-
Notifications
You must be signed in to change notification settings - Fork 73
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
Support istio egressGateways directly in umbrella #44
Comments
charden commented:
This change will impact all/most of the packages. We should consider creating an epic for it. |
charden commented:
A couple of thoughts... We'll need to go through each package and add DestinationRules to define TLS and VirtualServices to route traffic to the correct gateway for those known external services. Will TLS originate from the application or does the egress gateway need to perform TLS origination?
Do we want to setup networkpolicies to prevent bypassing the gateway? I assume this is all voluntary and can be disabled in the overrides. |
andrewshoell commented:
this might relate to https://repo1.dso.mil/groups/big-bang/-/epics/187 |
bb8-bot commented:
@benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
@benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
bb8-bot commented:
Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here Github Comment Mirrored Here @benjamin.hester.6 this issue has been inactive for 30 days and is being labelled as ~stale. If this issue is still required please take action by removing the ~stale label and commenting with an update, status, or justification. If this issue is not required please close it or label it as ~delete-me. If no action is taken this issue will be auto closed in 60 days. |
Bug
Description
specifying an egressGatway similar to
ingressGateways
leads to a schema validation error because the logic to support that in the umbrella chart is not present. Ensure we can supplyistio.egressGateways
via the umbrella chart and ensure it makes it through to the istio chart.The text was updated successfully, but these errors were encountered: