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 component to manage port 1022 for do-release-upgrade #571

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Conversation

cPholloway
Copy link
Contributor

Case RE-943: When performing an elevation from u20->u22, we rely on the do-release-upgrade script to perform the OS upgrade. do-release-upgrade will create a secondary SSH connection on port 1022 as a fallback mechanism in the event that the primary SSH connection is disrupted during the upgrade process. This change ensures that port 1022 is open before executing do-release-upgrade so that the fallback method will work if it is required during the upgrade. It also ensures that port 1022 is closed once the upgrade is complete if it was closed before the upgrade started.

Changelog: Add component to manage port 1022 for ELevations that rely on
the do-release-upgrade script to perform the OS upgrade

By submitting pull requests to this repo, I agree to the Contributor License Agreement which can be found at: https://github.com/cpanel/elevate/blob/main/docs/cPanel-CLA.pdf

@troglodyne
Copy link
Contributor

While I'm sure someone out there rips out UFW on principle, the grand majority will not. Looks like it ought to work.

troglodyne
troglodyne previously approved these changes Dec 6, 2024
toddr
toddr previously approved these changes Dec 6, 2024
Case RE-943: When performing an elevation from u20->u22, we rely on the
do-release-upgrade script to perform the OS upgrade.  do-release-upgrade
will create a secondary SSH connection on port 1022 as a fallback
mechanism in the event that the primary SSH connection is disrupted
during the upgrade process. This change ensures that port 1022 is open
before executing do-release-upgrade so that the fallback method will
work if it is required during the upgrade.  It also ensures that port
1022 is closed once the upgrade is complete if it was closed before the
upgrade started.

Changelog: Add component to manage port 1022 for ELevations that rely on
 the do-release-upgrade script to perform the OS upgrade
@cPholloway cPholloway dismissed stale reviews from toddr and troglodyne via b6003b6 December 10, 2024 16:27
@toddr toddr merged commit e3f5aa5 into main Dec 10, 2024
17 checks passed
@toddr toddr deleted the RE-943 branch December 10, 2024 17:26
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