Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test hard coding category in VS #10325

Closed
wants to merge 6 commits into from
Closed

Test hard coding category in VS #10325

wants to merge 6 commits into from

Conversation

arianaw66
Copy link
Contributor

@arianaw66 arianaw66 commented Nov 12, 2024

Description

Add category to Gloo Gateway enterprise CRDs so that the convenient kubectl get solo-io can be used to see all GG enterprise CRs on the cluster.

API changes

Added category to enterprise CRDs

Interesting decisions

This PR follows the example of solo-io#5592, which shows that solo-kit only generates the schema section of the CRD (see solo-io#5592 (comment)). Therefore, we hardcode the new category in the CRD yaml files.

Testing steps

Notes for reviewers

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@github-actions github-actions bot added keep pr updated signals bulldozer to keep pr up to date with base branch work in progress signals bulldozer to keep pr open (don't auto-merge) labels Nov 12, 2024
@arianaw66 arianaw66 removed the keep pr updated signals bulldozer to keep pr up to date with base branch label Nov 12, 2024
@arianaw66
Copy link
Contributor Author

Closing in favor of solo-io#5006

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work in progress signals bulldozer to keep pr open (don't auto-merge)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant