-
Notifications
You must be signed in to change notification settings - Fork 2.8k
chore: release chart for v0.18.0 #5633
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
chore: release chart for v0.18.0 #5633
Conversation
Welcome @elafarge! |
Hi @elafarge. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
/assign @stevehipwell |
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.
/ok-to-test |
@elafarge you should be good to rebase now. |
935b2c2
to
6332880
Compare
6332880
to
c4f02bc
Compare
Just did so :) EDIT: forgot the date update, just force-pushed 5 minutes later to fix that |
c4f02bc
to
0ea925d
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.
@elafarge could you please merge the ### Fixed
changelog sections (heading order is in the comment above un-released) and couple you please an an entry for your change to update the versions?
All taken care of @stevehipwell |
/lgtm |
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.
Thanks @elafarge.
/label tide/merge-method-squash
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stevehipwell 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 |
* chore: release chart for v0.18.0 * Add version change to the changelog and merge duplicated fixed entries
What does it do ?
Release a Helm chart for v0.18.0, solves: #5574
NOTE that it's my first time contributing to the external DNS project. I read the release process documentation and tried to stick with it as much as I could but may have missed something 😊
Motivation
Would love to upgrade external-dns on our infrastructure but since there are RBAC changes, a Helm chart upgrade will also be needed.
More