netgroup-polito / CrownLabs

Kubernetes-based Remote Laboratories

https://crownlabs.polito.it

Apache License 2.0

107 stars 42 forks source link

Tenant Autoenrolling feature - api & operator #877

Closed giacoliva closed 10 months ago

giacoliva commented 1 year ago

Description

Workspaces can now set AutoEnroll features with two flags in the Spec (no auto enroll / enroll request and manager's approval / completely self enroll) Tenants can now autoenroll in Workspaces enabled (with user or candidate role depending on settings)

Tenants are able to edit themselves in order to add or remove Workspaces with autoEnroll enabled

How Has This Been Tested?

Environment: KinD with CRDs installed, Tenant Operator and Webhooks running. Keycloak not configured.

[ ] through kubectl api - Tenant edit as root user
[ ] through kubectl api - Tenant edit as tenant user role (kubectl --as <tenantname>)
[ ] through kubectl api - Workspace edit as root user
[ ] through graphql - Tenant & Workspace edit as default user

kingmakerbot commented 1 year ago

Hi @giacoliva. 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: :boom: Breaking Change
kind/feature: :rocket: New Feature
kind/bug: :bug: Bug Fix
kind/cleanup: :broom: Code Refactoring
kind/docs: :memo: Documentation

QcFe commented 11 months ago

/deploy-staging

kingmakerbot commented 11 months ago

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

QcFe commented 10 months ago

/deploy-staging

kingmakerbot commented 10 months ago

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

QcFe commented 10 months ago

/merge

QcFe commented 10 months ago

/merge

kingmakerbot commented 10 months ago

Your staging environment has been correctly teared-down!