Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: Release process re-work #1947

Merged

Conversation

stevenhorsman
Copy link
Member

  • Update to reflect that we need to wait for the operator payload change before starting the peer-pods release

- Update to reflect that we need to wait for the operator
payload change before starting the peer-pods release

Signed-off-by: stevenhorsman <steven@uk.ibm.com>
@@ -30,11 +30,11 @@ the kata-containers release version.
go mod tidy
```
in the [cloud-api-adaptor](../src/cloud-api-adaptor/) directory and [csi-wrapper](../src/csi-wrapper/) directory.
1. The CoCo operator updates to use references to the other component releases and then releases itself
1. cloud-api-adaptor releases with the following phases detailed below:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: s/1/2

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ignore it.. In preview the sequence is proper.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I'm lazy/smart? and just label them all as 1. so that when we insert new items, or re-juggle the order, it's less work to update them

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The is super smart :-) .. Learnt a new way..

Copy link
Member

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Copy link
Member

@wainersm wainersm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @stevenhorsman !

@wainersm wainersm merged commit 9365f66 into confidential-containers:main Jul 25, 2024
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants