| ildikov | That would make jgriffith's life easier as well | 00:01 |
|---|---|---|
| *** armax has joined #openstack-cinder | 00:02 | |
| ildikov | Just one patch to review and total freedom after that :) | 00:02 |
| smcginnis | Easy peasy. | 00:02 |
| ildikov | ;) | 00:02 |
| ildikov | Just sayin' :) | 00:03 |
| *** itlinux has quit IRC | 00:07 | |
| *** ntpttr_laptop has quit IRC | 00:14 | |
| *** armax has quit IRC | 00:19 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder-specs master: Remove unit testing https://review.openstack.org/520823 | 00:21 |
| *** sticker has joined #openstack-cinder | 00:25 | |
| *** sdague has quit IRC | 00:25 | |
| *** wanghao has joined #openstack-cinder | 00:26 | |
| *** gouthamr has quit IRC | 00:31 | |
| *** markvoelker has quit IRC | 00:31 | |
| *** armax has joined #openstack-cinder | 00:33 | |
| *** zhurong has joined #openstack-cinder | 00:34 | |
| *** yongwc has joined #openstack-cinder | 00:34 | |
| *** armax_ has joined #openstack-cinder | 00:34 | |
| *** yangyapeng has quit IRC | 00:37 | |
| *** armax has quit IRC | 00:38 | |
| *** armax_ is now known as armax | 00:38 | |
| *** markvoelker has joined #openstack-cinder | 00:43 | |
| *** armax has quit IRC | 00:46 | |
| *** gmann_afk has joined #openstack-cinder | 00:48 | |
| gmann_afk | jungleboyj: smcginnis one quick query. is it intensional not to have policy enforcement on GET /os-host https://bugs.launchpad.net/cinder/+bug/1732808 | 00:49 |
| openstack | Launchpad bug 1732808 in Cinder "no policy enforment for GET /os-hosts" [Undecided,New] | 00:49 |
| jungleboyj | gmann_afk: Looking | 00:50 |
| *** gmann_afk is now known as gmann | 00:50 | |
| jungleboyj | gmann_afk: I think that is a valid bug. Let me look at something here. | 00:50 |
| jungleboyj | gmann: I think the documentation is right there. Anything that gives insight about the underlying hosts/backends/etc we limit to the administrator. | 00:52 |
| *** wangyong has joined #openstack-cinder | 00:53 | |
| gmann | jungleboyj: yea, those rules are admin only be default. | 00:54 |
| jungleboyj | gmann: Updated the bug. | 00:54 |
| gmann | jungleboyj: you do check admin thing here - https://github.com/openstack/cinder/blob/0cf910d4345c000e8c306b1cb2b2dd291975cf71/cinder/api/contrib/hosts.py#L149 | 00:55 |
| *** yongwc has quit IRC | 00:56 | |
| jungleboyj | gmann: Yeah, it is here too: https://github.com/openstack/cinder/blob/10a3f4e1c6f21effc79fd309628111c221543e0d/cinder/policies/hosts.py | 00:56 |
| gmann | jungleboyj: my concern is whether it should have policy control instead of checking admin from context like done in PUT os-host | 00:56 |
| jungleboyj | So, what is the bug? | 00:56 |
| gmann | bug is not to check the policy enforcement in SHOW method | 00:56 |
| gmann | instead PUT check the policy | 00:57 |
| *** liverpooler has joined #openstack-cinder | 00:57 | |
| gmann | this is just doc but it does not enforced in code - https://github.com/openstack/cinder/blob/10a3f4e1c6f21effc79fd309628111c221543e0d/cinder/policies/hosts.py | 00:58 |
| jungleboyj | gmann: Yeah, so it should be enforced for both. | 00:59 |
| gmann | yea, i also think so, i doubted it was missed in code | 01:00 |
| *** Apoorva has joined #openstack-cinder | 01:01 | |
| *** wanghao_ has joined #openstack-cinder | 01:04 | |
| *** wanghao has quit IRC | 01:04 | |
| *** wanghao has joined #openstack-cinder | 01:05 | |
| wangyong | Can any one please do a review for https://review.openstack.org/#/c/502455/. Which is a new driver for Inspur storage. Need another +2. | 01:05 |
| *** Apoorva has quit IRC | 01:06 | |
| *** wanghao_ has quit IRC | 01:08 | |
| *** Apoorva has joined #openstack-cinder | 01:08 | |
| jungleboyj | gmann: Looks like it is. Good catch. | 01:09 |
| *** gouthamr has joined #openstack-cinder | 01:10 | |
| *** cheneydc has joined #openstack-cinder | 01:17 | |
| *** ganso has quit IRC | 01:28 | |
| *** wanghao_ has joined #openstack-cinder | 01:28 | |
| *** wanghao has quit IRC | 01:31 | |
| *** zhurong has quit IRC | 01:34 | |
| *** Apoorva_ has joined #openstack-cinder | 01:57 | |
| *** Apoorva has quit IRC | 02:00 | |
| *** Apoorva_ has quit IRC | 02:01 | |
| *** salv-orlando has joined #openstack-cinder | 02:02 | |
| *** salv-orl_ has quit IRC | 02:05 | |
| tommylikehu | jungleboyj: do you mind to take a look at this one: https://review.openstack.org/#/c/515577/ | 02:12 |
| *** armax has joined #openstack-cinder | 02:13 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Check available capacity before creating resources https://review.openstack.org/509011 | 02:14 |
| jungleboyj | Yeah, I will take a look. | 02:14 |
| jungleboyj | Eating dinner now though. :-) | 02:14 |
| tommylikehu | jungleboyj: lol, sure! | 02:21 |
| *** yangyapeng has joined #openstack-cinder | 02:24 | |
| *** vivsoni has joined #openstack-cinder | 02:36 | |
| *** dalgaaf has quit IRC | 02:39 | |
| *** dalgaaf has joined #openstack-cinder | 02:39 | |
| *** lhx_ has joined #openstack-cinder | 02:40 | |
| *** namnh has joined #openstack-cinder | 02:45 | |
| *** vivsoni has quit IRC | 02:50 | |
| *** vivsoni_ has joined #openstack-cinder | 02:50 | |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for restoring backup are wrongly described https://review.openstack.org/520375 | 02:52 |
| *** itlinux has joined #openstack-cinder | 02:53 | |
| *** armax has quit IRC | 02:55 | |
| *** armax has joined #openstack-cinder | 02:59 | |
| *** salv-orl_ has joined #openstack-cinder | 03:00 | |
| *** dhinesh has quit IRC | 03:01 | |
| *** salv-orlando has quit IRC | 03:02 | |
| *** links has joined #openstack-cinder | 03:03 | |
| *** KeithMnemonic has quit IRC | 03:09 | |
| *** akekane has joined #openstack-cinder | 03:13 | |
| *** akekane is now known as abhishekk | 03:14 | |
| *** wanghao has joined #openstack-cinder | 03:21 | |
| *** wanghao_ has quit IRC | 03:25 | |
| *** psachin has joined #openstack-cinder | 03:31 | |
| *** itlinux has quit IRC | 03:36 | |
| *** itlinux has joined #openstack-cinder | 03:45 | |
| *** eharney has quit IRC | 03:49 | |
| *** eharney has joined #openstack-cinder | 03:50 | |
| *** vivsoni_ has quit IRC | 04:04 | |
| *** vivsoni has joined #openstack-cinder | 04:04 | |
| *** eharney has quit IRC | 04:09 | |
| *** vivsoni has quit IRC | 04:12 | |
| *** dave-mccowan has quit IRC | 04:19 | |
| *** itlinux has quit IRC | 04:24 | |
| *** itlinux has joined #openstack-cinder | 04:33 | |
| *** itlinux has quit IRC | 04:33 | |
| jungleboyj | tommylikehu: Done. | 04:35 |
| *** cheneydc has quit IRC | 04:39 | |
| *** sticker has quit IRC | 04:46 | |
| *** itlinux has joined #openstack-cinder | 04:49 | |
| *** cheneydc has joined #openstack-cinder | 04:59 | |
| *** pgadiya has joined #openstack-cinder | 05:05 | |
| *** wN has quit IRC | 05:13 | |
| *** wN has joined #openstack-cinder | 05:19 | |
| *** wN has quit IRC | 05:19 | |
| *** wN has joined #openstack-cinder | 05:19 | |
| *** armax has quit IRC | 05:23 | |
| *** yangyapeng has quit IRC | 05:24 | |
| *** vivsoni has joined #openstack-cinder | 05:30 | |
| vivsoni | Hi Team, please review | 05:30 |
| vivsoni | https://review.openstack.org/#/c/519979/ | 05:30 |
| vivsoni | its a cherry picked to stable/pike | 05:31 |
| *** gouthamr has quit IRC | 05:34 | |
| *** itlinux has quit IRC | 05:35 | |
| *** itlinux has joined #openstack-cinder | 05:38 | |
| *** zhonghua2 has quit IRC | 05:45 | |
| *** zhonghua2 has joined #openstack-cinder | 05:45 | |
| *** itlinux has quit IRC | 05:51 | |
| *** yangyapeng has joined #openstack-cinder | 05:52 | |
| *** wanghao_ has joined #openstack-cinder | 05:55 | |
| *** yangyapeng has quit IRC | 05:56 | |
| *** wanghao has quit IRC | 05:59 | |
| *** psachin_ has joined #openstack-cinder | 06:00 | |
| *** psachin has quit IRC | 06:01 | |
| *** wes_dillingham has quit IRC | 06:04 | |
| openstackgerrit | Merged openstack/cinder master: Fix earlier backup records can't be restored https://review.openstack.org/515577 | 06:06 |
| *** wanghao_ has quit IRC | 06:09 | |
| *** yangyapeng has joined #openstack-cinder | 06:10 | |
| *** yangyapeng has quit IRC | 06:15 | |
| *** zhonghua2 is now known as freesky-edward | 06:15 | |
| *** lpetrut_ has joined #openstack-cinder | 06:19 | |
| *** armaan has joined #openstack-cinder | 06:29 | |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for restoring backup are wrongly described https://review.openstack.org/520375 | 06:34 |
| *** yangyapeng has joined #openstack-cinder | 06:36 | |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for restoring backup are wrongly described https://review.openstack.org/520375 | 06:40 |
| *** zhurong has joined #openstack-cinder | 06:40 | |
| *** yangyapeng has quit IRC | 06:41 | |
| *** maestropandy has joined #openstack-cinder | 06:42 | |
| *** belmoreira has joined #openstack-cinder | 06:43 | |
| *** maestropandy has quit IRC | 06:45 | |
| *** yangyapeng has joined #openstack-cinder | 06:51 | |
| *** markvoelker has quit IRC | 06:59 | |
| *** armaan has quit IRC | 07:01 | |
| *** armaan has joined #openstack-cinder | 07:03 | |
| *** pcaruana has joined #openstack-cinder | 07:04 | |
| *** armaan_ has joined #openstack-cinder | 07:06 | |
| *** markstur has joined #openstack-cinder | 07:07 | |
| *** armaan has quit IRC | 07:09 | |
| *** markstur has quit IRC | 07:11 | |
| *** maestropandy has joined #openstack-cinder | 07:16 | |
| *** rcernin has quit IRC | 07:18 | |
| *** psachin_ has quit IRC | 07:20 | |
| *** namnh has quit IRC | 07:27 | |
| *** lpetrut_ has quit IRC | 07:31 | |
| *** psachin has joined #openstack-cinder | 07:33 | |
| *** maestropandy has quit IRC | 07:34 | |
| *** psachin has quit IRC | 07:38 | |
| *** lpetrut_ has joined #openstack-cinder | 07:39 | |
| *** maestropandy has joined #openstack-cinder | 07:39 | |
| *** psachin has joined #openstack-cinder | 07:39 | |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: V3 jsonschema validation: base schema framework https://review.openstack.org/515649 | 07:40 |
| openstackgerrit | Neha Alhat proposed openstack/cinder master: V3 jsonschema validation: snapshots https://review.openstack.org/520991 | 07:40 |
| *** zhurong has quit IRC | 07:43 | |
| *** wangyong has quit IRC | 07:47 | |
| *** markvoelker has joined #openstack-cinder | 08:00 | |
| *** AlexeyAbashkin has joined #openstack-cinder | 08:01 | |
| openstackgerrit | Merged openstack/cinder master: NetApp cDot: Fix manage volumes https://review.openstack.org/519716 | 08:09 |
| *** pgadiya has quit IRC | 08:10 | |
| *** maestropandy has quit IRC | 08:11 | |
| *** lpetrut_ has quit IRC | 08:11 | |
| *** markstur has joined #openstack-cinder | 08:14 | |
| *** markstur has quit IRC | 08:18 | |
| *** wanghao has joined #openstack-cinder | 08:18 | |
| *** alexchadin has joined #openstack-cinder | 08:24 | |
| *** armaan has joined #openstack-cinder | 08:27 | |
| *** amoralej|off is now known as amoralej | 08:27 | |
| *** jmlowe has quit IRC | 08:27 | |
| *** pgadiya has joined #openstack-cinder | 08:27 | |
| *** armaan_ has quit IRC | 08:30 | |
| *** vivsoni has quit IRC | 08:36 | |
| *** pgadiya has quit IRC | 08:40 | |
| *** lpetrut_ has joined #openstack-cinder | 08:43 | |
| *** vivsoni has joined #openstack-cinder | 08:45 | |
| *** belmoreira has quit IRC | 08:47 | |
| *** lpetrut_ has quit IRC | 08:50 | |
| *** lpetrut_ has joined #openstack-cinder | 08:50 | |
| *** sdatko has joined #openstack-cinder | 08:51 | |
| *** pgadiya has joined #openstack-cinder | 08:52 | |
| *** gaoyanami has joined #openstack-cinder | 08:52 | |
| *** maestropandy has joined #openstack-cinder | 08:53 | |
| *** lpetrut_ has quit IRC | 08:57 | |
| *** alexchadin has quit IRC | 09:05 | |
| *** alexchadin has joined #openstack-cinder | 09:05 | |
| *** e0ne has joined #openstack-cinder | 09:29 | |
| *** gkadam has joined #openstack-cinder | 09:32 | |
| *** wanghao has quit IRC | 09:42 | |
| openstackgerrit | Xiaojun Liao proposed openstack/cinder master: Fix v3 api-ref for restoring backup are wrongly described https://review.openstack.org/520375 | 09:44 |
| *** cheneydc has quit IRC | 09:44 | |
| *** alexchadin has quit IRC | 09:46 | |
| *** alexchadin has joined #openstack-cinder | 09:47 | |
| *** ganso has joined #openstack-cinder | 09:53 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Handle quota in 2 phases when managing resource. https://review.openstack.org/521044 | 09:53 |
| *** maestropandy has quit IRC | 09:57 | |
| *** alexchadin has quit IRC | 09:57 | |
| *** alexchadin has joined #openstack-cinder | 09:58 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Handle quota in 2 phases when managing resource. https://review.openstack.org/521044 | 09:59 |
| openstackgerrit | TommyLike proposed openstack/cinder master: Handle quota in 2 phases when managing resource. https://review.openstack.org/521044 | 10:08 |
| *** sapd__ has quit IRC | 10:09 | |
| *** sapd__ has joined #openstack-cinder | 10:14 | |
| *** markstur has joined #openstack-cinder | 10:14 | |
| *** maestropandy has joined #openstack-cinder | 10:18 | |
| *** markstur has quit IRC | 10:19 | |
| *** wanghao has joined #openstack-cinder | 10:27 | |
| *** lpetrut_ has joined #openstack-cinder | 10:30 | |
| openstackgerrit | Pooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: base schema framework https://review.openstack.org/515649 | 10:31 |
| *** gmann is now known as gmann_afk | 10:37 | |
| *** abhishekk has quit IRC | 11:00 | |
| *** alexchadin has quit IRC | 11:04 | |
| *** alexchadin has joined #openstack-cinder | 11:05 | |
| *** pgadiya has quit IRC | 11:09 | |
| *** pgadiya has joined #openstack-cinder | 11:10 | |
| *** lpetrut_ has quit IRC | 11:14 | |
| *** lpetrut has joined #openstack-cinder | 11:15 | |
| *** alexchadin has quit IRC | 11:17 | |
| *** maestropandy has quit IRC | 11:20 | |
| *** e0ne has quit IRC | 11:26 | |
| *** e0ne has joined #openstack-cinder | 11:26 | |
| *** pgadiya has quit IRC | 11:36 | |
| *** armaan has quit IRC | 11:39 | |
| *** mishino has joined #openstack-cinder | 11:41 | |
| *** e0ne has quit IRC | 11:41 | |
| *** e0ne has joined #openstack-cinder | 11:44 | |
| openstackgerrit | Hanxi Liu proposed openstack/cinder master: Enginefacde: update "group_type" and "volume_type" https://review.openstack.org/521067 | 11:58 |
| pooja_jadhav | smcginnis: Hi, are u there? | 12:00 |
| *** sdague has joined #openstack-cinder | 12:09 | |
| *** lhx_ has quit IRC | 12:21 | |
| *** lhx_ has joined #openstack-cinder | 12:21 | |
| *** zhaochao has quit IRC | 12:22 | |
| *** abishop has joined #openstack-cinder | 12:36 | |
| *** yangyapeng has quit IRC | 12:45 | |
| *** yangyapeng has joined #openstack-cinder | 12:46 | |
| *** gcb has quit IRC | 12:47 | |
| *** yangyapeng has quit IRC | 12:48 | |
| *** yangyapeng has joined #openstack-cinder | 12:49 | |
| *** yangyapeng has quit IRC | 12:53 | |
| *** kwathore has joined #openstack-cinder | 12:54 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Fix error handling and checks for generic volume groups https://review.openstack.org/521080 | 12:58 |
| *** psachin has quit IRC | 13:01 | |
| *** lhx_ has quit IRC | 13:13 | |
| *** lhx_ has joined #openstack-cinder | 13:13 | |
| *** lhx_ has quit IRC | 13:14 | |
| *** edmondsw has joined #openstack-cinder | 13:17 | |
| *** links has quit IRC | 13:19 | |
| *** wes_dillingham has joined #openstack-cinder | 13:23 | |
| *** markvoelker has quit IRC | 13:34 | |
| *** markvoelker has joined #openstack-cinder | 13:34 | |
| *** sbezverk has quit IRC | 13:35 | |
| *** sbezverk has joined #openstack-cinder | 13:36 | |
| *** dklyle has quit IRC | 13:48 | |
| *** dustins has joined #openstack-cinder | 14:04 | |
| *** maestropandy has joined #openstack-cinder | 14:07 | |
| *** maestropandy has quit IRC | 14:07 | |
| *** gouthamr has joined #openstack-cinder | 14:11 | |
| *** mriedem has joined #openstack-cinder | 14:18 | |
| *** mishino has quit IRC | 14:20 | |
| *** dave-mcc_ has joined #openstack-cinder | 14:24 | |
| *** lhx_ has joined #openstack-cinder | 14:30 | |
| *** netapp has joined #openstack-cinder | 14:34 | |
| dims | smcginnis : jungleboyj : can i get some reviews on a cinder related PR in k8s please? https://github.com/kubernetes/kubernetes/pull/55944 | 14:46 |
| smcginnis | dims: Will do! | 14:48 |
| smcginnis | pooja_jadhav: Here now. Still looking for me? | 14:49 |
| *** laurent\ has quit IRC | 14:53 | |
| *** psachin has joined #openstack-cinder | 14:59 | |
| *** KeithMnemonic has joined #openstack-cinder | 15:02 | |
| *** laurent\ has joined #openstack-cinder | 15:04 | |
| *** adrianofr has quit IRC | 15:06 | |
| *** Apoorva has joined #openstack-cinder | 15:08 | |
| *** eharney has joined #openstack-cinder | 15:08 | |
| jungleboyj | dims: Looks ok, think doing a backoff on the timeout isn't a bad idea though. | 15:10 |
| *** armax has joined #openstack-cinder | 15:11 | |
| *** e0ne_ has joined #openstack-cinder | 15:14 | |
| *** kwathore has quit IRC | 15:16 | |
| *** e0ne has quit IRC | 15:17 | |
| *** Apoorva has quit IRC | 15:21 | |
| *** kwathore has joined #openstack-cinder | 15:23 | |
| *** david-lyle has joined #openstack-cinder | 15:24 | |
| *** kwathore has quit IRC | 15:29 | |
| *** wanghao_ has joined #openstack-cinder | 15:30 | |
| *** itlinux has joined #openstack-cinder | 15:31 | |
| *** wanghao has quit IRC | 15:32 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add service_uuid FK to volumes https://review.openstack.org/519025 | 15:36 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add shared_targets flag to Volumes https://review.openstack.org/520676 | 15:39 |
| *** gaoyanami has quit IRC | 15:40 | |
| *** itlinux has quit IRC | 15:42 | |
| *** itlinux has joined #openstack-cinder | 15:45 | |
| *** markstur has joined #openstack-cinder | 15:45 | |
| *** crose has joined #openstack-cinder | 15:58 | |
| *** david-lyle has quit IRC | 16:02 | |
| *** david-lyle has joined #openstack-cinder | 16:03 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: Vzstorage: fix for create_cloned_volume https://review.openstack.org/508569 | 16:04 |
| *** AlexeyAbashkin has quit IRC | 16:05 | |
| cfriesen | anyone know what's up with the openstack-tox-py35 gate failures on https://review.openstack.org/#/c/520686/ ? | 16:12 |
| jungleboyj | cfriesen: Yeah, smcginnis has been working on trying to fix that for a while now. | 16:18 |
| jungleboyj | Not totally sure what is going on. Something weird coming out of a test case or two that causes it to puke. | 16:19 |
| jungleboyj | Long story short. It is being worked. | 16:21 |
| eharney | smcginnis: why does https://review.openstack.org/#/c/520823/ remove all of the "code-block" sections? are those not rst files now? | 16:22 |
| eharney | oh, in some files it removes them and some it adds them, hmm | 16:23 |
| jungleboyj | eharney: I was going to say, I didn't think it removed them all. | 16:23 |
| eharney | well, why is it removing any of them, then? | 16:23 |
| jungleboyj | Like where? I am clicking through here. | 16:25 |
| eharney | maybe it's only adding them and i just can't read today | 16:26 |
| jungleboyj | :-) I didn't think there was removal. | 16:26 |
| openstackgerrit | Merged openstack/cinder-specs master: Remove unit testing https://review.openstack.org/520823 | 16:26 |
| eharney | well, that's good then, never mind me | 16:27 |
| smcginnis | eharney: Yeah, is there a specific place? I don't think I removed any. | 16:29 |
| eharney | smcginnis: just in my imagination apparently, after reading through again | 16:29 |
| smcginnis | Switched some "::" to code-blocks where the formatting was weird. | 16:29 |
| smcginnis | eharney: Hah, fair enough. :) | 16:30 |
| jungleboyj | eharney: Can you share whatever you are smoking please? | 16:35 |
| eharney | jungleboyj: it's just stacks of printed out Cinder code | 16:36 |
| jungleboyj | eharney: Never mind. | 16:36 |
| smcginnis | Hash | 16:36 |
| smcginnis | Cinder hash | 16:36 |
| jungleboyj | He he ... has tables. | 16:37 |
| *** dhinesh has joined #openstack-cinder | 16:38 | |
| *** jistr has quit IRC | 16:39 | |
| *** gkadam has quit IRC | 16:41 | |
| *** gkadam has joined #openstack-cinder | 16:41 | |
| *** jistr has joined #openstack-cinder | 16:43 | |
| jungleboyj | *hash tables ... jeez, can't even make a joke today. | 16:48 |
| jungleboyj | @!t | 16:48 |
| _pewp_ | jungleboyj (╯°□°)╯︵ ┻━┻ | 16:48 |
| Swanson | I somehow don't think I should read a 400 page tax bill while I'm at work. Having a hard time not doing so, however. | 16:50 |
| *** Apoorva has joined #openstack-cinder | 16:51 | |
| jungleboyj | Swanson: Good lord. Really? | 16:51 |
| Swanson | jungleboyj, it is a big bill. Not that much stuff per page, however. | 16:51 |
| Swanson | They could use a better font. | 16:52 |
| *** psachin has quit IRC | 17:02 | |
| *** gkadam has quit IRC | 17:08 | |
| *** hemna_ has quit IRC | 17:23 | |
| *** hemna_ has joined #openstack-cinder | 17:23 | |
| *** adrianofr has joined #openstack-cinder | 17:28 | |
| adrianofr | Hi, guys. Can anyone pls take a look in this patch? https://review.openstack.org/#/c/518406/ | 17:34 |
| adrianofr | It's really small :) | 17:34 |
| *** Apoorva has quit IRC | 17:34 | |
| *** sri__ has quit IRC | 17:41 | |
| *** akekane has joined #openstack-cinder | 17:41 | |
| *** akekane is now known as abhishekk | 17:42 | |
| *** dhinesh has quit IRC | 17:43 | |
| *** salv-orlando has joined #openstack-cinder | 17:46 | |
| *** salv-orl_ has quit IRC | 17:46 | |
| *** e0ne_ has quit IRC | 17:49 | |
| *** ntpttr_laptop has joined #openstack-cinder | 17:50 | |
| smcginnis | adrianofr: No CI reporting on there yet. We need that. | 17:51 |
| *** lhx_ has quit IRC | 17:54 | |
| jungleboyj | adrianofr: I have it my +2, will let Sean +W when the 3rd party CI reports. | 17:54 |
| *** abhishekk has quit IRC | 18:07 | |
| ildikov | jgriffith: hey, I only have one quick question for this beautiful Saturday, oooops, Friday | 18:11 |
| ildikov | jgriffith: are you still up to writing up the multi-attach spec for Cinder? | 18:11 |
| ildikov | jgriffith: would be nice to capture those few things we will need, especially the policy bits, which I'm not an expert of | 18:11 |
| *** Apoorva has joined #openstack-cinder | 18:17 | |
| *** Apoorva has quit IRC | 18:17 | |
| *** Apoorva has joined #openstack-cinder | 18:18 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add service_uuid FK to volumes https://review.openstack.org/519025 | 18:20 |
| *** dustins has quit IRC | 18:23 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add shared_targets flag to Volumes https://review.openstack.org/520676 | 18:23 |
| adrianofr | @smcginnis Oh! I didn't realize that. I'll take care of it. Thanks! | 18:28 |
| adrianofr | @jungleboyj Thanks! | 18:28 |
| jungleboyj | ildikov: I am jealous of a beautiful Saturday. Haven't seen the sun since I left Sydney. | 18:38 |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: VNX: Fix cloning failure when system is busy https://review.openstack.org/520006 | 18:41 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: manageable volumes https://review.openstack.org/521158 | 18:49 |
| ildikov | jungleboyj: sadness :/ | 18:52 |
| jungleboyj | ildikov: c'est le vie Hope the time in NZ is awesome! | 18:53 |
| *** e0ne has joined #openstack-cinder | 19:03 | |
| *** exarr has joined #openstack-cinder | 19:09 | |
| *** dustins has joined #openstack-cinder | 19:09 | |
| exarr | Hi all. Ubuntu16.04/ocata cinder with an lvm backend on the local machine. Can't launch an instance with a 80G+ volume atached. | 19:09 |
| *** lpetrut has quit IRC | 19:09 | |
| exarr | I believe there's a patch out to fix/allow this to happen. | 19:10 |
| exarr | Anyone able to point me in the right direction? :-/ | 19:10 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Explicitly set expected log level for tests https://review.openstack.org/521165 | 19:14 |
| smcginnis | jgriffith: ^^ | 19:14 |
| *** e0ne has quit IRC | 19:15 | |
| smcginnis | exarr: What is the failure? | 19:15 |
| *** dhinesh has joined #openstack-cinder | 19:16 | |
| exarr | smcginnis: I can create volumes perfectly fine, and spawn new instances when the volume is <80G | 19:24 |
| exarr | But anoy more than that, and it says cannot find a node. | 19:24 |
| exarr | s/anoy/any | 19:24 |
| exarr | I believe it to be a well-known problem when using lvm as block storage I believe. Had hoped it had been addressed. | 19:25 |
| dhinesh | for testing cinder on the third party CI, is this https://github.com/openstack-infra/project-config-example/blob/master/jenkins/jobs/dsvm-cinder-driver.yaml test enough to be merged for the queens release? | 19:27 |
| smcginnis | exarr: So the LVM node is separate from your compute node, the compute node has 80G of storage, but when trying to create a volume booting from an 80G volume it filters out your compute node. Is that what you're saying? | 19:30 |
| *** itlinux has quit IRC | 19:31 | |
| smcginnis | dhinesh: I haven't used that, but based on DEVSTACK_GATE_TEMPEST_REGEX="volume", that looks correct. | 19:31 |
| *** itlinux has joined #openstack-cinder | 19:33 | |
| dhinesh | smcginnis: ok, have you used other alternatives then? | 19:33 |
| *** amoralej is now known as amoralej|off | 19:34 | |
| smcginnis | dhinesh: It's been awhile since I had to run a CI, but when I did, I just had scripts that set up devstack and ran "tox -e all_plugin volume". | 19:34 |
| exarr | smcginnis: Not quite. I have a single compute node, that runs compute/neutron/cinder as an AIO config (almost, separate horizon/keystone) | 19:34 |
| smcginnis | exarr: Are you using LVM thin provisioning? | 19:34 |
| exarr | smcginnis: Yes. | 19:35 |
| *** gouthamr has quit IRC | 19:35 | |
| smcginnis | exarr: Probably the same thing though. There is/was a Nova bug that would filter out by local compute node disk space by counting external (Cinder) used storage against the nova-compute space. | 19:36 |
| exarr | Even when launching /from/ a volume? | 19:37 |
| dhinesh | smcginnis: in that case should the devstack node also have the vendor specific cinder installed on it, even before it is merged? | 19:37 |
| smcginnis | dhinesh: Vendor specific configuration, yes. It needs to be set up to use your driver with your actual backend. | 19:38 |
| exarr | smcginnis: I have a 1TB vg, would you expect me to be able to spawn an instance with say 100G root disk? :-/ | 19:40 |
| smcginnis | exarr: Yes, but if it's filtering out that compute node because it wants to have 100G of local free space, it isn't going to matter. | 19:42 |
| dhinesh | smcginnis: ok thanks | 19:43 |
| smcginnis | dhinesh: No problem. | 19:44 |
| exarr | smcginnis: So, you're suggesting that because / has <80G free, it will not allocate an 80G+ lvs partition to an instance? | 19:48 |
| exarr | (had a few beers, bit slow) | 19:48 |
| exarr | ... that would correlate. | 19:50 |
| smcginnis | exarr: Yeah, that was the bug IIRC. | 19:50 |
| *** itlinux has quit IRC | 19:50 | |
| exarr | smcginnis: Has that been fixed for ocata? If this is the problem, then I'd be much bloomin happier :-D | 19:51 |
| smcginnis | exarr: I'm not sure. I know there were multiple attempts to fix it, but they were all shot down for some reason. | 19:51 |
| exarr | eek. | 19:51 |
| exarr | okay. I'll go and chop up some lvms. Thanks!! | 19:52 |
| smcginnis | exarr: You could ask in #openstack-nova. I know there were a few that should be familiar with it. | 19:53 |
| *** itlinux has joined #openstack-cinder | 19:54 | |
| exarr | thankyou!! | 19:57 |
| smcginnis | exarr: Good luck! | 19:57 |
| smcginnis | exarr: I do know some other operators used some of the proposed patches to get around it, so there are at least unmerged patches you might be able to get by with. | 19:57 |
| cfriesen | exarr: smcginnis: I think https://bugs.launchpad.net/nova/+bug/1469179 is what you're looking for. | 20:01 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt) | 20:01 |
| *** Apoorva has quit IRC | 20:01 | |
| smcginnis | cfriesen: Yep, that's the one! | 20:01 |
| smcginnis | exarr: Oh, last comment there: This has been fixed in newer versions with the resource providers and the use of the placement API. | 20:02 |
| *** lpetrut has joined #openstack-cinder | 20:06 | |
| *** vivsoni has quit IRC | 20:09 | |
| *** vivsoni has joined #openstack-cinder | 20:09 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder master: nimble: handle unicode strings in volume create https://review.openstack.org/521185 | 20:10 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: manageable volumes https://review.openstack.org/521158 | 20:21 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Update oslo.context RequestContext args https://review.openstack.org/517712 | 20:24 |
| abishop | hey folks, https://review.openstack.org/519432 looks ready for review | 20:32 |
| *** markstur has quit IRC | 20:42 | |
| *** itlinux has quit IRC | 20:43 | |
| *** eharney has quit IRC | 20:51 | |
| *** jmlowe has joined #openstack-cinder | 20:55 | |
| *** Apoorva has joined #openstack-cinder | 21:01 | |
| *** jmlowe has quit IRC | 21:01 | |
| *** wes_dillingham has quit IRC | 21:03 | |
| *** vivsoni has quit IRC | 21:13 | |
| *** vivsoni has joined #openstack-cinder | 21:14 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove ExceptionTestCase https://review.openstack.org/521192 | 21:17 |
| *** AlexeyAbashkin has joined #openstack-cinder | 21:19 | |
| *** ganso has quit IRC | 21:20 | |
| *** crose has quit IRC | 21:22 | |
| *** AlexeyAbashkin has quit IRC | 21:24 | |
| smcginnis | abishop: Do you happen to know anything about these "Could not load" message we get? http://paste.openstack.org/show/626658/ | 21:24 |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: Update api-ref to include volume_image_metadata https://review.openstack.org/520686 | 21:24 |
| abishop | smcginnis: yeah, it's coming from Castellan. One sec and I'll locate the details | 21:25 |
| smcginnis | abishop: And any suggestions how to get rid of them? :) | 21:26 |
| abishop | smcginnis: I think it's triggered when the key_manager backend isn't a "stevedore" driver | 21:27 |
| abishop | see https://github.com/openstack/castellan/blob/master/castellan/key_manager/__init__.py#L40 | 21:27 |
| abishop | smcginnis: msg is quite annoying, and I can see what it takes to squelch | 21:28 |
| smcginnis | abishop: I suppose our base test could do something like logger.getLogger('castellan').setLevel(logging.ERROR) or something like that for now. | 21:29 |
| smcginnis | abishop: But is the long term solution to use some external driver? | 21:29 |
| *** Apoorva_ has joined #openstack-cinder | 21:30 | |
| abishop | smcginnis: dunno, I spotted this, but haven't really investigated | 21:30 |
| abishop | smcginnis: maybe mock the driver load function? | 21:31 |
| smcginnis | abishop: Oh, that's a good idea. We shouldn't actually need a driver loaded for the unit tests. | 21:31 |
| abishop | right ;-/ | 21:32 |
| smcginnis | abishop: Thanks! | 21:32 |
| abishop | thx for the review! | 21:32 |
| *** salv-orl_ has joined #openstack-cinder | 21:32 | |
| smcginnis | abishop: No problem, thanks for fixing that. | 21:33 |
| *** Apoorva has quit IRC | 21:33 | |
| abishop | smcginnis: switching to barbican will likely turn over a number of rocks like this | 21:34 |
| smcginnis | abishop: Oh joy! :) | 21:34 |
| *** salv-orlando has quit IRC | 21:35 | |
| abishop | smcginnis: I have another one in the works, and that is the way keys are managed in backups | 21:35 |
| smcginnis | abishop: Cool! | 21:36 |
| openstackgerrit | Louie Kwan proposed openstack/cinder master: Allow purging of records less than 1 day old https://review.openstack.org/511567 | 21:41 |
| *** eharney has joined #openstack-cinder | 21:42 | |
| openstackgerrit | Merged openstack/os-brick master: Adding NVMEoF for initiator CLI https://review.openstack.org/482642 | 21:54 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Hey Matt, check this out https://review.openstack.org/521201 | 21:59 |
| smcginnis | mtreinish: Got a repro case for you ^^ | 22:00 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Suppress castellan log messages https://review.openstack.org/521202 | 22:03 |
| *** dustins has quit IRC | 22:03 | |
| *** abishop has quit IRC | 22:09 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder-specs master: switch from oslosphinx to openstackdocstheme https://review.openstack.org/481989 | 22:16 |
| *** edmondsw has quit IRC | 22:19 | |
| *** edmondsw has joined #openstack-cinder | 22:19 | |
| *** edmondsw_ has joined #openstack-cinder | 22:23 | |
| *** wes_dillingham has joined #openstack-cinder | 22:23 | |
| *** edmondsw has quit IRC | 22:24 | |
| *** edmondsw_ has quit IRC | 22:27 | |
| *** hoonetorg has quit IRC | 22:28 | |
| *** adrianofr has quit IRC | 22:28 | |
| *** dave-mcc_ has quit IRC | 22:30 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Suppress castellan log messages https://review.openstack.org/521202 | 22:31 |
| *** wes_dillingham has quit IRC | 22:32 | |
| openstackgerrit | Merged openstack/cinder-specs master: switch from oslosphinx to openstackdocstheme https://review.openstack.org/481989 | 22:33 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder-specs master: Clean up required packages https://review.openstack.org/521206 | 22:54 |
| *** lpetrut has quit IRC | 22:57 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Explicitly set expected log level for tests https://review.openstack.org/521165 | 22:58 |
| *** ntpttr_laptop has quit IRC | 22:58 | |
| *** wes_dillingham has joined #openstack-cinder | 23:01 | |
| *** imacdonn has quit IRC | 23:09 | |
| *** imacdonn has joined #openstack-cinder | 23:10 | |
| *** KeithMnemonic has quit IRC | 23:13 | |
| *** exarr has quit IRC | 23:19 | |
| *** Apoorva_ has quit IRC | 23:27 | |
| *** Apoorva has joined #openstack-cinder | 23:28 | |
| *** gouthamr_ has joined #openstack-cinder | 23:36 | |
| *** sdague has quit IRC | 23:40 | |
| *** edmondsw has joined #openstack-cinder | 23:44 | |
| *** edmondsw has quit IRC | 23:48 | |
| *** pcaruana has quit IRC | 23:54 | |
| *** wes_dillingham has quit IRC | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!