Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Test rook-ceph component in e2e tests #397

Closed
surajssd opened this issue May 7, 2020 · 2 comments · Fixed by #1153
Closed

Test rook-ceph component in e2e tests #397

surajssd opened this issue May 7, 2020 · 2 comments · Fixed by #1153

Comments

@surajssd
Copy link
Member

surajssd commented May 7, 2020

Right now we test only rook which essentially does not deploy anything important(it deploys operator which is kicked into action once we create a CR called CephCluster using rook-ceph component).

So this deploy rook-ceph via CI config. But before doing this we need to make sure there are machines available to run rook-ceph on. This component needs raw disk which it uses for its own purpose, they can't conflict with OpenEBS. So we need to add extra machines, label them so that rook is deployed there and now use some other component to test if it can create PVC from rook.

@surajssd surajssd mentioned this issue May 7, 2020
1 task
@surajssd
Copy link
Member Author

Add e2e monitoring tests as well while someone is working on this. As mentioned in #405 (comment)

invidian added a commit that referenced this issue Nov 2, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
@invidian
Copy link
Member

invidian commented Nov 2, 2020

Partially done in #1084. Will be finished in #1153.

invidian added a commit that referenced this issue Nov 5, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
invidian added a commit that referenced this issue Nov 6, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
invidian added a commit that referenced this issue Nov 6, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants