-
Notifications
You must be signed in to change notification settings - Fork 129
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
fix: Api controller addon upgrade v1.0.1 #331
Conversation
how has this been validated, are there any schema changes for the chart? |
there are no schema changes for the chart - they have introduced a new environment variable which can be provided via set values . Added DEFAULT_SERVICE_NETWORK environment variable that will automatically create a service network and its VPC association at the controller startup #502 apart from that the new version of the chart requires additional policies which are breaking changes Breaking Changes The required IAM policies have changed - tag:GetResources policy is now required for running the controller. Please update your existing service account with the new policy provided in examples/recommended-inline-policy.json. Creating a Gateway no longer creates a service network and its VPC association. Please refer to the documentation for more details. Existing service networks and their associations will not be deleted. |
the v1.0.2 is already out with some fixes |
This PR has been automatically marked as stale because it has been open 30 days |
Pull request closed due to inactivity. |
What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.
Motivation
More
pre-commit run -a
with this PRFor Moderators
Additional Notes