Skip to content

[deckhouse-cli] Update the vendored plugin to v0.0.30 - #434

Merged
ldmonster merged 1 commit into
mainfrom
chore/update-internal-package-plugin-to-v0.0.30
Aug 4, 2026
Merged

[deckhouse-cli] Update the vendored plugin to v0.0.30#434
ldmonster merged 1 commit into
mainfrom
chore/update-internal-package-plugin-to-v0.0.30

Conversation

@Glitchy-Sheep

@Glitchy-Sheep Glitchy-Sheep commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

d8 package is compiled into d8, so it never updates on its own. This syncs the vendored code with upstream v0.0.30 and brings one new check to d8 package verify.

What changed

New openapi linter for the schemas a package ships under openapi/:

  • rule advanced: x-deckhouse-ui-advanced is allowed on top-level settings only
  • the UI reads the marker nowhere else, so a nested one is now an error
  • the message names the property, for example properties.tuning.properties.cpuLimit
  • runs in all scopes: static, remote.bundle, remote.release
  • no .pkglint.yaml settings: the rule is a schema contract and always reports as an error
  • d8 package doc lists it

Verification

  • 158 files of the vendored tree match upstream v0.0.30 (23f3072), byte for byte
  • two files differ on purpose: packagecmd.go (d8 only) and pkg/cmd/cmd.go (no version subcommand)
  • go.mod needs nothing new, task build is green, linter reports 0 issues
  • ran the rule on a bootstrapped app package: a top-level marker passes, a nested one fails with the path

Notes

task test and task lint:check fail locally on the gost build tag, the same way they fail on clean main. #433 drops that tag, so this branch will be rebased on it.

- New `openapi` linter checks the schemas a package ships under `openapi/`.
- Its `advanced` rule allows `x-deckhouse-ui-advanced` on top-level settings only. The UI reads the marker nowhere else, so a nested one is an error and points at the offending property.
- The linter runs in every scope: `static`, `remote.bundle` and `remote.release`.
- It has no `.pkglint.yaml` settings. The rule is a schema contract, so it always reports as an error.
- `d8 package doc` lists the new linter and rule.

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@Glitchy-Sheep Glitchy-Sheep changed the title [package] Update the vendored plugin to v0.0.30 [deckhouse-cli] Update the vendored plugin to v0.0.30 Aug 4, 2026
@Glitchy-Sheep Glitchy-Sheep self-assigned this Aug 4, 2026
@Glitchy-Sheep Glitchy-Sheep added the enhancement New feature or request label Aug 4, 2026
@ldmonster
ldmonster merged commit f90757b into main Aug 4, 2026
9 checks passed
@ldmonster
ldmonster deleted the chore/update-internal-package-plugin-to-v0.0.30 branch August 4, 2026 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants