Skip to content

Conversation

@koralowiec
Copy link
Contributor

Description

Updated the meta file and CI workflow, so Ubuntu 24.04 is officially supported by the role

Definition of Done

  • The change is thoroughly documented.
  • The CI passes (R CMD check, linter, unit tests, spelling).
  • Any generated files have been updated (e.g. .Rd files with roxygen2::roxygenise())

@koralowiec koralowiec force-pushed the feat/ubuntu-2404-support branch from 12acd35 to 1068559 Compare June 17, 2025 06:32
@koralowiec koralowiec marked this pull request as draft June 17, 2025 06:34
@koralowiec koralowiec marked this pull request as ready for review June 17, 2025 06:35
@koralowiec koralowiec requested a review from tomascordovil June 18, 2025 11:35
platforms:
- &instance
name: instance-r
image: "geerlingguy/docker-${MOLECULE_DISTRO:-ubuntu2004}-ansible:latest"
Copy link
Contributor

Choose a reason for hiding this comment

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

are we keeping test with ubuntu2004?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The tests in CI are run for 20.04, 22.04 and 24.04. I think it doesn't matter that much what's the default here. When testing changes locally I typically run the tests against the newest and the oldest Ubuntu release

platform: linux/amd64
- <<: *instance
name: instance-r-rh
image: "registry.access.redhat.com/${MOLECULE_DISTRO:-ubi8}/ubi-init"
Copy link
Contributor

Choose a reason for hiding this comment

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

why reverting test to rhel8?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I probably reverted that by accident, but similarly to this #18 (comment) - In CI we run the tests against both releases, so the default here is not so important

@koralowiec koralowiec merged commit 471abd8 into main Jun 23, 2025
7 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.

3 participants