| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 05:34 |
|---|---|---|
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 05:44 |
| opendevreview | Rajat Dhasmana proposed openstack/tempest master: Update multiattach tests https://review.opendev.org/c/openstack/tempest/+/875372 | 06:50 |
| ricolin | gmann: -Thanks for the example, I updated that patch, please kidnly review it again :) | 07:26 |
| *** jpena|off is now known as jpena | 08:01 | |
| *** ykarel is now known as ykarel|lunch | 09:01 | |
| *** ykarel|lunch is now known as ykarel | 10:27 | |
| plibeau | hello guys, do you have sometime to review: https://review.opendev.org/c/openstack/rally-openstack/+/871452 tests are green now :) | 12:52 |
| opendevreview | Merged openstack/devstack master: Rehome functions to enable Neutron's segments integration https://review.opendev.org/c/openstack/devstack/+/818216 | 12:57 |
| opendevreview | Milana Levy proposed openstack/tempest master: Introduce a change that will allow more clients to make volumes https://review.opendev.org/c/openstack/tempest/+/874577 | 13:02 |
| frickler | plibeau: that is a rally patch, not qa. I don't think akurilin is on IRC any longer and I'm not sure if anyone else still does reviews there | 13:03 |
| kopecmartin | plibeau: hi, i'm not sure if anyone on this channel has core rights there | 13:09 |
| kopecmartin | oh, yeah, what frickler said :) | 13:10 |
| plibeau | thx guys :) | 13:22 |
| kopecmartin | gmann: hi, do we have any specific guidenance for the plugins in regards of SRBAC? - https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/872986 | 14:53 |
| kopecmartin | there seem to be some confusion .. f.e. should project reader tests inherit from project member class? | 14:53 |
| opendevreview | Pragadeeswaran Sathyanarayanan proposed openstack/tempest master: rm hardcoded cidr in test_create_server_multi_nic https://review.opendev.org/c/openstack/tempest/+/877292 | 15:19 |
| opendevreview | Pragadeeswaran Sathyanarayanan proposed openstack/tempest master: rm hardcoded cidr in test_create_server_multi_nic https://review.opendev.org/c/openstack/tempest/+/877292 | 15:23 |
| opendevreview | Pragadeeswaran Sathyanarayanan proposed openstack/tempest master: rm hardcoded cidr in test_create_server_multi_nic https://review.opendev.org/c/openstack/tempest/+/877292 | 15:26 |
| *** jpena is now known as jpena|off | 17:20 | |
| opendevreview | Rajat Dhasmana proposed openstack/tempest master: Update multiattach tests https://review.opendev.org/c/openstack/tempest/+/875372 | 18:16 |
| gmann | whoami-rajat: left some comment to put base method in admin class https://review.opendev.org/c/openstack/tempest/+/875372/6 | 18:19 |
| gmann | kopecmartin: commented on patch. there is no requirement of inheriting reader role test class from member | 18:29 |
| whoami-rajat | hi gmann , I've a doubt regarding your comment here https://review.opendev.org/c/openstack/tempest/+/875372/5/tempest/api/compute/volumes/test_attach_volume.py#346 | 18:34 |
| whoami-rajat | gmann, maybe I'm missing something but this class inherits from BaseAttachVolumeTest and that inherits from BaseV2ComputeTest and not the admin base class | 18:34 |
| whoami-rajat | gmann, do you want me to make BaseAttachVolumeTest inherit from BaseV2ComputeAdminTest? | 18:35 |
| gmann | whoami-rajat: ah, I thought these all are admin. | 18:37 |
| gmann | whoami-rajat: but it make me think more from user perspective that is multiattach volume a admin operation ? I am not seeing brian here, should we move to cinder channel ? | 18:38 |
| whoami-rajat | gmann, sure | 18:38 |
| gmann | thanks | 18:39 |
| opendevreview | Merged openstack/devstack master: Fix NotImplementedError in dbcounter on SQLA 2.x https://review.opendev.org/c/openstack/devstack/+/876601 | 18:52 |
| kopecmartin | gmann: thanks , we also have a fix for a quite old bug -> https://review.opendev.org/c/openstack/tempest/+/877292 | 19:56 |
| gmann | kopecmartin: ack, will check after lunch | 19:58 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 21:00 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 21:10 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 21:12 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 21:24 |
| opendevreview | Merged openstack/tempest master: Introduce a change that will allow more clients to make volumes https://review.opendev.org/c/openstack/tempest/+/874577 | 22:24 |
| opendevreview | Goutham Pacha Ravi proposed openstack/devstack-plugin-ceph master: WIP: Cleanup ceph install on cs9 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/877147 | 23:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!