Skip to content

Conversation

andrewjkeith
Copy link

@andrewjkeith andrewjkeith commented Jul 30, 2025

add separate labels to console ingress

@andrewjkeith andrewjkeith requested a review from a team as a code owner July 30, 2025 16:54
@andrewjkeith andrewjkeith force-pushed the feat_label branch 2 times, most recently from 1c0a6c1 to 9a68fff Compare July 30, 2025 17:15
@stefangries
Copy link
Collaborator

Doesn't that have to be added to values.schema.json as well?

@andrewjkeith
Copy link
Author

Doesn't that have to be added to values.schema.json as well?

it looks like there is a bunch of stuff missing from the values.schema.json file, should i run helm-schema against the vaules file to add it all or just the stuff in my MR?

@stefangries
Copy link
Collaborator

Thank you, I wasn't aware of that. It would be nice to have all the missing stuff added. Maybe in a separate pull request.

@jseiser
Copy link
Contributor

jseiser commented Sep 4, 2025

@andrewjkeith Any update on getting the values updated on this one?

@andrewjkeith
Copy link
Author

@stefangries i went ahead at refactored the values.schema.json, turns out it had several errors in there. let me know if you want a separate pull request

@stefangries
Copy link
Collaborator

Yes, please do separate pull requests for the refactoring and for this issue.

@andrewjkeith
Copy link
Author

@stefangries ok i just opened a new PR here, thanks: #858

@stefangries
Copy link
Collaborator

Thank you. If I'm not mistaken, the changes to values.schema.json are currently in both this pull request #841 and the other one #858. Could you please remove them from this pull request so that they are not in both?

Please excuse the late reply.

@andrewjkeith
Copy link
Author

@stefangries schema values from this PR have been removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants