Add functionality test based on microceph #52
Conversation
|
@taodd Can you rebase this with main? That should make the tests pass. I'll then test & review. |
pponnuvel
left a comment
pponnuvel
left a comment
There was a problem hiding this comment.
Looks good.
But there's a minor issue: the "ceph" cmd can be accessed directly as "microceph" cmd has its own flags & options. So need to replace all "ceph" cmds with "microceph.ceph".
because of this, the test is failing right now.
I also wonder if we need to do all the cleanups and whether we need to check if microceph already exists. Because each time, this is run, a new Ubuntu instance will be created and it won't have microceph.
Cleanups: while it's generally good to cleanup, but in this case, the whole Ubuntu instance will be wiped off regardless of test result. Also if any of the cleanup ops fail, it'll mark the test failed.
| TIMEOUT=120 | ||
| ELAPSED=0 | ||
| while [ $ELAPSED -lt $TIMEOUT ]; do | ||
| if ceph status | grep -q "HEALTH_OK\|HEALTH_WARN"; then |
| ELAPSED=$((ELAPSED + 5)) | ||
| done | ||
|
|
||
| ceph status |
|
|
||
| echo "=== Step 8: Create RBD pool and image for testing ===" | ||
| # Create RBD pool if it doesn't exist | ||
| if ! ceph osd pool ls | grep -q "^test_pool$"; then |
There was a problem hiding this comment.
s/ceph/microceph.ceph
and below too.
There was a problem hiding this comment.
Thanks Pon, let me try to uninstall the pre-installed ceph package and see if it works
|
It's still failing the same way: It suggests there's still the wrong "ceph" command. Perhaps the ceph packages you remove aren't comprehensive? Maybe ceph-common and other exist? I think it'd straightforward to just use |
No description provided.