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

Fix #2202 #2204

Closed
wants to merge 1 commit into from
Closed

Conversation

titilambert
Copy link
Contributor

@titilambert titilambert commented Jan 15, 2020

Description

This patch allows namespaced ambassador to get K8s secrets in an other namespace.

If you want to accept this PR, I will create an other for master

Related Issues

Testing

I deployed a namespaced ambassador, and I just added AMBASSADOR_CERTS_SINGLE_NAMESPACE env var with empty value.

        - name: AMBASSADOR_SINGLE_NAMESPACE
          value: "YES"
        - name: AMBASSADOR_CERTS_SINGLE_NAMESPACE
        - name: AMBASSADOR_NAMESPACE
          valueFrom:
            fieldRef:
              apiVersion: v1
              fieldPath: metadata.namespace

Then in my TLScontext I'm able the set a certificate from an other namespace. (the service account of ambassador has a rolebinding allowing reading secret in the other namespace)

Todos

  • Tests
  • Documentation

Other

@kflynn
Copy link
Member

kflynn commented Jan 27, 2020

Superseded by #2239, since this was targeted against release/0.86.1 and we won't be doing any more 0.86 releases. Thanks!!

@kflynn kflynn closed this Jan 27, 2020
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