Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[core] UAA cert injection broken #378

Closed
paulczar opened this issue Oct 1, 2019 · 0 comments · Fixed by #382
Closed

[core] UAA cert injection broken #378

paulczar opened this issue Oct 1, 2019 · 0 comments · Fixed by #382
Assignees
@ywk253100
Labels
target/1.2.1

Comments

@paulczar
Copy link
Contributor

paulczar commented Oct 1, 2019

The change #348 broke the ability to provide an external CA Cert for UAA (and other auth) as both secrets are trying to put files in /etc/core/ca, however in #348 the change was made to load all secrets in harbor-secret into the /etc/core/ca directory, kubernetes can't merge directory based secrets with file based secrets.
@ywk253100 ywk253100 self-assigned this Oct 9, 2019
ywk253100 added a commit to ywk253100/harbor-helm that referenced this issue Oct 9, 2019
@ywk253100
Mount the certificate of UAA in a seperated directory
90d742c 
Fixes goharbor#378, mount the certificate of UAA in a seperated directory

Signed-off-by: Wenkai Yin <yinw@vmware.com>
@ywk253100 ywk253100 mentioned this issue Oct 9, 2019
Merged
ywk253100 added a commit to ywk253100/harbor-helm that referenced this issue Oct 9, 2019
@ywk253100
Mount the certificate of UAA in a seperated directory
df7d8cd 
Fixes goharbor#378, mount the certificate of UAA in a seperated directory

Signed-off-by: Wenkai Yin <yinw@vmware.com>
@ywk253100 ywk253100 mentioned this issue Oct 9, 2019
Merged
ywk253100 added a commit to ywk253100/harbor-helm that referenced this issue Oct 10, 2019
@ywk253100
Mount the certificate of UAA in a seperated directory
6851082 
Fixes goharbor#378, mount the certificate of UAA in a seperated directory

Signed-off-by: Wenkai Yin <yinw@vmware.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@ywk253100 ywk253100

Labels
target/1.2.1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mount the certificate of UAA in a seperated directory ywk253100/harbor-helm
2 participants
@paulczar @ywk253100