test(robot): migrate test_backing_image_basic_operation #1867
Conversation
e2e/keywords/backing_image.resource
Outdated
| Verify all disk file status of backing image ${backing_image_name} are ready | ||
| all_disk_file_status_are_ready ${backing_image_name} | ||
|
|
||
| Verify try to clean up backing image ${backing_image_name} from a disk will fail |
There was a problem hiding this comment.
nit: Verify clean up backing image ${backing_image_name} from a disk will fail :)
e2e/keywords/backing_image.resource
Outdated
| Verify try to clean up backing image ${backing_image_name} from a disk will fail | ||
| Run Keyword And Expect Error * clean_up_backing_image_from_a_random_disk ${backing_image_name} | ||
|
|
||
| Verify try to delete backing image ${backing_image_name} will fail |
There was a problem hiding this comment.
nit: Verify delete backing image ${backing_image_name} will fail
| Verify try to delete backing image ${backing_image_name} will fail | ||
| Run Keyword And Expect Error * delete_backing_image ${backing_image_name} | ||
|
|
||
| Clean up backing image ${backing_image_name} from a disk |
There was a problem hiding this comment.
Not sure but can we clean up from desired disk if we know node name?
There was a problem hiding this comment.
Yes, it's possible. We can retrieve the node to know the uuid of the disk, and then clean up the backing image by uuid.
But maybe it's not required at this moment, since the test case is testing the cleaning up API works or not, specifying a node add no extra information to it.
e2e/keywords/volume.resource
Outdated
| @@ -10,9 +10,10 @@ Create volume ${volume_id} | |||
| ${volume_name} = generate_name_with_suffix volume ${volume_id} | |||
| create_volume ${volume_name} 2 3 | |||
|
|
|||
| Create volume ${volume_id} with frontend ${frontend_name} | |||
| Create volume ${volume_id} with | |||
| [Arguments] &{config} | |||
yangchiu
force-pushed
the
migrate-backing-image
branch
from
a9c8139
to
d22c9d8
Compare
|
This pull request is now in conflict. Could you fix it @yangchiu? 🙏 |
yangchiu
force-pushed
the
migrate-backing-image
branch
from
d22c9d8
to
abc247e
Compare
|
This pull request is now in conflict. Could you fix it @yangchiu? 🙏 |
yangchiu
force-pushed
the
migrate-backing-image
branch
from
abc247e
to
b6459ad
Compare
|
This pull request is now in conflict. Could you fix it @yangchiu? 🙏 |
Signed-off-by: Yang Chiu <yang.chiu@suse.com>
yangchiu
force-pushed
the
migrate-backing-image
branch
from
6cf95e8
to
c0f621c
Compare
Which issue(s) this PR fixes:
Issue longhorn/longhorn#8259
What this PR does / why we need it:
migrate test_backing_image_basic_operation
Special notes for your reviewer:
Additional documentation or context