Skip to content

Conversation

@dumbbell
Copy link
Collaborator

@dumbbell dumbbell commented Nov 20, 2024

… not any release.

Why

Once we release RabbitMQ 4.1.0, it means that the v4.1.x branch would be tested against 4.1.x, i.e. itself.

Note that we will have to update this pinned version from time to time if needed.

@dumbbell dumbbell requested a review from lhoguin November 20, 2024 13:27
@dumbbell dumbbell self-assigned this Nov 20, 2024
@dumbbell dumbbell changed the title GitHub Actions: Run mixed-version testion against 4.0.3 explicitly GitHub Actions: Run mixed-version testing against 4.0.3 explicitly Nov 20, 2024
@dumbbell dumbbell force-pushed the test-against-4.0.x-explicitly branch from 7546430 to ce7c91d Compare November 20, 2024 13:28
@dumbbell dumbbell marked this pull request as ready for review November 20, 2024 17:01
@dumbbell dumbbell marked this pull request as draft November 20, 2024 17:08
@dumbbell dumbbell force-pushed the test-against-4.0.x-explicitly branch from 1276f7e to dd49b7e Compare November 20, 2024 17:38
... not any release.

[Why]
Once we release RabbitMQ 4.1.0, it means that the `v4.1.x` branch would
be tested against 4.1.x, i.e. itself.

Note that we will have to update this pinned version from time to time
if needed.
[Why]
Using the `fetch-gh-release-asset` action outputs work fine for our
official releases, but it does not for our alphas published to
`rabbitmq/server-packages` because the filenames use another version
compared to the GitHub release.

[How]
Always derive the version from the archive filename. This is safe
regardless of what the GitHub release states.
[Why]
Our alpha packages published to `rabbitmq/server-packages` use letters
in addition to digits and dots.

[How]
Accept any string that starts with a digit as the version here. We just
enforce the prefix and the filename extension.

The digit at the beginning it here to exclude the
`rabbitmq-server-generic-unix-latest-toolchain-*.tar.xz` archive.
@dumbbell dumbbell force-pushed the test-against-4.0.x-explicitly branch from c81b148 to 37539d0 Compare November 21, 2024 09:58
@dumbbell dumbbell marked this pull request as ready for review November 21, 2024 10:25
@dumbbell dumbbell merged commit 91ca53b into main Nov 21, 2024
269 checks passed
@dumbbell dumbbell deleted the test-against-4.0.x-explicitly branch November 21, 2024 10:26
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.

3 participants