Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

operator API: replace embedded certificates with references to secrets #759

Closed
pohly opened this issue Sep 30, 2020 · 2 comments
Closed

operator API: replace embedded certificates with references to secrets #759

pohly opened this issue Sep 30, 2020 · 2 comments
Assignees
@avalluri
Labels
0.9

Comments

@pohly
Copy link
Contributor

pohly commented Sep 30, 2020

Having to copy custom certificates into the CR object implies that automatic certificate updates must be done through the PMEM-CSI API and thus will depend on custom scripting.

It may be better to reference secrets by name because then existing tooling for keeping secrets up-to-date can be used.
@pohly pohly added the 0.9 label Sep 30, 2020
@pohly
Copy link
Contributor Author

pohly commented Sep 30, 2020

Note that certificates are needed for controller/node communication. If we eliminate that part (see discussion in issue #733), then we don't need any certificates anymore.

@pohly pohly mentioned this issue Sep 30, 2020
Closed
5 tasks
@avalluri avalluri self-assigned this Oct 26, 2020
@pohly
Copy link
Contributor Author

pohly commented Jan 20, 2021

Became obsolete after PR #838 - now we have issue #857 😭

@pohly pohly closed this as completed Jan 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@avalluri avalluri

Labels
0.9
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pohly @avalluri