-
Notifications
You must be signed in to change notification settings - Fork 37
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
Publish custom rancher chart with operator #258
Publish custom rancher chart with operator #258
Conversation
Just nitpicking a bit, sorry. LGTM, though. I understand this is intended to publish updates to current Rancher version (in this case 2.8) and back-ports to other currently maintained versions (as we've been doing lately) will be done "manually". Is this correct? |
Looks like this needs a manual rebase. |
8048297
to
0379f57
Compare
Sorry, I know I'm being very picky but, can we update the typos in the name of the operator ( |
0379f57
to
c733e71
Compare
default: "dev-v2.7" | ||
prev_eks_operator: | ||
default: "dev-v2.8" | ||
prev_aks_operator: |
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.
prev_aks_operator: | |
prev_eks_operator: |
description: "Previous EKS operator version (e.g. 1.1.0-rc2)" | ||
required: true | ||
default: "" | ||
new_eks_operator: | ||
new_aks_operator: |
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.
new_aks_operator: | |
new_eks_operator: |
prev_operator_version: ${{ github.event.inputs.prev_aks_operator }} | ||
new_operator_version: ${{ github.event.inputs.new_aks_operator }} |
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.
prev_operator_version: ${{ github.event.inputs.prev_aks_operator }} | |
new_operator_version: ${{ github.event.inputs.new_aks_operator }} | |
prev_operator_version: ${{ github.event.inputs.prev_eks_operator }} | |
new_operator_version: ${{ github.event.inputs.new_eks_operator }} |
new_eks: | ||
description: "New EKS operator version (e.g. 1.1.0-rc2), don't include the 'v'" | ||
description: "New AKS operator version (e.g. 1.1.0-rc2), don't include the 'v'" |
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.
description: "New AKS operator version (e.g. 1.1.0-rc2), don't include the 'v'" | |
description: "New EKS operator version (e.g. 1.1.0-rc2), don't include the 'v'" |
Publish custom rancher chart with operator. Currently, there is no way to test changes in operator CRDs before tagging a release and merging an update PR in rancher/rancher. This PR adds a step to the nightly workflow to update rancher dependencies and build a chart based on that.
Fixes: rancher/highlander#20
Fixes: rancher/highlander#18
Fixes: rancher/highlander#21