*** alonma has quit IRC | 00:02 | |
*** mylu has quit IRC | 00:02 | |
*** hongbin has quit IRC | 00:04 | |
*** alonma has joined #openstack-cinder | 00:06 | |
*** mylu has joined #openstack-cinder | 00:07 | |
*** mylu has quit IRC | 00:10 | |
*** alonma has quit IRC | 00:11 | |
*** mylu has joined #openstack-cinder | 00:12 | |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 00:12 |
---|---|---|
*** csky has quit IRC | 00:13 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: WIP! Fix Rally job https://review.openstack.org/278119 | 00:14 |
*** alonma has joined #openstack-cinder | 00:15 | |
*** e0ne has quit IRC | 00:18 | |
*** alonma has quit IRC | 00:20 | |
*** mylu has quit IRC | 00:20 | |
notmorgan | so, good news everyone! I have all but 4 tests working with cinderclinet using keystoneauth! :) | 00:21 |
notmorgan | once i have this passed, who should i poke at to review the changes? I'd like to get this landed quickly so we're on KSA on the major client libs [hopefully occ too] | 00:22 |
hemna | gus, http://paste.openstack.org/show/486502/ | 00:25 |
hemna | after updating the latest patchset | 00:25 |
gus | "Error opening class fc_host" ? | 00:27 |
gus | The logs suggest it's running "systool -c fc_host -v" with uid/gid=0 and getting back that error | 00:28 |
gus | Does that work when you run that directly (perhaps with sudo)? | 00:28 |
*** esker has quit IRC | 00:28 | |
gus | .. and then later it's running "multipathd show status" and getting No such file or directory - so I assume you don't have multipathd installed? | 00:32 |
gus | That last seems to be the stacktrace that finally aborts your diediedie test | 00:32 |
*** dramakri has quit IRC | 00:35 | |
gus | oh, huh. | 00:35 |
gus | I wonder if PATH is never updated to include /sbin? | 00:36 |
gus | ... oh no, I think it should be, because the privsep-helper is still run through sudo and so should have the same environment as with rootwrap. | 00:36 |
*** geguileo has quit IRC | 00:38 | |
gus | aha. .. | 00:40 |
gus | I note that simply exec()ing "bogus" gives a different error return to exec()ing "sudo bogus" (or indeed "sh -c bogus") | 00:41 |
gus | I wonder if processutils.execute actually knows how to catch errors with shell=False. Looking.. | 00:41 |
*** akshai has quit IRC | 00:43 | |
*** prithiv has quit IRC | 00:45 | |
gus | huh, indeed putils.execute('foo') throws OSError(no such file or directory), and not ProcessExecutionError as expected. | 00:46 |
*** geguileo has joined #openstack-cinder | 00:46 | |
*** arch-nemesis has quit IRC | 00:50 | |
gus | .. and indeed putils.execute is documented as raising either ProcessExecutionError or OSError. | 00:51 |
gus | I claim this is an os-brick bug. | 00:51 |
gus | I believe lots of places where we currently catch only ProcessExecutionError should also catch OSError. | 00:52 |
gus | Now: should I convert OSError into ProcesExecutionError in priv_rootwrap.execute? | 00:52 |
gus | (as a workaround) | 00:52 |
*** mylu has joined #openstack-cinder | 00:58 | |
*** jungleboyj has quit IRC | 01:00 | |
*** markvoelker has joined #openstack-cinder | 01:01 | |
*** mylu has quit IRC | 01:02 | |
*** mylu has joined #openstack-cinder | 01:04 | |
*** garthb has quit IRC | 01:14 | |
*** alonma has joined #openstack-cinder | 01:16 | |
*** alonma has quit IRC | 01:20 | |
*** alonma has joined #openstack-cinder | 01:22 | |
*** fernnest is now known as fernnest_offWed | 01:24 | |
*** esker has joined #openstack-cinder | 01:26 | |
*** alonma has quit IRC | 01:27 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 01:27 |
*** fernnest_offWed is now known as fernnest | 01:27 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 01:28 |
*** mtanino has quit IRC | 01:29 | |
*** mylu has quit IRC | 01:31 | |
*** alonma has joined #openstack-cinder | 01:31 | |
*** leeantho has quit IRC | 01:34 | |
*** mylu has joined #openstack-cinder | 01:35 | |
*** alonma has quit IRC | 01:35 | |
*** shakamunyi has quit IRC | 01:40 | |
*** esker has quit IRC | 01:40 | |
*** esker has joined #openstack-cinder | 01:41 | |
*** laughterwym has joined #openstack-cinder | 01:44 | |
*** mylu has quit IRC | 01:44 | |
*** esker has quit IRC | 01:45 | |
*** laughterwym has quit IRC | 01:48 | |
*** bswartz has quit IRC | 01:49 | |
*** bswartz has joined #openstack-cinder | 01:50 | |
*** mylu has joined #openstack-cinder | 01:55 | |
*** sid14726 has joined #openstack-cinder | 01:55 | |
*** funeutron has joined #openstack-cinder | 01:56 | |
*** mylu has quit IRC | 01:58 | |
*** gouthamr has joined #openstack-cinder | 02:13 | |
*** itzdilip has joined #openstack-cinder | 02:20 | |
*** funeutron has quit IRC | 02:23 | |
*** mylu has joined #openstack-cinder | 02:30 | |
*** alonma has joined #openstack-cinder | 02:31 | |
*** alonma has quit IRC | 02:35 | |
*** dims_ has quit IRC | 02:45 | |
*** Poornima has joined #openstack-cinder | 02:48 | |
*** gouthamr_ has joined #openstack-cinder | 02:51 | |
*** gouthamr has quit IRC | 02:54 | |
*** dan_nguyen has quit IRC | 02:56 | |
*** links has joined #openstack-cinder | 03:00 | |
*** mudassirlatif has quit IRC | 03:06 | |
*** mudassirlatif has joined #openstack-cinder | 03:07 | |
*** cknight has quit IRC | 03:07 | |
*** csky has joined #openstack-cinder | 03:10 | |
*** logan-- has joined #openstack-cinder | 03:10 | |
*** mudassirlatif has quit IRC | 03:11 | |
*** alonma has joined #openstack-cinder | 03:14 | |
*** logan-- has quit IRC | 03:14 | |
*** mylu has quit IRC | 03:15 | |
*** mylu has joined #openstack-cinder | 03:16 | |
*** alonma has quit IRC | 03:18 | |
*** alonma has joined #openstack-cinder | 03:20 | |
*** logan- has quit IRC | 03:22 | |
*** logan- has joined #openstack-cinder | 03:23 | |
*** bardia has quit IRC | 03:23 | |
*** alonma has quit IRC | 03:25 | |
*** mylu has quit IRC | 03:26 | |
*** dims has joined #openstack-cinder | 03:27 | |
*** mylu has joined #openstack-cinder | 03:29 | |
smcginnis | notmorgan: Awesome on the keystoneauth! | 03:29 |
smcginnis | notmorgan: That was on my "this would be good if I can get to it someday" queue. | 03:29 |
smcginnis | notmorgan: Let me know when you have something and I'll check it out. I would like to get that switched too. | 03:29 |
notmorgan | Struggling with a few tests. Should have it wound up soon. | 03:29 |
smcginnis | notmorgan: Cool. Hopefully the struggling stays to a minimum. ;) | 03:30 |
notmorgan | It's just unwinding duplicated code | 03:31 |
*** funeutron has joined #openstack-cinder | 03:31 | |
notmorgan | Well copied code. | 03:32 |
notmorgan | Only 5 clients to fix this week. Cinder is the first. | 03:32 |
smcginnis | notmorgan: Hah, should hopefully be pretty trivial by the fifth. | 03:33 |
notmorgan | Each are wildly different :( | 03:33 |
notmorgan | Worst will be keystone client actually. :P | 03:34 |
*** xyang1 has joined #openstack-cinder | 03:34 | |
*** sheel has joined #openstack-cinder | 03:36 | |
*** wanghao_ has joined #openstack-cinder | 03:36 | |
*** dims has quit IRC | 03:37 | |
*** mylu has quit IRC | 03:38 | |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 03:39 |
*** mylu has joined #openstack-cinder | 03:39 | |
*** apoorvad has quit IRC | 03:40 | |
*** jwcroppe has joined #openstack-cinder | 03:41 | |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 03:42 |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 03:44 |
Poornima | Hi wanghao, swamireddy regarding https://review.openstack.org/#/c/235752/ | 03:48 |
Poornima | just removing blank space leading to gate failure | 03:48 |
*** bill_az has quit IRC | 03:49 | |
*** edmondsw has quit IRC | 03:49 | |
*** xyang1 has quit IRC | 03:50 | |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 03:51 |
smcginnis | Poornima: There are issues in gate right now. | 03:51 |
smcginnis | Poornima: Not sure if things are cleared up yet or not, but I've kicked off a recheck on there. | 03:51 |
Poornima | smcginnis, this is strange | 03:52 |
*** gouthamr_ has quit IRC | 03:52 | |
openstackgerrit | Merged openstack/cinder-specs: Remove Py33 support https://review.openstack.org/266111 | 04:05 |
*** shyama has joined #openstack-cinder | 04:09 | |
*** mylu has quit IRC | 04:15 | |
*** markvoelker has quit IRC | 04:15 | |
*** markvoelker has joined #openstack-cinder | 04:16 | |
*** alonma has joined #openstack-cinder | 04:20 | |
*** mylu has joined #openstack-cinder | 04:25 | |
*** alonma has quit IRC | 04:25 | |
*** mylu has quit IRC | 04:26 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove Cisco FC Zone Manager Driver https://review.openstack.org/278201 | 04:31 |
smcginnis | hemna: ^^ | 04:32 |
*** Poornima has quit IRC | 04:32 | |
*** funeutron has quit IRC | 04:40 | |
*** salv-orlando has joined #openstack-cinder | 04:41 | |
*** sgotliv has joined #openstack-cinder | 04:42 | |
*** salv-orl_ has quit IRC | 04:43 | |
*** mylu has joined #openstack-cinder | 04:44 | |
*** wanghao has quit IRC | 04:46 | |
*** wanghao_ is now known as wanghao | 04:46 | |
*** laughterwym has joined #openstack-cinder | 04:55 | |
*** laughterwym has quit IRC | 04:59 | |
*** Poornima has joined #openstack-cinder | 05:00 | |
*** mragupat has joined #openstack-cinder | 05:00 | |
*** alonma has joined #openstack-cinder | 05:04 | |
*** mragupat has quit IRC | 05:06 | |
*** mragupat has joined #openstack-cinder | 05:06 | |
*** chhavi has joined #openstack-cinder | 05:07 | |
*** alonma has quit IRC | 05:08 | |
*** alonma has joined #openstack-cinder | 05:10 | |
*** alonma has quit IRC | 05:14 | |
*** alonma has joined #openstack-cinder | 05:16 | |
*** alonma has quit IRC | 05:21 | |
*** alonma has joined #openstack-cinder | 05:22 | |
*** Poornima has quit IRC | 05:24 | |
*** anshul has joined #openstack-cinder | 05:24 | |
*** alonma has quit IRC | 05:27 | |
*** Poornima has joined #openstack-cinder | 05:29 | |
*** alonma has joined #openstack-cinder | 05:32 | |
*** anshul has quit IRC | 05:32 | |
*** dave-mccowan has quit IRC | 05:33 | |
*** Poornima has quit IRC | 05:34 | |
*** Poornima has joined #openstack-cinder | 05:36 | |
*** alonma has quit IRC | 05:37 | |
*** alkhodos has quit IRC | 05:39 | |
*** leeantho has joined #openstack-cinder | 05:39 | |
*** huangtianhua has quit IRC | 05:39 | |
*** huanan_L has quit IRC | 05:40 | |
*** huangtianhua has joined #openstack-cinder | 05:40 | |
*** huanan_L has joined #openstack-cinder | 05:40 | |
*** alonma has joined #openstack-cinder | 05:40 | |
*** mylu has quit IRC | 05:41 | |
*** Poornima has quit IRC | 05:42 | |
*** ak has joined #openstack-cinder | 05:43 | |
*** ianbrown_ has quit IRC | 05:44 | |
*** alonma has quit IRC | 05:45 | |
*** ianbrown_ has joined #openstack-cinder | 05:45 | |
*** mylu has joined #openstack-cinder | 05:46 | |
*** alonma has joined #openstack-cinder | 05:46 | |
*** anshul has joined #openstack-cinder | 05:48 | |
ak | hi guys enable_service c-bak should enable the cinder backup service | 05:49 |
ak | right can some one confirm ? | 05:49 |
*** alonma has quit IRC | 05:51 | |
*** alonma has joined #openstack-cinder | 05:53 | |
ak | hi guys | 05:56 |
ak | can some please help me out ? | 05:56 |
*** alonma has quit IRC | 05:57 | |
*** alonma has joined #openstack-cinder | 05:59 | |
*** sgotliv has quit IRC | 06:00 | |
*** alonma has quit IRC | 06:03 | |
*** mylu has quit IRC | 06:04 | |
*** mylu has joined #openstack-cinder | 06:05 | |
*** Poornima has joined #openstack-cinder | 06:07 | |
*** esker has joined #openstack-cinder | 06:08 | |
*** anshul has quit IRC | 06:09 | |
*** ak has quit IRC | 06:12 | |
*** mylu has quit IRC | 06:12 | |
*** mylu has joined #openstack-cinder | 06:13 | |
*** anshul has joined #openstack-cinder | 06:21 | |
*** esker has quit IRC | 06:21 | |
*** jwcroppe has quit IRC | 06:22 | |
*** ChubYann has quit IRC | 06:22 | |
*** nkrinner has joined #openstack-cinder | 06:25 | |
*** anshul has quit IRC | 06:26 | |
sheel | ak: yes, what;s the problem? | 06:26 |
sheel | ak:tell me | 06:27 |
*** lpetrut has joined #openstack-cinder | 06:30 | |
*** esker has joined #openstack-cinder | 06:32 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 06:34 |
*** alonma has joined #openstack-cinder | 06:41 | |
thingee | smcginnis: for the record, this reference implementation thing is starting to get annoying https://twitter.com/Thingee/status/697291484844851201 | 06:41 |
thingee | oh trust me, this was in my head the moment I started | 06:44 |
thingee | whoops wrong window | 06:44 |
*** alonma has quit IRC | 06:45 | |
sheel | :) | 06:46 |
*** alonma has joined #openstack-cinder | 06:47 | |
*** anshul has joined #openstack-cinder | 06:47 | |
*** csky has quit IRC | 06:48 | |
*** alonma has quit IRC | 06:51 | |
*** ianbrown_ has quit IRC | 06:52 | |
*** ianbrown_ has joined #openstack-cinder | 06:53 | |
*** alonma has joined #openstack-cinder | 06:53 | |
*** bvivek has joined #openstack-cinder | 06:56 | |
*** bvivek_ has joined #openstack-cinder | 06:56 | |
*** alonma has quit IRC | 06:58 | |
*** hparekh has joined #openstack-cinder | 07:00 | |
*** alonma has joined #openstack-cinder | 07:00 | |
*** chhavi has quit IRC | 07:04 | |
*** alonma has quit IRC | 07:05 | |
*** cfriesen has quit IRC | 07:06 | |
*** alonma has joined #openstack-cinder | 07:07 | |
*** alonma has quit IRC | 07:11 | |
*** alonma has joined #openstack-cinder | 07:13 | |
*** mylu has quit IRC | 07:13 | |
*** belmoreira has joined #openstack-cinder | 07:13 | |
*** arecknag has joined #openstack-cinder | 07:14 | |
*** rcernin has joined #openstack-cinder | 07:14 | |
*** alonma has quit IRC | 07:17 | |
*** chhavi has joined #openstack-cinder | 07:18 | |
*** alonma has joined #openstack-cinder | 07:19 | |
*** alonma has quit IRC | 07:23 | |
*** lpetrut has quit IRC | 07:23 | |
*** alonma has joined #openstack-cinder | 07:25 | |
*** mragupat has quit IRC | 07:27 | |
*** lpetrut has joined #openstack-cinder | 07:27 | |
*** alonma has quit IRC | 07:30 | |
*** mudassirlatif has joined #openstack-cinder | 07:30 | |
*** anshul has quit IRC | 07:34 | |
*** alonma has joined #openstack-cinder | 07:38 | |
*** alonma has quit IRC | 07:43 | |
*** alonma has joined #openstack-cinder | 07:44 | |
*** openstackgerrit has quit IRC | 07:47 | |
*** openstackgerrit has joined #openstack-cinder | 07:48 | |
*** alonma has quit IRC | 07:49 | |
*** alonma has joined #openstack-cinder | 07:51 | |
*** rcernin has quit IRC | 07:52 | |
*** isaacb has joined #openstack-cinder | 07:55 | |
*** lpetrut has quit IRC | 07:55 | |
*** andymaier has joined #openstack-cinder | 07:55 | |
*** alonma has quit IRC | 07:55 | |
*** esker has quit IRC | 07:56 | |
*** alonma has joined #openstack-cinder | 07:57 | |
*** e0ne has joined #openstack-cinder | 07:58 | |
*** Poornima is now known as Poornima|lunch | 07:58 | |
*** Poornima|lunch is now known as Poornima | 07:58 | |
*** huanan_L has quit IRC | 07:58 | |
*** esker has joined #openstack-cinder | 07:59 | |
*** huanan_L has joined #openstack-cinder | 07:59 | |
*** alonma has quit IRC | 08:01 | |
*** alonma has joined #openstack-cinder | 08:03 | |
*** leeantho has quit IRC | 08:05 | |
*** alonma has quit IRC | 08:07 | |
*** alonma has joined #openstack-cinder | 08:15 | |
*** rcernin has joined #openstack-cinder | 08:15 | |
*** sinese_ has joined #openstack-cinder | 08:17 | |
*** liverpooler has joined #openstack-cinder | 08:19 | |
*** alonma has quit IRC | 08:20 | |
*** shyama has quit IRC | 08:21 | |
*** alonma has joined #openstack-cinder | 08:22 | |
*** jistr|doc has joined #openstack-cinder | 08:22 | |
*** alonma has quit IRC | 08:26 | |
*** alonma has joined #openstack-cinder | 08:28 | |
*** ndipanov has joined #openstack-cinder | 08:32 | |
*** alonma has quit IRC | 08:32 | |
*** ndipanov has quit IRC | 08:33 | |
*** alonma has joined #openstack-cinder | 08:34 | |
*** alonma has quit IRC | 08:35 | |
*** alonma has joined #openstack-cinder | 08:35 | |
*** smoriya_ has quit IRC | 08:37 | |
*** e0ne has quit IRC | 08:39 | |
*** salv-orlando has quit IRC | 08:44 | |
*** shyama has joined #openstack-cinder | 08:52 | |
*** laughterwym has joined #openstack-cinder | 08:56 | |
*** laughterwym has quit IRC | 09:00 | |
openstackgerrit | Matan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver https://review.openstack.org/269624 | 09:12 |
openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver https://review.openstack.org/268344 | 09:12 |
*** ndipanov has joined #openstack-cinder | 09:13 | |
*** jwcroppe has joined #openstack-cinder | 09:22 | |
*** isaacb has quit IRC | 09:23 | |
*** alonma has quit IRC | 09:23 | |
*** jwcroppe has quit IRC | 09:26 | |
*** lpetrut has joined #openstack-cinder | 09:27 | |
*** alonma has joined #openstack-cinder | 09:28 | |
*** jistr|doc has quit IRC | 09:46 | |
*** mvk has joined #openstack-cinder | 09:52 | |
*** mvk has quit IRC | 09:56 | |
*** sheel has quit IRC | 10:07 | |
*** e0ne has joined #openstack-cinder | 10:07 | |
*** mvk has joined #openstack-cinder | 10:16 | |
*** sinese_ has quit IRC | 10:16 | |
*** yhayashi has quit IRC | 10:26 | |
*** alonma has quit IRC | 10:37 | |
*** alonma has joined #openstack-cinder | 10:37 | |
*** salv-orlando has joined #openstack-cinder | 10:41 | |
*** alonma has quit IRC | 10:42 | |
*** alonma has joined #openstack-cinder | 10:43 | |
*** dims has joined #openstack-cinder | 10:48 | |
*** alonma has quit IRC | 10:48 | |
*** alonma has joined #openstack-cinder | 10:49 | |
*** tpsilva has joined #openstack-cinder | 10:53 | |
*** alonma has quit IRC | 10:54 | |
*** alonma has joined #openstack-cinder | 10:56 | |
*** laughterwym has joined #openstack-cinder | 10:58 | |
*** alonma has quit IRC | 11:00 | |
*** alonma has joined #openstack-cinder | 11:02 | |
*** NightKhaos has quit IRC | 11:04 | |
*** lprice has joined #openstack-cinder | 11:05 | |
*** alonma has quit IRC | 11:06 | |
*** lprice1 has quit IRC | 11:07 | |
*** alonma has joined #openstack-cinder | 11:08 | |
*** laughterwym has quit IRC | 11:09 | |
*** NightKhaos has joined #openstack-cinder | 11:11 | |
*** alonma has quit IRC | 11:12 | |
*** alonma has joined #openstack-cinder | 11:16 | |
*** NightKhaos has quit IRC | 11:16 | |
*** NightKhaos has joined #openstack-cinder | 11:19 | |
*** alonma has quit IRC | 11:19 | |
*** alonma has joined #openstack-cinder | 11:20 | |
openstackgerrit | Sheel Rana proposed openstack/cinder-specs: Summary Messages for different cinder operations https://review.openstack.org/273938 | 11:22 |
*** changbl has quit IRC | 11:25 | |
openstackgerrit | Petrut Lucian proposed openstack/os-brick: [WIP] Add Windows connectors https://review.openstack.org/272522 | 11:31 |
*** ociuhandu has quit IRC | 11:35 | |
*** ildikov has quit IRC | 11:38 | |
*** erlon has joined #openstack-cinder | 11:43 | |
*** kmartin__ has quit IRC | 11:48 | |
*** sinese_ has joined #openstack-cinder | 11:49 | |
swamireddy | Poornima: Please fix the Jenkins failiures for https://review.openstack.org/#/c/235752/ | 11:52 |
*** xek has quit IRC | 11:53 | |
*** sid14726 has quit IRC | 11:54 | |
Poornima | swamireddy, going through http://logs.openstack.org/52/235752/8/check/gate-tempest-dsvm-large-ops/dc1f5ad/console.html | 11:54 |
*** permalac has joined #openstack-cinder | 11:55 | |
*** links has quit IRC | 11:56 | |
Poornima | swamireddy, I see number of failures in console but check through local setup patch works fine | 11:57 |
*** szblcs has joined #openstack-cinder | 11:58 | |
szblcs | hello | 11:58 |
szblcs | Can I use FreeNAS as a storage provider for OpenStack? I've got FreeNAS configured with iSCSI and NFS. | 11:58 |
*** mvk has quit IRC | 12:00 | |
swamireddy | Poornima: do "recheck" | 12:02 |
*** mvk has joined #openstack-cinder | 12:02 | |
*** ociuhandu has joined #openstack-cinder | 12:05 | |
*** Poornima has quit IRC | 12:10 | |
*** Poornima has joined #openstack-cinder | 12:12 | |
*** ildikov has joined #openstack-cinder | 12:22 | |
openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database https://review.openstack.org/153969 | 12:24 |
*** sinese_ has quit IRC | 12:25 | |
*** sinese_ has joined #openstack-cinder | 12:27 | |
*** edmondsw has joined #openstack-cinder | 12:29 | |
*** funenutron has joined #openstack-cinder | 12:29 | |
*** links has joined #openstack-cinder | 12:36 | |
*** ildikov has quit IRC | 12:38 | |
*** krotscheck_dcm is now known as krotscheck | 12:38 | |
*** bvivek_ has quit IRC | 12:40 | |
*** bvivek has quit IRC | 12:40 | |
*** ildikov has joined #openstack-cinder | 12:44 | |
*** vgridnev has joined #openstack-cinder | 12:44 | |
*** funenutron has quit IRC | 12:45 | |
*** alonma has quit IRC | 12:53 | |
*** sid14726 has joined #openstack-cinder | 12:57 | |
*** Poornima has quit IRC | 12:59 | |
*** alonma has joined #openstack-cinder | 12:59 | |
*** alonma has quit IRC | 13:03 | |
*** alonma has joined #openstack-cinder | 13:03 | |
*** bvivek has joined #openstack-cinder | 13:06 | |
*** bvivek_ has joined #openstack-cinder | 13:06 | |
*** e0ne has quit IRC | 13:09 | |
*** vgridnev has quit IRC | 13:09 | |
*** e0ne has joined #openstack-cinder | 13:09 | |
openstackgerrit | Dinesh Bhor proposed openstack/cinder: Fix 500 error if 'offset' is out of range https://review.openstack.org/278354 | 13:10 |
*** haomaiwa_ has joined #openstack-cinder | 13:10 | |
*** edmondsw has quit IRC | 13:11 | |
*** gouthamr has joined #openstack-cinder | 13:11 | |
*** gouthamr_ has joined #openstack-cinder | 13:13 | |
DuncanT | szblcs: I'm not aware of a FreeNAS plugin, so the only way you could do it without writing code would be to create a large lun on it, then run cinder/LVM on top of the lun | 13:15 |
DuncanT | szblcs: This works fine if your c-vol node has good networking | 13:16 |
*** gouthamr has quit IRC | 13:16 | |
*** martyturner has joined #openstack-cinder | 13:19 | |
*** vgridnev has joined #openstack-cinder | 13:19 | |
dulek | geguileo: I've just walked through c-api's volume flow with sborkowski to identify possible race conditions. | 13:19 |
szblcs | DuncanT: thanks. I'm a bit over my head. So you mean I create the iSCSI target in FreeNAS, then mount on the OpenStack node? | 13:19 |
geguileo | dulek: And how did it go? | 13:19 |
dulek | geguileo: And this is craaaazy… I mean - how paranoid we want to be? | 13:20 |
DuncanT | szblcs: Yes | 13:20 |
geguileo | dulek: XD XD XD | 13:20 |
dulek | geguileo: For example - what if source volume's size changes? | 13:20 |
szblcs | DuncanT: thank you, I'm going to try this. | 13:20 |
dulek | geguileo: Or source vol gets retyped? | 13:20 |
geguileo | dulek: That's a valid question | 13:21 |
dulek | geguileo: Or snapshot's AZ changes? | 13:21 |
*** vgridnev has quit IRC | 13:21 | |
dulek | geguileo: Or extra specs on volume type changes? | 13:21 |
DuncanT | dulek: does the cinder API allow those operations? If so, they are valid race conditions | 13:21 |
geguileo | dulek: Mmmmm, we really should be protecting those cases in the other operations | 13:21 |
dulek | DuncanT: Yeah, we don't lock source snap/vol from anything more than deletion. | 13:21 |
DuncanT | dulek: Editing extra specs does not change existing volumes, so that one at least is safe | 13:21 |
geguileo | dulek: Like not allowing an extend if it is being used as a source to create another volume | 13:22 |
dulek | DuncanT: Ah, good to know. | 13:22 |
geguileo | dulek: I think the way to go for now is to ignore the extra paranoia and add those cases as bugs | 13:22 |
geguileo | dulek: Like "extend action should not be allowed when the volume is used as a source volume" | 13:23 |
geguileo | dulek: Same for retype operation | 13:23 |
*** e0ne has quit IRC | 13:24 | |
dulek | Okay, so how about - we'll start with something simple (I think ensuring that source resource wasn't deleted while processing), and start to test "paranoid" cases to confirm if there's a bug to report? | 13:24 |
*** e0ne has joined #openstack-cinder | 13:25 | |
dulek | geguileo: I've identified places in your commits where some cases like that aren't taken into account. I can point these while reviewing, but I think these should be addressed in future commits, otherwise we end up with really complicated reviews. | 13:26 |
geguileo | dulek: I agree | 13:27 |
geguileo | dulek: I haven't taken those into account, because I was just removing races using the same logic we currently have | 13:27 |
*** vgridnev has joined #openstack-cinder | 13:27 | |
geguileo | dulek: I think we should open new bugs for those cases and then fix the swap-and-compare code to fix them | 13:27 |
openstackgerrit | Petrut Lucian proposed openstack/os-brick: [WIP] Add Windows connectors https://review.openstack.org/272522 | 13:31 |
dulek | geguileo: Okay, this sometimes seems to be pretty hard, but fixing them one-by-one may work. Or at least we'll know about crazy edge-cases. Thanks for opinion! | 13:31 |
geguileo | dulek: Thanks for looking into it | 13:32 |
*** gouthamr_ has quit IRC | 13:32 | |
*** csky has joined #openstack-cinder | 13:38 | |
*** xek has joined #openstack-cinder | 13:38 | |
*** permalac has quit IRC | 13:39 | |
*** akerr has joined #openstack-cinder | 13:39 | |
*** shyama_ has joined #openstack-cinder | 13:41 | |
*** timcl has joined #openstack-cinder | 13:41 | |
*** shyama has quit IRC | 13:42 | |
*** shyama_ is now known as shyama | 13:42 | |
*** ildikov has quit IRC | 13:43 | |
*** gouthamr has joined #openstack-cinder | 13:47 | |
*** dave-mccowan has joined #openstack-cinder | 13:51 | |
*** eharney has joined #openstack-cinder | 13:52 | |
*** alonma has quit IRC | 13:54 | |
*** laughterwym has joined #openstack-cinder | 13:54 | |
*** alonma has joined #openstack-cinder | 13:57 | |
*** akshai has joined #openstack-cinder | 13:59 | |
*** haomaiwa_ has quit IRC | 14:01 | |
*** haomaiwa_ has joined #openstack-cinder | 14:01 | |
*** sheel has joined #openstack-cinder | 14:04 | |
e0ne | eharney: hi Eric. could you please review my patch to NFS connector https://review.openstack.org/277788 | 14:05 |
*** rlrossit has joined #openstack-cinder | 14:06 | |
*** isaacb has joined #openstack-cinder | 14:10 | |
*** mtanino has joined #openstack-cinder | 14:12 | |
*** julim has joined #openstack-cinder | 14:15 | |
*** porrua has joined #openstack-cinder | 14:16 | |
smcginnis | thingee: Yeah, saw that. :[ | 14:16 |
*** bvivek_ has quit IRC | 14:17 | |
*** bvivek has quit IRC | 14:17 | |
*** eharney has quit IRC | 14:21 | |
*** cfriesen has joined #openstack-cinder | 14:21 | |
e0ne | eharney: thanks for the review | 14:21 |
krotscheck | I've got a middleware configuration update that should be pretty quick, any cores around to look at https://review.openstack.org/#/c/275751/ ? | 14:21 |
*** mriedem has joined #openstack-cinder | 14:22 | |
*** jordanP has joined #openstack-cinder | 14:22 | |
*** permalac has joined #openstack-cinder | 14:22 | |
smcginnis | krotscheck: Thanks for following up on that one. | 14:25 |
*** eharney has joined #openstack-cinder | 14:25 | |
krotscheck | smcginnis: Of course, thanks for the +2 :) | 14:25 |
*** krtaylor has quit IRC | 14:25 | |
*** sid14726 has quit IRC | 14:25 | |
*** links has quit IRC | 14:27 | |
*** changbl has joined #openstack-cinder | 14:29 | |
*** jgregor has joined #openstack-cinder | 14:30 | |
*** cknight has joined #openstack-cinder | 14:30 | |
*** crose has joined #openstack-cinder | 14:31 | |
*** ildikov has joined #openstack-cinder | 14:32 | |
*** bill_az has joined #openstack-cinder | 14:33 | |
*** akshai has quit IRC | 14:34 | |
*** szblcs has quit IRC | 14:34 | |
*** itzdilip has quit IRC | 14:34 | |
*** Vasanthi has quit IRC | 14:34 | |
*** pradipm has joined #openstack-cinder | 14:35 | |
*** xyang1 has joined #openstack-cinder | 14:37 | |
*** akshai has joined #openstack-cinder | 14:39 | |
*** knikolla has joined #openstack-cinder | 14:43 | |
jordanP | smcginnis, DuncanT hi. I am trying to add a new tempest scenario to validate volume extend. This is not tested in Tempest. The review is here: https://review.openstack.org/#/c/249100/ We have a disagreement with sdague. Maybe you can weight in ? | 14:44 |
jordanP | i think this test is super useful (just look a the 2 bug reports linked in the commit message) | 14:44 |
DuncanT | Sure, I'll take a look | 14:45 |
jordanP | I figured that if some cinder cores say 'yeah, we need this', maybe Sean will change his mind | 14:45 |
*** merooney has joined #openstack-cinder | 14:45 | |
jordanP | thanks ! | 14:45 |
smcginnis | jordanP: We've been having a lot of "disagreements" when it comes to tempest lately. :/ | 14:46 |
eharney | jordanP: i'll take a look there too. I updated my manage/unmanage tests last week based on your comments, thanks for the reviews there | 14:47 |
jordanP | smcginnis, how so ? I live in both world (tempest and cinder, mostly Tempest though) maybe I can help ? | 14:47 |
smcginnis | jordanP: Not sure if you can. Seems a lot of folks want to dictate what cinder can and can't implement for features. | 14:48 |
jordanP | hum that sucks | 14:48 |
jordanP | as far as I know the general idea is that "what can't be tested in the Gate can't land into projects" which I can understand. Now for a couple of weeks it seems the QA/infra community is worried about excessive test duration | 14:50 |
*** alonma has quit IRC | 14:50 | |
DuncanT | jordanP: We're having to re-evaluate the gate since the amount of resources dedicated to it started to look bounded. It's definitely a discussion worth having, but to block patches to tempest on it now is not good. I've commented along those lines on the patch | 14:51 |
smcginnis | jordanP: Yeah, I get the desire to test in gate. I just don't think that's practical to have a healthy, useful platform if it's strictly bound by that. | 14:52 |
jordanP | DuncanT, thanks a lot | 14:52 |
smcginnis | jordanP: My problem has been resistance to actually get tests to cover some of that functionality. | 14:52 |
*** esker has quit IRC | 14:52 | |
*** timcl has quit IRC | 14:53 | |
scottda | e0ne: DuncanT dulek smcginnis The patch set for api-microversions is passing Jenkins and could use some review attention: | 14:53 |
scottda | https://review.openstack.org/#/c/224910 | 14:53 |
DuncanT | smcginnis: I'm going to suggest that we split out some of the tempest tests so they only get run on cinder/nova commits, and that neutron look at doing the same... | 14:53 |
smcginnis | scottda: Great news! Thanks! | 14:53 |
DuncanT | scottda: Thanks! | 14:53 |
smcginnis | DuncanT: I'd be behind that. | 14:54 |
scottda | I'll put something on the meeting agenda for this as well. I can also schedule the demo I talked about during mid-cycle. | 14:54 |
*** alonma_ has joined #openstack-cinder | 14:54 | |
smcginnis | scottda: Excellent, I think that could help. | 14:54 |
scottda | And on the subject of Tempest tests...I've the same concerns/issues about how to get Tempest tests for microversions to land | 14:54 |
DuncanT | scottda: Got a link to the review, please? | 14:54 |
scottda | DuncanT: scroll up ^^^ | 14:55 |
scottda | https://review.openstack.org/#/c/224910 | 14:55 |
DuncanT | scottda: For the tempest bits | 14:55 |
*** xyang has joined #openstack-cinder | 14:55 | |
scottda | oh...no review yet | 14:55 |
scottda | for tempest | 14:55 |
DuncanT | scottda: Ok, thanks | 14:55 |
scottda | I'm going to start with 1 or more functional tests | 14:55 |
*** baumann has joined #openstack-cinder | 14:55 | |
scottda | I think that may be the best place to start. Plus cinder team has more control. I believe the Tempest bits will stretch out past the Mitaka deadlines. | 14:56 |
eharney | scottda: what about in-tree tempest tests? | 14:57 |
scottda | eharney: I'm all for that. smcginnis What is the status of your patch to move (some) Tempest tests into the Cinder tree? | 14:57 |
eharney | scottda: seems like a good fit for what you're doing if we can get it into a usable state soon (not sure how far out that is) | 14:58 |
smcginnis | scottda: It works, but I'm hung up on why they aren't picked up by CIs using the *volume* filtering. | 14:58 |
jordanP | in-tree tempest tests are nice but it's also fragile. You will have to import half of the tempest tree at runtime so each change in tempest can break your gate | 14:58 |
smcginnis | scottda: If I don't specify a regex it does end up getting run. | 14:58 |
jordanP | smcginnis, you need to run the tox target like this : tox -all-plugin -- volume | 14:58 |
smcginnis | jordanP: Ah! Thanks! | 14:58 |
*** diablo_rojo has joined #openstack-cinder | 14:59 | |
*** akshai has quit IRC | 14:59 | |
scottda | In the long run, I'd like to have it all: functional tests, some in-tree Tempest, some regular Tempest (nova<->cinder). I'm mostly concerned about the schedule and deadlines. | 14:59 |
smcginnis | jordanP: See, that's another one. When I went looking for help to figure that out, instead of getting useful information like changing the command line and warnings on fragility, I just started to get a lecture on not allowing features in. :/ | 14:59 |
* smcginnis is sorry to be venting | 15:00 | |
xyang | smcginnis: we need to ping to a specific commit. see what manila does: https://github.com/openstack/manila/blob/master/contrib/ci/common.sh#L3 | 15:00 |
jordanP | manila is doing this in-tree tempest tests but now they are stuck at some specific revision of tempest (https://github.com/openstack/manila/blob/5244bf372e7e40d5ff51c488e41a7e92b39a5a59/contrib/ci/pre_test_hook.sh#L76) | 15:00 |
smcginnis | xyang: What does that do? | 15:00 |
*** haomaiwa_ has quit IRC | 15:01 | |
jordanP | smcginnis, see my github link | 15:01 |
xyang | smcginnis: so manila test with that tempest commit | 15:01 |
xyang | smcginnis: it won't be broken by any checkin in master of tempest | 15:01 |
smcginnis | jordanP: Ah, gotcha. | 15:01 |
*** haomaiwa_ has joined #openstack-cinder | 15:01 | |
jordanP | smcginnis, sorry about your bad experience with QA folks, we try to be nice though :) | 15:01 |
*** akshai has joined #openstack-cinder | 15:01 | |
dulek | https://review.openstack.org/261110 - finally merged! | 15:01 |
smcginnis | jordanP: Haha, they are. | 15:01 |
smcginnis | jordanP: Just got frstrated on this issue. | 15:02 |
eharney | would it be reasonable at all to just start by using the stable n-1 tempest branch for in-tree? | 15:02 |
*** csky has quit IRC | 15:02 | |
dulek | We're one Gerrit restart from getting "Toggle CI" on Gerrits compare revisions pages. :D | 15:02 |
smcginnis | jordanP: For the record, QA folks are unsung heros. ;) | 15:02 |
smcginnis | dulek: Nice! | 15:03 |
smcginnis | eharney: I do think that's reasonable. | 15:03 |
eharney | jordanP: smcginnis: ^ | 15:03 |
*** timcl has joined #openstack-cinder | 15:04 | |
openstackgerrit | Clinton Knight proposed openstack/cinder: Filter & goodness functions in NetApp drivers https://review.openstack.org/278420 | 15:06 |
xyang | smcginnis: there are definitely challenges to have in-tree tempest tests but we can't test some functionalities without them unfortunately as those tests are not accepted by tempest | 15:07 |
smcginnis | xyang: That's how I feel. | 15:07 |
*** lprice has quit IRC | 15:07 | |
*** esp has joined #openstack-cinder | 15:09 | |
jordanP | eharney, there's no "stable branch" for tempest | 15:09 |
*** vgridnev has quit IRC | 15:09 | |
eharney | jordanP: oh, well, ok :) | 15:09 |
*** mtanino has quit IRC | 15:09 | |
jordanP | tempest has been "branch less" for almost 2 years now | 15:09 |
jordanP | it's "always stable" in a way :p | 15:09 |
*** vgridnev has joined #openstack-cinder | 15:11 | |
*** dustins has joined #openstack-cinder | 15:12 | |
*** permalac has quit IRC | 15:13 | |
*** esp has quit IRC | 15:13 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 15:15 |
*** daneyon has joined #openstack-cinder | 15:16 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Job Distribution to support HA A/A https://review.openstack.org/232595 | 15:17 |
*** daneyon__ has joined #openstack-cinder | 15:17 | |
*** vgridnev has quit IRC | 15:17 | |
openstackgerrit | Clinton Knight proposed openstack/cinder: Add variable QoS to NetApp cDOT drivers https://review.openstack.org/278430 | 15:19 |
*** daneyon_ has quit IRC | 15:20 | |
*** daneyon has quit IRC | 15:21 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Do not merge: check purpose https://review.openstack.org/278436 | 15:22 |
*** vincent_hou has joined #openstack-cinder | 15:24 | |
*** daneyon has joined #openstack-cinder | 15:24 | |
*** haypo has left #openstack-cinder | 15:25 | |
*** thangp has joined #openstack-cinder | 15:25 | |
*** daneyon__ has quit IRC | 15:27 | |
xyang | DuncanT: hi, are you okay with this one? https://review.openstack.org/#/c/257170/ | 15:27 |
DuncanT | xyang: Yup, approved | 15:28 |
xyang | DuncanT: you gave +1:) | 15:28 |
xyang | DuncanT: thanks! | 15:28 |
*** mvk has quit IRC | 15:29 | |
DuncanT | xyang: It'll be important when we start doing the error listing stuff | 15:29 |
anteaya | okay so extra-atcs | 15:29 |
anteaya | http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml#n922 | 15:29 |
xyang | DuncanT: sure | 15:29 |
*** mvk has joined #openstack-cinder | 15:29 | |
*** mragupat has joined #openstack-cinder | 15:30 | |
anteaya | so there used to be a file called extra-atcs and it appears it has disappeared in favour of putting the information in projects.yaml in the governance repo | 15:30 |
*** mvk has quit IRC | 15:30 | |
*** yuriy_n17 has joined #openstack-cinder | 15:30 | |
xyang | DuncanT: you probably noticed that there are a bunch of other patches for that same subject. Appreciate you can review some more when you have time:) | 15:30 |
DuncanT | xyang: Sure | 15:31 |
DuncanT | anteaya: Thanks | 15:31 |
anteaya | so smcginnis offers a patch to governance which adds whoever he wants to the projects.yaml file in that format | 15:31 |
*** mvk has joined #openstack-cinder | 15:31 | |
anteaya | welcome do ask me or fungi if there is anything that isn't clear | 15:31 |
anteaya | and offer the patch without delay as summit passes are starting to be issued | 15:31 |
anteaya | and the tc meeting is skipped next week | 15:31 |
*** daneyon_ has joined #openstack-cinder | 15:32 | |
*** rcernin has quit IRC | 15:33 | |
*** vgridnev has joined #openstack-cinder | 15:33 | |
mc_nair | DuncanT: were you able to dig up any of the background info for passing user encryption key in headers on any request that requires encryption? | 15:34 |
*** e0ne has quit IRC | 15:34 | |
*** daneyon has quit IRC | 15:35 | |
xyang | smcginnis: FYI - This one that I talked to DuncanT about (https://review.openstack.org/#/c/257170/) is for an approved cross-project spec https://review.openstack.org/#/c/156508. There are still quite a few patches outstanding | 15:35 |
*** e0ne has joined #openstack-cinder | 15:35 | |
*** wanghao has quit IRC | 15:37 | |
smcginnis | DuncanT, anteaya: Got it, thanks! | 15:38 |
smcginnis | xyang: OK, I'll take a look. | 15:38 |
*** krtaylor has joined #openstack-cinder | 15:38 | |
xyang | smcginnis: thanks | 15:38 |
*** arecknag has quit IRC | 15:39 | |
vincent_hou | eharney: what is your comment for https://review.openstack.org/#/c/250220/? I have not found any yet. | 15:39 |
eharney | vincent_hou: patch set 14 | 15:39 |
anteaya | smcginnis: awesome, thank you | 15:40 |
anteaya | smcginnis: if you can offer today then ttx might be able to get it rolling before he heads off for a week of holidays | 15:40 |
smcginnis | anteaya: Thanks for pointing that out. I had no idea about it. | 15:40 |
*** jungleboyj has joined #openstack-cinder | 15:40 | |
anteaya | smcginnis: glad DuncanT asked about it | 15:40 |
smcginnis | anteaya: Doing it right now. | 15:40 |
vincent_hou | eharney: you mean 'introduce a driver flag like _sparse_copy_volume_data'? | 15:40 |
anteaya | yeah so tough to keep folks informed about all the things | 15:40 |
anteaya | thanks for asking | 15:40 |
*** lprice has joined #openstack-cinder | 15:41 | |
vincent_hou | eharney: I added it in the new patch set. | 15:41 |
DuncanT | xyang: I've found 4 cinder-client patches for request id, I've got them all queued for review today | 15:42 |
eharney | vincent_hou: ok let me take a look | 15:42 |
xyang | DuncanT: great. thanks! | 15:42 |
vincent_hou | eharney: thx. | 15:43 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Do not merge: check purpose https://review.openstack.org/278436 | 15:44 |
*** vgridnev has quit IRC | 15:45 | |
*** vgridnev has joined #openstack-cinder | 15:45 | |
*** shyama has quit IRC | 15:45 | |
*** liverpooler has quit IRC | 15:46 | |
*** nkrinner has quit IRC | 15:49 | |
*** jseiler has joined #openstack-cinder | 15:53 | |
*** mtanino has joined #openstack-cinder | 15:56 | |
*** haomaiwa_ has quit IRC | 16:01 | |
*** sid14726 has joined #openstack-cinder | 16:01 | |
*** rcernin has joined #openstack-cinder | 16:01 | |
*** Guest67795 is now known as cfouts | 16:01 | |
*** haomaiwa_ has joined #openstack-cinder | 16:01 | |
*** salv-orlando has quit IRC | 16:01 | |
*** esp has joined #openstack-cinder | 16:04 | |
*** kmartin has joined #openstack-cinder | 16:05 | |
smcginnis | jbernard: Can you join the meeting? | 16:09 |
jbernard | smcginnis: yep | 16:10 |
jbernard | smcginnis: i thought noon | 16:10 |
smcginnis | jbernard: Thanks! | 16:10 |
*** esp has quit IRC | 16:11 | |
*** rajinir has joined #openstack-cinder | 16:11 | |
*** esker has joined #openstack-cinder | 16:12 | |
openstackgerrit | Merged openstack/python-cinderclient: Add Wrapper classes for list, dict, tuple https://review.openstack.org/257170 | 16:16 |
*** amoturi has joined #openstack-cinder | 16:16 | |
*** esker has quit IRC | 16:16 | |
*** garthb has joined #openstack-cinder | 16:17 | |
*** esp has joined #openstack-cinder | 16:17 | |
*** esker has joined #openstack-cinder | 16:17 | |
*** cbader has quit IRC | 16:22 | |
*** rcernin has quit IRC | 16:24 | |
*** esp has quit IRC | 16:24 | |
*** pradipm has quit IRC | 16:28 | |
*** raildo-afk is now known as raildo | 16:29 | |
*** jdurgin1 has joined #openstack-cinder | 16:31 | |
*** kfarr has joined #openstack-cinder | 16:33 | |
*** boris-42 has quit IRC | 16:33 | |
*** laughterwym has quit IRC | 16:34 | |
*** vgridnev has quit IRC | 16:36 | |
*** knomura has joined #openstack-cinder | 16:36 | |
*** jgriffith_away is now known as jgriffith | 16:37 | |
*** belmoreira has quit IRC | 16:41 | |
*** mudassirlatif has quit IRC | 16:41 | |
*** pradipm has joined #openstack-cinder | 16:42 | |
*** pradipm has quit IRC | 16:44 | |
*** alonma_ has quit IRC | 16:44 | |
*** alonma has joined #openstack-cinder | 16:45 | |
*** alonma has quit IRC | 16:49 | |
*** salv-orlando has joined #openstack-cinder | 16:50 | |
*** alonma has joined #openstack-cinder | 16:51 | |
*** apoorvad has joined #openstack-cinder | 16:51 | |
*** knomura has quit IRC | 16:51 | |
*** asselin has quit IRC | 16:52 | |
*** marcusvrn_ has joined #openstack-cinder | 16:53 | |
*** alonma has quit IRC | 16:55 | |
openstackgerrit | Michael Price proposed openstack/cinder: NetApp: E-Series remove snapshot limitations https://review.openstack.org/278515 | 16:55 |
mriedem | diablo_rojo: here is a great example of unnecessary DocImpact https://review.openstack.org/#/c/245827/ | 16:55 |
mriedem | b/c there are no docs for package requirements | 16:56 |
mriedem | especially unused ones | 16:56 |
mriedem | derp | 16:56 |
*** asselin has joined #openstack-cinder | 16:57 | |
diablo_rojo | mriedem: Ha ha thanks | 16:57 |
*** alonma has joined #openstack-cinder | 16:57 | |
*** leeantho has joined #openstack-cinder | 16:59 | |
*** jungleboyj has quit IRC | 16:59 | |
*** asselin__ has joined #openstack-cinder | 16:59 | |
jgriffith | mriedem: Nice one!!! | 17:00 |
*** haomaiwa_ has quit IRC | 17:01 | |
*** asselin has quit IRC | 17:01 | |
smcginnis | :) | 17:01 |
*** haomaiwang has joined #openstack-cinder | 17:01 | |
DuncanT | jgriffith: I'm worried that taking /too/ simple a solution will result in a design that is hard to move forward and require us to do another redesign with nasty migration/upgrade cases again | 17:01 |
jgriffith | DuncanT: haha, that's funny I thought the opposite | 17:02 |
geguileo | DuncanT: Would you be ok if first it gets implemented with polling | 17:02 |
*** alonma has quit IRC | 17:02 | |
jgriffith | DuncanT: but you and geguileo have way more insight here than I do so that's cool | 17:02 |
jgriffith | DuncanT: geguileo just throwing it out there | 17:02 |
geguileo | jgriffith: Oh, and that's why I brought it up | 17:02 |
jgriffith | we tend to build some overly complex things that end up being less than maintainable | 17:02 |
geguileo | jgriffith: Looking for an simpler alternative :-) | 17:02 |
smcginnis | jgriffith: +1 | 17:02 |
jgriffith | geguileo: :) | 17:02 |
smcginnis | bbl | 17:03 |
geguileo | jgriffith: That's what I thought when I was writting the specs | 17:03 |
jgriffith | geguileo: I wish I had something more concrete to offer up right now | 17:03 |
geguileo | We should try to do something less complicated | 17:03 |
DuncanT | geguileo: If you can explain how it would evolve in future (roughly) then maybe... I'm not a big fan of the new job distribution stuff already, I liked the cluster attribute stuff much more | 17:03 |
geguileo | DuncanT: Well, it's basically the same | 17:03 |
jgriffith | geguileo: effecting the scheduler seems like the right thing... which then comes back to "everything" should be going through the scheduler | 17:03 |
geguileo | DuncanT: But I just skip the whole renaming host to cluster | 17:03 |
geguileo | DuncanT: The rest is the same (same problems) | 17:04 |
DuncanT | geguileo: That makes the host disable stuff harder, since identifying a host is harder now | 17:04 |
geguileo | jgriffith: The problem with the scheduler approach is that it sends it to the same queue for all the nodes | 17:04 |
DuncanT | geguileo: host now means different things in different places | 17:04 |
geguileo | DuncanT: I believe it means the same everywhere | 17:04 |
DuncanT | geguileo: For when you're not talking to c-vol, it means a host.... when you're talking to c-vol it means several hosts | 17:05 |
*** alonma has joined #openstack-cinder | 17:05 | |
geguileo | DuncanT: But isn't that the same for the scheduler? | 17:06 |
DuncanT | geguileo: No... schedulers can run on different hosts... | 17:06 |
geguileo | DuncanT: Your are right, we just don't address them directly | 17:07 |
DuncanT | geguileo: Yup. Maybe we should do the same for c-vol - add 'cluster', which is a service name like scheduler is? | 17:07 |
DuncanT | geguileo: Old code will not be H/A aware, but that is ok | 17:07 |
DuncanT | geguileo: So both the hsot and the cluster queues will be there, and we can make stuff H/A (cluster) aware a bit at a time? | 17:08 |
*** e0ne has quit IRC | 17:08 | |
geguileo | DuncanT: It will take 3 releases to make the change | 17:08 |
geguileo | Rename host to cluster | 17:08 |
DuncanT | geguileo: No, don't rename, have both | 17:08 |
*** sid14726 has quit IRC | 17:09 | |
geguileo | DuncanT: But at some point host field will be not used | 17:09 |
geguileo | And we will want to drop it | 17:09 |
DuncanT | geguileo: host will be used for things like disables | 17:09 |
DuncanT | geguileo: and differentiating where things came from (stats updates etc) for debugging) | 17:09 |
*** alonma has quit IRC | 17:10 | |
DuncanT | geguileo: Maybe we drop it in the volumes table eventually, but initially leave it there | 17:10 |
geguileo | That's the same as we would have with this proposal host and node_name instead of cluster and node_name | 17:10 |
geguileo | DuncanT: If you really hate the concept of using host instead of cluster | 17:10 |
geguileo | I will take the renaming approach | 17:11 |
DuncanT | geguileo: give me a few minutes to type this up | 17:11 |
geguileo | But I think it will be a lot worse for us in the code and with backports | 17:11 |
*** alonma has joined #openstack-cinder | 17:11 | |
* mc_nair puts on his best salesman face... | 17:12 | |
mc_nair | do you know what's been missing from *your* life? Nested quota reviews.... but for a limited time.... https://review.openstack.org/#/c/274825/ | 17:12 |
*** xyang has quit IRC | 17:13 | |
*** xyang has joined #openstack-cinder | 17:13 | |
hemna | mc_nair, :) | 17:13 |
hemna | os-brick + privsep update | 17:13 |
hemna | gus and I have been working on getting his patch against os-brick working | 17:14 |
hemna | today is the first time that I've seen it work. | 17:14 |
hemna | http://paste.openstack.org/show/486583/ | 17:14 |
hemna | I'm not sure it's going to make the deadline for friday | 17:14 |
hemna | I've been using my brick_initiator.py script that lives in my diediedie repo on github. http://github.com/WaltHP/diediedie | 17:15 |
hemna | I'm going to start testing it against nova/cinder today | 17:15 |
*** vgridnev has joined #openstack-cinder | 17:15 | |
*** alonma has quit IRC | 17:16 | |
hemna | the os-brick privsep patch : https://review.openstack.org/#/c/277224/ | 17:17 |
mc_nair | hemna: is there anything I could help w/ review for the time being on that? or is it mostly just testing / stabilizing at this point? | 17:17 |
hemna | mc_nair, I haven't gone through the CI failures at this point because it just hasn't worked locally until today. | 17:18 |
mc_nair | gotcha | 17:18 |
hemna | I need to go through those as well as doing some tests | 17:18 |
*** alonma has joined #openstack-cinder | 17:18 | |
hemna | I'm going to try testing iSCSI first, then FC | 17:18 |
*** arch-nemesis has joined #openstack-cinder | 17:22 | |
hemna | cinder list | 17:22 |
hemna | ERROR: Versioning for this project requires either an sdist tarball, or access to an upstream git repository. Are you sure that git is installed? | 17:22 |
hemna | huh | 17:22 |
hemna | never seen that one before | 17:22 |
*** alonma has quit IRC | 17:22 | |
diablo_rojo | hemna: I've seen that one before, but I don't remember what had caused it.. | 17:23 |
diablo_rojo | hemna: Part of me wants to say it was some connection issue and when I reran it was fine. Idk. I'd have to go look. | 17:23 |
hemna | I get it every time | 17:23 |
hemna | it's odd. | 17:23 |
*** vincent_hou has quit IRC | 17:24 | |
DuncanT | geguileo: https://gist.github.com/Funcan/321e955ee4990f3c3ed2 | 17:26 |
diablo_rojo | hemna: Well do you have git installed? ;) | 17:27 |
DuncanT | hemna: I've seen that before on a devstack... I made a new one rather than trying to fix it | 17:27 |
hemna | hehe yah, that's probably what I'll do. | 17:27 |
hemna | fire up a new vagrant and be done with it. | 17:28 |
geguileo | DuncanT: I don't think that would work... | 17:29 |
*** leeantho has quit IRC | 17:29 | |
geguileo | DuncanT: What would you set in the host field of the Volume table? | 17:29 |
geguileo | DuncanT: As it should be the cluster | 17:30 |
DuncanT | geguileo: As it said, pick any host in the cluster | 17:30 |
hemna | gus, ping | 17:30 |
DuncanT | geguileo: For calls we've moved over to go via the scheduler, it will be ignored | 17:30 |
*** mudassirlatif has joined #openstack-cinder | 17:30 | |
geguileo | DuncanT: Then when we have to decide on which cluster it is, how do we do it? | 17:30 |
DuncanT | geguileo: The scheduler will know via the stats broadcast from the c-vol node which cluster it is in, if any | 17:31 |
*** mylu has joined #openstack-cinder | 17:31 | |
geguileo | Create will ignore it, but all other operations will pay attention to it | 17:31 |
DuncanT | geguileo: It will be *a* valid host to talk to that volume, every time, just not necessarily the only one | 17:31 |
geguileo | But it could be down | 17:31 |
DuncanT | geguileo: Yes. We fix the other operations to go via the scheduler one by one. Until they're all done (and there aren't that many) then you don't get HA | 17:32 |
DuncanT | geguileo: It means we can move (and test) them one by one | 17:33 |
DuncanT | geguileo: It's more patches, but each one will be simple | 17:33 |
geguileo | So the scheduler would decide which node gets to do each operation from the up and not disabled ones? | 17:33 |
DuncanT | Yes, as jgriffith suggested | 17:34 |
*** krtaylor has quit IRC | 17:34 | |
geguileo | But then we could be losing requests | 17:34 |
geguileo | Since we don't know immediately when a node is down | 17:34 |
geguileo | We could have sent that node a lot of requests until the heartbeat expires | 17:34 |
geguileo | I discarded the option of the scheduler choosing the specific node | 17:35 |
geguileo | When I started looking into this because that would create a lot of issues | 17:35 |
DuncanT | geguileo: The scheduler can send each rpc to the cluster and let rabbit pick a node... then you have to do host disabling in the manager, but that might be ok | 17:36 |
*** porrua has quit IRC | 17:37 | |
DuncanT | the disable can stop the node listening to the cluster queue, but not the node queue (so it can get enabled again) | 17:37 |
DuncanT | Disable would behave the same as today for none H/A calls, but that's ok too | 17:37 |
geguileo | That's the alternative I was proposing to the DB polling | 17:37 |
geguileo | But without changing everything to go through the scheduler | 17:38 |
DuncanT | Yup, I get that now. I think it is part of the solution for sure. Moving everything to go via the scheduler makes upgrade easier I think | 17:38 |
geguileo | Because my experience with creating a lot of patches for something is *not* good | 17:38 |
geguileo | DuncanT: I don't want to mix things | 17:38 |
DuncanT | Well, being too clever and doing things implicitly has hurt us before too | 17:39 |
*** boris-42 has joined #openstack-cinder | 17:39 | |
geguileo | What do you mean by doing things implicitly? | 17:39 |
DuncanT | Changing the meaning of the host field to not mean a host (yes, I'm aware people hack it today, but that comes with its own set of weird behaviours) | 17:40 |
DuncanT | Actually, if you add the cluster field, does it need things to go via the scheduler? If there's a cluster set, send the RPC there, job done | 17:41 |
geguileo | DuncanT: Ok, so for you using host is a no go | 17:41 |
geguileo | DuncanT: I can do everything exactly the same but using cluster field instead | 17:41 |
geguileo | DuncanT: I don't need to pass everything through the scheduler | 17:41 |
DuncanT | You'd need some way to set the cluster attribute for all existing volumes, but you could add an API call and let the scheduler fill them in in batches | 17:41 |
geguileo | DuncanT: That's not a problem, because that's part of the reason why we use 3 releases for the change | 17:42 |
geguileo | Since we support rolling upgrades | 17:42 |
geguileo | First we write to both fields the same value and only use the host field | 17:43 |
geguileo | Then we write to both values and use the cluster field | 17:43 |
DuncanT | geguileo: I don't think you need 3 releases | 17:43 |
geguileo | Finally we only read and write from the cluster field | 17:43 |
geguileo | And you are finally able to drop the host field | 17:43 |
geguileo | The last 2 happen on the same rolling upgrade | 17:44 |
DuncanT | Why write both and only use the host? | 17:44 |
geguileo | Because you have to be compatible while doing rolling upgrades | 17:44 |
geguileo | And you want to start populating the cluster field | 17:45 |
DuncanT | You can write it and use it if it is there, in the same release I think - it will never get set until there's a c-vol ready to receive it, and a scheduler filling it in | 17:45 |
*** krtaylor has joined #openstack-cinder | 17:45 | |
geguileo | You cannot use it in the first release | 17:45 |
geguileo | Because some old node may have only written to host | 17:46 |
DuncanT | Why not? | 17:46 |
geguileo | While you were upgrading | 17:46 |
geguileo | So host and cluster would be different | 17:46 |
DuncanT | If cluster is None, use host | 17:46 |
geguileo | It's not that simple | 17:46 |
DuncanT | Why not? | 17:47 |
*** eharney has quit IRC | 17:47 | |
geguileo | Because we may create it with a new version and migrate it with an old version | 17:48 |
geguileo | So we would end up with the cluster and the host being different | 17:48 |
geguileo | And the right one is the host | 17:48 |
geguileo | That's part of the pain of implementing this together with rolling upgrades | 17:48 |
geguileo | *Everything* is more complicated | 17:49 |
*** dustins has quit IRC | 17:49 | |
DuncanT | Ah, got you. Give me a few minutes to think, please :-) | 17:49 |
geguileo | DuncanT: Thanks :-) | 17:49 |
DuncanT | Pretty sure we can do it in one release via version pin walking, but I need to check | 17:50 |
DuncanT | It's a bit of a hack, but I'll try to gist it | 17:50 |
*** timcl has quit IRC | 17:52 | |
geguileo | DuncanT: I really hope you can figure out how to do it in just 1 release :-) | 17:52 |
*** rcernin has joined #openstack-cinder | 17:53 | |
*** martyturner has quit IRC | 17:53 | |
diablo_rojo | https://review.openstack.org/#/c/245629/ Anyone else care to have an opinion on a common policy scenario across all projects? DuncanT? eharney? Already got input from jgriffith and jungleboyj- thank you again :) | 17:55 |
*** sid14726 has joined #openstack-cinder | 17:55 | |
*** knomura has joined #openstack-cinder | 17:56 | |
*** knomura has quit IRC | 17:56 | |
*** mvk has quit IRC | 17:59 | |
*** martyturner has joined #openstack-cinder | 17:59 | |
*** haomaiwang has quit IRC | 18:01 | |
*** haomaiwa_ has joined #openstack-cinder | 18:01 | |
*** dustins has joined #openstack-cinder | 18:01 | |
*** igornsa_ has joined #openstack-cinder | 18:03 | |
*** esker has quit IRC | 18:04 | |
*** krtaylor has quit IRC | 18:08 | |
*** ociuhandu_ has joined #openstack-cinder | 18:08 | |
*** itlinux has quit IRC | 18:11 | |
*** ociuhandu has quit IRC | 18:11 | |
*** ociuhandu_ is now known as ociuhandu | 18:11 | |
*** porrua has joined #openstack-cinder | 18:13 | |
*** mylu has quit IRC | 18:16 | |
*** mylu has joined #openstack-cinder | 18:17 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove Cisco FC Zone Manager Driver https://review.openstack.org/278201 | 18:18 |
*** alonma has joined #openstack-cinder | 18:18 | |
*** jdurgin1 has quit IRC | 18:19 | |
*** alonma_ has joined #openstack-cinder | 18:19 | |
*** krtaylor has joined #openstack-cinder | 18:20 | |
*** e0ne has joined #openstack-cinder | 18:20 | |
openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning behavior https://review.openstack.org/275408 | 18:22 |
*** cbader has joined #openstack-cinder | 18:23 | |
*** alonma has quit IRC | 18:23 | |
*** alonma_ has quit IRC | 18:24 | |
DuncanT | diablo_rojo: Commented | 18:25 |
*** alonma has joined #openstack-cinder | 18:26 | |
diablo_rojo | DuncanT: Thank you :) | 18:26 |
DuncanT | geguileo: Im pretty sure I know how to do it, as long as nobody thinks it is too much of a hack. Thanks for going through the problems with me slowly until I understood them :-) | 18:26 |
geguileo | DuncanT: So you found a way to do it in only 1 release? | 18:27 |
geguileo | That's awesome!! | 18:27 |
*** mylu has quit IRC | 18:28 | |
*** mriedem has quit IRC | 18:28 | |
*** jordanP has quit IRC | 18:29 | |
geguileo | DuncanT: Can you tell me the general idea? | 18:29 |
geguileo | Or are you going to write something? | 18:29 |
*** ChubYann has joined #openstack-cinder | 18:29 | |
*** alonma has quit IRC | 18:30 | |
*** IlyaG has joined #openstack-cinder | 18:32 | |
*** eharney has joined #openstack-cinder | 18:33 | |
*** leeantho has joined #openstack-cinder | 18:34 | |
*** mriedem has joined #openstack-cinder | 18:36 | |
DuncanT | I'll write something | 18:38 |
DuncanT | Just having dinner | 18:39 |
geguileo | DuncanT: Ok, we can talk tomorrow, thanks | 18:39 |
*** JoseMello has joined #openstack-cinder | 18:41 | |
*** martyturner has quit IRC | 18:41 | |
*** martyturner has joined #openstack-cinder | 18:42 | |
*** cbader_ has joined #openstack-cinder | 18:46 | |
*** jgregor has quit IRC | 18:46 | |
*** liverpooler has joined #openstack-cinder | 18:47 | |
*** sid14726 has quit IRC | 18:49 | |
*** vincent_hou has joined #openstack-cinder | 18:49 | |
*** cbader has quit IRC | 18:50 | |
*** eharney has quit IRC | 18:50 | |
vincent_hou | patrickeast, smcginnis: Hi. I was wondering if you can help to look at this patch https://review.openstack.org/#/c/237387/. | 18:51 |
*** bardia has joined #openstack-cinder | 18:52 | |
*** leeantho has quit IRC | 18:52 | |
*** lpetrut has quit IRC | 18:53 | |
*** xyang has quit IRC | 18:53 | |
*** xyang has joined #openstack-cinder | 18:54 | |
*** mylu has joined #openstack-cinder | 18:54 | |
*** dramakri has joined #openstack-cinder | 18:57 | |
ameade | DuncanT, eharney: https://review.openstack.org/#/c/274958/ another look? | 18:59 |
DuncanT | ameade: Added to my pile | 18:59 |
ameade | ty sir | 18:59 |
*** thurloat is now known as thurloat_isgone | 19:00 | |
*** haomaiwa_ has quit IRC | 19:01 | |
*** haomaiwang has joined #openstack-cinder | 19:01 | |
*** eharney has joined #openstack-cinder | 19:03 | |
*** igornsa_ has quit IRC | 19:04 | |
*** xyang has quit IRC | 19:08 | |
*** xyang1 has quit IRC | 19:09 | |
*** mylu_ has joined #openstack-cinder | 19:11 | |
*** mylu has quit IRC | 19:15 | |
DuncanT | geguileo: https://gist.github.com/Funcan/7d21172017cb905fb71d | 19:18 |
*** mylu_ has quit IRC | 19:19 | |
openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 19:21 |
*** mylu has joined #openstack-cinder | 19:21 | |
*** mylu has quit IRC | 19:23 | |
*** mylu has joined #openstack-cinder | 19:24 | |
scottda | smcginnis: You back from meetings? | 19:25 |
*** alonma has joined #openstack-cinder | 19:26 | |
*** mylu has quit IRC | 19:27 | |
*** mylu has joined #openstack-cinder | 19:30 | |
*** alonma has quit IRC | 19:30 | |
*** mylu has quit IRC | 19:31 | |
*** angela-s has joined #openstack-cinder | 19:32 | |
*** alonma has joined #openstack-cinder | 19:32 | |
*** fthiagogv has quit IRC | 19:32 | |
*** jgregor has joined #openstack-cinder | 19:32 | |
*** jgregor has quit IRC | 19:33 | |
*** mylu has joined #openstack-cinder | 19:35 | |
*** jgregor has joined #openstack-cinder | 19:35 | |
*** jungleboyj has joined #openstack-cinder | 19:36 | |
*** alonma has quit IRC | 19:37 | |
*** mylu has quit IRC | 19:37 | |
*** sid14726 has joined #openstack-cinder | 19:38 | |
openstackgerrit | Merged openstack/cinder: HNAS driver: retry on Connection reset fix https://review.openstack.org/276357 | 19:41 |
*** mylu has joined #openstack-cinder | 19:42 | |
*** mylu has quit IRC | 19:43 | |
*** rajinir has quit IRC | 19:45 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: [LVM] Restore target config during ensure_export https://review.openstack.org/271424 | 19:45 |
*** krotscheck is now known as krotscheck_dcm | 19:47 | |
*** ociuhandu has quit IRC | 19:53 | |
*** merooney has quit IRC | 19:55 | |
*** jgriffith is now known as jgriffith_away | 19:57 | |
openstackgerrit | Merged openstack/cinder: Added osprofiler headers to cors middleware https://review.openstack.org/275751 | 19:57 |
*** jgregor has quit IRC | 19:57 | |
smcginnis | scottda: Back now. | 19:58 |
*** haomaiwang has quit IRC | 20:01 | |
*** haomaiwang has joined #openstack-cinder | 20:01 | |
DuncanT | jgriffith: You missed a bunch of comments from Xing on patch set 3 I think? | 20:01 |
*** mylu has joined #openstack-cinder | 20:01 | |
DuncanT | jgriffith_away: For replication | 20:01 |
scottda | smcginnis: I was wondering about the scheduling of a demo for microversions... | 20:03 |
scottda | You and hemna cannot come Fri morning. Should we push to Monday-ish? | 20:04 |
smcginnis | scottda: I'd like to be there, but I also don't want to be a roadblock. | 20:04 |
smcginnis | scottda: If it works best Friday then do it and I can catch up on the recording. | 20:04 |
smcginnis | scottda: Otherwise, Monday would work for me other than 1-2 CST. | 20:04 |
hemna | scottda, Monday would be better for me | 20:04 |
hemna | I can't do Friday morning | 20:04 |
smcginnis | Sorry, 1-2:30 | 20:05 |
scottda | OK. I'll just plan for Monday, morning like 16:00 UTC/9:00 MST | 20:05 |
scottda | and I'll send something out on the ML | 20:05 |
smcginnis | scottda: Works for me. | 20:05 |
scottda | thanks | 20:05 |
smcginnis | scottda: Thanks for arranging it. | 20:05 |
* scottda is off to lunch | 20:05 | |
hemna | ok thanks man | 20:06 |
*** jgregor has joined #openstack-cinder | 20:06 | |
*** xyang has joined #openstack-cinder | 20:08 | |
*** alonma has joined #openstack-cinder | 20:09 | |
*** xyang1 has joined #openstack-cinder | 20:12 | |
openstackgerrit | Kendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 20:13 |
*** andymaier has quit IRC | 20:13 | |
*** lpetrut has joined #openstack-cinder | 20:13 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Set volume listen port to test_service_listen_port https://review.openstack.org/278594 | 20:13 |
openstackgerrit | Eric Harney proposed openstack/cinder: Profiler: don't call trace_cls if profiler is not enabled https://review.openstack.org/278595 | 20:13 |
*** alonma has quit IRC | 20:13 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Profiler: make it possible to run without loading osprofiler https://review.openstack.org/278596 | 20:13 |
*** alonma has joined #openstack-cinder | 20:15 | |
openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 20:16 |
*** alonma has quit IRC | 20:20 | |
*** jgriffith_away is now known as jgriffith | 20:22 | |
*** dave-mcc_ has joined #openstack-cinder | 20:22 | |
*** dave-mccowan has quit IRC | 20:22 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI https://review.openstack.org/278605 | 20:22 |
*** alonma has joined #openstack-cinder | 20:22 | |
*** dave-mccowan has joined #openstack-cinder | 20:23 | |
*** ociuhandu has joined #openstack-cinder | 20:23 | |
hemna | new gerrit ui, I hate you. that is all. | 20:26 |
e0ne | :) | 20:26 |
smcginnis | The rants are getting much shorter. :) | 20:26 |
sheel | :) | 20:26 |
*** dave-mcc_ has quit IRC | 20:26 | |
hemna | my tapeworm tells me what to do. he's telling me to keep it brief. | 20:27 |
e0ne | hemna: any updates about privsep? I didn't have time to take a look on it today:( | 20:27 |
hemna | e0ne, so the latest patch works from the brick_initiator.py standpoint. | 20:27 |
hemna | but.... | 20:27 |
hemna | it makes nova lock up. | 20:27 |
hemna | ship it! | 20:27 |
*** alonma has quit IRC | 20:27 | |
hemna | e0ne, https://review.openstack.org/#/c/277224/ | 20:28 |
hemna | I just posted a review on it. | 20:28 |
smcginnis | Sounds like a nova problem. | 20:28 |
smcginnis | jk | 20:28 |
hemna | hehe | 20:28 |
hemna | e0ne, http://paste.openstack.org/show/486583/ | 20:28 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 20:28 |
hemna | e0ne, that's it 'working' | 20:28 |
hemna | at least from my command line script viewpoint. | 20:29 |
e0ne | :) | 20:29 |
hemna | I haven't had time to look into the n-cpu lock up yet. | 20:29 |
diablo_rojo | patrickeast: Hello today :) | 20:29 |
openstackgerrit | Sheel Rana proposed openstack/cinder: Api-microversions patch for bug #1535749 https://review.openstack.org/278605 | 20:29 |
openstack | bug 1535749 in python-cinderclient "cinder client does not have handling for "--bootable" parameter for cinder list command" [Medium,Fix released] https://launchpad.net/bugs/1535749 - Assigned to Sheel Rana (ranasheel2000) | 20:29 |
vincent_hou | eharney: Eric, could you take a look at https://review.openstack.org/#/c/250220/ to see if it is better? | 20:30 |
*** hemna is now known as hemnafk | 20:30 | |
e0ne | hemna: does it need any config changes for nova? | 20:30 |
*** e0ne has quit IRC | 20:32 | |
*** dims has quit IRC | 20:34 | |
sheel | diablo_rojo: | 20:34 |
sheel | diablo_rojo: hi | 20:34 |
*** vincent_hou has quit IRC | 20:34 | |
*** dims has joined #openstack-cinder | 20:35 | |
*** sid14726 has quit IRC | 20:37 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched https://review.openstack.org/246992 | 20:38 |
openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver https://review.openstack.org/268344 | 20:38 |
*** chhavi has quit IRC | 20:40 | |
*** IlyaG has quit IRC | 20:41 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: Api-microversions patch for bug #1535749 https://review.openstack.org/278605 | 20:41 |
openstack | bug 1535749 in python-cinderclient "cinder client does not have handling for "--bootable" parameter for cinder list command" [Medium,Fix released] https://launchpad.net/bugs/1535749 - Assigned to Sheel Rana (ranasheel2000) | 20:41 |
openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver https://review.openstack.org/268344 | 20:42 |
*** chris_morrell has joined #openstack-cinder | 20:43 | |
diablo_rojo | sheel: Hello | 20:43 |
sheel | diablo_rojo: could not find exactly where to raise bp in cross projects | 20:44 |
sheel | I was looking in at https://launchpad.net/openstack/ | 20:44 |
sheel | diablo_rojo: but it again is specific to project | 20:44 |
sheel | :( | 20:44 |
diablo_rojo | sheel: You want to create a spec, not a bp | 20:44 |
*** alexpilotti_ has joined #openstack-cinder | 20:44 | |
diablo_rojo | sheel: There is a cinder-specs repo and an openstack-specs repo | 20:45 |
sheel | yup, will create spec | 20:45 |
*** jgriffith is now known as jgriffith_away | 20:45 | |
smcginnis | sheel: https://github.com/openstack/openstack-specs | 20:46 |
diablo_rojo | smcginnis: Ha ha was just about to link that. Thanks :) | 20:46 |
sheel | smcginnis: diablo_rojo thanks | 20:46 |
*** knikolla has quit IRC | 20:50 | |
*** xyang has quit IRC | 20:50 | |
*** kfarr has quit IRC | 20:51 | |
sheel | smcginnis: As we are going to deprecate cli part...I think I should leave implementing https://blueprints.launchpad.net/python-cinderclient/+spec/retries-for-overlimit-response | 20:51 |
sheel | smcginnis: may be by marking it obsolete!! | 20:52 |
*** e0ne has joined #openstack-cinder | 20:53 | |
smcginnis | sheel: We're not going to deprecate our CLI yet. | 20:54 |
smcginnis | sheel: There's a lot of work to do with openstack client first. | 20:55 |
smcginnis | sheel: Eventually we may want to, but it will be a while before we get to that point. | 20:55 |
smcginnis | sheel: So I think your change should still go through. | 20:55 |
smcginnis | sheel: Thanks for checking! | 20:55 |
sheel | smcginnis: ok | 20:55 |
sheel | so, please approve as well.. | 20:55 |
smcginnis | sheel: Hah, I'll take a look in a bit. | 20:55 |
sheel | smcginnis: I will take it ahead in next week for sure.. | 20:55 |
sheel | smcginnis: thanks... | 20:56 |
openstackgerrit | Edward Hope-Morley proposed openstack/cinder: Conditionally restore display_name https://review.openstack.org/278629 | 20:57 |
openstackgerrit | Sheel Rana proposed openstack/cinder: Added Bootable filter for cinder list command https://review.openstack.org/278605 | 20:58 |
*** apoorvad has quit IRC | 20:59 | |
*** merooney has joined #openstack-cinder | 20:59 | |
*** haomaiwang has quit IRC | 21:01 | |
*** haomaiwang has joined #openstack-cinder | 21:01 | |
scottda | smcginnis: hemnafk Forgot, monday is a holiday. I'll schedule demo for Tues | 21:03 |
*** hemnafk is now known as hemna | 21:03 | |
hemna | ooh yah I forgot that too | 21:03 |
smcginnis | scottda: What are you, a government worker? :) | 21:03 |
*** haomaiw__ has joined #openstack-cinder | 21:04 | |
scottda | :) | 21:04 |
*** sinese_ has quit IRC | 21:04 | |
smcginnis | scottda: Does HPE have off Monday? | 21:04 |
hemna | yah I think so | 21:05 |
scottda | not sure about that. But kids have it off, so trip to Denver, etc. | 21:05 |
smcginnis | Ah, nice. | 21:05 |
scottda | We're hiring. | 21:05 |
* smcginnis brushes up resume | 21:06 | |
smcginnis | Hah! | 21:06 |
hemna | :) | 21:06 |
*** haomaiwang has quit IRC | 21:06 | |
scottda | hemna: How's the champagne and caviare in Roseville today? I think yesterday's was much better. | 21:07 |
smcginnis | Lol | 21:07 |
hemna | it was a bit different, they only had barrel aged stout and other beers today. I think we ran out of the caviar from our free lunches every day this week. | 21:08 |
*** e0ne has quit IRC | 21:09 | |
smcginnis | I get free coffee. :] | 21:09 |
diablo_rojo | scottda: Free cadillacs? ;) | 21:10 |
*** porrua has quit IRC | 21:10 | |
scottda | I ain't going there... | 21:10 |
* smcginnis shakes head | 21:10 | |
diablo_rojo | scottda: But I thought you liked cadaillacs and ham sandwhiches? | 21:11 |
diablo_rojo | scottda: Did you get a text last weekend from a number you didn't recognize? | 21:12 |
diablo_rojo | jungleboyj: ^^ | 21:13 |
scottda | yup, found out it was jay. | 21:13 |
diablo_rojo | scottda: Ha ha good. | 21:13 |
*** merooney has quit IRC | 21:14 | |
jungleboyj | scottda: How was that ham sandwich the other day? | 21:16 |
jungleboyj | smcginnis: So when is the demo now? | 21:17 |
scottda | I'm going to schedule the demo for Tues 16:00 UTC/9:00 MST . Does that work for you jungleboyj ? | 21:18 |
smcginnis | scottda: Works for me. | 21:18 |
jungleboyj | scottda: Yep. Cool. | 21:18 |
jungleboyj | Putting it on the calendar. | 21:18 |
*** alexpilotti_ has quit IRC | 21:19 | |
scottda | hemna: Is there any magic to streaming a hangout to youtube cinder channel? | 21:21 |
hemna | scottda, there are a few steps | 21:21 |
hemna | scottda, https://support.google.com/plus/answer/2553119?hl=en | 21:22 |
*** akerr has quit IRC | 21:23 | |
smcginnis | hemna: Do you need to do the streaming, or is scottda able to stream to the cinder channel? | 21:24 |
*** apoorvad has joined #openstack-cinder | 21:24 | |
hemna | whoever owns the youtube channel can stream to it | 21:25 |
hemna | when I'm on hangouts, I just switch my account over to the openstack-cinder user and then start a hangout on air | 21:26 |
smcginnis | scottda: So I probably need to add your account to that group. | 21:26 |
jgriffith_away | eharney: did you have new/outstanding patches on that DB unicode issue, or were you just pointing out that you started the work on volumes and the *rest* needs done? | 21:26 |
*** alonma has joined #openstack-cinder | 21:26 | |
scottda | smcginnis: is it as simple as adding Scott.dangelo@gmail ? | 21:27 |
smcginnis | scottda: I think so. I'll let you know if I need anything more than that. | 21:27 |
smcginnis | scottda: Did you get an email? | 21:29 |
*** dustins_ has joined #openstack-cinder | 21:29 | |
eharney | jgriffith_away: the latter | 21:29 |
*** jgriffith_away is now known as jgriffith | 21:29 | |
jgriffith | stoopid auto-away | 21:29 |
*** leeantho has joined #openstack-cinder | 21:29 | |
jgriffith | auto-im-back needs some help | 21:30 |
eharney | :) | 21:30 |
scottda | smcginnis: No email yet | 21:30 |
jgriffith | eharney: ok, thanks... just reading what I missed in the scroll back from this AM's meeting | 21:30 |
*** alonma has quit IRC | 21:30 | |
smcginnis | scottda: OK, I think it should work though. You're added. | 21:31 |
smcginnis | jgriffith: A lot to read there. | 21:31 |
jgriffith | smcginnis: yar, busy meeting | 21:31 |
jgriffith | smcginnis: I caught the second half at least | 21:31 |
smcginnis | No doubt | 21:31 |
smcginnis | jgriffith: Feeling any better today? | 21:31 |
*** dustins has quit IRC | 21:32 | |
*** dustins_ is now known as dustins | 21:33 | |
*** leeantho has quit IRC | 21:33 | |
*** leeantho has joined #openstack-cinder | 21:34 | |
*** diablo_rojo has quit IRC | 21:34 | |
*** shout28 has joined #openstack-cinder | 21:34 | |
openstackgerrit | Ryan McNair proposed openstack/cinder: Adds support for min/max volume size on vol_type https://review.openstack.org/278648 | 21:35 |
*** al_indigo has joined #openstack-cinder | 21:39 | |
*** salv-orl_ has joined #openstack-cinder | 21:39 | |
*** dave-mcc_ has joined #openstack-cinder | 21:39 | |
*** alonma has joined #openstack-cinder | 21:40 | |
*** rwall_ has joined #openstack-cinder | 21:40 | |
*** tpatzig_ has joined #openstack-cinder | 21:40 | |
*** cbader_2 has joined #openstack-cinder | 21:40 | |
*** erlon_ has joined #openstack-cinder | 21:41 | |
*** jordanP has joined #openstack-cinder | 21:41 | |
*** sheel_ has joined #openstack-cinder | 21:41 | |
*** mragupat_ has joined #openstack-cinder | 21:41 | |
*** dramakri1 has joined #openstack-cinder | 21:42 | |
*** dramakri1 has quit IRC | 21:42 | |
*** dramakri1 has joined #openstack-cinder | 21:43 | |
*** smcginni1 has joined #openstack-cinder | 21:44 | |
*** alonma has quit IRC | 21:44 | |
openstackgerrit | Sarvesh Ranjan proposed openstack/cinder: Confusing typos fixed. https://review.openstack.org/278649 | 21:44 |
*** wolsen has joined #openstack-cinder | 21:44 | |
*** bill_az_ has joined #openstack-cinder | 21:44 | |
*** mragupat has quit IRC | 21:45 | |
*** cknight has quit IRC | 21:45 | |
*** andymaier has joined #openstack-cinder | 21:45 | |
*** HenryG_ has joined #openstack-cinder | 21:47 | |
*** raildo is now known as raildo-afk | 21:47 | |
*** cbader_2 has quit IRC | 21:48 | |
*** cbader has joined #openstack-cinder | 21:48 | |
*** leeantho has quit IRC | 21:48 | |
*** dave-mccowan has quit IRC | 21:48 | |
*** dramakri has quit IRC | 21:48 | |
*** cbader_ has quit IRC | 21:48 | |
*** JoseMello has quit IRC | 21:48 | |
*** salv-orlando has quit IRC | 21:48 | |
*** bill_az has quit IRC | 21:48 | |
*** sheel has quit IRC | 21:48 | |
*** erlon has quit IRC | 21:48 | |
*** jgriffith has quit IRC | 21:48 | |
*** tpatzig has quit IRC | 21:48 | |
*** wolsen_ has quit IRC | 21:48 | |
*** smcginnis has quit IRC | 21:48 | |
*** rwall has quit IRC | 21:48 | |
*** zhenguo_ has quit IRC | 21:48 | |
*** al_indig_ has quit IRC | 21:48 | |
*** cfouts has quit IRC | 21:48 | |
*** HenryG has quit IRC | 21:48 | |
*** dhellmann has quit IRC | 21:48 | |
*** xyang has joined #openstack-cinder | 21:48 | |
*** rwall_ is now known as rwall | 21:48 | |
*** HenryG_ is now known as HenryG | 21:48 | |
*** sheel_ is now known as sheel | 21:48 | |
*** gnarld_ has joined #openstack-cinder | 21:48 | |
*** alonma has joined #openstack-cinder | 21:49 | |
*** dhellmann has joined #openstack-cinder | 21:49 | |
*** sgotliv has joined #openstack-cinder | 21:49 | |
*** erlon_ is now known as erlon | 21:49 | |
*** jgriffith has joined #openstack-cinder | 21:51 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/277228 | 21:52 |
*** HenryG has quit IRC | 21:52 | |
*** smcginni1 is now known as smcginnis | 21:52 | |
*** thangp has left #openstack-cinder | 21:53 | |
*** alonma has quit IRC | 21:54 | |
openstackgerrit | Alex O'Rourke proposed openstack/cinder: Pass a host's volumes to the driver upon fail-over https://review.openstack.org/278687 | 21:54 |
*** hemna has quit IRC | 21:55 | |
*** leeantho has joined #openstack-cinder | 21:55 | |
jgriffith | smcginnis: little bit, thanks! | 21:55 |
*** HenryG has joined #openstack-cinder | 21:56 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/278702 | 21:57 |
*** lpetrut has quit IRC | 21:58 | |
*** vgridnev has quit IRC | 21:58 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/266163 | 21:58 |
*** JoseMello has joined #openstack-cinder | 21:58 | |
*** alonma has joined #openstack-cinder | 21:58 | |
*** amoturi has left #openstack-cinder | 21:59 | |
*** zhenguo_ has joined #openstack-cinder | 22:00 | |
*** haomaiw__ has quit IRC | 22:01 | |
*** haomaiwang has joined #openstack-cinder | 22:01 | |
*** alonma has quit IRC | 22:03 | |
openstackgerrit | Alex O'Rourke proposed openstack/cinder: Pass a host's volumes to the driver upon fail-over https://review.openstack.org/278687 | 22:03 |
*** jgregor has quit IRC | 22:03 | |
*** baumann has left #openstack-cinder | 22:04 | |
*** mylu has quit IRC | 22:06 | |
*** cknight has joined #openstack-cinder | 22:06 | |
*** rcernin has quit IRC | 22:07 | |
*** alonma has joined #openstack-cinder | 22:08 | |
*** dustins has quit IRC | 22:10 | |
*** gnarld_ is now known as cfouts | 22:11 | |
openstackgerrit | Merged openstack/cinder: Wrap the method to calculate virtual free capacity https://review.openstack.org/257620 | 22:12 |
*** alonma has quit IRC | 22:13 | |
thingee | xyang: http://lists.openstack.org/pipermail/openstack/2016-February/015151.html | 22:14 |
thingee | xyang: I'm confused | 22:14 |
xyang1 | thingee: let me take a look | 22:14 |
xyang1 | thingee: what is your question | 22:14 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: WIP: Add in-tree tempest tests https://review.openstack.org/274471 | 22:15 |
thingee | xyang1: I couldn't really parse the question in this | 22:16 |
thingee | :) | 22:16 |
xyang1 | thingee: the email is not clear | 22:16 |
*** lprice1 has joined #openstack-cinder | 22:16 | |
xyang1 | thingee: let me take a good look and try to respond | 22:16 |
*** julim has quit IRC | 22:16 | |
thingee | xyang1: also saw the thread about xtremeio versus xio.py :P | 22:17 |
*** boris-42_ has joined #openstack-cinder | 22:17 | |
*** alonma has joined #openstack-cinder | 22:17 | |
*** julim has joined #openstack-cinder | 22:17 | |
*** dramakri has joined #openstack-cinder | 22:17 | |
*** julim has quit IRC | 22:17 | |
xyang1 | thingee: :) | 22:17 |
xyang1 | thingee: I think I responded, is there a new one | 22:18 |
xyang1 | thingee: we were calling xio at first but then was told to spell it out | 22:18 |
*** xyang has quit IRC | 22:19 | |
thingee | xyang1: no new message to that one, just found it funny because I've had the same confusion sometimes. | 22:19 |
xyang1 | thingee: I still get them mixed sometimes | 22:20 |
xyang1 | thingee: have to look at it twice | 22:20 |
*** dustins has joined #openstack-cinder | 22:20 | |
xyang1 | thingee: our CI is EMC xio ci. I named it before I learned we cannot use xio any more | 22:21 |
*** yuriy_n17_ has joined #openstack-cinder | 22:21 | |
thingee | ha | 22:21 |
*** alonma has quit IRC | 22:22 | |
*** chlong_ has joined #openstack-cinder | 22:22 | |
*** dramakri1 has quit IRC | 22:22 | |
*** boris-42 has quit IRC | 22:22 | |
*** lprice has quit IRC | 22:22 | |
*** dhellmann has quit IRC | 22:22 | |
*** yuriy_n17 has quit IRC | 22:22 | |
*** ndipanov has quit IRC | 22:22 | |
*** chlong has quit IRC | 22:22 | |
*** haomaiwang has quit IRC | 22:22 | |
*** leeantho has quit IRC | 22:22 | |
*** dhellmann has joined #openstack-cinder | 22:23 | |
*** haomaiwang has joined #openstack-cinder | 22:23 | |
*** leeantho has joined #openstack-cinder | 22:23 | |
*** ndipanov has joined #openstack-cinder | 22:23 | |
*** boris-42_ is now known as boris-42 | 22:24 | |
*** dramakri has quit IRC | 22:28 | |
*** IlyaG has joined #openstack-cinder | 22:28 | |
smcginnis | They used to be Xiotech, but now everyone wants to have a three letter name. :) | 22:28 |
*** cknight1 has joined #openstack-cinder | 22:30 | |
xyang1 | smcginnis: is it? | 22:30 |
smcginnis | EMC, XIO, HPE, IBM... :D | 22:30 |
xyang1 | smcginnis: I tried to ping rhedlind since yesterday but no response | 22:30 |
xyang1 | smcginnis: dell is 4 letter:) | 22:31 |
smcginnis | Hah, that will be the announcemnt after the acquisition, rename to DEL. :) | 22:31 |
xyang1 | smcginnis: we are in trouble:) | 22:32 |
smcginnis | hehe | 22:32 |
*** cknight has quit IRC | 22:32 | |
*** jordanP has quit IRC | 22:33 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Operations and timeout issues https://review.openstack.org/272760 | 22:34 |
*** jwang has quit IRC | 22:34 | |
*** eharney has quit IRC | 22:35 | |
*** jwang has joined #openstack-cinder | 22:35 | |
*** salv-orlando has joined #openstack-cinder | 22:42 | |
*** mylu has joined #openstack-cinder | 22:42 | |
xyang1 | thingee: the last email was not clear at all. I sent a question | 22:43 |
thingee | xyang1: ha ok | 22:43 |
xyang1 | thingee: looks like there are a bunch more emails on this thread, I will go thru those | 22:43 |
*** salv-orl_ has quit IRC | 22:44 | |
*** lprice1 has quit IRC | 22:45 | |
*** jgriffith is now known as jgriffith_away | 22:46 | |
*** mylu has quit IRC | 22:48 | |
*** IlyaG has quit IRC | 22:50 | |
*** mriedem has quit IRC | 22:52 | |
*** crose has quit IRC | 22:53 | |
*** IlyaG has joined #openstack-cinder | 22:54 | |
*** merooney has joined #openstack-cinder | 22:56 | |
*** mylu has joined #openstack-cinder | 22:57 | |
*** xyang has joined #openstack-cinder | 22:57 | |
*** dramakri has joined #openstack-cinder | 22:58 | |
*** JoseMello has quit IRC | 22:58 | |
*** dramakri has quit IRC | 22:59 | |
*** haomaiwang has quit IRC | 23:01 | |
*** IlyaG has quit IRC | 23:01 | |
*** haomaiwa_ has joined #openstack-cinder | 23:01 | |
*** jgriffith_away is now known as jgriffith | 23:02 | |
*** arch-nemesis has quit IRC | 23:05 | |
*** sheel has quit IRC | 23:07 | |
*** rlrossit has quit IRC | 23:07 | |
*** IlyaG has joined #openstack-cinder | 23:08 | |
*** jungleboyj has quit IRC | 23:09 | |
*** sid14726 has joined #openstack-cinder | 23:12 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 23:12 |
*** xyang has quit IRC | 23:13 | |
openstackgerrit | Merged openstack/cinder: Remove old client version checks from 3PAR driver https://review.openstack.org/268346 | 23:16 |
*** mylu has quit IRC | 23:16 | |
*** erlon has quit IRC | 23:16 | |
*** alonma has joined #openstack-cinder | 23:18 | |
*** dustins has quit IRC | 23:18 | |
*** gouthamr has quit IRC | 23:18 | |
*** mylu has joined #openstack-cinder | 23:19 | |
*** mylu has quit IRC | 23:21 | |
smcginnis | angela-s: ping | 23:21 |
*** mylu has joined #openstack-cinder | 23:21 | |
*** alonma has quit IRC | 23:22 | |
*** mylu has quit IRC | 23:23 | |
*** mylu has joined #openstack-cinder | 23:24 | |
*** alonma has joined #openstack-cinder | 23:27 | |
*** hemna has joined #openstack-cinder | 23:29 | |
*** alonma has quit IRC | 23:31 | |
*** cknight1 has quit IRC | 23:32 | |
*** edmondsw has joined #openstack-cinder | 23:34 | |
*** mragupat_ has quit IRC | 23:35 | |
*** andymaier has quit IRC | 23:38 | |
angela-s | smcginnis: pong | 23:39 |
aorourke | jgriffith, do you mind if I make a revision to your patch to include passing in the volumes? That way I can abandon the separate patch I have and not worry about constantly rebasing it? | 23:43 |
*** merooney has quit IRC | 23:44 | |
Swanson | aorourke, which patch is getting volumes sent in? | 23:45 |
hemna | Swanson, https://review.openstack.org/#/c/278687 | 23:46 |
aorourke | Swanson, https://review.openstack.org/#/c/278687 | 23:46 |
*** chlong_ has quit IRC | 23:46 | |
*** mylu has quit IRC | 23:47 | |
Swanson | aorourke, hemna, oh! So they are going to be sent down! | 23:48 |
hemna | Swanson, :) yah looking that way | 23:49 |
Swanson | I'd just come to grips with the fact that they weren't. | 23:49 |
hemna | one of our arrays can't do failover at all w/o it. | 23:49 |
jgriffith | aorourke: sure, or I can do it and add you as co-contributor | 23:49 |
jgriffith | aorourke: I'll be making another pass at it here shortly | 23:49 |
jgriffith | aorourke: but either way is fine by me | 23:50 |
Swanson | hemna, it helps but it isn't required. If I have the info I can break all the replications immediately. Without I have to wait for the next cinder call involving each impacted volume. That would likely be the next attach. | 23:51 |
aorourke | jgriffith, I don't mind making the revision, as long as it doesn't complicate things | 23:53 |
jgriffith | aorourke: nahh... what could go wrong ;) | 23:53 |
Swanson | how many madison and sacagawea $1 coins can I use to pay a pizza delivery guy before they punch me in the face? | 23:57 |
*** martyturner has quit IRC | 23:58 | |
*** cknight has joined #openstack-cinder | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!