Skip to content

Update certificate key usage to match documentation #17987

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Apr 2, 2025
Merged

Update certificate key usage to match documentation #17987

merged 2 commits into from
Apr 2, 2025

Conversation

ehelms
Copy link
Contributor

@ehelms ehelms commented Mar 20, 2025

Problem Statement

The custom certificates that are generated in testing should match documentation and that is being updated here: theforeman/foreman-documentation#3700 (comment)

Solution

Related Issues

@evgeni
Copy link
Member

evgeni commented Mar 21, 2025 

trigger: test-robottelo
pytest: -vv tests/foreman/sys/test_katello_certs_check.py

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10672
Build Status: SUCCESS
PRT Comment: pytest -vv tests/foreman/sys/test_katello_certs_check.py --external-logging
Test Result : ========== 5 passed, 1 deselected, 40 warnings in 2856.92s (0:47:36) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 21, 2025
@Gauravtalreja1
Copy link
Collaborator

is this change intended only for stream or also for 6.17.z? could you confirm and add appropriate labels for cherrypick?

@Gauravtalreja1 Gauravtalreja1 requested a review from a team March 26, 2025 09:27
@ehelms ehelms added Stream Introduced in or relating directly to Satellite Stream/Master CherryPick PR needs CherryPick to previous branches 6.17.z labels Apr 1, 2025
@ehelms
Copy link
Contributor Author

ehelms commented Apr 1, 2025

is this change intended only for stream or also for 6.17.z? could you confirm and add appropriate labels for cherrypick?

Done.

@jameerpathan111 jameerpathan111 added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Apr 2, 2025
@jameerpathan111 jameerpathan111 merged commit ff901df into SatelliteQE:master Apr 2, 2025
11 of 14 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 2, 2025
@ehelms @evgeni @web-flow
Update certificate key usage to match documentation (#17987)
55618e2 
Co-authored-by: Evgeni Golov <[email protected]>
(cherry picked from commit ff901df)
@Satellite-QE Satellite-QE mentioned this pull request Apr 2, 2025
Merged
@aneta-petrova aneta-petrova mentioned this pull request Apr 9, 2025
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@evgeni evgeni evgeni approved these changes

@ekohl ekohl ekohl approved these changes

@jameerpathan111 jameerpathan111 jameerpathan111 approved these changes

Assignees

@ehelms ehelms

Labels
6.17.z AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ehelms @evgeni @Satellite-QE @Gauravtalreja1 @ekohl @jameerpathan111