Repository navigation
Set the chart version for the v0.2.0 release - #17
Open
Timofei Larkin (lllamnyp) wants to merge 1 commit into
Open
Timofei Larkin (lllamnyp) wants to merge 1 commit into
Timofei Larkin (lllamnyp) wants to merge 1 commit into
Conversation
Cozystack vendors chart/address-controller/ from the tagged commit, so Chart.yaml's version and appVersion, and the Makefile's CHART_VERSION, must match the v0.2.0 tag. The operating guide's schema table moves its unreleased row to v0.2.0. Assisted-By: LLM Signed-off-by: Timofei Larkin <lllamnyp@gmail.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sets the chart's
version/appVersionand the Makefile'sCHART_VERSIONto 0.2.0, ahead of taggingv0.2.0. Cozystack copieschart/address-controller/from the tagged commit, so the chart version has to match the tag. v0.1.0 was prepared the same way (#4). The schema table indocs/operating.mdmoves its "unreleased" row to v0.2.0.After merge, tagging
v0.2.0on main makes the release workflow publishghcr.io/cozystack/address-controller:0.2.0and:0.2.Changes since v0.1.0
local.sdn.cozystack.io/ip-address-classor the class markedis-auto-claim-class.local.sdn.cozystack.io/auto-claim: "false".reclaimPolicy.Lostaddress is not replaced, reclaim leavesConflict/Lostas they are, and pre-bindings are checked for class and family.releaseDelay(Return Released addresses to the pool after a class's releaseDelay #11): a class'sReleasedaddresses return toAvailableafter the delay, unless annotatedheld.IPAddressgains a core-setBoundcondition.ServiceMonitorandPrometheusRule.IPAddressClaim.spec.addressName.docs/operating.md.Upgrade notes
make install-crds; Helm does not upgradecrds/. The schema changes are additive:IPAddressClass.spec.releaseDelay, and theBoundcondition onIPAddress.get/list/watch/patch,createon claims, andupdateon class finalizers. The chart's regenerated role carries them.is-auto-claim-classor a Service names a class. Before marking one, deploy a metallb-iad build with the placeholder opt-out (Opt placeholders out of the core's auto-claim metallb-iad#5).Supersedes #6.