Skip to content
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.

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

Add: Reset custom SSL certificate to default self-signed #2886

Conversation

mjivraja
Copy link
Contributor

@mjivraja mjivraja commented Mar 14, 2024

Documented the procedure for end-users to revert from Custom SSL to default self-signed certificate for Project, SmartProxy, and Hosts guide. This was required in the case where Custom SSL certificates were expired for Project and CA was not available to provide new signed certificates that caused business outages or production delays. Also, I fixed minor things in Deploying custom SSL certificates to hosts

https://bugzilla.redhat.com/show_bug.cgi?id=1874045

Please cherry-pick my commits into:

  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (planned Satellite 6.15)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6/6.7)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only; orcharhino 6.4/6.5 on EL8 only)
  • We do not accept PRs for Foreman older than 3.3.

@mjivraja mjivraja added the Needs tech review Requires a review from the technical perspective label Mar 14, 2024
@mjivraja mjivraja force-pushed the Revert-from-custom-SSL-to-default-master branch 4 times, most recently from 623a99f to 3fb466d Compare March 15, 2024 06:48
Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

Left some comments, I did not test the procedure myself yet.

@mjivraja mjivraja force-pushed the Revert-from-custom-SSL-to-default-master branch 2 times, most recently from bab196a to 85f53e9 Compare March 21, 2024 10:43
@mjivraja mjivraja requested a review from ehelms April 10, 2024 14:55
@mjivraja mjivraja force-pushed the Revert-from-custom-SSL-to-default-master branch from 85f53e9 to c11fdce Compare April 24, 2024 12:52
@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author Needs re-review and removed Waiting on contributor Requires an action from the author labels Apr 25, 2024
mjivraja and others added 6 commits April 30, 2024 11:48
Documented the procedure for end-users to revert from Custom SSL to
default self-signed certificate for Project, SmartProxy, and Hosts
guide. This was required in the case where Custom SSL certificate were
expired for Project and CA was not available to provide new signed
certificates that caused business outages or production delays. Also,
fixed minor things in Deploying custom SSL certificate to hosts

https://bugzilla.redhat.com/show_bug.cgi?id=1874045

Rerun GitHub Actions
Co-authored-by: Maximilian Kolb <mail@maximilian-kolb.de>
@mjivraja mjivraja force-pushed the Revert-from-custom-SSL-to-default-master branch from a4d8536 to 1a9a2ed Compare April 30, 2024 06:31
Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

diff LGTM.

@maximiliankolb maximiliankolb merged commit bb8131c into theforeman:master May 16, 2024
8 checks passed
maximiliankolb pushed a commit that referenced this pull request May 16, 2024
@maximiliankolb
Copy link
Contributor

Merged to "master" and cherry-picked to "3.10":
7bc995d..d442759 3.10 -> 3.10

@mjivraja mjivraja deleted the Revert-from-custom-SSL-to-default-master branch May 16, 2024 13:34
mjivraja added a commit to mjivraja/foreman-documentation that referenced this pull request May 16, 2024
mjivraja added a commit to mjivraja/foreman-documentation that referenced this pull request May 16, 2024
mjivraja added a commit to mjivraja/foreman-documentation that referenced this pull request May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs tech review Requires a review from the technical perspective
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants