-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
contrib: update release guide with helm/ansible operator and website info #3383
contrib: update release guide with helm/ansible operator and website info #3383
Conversation
@@ -306,26 +330,17 @@ To github.com:operator-framework/operator-sdk.git | |||
|
|||
Now that the branch exists, you need to make the post-release PR for the new release branch. To do this, simply follow the same steps as in [step 3](#3-create-a-pr-for-post-release-version-updates) with the addition of changing the branch name in the `go.mod` scaffold from `master` to the new branch (for example, `v1.3.x`). Then, make the PR against the new branch. | |||
|
|||
### 6. Updating the Homebrew formula |
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.
Should we not update the homebrew formula anymore?
The website-related steps are incorrect and need re-doing. Specifically, the release PR/tag need to include changes to /hold |
1cec65a
to
51e76e1
Compare
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.
Minor change requested, no need to re-review
```toml | ||
[[params.versions]] | ||
version = "v1.3" | ||
url = "https://v1-3-x.sdk.operatorframework.io" |
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.
As mentioned above, this link won't work (and will fail CI) unless Netlify is already building this branch. The configuration on the netlify side cannot be done completely ahead of time though, the github branch must be created and pushed first.
I think if we add a git push
here that should cover it.
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.
Done
/hold cancel |
51e76e1
to
613e5af
Compare
…and ansible/helm operator release binaries
613e5af
to
4af840c
Compare
Description of the change:
Motivation for the change: new release steps are necessary to document.
/cc @asmacdo @camilamacedo86 @varshaprasad96
/kind documentation
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs