Skip to content

Add warning about limited ongoing technical support for RHEL 6 upgrades.#332

Merged
spetrosi merged 2 commits intoredhat-cop:mainfrom
PeterMocary:rhel6-not-supported-warning
Jan 30, 2026
Merged

Add warning about limited ongoing technical support for RHEL 6 upgrades.#332
spetrosi merged 2 commits intoredhat-cop:mainfrom
PeterMocary:rhel6-not-supported-warning

Conversation

@PeterMocary
Copy link
Collaborator

@PeterMocary PeterMocary commented Jan 8, 2026

Although this collection allows for upgrades of RHEL 6, the upgrades are not fully supported by Red Hat. For more information refer to the RHEL In-place upgrade Support Policy.

The warning is currently added to the upgrade and analysis roles. It is in the analysis-preupg.yml and redhat-upgrade-tool-upgrade.yml‎ to leverage the version check happening before these files are included. I wanted to make the warning part of the main.yml files while making it more generic for easier inclusion of additional versions later, however, the title of the task would be shown when the warn task is skipped, which is not desired.

Jira: RHELMISC-22117

@PeterMocary PeterMocary force-pushed the rhel6-not-supported-warning branch 2 times, most recently from 2751570 to 073670a Compare January 10, 2026 16:21
@PeterMocary PeterMocary changed the title add warning about RHEL 6 being upgraded using redhat-upgrade-tool Add warning about limited ongoing technical support for RHEL 6 upgrades. Jan 10, 2026
@PeterMocary
Copy link
Collaborator Author

  • Updated the warning formulation according to the RHEL In-place upgrade Support Policy while also referring to it.
  • Added the warning to the analysis role.
  • Added the changelog fragment.

Copy link
Collaborator

@spetrosi spetrosi left a comment

Choose a reason for hiding this comment

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

I suggest renaming roles/analysis/tasks/analysis-preupg.yml and roles/upgrade/tasks/redhat-upgrade-tool-upgrade.yml to finish with _el6.yml so that its clear that they are run on EL6 only. This PR seems like a good place to do it. Other than that, looks good to me.

@PeterMocary PeterMocary force-pushed the rhel6-not-supported-warning branch 2 times, most recently from 19a5d50 to 4d5151e Compare January 14, 2026 10:45
@PeterMocary
Copy link
Collaborator Author

  • Addressed the text related suggestions.
  • Renamed the rhel6 related files with -el6 suffix as suggested.
  • Rebased onto main.

@PeterMocary PeterMocary marked this pull request as ready for review January 14, 2026 11:08
@PeterMocary PeterMocary force-pushed the rhel6-not-supported-warning branch from 4d5151e to 0b4a488 Compare January 16, 2026 14:37
@PeterMocary PeterMocary requested a review from bocekm January 16, 2026 14:40
@PeterMocary PeterMocary force-pushed the rhel6-not-supported-warning branch from 0b4a488 to ea1eba4 Compare January 16, 2026 20:40
@PeterMocary
Copy link
Collaborator Author

Updated the recommendations for RHEL 7 as well. Thanks @bocekm for help!

@jeffmcutter
Copy link
Collaborator

In the readme, would not the same warning apply to analysis as upgrade? It's not leapp for 6 to 7 either.

@PeterMocary PeterMocary force-pushed the rhel6-not-supported-warning branch 2 times, most recently from ec29b96 to e9e0207 Compare January 19, 2026 10:56
@PeterMocary
Copy link
Collaborator Author

PeterMocary commented Jan 19, 2026

In the readme, would not the same warning apply to analysis as upgrade? It's not leapp for 6 to 7 either.

I've updated the readme a bit to make it more clear the RHEL 6 analysis uses the Preupgrade Assistant instead of leapp. I'm not sure, if that is what you meant though.

Some of the tasks files in analysis and upgrade roles are exclusive to
upgrades of RHEL 6. These files use different tools than the rest of the
collection to perform pre-upgrade analysis (preupg) and the upgrade
itself (redhat-upgrade-tool).

This patch adds suffix to the names of these files to distinguish them
event further.
@PeterMocary PeterMocary force-pushed the rhel6-not-supported-warning branch from e9e0207 to 310d908 Compare January 26, 2026 09:37
@PeterMocary PeterMocary requested review from bocekm and jeffmcutter and removed request for bocekm January 27, 2026 15:01
@richm
Copy link
Collaborator

richm commented Jan 29, 2026

@bocekm @jeffmcutter ok to merge?

Copy link
Collaborator

@jeffmcutter jeffmcutter left a comment

Choose a reason for hiding this comment

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

LGTM

@spetrosi spetrosi merged commit 0743eec into redhat-cop:main Jan 30, 2026
29 of 65 checks passed
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.

5 participants