Add PVC Provisioning Job

[AleTopp]

Description

Added Job that changes the owner of the MyDrive from root (default) to user 1010 (CrownLabsUserID), so that it is writable by tenants inside Instances deployed as containers (that run as user CrownLabsUserID). If the Job completes, it adds a label to the PVC pvc-provisioning: completed to flag it.

Fixes issue: NFS MyDrive cannot be written from containers since owner is root

~ Alessio Giliberti, Chiara Mercurio

[kingmakerbot]

Hi @AleTopp. Thanks for your PR.

I am @kingmakerbot.
You can interact with me issuing a slash command in the first line of a comment.
Currently, I understand the following commands:

• /rebase: Rebase this PR onto the master branch
• /merge: Merge this PR into the master branch
• /hold: Adds hold label to prevent merging with /merge
• /unhold: Removes the hold label to allow merging with /merge
• /deploy-staging: Deploy a staging environment to test this PR (the build-all flag enables user environments building)
• /undeploy-staging: Manually undeploy the staging environment

Make sure this PR appears in the CrownLabs changelog, adding one of the following labels:

• kind/breaking: 💥 Breaking Change
• kind/feature: 🚀 New Feature
• kind/bug: 🐛 Bug Fix
• kind/cleanup: 🧹 Code Refactoring
• kind/docs: 📝 Documentation

[QcFe]

Managed to have a look. Overall logic now seems fine. Minor changes:

• switch from CreateOrUpdate to r.Update for pvc labels, since for sure you already have the pvc there.
• change 'pvc-provisioning' to 'mydrive-provisioning'. Also I'd likely add a new 'pending' value that can be used to retrieve all tenants that for any reason are stuck on provisioning.
• as you can see here there are issues with linting. you can go to your go settings in vscode and select golangci-lint as the linter, to get in sync with the linter we're using in the pipeline
• we'll still deploy now and, if it works, we can still already test the logic

[QcFe]

/deploy-staging

[kingmakerbot]

Your staging environment has been correctly deployed/updated!
Available here: Frontend, Qlkube
Operators: add the crownlabs.polito.it/operator-selector=staging-916 label to your tenant.

[QcFe]

almost there :)

[QcFe]

/deploy-staging

[kingmakerbot]

Your staging environment has been correctly deployed/updated!
Available here: Frontend, Qlkube
Operators: add the crownlabs.polito.it/operator-selector=staging-916 label to your tenant.

[AleTopp]

/deploy-staging

[QcFe]

/deploy-staging

[kingmakerbot]

Your staging environment has been correctly deployed/updated!
Available here: Frontend, Qlkube
Operators: add the crownlabs.polito.it/operator-selector=staging-916 label to your tenant.