-
Notifications
You must be signed in to change notification settings - Fork 470
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
Add Gateway RBAC and tests #536
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly looks good, looks like we're just missing adding the --feature-gates=Gateway=true
to the test setup so that the helm test can complete.
72cbac1
to
b5cd454
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once CI is happy 👍
9ef1798
to
3ec1a8e
Compare
Move tests from non-main workflow into main PR workflow and remove non-main workflow. Remove 1.x -> 2.x transitional tests.
Always run chart-testing even if no changes are detected. An unknown issue is preventing ct list-changed from detecting changes.
What this PR does / why we need it:
Adds Gateway RBAC and tests.
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Notes for reviewers
CI was a combination of misconfigured and broken after the removal of next. Integration tests have moved into the main PR workflow. chart-testing is being very annoying and reporting that there are no changes even though running it locally says there are. I don't much care to dig into this ATM, since for our purposes we only have the single chart, almost all changes update it, and simply running tests on everything always is generally fine (the change detection stuff was more relevant on the old helm/charts repo, which had ALL the charts and only changed one in any given PR).
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
main
branch.