Skip to content

DOC: Custom CA certificates on K8s - #5017

Open
mbussolotto wants to merge 1 commit into
uyuni-project:masterfrom
mbussolotto:custom-cas
Open

DOC: Custom CA certificates on K8s#5017
mbussolotto wants to merge 1 commit into
uyuni-project:masterfrom
mbussolotto:custom-cas

Conversation

@mbussolotto

Copy link
Copy Markdown
Member

Description

Custom CA certificates on K8s

Target branches

  • master

Links

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds documentation for using an optional custom-cas ConfigMap to inject additional CA certificates into the Uyuni server pod trust store on Kubernetes, and updates the storage/PVC list accordingly.

Changes:

  • Document how to provide additional CA certs via an optional custom-cas ConfigMap and when a deployment restart is required.
  • Remove the ca-certs PVC entry from the list of chart-created PVCs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 6 comments.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@mbussolotto
mbussolotto marked this pull request as ready for review June 24, 2026 08:26

@cbosdo cbosdo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I  wonder how we can handle these new certs we drop in that folder

----
kubectl apply -n $NAMESPACE -f ca-certs-rescue.yaml
kubectl wait --for=condition=Ready pod/ca-certs-rescue -n $NAMESPACE --timeout=300s
kubectl cp $NAMESPACE/ca-certs-rescue:/data ./old-ca-certs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you cat them, there is no need to copy them around

containers:
- name: rescue
image: busybox:stable
command: ["sleep", "3600"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to sleed, even though it's hot these days… just call cat them all out and get them in the pod logs or stdout.

@mbussolotto
mbussolotto marked this pull request as draft June 24, 2026 15:07
@mbussolotto mbussolotto changed the title Custom CA certificates on K8s DOC: Custom CA certificates on K8s Jul 3, 2026
@mbussolotto
mbussolotto removed the request for review from aaannz July 3, 2026 13:35
@mbussolotto
mbussolotto force-pushed the custom-cas branch 3 times, most recently from b2ea652 to c03487b Compare July 13, 2026 12:32
@mbussolotto
mbussolotto marked this pull request as ready for review July 13, 2026 13:47
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants