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

Install rbd-nbd before running device map in krbd #3045

Merged

Conversation

manasagowri
Copy link
Contributor

Baremetal pipeline fix -

http://magna002.ceph.redhat.com/cephci-jenkins/test-runs/baremetal/18.2.0-44/rados/2/tier-2_rbd_encryption/ -
Since we create cluster and configure client only once, we would not have installed rbd-nbd in the previous tests. We need to make sure it is installed before running any of the encryption tests.

Success Log - http://magna002.ceph.redhat.com/cephci-jenkins/cephci-run-WYVBE4/

@sunilangadi2 sunilangadi2 added the RBD Rados Bock Device label Oct 19, 2023
Signed-off-by: manasagowri <manasagowri16@gmail.com>
Copy link
Contributor

@sunilangadi2 sunilangadi2 left a comment

Choose a reason for hiding this comment

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

changes LGTM.

@openshift-ci openshift-ci bot added the lgtm Add this label when the PR is good to be merged label Oct 19, 2023
@manasagowri manasagowri added the approved Override label till owners file is created label Oct 19, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 19, 2023

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: manasagowri, sunilangadi2

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

@openshift-ci openshift-ci bot merged commit 1210d68 into red-hat-storage:master Oct 19, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Override label till owners file is created lgtm Add this label when the PR is good to be merged RBD Rados Bock Device
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants