| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add QoS support https://review.opendev.org/c/openstack/cinder/+/847730 | 03:16 |
|---|---|---|
| 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 | Konrad Gube proposed openstack/cinder-specs master: Re-propose extend volume completion action for 2023.2 https://review.opendev.org/c/openstack/cinder-specs/+/877230 | 08:39 |
| *** shoffmann1 is now known as shoffmann | 10:17 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Deleting a volume in 'downloading' state https://review.opendev.org/c/openstack/cinder/+/826607 | 11:06 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE 3PAR: Convert volume v2 to base vol properly https://review.opendev.org/c/openstack/cinder/+/862679 | 11:56 |
| happystacker | Hi reviewers, can someone take a look at our two issues for Antelope final RC https://review.opendev.org/c/openstack/cinder/+/858370 and https://review.opendev.org/c/openstack/cinder/+/797970 ? | 13:09 |
| happystacker | thks in advance | 13:09 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: zuul: make the SRBAC job voting https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/877285 | 13:51 |
| Dessira | IDENTIFY | 14:07 |
| enriquetaso | hi Dessira | 14:13 |
| Dessira | hi enriquetaso | 14:14 |
| *** User__ is now known as Toheeb | 16:14 | |
| opendevreview | Tony Saad proposed openstack/os-brick master: Dell PowerFlex: Unnecessary login happen https://review.opendev.org/c/openstack/os-brick/+/876284 | 17:21 |
| whoami-rajat | gmann, so creating multiattach volume is itself not an admin operation, but first we require an admin to create a type so non-admins can create the multiattach volume | 18:39 |
| whoami-rajat | using that type | 18:39 |
| gmann | whoami-rajat: rosmaita: on this comment where we need admin creds to create volume_type - https://review.opendev.org/c/openstack/tempest/+/875372/5/tempest/api/compute/volumes/test_attach_volume.py#384 | 18:40 |
| gmann | whoami-rajat: yeah so how users (non-admin) request volume_type creation to admin ? | 18:40 |
| rosmaita | gmann: it's up to the operator to create those | 18:41 |
| rosmaita | presumably the cloud has some support or docs to explain that to users | 18:41 |
| whoami-rajat | gmann, i think it's part of a deployment process, a day-2 operation is for admins to create the required volume types and if not created, non-admins can ask their operators to create one | 18:41 |
| gmann | like flavor in nova ? | 18:42 |
| whoami-rajat | gmann, does flavors have a 1 to 1 mapping with the compute node? | 18:44 |
| whoami-rajat | like using a particular flavor will enforce creation of a VM in a particular compute node | 18:44 |
| gmann | whoami-rajat: no, they are used for instance signature by users | 18:44 |
| gmann | no | 18:44 |
| gmann | but I think I got the concept here for volume_type | 18:44 |
| whoami-rajat | ok, volume types are generally linked to a particular backend | 18:44 |
| whoami-rajat | and we add properties to the type specific to that backend | 18:45 |
| whoami-rajat | like a multiattach volume type if the backend supports it | 18:45 |
| gmann | flavor in nova is kind of pre-prepared stuff by operator for users resources | 18:45 |
| gmann | ok | 18:45 |
| gmann | whoami-rajat: on tempest tests change (adding admin creds ) let me check/think what we can do as any test require admin creds needs to go to admin dir so that any test env with no admin access can run tempest easily (for example interop certification program) | 18:45 |
| whoami-rajat | that kind of analogy works for volume types as well, volume types are pre-prepared by admins and doesn't need to be created/deleted often | 18:46 |
| gmann | I see | 18:47 |
| whoami-rajat | gmann, ack, yeah i faced the error hence added the admin creds, don't know if there is a better way to do it since we need to setup the admin client for volume types | 18:47 |
| gmann | now testing 'multiattach' as admin or non-admin in tempest is kind of tricky | 18:47 |
| gmann | one way is to create multiattach volume type in devstack (like it does for flavor) and use that in non-admin multiattach test in tempest. It will match with operator/user actual operation. | 18:49 |
| gmann | because I am hesitate to move/test multiattach tests as admin operation in tempest | 18:49 |
| gmann | if making as admin then one example can be think for interop as they will not be able to test multiattach if they are/want to. | 18:50 |
| whoami-rajat | hmm, so there will be a multiattach volume type in every devstack deployment and it will be useless in most of the cases, I don't have any issues adding it if the team agrees | 18:50 |
| whoami-rajat | rosmaita, what do you think about this? ^ | 18:52 |
| gmann | whoami-rajat: not every deployement but if ENABLE_VOLUME_MULTIATTACH is true | 18:53 |
| whoami-rajat | gmann, oh, yes that sounds good then | 18:53 |
| whoami-rajat | gmann, do we run tempest in anything apart from devstack deployment? | 18:55 |
| rosmaita | gmann: i think the way this has been handled in the past is to put tests that need admin setup in the tempest.api.volume.admin package | 18:57 |
| gmann | whoami-rajat: yes for that we can skip the test if we do not find multiattach volume type in cinder. | 18:57 |
| gmann | rosmaita: yeah but multiattach is not admin operation so we should not put all multiattach operation test there | 18:57 |
| whoami-rajat | ok | 18:58 |
| gmann | whoami-rajat: here we can create, comment in patch also - https://github.com/openstack/devstack/blob/3894077e6c002d5238423a17846bd2c1afc6a00b/lib/cinder#L345 | 18:58 |
| gmann | rosmaita: if operation can be done only by admin or most of the things needed by that operation is admin then yes we need to put that under admin dir | 18:59 |
| whoami-rajat | gmann, thanks | 18:59 |
| rosmaita | gmann: makes sense | 18:59 |
| rosmaita | whoami-rajat: sounds like having devstack create the type is the way to go here | 19:00 |
| gmann | whoami-rajat: rosmaita: I have not cut the devstack branch yet so holding this to get your change in https://review.opendev.org/c/openstack/releases/+/877123 | 19:01 |
| gmann | I will be watching the changes and update release patch | 19:01 |
| 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 |
| gouthamr | pbkac | 21:25 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fixed docstring description for test_boot_cloned_encryted_volume https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 21:25 |
| 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/!