| *** yangyapeng has joined #openstack-cinder | 00:09 | |
| *** jose-phillips has quit IRC | 00:09 | |
| *** jistr has quit IRC | 00:11 | |
| *** yangyapeng has quit IRC | 00:14 | |
| *** jose-phillips has joined #openstack-cinder | 00:15 | |
| *** zhurong has joined #openstack-cinder | 00:15 | |
| *** jistr has joined #openstack-cinder | 00:16 | |
| *** sandk has quit IRC | 00:17 | |
| *** sandk has joined #openstack-cinder | 00:19 | |
| *** yangyapeng has joined #openstack-cinder | 00:20 | |
| *** sdague has quit IRC | 00:22 | |
| *** sandk has quit IRC | 00:23 | |
| *** sandk has joined #openstack-cinder | 00:24 | |
| *** yangyapeng has quit IRC | 00:25 | |
| *** sandk has quit IRC | 00:25 | |
| *** chhavi__ has quit IRC | 00:26 | |
| *** EmilienM has quit IRC | 00:30 | |
| *** edmondsw has joined #openstack-cinder | 00:36 | |
| *** edmondsw has quit IRC | 00:41 | |
| *** dave-mccowan has joined #openstack-cinder | 00:48 | |
| *** jose-phillips has quit IRC | 01:00 | |
| *** jose-phillips has joined #openstack-cinder | 01:05 | |
| *** nicolasbock has joined #openstack-cinder | 01:17 | |
| openstackgerrit | Merged openstack/cinder master: Add microversion and volume status limit to revert api ref doc https://review.openstack.org/529799 | 01:29 |
|---|---|---|
| openstackgerrit | Merged openstack/cinder master: V2/V3 json schema validation: snapshot manage https://review.openstack.org/528631 | 01:29 |
| *** Apoorva_ has joined #openstack-cinder | 01:33 | |
| *** Apoorva has quit IRC | 01:37 | |
| *** Apoorva_ has quit IRC | 01:38 | |
| *** hemna_ has quit IRC | 01:39 | |
| *** hoangcx has quit IRC | 01:55 | |
| *** hoangcx has joined #openstack-cinder | 01:55 | |
| *** yangyapeng has joined #openstack-cinder | 02:11 | |
| *** threestrands_ has joined #openstack-cinder | 02:14 | |
| *** threestrands_ has joined #openstack-cinder | 02:14 | |
| *** eharney has quit IRC | 02:16 | |
| *** chenying_ has joined #openstack-cinder | 02:16 | |
| *** chenying has quit IRC | 02:16 | |
| *** threestrands has quit IRC | 02:16 | |
| *** zzzeek has quit IRC | 02:19 | |
| *** zzzeek has joined #openstack-cinder | 02:20 | |
| *** namnh has joined #openstack-cinder | 02:20 | |
| *** zzzeek has quit IRC | 02:20 | |
| *** zzzeek has joined #openstack-cinder | 02:21 | |
| *** markstur has joined #openstack-cinder | 02:24 | |
| *** edmondsw has joined #openstack-cinder | 02:24 | |
| *** Brin has joined #openstack-cinder | 02:26 | |
| *** edmondsw has quit IRC | 02:28 | |
| *** eharney has joined #openstack-cinder | 02:30 | |
| *** mriedem has quit IRC | 02:31 | |
| *** harlowja has quit IRC | 02:36 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 02:37 | |
| *** markstur has quit IRC | 02:50 | |
| *** imran_ansari has joined #openstack-cinder | 02:56 | |
| *** Dinesh_Bhor has quit IRC | 03:12 | |
| *** nicolasbock has quit IRC | 03:14 | |
| *** ChanServ sets mode: +o smcginnis | 03:16 | |
| *** imran_ansari has quit IRC | 03:28 | |
| *** links has joined #openstack-cinder | 03:29 | |
| *** Apoorva has joined #openstack-cinder | 03:30 | |
| *** markstur has joined #openstack-cinder | 03:31 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 03:32 | |
| *** links has quit IRC | 03:34 | |
| *** zhurong has quit IRC | 03:34 | |
| *** markstur_ has joined #openstack-cinder | 03:35 | |
| *** markstur has quit IRC | 03:38 | |
| *** eharney has quit IRC | 03:38 | |
| *** eharney has joined #openstack-cinder | 03:39 | |
| *** moshele has joined #openstack-cinder | 03:39 | |
| *** s-shiono has joined #openstack-cinder | 03:42 | |
| *** abhishekk has joined #openstack-cinder | 03:51 | |
| *** links has joined #openstack-cinder | 03:56 | |
| *** wanghao has joined #openstack-cinder | 04:01 | |
| *** wanghao has quit IRC | 04:01 | |
| *** wanghao has joined #openstack-cinder | 04:01 | |
| *** s-shiono_ has joined #openstack-cinder | 04:06 | |
| *** s-shiono has quit IRC | 04:09 | |
| *** moshele has quit IRC | 04:12 | |
| *** edmondsw has joined #openstack-cinder | 04:13 | |
| *** Dinesh_Bhor has quit IRC | 04:14 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 04:17 | |
| *** edmondsw has quit IRC | 04:17 | |
| *** psachin has joined #openstack-cinder | 04:33 | |
| *** eharney has quit IRC | 04:34 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: create snapshot failed with no option name is given https://review.openstack.org/531289 | 04:36 |
| *** ykarel|away has joined #openstack-cinder | 04:45 | |
| *** ykarel|away is now known as ykarel | 04:59 | |
| *** chhavi__ has joined #openstack-cinder | 05:00 | |
| *** dave-mccowan has quit IRC | 05:06 | |
| *** wanghao has quit IRC | 05:08 | |
| *** wanghao has joined #openstack-cinder | 05:08 | |
| ykarel | smcginnis, hi | 05:14 |
| ykarel | smcginnis, winston-d can you check my comment on https://review.openstack.org/#/c/520991/ and confirm | 05:15 |
| *** Brin has quit IRC | 05:20 | |
| *** Brin has joined #openstack-cinder | 05:20 | |
| *** Dinesh_Bhor has quit IRC | 05:49 | |
| *** zengyingzhe has joined #openstack-cinder | 05:51 | |
| ykarel | smcginnis, winston-d created bug: https://bugs.launchpad.net/cinder/+bug/1741402 for ^^ | 05:51 |
| openstack | Launchpad bug 1741402 in Cinder "Backward Incompatible change: https://review.openstack.org/#/c/520991" [Undecided,New] | 05:51 |
| *** Apoorva has quit IRC | 05:55 | |
| *** wanghao has quit IRC | 05:56 | |
| *** wanghao has joined #openstack-cinder | 05:58 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:03 | |
| *** Dinesh_Bhor has quit IRC | 06:04 | |
| *** markstur_ has quit IRC | 06:11 | |
| *** threestrands_ has quit IRC | 06:13 | |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: Add project_id to backup details https://review.openstack.org/531309 | 06:40 |
| *** wanghao_ has joined #openstack-cinder | 06:42 | |
| *** wanghao has quit IRC | 06:42 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: create snapshot failed with no option name is given https://review.openstack.org/531289 | 06:43 |
| openstackgerrit | yatin proposed openstack/cinder master: Allow zero length name to be None https://review.openstack.org/531312 | 06:45 |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: Add project_id to backup details https://review.openstack.org/531309 | 06:47 |
| *** lhx_ has joined #openstack-cinder | 06:54 | |
| *** armaan_ has joined #openstack-cinder | 06:54 | |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: Add project_id to backup details https://review.openstack.org/531309 | 06:54 |
| *** pgadiya has joined #openstack-cinder | 07:06 | |
| *** claudiub has joined #openstack-cinder | 07:14 | |
| *** lhx_ has quit IRC | 07:14 | |
| *** lhx_ has joined #openstack-cinder | 07:15 | |
| *** armax has quit IRC | 07:29 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 07:32 | |
| *** Dinesh_Bhor has quit IRC | 07:47 | |
| *** zhurong has joined #openstack-cinder | 07:48 | |
| *** edmondsw has joined #openstack-cinder | 07:49 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 07:52 | |
| openstackgerrit | Merged openstack/cinder master: Handle InvalidVolume when creating a volume attachment https://review.openstack.org/531011 | 07:53 |
| *** edmondsw has quit IRC | 07:53 | |
| *** ykarel is now known as ykarel|lunch | 07:58 | |
| *** markvoelker has joined #openstack-cinder | 08:09 | |
| *** tesseract has joined #openstack-cinder | 08:12 | |
| *** gouthamr has quit IRC | 08:13 | |
| *** yangyapeng has quit IRC | 08:15 | |
| *** yangyapeng has joined #openstack-cinder | 08:15 | |
| *** armaan_ has quit IRC | 08:15 | |
| *** armaan has joined #openstack-cinder | 08:16 | |
| *** zhurong has quit IRC | 08:16 | |
| *** armaan has quit IRC | 08:16 | |
| *** zhurong has joined #openstack-cinder | 08:27 | |
| *** rcernin has quit IRC | 08:28 | |
| *** armaan has joined #openstack-cinder | 08:31 | |
| *** Dinesh_Bhor has quit IRC | 08:36 | |
| *** Brin has quit IRC | 08:43 | |
| *** markvoelker has quit IRC | 08:44 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 08:47 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Refactored cinder tempest tests to cinder_tempest_plugin https://review.openstack.org/529824 | 08:52 |
| *** moshele has joined #openstack-cinder | 08:52 | |
| *** armax has joined #openstack-cinder | 08:55 | |
| *** s-shiono_ has quit IRC | 08:59 | |
| *** ykarel|lunch is now known as ykarel | 09:01 | |
| *** stakeda has quit IRC | 09:09 | |
| *** moshele has quit IRC | 09:12 | |
| *** armax has quit IRC | 09:14 | |
| *** johnthetubaguy has quit IRC | 09:17 | |
| *** johnthetubaguy has joined #openstack-cinder | 09:18 | |
| *** lpetrut has joined #openstack-cinder | 09:31 | |
| gcb | jungleboyj: could you please help contact with eharney about https://review.openstack.org/528618 | 09:32 |
| *** cristicalin has joined #openstack-cinder | 09:34 | |
| *** ykarel_ has joined #openstack-cinder | 09:36 | |
| openstackgerrit | Jeremy Zhang proposed openstack/python-cinderclient master: Fix 'cluster' paramter using for v3 volume manage https://review.openstack.org/531344 | 09:36 |
| *** edmondsw has joined #openstack-cinder | 09:37 | |
| *** ykarel has quit IRC | 09:38 | |
| *** cristicalin has quit IRC | 09:39 | |
| *** wanghao_ has quit IRC | 09:40 | |
| *** markvoelker has joined #openstack-cinder | 09:41 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Revert volume consumed capacity when rescheduling https://review.openstack.org/531345 | 09:41 |
| *** edmondsw has quit IRC | 09:42 | |
| *** claudiub|2 has joined #openstack-cinder | 09:51 | |
| *** claudiub has quit IRC | 09:54 | |
| *** Dinesh_Bhor has quit IRC | 10:09 | |
| *** e0ne has joined #openstack-cinder | 10:12 | |
| *** markvoelker has quit IRC | 10:15 | |
| *** hoangcx has quit IRC | 10:15 | |
| *** ykarel_ is now known as ykarel | 10:17 | |
| *** Brin has joined #openstack-cinder | 10:28 | |
| Brin | Hi tommy | 10:28 |
| *** gcb has quit IRC | 10:29 | |
| Brin | https://review.openstack.org/#/c/531309/ this review I find it on ocata. And it exists on Queen, and I use the context is admin. | 10:31 |
| *** ykarel_ has joined #openstack-cinder | 10:33 | |
| *** namnh_ has joined #openstack-cinder | 10:33 | |
| *** namnh_ has quit IRC | 10:34 | |
| Brin | Thanks for reviewing my code.If you have your own environment, I suggest you try to get it. If it is my understanding is not in place, I will make changes. | 10:34 |
| *** ykarel has quit IRC | 10:36 | |
| *** namnh has quit IRC | 10:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder-tempest-plugin master: Updated from global requirements https://review.openstack.org/531353 | 10:38 |
| *** moshele has joined #openstack-cinder | 10:42 | |
| *** sdague has joined #openstack-cinder | 10:43 | |
| *** moshele has quit IRC | 10:44 | |
| openstackgerrit | xiaoqin proposed openstack/cinder master: Storwize: correct in-use replication vol status https://review.openstack.org/531357 | 10:46 |
| *** abhishekk has quit IRC | 10:49 | |
| *** Brin has quit IRC | 10:52 | |
| *** ykarel_ is now known as ykarel | 10:53 | |
| *** zhurong has quit IRC | 10:57 | |
| *** chhavi__ has quit IRC | 10:58 | |
| *** chhavi has joined #openstack-cinder | 11:01 | |
| *** pcaruana has joined #openstack-cinder | 11:02 | |
| *** markvoelker has joined #openstack-cinder | 11:11 | |
| openstackgerrit | yatin proposed openstack/cinder master: Allow zero length name to be None https://review.openstack.org/531312 | 11:15 |
| *** nicolasbock has joined #openstack-cinder | 11:35 | |
| *** markvoelker has quit IRC | 11:45 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Fix mock for eventlet.greenthread.sleep in Instorage tests https://review.openstack.org/531363 | 11:57 |
| *** abishop has quit IRC | 12:12 | |
| *** moshele has joined #openstack-cinder | 12:42 | |
| *** markvoelker has joined #openstack-cinder | 12:42 | |
| *** moshele has quit IRC | 12:51 | |
| *** pooja_jadhav has quit IRC | 12:57 | |
| *** hoonetorg has quit IRC | 12:57 | |
| *** abishop has joined #openstack-cinder | 13:00 | |
| *** dillaman has joined #openstack-cinder | 13:03 | |
| *** abishop has quit IRC | 13:04 | |
| *** ykarel has quit IRC | 13:05 | |
| *** abishop has joined #openstack-cinder | 13:06 | |
| *** psachin has quit IRC | 13:09 | |
| *** lhx_ has quit IRC | 13:10 | |
| *** arnewiebalck_ has joined #openstack-cinder | 13:11 | |
| *** erlon has joined #openstack-cinder | 13:11 | |
| *** arnewiebalck_ has quit IRC | 13:11 | |
| *** edmondsw has joined #openstack-cinder | 13:13 | |
| *** markvoelker has quit IRC | 13:15 | |
| *** edmondsw has quit IRC | 13:17 | |
| *** arnewiebalck_ has joined #openstack-cinder | 13:21 | |
| *** dave-mccowan has joined #openstack-cinder | 13:22 | |
| *** eharney has joined #openstack-cinder | 13:24 | |
| *** slunkad has joined #openstack-cinder | 13:32 | |
| *** pgadiya has quit IRC | 13:36 | |
| *** markvoelker has joined #openstack-cinder | 13:36 | |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Run backup-restore operations on native thread https://review.openstack.org/518316 | 13:36 |
| *** lhx_ has joined #openstack-cinder | 13:44 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: Add Windows volume backup support https://review.openstack.org/528352 | 13:45 |
| *** nsingh has joined #openstack-cinder | 13:46 | |
| *** bhagyashri_s has joined #openstack-cinder | 13:47 | |
| *** links has quit IRC | 13:47 | |
| *** arnewiebalck_ has quit IRC | 13:48 | |
| openstackgerrit | Jeremy Zhang proposed openstack/python-cinderclient master: Fix 'cluster' paramter using for v3 volume manage https://review.openstack.org/531344 | 13:49 |
| *** niraj_singh has quit IRC | 13:49 | |
| *** bhagyashris has quit IRC | 13:49 | |
| *** tonyb has quit IRC | 13:49 | |
| *** tonyb has joined #openstack-cinder | 13:50 | |
| *** jistr is now known as jistr|biab | 13:51 | |
| *** pgadiya has joined #openstack-cinder | 13:53 | |
| *** edmondsw has joined #openstack-cinder | 13:53 | |
| mnaser | so nova ignores volume detach failures when deleting an instance which could lead in a condition of a volume with attachments to a server that is deleted. is there a workflow to handle this as i haven't found a way to delete those volumes | 13:53 |
| *** pgadiya has quit IRC | 13:53 | |
| mnaser | and by delete, i mean delete them cleanly without having to muck around the db | 13:54 |
| openstackgerrit | Merged openstack/cinder master: Mock FixedIntervalLoopingCall for Huawei unit tests https://review.openstack.org/531181 | 13:54 |
| smcginnis | mnaser: In that case, I think you would need to use "cinder force-delete" | 13:55 |
| mnaser | smcginnis: force delete complained that it could not delete an attached volume :< | 13:55 |
| mnaser | Delete for volume 973d5a3d-07f3-4d05-b62c-8dfe746d298c failed: Invalid volume: Volume must not be migrating, attached, belong to a group, have snapshots or be disassociated from snapshots after volume transfer. (HTTP 400) (Request-ID: req-05c2448e-e086-4423-95b5-becc2c60e28d) | 13:55 |
| mnaser | ERROR: Unable to force delete any of the specified volumes. | 13:55 |
| smcginnis | mnaser: OK, then I think first | 13:55 |
| smcginnis | "cinder reset-state --state available $ID" | 13:56 |
| smcginnis | mnaser: Then "cinder delete" | 13:56 |
| * smcginnis wonders why we have a "force" that doesn't allow deleting unless it's not attached | 13:57 | |
| mnaser | smcginnis: already tried resetting state to available, the status of the volume is 'available' but it still has attachments | 13:57 |
| smcginnis | mnaser: Sorry - "cinder reset-state --state available --attach-status detached $ID" | 13:58 |
| mnaser | ahhh the attach status is what i was missing | 13:58 |
| mnaser | ok wonderful | 13:59 |
| mnaser | i couldnt find that documented anywhere online, i found a lot of db hacks | 13:59 |
| mnaser | but i was sure there must have been a clean way to do this | 13:59 |
| mnaser | thank you very much smcginnis.. is there a good place for us to add this in cinder docs (i can propose the patch) | 14:00 |
| smcginnis | mnaser: It's a bit of a mess right now, but somewhere under our admin guide would be good: https://docs.openstack.org/cinder/latest/admin/index.html | 14:02 |
| smcginnis | mnaser: I think we need to reorganize all that, but if you want to propose that to somewhere somewhat logical, we may shift it around later, but at least it would be captured then. | 14:03 |
| mnaser | smcginnis: yeah at least have it written down somewhere | 14:03 |
| * mnaser adds to personal log | 14:03 | |
| smcginnis | mnaser: Thanks! | 14:04 |
| *** claudiub|2 is now known as claudiub | 14:08 | |
| *** mriedem has joined #openstack-cinder | 14:12 | |
| *** KeithMnemonic has joined #openstack-cinder | 14:31 | |
| mnaser | smcginnis: i added a topic for the next meeting a while back in regarding the performance of volume usage checks for the rbd driver. if there's someone i should reach out to who should be there, that would be helpful to know so we can discuss it | 14:32 |
| *** jmlowe has quit IRC | 14:43 | |
| *** jmlowe has joined #openstack-cinder | 14:45 | |
| *** jmlowe has quit IRC | 14:48 | |
| *** nicolasbock has quit IRC | 14:51 | |
| smcginnis | mnaser: jbernard is/was the primary driver maintainer for rbd. Not around a lot, but he is here from time to time. | 14:52 |
| smcginnis | mnaser: Would be good to talk to him or maybe eharney. | 14:52 |
| mnaser | smcginnis: ok cool, i'll try to see if i can reach out in some way, if not i don't mind making an effort on my side because its absolutely kills performance (agent just pretty much gets stuck down) | 14:54 |
| mnaser | maybe its a more of an architectural change that should happen where the statistic collection happens in a different thread | 14:54 |
| smcginnis | mnaser: Maybe if it's specific to that driver, it could possibly do something like have a periodic background check so it's got recent data cached or something. | 14:56 |
| smcginnis | Anyway, worth exploring if there is a way to improve things. | 14:56 |
| eharney | mnaser: i actually thought we had fixed the performance issues w/ rbd stats polling a bit ago | 14:56 |
| smcginnis | eharney: Wasn't there a lib update that helped with that too? | 14:56 |
| mnaser | eharney: if you have a few minutes i can go through what i identified that is causing a ton of problems for us | 14:56 |
| eharney | smcginnis: i'm thinking of rbd driver changes | 14:57 |
| eharney | mnaser: sure | 14:57 |
| mnaser | eharney: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L380-L404 | 14:57 |
| mnaser | so one of the things in that section that is not optimized is the possiblity that the pool is being used by something else. in our case, we share the same pool for legacy reasons, so it's going through *all* volumes instead of the ones provisioned by cinder | 14:58 |
| mnaser | which can include epheremal volumes for nova | 14:58 |
| eharney | ah, i understand, the previous fix i was thinking of was in that same method, to no longer use diff_iterate | 14:59 |
| mnaser | oh yes, the removal of diff_iterate has made things much better | 14:59 |
| eharney | approximately how many volumes/rbd images do you have in the pool? | 15:00 |
| mnaser | it went from unusable with diff_iterate to like agent blipping off and on because its not reporting in time (so service is marked down) | 15:01 |
| openstackgerrit | Merged openstack/cinder master: Storwize: add hyperswap volume support https://review.openstack.org/492414 | 15:02 |
| mnaser | eharney: in this case, a cluster with 1600 rbd volumes | 15:02 |
| mnaser | 802 volumes | 15:02 |
| mnaser | so it would reduce the number of size checks by half in this case at least | 15:02 |
| eharney | we actually used to try to filter out non-cinder volumes, and stopped doing that as part of the overprovisioning fixes | 15:04 |
| *** jistr|biab is now known as jistr | 15:04 | |
| mnaser | eharney: that loop seems a bit redundant to me given that _get_usage_info() should technically give the same # as the sum of volumes provisioned in the db | 15:05 |
| mnaser | assuming everything goes okay | 15:05 |
| mnaser | so could a simple db query to replace that be enough? | 15:05 |
| *** _ix has joined #openstack-cinder | 15:06 | |
| *** ykarel has joined #openstack-cinder | 15:06 | |
| eharney | we use a db query for part of this (allocated_capacity_gb) -- i'd need to go read geguileo's spec again to understand the exact relationship between the different values here | 15:07 |
| eharney | the general idea is that when calculating the overprovisioning ratio, we need to pull some of the info from rbd itself on space usage, and some is pulled from cinder | 15:07 |
| eharney | (he's back in the office next week and could surely help us with this) | 15:08 |
| mnaser | eharney: ok cool, i added it as an item for next meeting so if you can ping him to discuss it so that we can see if we can 'rethink' how we do that part | 15:10 |
| eharney | great | 15:10 |
| *** hoonetorg has joined #openstack-cinder | 15:10 | |
| mnaser | for now we just don't call that function and set it to 0 .. seems tobe an ok workaround | 15:11 |
| mnaser | but i dont like it | 15:11 |
| melwitt | smcginnis, jungleboyj: hey, I was wondering if it would be possible to have .z releases for os-brick stable/pike and stable/ocata since the backports merged? https://review.openstack.org/#/q/status:merged+project:openstack/os-brick+topic:bug/1724573 | 15:11 |
| smcginnis | melwitt: Shouldn't be a problem. I'll leave it to jungleboyj to propose the release though. | 15:12 |
| *** _ix has quit IRC | 15:13 | |
| melwitt | cool, thank you | 15:13 |
| jungleboyj | eharney: Did you see the request from gcb to unblock this? https://review.openstack.org/528618 | 15:14 |
| jungleboyj | melwitt: Sure. We can do that. | 15:14 |
| jungleboyj | smcginnis: You want me to propose those? | 15:14 |
| eharney | jungleboyj: nope, thanks for the heads up | 15:15 |
| jungleboyj | eharney: No problem. He pinged me earlier. Thanks for fixing that. | 15:15 |
| smcginnis | jungleboyj: Yeah, that would be good. Then Monday I we can get them out (no releases on Friday). | 15:16 |
| jungleboyj | smcginnis: Right. I am pretty booked this morning so I can tackle it later today. | 15:16 |
| smcginnis | jungleboyj: Sounds good. | 15:16 |
| melwitt | thanks jungleboyj | 15:17 |
| jungleboyj | melwitt: No problem. Glad to help. | 15:17 |
| *** arnewiebalck_ has joined #openstack-cinder | 15:17 | |
| eharney | is anyone looking at the tempest-full failures on stable? seems to be a tempest/keystone issue | 15:18 |
| jungleboyj | eharney: I saw that but haven't had a chance to look at it. | 15:19 |
| *** gouthamr has joined #openstack-cinder | 15:22 | |
| smcginnis | eharney: I asked in infra a couple times. I think Andreas may be working on it. Or at least is aware of the issue. | 15:22 |
| smcginnis | But hasn't been around to share. | 15:22 |
| smcginnis | Something needs to be backported for zuul v3 jobs to stable branches. | 15:23 |
| eharney | ok | 15:24 |
| chandankumar | eharney: smcginnis please have a look on these reviews https://review.openstack.org/531206 https://review.openstack.org/529824 and https://review.openstack.org/531361 | 15:24 |
| chandankumar | all related to cinder tempest plugin | 15:25 |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Fix: Validate fail when create group snapshot https://review.openstack.org/531393 | 15:26 |
| eharney | chandankumar: ok. i don't really know enough about the details to have much input on Luigi's comment on the first patch, but it sounds like a good idea | 15:26 |
| chandankumar | eharney: i am updating that patch | 15:26 |
| chandankumar | currently working on that | 15:26 |
| chandankumar | just spoke with him on #rdo channel for the same | 15:27 |
| *** jmlowe has joined #openstack-cinder | 15:32 | |
| mriedem | did you guys notice that the cinder-specs docs aren't getting updated? https://specs.openstack.org/openstack/cinder-specs/ | 15:37 |
| mriedem | e.g. jgriffith's multiattach spec isn't in there | 15:37 |
| * mriedem asks in infra | 15:37 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 15:39 |
| chandankumar | eharney: ^^ | 15:39 |
| smcginnis | mriedem: Saw the response in infra. Wasn't aware of that. | 15:39 |
| mriedem | i guess just need to merge something | 15:39 |
| *** zhaochao has quit IRC | 15:39 | |
| smcginnis | mriedem: I'll put up a clean up patch just to get it through. | 15:40 |
| mriedem | ok | 15:40 |
| Roamer` | hm, I wonder if it would be too forward and self-serving of me to ask about the possibility of releasing cinder or at least os-brick, so that a "recheck" on the StorPool Nova driver has a chance to pass? :) | 15:40 |
| Roamer` | actually, come to think of it, it's just os-brick | 15:41 |
| eharney | chandankumar: out of curiosity, does "description" in .zuul.yaml ever show up anywhere? | 15:41 |
| chandankumar | eharney: i have not checked it | 15:42 |
| *** liverpooler has joined #openstack-cinder | 15:43 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder-specs master: Doc8 and file cleanup https://review.openstack.org/531396 | 15:47 |
| *** stephenfin is now known as finucannot | 15:53 | |
| *** markstur has joined #openstack-cinder | 16:01 | |
| *** jmlowe has quit IRC | 16:02 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 16:03 |
| *** dalgaaf has joined #openstack-cinder | 16:04 | |
| *** arnewiebalck_ has quit IRC | 16:08 | |
| *** mtreinish has quit IRC | 16:08 | |
| *** mtreinish has joined #openstack-cinder | 16:09 | |
| openstackgerrit | Merged openstack/cinder-specs master: Doc8 and file cleanup https://review.openstack.org/531396 | 16:10 |
| *** arnewiebalck_ has joined #openstack-cinder | 16:11 | |
| *** stvnoyes has joined #openstack-cinder | 16:13 | |
| openstackgerrit | Eric Harney proposed openstack/os-brick master: Remove requirement on oslo.serialization https://review.openstack.org/531409 | 16:15 |
| *** moshele has joined #openstack-cinder | 16:17 | |
| jbernard | smcginnis: o/ | 16:18 |
| *** moshele has quit IRC | 16:19 | |
| smcginnis | jbernard: Hey! I think eharney was able to answer questions. | 16:20 |
| andreykurilin | hi cinder team! | 16:20 |
| andreykurilin | I need a help | 16:21 |
| *** jmlowe has joined #openstack-cinder | 16:21 | |
| *** dalgaaf has quit IRC | 16:21 | |
| andreykurilin | It looks like after recent changes in Cinder API request validation, several rally scenarios started to fail | 16:22 |
| andreykurilin | see http://logs.openstack.org/04/528504/2/check/legacy-rally-dsvm-rally-cinder/8377bbc/rally-plot/results.html.gz#/CinderVolumeBackups.create_incremental_volume_backup/failures | 16:22 |
| andreykurilin | snapshot_id by default is None in cinderclient (https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/volume_backups.py#L49) | 16:22 |
| andreykurilin | And cinderclient transmits it to the API side anyway. even in case of creating a backup from a volume | 16:23 |
| eharney | andreykurilin: i think this is https://bugs.launchpad.net/cinder/+bug/1741191 | 16:23 |
| openstack | Launchpad bug 1741191 in Cinder "Create snapshot failed on cinder client" [High,In progress] - Assigned to yatin (yatinkarel) | 16:23 |
| andreykurilin | yes, it is | 16:23 |
| andreykurilin | eharney: thanks | 16:23 |
| eharney | there's a patch in review | 16:24 |
| Roamer` | smcginnis, jungleboyj, there was a bit of discussion about the StorPool Nova attachment driver in #openstack-nova just now, and mriedem pointed out that there really ought to be an os-brick release containing the StorPool driver first. I imagine that there's something scheduled for around milestone 2, but is there a chance to move it up a bit? Or am I being too annoying here, too? :) | 16:24 |
| mriedem | the final release date for os-brick is jan 18 | 16:25 |
| mriedem | but that's not to say one could happen earlier | 16:25 |
| mriedem | jan 25 is feature freeze | 16:25 |
| mriedem | and milestone 3 | 16:25 |
| smcginnis | Roamer`: Yes, you will need a full release with your connector. There are a few other things pending for os-brick as well. | 16:27 |
| Roamer` | right, trust me to get the milestones wrong... | 16:27 |
| smcginnis | Roamer`: jungleboyj plans on doing new os-brick releases soon I believe. | 16:27 |
| smcginnis | mriedem: Looks like the cinder-specs post job completed. | 16:28 |
| mriedem | smcginnis: nice https://specs.openstack.org/openstack/cinder-specs/specs/queens/enable-multiattach.html | 16:29 |
| jungleboyj | smcginnis: Roamer` Sure, I can respin os-brick later today. | 16:29 |
| jungleboyj | Release will come out Monday then. | 16:30 |
| Roamer` | jungleboyj, that'd be great! (and yeah, I did notice the "no releases on Friday" line earlier, quite reasonable) | 16:30 |
| jungleboyj | :-) | 16:31 |
| jungleboyj | Roamer`: Best to not blow stuff up on Friday. | 16:31 |
| *** e0ne has quit IRC | 16:34 | |
| *** _ix has joined #openstack-cinder | 16:35 | |
| eharney | we seem to have lost our filtering that skips tempest jobs on patches that only change unit tests: https://review.openstack.org/#/c/531197/ | 16:37 |
| eharney | but it looks like it's still in project-config | 16:37 |
| eharney | hrrmm... | 16:37 |
| smcginnis | eharney: That had been working, at least as of a couple weeks ago. | 16:38 |
| eharney | smcginnis: yeah, this is the first time i've noticed it, not that i've been really looking... | 16:38 |
| smcginnis | eharney: Wait, did we skip tempest on unit test changes? I thought it was just docs-only and things like that. | 16:39 |
| *** armax has joined #openstack-cinder | 16:39 | |
| eharney | smcginnis: yes, see https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml?id=0a93ac#n3884 | 16:39 |
| eharney | line 3892 specifically | 16:40 |
| *** openstackstatus has quit IRC | 16:40 | |
| smcginnis | hmm | 16:40 |
| *** openstackstatus has joined #openstack-cinder | 16:40 | |
| *** ChanServ sets mode: +v openstackstatus | 16:40 | |
| *** mikal has quit IRC | 16:43 | |
| *** mikal has joined #openstack-cinder | 16:45 | |
| eharney | smcginnis: and it actually did filter out a lot of jobs, like devstack-plugin-ceph etc -- i guess tempest-full is somehow special here | 16:45 |
| smcginnis | eharney: Might have to ask in infra why that would be. | 16:50 |
| eharney | yep, will do | 16:50 |
| *** ykarel has quit IRC | 16:51 | |
| *** Apoorva has joined #openstack-cinder | 16:51 | |
| *** armaan has quit IRC | 16:57 | |
| *** armaan has joined #openstack-cinder | 16:58 | |
| *** mriedem is now known as mriedem_luncharo | 17:04 | |
| *** Apoorva has quit IRC | 17:04 | |
| eharney | smcginnis: i'll worry later about actually making a filter for this -- satisfied enough for now just knowing that it's not all broken | 17:07 |
| smcginnis | eharney: Yep, good enough for now. | 17:08 |
| *** ykarel has joined #openstack-cinder | 17:09 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 17:10 |
| *** lhx_ has quit IRC | 17:11 | |
| *** dalgaaf has joined #openstack-cinder | 17:21 | |
| *** pcaruana has quit IRC | 17:21 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: On None connector terminate_connection unmaps all https://review.openstack.org/531231 | 17:25 |
| *** lpetrut has quit IRC | 17:31 | |
| *** ykarel has quit IRC | 17:33 | |
| *** ykarel has joined #openstack-cinder | 17:34 | |
| *** danpawlik has quit IRC | 17:37 | |
| *** ykarel has quit IRC | 17:42 | |
| *** dobson has quit IRC | 17:42 | |
| *** dobson has joined #openstack-cinder | 17:46 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 17:50 |
| *** Apoorva has joined #openstack-cinder | 17:54 | |
| *** dave-mccowan has quit IRC | 17:57 | |
| *** lpetrut has joined #openstack-cinder | 18:02 | |
| *** dalgaaf has quit IRC | 18:09 | |
| *** harlowja has joined #openstack-cinder | 18:09 | |
| *** dalgaaf has joined #openstack-cinder | 18:13 | |
| *** mriedem_luncharo is now known as mriedem | 18:17 | |
| *** chhavi has quit IRC | 18:17 | |
| *** _pewp_ has quit IRC | 18:20 | |
| *** _pewp_ has joined #openstack-cinder | 18:21 | |
| openstackgerrit | Chandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest https://review.openstack.org/531206 | 18:25 |
| *** chandankumar is now known as {archer} | 18:27 | |
| *** armaan has quit IRC | 18:44 | |
| *** armaan has joined #openstack-cinder | 18:47 | |
| *** openstack has joined #openstack-cinder | 21:17 | |
| *** ChanServ sets mode: +o openstack | 21:17 | |
| *** jose-phillips has quit IRC | 21:17 | |
| *** jose-phillips has joined #openstack-cinder | 21:20 | |
| *** tonyb has quit IRC | 21:21 | |
| *** erlon has quit IRC | 21:27 | |
| *** jose-phillips has quit IRC | 21:29 | |
| *** lpetrut has quit IRC | 21:29 | |
| *** jose-phillips has joined #openstack-cinder | 21:30 | |
| *** lpetrut has joined #openstack-cinder | 21:30 | |
| *** gouthamr has joined #openstack-cinder | 21:47 | |
| *** Apoorva has joined #openstack-cinder | 21:48 | |
| *** dave-mccowan has quit IRC | 21:50 | |
| *** jose-phillips has quit IRC | 21:55 | |
| *** Apoorva_ has joined #openstack-cinder | 21:56 | |
| *** jose-phillips has joined #openstack-cinder | 21:58 | |
| *** abishop has quit IRC | 21:58 | |
| *** Apoorva has quit IRC | 21:59 | |
| *** dalgaaf has quit IRC | 22:00 | |
| *** dalgaaf has joined #openstack-cinder | 22:02 | |
| *** gouthamr has quit IRC | 22:07 | |
| *** gouthamr has joined #openstack-cinder | 22:12 | |
| *** armaan_ has quit IRC | 22:14 | |
| *** gouthamr has quit IRC | 22:15 | |
| *** gouthamr has joined #openstack-cinder | 22:15 | |
| *** jose-phillips has quit IRC | 22:16 | |
| *** gouthamr has quit IRC | 22:16 | |
| *** edmondsw has quit IRC | 22:22 | |
| *** edmondsw has joined #openstack-cinder | 22:23 | |
| openstackgerrit | Eric Young proposed openstack/cinder master: ScaleIO: Enable multiattach for volumes https://review.openstack.org/531511 | 22:26 |
| *** edmondsw has quit IRC | 22:26 | |
| *** arnewiebalck_ has joined #openstack-cinder | 22:27 | |
| *** jose-phillips has joined #openstack-cinder | 22:29 | |
| *** mchlumsky has quit IRC | 22:41 | |
| *** e0ne has joined #openstack-cinder | 22:42 | |
| *** jose-phillips has quit IRC | 22:42 | |
| *** arnewiebalck_ has quit IRC | 23:01 | |
| *** david-lyle has joined #openstack-cinder | 23:04 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: Windows: allow multiple c-vol backends per service https://review.openstack.org/522806 | 23:06 |
| *** _ix has quit IRC | 23:10 | |
| *** moshele has joined #openstack-cinder | 23:13 | |
| *** amoralej|off has quit IRC | 23:16 | |
| *** sandk has joined #openstack-cinder | 23:17 | |
| *** lpetrut has quit IRC | 23:19 | |
| *** lpetrut has joined #openstack-cinder | 23:20 | |
| *** moshele has quit IRC | 23:21 | |
| *** lpetrut has quit IRC | 23:28 | |
| *** Apoorva_ has quit IRC | 23:28 | |
| *** stvnoyes has quit IRC | 23:35 | |
| *** e0ne has quit IRC | 23:40 | |
| *** sandk has quit IRC | 23:42 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add policy for multiattach https://review.openstack.org/531521 | 23:45 |
| *** liverpooler has quit IRC | 23:48 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add policy for multiattach https://review.openstack.org/531521 | 23:49 |
| *** liverpooler has joined #openstack-cinder | 23:50 | |
| *** jose-phillips has joined #openstack-cinder | 23:52 | |
| *** sdague has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!