Skip to content

ncm-spma: support for ssl client settings for yumng and dnf backend.#1852

Merged
jrha merged 2 commits intoquattor:mainfrom
aka7:ssl_cert_support
Mar 13, 2026
Merged

ncm-spma: support for ssl client settings for yumng and dnf backend.#1852
jrha merged 2 commits intoquattor:mainfrom
aka7:ssl_cert_support

Conversation

@aka7
Copy link
Copy Markdown
Contributor

@aka7 aka7 commented Feb 4, 2026

Add the code for ssl client settings for yumng and dnf backend.
schema exists but no code to add it.

  • Why the change is necessary.
    Although the schema exists for protocols settings to add verify cacert clientkey clientcert, etc, but adding it has no effect on the config generation for that repo. at least on two backends we use, yumng and dnf backend.

  • What backwards incompatibility it may introduce.
    None.

Add the code for ssl client settings for yumng and dnf backend.
@aka7 aka7 force-pushed the ssl_cert_support branch from c66a18e to dbe0ac9 Compare February 4, 2026 12:09
@aka7 aka7 requested review from jrha, ned21 and stdweird February 4, 2026 12:10
@ned21
Copy link
Copy Markdown
Contributor

ned21 commented Feb 4, 2026

Would it be possible to have a unit test addition for these settings?

@jrha jrha added this to the 26.2 milestone Feb 11, 2026
@ned21
Copy link
Copy Markdown
Contributor

ned21 commented Feb 13, 2026

LGTM but will let someone else approve.

adding missing tests for dnf backend. plus yumng ssl config tests
@aka7 aka7 force-pushed the ssl_cert_support branch from d8eec26 to a09177c Compare March 5, 2026 13:26
@jrha jrha merged commit e567c22 into quattor:main Mar 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants