Skip to content

Comments

console: add/update values.schema.json#537

Merged
chrisseto merged 1 commit intomainfrom
chris/p/console-schema
Mar 19, 2025
Merged

console: add/update values.schema.json#537
chrisseto merged 1 commit intomainfrom
chris/p/console-schema

Conversation

@chrisseto
Copy link
Contributor

This commit updates gen to produce JSON schemas for console. In order to support this without conflicts, the console chart has been bumped to v3 which allows it to co-exist with the redpanda chart with currently uses an unversioned import of console for subcharting.

@RafalKorepta
Copy link
Contributor

The following error:

Error:      	Received unexpected error:
--
  | values don't meet the specifications of the schema(s) in the following chart(s):
  | console:
  | - service.targetPort: Invalid type. Expected: integer, given: null
  | - ingress.className: Invalid type. Expected: string, given: null

and

Error Trace:	/work/charts/console/chart_test.go:89
--
  | Error:      	Received unexpected error:
  | values don't meet the specifications of the schema(s) in the following chart(s):
  | console:
  | - podSecurityContext.fsGroupChangePolicy: podSecurityContext.fsGroupChangePolicy must be one of the following: "OnRootMismatch", "Always"
  | Test:       	TestTemplate/case-021

was solved by 36ee3d0

@chrisseto chrisseto force-pushed the chris/p/console-schema branch from 6bbba22 to e7a1d1c Compare March 18, 2025 20:52
This commit updates `gen` to produce JSON schemas for `console`. In order to
support this without conflicts, the console chart has been bumped to `v3` which
allows it to co-exist with the redpanda chart with currently uses an
unversioned import of console for subcharting.
@chrisseto chrisseto force-pushed the chris/p/console-schema branch from e7a1d1c to a06f3a2 Compare March 18, 2025 21:16
@chrisseto
Copy link
Contributor Author

Fixed!

@chrisseto chrisseto merged commit 92c19f4 into main Mar 19, 2025
12 checks passed
@chrisseto chrisseto deleted the chris/p/console-schema branch March 19, 2025 17:17
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