netgroup-polito / CrownLabs

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

Normalize Instance status phase #755

Closed QcFe closed 2 years ago

QcFe commented 2 years ago

Description

This PR removes the old VMI text from Instance status.

kingmakerbot commented 2 years ago

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

kingmakerbot commented 2 years ago

The generated files in the operators/ folder appear to be out-of-date.

Please, ensure you are using the correct version of controller-gen and re-run:

make generate
make manifests
Here it is an excerpt of the diff: ```diff diff --git a/operators/deploy/crds/crownlabs.polito.it_instances.yaml b/operators/deploy/crds/crownlabs.polito.it_instances.yaml index 33bddd7..7e299f3 100644 --- a/operators/deploy/crds/crownlabs.polito.it_instances.yaml +++ b/operators/deploy/crds/crownlabs.polito.it_instances.yaml @@ -181,7 +181,7 @@ spec: - Running - Ready - Stopping - - Off + - "Off" - Failed - CreationLoopBackoff type: string ```
giorio94 commented 2 years ago

/merge