docs(platform-package): sync the publishing.* table with the platform values - #679
Open
Aleksei Sviridkin (lexfrei) wants to merge 1 commit into
Open
docs(platform-package): sync the publishing.* table with the platform values#679Aleksei Sviridkin (lexfrei) wants to merge 1 commit into
Aleksei Sviridkin (lexfrei) wants to merge 1 commit into
Conversation
… values The publishing.* reference on the next trunk documented five publishing.exposureClass.* keys the platform no longer has: the ExposureClass/ServiceExposure machinery was dropped for the native Service loadBalancerClass (cozystack/cozystack#3218), and packages/core/platform/values.yaml carries no exposureClass key. Remove those rows and rewrite the publishing.exposure note that still named exposureClass as the successor. Add the publishing.loadBalancerClass row that replaced them and never had one: it names the LoadBalancer controller for the host ingress Service, the address pool or announcer is provisioned out of band, and because spec.loadBalancerClass is immutable an existing Service has to be recreated once when the value changes. Add the publishing.serviceDomain row for the new display-only DNS suffix the dashboard shows under the external IP of LoadBalancer Services (cozystack/cozystack#4035). Assisted-by: LLM Signed-off-by: Aleksei Sviridkin <f@lex.la>
Aleksei Sviridkin (lexfrei)
requested review from
Andrei Kvapil (kvaps),
Timofei Larkin (lllamnyp),
myasnikovdaniil and
Timur Tukaev (tym83)
as code owners
September 2, 2026 21:12
✅ Deploy Preview for cozystack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
11 tasks
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
1 participant
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.
The
publishing.*table on the platform-package page documented fivepublishing.exposureClass.*keys that the platform no longer has (cozystack/cozystack#3218 dropped ExposureClass for the nativeloadBalancerClass) and had no row forpublishing.loadBalancerClass. This removes the five rows, addsloadBalancerClasswith the description from the platform values comment, and addspublishing.serviceDomain, the display-only suffix the dashboard Services tab appends to LoadBalancer Service names, which lands in cozystack/cozystack#4035. The paragraph under the table that namedpublishing.exposureClassas the successor ofpublishing.exposurenow points atexternalIPsandloadBalancerClassinstead.serviceDomainis not on cozystack main yet, so this should merge after cozystack/cozystack#4035; the other two changes stand on their own. The same five stale rows sit on the v1.6 page too, and that page is missingloadBalancerClassalthough v1.6.0 shipped with it; it is not touched here and needs its own backport.Rendered with the hugo preview environment: both new rows in place, no
exposureClassleft on the page. The i18n check is red on main already (stale zh-cn index), unrelated to this change.