| opendevreview | OpenStack Release Bot proposed openstack/python-cyborgclient master: Update master for stable/2026.2 https://review.opendev.org/c/openstack/python-cyborgclient/+/1003451 | 10:43 |
|---|---|---|
| chandankumar | sean-k-mooney: Hello, Here is the nvme cleanup strategy test result: https://gist.github.com/raukadah/fdccdecb7c92fc932ac49d2fb7ccad9f | 12:07 |
| chandankumar | Do let me know is there anything else I need to include in this. | 12:08 |
| sean-k-mooney | so i hav enot looked at that specificly in tempet but this is actully wrtign data to the device and verifying that that data is not present as part of the test? or is it just booting the vm with the device and makign sure the cleian up didnt fail | 12:11 |
| sean-k-mooney | and did you capture service log fo the calls | 12:12 |
| sean-k-mooney | ah you do have the agent output | 12:12 |
| chandankumar | I have captured cyborg-agent logs | 12:12 |
| chandankumar | The current test is done with cirros image where we donot write data to the device | 12:12 |
| chandankumar | so it is a plain vm boot with the device | 12:13 |
| chandankumar | and making sure cleanup works fine | 12:13 |
| sean-k-mooney | ack can you do a one of test where you echo a fixed byte strign tot he /dev/nvme... device | 12:13 |
| sean-k-mooney | stop the vm an check its there on the host | 12:14 |
| sean-k-mooney | and hten delete it and check it after | 12:14 |
| sean-k-mooney | just to confirm it worked onces | 12:14 |
| sean-k-mooney | ill review your logs later but if we can do that i think we are good | 12:14 |
| sean-k-mooney | the current testign show the nvme cli arges are valid | 12:14 |
| sean-k-mooney | we dont really need to test nvme cli itself | 12:15 |
| sean-k-mooney | i just want to see it work once | 12:15 |
| sean-k-mooney | to make sure its not siglently failing adn we are not seeign it in the logs | 12:15 |
| chandankumar | sure | 12:15 |
| chandankumar | Let me do that | 12:15 |
| chandankumar | sean-k-mooney: I think it is working with data write https://gist.github.com/raukadah/fdccdecb7c92fc932ac49d2fb7ccad9f#6-manual-data-verification-test | 14:05 |
| chandankumar | Feel free to comment to add more verification | 14:05 |
| chandankumar | I have not cpatured logs for this one | 14:06 |
| opendevreview | Merged openstack/cyborg master: Add device_state field to Device and DriverDevice https://review.opendev.org/c/openstack/cyborg/+/999933 | 19:03 |
| opendevreview | sean mooney proposed openstack/cyborg master: Remove the standalone API server https://review.opendev.org/c/openstack/cyborg/+/1003596 | 19:42 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!