-
Notifications
You must be signed in to change notification settings - Fork 26
Created new yamls for busybox workloads for provider mdr validation #106
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
base: master
Are you sure you want to change the base?
Conversation
…class details for busybox workloads Signed-off-by: Amrita Mahapatra <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: amr1ta The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@amr1ta i think its better not touch mdr folder. Lets create seperate directory for mdr-converged and copy the yamls to that directory. There we can make changes |
…s added under existing mdr folder
@mashetty330 addressed in latest commit. |
spec: | ||
project: default | ||
source: | ||
repoURL: https://github.com/red-hat-storage/ocs-workloads |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be changed accordingly. We need to reference the workload under provider-mdr
source: | ||
repoURL: https://github.com/red-hat-storage/ocs-workloads | ||
targetRevision: master | ||
path: mdr/cnv-workload/vm-resources/vm-workload-1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
source: | ||
repoURL: https://github.com/red-hat-storage/ocs-workloads | ||
targetRevision: master | ||
path: mdr/cnv-workload/vm-resources/vm-workload-1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
resources: | ||
requests: | ||
storage: 512Mi | ||
storageClassName: ocs-storagecluster-ceph-rbd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
storageclass needs to be rbd virtualization
Created separate pod and pvc yamls for provider mdr with required storagclass details for busybox workloads.