| jungleboyj | smcginnis: http://logs.openstack.org/92/509292/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ee4f503/console.html | 00:00 |
|---|---|---|
| smcginnis | jungleboyj: Yeah, looks like the same thing: http://logs.openstack.org/92/509292/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ee4f503/logs/screen-c-bak.txt.gz#_Oct_05_15_28_30_220216 | 00:01 |
| jungleboyj | Remote error: ProcessExecutionError Unexpected error while running command.\nCommand: sudo cinder-rootwrap /etc/cinder/rootwrap.conf lvchange -a y --yes -K stack-volumes-lvmdriver-1/_snapshot-6544f849-be4e-437e-9f65-c5cfaf78a6ff\nExit code: 5\nStdout: u''\nSt", "is_incremental": false, "id": "dd6bdd20-1b60-4782-92c8-c9a856a53019", "size": 1}} | 00:01 |
| jungleboyj | He he. Found that at the same time as you. | 00:02 |
| jungleboyj | Ok, so that is the thin provisioning issue. So the other one is the issues with cascade delete that I am intermittently seeing. | 00:02 |
| jungleboyj | Do you know if we have bugs open for either of those yet? | 00:03 |
| smcginnis | Well, it's just the one bug. | 00:11 |
| *** ntpttr_laptop has quit IRC | 00:12 | |
| * jungleboyj is confused | 00:12 | |
| jungleboyj | smcginnis: So are both problems caused by the thin provisioning with LVM issues we think? | 00:12 |
| smcginnis | The error "device-mapper: reload ioctl on (252:4) failed: Invalid argument". It's just bubbling up in multiple places. | 00:13 |
| jungleboyj | Ok. I misinterpreted what you had said earlier. | 00:14 |
| jungleboyj | Sorry. | 00:14 |
| *** hemna__ has quit IRC | 00:14 | |
| smcginnis | jungleboyj: Seems to be a timing issue since we don't always see it. | 00:15 |
| smcginnis | Timing with itself, or timing between it and other concurrent operations that are going on at the time. | 00:15 |
| jungleboyj | Yeah, I was trying to get a VM up on the PowerEdge box to try and recreate. | 00:16 |
| jungleboyj | Gotta go feed the boys. Back online later. | 00:17 |
| *** crushil has joined #openstack-cinder | 00:19 | |
| *** crushil has quit IRC | 00:23 | |
| *** crushil has joined #openstack-cinder | 00:25 | |
| *** hemna__ has joined #openstack-cinder | 00:25 | |
| *** crushil has quit IRC | 00:26 | |
| *** savvas has joined #openstack-cinder | 00:38 | |
| *** Nel1x has quit IRC | 00:44 | |
| *** savvas has quit IRC | 00:58 | |
| *** crushil has joined #openstack-cinder | 01:01 | |
| *** markstur has quit IRC | 01:04 | |
| *** Apoorva has quit IRC | 01:11 | |
| *** openstackstatus has quit IRC | 01:14 | |
| *** openstackstatus has joined #openstack-cinder | 01:14 | |
| *** ChanServ sets mode: +v openstackstatus | 01:14 | |
| *** mriedem has quit IRC | 01:26 | |
| *** zigo has quit IRC | 01:27 | |
| *** dikonoor has joined #openstack-cinder | 01:29 | |
| *** zigo has joined #openstack-cinder | 01:31 | |
| *** markvoelker has joined #openstack-cinder | 01:52 | |
| *** markstur has joined #openstack-cinder | 01:57 | |
| *** wN has quit IRC | 02:04 | |
| *** sdake has quit IRC | 02:04 | |
| *** sweston has quit IRC | 02:04 | |
| *** sweston has joined #openstack-cinder | 02:05 | |
| *** wiggin15 has quit IRC | 02:05 | |
| *** wiggin15 has joined #openstack-cinder | 02:05 | |
| *** andymccr has quit IRC | 02:06 | |
| *** rha has quit IRC | 02:06 | |
| *** lyarwood has quit IRC | 02:06 | |
| *** wN has joined #openstack-cinder | 02:06 | |
| *** wN has quit IRC | 02:06 | |
| *** wN has joined #openstack-cinder | 02:06 | |
| *** twouters has quit IRC | 02:06 | |
| *** gerhardqux has quit IRC | 02:06 | |
| *** gerhardqux has joined #openstack-cinder | 02:06 | |
| *** mtreinish has quit IRC | 02:08 | |
| *** lyarwood has joined #openstack-cinder | 02:08 | |
| *** jistr has quit IRC | 02:08 | |
| *** john5223_ has quit IRC | 02:08 | |
| *** savvas has joined #openstack-cinder | 02:09 | |
| *** stephenfin has quit IRC | 02:09 | |
| *** dirk_ has joined #openstack-cinder | 02:09 | |
| *** enriquetaso_ has joined #openstack-cinder | 02:09 | |
| *** jistr has joined #openstack-cinder | 02:10 | |
| *** mtreinish has joined #openstack-cinder | 02:10 | |
| openstackgerrit | Merged openstack/cinder master: Remove QNAP volume driver https://review.openstack.org/501331 | 02:11 |
| *** twouters has joined #openstack-cinder | 02:11 | |
| *** twouters has quit IRC | 02:11 | |
| *** twouters has joined #openstack-cinder | 02:11 | |
| *** stephenfin has joined #openstack-cinder | 02:11 | |
| *** enriquetaso has quit IRC | 02:12 | |
| *** dirk has quit IRC | 02:12 | |
| *** kambiz has quit IRC | 02:12 | |
| *** logan- has quit IRC | 02:12 | |
| *** enriquetaso_ is now known as enriquetaso | 02:12 | |
| *** dirk_ is now known as dirk | 02:12 | |
| *** sdake has joined #openstack-cinder | 02:13 | |
| *** andymccr has joined #openstack-cinder | 02:13 | |
| *** sdake is now known as Guest2849 | 02:13 | |
| *** savvas has quit IRC | 02:14 | |
| *** kambiz has joined #openstack-cinder | 02:14 | |
| *** rha has joined #openstack-cinder | 02:14 | |
| *** logan- has joined #openstack-cinder | 02:15 | |
| *** crushil has quit IRC | 02:17 | |
| *** crushil has joined #openstack-cinder | 02:22 | |
| *** mgagne has quit IRC | 02:23 | |
| *** mgagne has joined #openstack-cinder | 02:24 | |
| *** mgagne is now known as Guest66098 | 02:24 | |
| *** markvoelker has quit IRC | 02:27 | |
| *** jwcroppe has joined #openstack-cinder | 02:30 | |
| *** diegows_ has joined #openstack-cinder | 02:32 | |
| *** dave-mccowan has quit IRC | 02:33 | |
| *** diegows has quit IRC | 02:35 | |
| *** sapd has joined #openstack-cinder | 02:38 | |
| *** sapd has quit IRC | 02:38 | |
| *** ericyoung has quit IRC | 02:38 | |
| *** ericyoung has joined #openstack-cinder | 02:41 | |
| *** crushil has quit IRC | 02:52 | |
| *** crushil has joined #openstack-cinder | 02:53 | |
| *** nicolasbock_ has quit IRC | 03:00 | |
| *** nicolasbock has quit IRC | 03:01 | |
| *** edmondsw has joined #openstack-cinder | 03:04 | |
| *** edmondsw has quit IRC | 03:09 | |
| *** felipemonteiro_ has joined #openstack-cinder | 03:12 | |
| openstackgerrit | Merged openstack/cinder master: Dell EMC PS: Fix over-subscription ratio stats https://review.openstack.org/507647 | 03:17 |
| *** markvoelker has joined #openstack-cinder | 03:24 | |
| *** wes_dillingham has quit IRC | 03:29 | |
| *** wes_dillingham has joined #openstack-cinder | 03:30 | |
| *** wes_dillingham has quit IRC | 03:30 | |
| *** links has joined #openstack-cinder | 03:32 | |
| *** mriedem has joined #openstack-cinder | 03:35 | |
| *** abhitechie has quit IRC | 03:36 | |
| *** mriedem has quit IRC | 03:42 | |
| *** gouthamr has quit IRC | 03:42 | |
| *** crushil has quit IRC | 03:52 | |
| *** crushil has joined #openstack-cinder | 03:53 | |
| *** markvoelker has quit IRC | 03:57 | |
| *** gkadam has joined #openstack-cinder | 04:01 | |
| *** arnewiebalck_ has joined #openstack-cinder | 04:05 | |
| *** armax has quit IRC | 04:23 | |
| *** felipemonteiro_ has quit IRC | 04:49 | |
| openstackgerrit | Merged openstack/cinder master: docs: Fix typo error in blockstorage-driver-filter-weighing.rst https://review.openstack.org/504304 | 04:49 |
| pooja_jadhav | diablo_rojo: Hi nelson, Please help me to review these patches. https://review.openstack.org/#/c/434894/ and https://review.openstack.org/#/c/434898/ | 04:51 |
| *** edmondsw has joined #openstack-cinder | 04:52 | |
| *** markvoelker has joined #openstack-cinder | 04:54 | |
| *** edmondsw has quit IRC | 04:57 | |
| *** markstur has quit IRC | 05:02 | |
| *** markstur has joined #openstack-cinder | 05:03 | |
| *** psachin has joined #openstack-cinder | 05:05 | |
| *** lpetrut_ has joined #openstack-cinder | 05:06 | |
| *** markstur has quit IRC | 05:08 | |
| *** arnewiebalck_ has quit IRC | 05:15 | |
| *** jwcroppe has quit IRC | 05:15 | |
| *** markvoelker has quit IRC | 05:27 | |
| *** kwathore has joined #openstack-cinder | 05:27 | |
| *** ChubYann has quit IRC | 05:34 | |
| *** crushil has quit IRC | 05:37 | |
| openstackgerrit | Merged openstack/cinder master: Doc: Configuration: Remove some outdated Ceph info https://review.openstack.org/509207 | 05:49 |
| *** e0ne has joined #openstack-cinder | 05:55 | |
| *** spectr has joined #openstack-cinder | 05:56 | |
| *** e0ne has quit IRC | 06:02 | |
| *** jwcroppe has joined #openstack-cinder | 06:03 | |
| *** lpetrut_ has quit IRC | 06:06 | |
| *** hemna_ has joined #openstack-cinder | 06:07 | |
| *** hemna__ has quit IRC | 06:11 | |
| *** jwcroppe has quit IRC | 06:14 | |
| *** rcernin has joined #openstack-cinder | 06:17 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: [WIP] Adds Overlay Volumes Created from Snapshots to Quobyte https://review.openstack.org/507050 | 06:20 |
| *** markvoelker has joined #openstack-cinder | 06:24 | |
| *** pcaruana has joined #openstack-cinder | 06:24 | |
| *** pgadiya has joined #openstack-cinder | 06:27 | |
| *** pgadiya has quit IRC | 06:27 | |
| *** e0ne has joined #openstack-cinder | 06:31 | |
| *** rcernin has quit IRC | 06:41 | |
| *** links has quit IRC | 06:42 | |
| *** rcernin has joined #openstack-cinder | 06:43 | |
| *** e0ne has quit IRC | 06:46 | |
| *** isaacb has joined #openstack-cinder | 06:46 | |
| *** swamireddy has quit IRC | 06:55 | |
| *** markvoelker has quit IRC | 06:58 | |
| *** wanghao has joined #openstack-cinder | 06:59 | |
| *** links has joined #openstack-cinder | 06:59 | |
| *** swamireddy has joined #openstack-cinder | 07:04 | |
| *** jwcroppe has joined #openstack-cinder | 07:05 | |
| *** wanghao has quit IRC | 07:09 | |
| *** tesseract has joined #openstack-cinder | 07:25 | |
| *** wanghao has joined #openstack-cinder | 07:27 | |
| *** jwcroppe has quit IRC | 07:27 | |
| *** isaacb has quit IRC | 07:30 | |
| *** wanghao has quit IRC | 07:33 | |
| *** wanghao has joined #openstack-cinder | 07:33 | |
| *** wanghao has quit IRC | 07:34 | |
| *** wanghao has joined #openstack-cinder | 07:34 | |
| *** wanghao has quit IRC | 07:35 | |
| *** links has quit IRC | 07:46 | |
| *** e0ne has joined #openstack-cinder | 07:47 | |
| *** markvoelker has joined #openstack-cinder | 07:54 | |
| *** mlakat has quit IRC | 07:56 | |
| *** links has joined #openstack-cinder | 07:59 | |
| *** mlakat has joined #openstack-cinder | 08:28 | |
| *** markvoelker has quit IRC | 08:28 | |
| *** edmondsw has joined #openstack-cinder | 08:28 | |
| openstackgerrit | Alexandru Muresan proposed openstack/cinder master: RemoteFS: revert snapshot support https://review.openstack.org/510016 | 08:33 |
| *** edmondsw has quit IRC | 08:33 | |
| *** psachin has quit IRC | 08:34 | |
| *** jwcroppe has joined #openstack-cinder | 08:49 | |
| *** jwcroppe has quit IRC | 08:58 | |
| *** dalgaaf has quit IRC | 09:18 | |
| *** ociuhandu has joined #openstack-cinder | 09:18 | |
| *** psachin has joined #openstack-cinder | 09:18 | |
| *** dalgaaf has joined #openstack-cinder | 09:19 | |
| *** ociuhandu has quit IRC | 09:21 | |
| *** markvoelker has joined #openstack-cinder | 09:25 | |
| *** spectr has quit IRC | 09:26 | |
| ArchiFleKs | Hi, I'm running into this in latest master when running cinder-manage db online_data_migrations (http://paste.openstack.org/show/622815/) | 09:28 |
| ArchiFleKs | TypeError: Message objects do not support addition. | 09:28 |
| *** spectr has joined #openstack-cinder | 09:34 | |
| *** spectr has quit IRC | 09:34 | |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/cinder master: Cleanup test-requirements https://review.openstack.org/510033 | 09:36 |
| *** spectr has joined #openstack-cinder | 09:36 | |
| *** spectr has quit IRC | 09:36 | |
| *** spectr has joined #openstack-cinder | 09:37 | |
| *** spectr has quit IRC | 09:38 | |
| *** spectr has joined #openstack-cinder | 09:40 | |
| *** spectr has quit IRC | 09:40 | |
| *** spectr has joined #openstack-cinder | 09:41 | |
| *** spectr has quit IRC | 09:43 | |
| *** spectr has joined #openstack-cinder | 09:43 | |
| *** kwathore_ has joined #openstack-cinder | 09:52 | |
| *** kwathore has quit IRC | 09:52 | |
| *** spectr has quit IRC | 09:56 | |
| *** spectr has joined #openstack-cinder | 09:57 | |
| *** spectr has quit IRC | 09:57 | |
| *** spectr has joined #openstack-cinder | 09:58 | |
| *** markvoelker has quit IRC | 09:58 | |
| *** stephenfin is now known as finucannot | 10:05 | |
| *** edmondsw has joined #openstack-cinder | 10:16 | |
| *** edmondsw has quit IRC | 10:21 | |
| *** sdague has joined #openstack-cinder | 10:28 | |
| *** Dinesh_Bhor has quit IRC | 10:35 | |
| *** jwcroppe has joined #openstack-cinder | 10:39 | |
| *** jwcroppe has quit IRC | 10:47 | |
| *** ganso has joined #openstack-cinder | 10:48 | |
| *** savvas has joined #openstack-cinder | 10:55 | |
| *** markvoelker has joined #openstack-cinder | 10:56 | |
| *** nicolasbock has joined #openstack-cinder | 11:10 | |
| *** nicolasbock_ has joined #openstack-cinder | 11:10 | |
| *** nicolasbock_ has quit IRC | 11:15 | |
| *** nicolasbock has quit IRC | 11:15 | |
| *** dave-mccowan has joined #openstack-cinder | 11:20 | |
| *** nicolasbock has joined #openstack-cinder | 11:27 | |
| *** nicolasbock_ has joined #openstack-cinder | 11:27 | |
| *** nicolasbock_ has quit IRC | 11:29 | |
| *** nicolasbock has quit IRC | 11:29 | |
| *** markvoelker has quit IRC | 11:29 | |
| *** takedakn has joined #openstack-cinder | 11:34 | |
| *** abishop has joined #openstack-cinder | 11:52 | |
| *** spectr has quit IRC | 12:02 | |
| *** spectr has joined #openstack-cinder | 12:03 | |
| *** edmondsw has joined #openstack-cinder | 12:05 | |
| *** wes_dillingham has joined #openstack-cinder | 12:07 | |
| *** edmondsw has quit IRC | 12:09 | |
| hemna_ | It's Friday peeps! | 12:17 |
| *** tpsilva has joined #openstack-cinder | 12:19 | |
| *** MVenesio has joined #openstack-cinder | 12:21 | |
| *** nicolasbock has joined #openstack-cinder | 12:22 | |
| *** markvoelker has joined #openstack-cinder | 12:26 | |
| *** jwcroppe has joined #openstack-cinder | 12:28 | |
| *** spectr has quit IRC | 12:28 | |
| *** jdwidari has joined #openstack-cinder | 12:30 | |
| *** markvoelker has quit IRC | 12:31 | |
| *** markvoelker has joined #openstack-cinder | 12:31 | |
| *** jwcroppe has quit IRC | 12:36 | |
| *** spectr has joined #openstack-cinder | 12:39 | |
| *** nicolasbock has quit IRC | 12:39 | |
| *** mchlumsky has joined #openstack-cinder | 12:40 | |
| *** erlon has joined #openstack-cinder | 12:41 | |
| *** alexchadin has joined #openstack-cinder | 12:45 | |
| *** takedakn has quit IRC | 12:47 | |
| *** nicolasbock has joined #openstack-cinder | 12:51 | |
| *** mchlumsky has quit IRC | 12:56 | |
| *** takedakn has joined #openstack-cinder | 12:58 | |
| *** mchlumsky has joined #openstack-cinder | 13:01 | |
| *** crushil has joined #openstack-cinder | 13:02 | |
| e0ne | hemna_: it's almost Friday's evening! | 13:04 |
| *** psachin has quit IRC | 13:05 | |
| *** nicolasbock has quit IRC | 13:05 | |
| *** nicolasbock has joined #openstack-cinder | 13:05 | |
| *** takedakn has quit IRC | 13:05 | |
| *** takedakn has joined #openstack-cinder | 13:07 | |
| *** eharney has joined #openstack-cinder | 13:12 | |
| *** lbragstad has joined #openstack-cinder | 13:15 | |
| *** dustins has joined #openstack-cinder | 13:17 | |
| *** takedakn has quit IRC | 13:18 | |
| *** gkadam has quit IRC | 13:18 | |
| *** dansmith is now known as superdan | 13:28 | |
| *** jmlowe_ has quit IRC | 13:29 | |
| *** jmlowe has joined #openstack-cinder | 13:30 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove Reduxio volume driver https://review.openstack.org/501332 | 13:32 |
| *** jmlowe_ has joined #openstack-cinder | 13:33 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove Tegile volume driver https://review.openstack.org/501333 | 13:35 |
| *** spectr has quit IRC | 13:35 | |
| *** jmlowe has quit IRC | 13:35 | |
| *** lbragstad has quit IRC | 13:36 | |
| *** lbragstad has joined #openstack-cinder | 13:36 | |
| *** spectr has joined #openstack-cinder | 13:36 | |
| *** mriedem has joined #openstack-cinder | 13:39 | |
| *** gouthamr has joined #openstack-cinder | 13:39 | |
| *** wes_dillingham has quit IRC | 13:45 | |
| *** wes_dillingham has joined #openstack-cinder | 13:46 | |
| *** links has quit IRC | 13:48 | |
| *** edmondsw has joined #openstack-cinder | 13:53 | |
| *** jdwidari has quit IRC | 13:54 | |
| *** jwcroppe has joined #openstack-cinder | 13:55 | |
| * hemna_ is ceph-ing | 13:57 | |
| *** edmondsw has quit IRC | 13:57 | |
| smcginnis | I've heard of that. | 13:57 |
| hemna_ | cephilus ? | 13:57 |
| hemna_ | :P | 13:57 |
| smcginnis | Hah! | 13:57 |
| *** catintheroof has joined #openstack-cinder | 14:05 | |
| *** catinthe_ has joined #openstack-cinder | 14:06 | |
| *** catintheroof has quit IRC | 14:07 | |
| *** spectr has quit IRC | 14:11 | |
| *** alexchadin has quit IRC | 14:15 | |
| *** gouthamr has quit IRC | 14:16 | |
| *** savvas has quit IRC | 14:19 | |
| *** felipemonteiro_ has joined #openstack-cinder | 14:21 | |
| *** alexchadin has joined #openstack-cinder | 14:22 | |
| *** felipemonteiro__ has joined #openstack-cinder | 14:22 | |
| *** alexchadin has quit IRC | 14:23 | |
| *** felipemonteiro_ has quit IRC | 14:26 | |
| *** armax has joined #openstack-cinder | 14:26 | |
| jungleboyj | TGIF ... been quite the week. | 14:36 |
| * jungleboyj doesn't want to get cephilus from hemna_ | 14:36 | |
| *** ntpttr_laptop has joined #openstack-cinder | 14:50 | |
| hemna_ | it spreads through irc man....you probably already have it. | 14:51 |
| *** markstur has joined #openstack-cinder | 14:52 | |
| smcginnis | Anyone know under what conditions an LVM thin pool is "suspended"? | 14:55 |
| hemna_ | overprovisioned? | 14:55 |
| smcginnis | Thinly provisioned, but shouldn't be overprovisioned. Happening in our tests, and I don't think we really have anything that actually writes much data. | 14:57 |
| *** ntpttr_laptop has quit IRC | 14:58 | |
| twouters | google says that the origin device is suspended during snapshots :) | 15:01 |
| jungleboyj | twouters: You googled better than I. | 15:02 |
| smcginnis | twouters: Oh, hmm. So maybe it is still in the process of creating the snapshot when we try to activate it. Thanks! | 15:02 |
| smcginnis | That could be a good clue. | 15:02 |
| smcginnis | We may need to add a check in there to wait for completion. | 15:02 |
| smcginnis | Could also explain the intermittent nature of the failures we've been seeing. | 15:03 |
| twouters | jungleboyj: "lvm thin suspend" :p | 15:03 |
| *** spectr has joined #openstack-cinder | 15:04 | |
| *** spectr has quit IRC | 15:04 | |
| *** links has joined #openstack-cinder | 15:07 | |
| *** gouthamr has joined #openstack-cinder | 15:07 | |
| *** mriedem is now known as ronlund | 15:09 | |
| *** MVenesio has quit IRC | 15:16 | |
| *** rcernin has quit IRC | 15:19 | |
| Swanson | "cephilus" is genius. | 15:22 |
| *** shvepsy has quit IRC | 15:25 | |
| *** shvepsy has joined #openstack-cinder | 15:25 | |
| *** jwcroppe has quit IRC | 15:28 | |
| *** jwcroppe has joined #openstack-cinder | 15:29 | |
| *** savvas has joined #openstack-cinder | 15:29 | |
| *** jwcroppe has quit IRC | 15:33 | |
| *** savvas has quit IRC | 15:34 | |
| *** jwcroppe has joined #openstack-cinder | 15:36 | |
| *** edmondsw has joined #openstack-cinder | 15:41 | |
| *** ildikov is now known as coffee_cat | 15:45 | |
| *** edmondsw has quit IRC | 15:45 | |
| *** markstur_ has joined #openstack-cinder | 15:59 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add retries to LVM logical volume activation https://review.openstack.org/510168 | 16:00 |
| smcginnis | Optimistically marking review 510168 as "Closes-bug". If that merges and we still see this I will reopen. | 16:01 |
| *** Apoorva has joined #openstack-cinder | 16:01 | |
| smcginnis | jungleboyj, hemna_, e0ne, all: Was hoping for a more deterministic fix, but I think that might do it. ^^ | 16:02 |
| *** Apoorva has quit IRC | 16:02 | |
| smcginnis | I'd like to get that pushed through and see if we observe any improvement. | 16:02 |
| *** Apoorva has joined #openstack-cinder | 16:02 | |
| jungleboyj | smcginnis: Let me look. | 16:02 |
| *** markstur has quit IRC | 16:02 | |
| jgriffith | smcginnis in the past we've introduced some forced delays to pin that down, might be worth trying again | 16:03 |
| smcginnis | We can also check out the logs for any sign that it has been doing the retries and succeeding. That could give us a little more assurance it's actually working. | 16:03 |
| smcginnis | jgriffith: I thought about putting in some actual greenthread.sleep(), but figured with the retry we wouldn't be blocking if we didn't need to be. | 16:03 |
| smcginnis | jgriffith: But that may be another option. | 16:03 |
| jgriffith | specifically the lvm calls rapid fire from tempest sometimes get a bit hay wire, so we would introduce a delay after a create of a few seconds | 16:03 |
| jgriffith | yeah, the retry thing is funky though with device-mapper | 16:04 |
| jgriffith | sometimes you bork device-mapper to the point where it doesn't recover, or it waits for it's time out in some cases before letting you do anything | 16:04 |
| smcginnis | jgriffith: We could if do "if lvm_type == thin: sleep" | 16:04 |
| jgriffith | which IIRC the default is like 30 seconds or something :( | 16:04 |
| jgriffith | smcginnis that's not a bad idea | 16:04 |
| jgriffith | the sleep if thin | 16:04 |
| smcginnis | Hmm. I put the backoff delay to 2 seconds with 5 retries thinking that could be enough, but maybe we want to make that even biggerer if needed. | 16:05 |
| jgriffith | nah, I'd try that first | 16:05 |
| smcginnis | Do like a 10 second backoff. Then by the 5th attempt it could wait long enough for DM to get straight. | 16:05 |
| jgriffith | well, you could start with what you suggested there and collect some data, then go up if needed. Or start the other way and work down | 16:06 |
| jgriffith | you're call, six one, half dozen the other I guess | 16:06 |
| smcginnis | jgriffith: I wanted to start a little conservative so we didn't introduce too big of a slow down if it wasn't actually needed. But yeah, let's see how it goes. | 16:07 |
| jgriffith | i don't think in this case you're hitting the dm or lvm internal timeout mechanisms though FWIW | 16:07 |
| smcginnis | Yeah, if it's 30 seconds this is going to fail before then. | 16:07 |
| jgriffith | lvm activiation has been a bit of a pain ever since it was introduced | 16:07 |
| smcginnis | Really seems like it shouldn't be this difficult. | 16:08 |
| jgriffith | we fumbled around like this quite a bit | 16:08 |
| jgriffith | I know right!!! | 16:08 |
| smcginnis | Fingers crossed this will get us past it. If so, we should probably backport to pike. | 16:08 |
| smcginnis | Not really happy this has been around for so long and we've just bypassed it by having devstack switch things to thick provisioned. :/ | 16:09 |
| jungleboyj | smcginnis: Thank you for digging into this. | 16:10 |
| smcginnis | Maybe we should take the entire local_dev/lvm file and put a retry with a 30 second delay on every call. :D | 16:10 |
| smcginnis | jungleboyj: I don't know if you've been getting any of it, but I've been getting more and more grumbling from others about the test failures, so I'm hoping we can get past this. | 16:11 |
| jgriffith | LULZ | 16:11 |
| jgriffith | Let me know if you need help on it, I may get my stuff fixed and finished today | 16:11 |
| smcginnis | jgriffith: We're actually almost to that point if you scan through that file. :) | 16:11 |
| jgriffith | we need to rewrite all of that :) | 16:11 |
| smcginnis | jgriffith: Cool, we'll see how it goes. | 16:11 |
| smcginnis | jgriffith: If we still get failures, I'd love to have your eyes on it with your LVM experience. I know it's much more than mine. | 16:12 |
| jgriffith | It's too bad, once a year or so we have these crazy intermittent LVM issues that just kill us | 16:12 |
| smcginnis | Would be great if there was a good API for interacting with LVM. Other than running commands and parsing output. | 16:13 |
| *** pcaruana has quit IRC | 16:13 | |
| *** jwcroppe has quit IRC | 16:13 | |
| jgriffith | smcginnis https://pypi.python.org/pypi/lvm2py/ | 16:13 |
| smcginnis | jgriffith: Have you played around with that at all? If it's the one I looked at awhile back, it didn't seem like it was very active. | 16:14 |
| jgriffith | There are several I have found and messed with over the years, but none seemed that great | 16:14 |
| smcginnis | Uh, yeah - Uploaded on 2012-06-08 | 16:14 |
| smcginnis | I think LVM has had a few tweaks since then. | 16:14 |
| jgriffith | yeah, that's the problem... most are about 5 years old | 16:15 |
| smcginnis | I suppose we could fork it and make it into what we need it to be. | 16:15 |
| smcginnis | Would take some work - "the resize of Physical and Logical volumes is not implemented in the api yet" | 16:16 |
| *** xyang1 has joined #openstack-cinder | 16:16 | |
| jungleboyj | smcginnis: I haven't been getting that, but it clearly has become insane. | 16:16 |
| *** xyang1 has quit IRC | 16:16 | |
| *** jwcroppe has joined #openstack-cinder | 16:16 | |
| *** xyang1 has joined #openstack-cinder | 16:17 | |
| smcginnis | Definitely picked up: http://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22device-mapper:%20reload%20ioctl%20on%5C%22%20AND%20%20%20message:%5C%22failed:%20Invalid%20argument%5C%22%20AND%20%20%20tags:%5C%22screen-c-vol.txt%5C%22%20AND%20voting:1&from=7d | 16:17 |
| jungleboyj | smcginnis: Yeah, I saw that yesterday. | 16:17 |
| coffee_cat | smcginnis: problem with forks is you need to maintain it, or convince jgriffith to do that for you as he's advocating on LVM needing some love from all of us :) | 16:17 |
| jgriffith | coffee_cat or just *steal it* and adopt it as your own offspring | 16:18 |
| coffee_cat | jgriffith: if it fits, then why not? | 16:19 |
| smcginnis | It at least fits in and would be used by Cinder. But yeah, we don't really need to look for more things to maintain. | 16:19 |
| * jgriffith has pretty much given up on the dream of many people caring about LVM | 16:19 | |
| coffee_cat | jgriffith: sadness :( | 16:19 |
| jgriffith | there's definitely a few but not as many as there should be | 16:19 |
| smcginnis | We need to start selling an LVM Cinder appliance. ;) | 16:20 |
| jungleboyj | I think I am going to be using LVM for the cloud I am putting together. :-) | 16:20 |
| coffee_cat | smcginnis: it only takes good marketing :) | 16:20 |
| smcginnis | Just take one of these and preconfigure it with c-vol: https://www.supermicro.com/products/nfo/NVMe.cfm | 16:21 |
| * coffee_cat is hearing jungleboyj volunteering for maintenance :) | 16:21 | |
| jgriffith | smcginnis +1 | 16:21 |
| smcginnis | Plug into your rack and perform a couple config steps. Cloud storage ready to go. | 16:21 |
| jungleboyj | :-) | 16:22 |
| coffee_cat | smcginnis: that brings up old memories of changing a motherboard in a supermicro server and I couldn't find cables anywhere that would've fit, but at least everyone reminded me supermicro is terrible on the way... :) | 16:23 |
| smcginnis | coffee_cat: Really? I've had good luck with them. | 16:23 |
| smcginnis | My worst was something like a 6U Sun server I maintained. Had to replace the motherboard. | 16:23 |
| smcginnis | Had to go out and find a super long screwdriver to be able to get to one of the screws. | 16:24 |
| coffee_cat | smcginnis: it was a success story at the end, and trying to find a cable the night before Christmas might've been a bit ambitious of me as well :) | 16:24 |
| smcginnis | coffee_cat: Haha, bad timing. | 16:24 |
| coffee_cat | smcginnis: lol, if even the screwdriver doesn't fit, that really sucks... :/ :) | 16:25 |
| coffee_cat | sorry, didn't want to hijack the chat | 16:25 |
| smcginnis | Ease of maintenance was definitely not on their minds when they designed it. | 16:26 |
| coffee_cat | I don't think that's ever on the minds for real TBH... | 16:27 |
| *** dikonoor has quit IRC | 16:28 | |
| smcginnis | I've actually been impressed with some of them now with tool-less access and things. | 16:28 |
| smcginnis | Even rack rails have gotten so much better that just snap in. | 16:28 |
| coffee_cat | oh nice! | 16:30 |
| coffee_cat | we should adapt that in software too :) | 16:30 |
| smcginnis | ;) | 16:31 |
| *** dikonoor has joined #openstack-cinder | 16:38 | |
| Swanson | I'll use LVM when I make my cloud running on the 12 unneeded cores of the thread ripper CPU I want to buy for no good reason. | 16:38 |
| smcginnis | Working well with my Ryzen 1700. :) | 16:41 |
| Swanson | puny 8 cores? | 16:41 |
| smcginnis | Two threads per core though. | 16:41 |
| smcginnis | My 32G RAM is the more limiting factor. | 16:42 |
| Swanson | Yeah. I love when you get a 32 core server and they give you 32GB. Like, yay! | 16:44 |
| openstackgerrit | Merged openstack/cinder master: Remove Reduxio volume driver https://review.openstack.org/501332 | 16:45 |
| Swanson | I've powered none of your stuff up in a month. It's like you never worked here. | 16:45 |
| Swanson | :) | 16:45 |
| smcginnis | Hah | 16:46 |
| smcginnis | If no one wants that stack of servers, I can pull up to the loading dock in about 30 minutes. | 16:46 |
| openstackgerrit | Sorin Sbarnea proposed openstack/cinder master: bump pylint to recent version https://review.openstack.org/509755 | 16:46 |
| *** wes_dillingham has quit IRC | 16:47 | |
| Swanson | Oh, when I get a chance I'm having your's and Mike's moved into a rack where I will install Openstack on a huge cluster and run one single instance of ubuntu on it. | 16:48 |
| Swanson | Unseen performance art is the new performance art. | 16:48 |
| smcginnis | There's massive compute power sitting idle in that lab. I think you need to start mining for bitcoin or something. | 16:49 |
| smcginnis | Or process all of Seti@Home over the weekend. | 16:50 |
| * hemna_ misses the days of using screen for devstack | 17:00 | |
| hemna_ | seriously hate systemd | 17:00 |
| Swanson | hemna, In openstack years devstack was 60 years ago. | 17:01 |
| Swanson | Sorry, screen | 17:01 |
| hemna_ | Swanson, you had me there for a second...I was like....did everyone move to containers for dev already? | 17:02 |
| hemna_ | :P | 17:02 |
| Swanson | hemna, That's coming up in 60 openstack years (or 6 months). | 17:02 |
| *** melwitt is now known as jgwentworth | 17:02 | |
| *** openstackstatus has quit IRC | 17:05 | |
| *** openstack has joined #openstack-cinder | 17:09 | |
| *** ChanServ sets mode: +o openstack | 17:09 | |
| *** ntpttr_laptop has joined #openstack-cinder | 17:17 | |
| *** edmondsw has joined #openstack-cinder | 17:29 | |
| *** links has quit IRC | 17:32 | |
| *** edmondsw has quit IRC | 17:33 | |
| *** MVenesio has joined #openstack-cinder | 17:40 | |
| *** armax has quit IRC | 17:44 | |
| openstackgerrit | Merged openstack/cinder master: Fix tempest test revet_client typo https://review.openstack.org/508995 | 17:44 |
| *** oomichi_afk is now known as oomichi | 17:44 | |
| *** MVenesio has quit IRC | 17:45 | |
| *** ntpttr_laptop has quit IRC | 17:45 | |
| *** ntpttr_laptop has joined #openstack-cinder | 17:45 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Fix migration 112 to use live_data_migration API https://review.openstack.org/510201 | 18:07 |
| jgriffith | smcginnis jungleboyj show some love: https://review.openstack.org/#/c/509622/ | 18:13 |
| jgriffith | It's good, I totally P-Promise | 18:13 |
| jungleboyj | jgriffith: Your studder makes me nervous. ;-) | 18:14 |
| smcginnis | jgriffith: Haha, looked fine from me, but I haven't had a chance to try it out. I mostly trust you though. :) | 18:14 |
| jgriffith | smcginnis LOL... https://review.openstack.org/510201 is EXACTLY why no one should trust me!!! | 18:15 |
| jgriffith | well.. you should trust me on 510201! | 18:15 |
| *** lbragstad has quit IRC | 18:16 | |
| jgriffith | You just shouldn't trust me due to the fact that I created the thing that needed fixing in the first place | 18:16 |
| * jgriffith should probably just stop talking now | 18:16 | |
| smcginnis | But oh noes, what about all those people that have already deployed that using CD. We need to support that for the next 5 years and force everyone else to suffer because there might be one person out there that deployed it already. | 18:16 |
| smcginnis | Too snarky? | 18:16 |
| jgriffith | smcginnis screw-em | 18:17 |
| jgriffith | although on the bright side, it won't hurt them, it just becomes a noop | 18:17 |
| jgriffith | smcginnis and you can NEVER be "too snarky" | 18:17 |
| jgriffith | especially on that particular topic :) | 18:18 |
| smcginnis | :) | 18:18 |
| jgriffith | I didn't figure out the weird i18n thing though | 18:18 |
| jgriffith | which is annoying | 18:18 |
| smcginnis | jgriffith: Which was that? | 18:18 |
| coffee_cat | jgriffith: what's the exact issue with that there? | 18:19 |
| coffee_cat | jgriffith: just for my education | 18:19 |
| jgriffith | for some reason if I put the i18n symbol in front of the prettyprint items (cmd/manage.py) it raises | 18:19 |
| jgriffith | I put a comment in the code on that patch with the error | 18:19 |
| jgriffith | and a "FIXME", to say "yeah, I know but it doesn't work and I don't know how to fix it right now" | 18:20 |
| jgriffith | maybe next week | 18:20 |
| coffee_cat | is that used anywhere else for those prettyprint tables? | 18:20 |
| smcginnis | Oh right, saw that. | 18:20 |
| jgriffith | or next year | 18:20 |
| coffee_cat | smcginnis: that was a +2 of trust :) | 18:21 |
| smcginnis | Eh, a +2 of "good enough". ;) | 18:21 |
| jgriffith | hey... hold on now, I worked hard on that :) | 18:22 |
| coffee_cat | smcginnis: I know, I read slow... :) | 18:22 |
| smcginnis | jgriffith: I'm guessing because _() doesn't actually return a string, so pprint is probably like what the hell is this? | 18:22 |
| coffee_cat | jgriffith: no one said you didn't :) | 18:22 |
| smcginnis | I think. | 18:23 |
| smcginnis | But meh. | 18:23 |
| jgriffith | smcginnis indeed, but Nova made it work and it really bothered me | 18:23 |
| smcginnis | Hmm, well we can't have that. | 18:24 |
| jgriffith | coffee_cat to answer your earlier question "no" we're not using it elsewhere in Cinder, we do use in the client but we don't i18n it there either | 18:24 |
| jgriffith | smcginnis :) | 18:24 |
| openstackgerrit | Jon Bernard proposed openstack/cinder-specs master: Update cheesecake promotion specification https://review.openstack.org/408830 | 18:24 |
| *** lbragstad has joined #openstack-cinder | 18:25 | |
| jgriffith | Think I'll get lunch before reworking the actual patches I wanted to get done this week :( | 18:25 |
| *** markstur_ has quit IRC | 18:28 | |
| coffee_cat | jgriffith: it's weird though that you removed one column and renamed two more and i18 freaked out from that but not the original version... | 18:28 |
| *** psachin has joined #openstack-cinder | 18:29 | |
| jgriffith | coffee_cat ? | 18:29 |
| jgriffith | No... that's not why it errors out | 18:29 |
| coffee_cat | I didn't fully put together the pieces yet | 18:30 |
| coffee_cat | just tried to follow some of the changes | 18:30 |
| jgriffith | coffee_cat so line #313 - 315 | 18:30 |
| jgriffith | [_('Migration'), | 18:31 |
| jgriffith | _('Total Needed'), | 18:31 |
| jgriffith | _('Completed')] | 18:31 |
| jgriffith | in other words, sending the table headers through the translator | 18:31 |
| jgriffith | I believe I could work around the type error by translating them into a variable | 18:32 |
| jgriffith | but honestly don't know if it's worth messing with | 18:32 |
| jgriffith | or if it's the "right" thing to do | 18:32 |
| jgriffith | and like I said, it's annoying that it works elsewhere and I don't know what's different | 18:33 |
| coffee_cat | but if you check the base on the review it looks like it was translated | 18:33 |
| smcginnis | I don't think we translate CLI stuff normally. | 18:33 |
| jgriffith | anyway, it's not really worth any more of my CPU cycles | 18:33 |
| jgriffith | smcginnis that's good enough for me | 18:33 |
| smcginnis | Obsess and speculate. It's the only way. | 18:33 |
| jgriffith | I shouldn't have mentioned it here or in the comments :) | 18:33 |
| jgriffith | because now it'll be a *thing* for at least 30 minutes | 18:34 |
| coffee_cat | jgriffith: fair enough, go get lunch, I can let this go :) | 18:34 |
| smcginnis | Oh, it was translated before. Hmm. Oh well. | 18:34 |
| smcginnis | Go eat. That's more important. | 18:34 |
| coffee_cat | smcginnis: yeah, that got me thinking as I would guess that the test should've caught that even if nothing else used this thing | 18:35 |
| coffee_cat | smcginnis: or maybe even that didn't... | 18:35 |
| smcginnis | ¯\_(ツ)_/¯ | 18:36 |
| jgriffith | smcginnis wait what? | 18:37 |
| jgriffith | oh | 18:37 |
| jgriffith | no... see so funny thing | 18:37 |
| jgriffith | there was a bunch of online_data_migration code in there | 18:37 |
| jgriffith | but it was never called | 18:37 |
| jgriffith | unit tests are "unit tests" and they cheat | 18:37 |
| jgriffith | beause as per usual we just mock away all the things | 18:38 |
| smcginnis | Shocked. :) | 18:38 |
| jgriffith | I know!! It's hard to believe | 18:38 |
| coffee_cat | jgriffith: yeah, that was my guess | 18:40 |
| coffee_cat | jgriffith: sorry to keep you from eating because of this, my bad | 18:40 |
| coffee_cat | as for the tests, sadness... | 18:40 |
| smcginnis | coffee_cat: He got donuts the other day. He has enough calories to survive for awhile. ;) | 18:40 |
| coffee_cat | coffee_cat: haha, you're right! | 18:41 |
| coffee_cat | coffee_cat: let me think then how we could increase jgriffith's heart rate to burn those extra calories :) | 18:42 |
| coffee_cat | I shouldn't drink more coffee I guess, lol | 18:42 |
| smcginnis | _shouldn't_ drink more coffee? Are you OK? | 18:43 |
| coffee_cat | smcginnis: well, as I'm talking to myself in public now I guess I'm not... :) | 18:47 |
| jungleboyj | A healthy person shouldn't say they shouldn't drink more coffee. ;-) | 18:47 |
| *** savvas has joined #openstack-cinder | 18:48 | |
| openstackgerrit | Dhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation https://review.openstack.org/433341 | 18:50 |
| openstackgerrit | Dhinesh Balasubramaniam proposed openstack/cinder master: Review comments fixed https://review.openstack.org/510210 | 18:50 |
| hemna_ | anyone see this trying to stand up cinder rbd driver ? | 19:05 |
| hemna_ | http://paste.openstack.org/show/622880/ | 19:05 |
| *** chlong has joined #openstack-cinder | 19:06 | |
| openstackgerrit | Dhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation https://review.openstack.org/433341 | 19:11 |
| *** mchlumsky has quit IRC | 19:12 | |
| *** mchlumsky has joined #openstack-cinder | 19:14 | |
| *** edmondsw has joined #openstack-cinder | 19:17 | |
| *** mchlumsky has quit IRC | 19:20 | |
| *** edmondsw has quit IRC | 19:22 | |
| *** wes_dillingham has quit IRC | 19:24 | |
| *** ChubYann has joined #openstack-cinder | 19:26 | |
| hemna_ | fwiw my ceph keyring file was empty | 19:26 |
| hemna_ | that fixed it | 19:26 |
| hemna_ | very bad error for what the problem was | 19:26 |
| smcginnis | hemna_: Wow, not even a clue to point in the general right area to troubleshoot. | 19:28 |
| openstackgerrit | Jon Bernard proposed openstack/cinder master: RBD: support driver-assisted volume migration https://review.openstack.org/256091 | 19:28 |
| smcginnis | jgriffith, hemna_: Would be great to see if this gets rid of our LVM issues: https://review.openstack.org/#/c/510168/ | 19:29 |
| jungleboyj | smcginnis: You want me to just merge it? | 19:30 |
| smcginnis | jungleboyj: That's good to me. | 19:30 |
| smcginnis | We can always revert or try something in addition to this if it ends up not fixing the job failures. | 19:30 |
| jungleboyj | It isn't going to make anything worse. You reached out earlier. | 19:30 |
| smcginnis | Right | 19:31 |
| *** tesseract has quit IRC | 19:31 | |
| jungleboyj | I just wasn't sure after the earlier discussion if there was more you wanted to do, but I am good starting here. | 19:31 |
| smcginnis | Things won't suck any more than they do now. ;) | 19:31 |
| *** hemna_ has quit IRC | 19:31 | |
| smcginnis | I'm hoping this clears up the failures. No immediate plans to do more unless we find the need to do so. | 19:31 |
| smcginnis | Double whammy! :) | 19:32 |
| jungleboyj | He he. | 19:33 |
| jungleboyj | Problem is John is being the strong silent type. | 19:33 |
| smcginnis | His mouth is too full with donuts for lunch. | 19:33 |
| jungleboyj | He he. | 19:34 |
| jgriffith | No donuts today!! I'm having a Kale, BlueBerry, Quiona and goat cheese salad | 19:34 |
| jungleboyj | Ok, stupid question, but is there a way to get devstack to update the install requirements for me? | 19:35 |
| * jungleboyj make a fancy oooooo sound | 19:35 | |
| smcginnis | jgriffith: We're not buying it. | 19:35 |
| jgriffith | smcginnis dang | 19:35 |
| jgriffith | they're on to me | 19:35 |
| jgriffith | jungleboyj you mean cinders install requirements? | 19:35 |
| jgriffith | Just by overriding them in the checkout as far as I know | 19:35 |
| jungleboyj | Yeah. When I run devstack right now it keeps failing because I have old versions installed and I am having to install updates by hand which doesn't seem right. | 19:36 |
| jgriffith | clean.sh maybe? | 19:36 |
| jungleboyj | Tried that. | 19:36 |
| smcginnis | pip or apt requirements? | 19:37 |
| jungleboyj | Pip. | 19:37 |
| jgriffith | jungleboyj by hand? You should just use the requirements file manually | 19:38 |
| jungleboyj | jgriffith: Teach me oh wise one. | 19:38 |
| jgriffith | "sudo pip install -r ./requirements.txt" | 19:38 |
| smcginnis | It seems to get in weird states sometimes when packages are already installed. | 19:39 |
| smcginnis | If it's failing on a specific one, manually remove the package, then run it again. | 19:39 |
| coffee_cat | smcginnis: doesn't --upgrade work for those? | 19:39 |
| jungleboyj | Dear lord. I didn't know I could just run the file like that. Thank you. | 19:39 |
| smcginnis | coffee_cat: If it's actually upgrading. | 19:39 |
| coffee_cat | smcginnis: good point | 19:40 |
| jgriffith | So... I need some git foo for my least favorite thing that I screw up EVERY time I try it | 19:41 |
| jgriffith | I have patch A in flight | 19:41 |
| jgriffith | I then created patch-B | 19:41 |
| jgriffith | It's no proposed as well | 19:41 |
| coffee_cat | cherry-pick | 19:41 |
| smcginnis | Make B dependent of A? | 19:41 |
| jgriffith | :) | 19:41 |
| jgriffith | Nope | 19:41 |
| jgriffith | Make A dep on B | 19:42 |
| jgriffith | I cerry picked B into A, that's fine | 19:42 |
| jgriffith | will it now be dep on B though when I push it to gerrit? | 19:42 |
| smcginnis | Reverse the dependencies? | 19:42 |
| coffee_cat | cherry picked B in A? | 19:42 |
| jgriffith | yes | 19:42 |
| coffee_cat | you need to cherry-pick A on top of B if you want them in a chain | 19:43 |
| coffee_cat | with A depending on B | 19:43 |
| smcginnis | But if B is already on A you'll need to take a step back. | 19:43 |
| jgriffith | luckily it is not | 19:43 |
| coffee_cat | B is proposed already so it's just a download with checkout | 19:44 |
| jgriffith | right... but!!! :) | 19:44 |
| coffee_cat | or if B would be on A then a reset --hard | 19:44 |
| smcginnis | OK, checkout B's branch, git cherry-pick $A_HASH, clean up if needed, git review. | 19:44 |
| jgriffith | https://review.openstack.org/#/c/509005/ | 19:44 |
| jgriffith | is the target here | 19:44 |
| jgriffith | the trick is that has Service object changes that are no longer needed/good | 19:45 |
| smcginnis | Just amend that one with the second commit changes and don't bother? | 19:45 |
| jgriffith | I want the changes in https://review.openstack.org/#/c/510201/ | 19:45 |
| coffee_cat | smcginnis: no, don't amend please | 19:45 |
| jgriffith | coffee_cat no? I use amend all the time | 19:45 |
| coffee_cat | jgriffith: you can soft reset B and remove the files from the commit that contain those changes | 19:46 |
| coffee_cat | jgriffith: then cherry-pick A on top of it | 19:46 |
| coffee_cat | jgriffith: I thought smcginnis means making one patch out of the two | 19:47 |
| coffee_cat | oh ok, I guess amend can work too | 19:47 |
| smcginnis | If the second one is just fixing up things in the first one, then easier just merging them into one and not bothering with dependencies. | 19:47 |
| jgriffith | You people are all crazy :) | 19:47 |
| coffee_cat | smcginnis: no, those patches are two separate things, don't merge them please | 19:48 |
| * jgriffith just makes a new patch | 19:48 | |
| coffee_cat | jgriffith: why? | 19:48 |
| smcginnis | I'll stop talking. It's probably not helping that coffee_cat and I have different approaches. | 19:48 |
| coffee_cat | jgriffith: why would you make a new patch? | 19:48 |
| jgriffith | coffee_cat why are you crazy? I dunno, could be something in your water, perhaps genetic, maybe just enviromental impacts... who knows | 19:48 |
| jgriffith | :) | 19:49 |
| openstackgerrit | Merged openstack/cinder master: Update BlockBox https://review.openstack.org/509622 | 19:49 |
| coffee_cat | coffee_cat: so you actually think I'm crazy | 19:49 |
| coffee_cat | jgriffith: ^^ | 19:49 |
| jgriffith | LOL | 19:49 |
| jgriffith | No, I'm just messing with you | 19:49 |
| coffee_cat | I log off for today | 19:49 |
| jgriffith | and actually if you scroll back you'll notice I said "YOU ALL" | 19:49 |
| coffee_cat | jgriffith: but I can fix your git issue without creating a new patch | 19:49 |
| *** markstur has joined #openstack-cinder | 19:50 | |
| coffee_cat | jgriffith: well, that includes me too :) | 19:50 |
| jgriffith | I'm just trying to be "inclusive" | 19:50 |
| jgriffith | :) | 19:50 |
| coffee_cat | oh how nice of you :) | 19:50 |
| jgriffith | ok... so; what you're suggesting is that I checkout https://review.openstack.org/#/c/510201/ | 19:51 |
| jgriffith | and I then cherry pick https://review.openstack.org/#/c/509005/ on top of it | 19:51 |
| *** savvas has quit IRC | 19:51 | |
| coffee_cat | yes | 19:51 |
| jgriffith | and then just clean up the disaster that is merge-conflicts that occur :) | 19:51 |
| coffee_cat | and as you have the service uuid changes in the first one and that's where you want it now that I think of it it should work | 19:51 |
| jgriffith | ok... I'll give it a go, what have I got to loose :) | 19:52 |
| coffee_cat | git might complains a bit when you cherry-pick you just fix it as you go | 19:52 |
| coffee_cat | it shouldn't have that many disasters | 19:52 |
| *** eharney has quit IRC | 19:52 | |
| *** arnewiebalck_ has joined #openstack-cinder | 19:54 | |
| *** markstur has quit IRC | 19:54 | |
| coffee_cat | on the other hand I hate that you do a simple rebase on a Nova patch and the test suddenly starts to complain that you're using the DB where you shouldn't... | 19:54 |
| coffee_cat | is this really a Friday??? | 19:55 |
| jgriffith | AHHHHH!!! My changes, my beautiful changes they're all gone!!!! | 19:55 |
| jgriffith | Just kidding | 19:55 |
| jgriffith | seems like that worked :) | 19:55 |
| jgriffith | thanks coffee_cat | 19:55 |
| coffee_cat | jgriffith: anytime :) | 19:55 |
| jgriffith | one thing missing though | 19:56 |
| coffee_cat | what? | 19:56 |
| jgriffith | if I were to do a "git review" as it is right now it's going to submit it as the original patch as an update no? | 19:56 |
| jgriffith | What I mean by original is if I currently do a git status, I'm on bug/1721837 branch stll | 19:57 |
| coffee_cat | it will link the two together as the ChangeId's are the same | 19:57 |
| jgriffith | that's no bueno | 19:57 |
| jgriffith | ok, we'll find out in a bit :) | 19:57 |
| coffee_cat | I mean when you push the changes | 19:57 |
| smcginnis | Yeah, just don't get rid of the change ID in the commit. | 19:57 |
| *** jwcroppe has quit IRC | 19:57 | |
| coffee_cat | or that's what's supposed to happen, but I don't use git review so it might be "smarter" than all of us... :) | 19:58 |
| *** jwcroppe has joined #openstack-cinder | 19:58 | |
| *** jwcroppe_ has joined #openstack-cinder | 20:00 | |
| *** jwcroppe has quit IRC | 20:02 | |
| *** john5223_ has joined #openstack-cinder | 20:02 | |
| *** e0ne has quit IRC | 20:05 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add shared_targets and backend_id to volumes https://review.openstack.org/509005 | 20:07 |
| jgriffith | excellent! Thanks for the coaching coffee_cat | 20:08 |
| coffee_cat | jgriffith: my pleasure, happy it worked :) | 20:09 |
| coffee_cat | jgriffith: and never listen to smcginnis ;) | 20:09 |
| *** ChubYann has quit IRC | 20:10 | |
| smcginnis | :P | 20:17 |
| *** gouthamr has quit IRC | 20:21 | |
| *** catinthe_ has quit IRC | 20:28 | |
| *** spectr has joined #openstack-cinder | 20:34 | |
| *** spectr has quit IRC | 20:36 | |
| *** david-lyle has quit IRC | 20:38 | |
| *** e0ne has joined #openstack-cinder | 20:39 | |
| openstackgerrit | Merged openstack/cinder master: Add ploop to parallels naming conversion https://review.openstack.org/507193 | 20:43 |
| *** crushil has quit IRC | 20:48 | |
| *** ChubYann has joined #openstack-cinder | 20:50 | |
| *** markstur has joined #openstack-cinder | 20:51 | |
| *** dikonoor has quit IRC | 20:52 | |
| *** arnewiebalck_ has quit IRC | 20:57 | |
| *** e0ne has quit IRC | 20:57 | |
| *** abishop has quit IRC | 20:58 | |
| openstackgerrit | Merged openstack/cinder master: Add retries to LVM logical volume activation https://review.openstack.org/510168 | 20:59 |
| *** savvas has joined #openstack-cinder | 21:02 | |
| *** jwcroppe_ has quit IRC | 21:03 | |
| *** edmondsw has joined #openstack-cinder | 21:05 | |
| *** jwcroppe has joined #openstack-cinder | 21:05 | |
| *** savvas has quit IRC | 21:06 | |
| *** edmondsw has quit IRC | 21:10 | |
| *** dustins has quit IRC | 21:14 | |
| *** crushil has joined #openstack-cinder | 21:16 | |
| *** david-lyle has joined #openstack-cinder | 21:19 | |
| *** xyang1 has quit IRC | 21:20 | |
| *** felipemonteiro__ has quit IRC | 21:21 | |
| *** wes_dillingham has joined #openstack-cinder | 21:23 | |
| *** wes_dillingham has quit IRC | 21:24 | |
| *** ronlund is now known as mriedem | 21:29 | |
| *** dklyle has joined #openstack-cinder | 21:39 | |
| *** tpsilva has quit IRC | 21:39 | |
| *** david-lyle has quit IRC | 21:41 | |
| *** dklyle has quit IRC | 21:41 | |
| *** dklyle has joined #openstack-cinder | 21:42 | |
| *** lbragstad has quit IRC | 21:43 | |
| *** gnufied has quit IRC | 21:46 | |
| *** wes_dillingham has joined #openstack-cinder | 21:51 | |
| *** dklyle has quit IRC | 21:52 | |
| *** wes_dillingham has quit IRC | 21:52 | |
| *** wes_dillingham has joined #openstack-cinder | 21:53 | |
| *** wes_dillingham has quit IRC | 21:53 | |
| *** wes_dillingham has joined #openstack-cinder | 21:54 | |
| *** wes_dillingham has quit IRC | 21:54 | |
| *** wes_dillingham has joined #openstack-cinder | 21:55 | |
| *** wes_dillingham has quit IRC | 21:56 | |
| *** wes_dillingham has joined #openstack-cinder | 21:57 | |
| *** wes_dillingham has quit IRC | 21:57 | |
| *** jwcroppe has quit IRC | 21:58 | |
| *** jwcroppe has joined #openstack-cinder | 21:59 | |
| *** Apoorva_ has joined #openstack-cinder | 22:00 | |
| *** Apoorva has quit IRC | 22:01 | |
| *** ntpttr_laptop has quit IRC | 22:02 | |
| *** Apoorva_ has quit IRC | 22:02 | |
| *** Apoorva has joined #openstack-cinder | 22:02 | |
| *** Apoorva has quit IRC | 22:03 | |
| *** jwcroppe has quit IRC | 22:03 | |
| *** Apoorva has joined #openstack-cinder | 22:03 | |
| *** jgwentworth is now known as melwitt | 22:08 | |
| *** chlong has quit IRC | 22:14 | |
| *** armax has joined #openstack-cinder | 22:23 | |
| *** gouthamr has joined #openstack-cinder | 22:24 | |
| *** armax has quit IRC | 22:28 | |
| *** erlon has quit IRC | 22:30 | |
| *** shvepsy_ has joined #openstack-cinder | 22:31 | |
| *** armax has joined #openstack-cinder | 22:32 | |
| *** armax has quit IRC | 22:33 | |
| *** shvepsy has quit IRC | 22:33 | |
| *** crushil has quit IRC | 22:41 | |
| jgriffith | I'll listen to smcginnis before I listen to the voices inside my head :) | 22:43 |
| jgriffith | well.. depending on what they say | 22:44 |
| jgriffith | but usually | 22:44 |
| *** markstur has quit IRC | 22:49 | |
| *** ntpttr_laptop has joined #openstack-cinder | 22:50 | |
| *** edmondsw has joined #openstack-cinder | 22:53 | |
| *** edmondsw has quit IRC | 22:58 | |
| jgriffith | anybody still around? | 23:02 |
| *** crushil has joined #openstack-cinder | 23:05 | |
| *** gouthamr has quit IRC | 23:10 | |
| mtreinish | jgriffith: I am here | 23:11 |
| jgriffith | mtreinish hey... so I actually was hoping to ping you earlier | 23:11 |
| * mtreinish wasn't expecting that | 23:11 | |
| jgriffith | I forget :) | 23:11 |
| jgriffith | but anyway :) | 23:12 |
| jgriffith | I was going to see if there was anybody with a running devstack install up (with cinder) | 23:12 |
| jgriffith | because I just noticed I can't do something simple that I thought I should be able to do | 23:12 |
| jgriffith | if you `mysql cinder` | 23:12 |
| jgriffith | then do `select binary from services;` | 23:12 |
| jgriffith | it results in a syntax error, which is completely odd to me | 23:13 |
| jgriffith | only thing I wondered is if "binary" is some sort of mysql keyword or something | 23:13 |
| mtreinish | jgriffith: yeah it's a type: https://dev.mysql.com/doc/refman/5.7/en/binary-varbinary.html | 23:14 |
| jgriffith | Ahh... yes, well how about that | 23:14 |
| jgriffith | well that's inconvenient | 23:15 |
| *** Apoorva has quit IRC | 23:16 | |
| *** Apoorva has joined #openstack-cinder | 23:17 | |
| *** ganso has quit IRC | 23:18 | |
| jgriffith | so I wonder how I specify that I actually want to use a column name there | 23:18 |
| jgriffith | Oh :) | 23:19 |
| jgriffith | `select services.binar from services` | 23:19 |
| jgriffith | mtreinish thanks for the help :) | 23:20 |
| jgriffith | mriedem if you get a chance https://review.openstack.org/#/c/510201/ | 23:21 |
| jgriffith | mriedem I'm updating https://review.openstack.org/#/c/509005/ to build off of that as well | 23:22 |
| jgriffith | mtreinish oh.. I remember, stestr question | 23:23 |
| jgriffith | but I need to run.... I'll hit you up later | 23:23 |
| *** crushil has quit IRC | 23:24 | |
| *** jgriffith is now known as jgriffith_ | 23:24 | |
| *** crushil has joined #openstack-cinder | 23:38 | |
| *** david-lyle has joined #openstack-cinder | 23:42 | |
| *** harlowja has quit IRC | 23:45 | |
| *** markstur has joined #openstack-cinder | 23:45 | |
| *** markstur has quit IRC | 23:49 | |
| *** ntpttr_laptop has quit IRC | 23:50 | |
| *** markvoelker has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!