| *** dave-mccowan has joined #openstack-cinder | 00:18 | |
| *** edmondsw has joined #openstack-cinder | 00:46 | |
| *** edmondsw has quit IRC | 00:51 | |
| *** zhaochao has joined #openstack-cinder | 01:07 | |
| *** yangyapeng has joined #openstack-cinder | 01:15 | |
| *** openstackgerrit has joined #openstack-cinder | 01:30 | |
| openstackgerrit | junbo.li proposed openstack/cinder master: Handle TZ change in iso8601 >=1.12.0 https://review.openstack.org/536182 | 01:30 |
|---|---|---|
| *** zhurong has joined #openstack-cinder | 01:30 | |
| *** zhurong has quit IRC | 01:33 | |
| openstackgerrit | junbo.li proposed openstack/cinder master: Handle TZ change in iso8601 >=1.12.0 https://review.openstack.org/536182 | 01:39 |
| *** _dalgaaf has joined #openstack-cinder | 01:42 | |
| *** dalgaaf has quit IRC | 01:46 | |
| *** markvoelker has joined #openstack-cinder | 01:47 | |
| *** daidv has joined #openstack-cinder | 01:51 | |
| openstackgerrit | Merged openstack/cinder master: VMAX driver - QOS settings incorrectly applied on Default storage group https://review.openstack.org/532911 | 01:57 |
| openstackgerrit | Merged openstack/cinder master: VMAX driver - retype fix where workload is None https://review.openstack.org/533170 | 01:57 |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:06 | |
| *** KeithMnemonic has quit IRC | 02:06 | |
| *** gcb has joined #openstack-cinder | 02:09 | |
| *** crushil_ has joined #openstack-cinder | 02:11 | |
| *** markvoelker has quit IRC | 02:21 | |
| *** kaisers__ has joined #openstack-cinder | 02:27 | |
| *** kaisers_ has quit IRC | 02:30 | |
| *** zhurong has joined #openstack-cinder | 02:32 | |
| daidv | smcginnis, Hi | 02:45 |
| daidv | Can I ask you a little about the LVM volume multi attachment? | 02:46 |
| *** crushil_ has quit IRC | 02:53 | |
| *** markvoelker has joined #openstack-cinder | 03:17 | |
| *** psachin has joined #openstack-cinder | 03:39 | |
| *** markvoelker has quit IRC | 03:51 | |
| *** yangyapeng has quit IRC | 03:55 | |
| *** wanghao has quit IRC | 04:00 | |
| *** wanghao has joined #openstack-cinder | 04:00 | |
| *** links has joined #openstack-cinder | 04:00 | |
| *** wanghao has quit IRC | 04:03 | |
| *** wanghao has joined #openstack-cinder | 04:04 | |
| *** chhavi__ has joined #openstack-cinder | 04:05 | |
| *** abhishekk has joined #openstack-cinder | 04:05 | |
| *** crushil_ has joined #openstack-cinder | 04:10 | |
| *** sapd has quit IRC | 04:14 | |
| *** dave-mccowan has quit IRC | 04:21 | |
| *** edmondsw has joined #openstack-cinder | 04:22 | |
| *** edmondsw has quit IRC | 04:27 | |
| *** sapd has joined #openstack-cinder | 04:28 | |
| *** ykarel|away has joined #openstack-cinder | 04:34 | |
| *** yangyapeng has joined #openstack-cinder | 04:34 | |
| *** crushil_ has quit IRC | 04:36 | |
| *** zhurong has quit IRC | 04:39 | |
| *** pgadiya has joined #openstack-cinder | 04:39 | |
| *** vivsoni has joined #openstack-cinder | 04:46 | |
| *** markvoelker has joined #openstack-cinder | 04:48 | |
| *** armaan has quit IRC | 04:56 | |
| *** armaan has joined #openstack-cinder | 04:57 | |
| *** yangyapeng has quit IRC | 05:16 | |
| *** markvoelker has quit IRC | 05:21 | |
| *** kaisers__ has quit IRC | 05:23 | |
| *** kaisers_ has joined #openstack-cinder | 05:33 | |
| *** ykarel|away is now known as ykarel | 05:34 | |
| *** stakeda has joined #openstack-cinder | 05:40 | |
| *** threestrands has quit IRC | 05:40 | |
| *** yangyapeng has joined #openstack-cinder | 05:41 | |
| *** zhurong has joined #openstack-cinder | 05:45 | |
| *** yangyapeng has quit IRC | 05:45 | |
| *** sapd has quit IRC | 05:51 | |
| *** lkuchlan has joined #openstack-cinder | 05:55 | |
| *** sapd has joined #openstack-cinder | 05:56 | |
| *** yangyapeng has joined #openstack-cinder | 05:59 | |
| *** wanghao_ has joined #openstack-cinder | 05:59 | |
| *** wanghao has quit IRC | 06:02 | |
| *** Dinesh_Bhor has quit IRC | 06:02 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:04 | |
| *** Dinesh_Bhor has quit IRC | 06:08 | |
| *** edmondsw has joined #openstack-cinder | 06:11 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:14 | |
| *** armaan has quit IRC | 06:14 | |
| *** Dinesh_Bhor has quit IRC | 06:14 | |
| *** armaan has joined #openstack-cinder | 06:14 | |
| *** edmondsw has quit IRC | 06:15 | |
| *** markvoelker has joined #openstack-cinder | 06:18 | |
| *** jmlowe_ has joined #openstack-cinder | 06:22 | |
| *** sapd has quit IRC | 06:24 | |
| *** sapd has joined #openstack-cinder | 06:24 | |
| *** jmlowe has quit IRC | 06:26 | |
| *** markvoelker has quit IRC | 06:51 | |
| *** dikonoor has joined #openstack-cinder | 06:59 | |
| *** alexchadin has joined #openstack-cinder | 07:05 | |
| openstackgerrit | wanghao proposed openstack/cinder master: Transfer snapshots with volumes https://review.openstack.org/533564 | 07:07 |
| *** nikeshm has joined #openstack-cinder | 07:08 | |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: V3 jsonschema validation: volume_type_access https://review.openstack.org/536237 | 07:14 |
| *** rcernin has quit IRC | 07:17 | |
| nikeshm | https://review.openstack.org/#/c/522746/ https://review.openstack.org/#/c/522746/ | 07:21 |
| nikeshm | please +2 in these twos | 07:21 |
| nikeshm | they are now rebased | 07:21 |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Refactored cinder tempest tests to cinder_tempest_plugin https://review.openstack.org/529824 | 07:23 |
| *** lkuchlan has quit IRC | 07:34 | |
| openstackgerrit | wanghao proposed openstack/cinder master: Support to query volume filter by changes since/before https://review.openstack.org/352688 | 07:39 |
| *** ykarel is now known as ykarel|lunch | 07:41 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder master: Use cinder-tempest-plugin in lvm-lio jobs https://review.openstack.org/536239 | 07:42 |
| *** markvoelker has joined #openstack-cinder | 07:48 | |
| *** sapd_ has joined #openstack-cinder | 07:48 | |
| *** sapd has quit IRC | 07:48 | |
| *** pcaruana has joined #openstack-cinder | 07:52 | |
| *** edmondsw has joined #openstack-cinder | 07:59 | |
| *** adisky__ has joined #openstack-cinder | 08:01 | |
| *** edmondsw has quit IRC | 08:03 | |
| *** lkuchlan has joined #openstack-cinder | 08:09 | |
| evrardjp | good morning! | 08:14 |
| *** alexchadin has quit IRC | 08:15 | |
| evrardjp | Can someone tell where the tempest api tests are? | 08:15 |
| evrardjp | In pike it used to be there: https://github.com/openstack/cinder/tree/stable/pike/cinder/tests/tempest/api | 08:15 |
| evrardjp | master is another game | 08:15 |
| evrardjp | docs isn't really fully explicit: https://docs.openstack.org/cinder/latest/contributor/testing.html | 08:15 |
| *** AlexeyAbashkin has joined #openstack-cinder | 08:15 | |
| *** alexchadin has joined #openstack-cinder | 08:18 | |
| *** tesseract has joined #openstack-cinder | 08:19 | |
| *** ykarel|lunch is now known as ykarel | 08:20 | |
| *** wanghao_ has quit IRC | 08:21 | |
| *** markvoelker has quit IRC | 08:21 | |
| *** wanghao has joined #openstack-cinder | 08:22 | |
| *** danpawlik has joined #openstack-cinder | 08:29 | |
| *** zhaochao has quit IRC | 08:31 | |
| *** zhaochao has joined #openstack-cinder | 08:31 | |
| evrardjp | found it. | 08:44 |
| Roamer` | evrardjp, I was *just* about to ask if tempest/api/volume/ is what you're looking for | 08:45 |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: Remove extra spaces of the volume creation request body https://review.openstack.org/536254 | 08:47 |
| evrardjp | Roamer`: yes, in the new repo :) | 08:54 |
| *** gaurangt has quit IRC | 08:56 | |
| *** gaurangt has joined #openstack-cinder | 08:56 | |
| *** alexchadin has quit IRC | 08:59 | |
| openstackgerrit | Gao Fei proposed openstack/cinder master: Modify broken link https://review.openstack.org/536261 | 09:04 |
| openstackgerrit | Gao Fei proposed openstack/cinder master: Remove unreachable link https://review.openstack.org/536265 | 09:12 |
| *** markvoelker has joined #openstack-cinder | 09:19 | |
| *** markvoelker has quit IRC | 09:23 | |
| *** junboli has joined #openstack-cinder | 09:23 | |
| *** wanghao has quit IRC | 09:31 | |
| *** wanghao has joined #openstack-cinder | 09:31 | |
| *** wanghao has quit IRC | 09:31 | |
| *** wanghao has joined #openstack-cinder | 09:32 | |
| *** gcb has quit IRC | 09:32 | |
| *** wanghao has quit IRC | 09:32 | |
| *** wanghao has joined #openstack-cinder | 09:33 | |
| *** wanghao has quit IRC | 09:33 | |
| *** wanghao has joined #openstack-cinder | 09:33 | |
| *** wanghao has quit IRC | 09:34 | |
| *** wanghao has joined #openstack-cinder | 09:34 | |
| *** wanghao has quit IRC | 09:35 | |
| *** wanghao has joined #openstack-cinder | 09:35 | |
| *** wanghao has quit IRC | 09:35 | |
| *** wanghao has joined #openstack-cinder | 09:36 | |
| *** wanghao has quit IRC | 09:36 | |
| *** wanghao has joined #openstack-cinder | 09:36 | |
| *** wanghao has quit IRC | 09:37 | |
| *** wanghao has joined #openstack-cinder | 09:37 | |
| *** wanghao has quit IRC | 09:38 | |
| *** wanghao has joined #openstack-cinder | 09:38 | |
| *** junboli has quit IRC | 09:38 | |
| *** wanghao has quit IRC | 09:38 | |
| *** wanghao has joined #openstack-cinder | 09:39 | |
| *** wanghao has quit IRC | 09:40 | |
| *** wanghao has joined #openstack-cinder | 09:40 | |
| *** bhagyashris has joined #openstack-cinder | 09:44 | |
| *** edmondsw has joined #openstack-cinder | 09:47 | |
| openstackgerrit | Andreas Scheuring proposed openstack/os-brick master: [WIP] s390x fc: Fix LUN format for Ubuntu with ds8k https://review.openstack.org/535396 | 09:49 |
| openstackgerrit | Andreas Scheuring proposed openstack/os-brick master: s390x fc: Fix device path for Ubuntu with ds8k https://review.openstack.org/535396 | 09:51 |
| *** izombie has joined #openstack-cinder | 09:52 | |
| *** edmondsw has quit IRC | 09:52 | |
| *** pooja_jadhav has joined #openstack-cinder | 09:54 | |
| *** pooja-jadhav has joined #openstack-cinder | 09:54 | |
| *** pooja-jadhav has quit IRC | 09:54 | |
| *** e0ne has joined #openstack-cinder | 09:55 | |
| *** chhagarw has joined #openstack-cinder | 09:58 | |
| *** chhavi__ has quit IRC | 09:58 | |
| *** pgadiya has quit IRC | 09:58 | |
| *** pgadiya has joined #openstack-cinder | 09:59 | |
| *** chhagarw has quit IRC | 10:04 | |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: Remove extra spaces of the volume creation request body https://review.openstack.org/536254 | 10:07 |
| *** alexchadin has joined #openstack-cinder | 10:16 | |
| *** yangyapeng has quit IRC | 10:18 | |
| pooja_jadhav | smcginnis, tommylikehu: Hi, Before schema changes, taking example of create group snashot, group_id is mandatory parameter and if user passes like(group_id: ""/" "/null) then it raises 404 (itemNotFound). | 10:20 |
| pooja_jadhav | After schema changes as user is not passing valid uuid format for group_id, we are raising 400 (BadRequest). So is this convinient that we should raise BadRequest only when user passes invalid format of group_id ? | 10:20 |
| *** e0ne has quit IRC | 10:21 | |
| *** e0ne has joined #openstack-cinder | 10:25 | |
| *** itlinux has joined #openstack-cinder | 10:32 | |
| *** zhurong has quit IRC | 10:33 | |
| *** yangyapeng has joined #openstack-cinder | 10:38 | |
| *** itlinux has quit IRC | 10:41 | |
| *** yangyapeng has quit IRC | 10:44 | |
| *** yangyapeng has joined #openstack-cinder | 10:49 | |
| *** zhurong has joined #openstack-cinder | 10:51 | |
| *** yangyapeng has quit IRC | 10:53 | |
| *** tpsilva has joined #openstack-cinder | 10:56 | |
| *** lpetrut has joined #openstack-cinder | 11:00 | |
| *** abhishekk has quit IRC | 11:02 | |
| *** lpetrut has quit IRC | 11:03 | |
| *** kwathore_ has quit IRC | 11:13 | |
| *** vivsoni has quit IRC | 11:13 | |
| *** kwathore has joined #openstack-cinder | 11:14 | |
| *** vivsoni has joined #openstack-cinder | 11:14 | |
| *** ganso has joined #openstack-cinder | 11:16 | |
| *** yangyapeng has joined #openstack-cinder | 11:17 | |
| *** markvoelker has joined #openstack-cinder | 11:19 | |
| *** links has quit IRC | 11:23 | |
| *** gaurangt has quit IRC | 11:24 | |
| *** gaurangt has joined #openstack-cinder | 11:25 | |
| *** alexchadin has quit IRC | 11:28 | |
| *** _ix has quit IRC | 11:28 | |
| *** AlexeyAbashkin has quit IRC | 11:32 | |
| *** openstackgerrit has quit IRC | 11:33 | |
| *** alexchadin has joined #openstack-cinder | 11:33 | |
| *** links has joined #openstack-cinder | 11:36 | |
| *** yangyapeng has quit IRC | 11:37 | |
| *** yangyapeng has joined #openstack-cinder | 11:37 | |
| tommylikehu | hey pooja_jadhav I think it's fine as long as that will behave the same as volume&snapshot after all of the changes. | 11:37 |
| smcginnis | pooja_jadhav: I don't believe a group ID should ever be anything other than a UUID, so I think that is fine. We are able to fail quicker than before and avoid unnecessary lookups in the DB. | 11:37 |
| *** openstackgerrit has joined #openstack-cinder | 11:40 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [TEST][DNM] Use countable resource for quota reserve/commit https://review.openstack.org/536341 | 11:40 |
| *** AlexeyAbashkin has joined #openstack-cinder | 11:41 | |
| *** _ix has joined #openstack-cinder | 11:41 | |
| pooja_jadhav | tommylikehu, smcginnis: Thanks for you opinion :) | 11:44 |
| *** izombie has quit IRC | 11:44 | |
| *** psachin has quit IRC | 11:53 | |
| *** e0ne has quit IRC | 11:54 | |
| *** markvoelker has quit IRC | 11:54 | |
| *** tesseract-RH has joined #openstack-cinder | 11:55 | |
| *** tesseract has quit IRC | 11:57 | |
| *** _ix_ has joined #openstack-cinder | 12:09 | |
| *** _ix has quit IRC | 12:12 | |
| *** dave-mccowan has joined #openstack-cinder | 12:12 | |
| *** dave-mcc_ has joined #openstack-cinder | 12:21 | |
| *** dave-mccowan has quit IRC | 12:22 | |
| *** alexchadin has quit IRC | 12:35 | |
| *** dikonoor has quit IRC | 12:39 | |
| *** alexchadin has joined #openstack-cinder | 12:42 | |
| *** markvoelker has joined #openstack-cinder | 12:51 | |
| *** deepak_ has quit IRC | 12:58 | |
| *** dikonoor has joined #openstack-cinder | 13:02 | |
| mnaser | smcginnis: https://review.openstack.org/#/c/535914/ if i can have some eyes so we can backport to stable/pike | 13:02 |
| *** markvoelker has quit IRC | 13:02 | |
| smcginnis | mnaser: Sure, I'll take a look now. | 13:02 |
| *** markvoelker has joined #openstack-cinder | 13:02 | |
| *** erlon has joined #openstack-cinder | 13:04 | |
| erlon | geguileo: ping | 13:05 |
| geguileo | erlon: pong | 13:05 |
| erlon | geguileo, hey, good morning, so, have addressed the comments on the opverprovisioning patch | 13:11 |
| erlon | when you have a chance to look it | 13:11 |
| *** deepak_ has joined #openstack-cinder | 13:12 | |
| geguileo | erlon: Thanks, I'll have a look now | 13:12 |
| erlon | so, some ofyour suggestions in the drivers I avoided on purpose to to that, because we don't want to change drivers | 13:12 |
| geguileo | erlon: though it seems you have a pep8 error | 13:12 |
| geguileo | erlon: the bp mentioned that drivers needed to be fixed | 13:12 |
| erlon | I don't think its a good idea to change options in drivers without notifying the vendor | 13:12 |
| geguileo | erlon: we can have separate patches for that | 13:13 |
| *** moshele has joined #openstack-cinder | 13:13 | |
| erlon | aaan my management didn't liked the idea of spent too much time on that as well | 13:13 |
| erlon | geguileo, hmm, Ill check the pep8 then, didn't noticed | 13:13 |
| erlon | *aand | 13:14 |
| *** crushil_ has joined #openstack-cinder | 13:15 | |
| *** abishop has joined #openstack-cinder | 13:16 | |
| *** izombie has joined #openstack-cinder | 13:17 | |
| *** e0ne has joined #openstack-cinder | 13:18 | |
| izombie | smcginnis: https://review.openstack.org/#/c/534363/ doesn't appears to be fixed on rebasing. Any suggestions on debugging migration code? | 13:21 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder master: Overprovisioning Improvements https://review.openstack.org/534854 | 13:26 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder master: Netapp Ontap: Adds support for auto-max-over-subscription https://review.openstack.org/534855 | 13:26 |
| *** zhurong has quit IRC | 13:28 | |
| *** chandankumar has joined #openstack-cinder | 13:29 | |
| *** gcb has joined #openstack-cinder | 13:39 | |
| *** liverpooler has joined #openstack-cinder | 13:41 | |
| *** izombie has quit IRC | 13:41 | |
| *** pgadiya has quit IRC | 13:42 | |
| openstackgerrit | Merged openstack/cinder master: Rename 'WindowsDriver' to 'WindowsISCSIDriver' https://review.openstack.org/535319 | 13:43 |
| openstackgerrit | Merged openstack/cinder master: Remove extra spaces of the volume creation request body https://review.openstack.org/536254 | 13:43 |
| *** edmondsw_ has joined #openstack-cinder | 13:46 | |
| *** itlinux has joined #openstack-cinder | 13:47 | |
| *** lhx_ has joined #openstack-cinder | 13:48 | |
| *** pchavva has joined #openstack-cinder | 13:52 | |
| *** dustins has joined #openstack-cinder | 13:55 | |
| *** ykarel is now known as ykarel|afk | 13:56 | |
| *** edmondsw_ is now known as edmondsw | 13:56 | |
| *** yangyapeng has quit IRC | 13:58 | |
| *** crushil has quit IRC | 13:58 | |
| *** ykarel|afk has quit IRC | 14:03 | |
| *** finucannot is now known as stephenfin | 14:08 | |
| *** itlinux has quit IRC | 14:09 | |
| *** izombie has joined #openstack-cinder | 14:15 | |
| *** dillaman has quit IRC | 14:20 | |
| *** alexchadin has quit IRC | 14:25 | |
| *** eharney has joined #openstack-cinder | 14:26 | |
| *** mriedem has joined #openstack-cinder | 14:27 | |
| *** zhurong has joined #openstack-cinder | 14:28 | |
| *** Guest86649 is now known as dansmith | 14:29 | |
| *** ykarel|afk has joined #openstack-cinder | 14:29 | |
| *** yangyapeng has joined #openstack-cinder | 14:32 | |
| *** crushil has joined #openstack-cinder | 14:34 | |
| smcginnis | izombie: On quick look, one of those is probably a transient error that we see from time to time with notifications. But the other failures look like the API response is changing. | 14:36 |
| smcginnis | izombie: Did that pass for you running locally? | 14:36 |
| izombie | smcginnis: Nope, it didn't. It's throwing 400 error and no response once. Logs aren't much of help. | 14:41 |
| *** moshele has quit IRC | 14:59 | |
| *** izombie has quit IRC | 15:02 | |
| *** namnh has joined #openstack-cinder | 15:05 | |
| *** r-daneel has joined #openstack-cinder | 15:09 | |
| *** crushil_ has quit IRC | 15:11 | |
| *** stelucz_ has joined #openstack-cinder | 15:15 | |
| *** stelucz_ is now known as stelucz | 15:17 | |
| *** lhx_ has quit IRC | 15:18 | |
| *** lhx_ has joined #openstack-cinder | 15:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder master: Add the nova-multiattach job to the check/gate queues https://review.openstack.org/532695 | 15:33 |
| evrardjp | I think I might be doing something wrong with my tempest. | 15:36 |
| evrardjp | Roamer`: are you still there by any chance? | 15:37 |
| *** zhaochao has quit IRC | 15:37 | |
| evrardjp | so because master doesn't have the same plugins as it had days ago, I changed the path of the plugins to install to something like https://review.openstack.org/#/c/535791/4/tests/os_cinder-overrides.yml | 15:37 |
| *** izombie has joined #openstack-cinder | 15:38 | |
| *** namnh has quit IRC | 15:38 | |
| evrardjp | my tempest whitelist seem ok because you didn't change the path of the tests (but I have no knowledge whatsoever of tempest) | 15:38 |
| izombie | smcginnis: Did I miss anything? :P Got disconnected. | 15:38 |
| *** namnh has joined #openstack-cinder | 15:39 | |
| *** namnh has quit IRC | 15:39 | |
| evrardjp | yet I got: ERROR tempest.test_discover.plugins [-] Could not load 'cinder_tests': No module named cinder.tests.tempest.plugin | 15:39 |
| *** namnh has joined #openstack-cinder | 15:39 | |
| *** namnh has quit IRC | 15:39 | |
| evrardjp | does that ring a bell to anyone? | 15:39 |
| *** namnh has joined #openstack-cinder | 15:40 | |
| ericyoung | the in-project tempest plugins got removed late last week | 15:40 |
| *** namnh has joined #openstack-cinder | 15:40 | |
| *** wiggin15 has quit IRC | 15:40 | |
| *** ameade has quit IRC | 15:40 | |
| *** jungleboyj has quit IRC | 15:40 | |
| *** apopovych has quit IRC | 15:40 | |
| *** enriquetaso has quit IRC | 15:40 | |
| *** DuncanT has quit IRC | 15:40 | |
| *** catmando has quit IRC | 15:40 | |
| *** BlackDex has quit IRC | 15:40 | |
| *** twouters has quit IRC | 15:40 | |
| *** niedbalski has quit IRC | 15:40 | |
| evrardjp | ericyoung: indeed ... | 15:40 |
| *** twouters_ has joined #openstack-cinder | 15:40 | |
| evrardjp | ericyoung: and ? | 15:40 |
| *** BlackDex has joined #openstack-cinder | 15:40 | |
| ericyoung | they are going into a new project but I don;t think have made it there yet | 15:40 |
| *** niedbalski_ has joined #openstack-cinder | 15:40 | |
| *** catmando has joined #openstack-cinder | 15:41 | |
| *** apopovych has joined #openstack-cinder | 15:41 | |
| evrardjp | yeah I think there are there: https://review.openstack.org/#/c/535791/4/tests/os_cinder-overrides.yml | 15:41 |
| *** enriquetaso has joined #openstack-cinder | 15:41 | |
| evrardjp | but I think there is something wrong indeed | 15:41 |
| *** jungleboyj has joined #openstack-cinder | 15:41 | |
| *** wiggin15 has joined #openstack-cinder | 15:41 | |
| evrardjp | I think the plugin got this content: | 15:41 |
| ericyoung | sounds like it. I am not sure where it stands. I just know that my patch for the tests is now obsolete :( | 15:41 |
| evrardjp | /lib/python2.7/site-packages/cinder_tempest_plugin-0.0.1.dev50-py2.7.egg-info/entry_points.txt:cinder_tests = cinder.tests.tempest.plugin:CinderTempestPlugin | 15:41 |
| *** ameade has joined #openstack-cinder | 15:41 | |
| *** DuncanT has joined #openstack-cinder | 15:41 | |
| evrardjp | ericyoung: I feel sad for you :[ | 15:42 |
| evrardjp | what I know is that our gates are broken due to that... I may need to pin cinder to a different version in the meantime. | 15:42 |
| ericyoung | evrardjp, was only a few lines of code so not horrible | 15:42 |
| smcginnis | izombie: No, I'm not sure. But if it fails locally, at least you can debug it and see where the problem is. | 15:43 |
| ericyoung | evrardjp, the patch to get the plugin to the new project is https://review.openstack.org/#/c/529824/ | 15:43 |
| *** twouters_ has quit IRC | 15:44 | |
| *** twouters_ has joined #openstack-cinder | 15:44 | |
| smcginnis | evrardjp: Is that actually causing errors? Tempest tests are in cinder_tempest_plugin. | 15:44 |
| smcginnis | There's that patch to rename the path. ^^ | 15:44 |
| *** twouters_ is now known as twouters | 15:44 | |
| smcginnis | I didn't realize that was actually causing failures in the mean time though. | 15:44 |
| evrardjp | our gates define where we take the tempest plugins, and which to whitelist :) | 15:44 |
| smcginnis | jungleboyj or other cores: If anyone is around, please take a look at the patch above. | 15:45 |
| evrardjp | but my concern is different | 15:45 |
| smcginnis | Otherwise I can push it through if no one else is around. | 15:45 |
| evrardjp | let me review that quick :D | 15:45 |
| smcginnis | evrardjp: What's your concern? | 15:45 |
| jungleboyj | smcginnis: Looking. | 15:45 |
| evrardjp | do we still need the .plugin? | 15:46 |
| smcginnis | evrardjp: Not sure if I understand. | 15:46 |
| evrardjp | my bad, misread :p | 15:46 |
| evrardjp | yeah should be good for me. | 15:46 |
| smcginnis | OK, good. :) | 15:47 |
| evrardjp | as usual, thanks all | 15:47 |
| evrardjp | well let me double check to see if I can break more stuff with it. | 15:48 |
| smcginnis | I would have pushed that along earlier but I didn't realize the impact. | 15:48 |
| evrardjp | sorry, I meant FIX more stuff. | 15:48 |
| smcginnis | Hah :) | 15:48 |
| evrardjp | bad working habit. | 15:48 |
| evrardjp | :p | 15:48 |
| smcginnis | It's good to break stuff. ;) | 15:48 |
| evrardjp | Breaking is experiencing! | 15:48 |
| evrardjp | :D | 15:48 |
| *** felipemonteiro_ has joined #openstack-cinder | 15:49 | |
| evrardjp | let me try that patch real quick | 15:49 |
| *** alkhodos has joined #openstack-cinder | 15:51 | |
| *** david-lyle has joined #openstack-cinder | 15:53 | |
| *** niedbalski_ is now known as niedbalski | 15:54 | |
| *** niedbalski has joined #openstack-cinder | 15:54 | |
| evrardjp | I think that changes in cinder-tempest-plugin should trigger a run of tempest too, just in case :) | 15:55 |
| *** felipemonteiro__ has joined #openstack-cinder | 15:55 | |
| evrardjp | I know I start to be picky and all that! | 15:55 |
| eharney | that is being done here: https://review.openstack.org/#/c/531206/ | 15:55 |
| smcginnis | Definitely need more than pep8. | 15:56 |
| *** lkuchlan has quit IRC | 15:57 | |
| *** felipemonteiro_ has quit IRC | 15:58 | |
| *** markstur has joined #openstack-cinder | 15:59 | |
| evrardjp | if you want, we have cross repo testing. We can make it work for you | 15:59 |
| evrardjp | We already have that for keystone | 16:00 |
| *** markstur has quit IRC | 16:03 | |
| *** ykarel|afk is now known as ykarel | 16:05 | |
| *** izombie has quit IRC | 16:06 | |
| evrardjp | if you're interested by that kind of testing, ping me anytime | 16:06 |
| smcginnis | evrardjp: I meant to check on that. I know there are existing jobs out there that should work. Do you have a pointer for that? | 16:06 |
| jungleboyj | smcginnis: eharney xyang vkmc is asking if we have any projects that would go for about 3 months that we could use to help mentor a Google Summer of Code intern. | 16:06 |
| evrardjp | or let's discuss that at the ptg if you're not in a hurry | 16:06 |
| jungleboyj | Any ideas? I would be willing to do the mentoring but need an idea of what they can work on. | 16:07 |
| *** lpetrut has joined #openstack-cinder | 16:07 | |
| evrardjp | smcginnis: yeah, I can discuss with you on what we did for keystone for their upgrade testing | 16:07 |
| smcginnis | jungleboyj: I'll have to think about that. Nothing coming to mind immediately that would be a nice neat 3 month task, but I'm sure we can find something. | 16:07 |
| jungleboyj | Ok. We should chat soon as they need ideas to propose yet today. :-( | 16:08 |
| smcginnis | evrardjp: That would be cool. We need something in that repo so we don't find out too late that we've merged something problematic. | 16:08 |
| xyang | jungleboyj: I don't have something in mind right now either. | 16:08 |
| evrardjp | smcginnis: https://github.com/openstack/keystone/blob/master/.zuul.yaml#L72-L87 that's what they have | 16:08 |
| evrardjp | basically that triggers our job, so you'd have to replace os_keystone with os_cinder | 16:09 |
| evrardjp | that's for upgrade testing | 16:09 |
| evrardjp | if you want a standard deploy, you'd also have to replace tox_env: upgrade by tox_env: deploy if I am not mistaken | 16:09 |
| evrardjp | or functional | 16:09 |
| evrardjp | let me check | 16:09 |
| *** crose has joined #openstack-cinder | 16:09 | |
| smcginnis | evrardjp: So that will cause keystone changes to run ansible playbooks to upgrade to the latest change and make sure it doesn't break that, right? | 16:09 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Fix mock for sleep mocks in volume drivers tests https://review.openstack.org/531363 | 16:09 |
| smcginnis | So kind of like grenade, but validates the ansible bits. | 16:10 |
| evrardjp | smcginnis: yes, we deploy infra needed for keystone + we deploy pike keystone + we upgrade to queens keystone | 16:10 |
| evrardjp | yes kinda like grenade | 16:10 |
| jungleboyj | e0ne: You have any 3 month type projects that an intern could work on for Cinder? | 16:10 |
| evrardjp | let me check the name we use for deploys | 16:11 |
| evrardjp | in case you just want to test deployments | 16:11 |
| e0ne | jungleboyj: hi. what do you mean? | 16:11 |
| smcginnis | evrardjp: Cool! I'll have to play around with that a little later. | 16:11 |
| jungleboyj | vkmc is looking for a project that we could give to a Google Summer of Code intern and then help them through the project. They needs some ideas today. | 16:12 |
| evrardjp | smcginnis: functional would be the name to use for simple deploy | 16:12 |
| smcginnis | evrardjp: Great - noted! | 16:12 |
| evrardjp | smcginnis: Well, right now it's broken due to the thing we just discussed above :p | 16:12 |
| smcginnis | evrardjp: :) | 16:12 |
| smcginnis | evrardjp: Hopefully that won't take too long to make it through gate. | 16:13 |
| xyang | jungleboyj: do we have anything here that we are looking for volunteers?https://etherpad.openstack.org/p/cinder-spec-review-tracking | 16:13 |
| *** markstur has joined #openstack-cinder | 16:13 | |
| evrardjp | yeah with pep8 only testing it's gonna be really fast | 16:13 |
| xyang | jungleboyj: I wonder if there is any spec already merged (or in good shape) that needs someone to work on | 16:13 |
| smcginnis | evrardjp: True! | 16:13 |
| *** mvk has quit IRC | 16:13 | |
| evrardjp | smcginnis: If you're willing to use our gate systems, I can explain you that in more details, because it's still very tangled right now. | 16:14 |
| evrardjp | not really gates, I mean tests... You can use it for whatever, and should probably start with nv, just to get familiar with it :) | 16:14 |
| evrardjp | my english is terribad! | 16:14 |
| e0ne | jungleboyj: it would be great to help me with generic backups | 16:14 |
| smcginnis | evrardjp: Hah, it's better than some people I know in the US. ;) | 16:15 |
| evrardjp | I want names! | 16:15 |
| evrardjp | :p | 16:15 |
| smcginnis | evrardjp: I can't really get to it right away, but I'll try putting up a patch to that later and we can see how it looks. | 16:15 |
| e0ne | jungleboyj: we can discuss such things on meeting and/or ptg | 16:15 |
| jungleboyj | e0ne: Oh yes, that needs work and is definitely an important piece of Cinder. | 16:15 |
| evrardjp | haha sure no worries. I am not in US, so it might take some time before my answer. | 16:15 |
| evrardjp | but don't hesitate to discuss with us, we're happy to help. | 16:16 |
| jungleboyj | e0ne: Sure, we just need an idea to get to vkmc. We can talk more about the details at the PTG. | 16:16 |
| evrardjp | and are open to suggestions for improvements! | 16:16 |
| smcginnis | evrardjp: Yeah, I'll try bug you in Belgian friendly times. | 16:16 |
| evrardjp | woot! | 16:16 |
| e0ne | jungleboyj: I'll have more time to get it done for Rocky. I really hope to finish it | 16:16 |
| evrardjp | jungleboyj: you remind me that I should do that too. :D | 16:17 |
| evrardjp | anyway, thanks for the chat, talk to you laterz! | 16:17 |
| *** itlinux has joined #openstack-cinder | 16:17 | |
| jungleboyj | e0ne: I am happy to help with the initial mentoring and education if you can help with the guidance on the technical side. | 16:18 |
| *** david-lyle has quit IRC | 16:19 | |
| e0ne | jungleboyj: sure, I'll do my best to help you with this | 16:19 |
| jungleboyj | e0ne: Awesome. Thank you! | 16:21 |
| e0ne | jungleboyj: np. you're welcome! | 16:21 |
| *** namnh has quit IRC | 16:24 | |
| *** namnh has joined #openstack-cinder | 16:25 | |
| *** itlinux has quit IRC | 16:26 | |
| *** AlexeyAbashkin has quit IRC | 16:27 | |
| *** eharney has quit IRC | 16:27 | |
| *** crushil has quit IRC | 16:28 | |
| *** _pewp_ has quit IRC | 16:28 | |
| *** david-lyle has joined #openstack-cinder | 16:28 | |
| *** _pewp_ has joined #openstack-cinder | 16:29 | |
| *** namnh has quit IRC | 16:30 | |
| *** zhurong has quit IRC | 16:31 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: allow snapshot ro attach https://review.openstack.org/528350 | 16:33 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: Add Windows volume backup support https://review.openstack.org/528352 | 16:33 |
| *** namnh has joined #openstack-cinder | 16:35 | |
| *** moshele has joined #openstack-cinder | 16:36 | |
| *** namnh has quit IRC | 16:39 | |
| *** moshele has quit IRC | 16:40 | |
| *** ykarel is now known as ykarel|afk | 16:44 | |
| *** kien-ha has joined #openstack-cinder | 16:46 | |
| *** dikonoor has quit IRC | 16:47 | |
| *** tesseract-RH has quit IRC | 16:51 | |
| *** _pewp_ has quit IRC | 16:53 | |
| *** _pewp_ has joined #openstack-cinder | 16:53 | |
| kien-ha | Hello everyone, I recently gained an interest in device driver development but ]] | 16:56 |
| kien-ha | I was wondering if this is the right place to be | 16:56 |
| jungleboyj | e0ne: I see you are +1 on the SMBFS patch. That kind of looks like a feature though and I feel we are a little late in the game for that. | 16:56 |
| openstackgerrit | Merged openstack/cinder master: Move disk space check to run when needed https://review.openstack.org/535914 | 16:57 |
| openstackgerrit | Merged openstack/cinder master: VMAX doc - Queens feature documentation https://review.openstack.org/533147 | 16:57 |
| openstackgerrit | Merged openstack/cinder master: ScaleIO: Enable multiattach for volumes https://review.openstack.org/531511 | 16:57 |
| openstackgerrit | Merged openstack/cinder master: [VNX]Add `force detach` support https://review.openstack.org/534157 | 16:57 |
| e0ne | jungleboyj: do you mean this one https://review.openstack.org/528350? | 16:57 |
| jungleboyj | kien-ha: Well, the drivers that are written for Cinder control storage backends. It isn't low level device driver development but more high level. | 16:57 |
| jungleboyj | e0ne: Yes. | 16:57 |
| e0ne | jungleboyj: removed my +2. I reviewed it last week, so it had a chance to be merged before FF | 16:59 |
| jungleboyj | Ah ... e0ne I didn't realize that. You think it is safe to get in since it was a case of me not looking at it? | 16:59 |
| *** crose has quit IRC | 16:59 | |
| e0ne | jungleboyj: it's up to you. we've got such problems every release: some patches are not reviewed before FF | 17:00 |
| jungleboyj | e0ne: I know. I tried to get through everything the end of last week but obviously missed some things. Let me take a look at that one. | 17:01 |
| e0ne | need to head home. bb in ~2 hours | 17:01 |
| jungleboyj | e0ne: Ok. Safe travels. :-) | 17:01 |
| e0ne | thanks:) | 17:01 |
| *** e0ne has quit IRC | 17:03 | |
| kien-ha | jungleboyj: I see. Do I need specific knowledge/experience to help development of Cinder? | 17:03 |
| kien-ha | I've never developed for drivers before but I do wish to learn more about them. | 17:04 |
| *** felipemonteiro__ has quit IRC | 17:04 | |
| jungleboyj | kien-ha: Experience with Python and storage are helpful. | 17:04 |
| kien-ha | jungleboyj: I do have Python experience from developing on JJB . Not too sure about the 'storage' side of things | 17:06 |
| *** pcaruana has quit IRC | 17:11 | |
| *** lhx_ has quit IRC | 17:12 | |
| jungleboyj | kien-ha: Ok. Well, what I would recommend, if you understand python, is to spend some time doing code reviews and see if the project is of interest to you. We desperately need more help with reviews. | 17:12 |
| jungleboyj | kien-ha: That is how I got started with Cinder and figured things out from there. | 17:12 |
| *** moshele has joined #openstack-cinder | 17:13 | |
| kien-ha | jungleboyj: Brilliant. I'll do just that then. Thanks! | 17:15 |
| jungleboyj | kien-ha: Let me know if you have questions as you are learning. | 17:16 |
| kien-ha | jungleboyj: will do. | 17:17 |
| *** moshele has quit IRC | 17:24 | |
| *** winston-d has quit IRC | 17:26 | |
| *** jose-phillips has joined #openstack-cinder | 17:26 | |
| *** ntpttr_ has joined #openstack-cinder | 17:27 | |
| *** ntpttr_ has quit IRC | 17:27 | |
| jgriffith | smcginnis: jungleboyj I believe smcginnis is correct about return after the yield here: https://review.openstack.org/#/c/528352/4 | 17:30 |
| *** crushil has joined #openstack-cinder | 17:31 | |
| jgriffith | I added my 2 cents, I *believe* it's ok to have the return, but if it's truly a generator it'll never get there | 17:31 |
| jgriffith | so it is not "correct" | 17:31 |
| jgriffith | lpetrut: ^^ | 17:31 |
| *** crushil has left #openstack-cinder | 17:31 | |
| smcginnis | Yeah, probably OK, but unnecessary. | 17:31 |
| jgriffith | smcginnis: agreed | 17:32 |
| jgriffith | and if nothing else it's confusing | 17:32 |
| smcginnis | ++ | 17:32 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: allow snapshot ro attach https://review.openstack.org/528350 | 17:37 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: Add Windows volume backup support https://review.openstack.org/528352 | 17:37 |
| openstackgerrit | Merged openstack/cinder-tempest-plugin master: Refactored cinder tempest tests to cinder_tempest_plugin https://review.openstack.org/529824 | 17:39 |
| lpetrut | jgriffith: smcginnis: here's a small snippet, showing why we actually do need the return there: http://paste.openstack.org/show/650203/ | 17:41 |
| lpetrut | sorry for the late reply | 17:41 |
| *** izombie has joined #openstack-cinder | 17:42 | |
| jgriffith | lpetrut: huh... so is that something to do with some magic in the decorator? | 17:42 |
| jgriffith | lpetrut: I would expect the yield with any generator to replace the return, so I find that odd | 17:43 |
| *** kaisers_ has quit IRC | 17:43 | |
| smcginnis | lpetrut: Ah, I see now. Thanks. Yields can be confusing. | 17:43 |
| jgriffith | I believe what you're saying; just trying to understand *why* :) | 17:43 |
| smcginnis | jgriffith: I don't believe it will return, it actually will still execute the rest. | 17:43 |
| smcginnis | jgriffith: I've seen it used to return a value but still do some async work that needs to be done before the next thing to return. | 17:44 |
| lpetrut | contextlib just calls .next() on the returned generator, expecting the second call to raise an IterationError, which happens when the function returns | 17:44 |
| jgriffith | Ahhh! | 17:44 |
| jgriffith | so the return is a "stop" so to speak | 17:45 |
| lpetrut | yep | 17:45 |
| jgriffith | ie don't iterate any more | 17:45 |
| jgriffith | ok, I get it | 17:45 |
| jgriffith | sorry for the confusion | 17:45 |
| lpetrut | no worries, thanks for reviewing the change :) | 17:45 |
| smcginnis | lpetrut: Thanks for the clarification. | 17:45 |
| *** jdillaman has joined #openstack-cinder | 17:47 | |
| lpetrut | I do have a small issue though: the patch depends on an os-brick change: https://review.openstack.org/#/c/528237/ we've just managed to get the os-brick ci up (it had some planned downtime after we've moved some hardware) | 17:48 |
| lpetrut | do you think there's any chance we may get an os-win release in time? | 17:48 |
| lpetrut | os-brick* | 17:48 |
| *** izombie has quit IRC | 17:48 | |
| *** izombie_ has joined #openstack-cinder | 17:49 | |
| *** hoonetorg has quit IRC | 17:50 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 17:54 | |
| smcginnis | Hmm, no. Non-library freeze was last Thursday, so only critical library bugs can be released now. | 17:55 |
| *** mvenesio has joined #openstack-cinder | 17:56 | |
| *** AlexeyAbashkin has quit IRC | 17:56 | |
| lpetrut | got it, thanks. I saw that q-3 (Jan 22 - Jan 26) was marked as deadline for "Final release for client libraries" https://releases.openstack.org/queens/schedule.html, wasn't aware about the Thursday's deadline | 17:58 |
| *** kien-ha has quit IRC | 17:58 | |
| smcginnis | Yeah, non-client last week, client this week, then we have a couple weeks for requirements updates to settle before the RC. | 17:59 |
| lpetrut | oh, right. makes sense | 18:01 |
| izombie_ | how do we debug unit-test failures? I looked into logs, but it only prints the assertion failures and not the reason request failed. | 18:01 |
| *** izombie_ is now known as ik__ | 18:02 | |
| *** ik__ is now known as izombie | 18:02 | |
| *** nchawla3 has joined #openstack-cinder | 18:04 | |
| Roamer` | izombie, you can look at the logs of the services that processed the requests... and that's pretty much all there is | 18:04 |
| *** rmcall has joined #openstack-cinder | 18:05 | |
| *** nchawla3 has left #openstack-cinder | 18:06 | |
| *** hoonetorg has joined #openstack-cinder | 18:07 | |
| *** david-lyle has quit IRC | 18:09 | |
| *** moshele has joined #openstack-cinder | 18:10 | |
| smcginnis | izombie: Here are some instructions: https://review.openstack.org/#/c/533813/ | 18:10 |
| izombie | smcginnis: checking it | 18:11 |
| *** felipemonteiro has joined #openstack-cinder | 18:12 | |
| *** e0ne has joined #openstack-cinder | 18:12 | |
| *** felipemonteiro_ has joined #openstack-cinder | 18:15 | |
| *** e0ne has quit IRC | 18:16 | |
| *** felipemonteiro has quit IRC | 18:18 | |
| *** wanghao_ has joined #openstack-cinder | 18:23 | |
| *** wanghao has quit IRC | 18:24 | |
| *** prashkre has joined #openstack-cinder | 18:26 | |
| mnaser | thanks for the +2's, backport - https://review.openstack.org/#/c/536529/ :) | 18:27 |
| *** kaisers_ has joined #openstack-cinder | 18:27 | |
| *** Cibo has joined #openstack-cinder | 18:28 | |
| *** felipemonteiro_ has quit IRC | 18:29 | |
| *** pcaruana has joined #openstack-cinder | 18:32 | |
| *** kien-ha has joined #openstack-cinder | 18:33 | |
| *** e0ne has joined #openstack-cinder | 18:36 | |
| *** jmlowe_ has quit IRC | 18:37 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 18:38 | |
| *** AlexeyAbashkin has quit IRC | 18:42 | |
| *** ykarel|afk has quit IRC | 18:44 | |
| *** eharney has joined #openstack-cinder | 18:48 | |
| izombie | Any idea why a request would get `There is no such action: None` even though routing args has action. https://pastebin.com/raw/z1C3Zfa1 | 18:52 |
| *** lpetrut has quit IRC | 18:54 | |
| *** links has quit IRC | 18:55 | |
| *** jmlowe has joined #openstack-cinder | 19:00 | |
| *** kien-ha has quit IRC | 19:01 | |
| prashkre | Hello Everyone. Could someone help me on my query? | 19:04 |
| *** prashkre_ has joined #openstack-cinder | 19:06 | |
| *** prashkre has quit IRC | 19:06 | |
| *** david-lyle has joined #openstack-cinder | 19:07 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 19:08 | |
| smcginnis | izombie: Maybe a bad mock? | 19:09 |
| *** winston-d has joined #openstack-cinder | 19:10 | |
| *** adisky__ has quit IRC | 19:11 | |
| izombie | smcginnis: probably. the api router for v3 api is strangely v2. | 19:14 |
| prashkre_ | Method ssh_execute() at https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/processutils.py#L504 is being invoked at many places to run command through ssh and return command output. For example, when getting all volume detials from a storage, if some volume name has substrings for these kind of strings ['adminPass', 'admin_pass', 'password', 'admin_password', 'auth_token', 'new_pass', 'auth_password', 'secret_uuid', | 19:18 |
| prashkre_ | 'secret', 'sys_pswd', 'token', 'configdrive', 'CHAPPASSWORD', 'encrypted_key'] then output is being masked with "strutils.mask_password(stdout)" at https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/processutils.py#L540. Due to this getting a malformed output. | 19:18 |
| prashkre_ | Is there any situation that we get sensitive information from external sources, whose output needs to be masked as above? | 19:18 |
| *** armaan has quit IRC | 19:19 | |
| *** prashkre_ is now known as prashkre | 19:19 | |
| *** prashkre is now known as prashkre_ | 19:19 | |
| *** armaan has joined #openstack-cinder | 19:20 | |
| *** AlexeyAbashkin has quit IRC | 19:20 | |
| *** itlinux has joined #openstack-cinder | 19:23 | |
| smcginnis | Infra folks (cc: dhellmann) - Are there any planned upcoming changes that may impact releases that we should be aware of? | 19:25 |
| smcginnis | Oops, wrong channel. Hah! | 19:26 |
| *** itlinux has quit IRC | 19:28 | |
| *** prashkre_ has quit IRC | 19:34 | |
| izombie | smcginnis: Fixed. It was right in front. | 19:39 |
| openstackgerrit | Ibad Khan proposed openstack/cinder master: Fix: request-id missing in volume action's response header. https://review.openstack.org/534363 | 19:39 |
| *** kien-ha has joined #openstack-cinder | 19:40 | |
| smcginnis | izombie: Awesome! | 19:41 |
| *** itlinux has joined #openstack-cinder | 19:43 | |
| *** pcaruana has quit IRC | 19:43 | |
| *** itlinux has quit IRC | 19:46 | |
| openstackgerrit | Ibad Khan proposed openstack/cinder master: Fix: request-id missing in volume action's response header. https://review.openstack.org/534363 | 19:46 |
| *** izombie has quit IRC | 19:48 | |
| *** dave-mcc_ is now known as dave-mcowan | 19:59 | |
| *** dave-mcowan is now known as dave-mccowan | 19:59 | |
| *** dave-mccowan is now known as dave-mcc_ | 20:00 | |
| *** jmlowe has quit IRC | 20:00 | |
| *** namnh has joined #openstack-cinder | 20:02 | |
| *** namnh has quit IRC | 20:03 | |
| *** namnh has joined #openstack-cinder | 20:03 | |
| *** harlowja has joined #openstack-cinder | 20:04 | |
| *** jmlowe has joined #openstack-cinder | 20:06 | |
| *** namnh has quit IRC | 20:07 | |
| *** erlon has quit IRC | 20:07 | |
| *** kwathore_ has joined #openstack-cinder | 20:16 | |
| *** kwathore has quit IRC | 20:20 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble: update config doc to add missing extra-specs support https://review.openstack.org/536555 | 20:22 |
| *** moshele has quit IRC | 20:27 | |
| *** namnh_ has joined #openstack-cinder | 20:32 | |
| *** namnh_ has quit IRC | 20:33 | |
| *** namnh_ has joined #openstack-cinder | 20:33 | |
| *** johnthetubaguy has quit IRC | 20:33 | |
| *** itlinux has joined #openstack-cinder | 20:38 | |
| *** jmlowe has quit IRC | 20:39 | |
| *** johnthetubaguy has joined #openstack-cinder | 20:40 | |
| *** namnh_ has quit IRC | 20:45 | |
| *** r-daneel has quit IRC | 20:46 | |
| *** alkhodos has quit IRC | 20:50 | |
| ildikov | mriedem: hi, welcome back :) | 20:55 |
| ildikov | mriedem: if you're around, it would be great to have a quick chat about this one: https://review.openstack.org/#/c/535863/ | 20:55 |
| ildikov | mriedem: or well, the bug itself that'll report soon | 20:56 |
| *** liverpooler has quit IRC | 20:59 | |
| *** r-daneel has joined #openstack-cinder | 21:04 | |
| *** dave-mcc_ has quit IRC | 21:05 | |
| mriedem | ildikov: i'm here now | 21:06 |
| *** markstur has quit IRC | 21:17 | |
| *** kien-ha has quit IRC | 21:20 | |
| *** e0ne has quit IRC | 21:22 | |
| ildikov | mriedem: sorry, back | 21:24 |
| ildikov | mriedem: really just a heads up on that the old way of creating a multi-attach volume is basically for anyone who can create a volume, while creating a multiattach volume type is an admin only operation | 21:25 |
| mriedem | yeah i know | 21:26 |
| mriedem | i think i pointed out something like that in one of the cinder patches to deprecate the old multiattach way or the one that added the new multiattach policy stuff, can't remember | 21:27 |
| mriedem | you basically have to deprecate the multiattach request parameter in the volume create API to get non-admin users to stop using it | 21:27 |
| mriedem | i.e. remove the request parameter in a microversion | 21:28 |
| ildikov | I think it is deprecated now, but I either missed or forgot about this by now | 21:28 |
| openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble: update config doc to add missing extra-specs support https://review.openstack.org/536555 | 21:28 |
| ildikov | and wanted to be sure we're on the same page on this | 21:28 |
| mriedem | it's "deprecated" in that the cinder-api logs say it is, but the user doesn't know that | 21:28 |
| mriedem | unless a new microversion landed in cinder last week | 21:28 |
| ildikov | not that I know of | 21:29 |
| smcginnis | It never "worked" in the first place since it was only part of the solution. | 21:29 |
| ildikov | smcginnis: creating a multi-attach volume always worked, attaching it multiple times is what never worked :) | 21:29 |
| smcginnis | ildikov: Yeah, which is why I say it never worked. :) | 21:30 |
| ildikov | smcginnis: according to tempest right now the old way is the only way to create a multi-attach volume, but don't tell anyone :) | 21:30 |
| *** dave-mccowan has joined #openstack-cinder | 21:30 | |
| smcginnis | Need to hack in and edit the channel log to strike that comment. :) | 21:32 |
| *** jmlowe has joined #openstack-cinder | 21:33 | |
| *** markstur has joined #openstack-cinder | 21:38 | |
| ildikov | :) | 21:38 |
| ildikov | I will try my best to add tests to the bug fix patch | 21:39 |
| ildikov | not gonna happen today though | 21:39 |
| *** lpetrut has joined #openstack-cinder | 21:47 | |
| *** dave-mcc_ has joined #openstack-cinder | 21:51 | |
| *** edmondsw has quit IRC | 21:52 | |
| *** dave-mccowan has quit IRC | 21:52 | |
| *** liverpooler has joined #openstack-cinder | 21:58 | |
| *** _ix has joined #openstack-cinder | 22:00 | |
| *** pchavva has quit IRC | 22:01 | |
| *** _ix_ has quit IRC | 22:03 | |
| *** mvenesio has quit IRC | 22:06 | |
| openstackgerrit | Merged openstack/cinder master: Storwize: fix an incorrect temporary parameter name https://review.openstack.org/534665 | 22:09 |
| *** abishop has quit IRC | 22:09 | |
| openstackgerrit | Merged openstack/cinder master: Update Volume Encryption Documentation https://review.openstack.org/535850 | 22:09 |
| *** mvk has joined #openstack-cinder | 22:15 | |
| *** openstackgerrit has quit IRC | 22:18 | |
| *** rcernin has joined #openstack-cinder | 22:19 | |
| *** lpetrut has quit IRC | 22:24 | |
| *** lpetrut has joined #openstack-cinder | 22:25 | |
| *** markstur has quit IRC | 22:28 | |
| *** lpetrut has quit IRC | 22:30 | |
| *** lpetrut has joined #openstack-cinder | 22:30 | |
| *** markstu__ has joined #openstack-cinder | 22:37 | |
| *** openstackgerrit has joined #openstack-cinder | 22:38 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: RBD: Support encrypted volumes https://review.openstack.org/534811 | 22:38 |
| *** Cibo has quit IRC | 22:40 | |
| *** Cibo has joined #openstack-cinder | 22:41 | |
| *** markstu__ has quit IRC | 22:41 | |
| *** lpetrut has quit IRC | 22:47 | |
| *** dikonoor has joined #openstack-cinder | 22:53 | |
| *** dustins has quit IRC | 23:02 | |
| *** markstur has joined #openstack-cinder | 23:03 | |
| *** liverpooler has quit IRC | 23:04 | |
| *** eharney has quit IRC | 23:09 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 23:18 | |
| *** AlexeyAbashkin has quit IRC | 23:23 | |
| *** wiggin15 has quit IRC | 23:26 | |
| *** enriquetaso has quit IRC | 23:26 | |
| *** jungleboyj has quit IRC | 23:26 | |
| *** tpsilva has quit IRC | 23:26 | |
| *** apopovych has quit IRC | 23:26 | |
| *** wiggin15 has joined #openstack-cinder | 23:26 | |
| *** enriquetaso has joined #openstack-cinder | 23:26 | |
| *** tpsilva has joined #openstack-cinder | 23:26 | |
| *** zeestrat has quit IRC | 23:26 | |
| *** apopovych has joined #openstack-cinder | 23:26 | |
| *** jungleboyj has joined #openstack-cinder | 23:26 | |
| *** DuncanT has quit IRC | 23:26 | |
| *** niedbalski has quit IRC | 23:26 | |
| *** icey has quit IRC | 23:26 | |
| *** catmando has quit IRC | 23:26 | |
| *** zeestrat has joined #openstack-cinder | 23:26 | |
| *** rajinir has quit IRC | 23:26 | |
| *** catmando has joined #openstack-cinder | 23:27 | |
| *** icey has joined #openstack-cinder | 23:27 | |
| *** rajinir has joined #openstack-cinder | 23:27 | |
| *** lennyb has quit IRC | 23:27 | |
| *** DuncanT has joined #openstack-cinder | 23:27 | |
| *** thingee has quit IRC | 23:28 | |
| *** lennyb has joined #openstack-cinder | 23:28 | |
| *** niedbalski has joined #openstack-cinder | 23:29 | |
| *** markstur_ has joined #openstack-cinder | 23:30 | |
| *** markstur has quit IRC | 23:33 | |
| *** _ix has quit IRC | 23:35 | |
| *** ganso has quit IRC | 23:38 | |
| *** liverpooler has joined #openstack-cinder | 23:55 | |
| *** thingee has joined #openstack-cinder | 23:55 | |
| *** openstack has quit IRC | 23:55 | |
| *** openstack has joined #openstack-cinder | 23:59 | |
| *** ChanServ sets mode: +o openstack | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!