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

Fixes #36666 - Add support for "channel" parameter #130

Merged
merged 3 commits into from
Feb 13, 2024

Conversation

andreamtp
Copy link
Contributor

This commit introduce the possibility to pass the "channel" parameter to
leapp, so during an upgrade can be targetted release 8.6 EUS, even if
source is 7.9 "main".

ref. https://projects.theforeman.org/issues/36666

@andreamtp andreamtp changed the title fix 36666 - Add support for "channel" parameter Fixes #36666 - Add support for "channel" parameter Aug 17, 2023
@andreamtp andreamtp marked this pull request as ready for review August 17, 2023 18:31
@stejskalleos stejskalleos self-assigned this Aug 24, 2023
This commit introduce the possibility to pass the "channel" parameter to
leapp, so during an upgrade can be targetted release 8.6 EUS, even if
source is 7.9 "plain".

Channels list is take from upstream:
https://github.com/oamg/leapp-repository/blob/84d6ce3073e646e8740b72a5e7edda056c1b324a/commands/preupgrade/__init__.py#L28

ref. https://projects.theforeman.org/issues/36666
@andreamtp andreamtp force-pushed the 36666-channel branch 2 times, most recently from 63a1bf4 to 15fbc7d Compare November 9, 2023 00:07
@andreamtp
Copy link
Contributor Author

I've addressed all requested changes, can this PR be reviewed please?

Copy link
Contributor

@stejskalleos stejskalleos left a comment

Choose a reason for hiding this comment

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

bundle exec rails db:seed was failing, due to some error during parsing the description. Adding quotes fixed the problem

app/views/foreman_leapp/job_templates/leapp_preupgrade.erb Outdated Show resolved Hide resolved
app/views/foreman_leapp/job_templates/leapp_upgrade.erb Outdated Show resolved Hide resolved
@andreamtp
Copy link
Contributor Author

Thanks, done.

Unfort. packit fails for unrelated problems...

@stejskalleos stejskalleos merged commit b0dd0b6 into theforeman:master Feb 13, 2024
11 of 12 checks passed
@stejskalleos
Copy link
Contributor

Thanks @andreamtp !

@andreamtp andreamtp deleted the 36666-channel branch February 20, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants