Skip to content

Conversation

@imprateeksh
Copy link
Member

Description

This PR addresses the change to include RHEL9 in the module.
Issue - #590

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content
  • Added support for RHEL9 (RHEL_9_64)

Run the pipeline

If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.

Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:

/run pipeline

Checklist for reviewers

  • If relevant, a test for the change is included or updated with this PR.
  • If relevant, documentation for the change is included or updated with this PR.

For mergers

  • Use a conventional commit message to set the release level. Follow the guidelines.
  • Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
  • Use the Squash and merge option.

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh imprateeksh requested a review from ocofaigh February 7, 2025 12:49
@imprateeksh
Copy link
Member Author

imprateeksh commented Feb 7, 2025

Updated multizone cluster example to use RHEL_9_64 operating system. As this test is part of other_tests.go , currently running test locally. Once completed will share the logs.

All resources were created successfully here, 500 errors were seen during destroy, resulting in test failure.

@imprateeksh
Copy link
Member Author

Here are the logs for the mzr cluster but this has failed, though does not looks like failed because of RHEL_9_64 changes.
logs-mzr-eg.txt

Will give another try.

@imprateeksh
Copy link
Member Author

The PR got failed due to some other error which is not related to the changes.
image

@imprateeksh
Copy link
Member Author

/run pipeline

@imprateeksh
Copy link
Member Author

other_tests - Passed test TestRunMultiClusterExample. Logs are shared here:
logs-mzr-eg-2.txt

@ocofaigh ocofaigh merged commit 4f52a84 into main Feb 10, 2025
2 checks passed
@ocofaigh ocofaigh deleted the 590-os-rh9 branch February 10, 2025 10:34
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 3.40.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants