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

inject foreman version for stable plugin builds #421

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Feb 27, 2024

this allows to have the same plugin version to have different builds for different foreman releases, thus making it possible to cleanly upgrade to plugin builds that match the foreman release even if there are no plugin version differences.

practical example: foreman_puppet 6.2.0 exists in 3.8, 3.9, 3.10 and nightly, but in 3.10 and nightly it needs to be built with Webpack5 and not 3 to work.
before this change, all builds ended up being 6.2.0-1 and apt would not consider to upgrade those when coming from 3.9 to 3.10, creating a broken setup.
with this change, the versions would be:
6.2.0-1~fm3.8
6.2.0-1~fm3.9
6.2.0-1~fm3.10
6.2.0-1
which apt will gladly upgrade in that order

It would be cool if we could name the last one 6.2.0-1~3.11, but there is no way for us to know that the next version will be 3.11 at that point

this allows to have the same plugin version to have different builds for
different foreman releases, thus making it possible to cleanly upgrade
to plugin builds that match the foreman release even if there are no
plugin version differences.

practical example: foreman_puppet 6.2.0 exists in 3.8, 3.9, 3.10 and
nightly, but in 3.10 and nightly it needs to be built with Webpack5 and
not 3 to work.
before this change, all builds ended up being 6.2.0-1 and apt would not
consider to upgrade those when coming from 3.9 to 3.10, creating a
broken setup.
with this change, the versions would be:
6.2.0-1~fm3.8
6.2.0-1~fm3.9
6.2.0-1~fm3.10
6.2.0-1
which apt will gladly upgrade in that order

It would be cool if we could name the last one 6.2.0-1~3.11, but there
is no way for us to know that the next version will be 3.11 at that
point
@evgeni evgeni merged commit a13893e into theforeman:master Feb 27, 2024
2 checks passed
@evgeni evgeni deleted the plugin-fm-version branch February 27, 2024 12:34
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.

2 participants