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

Add fence_aws Fence Agent #30

Merged
merged 1 commit into from
Apr 4, 2023

Conversation

razo7
Copy link
Member

@razo7 razo7 commented Mar 30, 2023

highavailability/ha repository is required for downloading the fence-agents-aws package and fence_aws FA.

@openshift-ci openshift-ci bot requested review from clobrano and slintes March 30, 2023 10:00
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 30, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: razo7

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@razo7
Copy link
Member Author

razo7 commented Mar 30, 2023

/retest

@slintes
Copy link
Member

slintes commented Mar 30, 2023

looks like dnf config-manager --set-enabled ha doesn't work on the CI image

@razo7
Copy link
Member Author

razo7 commented Mar 30, 2023

looks like dnf config-manager --set-enabled ha doesn't work on the CI image

And in Centos it does work 🤷🏻

ha repo is required for downloading the fence-agents-aws package
@razo7
Copy link
Member Author

razo7 commented Apr 4, 2023

CI image has been changed to CentOS stream 8 - openshift/release#37985

@slintes
Copy link
Member

slintes commented Apr 4, 2023

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Apr 4, 2023
@openshift-merge-robot openshift-merge-robot merged commit 43e7b35 into medik8s:main Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants