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

Switch out the envoy reverse proxy for caddy #905

Merged
merged 8 commits into from
Jan 17, 2024

Conversation

SteveHNH
Copy link
Contributor

@SteveHNH SteveHNH commented Dec 6, 2023

Envoy dropped support for the only version that will run on RHEL8 and no longer does security patches. This impacts secure environments since we can't use later images for envoy. Instead, we can swap out for caddy which effectively does the same thing

RHCLOUD-29123

@SteveHNH SteveHNH requested a review from psav December 6, 2023 14:47
@SteveHNH
Copy link
Contributor Author

SteveHNH commented Dec 6, 2023

/retest

@SteveHNH
Copy link
Contributor Author

SteveHNH commented Dec 6, 2023

looks like this failing test is due to a wget to feature flags getting a 404. Not sure the solution to that just yet

@SteveHNH
Copy link
Contributor Author

SteveHNH commented Dec 7, 2023

/retest

Copy link
Collaborator

@psav psav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy to make these changes if you're too busy on return - just let me know

Signed-off-by: Stephen Adams <stephen.adams@redhat.com>
Due to envoy no longer supporting RHEL8, we had to make a decision to
switch to a different reverse proxy for TLS termination inside a pod.
  Caddy seemed like the obvious choice given that we already use it for
  other purposes.

Signed-off-by: Stephen Adams <stephen.adams@redhat.com>
Signed-off-by: Stephen Adams <stephen.adams@redhat.com>
@SteveHNH
Copy link
Contributor Author

SteveHNH commented Jan 8, 2024

/retest

1 similar comment
@SteveHNH
Copy link
Contributor Author

SteveHNH commented Jan 9, 2024

/retest

Signed-off-by: Stephen Adams <stephen.adams@redhat.com>
Signed-off-by: Stephen Adams <stephen.adams@redhat.com>
@SteveHNH SteveHNH force-pushed the envoy_to_caddy branch 4 times, most recently from ff0b215 to 3161c67 Compare January 11, 2024 21:21
Signed-off-by: Stephen Adams <stephen.adams@redhat.com>
@SteveHNH SteveHNH requested a review from psav January 15, 2024 15:29
@psav psav merged commit ef4b69e into RedHatInsights:master Jan 17, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants