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

[#185765129] Add tls to cdn broker #3548

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

malcgds
Copy link
Contributor

@malcgds malcgds commented Dec 20, 2023

What

Added cdn broker that has been updated to enable tls between the load balancer and the broker.
Removed artefacts related to redundant Let's Encrypt usage.

How to review

  1. Run the branch into a dev env.
  2. After the updated broker has been installed check the state of the vm and cdn-broker logs using bosh.

🚨⚠️ Please do not merge this pull request via the GitHub UI ⚠️🚨

@malcgds malcgds force-pushed the 185765129-add-tls-to-cdn-broker branch from ad78893 to a19723a Compare December 21, 2023 12:24
@dark5un dark5un force-pushed the 185765129-add-tls-to-cdn-broker branch from a19723a to 2c872bb Compare January 16, 2024 07:43
@dark5un dark5un mentioned this pull request Jan 19, 2024
@kr8n3r
Copy link
Contributor

kr8n3r commented Jan 31, 2024

is this still relevant?

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