-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update konflux documentation #294
Update konflux documentation #294
Conversation
362d437
to
afb6c35
Compare
docs/ocp-release.md
Outdated
@@ -1,10 +1,10 @@ | |||
### Release the operator (for OCP) | |||
|
|||
1. Choose a snapshot from the list of snapshots and double check post-submits passed for it. | |||
1. Creat a new branch off of `main`. example `v1.0`, `v0.9` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the creation of the branch shouldn't stay here. If we want to release 1.0.z,we should only need to select the snapshot and run the release with it and referring the 1.0 application (and branch).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can make step 1 optional if we are doing that. Just wanted to document the branch name, but maybe that is to basic.
4498a44
to
801ff20
Compare
080e6f8
to
54e637b
Compare
/test integration |
/test unit |
@aleskandro: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
0fc08ca
to
14a9da2
Compare
14a9da2
to
9601aee
Compare
69bbd04
to
31bb872
Compare
31bb872
to
66a4a93
Compare
docs/ocp-release.md
Outdated
|
||
### Create a new FBC fragment for a new OCP release | ||
10. Create a new Release for the operator snapshot in the previous step. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we also need to update the RPA to allow the new 'tag' (e.g., v1.0.1) and create it in comet.
66a4a93
to
d7f1f30
Compare
@AnnaZivkovic: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Thanks @AnnaZivkovic feel free to |
(I would squash the two commits and rebase first) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aleskandro The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
No description provided.