Skip to content

Test refresh of internal CA doesn't break the bundle#671

Open
evgeni wants to merge 1 commit into
masterfrom
ca-doesn-break-proxy
Open

Test refresh of internal CA doesn't break the bundle#671
evgeni wants to merge 1 commit into
masterfrom
ca-doesn-break-proxy

Conversation

@evgeni

@evgeni evgeni commented Jul 17, 2026

Copy link
Copy Markdown
Member

Why are you introducing these changes? (Problem description, related links)

What are the changes introduced in this pull request?

How to test this pull request

Steps to reproduce:

Checklist

  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)

- name: Generate certificates bundle on quadlet
run: |
./foremanctl certificate-bundle proxy.example.com \
--certificate-source=${{ matrix.certificate_source }} \

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This was what was making the tests pass before. Now we explicitly store only the source in L434.

Comment thread .github/workflows/test.yml Outdated
@evgeni
evgeni force-pushed the ca-doesn-break-proxy branch from 94e634d to d3ca9c0 Compare July 17, 2026 10:42
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.

1 participant