netgroup-polito / CrownLabs

Kubernetes-based Remote Laboratories
https://crownlabs.polito.it
Apache License 2.0
107 stars 42 forks source link

Instance controller: remove generation changed predicate #718

Closed giorio94 closed 2 years ago

giorio94 commented 2 years ago

Description

This PR removes the generation changed predicate from the instance controller, since it is the likely culprit for the status glitch observed during the exam simulation, as it might prevent the status from being correctly updated in case of race conditions.

Fixes # (issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

kingmakerbot commented 2 years ago

Hi @giorio94. 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:

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

giorio94 commented 2 years ago

/rebase

QcFe commented 2 years ago

/deploy-staging

kingmakerbot commented 2 years ago

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

QcFe commented 2 years ago

/merge

kingmakerbot commented 2 years ago

Your staging environment has been correctly teared-down!