Skip to content

fix: match archibot httpGet.port ignore keys to jsonschema 4.19 json_path - #84

Merged
vl-kp merged 2 commits into
mainfrom
fix/archibot-httpget-port-ignore-json-path
Jul 10, 2026
Merged

fix: match archibot httpGet.port ignore keys to jsonschema 4.19 json_path#84
vl-kp merged 2 commits into
mainfrom
fix/archibot-httpget-port-ignore-json-path

Conversation

@vl-kp

@vl-kp vl-kp commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix IGNORED_VALIDATION_ERRORS keys for archibot / archibot-db-gateway httpGet.port to use dot notation (services.archibot-db-gateway), matching error.json_path from jsonschema==4.19.0 (the version pinned in the hook).
  • The v0.61.0 ignore used bracket notation (services['archibot-db-gateway']), so it never matched and CI still failed on values-dev-tech-support.yaml.

Follow-up to #83. Unblocks cloud-platform-gitops#413.

Test plan

  • Confirm with jsonschema==4.19.0 that error.json_path is $.platform-managed-chart.services.archibot-db-gateway.livenessProbe.httpGet (and readiness equivalent).
  • After release/tag + bump rev in cloud-platform-gitops .pre-commit-config.yaml, re-run PR 413 pre-commit and confirm tech-support instance dev passes with httpGet.port: 8080.

Made with Cursor

vl-kp added 2 commits July 10, 2026 13:22
…path

CI pins jsonschema==4.19.0, which emits hyphenated service keys with dot
notation (services.archibot-db-gateway), not brackets. The v0.61.0 ignore
never matched, so values-dev-tech-support still failed validation.
…path

CI pins jsonschema==4.19.0, which emits hyphenated service keys with dot
notation (services.archibot-db-gateway), not brackets. The v0.61.0 ignore
never matched, so values-dev-tech-support still failed validation.
@vl-kp
vl-kp requested a review from a team as a code owner July 10, 2026 05:22
@vl-kp
vl-kp merged commit f0f751b into main Jul 10, 2026
2 checks passed
@vl-kp
vl-kp deleted the fix/archibot-httpget-port-ignore-json-path branch July 10, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants