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

sap_ha_pacemaker_cluster: fix(RHEL): packages on AWS #857

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

ja9fuchs
Copy link
Contributor

@ja9fuchs ja9fuchs commented Sep 4, 2024

In RHEL 9 the package awscli was moved into a bundle with the fence-agents and the AWS platform resource agents are shipped in a separate package that automatically pulls in the dependency to ha-cloud-support, which contains the aws binary as well.

Verified on RHEL 8.6 that it still behaves as before and installs the awscli (version 1) package.
Tested on RHEL 9.2. The fix installs the right packages and the aws cli is present through the HA packages.

ha-cloud-support-4.10.0-43.el9_2.7.x86_64
resource-agents-cloud-4.10.0-34.el9_2.5.x86_64

aws is now in a dedicated path, for use by the AWS fence and resource agents.

# /usr/lib/fence-agents/support/awscli/bin/aws --version
aws-cli/2.2.15 Python/3.9.16 Linux/5.14.0-284.82.1.el9_2.x86_64 source/x86_64.rhel.9 prompt/off

The dedicated package awscli2 will be part of RHEL through AppStream from 9.5 onwards.
The empty list element resulting from the conditional for anything <9.5 is removed from the list in the task that processes the list.

Copy link
Contributor

@marcelmamula marcelmamula left a comment

Choose a reason for hiding this comment

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

LGTM

@ja9fuchs ja9fuchs merged commit 2461cc5 into sap-linuxlab:dev Sep 5, 2024
3 of 4 checks passed
@ja9fuchs ja9fuchs deleted the fix-aws-rhel branch September 5, 2024 09:01
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