-
Notifications
You must be signed in to change notification settings - Fork 36
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
[OSPRH-12378] Set X-Forwarded-Proto only if empty #584
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jlarriba The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This looks good, but I'm wondering if we need something similar for aodh
|
That conf file is currently lacking the line we are replacing:
I wonder if this was a bug or intended. |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/b30c45477d4b47a997af0a49f243b39d ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 25m 29s |
42ae0a1
to
c03f760
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2504793ae7b8467ea573663c969bb67c ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 10m 03s |
recheck It is an issue deploying the dataplane deployment: |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0ef73603c09c4cd8b5e248ebed29a545 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 04m 49s |
No description provided.