| caiqilong | Hi, core reviewers, my patch "https://review.opendev.org/c/openstack/cinder/+/816443" needs other votes or suggestions. Please take a look in spare time. | 01:18 |
|---|---|---|
| opendevreview | Tushar Trambak Gite proposed openstack/python-cinderclient master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/python-cinderclient/+/819259 | 07:03 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinderlib master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/cinderlib/+/819258 | 07:42 |
| *** brinzhang_ is now known as brinzhang | 08:53 | |
| *** dviroel|afk is now known as dviroel | 10:52 | |
| caiqilong | Hi, core reviewers, my patch "https://review.opendev.org/c/openstack/cinder/+/816443" needs other votes or suggestions. Please take a look in spare time | 12:53 |
| whoami-rajat | hey tosky | 13:15 |
| tosky | whoami-rajat: hi | 13:19 |
| whoami-rajat | tosky, i don't exactly remember the conclusion of our discussion regarding the cinder-tempest-plugin patches | 13:20 |
| whoami-rajat | the tempest stable manager import patch depends on one of my tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/790899 | 13:20 |
| tosky | whoami-rajat: I needed to review the underlying patch | 13:20 |
| whoami-rajat | oh ok | 13:20 |
| whoami-rajat | just for an overview, we used to launch an instance, do data change, create snapshot (until 3 snapshots), then launch an instance from each snapshot = 4 instances | 13:23 |
| whoami-rajat | now I'm just creating one instance and attaching those snapshots (volume from the snapshots) to it then detach | 13:23 |
| whoami-rajat | so less resources consumed | 13:23 |
| tosky | uhm | 13:26 |
| tosky | but isn't the tested case a bit different? In the new way it's possible to check if the volume can be restored and its content can be accessed, but not that it can be used to boot | 13:27 |
| whoami-rajat | but we aren't checking anything related to boot/non-boot volumes, we are just checking the consistency of snapshots that if we write data and create snapshots, does the data remain intact when we restore from that snapshot | 13:29 |
| tosky | right, I see now | 13:30 |
| tosky | the test is just abut data integrity | 13:30 |
| tosky | oki | 13:30 |
| whoami-rajat | yes, exactly | 13:30 |
| whoami-rajat | thanks tosky | 13:44 |
| whoami-rajat | geguileo, hey, if you're around can you take a look at the improvements in the snapshots tests you suggested ? https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/790899 | 13:45 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Re-use existing initiator group/host. https://review.opendev.org/c/openstack/cinder/+/787764 | 13:58 |
| opendevreview | Abhishek Kekane proposed openstack/devstack-plugin-ceph master: PoC Allow glance ceph osd cinder and nova pool access https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/819476 | 14:36 |
| *** dviroel is now known as dviroel|lunch | 15:08 | |
| *** dviroel|lunch is now known as dviroel | 16:25 | |
| *** dviroel is now known as dviroel|OUT | 20:28 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!