*** haomaiwa_ has quit IRC | 00:01 | |
*** lcurtis has quit IRC | 00:01 | |
*** haomaiwang has joined #openstack-cinder | 00:01 | |
hemna | so, to clarify | 00:05 |
---|---|---|
hemna | if we have multi-targets | 00:05 |
hemna | and you failover | 00:05 |
hemna | how do we pick which target ? | 00:05 |
hemna | and know later on that we've picked target A vs. B ? | 00:05 |
patrickeast | hemna: it can be specified by the caller | 00:05 |
patrickeast | but otherwise up to the driver | 00:05 |
patrickeast | then you return the id of the one you did fail over to | 00:06 |
hemna | passed as a param in the cinderclient failover call ? | 00:06 |
*** baojg has joined #openstack-cinder | 00:06 | |
patrickeast | and its stored in that like active_backend_id or whatever column in the service table | 00:06 |
hemna | we currently have list replication targets to get a list of them for a host | 00:06 |
patrickeast | yea i think so | 00:06 |
patrickeast | yea | 00:06 |
patrickeast | there was some discussion about that earlier | 00:06 |
hemna | and the admin uses one of those in the failover command | 00:06 |
patrickeast | yep | 00:06 |
hemna | ok sorry I missed that | 00:06 |
patrickeast | same idea | 00:06 |
hemna | so same thing | 00:06 |
hemna | ok | 00:06 |
hemna | done | 00:06 |
hemna | ship it. | 00:06 |
patrickeast | easy as cheesecake | 00:06 |
jgriffith | patrickeast: hemna exactly... look at the call in cinder.volume.manager in my patch | 00:07 |
jgriffith | patrickeast: hemna there's an optional param to specify "where' you want to failover to | 00:07 |
jgriffith | patrickeast: hemna and even a comment that points out what to do if blank | 00:07 |
hemna | so Host is a target ? | 00:08 |
hemna | or is that a cinder host ? | 00:08 |
*** JoseMello has quit IRC | 00:08 | |
patrickeast | hemna: its the 'secondary' param on the driver method, which should just be a string that matches the id your driver wants to use | 00:09 |
patrickeast | like the id from list_replication_targets | 00:09 |
hemna | https://review.openstack.org/#/c/275797/1/cinder/volume/manager.py | 00:09 |
jgriffith | poor choice of names, updating the comment now | 00:10 |
hemna | in failover_replication ? | 00:10 |
patrickeast | oh | 00:10 |
hemna | I only see context, host=None | 00:10 |
patrickeast | yea its called secondary on the driver method, but yea that one | 00:10 |
patrickeast | as;dlkjads | 00:10 |
patrickeast | stupid fc hbas are going offline again | 00:10 |
hemna | damnit | 00:11 |
jgriffith | hemna: patrickeast see my response to the comment. Host is a *terrible* name | 00:11 |
hemna | ok phew! | 00:11 |
jgriffith | hemna: patrickeast I should change it to "backend_id" or "backend_target_id" or whatever we call it in the cinder.conf file | 00:11 |
hemna | I was confused | 00:11 |
jgriffith | sorry about that | 00:11 |
patrickeast | sounds good | 00:11 |
hemna | very cool | 00:12 |
jgriffith | remember we're skipping the whole manage/unmanaged nonsense | 00:12 |
hemna | jgriffith, thanks so much for work on this. | 00:12 |
jgriffith | even if it is managed it doesn't matter | 00:12 |
jgriffith | hemna: sure... who knows, maybe this time I'll get it right and it'll stick | 00:12 |
jgriffith | I'm not doing it again just so everybody knows :) | 00:12 |
hemna | I am kinda thinking this is more like straight up unmanaged, in the respect that the driver instance has to switch arrays it's talking to depending on failed over or not. | 00:13 |
jgriffith | hemna: right | 00:13 |
jgriffith | hemna: but the other thing is if that array is configured in Cinder as well... then that's fine too | 00:13 |
smcginnis | jgriffith: No repl v3 for you? | 00:13 |
jgriffith | hemna: it flat out doesn't matter | 00:13 |
hemna | yah | 00:13 |
jgriffith | smcginnis: NOOOOOO please | 00:13 |
hemna | :) | 00:13 |
smcginnis | :) | 00:13 |
hemna | hey, I tried multiattach 3 times then gave up. | 00:14 |
jgriffith | smcginnis: Not that I don't like this sort of stuff | 00:14 |
hemna | 3rd time's a charm! | 00:14 |
hemna | eerr quit! | 00:14 |
patrickeast | haha | 00:14 |
jgriffith | hemna: that's not dead yet | 00:14 |
smcginnis | Just a flesh wound. | 00:14 |
hemna | at least with replication, it's all in cinder. | 00:14 |
jgriffith | smcginnis: :) | 00:15 |
jgriffith | hemna: true-dat | 00:15 |
jgriffith | alright, I'm out... see ya'll later | 00:15 |
patrickeast | cya | 00:15 |
smcginnis | See ya | 00:15 |
*** baojg has quit IRC | 00:16 | |
patrickeast | huh... if i unbind/bind a pci device and get a notice that root has new mail its probably a bad sigh eh? | 00:16 |
hemna | patrickeast, smcginnis if you guys are really bored, I posted this up today and would like some feedback....https://review.openstack.org/#/c/275943/ | 00:16 |
hemna | hehe | 00:16 |
smcginnis | hemna: Oooh, interesting. | 00:17 |
smcginnis | The platform filtering is interesting. | 00:18 |
hemna | I don't think it's fully flushed out yet | 00:18 |
hemna | but it's an idea I was toying with | 00:18 |
smcginnis | Though I believe the TC has states Linux is the only officially supported platform. | 00:18 |
*** baojg has joined #openstack-cinder | 00:18 | |
hemna | because there is a WIP Windows connector patch in gerrit. | 00:18 |
hemna | which is needed by the hyperV support in nova | 00:19 |
*** PsionTheory has quit IRC | 00:19 | |
hemna | there are several Nova patches in flight that deal with adding FC support to hyperV support. | 00:19 |
smcginnis | hemna: I'm for that. Would be cool if we can handle different platforms. | 00:19 |
hemna | yah | 00:19 |
hemna | I'm not sure this is the right way to do it, but it's A way | 00:20 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/275729 | 00:20 |
hemna | I was hoping there was a pattern for this type of thing already | 00:20 |
hemna | but I just hacked this out today | 00:20 |
smcginnis | hemna: Just scanning through so far, but looks good to nme. | 00:21 |
hemna | been waiting for jenkins results on it | 00:21 |
hemna | but check has been backed up 300 years now it seems | 00:21 |
smcginnis | Jenkins isn't too happy today. :/ | 00:21 |
*** mtanino has quit IRC | 00:22 | |
smcginnis | But it does look like a patch to fix the issues with getting 502's and 503's and causing restarts is making it through! :) | 00:22 |
hemna | oh good | 00:22 |
hemna | can we just all revert back to the old version :P | 00:23 |
hemna | *sigh* | 00:23 |
smcginnis | hehe | 00:23 |
*** r-daneel has joined #openstack-cinder | 00:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/266163 | 00:25 |
*** bill_az has quit IRC | 00:31 | |
*** dustins has quit IRC | 00:36 | |
*** cburgess_ has quit IRC | 00:39 | |
*** chlong has joined #openstack-cinder | 00:40 | |
*** cburgess has joined #openstack-cinder | 00:42 | |
*** pratap has joined #openstack-cinder | 00:44 | |
*** zhangjn has joined #openstack-cinder | 00:46 | |
*** Fdaisuke has quit IRC | 00:52 | |
*** alonma has joined #openstack-cinder | 00:54 | |
*** cheneydc has joined #openstack-cinder | 00:57 | |
*** alonma has quit IRC | 00:58 | |
*** haomaiwang has quit IRC | 01:01 | |
*** jamielennox is now known as jamielennox|away | 01:01 | |
*** haomaiwang has joined #openstack-cinder | 01:01 | |
*** alonma has joined #openstack-cinder | 01:02 | |
*** smoriya_ has joined #openstack-cinder | 01:04 | |
*** qeelee has joined #openstack-cinder | 01:07 | |
*** alonma has quit IRC | 01:07 | |
*** alonma has joined #openstack-cinder | 01:09 | |
*** laughterwym has joined #openstack-cinder | 01:10 | |
*** laughterwym has quit IRC | 01:10 | |
*** laughterwym has joined #openstack-cinder | 01:10 | |
*** laughterwym has quit IRC | 01:10 | |
*** alonma has quit IRC | 01:14 | |
*** laughterwym has joined #openstack-cinder | 01:14 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add description when rename LUN in Huawei driver https://review.openstack.org/271603 | 01:16 |
*** huanan_L has quit IRC | 01:17 | |
*** cknight has joined #openstack-cinder | 01:17 | |
*** huanan_L has joined #openstack-cinder | 01:17 | |
*** alonma has joined #openstack-cinder | 01:19 | |
*** zhangjn has quit IRC | 01:19 | |
*** mylu has quit IRC | 01:19 | |
*** alonma has quit IRC | 01:23 | |
*** alonma has joined #openstack-cinder | 01:25 | |
*** mylu has joined #openstack-cinder | 01:26 | |
*** garthb has quit IRC | 01:27 | |
*** alonma has quit IRC | 01:29 | |
*** mylu has quit IRC | 01:31 | |
*** alonma has joined #openstack-cinder | 01:31 | |
*** zhangjn has joined #openstack-cinder | 01:33 | |
*** alonma has quit IRC | 01:36 | |
*** Lee1092 has joined #openstack-cinder | 01:38 | |
*** geguileo has quit IRC | 01:38 | |
*** lprice1 has quit IRC | 01:38 | |
*** zhangjn has quit IRC | 01:40 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Don't show chap password in log https://review.openstack.org/271595 | 01:40 |
*** r-daneel has quit IRC | 01:41 | |
*** leeantho has quit IRC | 01:41 | |
*** lixiaoy1 has joined #openstack-cinder | 01:44 | |
*** zhangjn_ has quit IRC | 01:45 | |
*** zhangjn has joined #openstack-cinder | 01:46 | |
*** ericksonsantos has quit IRC | 01:50 | |
*** cknight has quit IRC | 01:52 | |
*** mgarza_ has joined #openstack-cinder | 01:55 | |
*** mylu has joined #openstack-cinder | 01:57 | |
*** ericksonsantos has joined #openstack-cinder | 01:59 | |
*** laughterwym has quit IRC | 02:00 | |
*** qeelee has quit IRC | 02:00 | |
*** laughterwym has joined #openstack-cinder | 02:00 | |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwa_ has joined #openstack-cinder | 02:01 | |
*** laughter_ has joined #openstack-cinder | 02:02 | |
*** laughte__ has joined #openstack-cinder | 02:03 | |
*** bardia has quit IRC | 02:03 | |
*** qeelee has joined #openstack-cinder | 02:04 | |
*** davechen has joined #openstack-cinder | 02:05 | |
*** laughterwym has quit IRC | 02:05 | |
*** laughter_ has quit IRC | 02:06 | |
*** baojg has quit IRC | 02:07 | |
openstackgerrit | LisaLi proposed openstack/cinder: Clean uploading volume when c-vol restarts https://review.openstack.org/275480 | 02:07 |
*** cknight has joined #openstack-cinder | 02:08 | |
*** baojg has joined #openstack-cinder | 02:10 | |
*** lixiaoy1 has quit IRC | 02:11 | |
*** qeelee has quit IRC | 02:11 | |
openstackgerrit | LisaLi proposed openstack/cinder: Clean uploading volume when c-vol restarts https://review.openstack.org/275480 | 02:11 |
*** lixiaoy1 has joined #openstack-cinder | 02:12 | |
*** qeelee has joined #openstack-cinder | 02:18 | |
*** alejandrito has joined #openstack-cinder | 02:20 | |
*** baojg has quit IRC | 02:23 | |
*** baojg has joined #openstack-cinder | 02:23 | |
*** laughte__ has quit IRC | 02:27 | |
*** laughterwym has joined #openstack-cinder | 02:27 | |
*** alonma has joined #openstack-cinder | 02:31 | |
*** jamielennox|away is now known as jamielennox | 02:32 | |
*** mudassirlatif has quit IRC | 02:32 | |
*** qeelee has quit IRC | 02:35 | |
*** alonma has quit IRC | 02:35 | |
*** qeelee has joined #openstack-cinder | 02:36 | |
*** qeelee has quit IRC | 02:45 | |
openstackgerrit | Apoorva Deshpande proposed openstack/cinder: Tintri image direct clone https://review.openstack.org/261126 | 02:46 |
openstackgerrit | LisaLi proposed openstack/cinder: Create volume in same zone when backup restore https://review.openstack.org/272404 | 02:48 |
*** baojg has quit IRC | 02:51 | |
*** sheel has joined #openstack-cinder | 02:52 | |
*** baojg has joined #openstack-cinder | 02:52 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Don't create cgsnapshot if empty consistencygroup https://review.openstack.org/276023 | 02:54 |
*** thurloat_isgone is now known as thurloat | 02:54 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: Use of metadata id instead of metadata name https://review.openstack.org/274589 | 02:56 |
*** apoorvad has quit IRC | 02:59 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** baojg has quit IRC | 03:02 | |
*** baojg_ has joined #openstack-cinder | 03:02 | |
*** qeelee has joined #openstack-cinder | 03:02 | |
openstackgerrit | LisaLi proposed openstack/cinder: Create volume in same zone when backup restore https://review.openstack.org/272404 | 03:04 |
*** baojg_ has quit IRC | 03:05 | |
*** jamielennox is now known as jamielennox|away | 03:06 | |
*** baojg has joined #openstack-cinder | 03:06 | |
*** baojg has quit IRC | 03:09 | |
*** baojg has joined #openstack-cinder | 03:09 | |
*** jamielennox|away is now known as jamielennox | 03:11 | |
*** qeelee has quit IRC | 03:12 | |
*** gouthamr_ has quit IRC | 03:12 | |
*** qeelee has joined #openstack-cinder | 03:13 | |
*** baojg has quit IRC | 03:16 | |
*** baojg has joined #openstack-cinder | 03:16 | |
*** alejandrito has quit IRC | 03:17 | |
*** gouthamr has joined #openstack-cinder | 03:18 | |
*** qeelee has quit IRC | 03:21 | |
*** alonma has joined #openstack-cinder | 03:32 | |
*** gouthamr has quit IRC | 03:34 | |
*** alonma has quit IRC | 03:36 | |
*** dims has joined #openstack-cinder | 03:37 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Don't create cgsnapshot if empty consistencygroup https://review.openstack.org/276023 | 03:39 |
*** cheneydc_ has joined #openstack-cinder | 03:40 | |
*** cheneydc_ has quit IRC | 03:40 | |
*** alonma has joined #openstack-cinder | 03:41 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Don't create cgsnapshot if empty consistencygroup https://review.openstack.org/276023 | 03:41 |
*** alonma has quit IRC | 03:45 | |
*** baojg has quit IRC | 03:46 | |
*** alonma has joined #openstack-cinder | 03:49 | |
*** links has joined #openstack-cinder | 03:49 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI https://review.openstack.org/272229 | 03:50 |
*** vivekd has joined #openstack-cinder | 03:52 | |
*** dims has quit IRC | 03:53 | |
*** alonma has quit IRC | 03:53 | |
*** salv-orl_ has joined #openstack-cinder | 03:54 | |
*** baojg has joined #openstack-cinder | 03:54 | |
*** esker has quit IRC | 03:55 | |
*** alonma has joined #openstack-cinder | 03:56 | |
*** salv-orlando has quit IRC | 03:58 | |
*** vivekd has quit IRC | 03:59 | |
*** alonma has quit IRC | 04:00 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:01 | |
*** alonma has joined #openstack-cinder | 04:04 | |
*** esker has joined #openstack-cinder | 04:05 | |
*** laughter_ has joined #openstack-cinder | 04:06 | |
*** mgarza_ has quit IRC | 04:06 | |
*** laughterwym has quit IRC | 04:08 | |
*** alonma has quit IRC | 04:09 | |
*** daneyon_ has joined #openstack-cinder | 04:11 | |
*** boris-42 has quit IRC | 04:11 | |
*** briancurtin has quit IRC | 04:11 | |
*** fmccrthy has quit IRC | 04:12 | |
*** cppforlife_ has quit IRC | 04:12 | |
*** jamielennox is now known as jamielennox|away | 04:12 | |
*** Lee1092 has quit IRC | 04:12 | |
*** daneyon has quit IRC | 04:12 | |
*** mtanino__ has quit IRC | 04:12 | |
*** tpeoples has quit IRC | 04:12 | |
*** Nakato has quit IRC | 04:12 | |
*** abhi has quit IRC | 04:13 | |
*** tbarron has quit IRC | 04:13 | |
*** jroll has quit IRC | 04:13 | |
*** gus has quit IRC | 04:13 | |
*** chlong has quit IRC | 04:13 | |
*** Nakato has joined #openstack-cinder | 04:13 | |
*** alonma has joined #openstack-cinder | 04:14 | |
*** dosaboy_ has quit IRC | 04:14 | |
*** tbarron has joined #openstack-cinder | 04:14 | |
*** gus has joined #openstack-cinder | 04:14 | |
*** jroll has joined #openstack-cinder | 04:14 | |
*** mylu has quit IRC | 04:14 | |
*** EinstCrazy has joined #openstack-cinder | 04:14 | |
*** dosaboy has joined #openstack-cinder | 04:15 | |
*** abhi has joined #openstack-cinder | 04:15 | |
*** mtanino__ has joined #openstack-cinder | 04:16 | |
*** fmccrthy has joined #openstack-cinder | 04:16 | |
*** mylu has joined #openstack-cinder | 04:17 | |
*** tpeoples has joined #openstack-cinder | 04:17 | |
*** lprice has joined #openstack-cinder | 04:17 | |
*** xek_ has joined #openstack-cinder | 04:18 | |
*** boris-42 has joined #openstack-cinder | 04:18 | |
*** rwall_ has joined #openstack-cinder | 04:19 | |
*** amit213 has quit IRC | 04:19 | |
*** rwall has quit IRC | 04:19 | |
*** SergeyLukjanov has quit IRC | 04:19 | |
*** alonma has quit IRC | 04:19 | |
*** mriedem_afk has quit IRC | 04:19 | |
*** frakt has quit IRC | 04:19 | |
*** smcginnis has quit IRC | 04:19 | |
*** rwall_ is now known as rwall | 04:19 | |
*** frakt has joined #openstack-cinder | 04:19 | |
*** xek has quit IRC | 04:19 | |
*** SergeyLukjanov has joined #openstack-cinder | 04:20 | |
*** briancurtin has joined #openstack-cinder | 04:20 | |
*** smcginnis has joined #openstack-cinder | 04:20 | |
*** cknight has quit IRC | 04:21 | |
*** shyama has joined #openstack-cinder | 04:22 | |
*** Lee1092 has joined #openstack-cinder | 04:23 | |
*** cppforlife_ has joined #openstack-cinder | 04:23 | |
*** alonma has joined #openstack-cinder | 04:26 | |
*** haomaiwang has quit IRC | 04:28 | |
*** chlong has joined #openstack-cinder | 04:28 | |
*** alonma has quit IRC | 04:30 | |
*** chhavi has joined #openstack-cinder | 04:33 | |
*** vivekd has joined #openstack-cinder | 04:35 | |
*** shausy has joined #openstack-cinder | 04:42 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Don't create cgsnapshot if consistencygroup is empty https://review.openstack.org/276023 | 04:47 |
*** vivekd has quit IRC | 04:48 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Don't create cgsnapshot if consistencygroup is empty https://review.openstack.org/276023 | 04:50 |
*** bkumar has joined #openstack-cinder | 04:50 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support https://review.openstack.org/255015 | 04:52 |
openstackgerrit | Sheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI https://review.openstack.org/272229 | 05:04 |
openstackgerrit | Accela Zhao proposed openstack/cinder: Don't create cgsnapshot if cg is empty https://review.openstack.org/276023 | 05:05 |
*** abhishekk has joined #openstack-cinder | 05:08 | |
*** DericHorn-HP has joined #openstack-cinder | 05:10 | |
*** vivekd has joined #openstack-cinder | 05:11 | |
*** boris-42 has quit IRC | 05:13 | |
*** DericHorn-HP has quit IRC | 05:17 | |
*** haomaiwa_ has joined #openstack-cinder | 05:20 | |
*** alonma has joined #openstack-cinder | 05:20 | |
*** angela-s has quit IRC | 05:24 | |
*** alonma has quit IRC | 05:24 | |
*** laughter_ has quit IRC | 05:29 | |
*** laughterwym has joined #openstack-cinder | 05:30 | |
*** jwcroppe has joined #openstack-cinder | 05:33 | |
*** markvoelker_ has quit IRC | 05:34 | |
*** shausy has quit IRC | 05:35 | |
*** leeantho has joined #openstack-cinder | 05:35 | |
*** shausy has joined #openstack-cinder | 05:35 | |
*** abhishekk has quit IRC | 05:38 | |
*** qeelee has joined #openstack-cinder | 05:39 | |
*** krtaylor has quit IRC | 05:44 | |
*** baojg has quit IRC | 05:46 | |
*** vgridnev has joined #openstack-cinder | 05:51 | |
*** krtaylor has joined #openstack-cinder | 05:56 | |
*** itzdilip has joined #openstack-cinder | 06:00 | |
*** haomaiwa_ has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** krtaylor has quit IRC | 06:02 | |
*** baojg has joined #openstack-cinder | 06:02 | |
*** vivekd_ has joined #openstack-cinder | 06:11 | |
*** vivekd has quit IRC | 06:12 | |
*** vivekd_ is now known as vivekd | 06:12 | |
*** krtaylor has joined #openstack-cinder | 06:15 | |
*** haomaiwang has quit IRC | 06:18 | |
*** haomaiwang has joined #openstack-cinder | 06:19 | |
*** lpetrut has joined #openstack-cinder | 06:20 | |
*** alonma has joined #openstack-cinder | 06:20 | |
*** alonma has quit IRC | 06:24 | |
*** mylu has quit IRC | 06:26 | |
*** alonma has joined #openstack-cinder | 06:27 | |
*** tongli has quit IRC | 06:30 | |
*** alonma has quit IRC | 06:32 | |
*** alonma has joined #openstack-cinder | 06:34 | |
*** markvoelker has joined #openstack-cinder | 06:35 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/276060 | 06:38 |
*** alonma has quit IRC | 06:38 | |
*** markvoelker has quit IRC | 06:39 | |
*** vivekd has quit IRC | 06:40 | |
*** cfriesen has quit IRC | 06:40 | |
*** alonma has joined #openstack-cinder | 06:40 | |
*** mylu has joined #openstack-cinder | 06:41 | |
*** alonma has quit IRC | 06:44 | |
*** alonma has joined #openstack-cinder | 06:46 | |
*** alonma has quit IRC | 06:50 | |
*** jwcroppe has quit IRC | 06:51 | |
*** jwcroppe has joined #openstack-cinder | 06:52 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: manage_existing_get_size for VMDK driver https://review.openstack.org/276063 | 06:52 |
*** alonma has joined #openstack-cinder | 06:53 | |
*** lpetrut has quit IRC | 06:54 | |
*** krtaylor has quit IRC | 06:54 | |
*** jwcroppe has quit IRC | 06:56 | |
*** alonma has quit IRC | 06:57 | |
*** alonma has joined #openstack-cinder | 06:59 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: manage_existing_get_size for VMDK driver https://review.openstack.org/276063 | 07:00 |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwa_ has joined #openstack-cinder | 07:01 | |
*** belmoreira has joined #openstack-cinder | 07:02 | |
*** alonma has quit IRC | 07:03 | |
*** alonma has joined #openstack-cinder | 07:05 | |
*** krtaylor has joined #openstack-cinder | 07:06 | |
*** markus_zz has joined #openstack-cinder | 07:06 | |
*** markus_zz has quit IRC | 07:07 | |
*** alonma has quit IRC | 07:09 | |
*** salv-orl_ has quit IRC | 07:13 | |
*** alonma has joined #openstack-cinder | 07:14 | |
*** avishay has joined #openstack-cinder | 07:15 | |
openstackgerrit | LisaLi proposed openstack/cinder: Update quota when volume type renames. https://review.openstack.org/267999 | 07:16 |
*** alonma has quit IRC | 07:19 | |
*** lixiaoy1 has quit IRC | 07:19 | |
*** alonma has joined #openstack-cinder | 07:21 | |
*** lixiaoy1 has joined #openstack-cinder | 07:21 | |
openstackgerrit | Sheel Rana proposed openstack/cinder: Use of metadata id instead of metadata name https://review.openstack.org/274589 | 07:22 |
*** alonma has quit IRC | 07:25 | |
*** lpetrut has joined #openstack-cinder | 07:26 | |
*** alonma has joined #openstack-cinder | 07:27 | |
*** leeantho has quit IRC | 07:29 | |
*** lpetrut has quit IRC | 07:30 | |
*** alonma has quit IRC | 07:31 | |
*** mylu has quit IRC | 07:32 | |
*** alonma has joined #openstack-cinder | 07:33 | |
sheel | DuncanT: Hi | 07:35 |
sheel | DuncanT: Please review https://review.openstack.org/#/c/272229/ as per time availibility on your end.. | 07:36 |
*** markvoelker has joined #openstack-cinder | 07:36 | |
sheel | DuncanT: thank you... | 07:36 |
*** alonma has quit IRC | 07:37 | |
*** alonma has joined #openstack-cinder | 07:39 | |
*** markvoelker has quit IRC | 07:41 | |
*** lixiaoy11 has joined #openstack-cinder | 07:41 | |
*** lixiaoy1 has quit IRC | 07:41 | |
*** lixiaoy1 has joined #openstack-cinder | 07:42 | |
*** lixiaoy11 has quit IRC | 07:42 | |
*** lixiaoy11 has joined #openstack-cinder | 07:43 | |
*** lixiaoy1 has quit IRC | 07:43 | |
*** alonma has quit IRC | 07:43 | |
*** alonma has joined #openstack-cinder | 07:45 | |
*** arecknag has joined #openstack-cinder | 07:47 | |
*** anshul has joined #openstack-cinder | 07:48 | |
*** nkrinner has joined #openstack-cinder | 07:49 | |
*** alonma has quit IRC | 07:50 | |
*** alonma has joined #openstack-cinder | 07:52 | |
*** alonma has quit IRC | 07:56 | |
*** mvk_ has quit IRC | 07:57 | |
*** alonma has joined #openstack-cinder | 07:58 | |
*** haomaiwa_ has quit IRC | 08:01 | |
*** haomaiwa_ has joined #openstack-cinder | 08:01 | |
*** alonma has quit IRC | 08:02 | |
*** lixiaoy11 has left #openstack-cinder | 08:03 | |
*** lixiaoy1 has joined #openstack-cinder | 08:03 | |
*** alonma has joined #openstack-cinder | 08:04 | |
*** sinese has joined #openstack-cinder | 08:05 | |
*** ChubYann has quit IRC | 08:07 | |
*** vgridnev has quit IRC | 08:08 | |
*** alonma has quit IRC | 08:08 | |
*** alonma has joined #openstack-cinder | 08:10 | |
*** alonma has quit IRC | 08:14 | |
*** laughterwym has quit IRC | 08:14 | |
*** laughterwym has joined #openstack-cinder | 08:15 | |
*** laughterwym has quit IRC | 08:15 | |
*** laughterwym has joined #openstack-cinder | 08:15 | |
*** laughterwym has quit IRC | 08:15 | |
*** laughterwym has joined #openstack-cinder | 08:16 | |
*** alonma has joined #openstack-cinder | 08:16 | |
openstackgerrit | yangweiwei proposed openstack/cinder: Update the method of backup-restore https://review.openstack.org/275542 | 08:18 |
*** alonma has quit IRC | 08:20 | |
*** nkrinner has quit IRC | 08:20 | |
*** laughterwym has quit IRC | 08:21 | |
*** nkrinner has joined #openstack-cinder | 08:23 | |
*** vgridnev has joined #openstack-cinder | 08:25 | |
*** mvk_ has joined #openstack-cinder | 08:26 | |
*** alonma has joined #openstack-cinder | 08:27 | |
*** jistr has joined #openstack-cinder | 08:29 | |
*** alonma has quit IRC | 08:31 | |
*** xinyong is now known as xiangxinyong | 08:34 | |
*** andymaier has joined #openstack-cinder | 08:34 | |
*** vgridnev has quit IRC | 08:46 | |
*** slunkad has quit IRC | 08:50 | |
*** jordanP has joined #openstack-cinder | 08:52 | |
*** slunkad has joined #openstack-cinder | 08:52 | |
*** liverpooler has joined #openstack-cinder | 08:53 | |
*** haomaiwa_ has quit IRC | 09:01 | |
*** haomaiwa_ has joined #openstack-cinder | 09:01 | |
*** avishay has quit IRC | 09:01 | |
*** baojg has quit IRC | 09:04 | |
*** baojg has joined #openstack-cinder | 09:07 | |
swamireddy | is Cinder support the volume encryption? | 09:11 |
openstackgerrit | yangweiwei proposed openstack/cinder: Update the status of backup-restore https://review.openstack.org/275542 | 09:14 |
*** jistr has quit IRC | 09:16 | |
*** avishay has joined #openstack-cinder | 09:17 | |
*** openstackgerrit has quit IRC | 09:17 | |
*** openstackgerrit has joined #openstack-cinder | 09:18 | |
*** alonma has joined #openstack-cinder | 09:19 | |
sheel | swamireddy: yes | 09:21 |
sheel | swamireddy: but as far as i know this support is only provided by IBM driver | 09:21 |
swamireddy | sheel: Thanks...Ok...I see that - encrytion supports in K release onwards only.. | 09:25 |
sheel | swamireddy:yup | 09:25 |
swamireddy | sheel: ceph backend might not be support ATM... | 09:26 |
*** lpetrut has joined #openstack-cinder | 09:30 | |
*** qeelee has quit IRC | 09:33 | |
*** tpsilva has joined #openstack-cinder | 09:34 | |
*** markvoelker has joined #openstack-cinder | 09:37 | |
*** geguileo has joined #openstack-cinder | 09:37 | |
*** abhishekk has joined #openstack-cinder | 09:40 | |
*** markvoelker has quit IRC | 09:41 | |
openstackgerrit | wanghao proposed openstack/cinder: Uploading volume to specific image https://review.openstack.org/270073 | 09:42 |
*** isaacb has joined #openstack-cinder | 09:42 | |
*** qeelee has joined #openstack-cinder | 09:43 | |
*** ociuhandu has joined #openstack-cinder | 09:45 | |
*** ociuhandu has quit IRC | 09:49 | |
*** ociuhandu has joined #openstack-cinder | 09:49 | |
*** sgotliv has joined #openstack-cinder | 09:50 | |
*** dalgaaf has joined #openstack-cinder | 09:53 | |
*** davechen has left #openstack-cinder | 09:57 | |
*** haomaiwa_ has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
*** cheneydc has quit IRC | 10:02 | |
*** ildikov has quit IRC | 10:07 | |
DuncanT | Encryption should be supported at the hypervisor by any iSCSI driver, and at the backend by a bunch of drivers | 10:11 |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder: Support https keystone CA checking in volume quotas https://review.openstack.org/272437 | 10:11 |
*** qeelee has quit IRC | 10:12 | |
*** e0ne has joined #openstack-cinder | 10:14 | |
*** esp has joined #openstack-cinder | 10:20 | |
*** baojg_ has joined #openstack-cinder | 10:24 | |
*** esp has quit IRC | 10:24 | |
*** baojg has quit IRC | 10:25 | |
sheel | swamireddy: hmm, i think so | 10:27 |
*** dguryanov has joined #openstack-cinder | 10:30 | |
*** avishay has quit IRC | 10:36 | |
*** jwang__ has joined #openstack-cinder | 10:37 | |
*** baojg_ has quit IRC | 10:38 | |
*** smoriya has quit IRC | 10:38 | |
*** smoriya_afk has joined #openstack-cinder | 10:39 | |
*** smoriya_afk is now known as smoriya | 10:39 | |
*** kmartin__ has joined #openstack-cinder | 10:39 | |
*** fernnest__ has joined #openstack-cinder | 10:39 | |
*** pots2 has joined #openstack-cinder | 10:39 | |
*** jwang_ has quit IRC | 10:40 | |
*** pots has quit IRC | 10:40 | |
*** kmartin_ has quit IRC | 10:40 | |
*** fernnest_ has quit IRC | 10:40 | |
*** ndipanov has joined #openstack-cinder | 10:44 | |
*** alonma has quit IRC | 10:45 | |
*** alonma has joined #openstack-cinder | 10:45 | |
*** baojg has joined #openstack-cinder | 10:45 | |
*** alonma has quit IRC | 10:49 | |
*** ociuhandu has quit IRC | 10:50 | |
*** alonma has joined #openstack-cinder | 10:51 | |
openstackgerrit | Matan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver https://review.openstack.org/269624 | 10:52 |
*** baojg has quit IRC | 10:53 | |
*** salv-orlando has joined #openstack-cinder | 10:54 | |
*** haomaiwang has quit IRC | 10:54 | |
*** alonma has quit IRC | 10:55 | |
*** alonma has joined #openstack-cinder | 10:57 | |
*** qeelee has joined #openstack-cinder | 10:59 | |
*** alonma has quit IRC | 11:02 | |
*** alonma has joined #openstack-cinder | 11:04 | |
yuriy_n17 | geguileo: Hi! Sorry for disturbing. Please review https://review.openstack.org/#/c/263257/ whenever you have free time. Thanks a lot for advance. | 11:05 |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Method not being called for V3 https://review.openstack.org/252066 | 11:08 |
*** alonma has quit IRC | 11:08 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI https://review.openstack.org/272232 | 11:09 |
*** qeelee has quit IRC | 11:10 | |
*** alonma has joined #openstack-cinder | 11:11 | |
*** dims has joined #openstack-cinder | 11:12 | |
*** baojg has joined #openstack-cinder | 11:13 | |
*** qeelee has joined #openstack-cinder | 11:15 | |
*** alonma has quit IRC | 11:15 | |
*** alonma has joined #openstack-cinder | 11:17 | |
*** alonma has joined #openstack-cinder | 11:18 | |
*** ankit_ag has joined #openstack-cinder | 11:21 | |
*** yhayashi has quit IRC | 11:22 | |
swamireddy | DuncanT: yes....so it iSCSI driver should have the encryption functionality and its respective backend also suppor the encryption....is this correct? | 11:24 |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched https://review.openstack.org/246992 | 11:24 |
DuncanT | swamireddy: for iSCSI, cinder can do the encryption at the hypervisor, so it doesn't matter what the backend does | 11:25 |
*** sheel has quit IRC | 11:32 | |
*** erlon has joined #openstack-cinder | 11:33 | |
*** haomaiwa_ has joined #openstack-cinder | 11:37 | |
*** markvoelker has joined #openstack-cinder | 11:38 | |
*** rockeychen has joined #openstack-cinder | 11:41 | |
*** markvoelker has quit IRC | 11:43 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Fix variable scope issue in try-except https://review.openstack.org/276179 | 11:43 |
*** qeelee has quit IRC | 11:47 | |
*** qeelee has joined #openstack-cinder | 11:48 | |
*** bkumar has quit IRC | 11:50 | |
*** merooney has joined #openstack-cinder | 11:51 | |
*** jistr has joined #openstack-cinder | 11:56 | |
*** resker has joined #openstack-cinder | 11:58 | |
*** chhavi has quit IRC | 11:58 | |
dguryanov | Hello! | 12:00 |
*** haomaiwa_ has quit IRC | 12:01 | |
*** esker has quit IRC | 12:01 | |
dguryanov | Is it possible to upload patch to an abandoned review? | 12:01 |
*** haomaiwang has joined #openstack-cinder | 12:01 | |
dguryanov | There is a review, which abandoned until newton https://review.openstack.org/#/c/238894/ | 12:01 |
dguryanov | But a set of my patches depends on it | 12:01 |
dguryanov | I'd like to just upload rebased version | 12:02 |
*** xiangxinyong has quit IRC | 12:02 | |
*** xek_ is now known as xek | 12:08 | |
*** ildikov has joined #openstack-cinder | 12:09 | |
*** chhavi has joined #openstack-cinder | 12:10 | |
*** merooney has quit IRC | 12:10 | |
*** merooney has joined #openstack-cinder | 12:12 | |
*** vgridnev has joined #openstack-cinder | 12:13 | |
*** merooney has quit IRC | 12:13 | |
*** qeelee has quit IRC | 12:13 | |
*** qeelee has joined #openstack-cinder | 12:14 | |
e0ne | dguryanov: hi, no, you have to restrore patch | 12:17 |
dguryanov | It seems I don't have permissions | 12:17 |
*** merooney has joined #openstack-cinder | 12:18 | |
*** EinstCrazy has quit IRC | 12:19 | |
*** qeelee has quit IRC | 12:19 | |
*** EinstCrazy has joined #openstack-cinder | 12:20 | |
e0ne | dguryanov: please, ask DuncanT | 12:21 |
flip214 | can somebody help me with a devstack failure? http://logs.openstack.org/38/276138/1/check/gate-drbd-devstack-bashate/b076b2f/console.html#_2016-02-04_10_56_15_178 | 12:21 |
e0ne | dguryanov: from my PoV it's enough to get -2 | 12:21 |
dguryanov | e0ne: OK, thanks! | 12:21 |
flip214 | or, more correct, bashate. | 12:21 |
dguryanov | e0ne: Do you mean set -2 to review instead of abandoning? | 12:22 |
*** raildo-afk is now known as raildo | 12:23 | |
e0ne | dguryanov: yes. but maybe DuncanT has some reasons to abandone it, so I would like to ask him | 12:23 |
dguryanov | e0ne: OK | 12:23 |
dguryanov | DuncanT: Hello! Could you, please, temporary reopen review https://review.openstack.org/#/c/238894/? I'd like to upload rebased version of this patch, because my patches depend on it :( | 12:26 |
*** baojg has quit IRC | 12:28 | |
*** baojg has joined #openstack-cinder | 12:31 | |
*** salv-orlando has quit IRC | 12:32 | |
DuncanT | Done | 12:37 |
DuncanT | I was told that patch series is entirely optional, so I abandoned the series since we already have a whole bunch of coded changing for rolling upgrades, and I was reluctant to see more added | 12:37 |
DuncanT | We need to keep the amount of the change sane for the inevitable debug work when something goes wrong | 12:38 |
openstackgerrit | Marc Koderer proposed openstack/cinder: Fix issue in hacking with underscore imports https://review.openstack.org/276198 | 12:44 |
*** salv-orlando has joined #openstack-cinder | 12:46 | |
*** tpatzig has quit IRC | 12:47 | |
*** mkoderer has quit IRC | 12:49 | |
* DuncanT appologies to mkoderer for that review, but it needs it | 12:50 | |
*** erlon_ has joined #openstack-cinder | 12:50 | |
*** fernnest_ has joined #openstack-cinder | 12:52 | |
*** erlon has quit IRC | 12:52 | |
*** erlon_ is now known as erlon | 12:52 | |
*** timcl has joined #openstack-cinder | 12:52 | |
*** fernnest__ has quit IRC | 12:53 | |
*** knikolla has quit IRC | 12:53 | |
*** markvoelker has joined #openstack-cinder | 12:53 | |
*** mc_nair has quit IRC | 12:54 | |
*** laughterwym has joined #openstack-cinder | 12:55 | |
*** takedakn has joined #openstack-cinder | 12:55 | |
*** knikolla has joined #openstack-cinder | 12:55 | |
*** mc_nair has joined #openstack-cinder | 12:57 | |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder: Support https keystone CA checking in volume quotas https://review.openstack.org/272437 | 12:58 |
*** markvoelker has quit IRC | 12:58 | |
*** merooney has quit IRC | 12:58 | |
*** haomaiwang has quit IRC | 13:01 | |
*** haomaiwa_ has joined #openstack-cinder | 13:01 | |
*** tpatzig has joined #openstack-cinder | 13:03 | |
*** mkoderer has joined #openstack-cinder | 13:03 | |
*** dguryanov has quit IRC | 13:04 | |
*** merooney has joined #openstack-cinder | 13:07 | |
*** laughterwym has quit IRC | 13:09 | |
*** markvoelker has joined #openstack-cinder | 13:11 | |
*** alonma has quit IRC | 13:14 | |
*** ociuhandu has joined #openstack-cinder | 13:15 | |
*** smoriya_ has quit IRC | 13:16 | |
*** takedakn has quit IRC | 13:17 | |
*** alonma has joined #openstack-cinder | 13:21 | |
*** abhishekk has quit IRC | 13:22 | |
*** akerr has joined #openstack-cinder | 13:25 | |
*** baojg has quit IRC | 13:27 | |
*** sinese has quit IRC | 13:28 | |
*** baojg has joined #openstack-cinder | 13:28 | |
*** sinese has joined #openstack-cinder | 13:29 | |
*** sinese has quit IRC | 13:29 | |
*** edmondsw has joined #openstack-cinder | 13:31 | |
*** porrua has joined #openstack-cinder | 13:32 | |
*** ankit_ag has quit IRC | 13:33 | |
*** takedakn has joined #openstack-cinder | 13:33 | |
*** arthurrdg has joined #openstack-cinder | 13:34 | |
*** rockeychen has quit IRC | 13:34 | |
*** rockeychen has joined #openstack-cinder | 13:34 | |
*** ildikov has quit IRC | 13:34 | |
*** david-lyle has quit IRC | 13:38 | |
*** diablo_rojo has joined #openstack-cinder | 13:39 | |
*** mylu has joined #openstack-cinder | 13:39 | |
*** gouthamr has joined #openstack-cinder | 13:39 | |
*** merooney has quit IRC | 13:41 | |
*** arthurrdg_ has joined #openstack-cinder | 13:41 | |
*** shausy has quit IRC | 13:42 | |
*** leseb_ has quit IRC | 13:42 | |
*** gouthamr_ has joined #openstack-cinder | 13:43 | |
*** mylu has quit IRC | 13:43 | |
*** arthurrdg has quit IRC | 13:43 | |
*** gouthamr has quit IRC | 13:46 | |
*** merooney has joined #openstack-cinder | 13:50 | |
openstackgerrit | Philipp Marek proposed openstack/cinder: DRBD: Compat for current DRBDmanage versions. https://review.openstack.org/276242 | 13:52 |
*** links has quit IRC | 13:56 | |
akerr | Hiya cores, wondering if I could get a +Workflow on my backport? https://review.openstack.org/#/c/273732/ | 13:57 |
mkoderer | DuncanT: do we have unit tests for hacking rules? ( https://review.openstack.org/#/c/276198/ ) | 13:59 |
smcginnis | mkoderer: Yep, let me get a link for you. | 14:00 |
*** haomaiwa_ has quit IRC | 14:01 | |
smcginnis | mkoderer: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_hacking.py | 14:01 |
*** ildikov has joined #openstack-cinder | 14:01 | |
*** haomaiwa_ has joined #openstack-cinder | 14:01 | |
*** rlrossit has joined #openstack-cinder | 14:02 | |
smcginnis | akerr: Done | 14:02 |
akerr | smcginnis: thanks! | 14:03 |
*** sheel has joined #openstack-cinder | 14:03 | |
mkoderer | smcginnis: ok thx! need to check if Manila has them too .. | 14:03 |
smcginnis | mkoderer: They are useful. The few hacking checks I've done - I caught issues writing the tests. :) | 14:04 |
*** barra204 has quit IRC | 14:04 | |
DuncanT | mkoderer: What he said ;-) | 14:06 |
*** shakamunyi has joined #openstack-cinder | 14:06 | |
smcginnis | :) | 14:06 |
*** julim has joined #openstack-cinder | 14:07 | |
*** merooney has quit IRC | 14:07 | |
*** dims_ has joined #openstack-cinder | 14:10 | |
*** merooney has joined #openstack-cinder | 14:11 | |
*** dims has quit IRC | 14:12 | |
*** gouthamr_ has quit IRC | 14:15 | |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in smbfs volume driver https://review.openstack.org/261551 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in vzstorage volume driver https://review.openstack.org/261550 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in netapp_nfs unit test https://review.openstack.org/261557 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in glusterfs volume driver https://review.openstack.org/261552 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in scality volume driver https://review.openstack.org/261553 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/270908 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in quobyte volume driver https://review.openstack.org/261554 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in nfs volume driver https://review.openstack.org/261555 | 14:17 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Update initialize_connection to use versionedobjects https://review.openstack.org/238894 | 14:17 |
openstackgerrit | Liucheng Jiang proposed openstack/cinder: Huawei: Implement v2 replication (managed) https://review.openstack.org/261246 | 14:17 |
*** takedakn has quit IRC | 14:19 | |
*** salv-orlando has quit IRC | 14:20 | |
*** mylu has joined #openstack-cinder | 14:23 | |
*** salv-orlando has joined #openstack-cinder | 14:24 | |
*** gouthamr has joined #openstack-cinder | 14:26 | |
*** sgotliv has quit IRC | 14:28 | |
*** timcl has quit IRC | 14:28 | |
*** arthurrdg_ has quit IRC | 14:35 | |
*** merooney has quit IRC | 14:36 | |
*** resker has quit IRC | 14:36 | |
*** timcl has joined #openstack-cinder | 14:37 | |
*** jgregor has joined #openstack-cinder | 14:37 | |
*** takedakn has joined #openstack-cinder | 14:38 | |
*** mylu has quit IRC | 14:40 | |
*** takedakn has quit IRC | 14:41 | |
*** mylu has joined #openstack-cinder | 14:43 | |
openstackgerrit | Marc Koderer proposed openstack/cinder: Fix issue in hacking with underscore imports https://review.openstack.org/276198 | 14:44 |
*** takedakn has joined #openstack-cinder | 14:45 | |
*** xyang1 has joined #openstack-cinder | 14:47 | |
*** merooney has joined #openstack-cinder | 14:48 | |
*** takedakn has quit IRC | 14:49 | |
*** cfriesen has joined #openstack-cinder | 14:52 | |
*** markstur_ has joined #openstack-cinder | 14:53 | |
*** markstur_ has left #openstack-cinder | 14:55 | |
*** dustins has joined #openstack-cinder | 14:56 | |
*** sgotliv has joined #openstack-cinder | 14:57 | |
*** eharney has joined #openstack-cinder | 15:00 | |
*** cknight has joined #openstack-cinder | 15:00 | |
*** haomaiwa_ has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-cinder | 15:01 | |
*** esp has joined #openstack-cinder | 15:01 | |
*** daneyon_ has quit IRC | 15:01 | |
*** edtubill has joined #openstack-cinder | 15:08 | |
*** esp has quit IRC | 15:10 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI https://review.openstack.org/272232 | 15:12 |
*** jwcroppe has joined #openstack-cinder | 15:14 | |
*** jungleboyj has quit IRC | 15:15 | |
sheel | Dear All, | 15:17 |
sheel | need some eyes for https://review.openstack.org/#/c/274589/6 | 15:17 |
sheel | scottda: hi there | 15:19 |
sheel | scottda: could we discuss today for micro version for https://review.openstack.org/#/c/272229 ? | 15:20 |
*** chhavi has quit IRC | 15:21 | |
aorourke | patrickeast, congrats on the core position :) | 15:22 |
sheel | 11->12 patrickeast:All the Best for your new Role.. | 15:25 |
*** baumann has joined #openstack-cinder | 15:27 | |
diablo_rojo | patrickeast: Congrats Patrick :) | 15:28 |
*** mgarza has joined #openstack-cinder | 15:29 | |
*** e0ne has quit IRC | 15:29 | |
*** e0ne has joined #openstack-cinder | 15:30 | |
*** esker has joined #openstack-cinder | 15:32 | |
*** vgridnev has quit IRC | 15:32 | |
*** baojg has quit IRC | 15:33 | |
*** julim has quit IRC | 15:33 | |
*** arch-nemesis has joined #openstack-cinder | 15:35 | |
*** julim has joined #openstack-cinder | 15:35 | |
*** mylu has quit IRC | 15:36 | |
*** baojg has joined #openstack-cinder | 15:37 | |
*** baojg has quit IRC | 15:37 | |
*** mylu has joined #openstack-cinder | 15:39 | |
*** gouthamr_ has joined #openstack-cinder | 15:40 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Fix variable scope issue in try-except https://review.openstack.org/276179 | 15:43 |
*** gouthamr has quit IRC | 15:43 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Inherit multiattach from volume source https://review.openstack.org/276312 | 15:43 |
*** crose has joined #openstack-cinder | 15:46 | |
scottda | sheel: Hi. I looked at your patch. Yes, when microversions has landed this could microversioned.. | 15:47 |
openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 15:47 |
*** alyson_ has joined #openstack-cinder | 15:48 | |
*** alyson_ has quit IRC | 15:48 | |
*** JoseMello has joined #openstack-cinder | 15:48 | |
*** alyson_ has joined #openstack-cinder | 15:49 | |
*** rockeychen has quit IRC | 15:51 | |
*** zhipeng has joined #openstack-cinder | 15:51 | |
*** rockeychen has joined #openstack-cinder | 15:52 | |
sheel | scottda: ok, thanks for confirmation | 15:53 |
sheel | scottda | 15:53 |
sheel | scottda: could you give some link to have some idea about micro version | 15:53 |
openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 15:54 |
sheel | scottda: it wil help me to write code if required to release this patch for micro versioning | 15:55 |
*** garthb has joined #openstack-cinder | 15:55 | |
sheel | scottda: may be some code, patch, anything.. | 15:55 |
scottda | sheel: https://review.openstack.org/#/c/224910/17 | 15:59 |
scottda | sheel: There is a devref doc there as well | 15:59 |
patrickeast | aorourke: sheel: diablo_rojo : thanks! | 16:00 |
sheel | scottda:hmm, quite large patch :) | 16:01 |
*** haomaiwang has quit IRC | 16:01 | |
scottda | To use microversions you need the code in a class that inherits from wsgi.Controller | 16:01 |
scottda | i.e. cinder/api/v2/volumes.py | 16:01 |
*** haomaiwa_ has joined #openstack-cinder | 16:01 | |
scottda | But I need to add a v3 endpoint first.... | 16:01 |
smcginnis | patrickeast: You see the shiny new -2/+2 options on the review page yet? :) | 16:01 |
patrickeast | smcginnis: yep :D | 16:02 |
sheel | scottda: great | 16:02 |
*** dustins_ has joined #openstack-cinder | 16:02 | |
*** mylu has quit IRC | 16:02 | |
*** mtanino has joined #openstack-cinder | 16:02 | |
sheel | scottda: thnx for explanation | 16:02 |
*** jungleboyj has joined #openstack-cinder | 16:02 | |
*** dustins has quit IRC | 16:03 | |
sheel | patrickeast: May be you can have feeling of pressing +2 button on https://review.openstack.org/#/c/272229/ | 16:03 |
sheel | patrickeast: ;) | 16:04 |
smcginnis | Hah, the review requests are already coming in. :) | 16:04 |
patrickeast | haha | 16:04 |
*** r-daneel has joined #openstack-cinder | 16:04 | |
patrickeast | i'll check it out in a little bit, on mobile right now | 16:05 |
scottda | sheel: I could walk you through the changes you will need, but it is getting late in the cycle and I am not sure microversions will get in before your patch. I think reviewers should consider this for your patch, and whether your change will break anyone. | 16:05 |
sheel | patrickeast : thanks | 16:05 |
sheel | scottda: patch is about to release.. | 16:06 |
*** alkhodos has joined #openstack-cinder | 16:06 | |
sheel | scottda: actually there were 2 versions of patch | 16:06 |
sheel | for one we need micro version support so that it does not break existing behavior | 16:06 |
scottda | ok, well micorversion patch is NOT about to release. | 16:06 |
sheel | scottda: hehe | 16:07 |
sheel | scottda: will it in mitaka or Newton? | 16:07 |
sheel | scottda: my patch can wait...there is no urgency as such | 16:08 |
scottda | sheel: Unsure about mitaka. I don't think it could/should merge at the last minute, which is coming up soon. | 16:08 |
*** merooney has quit IRC | 16:08 | |
sheel | scottda: yup, no issues... | 16:09 |
*** merooney has joined #openstack-cinder | 16:09 | |
sheel | scottda: I will follow up with your patch(you shared)..may be will ping you once your patch is released.. | 16:09 |
Swanson | Can't help but notice something murdered my (Sean's. I murdered my) CI overnight. | 16:10 |
*** arecknag has quit IRC | 16:10 | |
*** lprice1 has joined #openstack-cinder | 16:10 | |
smcginnis | Swanson: puppet apply something bad? | 16:12 |
Swanson | smcginnis, you use puppet with your CI? | 16:13 |
mtanino | smcginnis: Hi, Could you revisit to this image metadata support? Duncan-T was agreed the approach. I suppose just need few reviews. https://review.openstack.org/#/c/258649/ | 16:14 |
*** lprice has quit IRC | 16:14 | |
*** alkhodos has quit IRC | 16:14 | |
*** nkrinner has quit IRC | 16:15 | |
*** xyang1 has quit IRC | 16:15 | |
smcginnis | mtanino: Sure, I'll take a look. | 16:16 |
mtanino | smcginnis: Thank you. | 16:16 |
sheel | smcginnis: Hi | 16:17 |
sheel | smcginnis: please this one as well : https://review.openstack.org/#/c/274589/ | 16:17 |
smcginnis | Added to my open tabs... | 16:18 |
*** liverpooler has quit IRC | 16:18 | |
sheel | smcginnis: thank you | 16:19 |
mtanino | then firefox will be crashed... | 16:20 |
smcginnis | Usually other apps start acting weird from lack of memory available I think. | 16:21 |
smcginnis | Then I know it's time to clean up some tabs. :) | 16:21 |
*** mylu has joined #openstack-cinder | 16:21 | |
*** esker has quit IRC | 16:23 | |
sheel | hehe.. | 16:24 |
*** david-lyle has joined #openstack-cinder | 16:24 | |
*** esker has joined #openstack-cinder | 16:24 | |
*** alonma has quit IRC | 16:26 | |
*** alonma has joined #openstack-cinder | 16:27 | |
sheel | mtanino: hi | 16:29 |
mtanino | sheel: hi, | 16:29 |
sheel | mtanino: sorry, but i have conflicts on comment on https://review.openstack.org/#/c/274589/ | 16:29 |
sheel | may be need to understand your view on this | 16:30 |
*** anshul has quit IRC | 16:30 | |
*** merooney has quit IRC | 16:30 | |
sheel | mtanino: actually, this functionality will not change existing behavior | 16:30 |
sheel | mtanino: rather will only eliminate one corner case | 16:30 |
sheel | mtanino: so, there should not be any APIImpact | 16:30 |
sheel | mtanino: pleaes let me know your views in case i am missing anything | 16:31 |
*** alonma has quit IRC | 16:31 | |
mtanino | sheel: I thought this changes existing behaviour(seems it's a bug to delete all same name metadata). | 16:32 |
sheel | mtanino: hmm, may be I could have given some more details in commit message.. | 16:32 |
*** geguileo has quit IRC | 16:32 | |
mtanino | sheel: let me check the definition of APIImpact again. | 16:33 |
*** alonma has joined #openstack-cinder | 16:33 | |
sheel | mtanino: Ok sure.. | 16:33 |
mtanino | sheel: but I recommend to add release note too. | 16:33 |
*** dguryanov has joined #openstack-cinder | 16:33 | |
dguryanov | DuncanT: Thanks! | 16:33 |
sheel | mtanino: ah, but this is not changing any expected behavior... | 16:34 |
sheel | mtanino: I dont think even some one create image metadata with same name | 16:34 |
sheel | mtanino: this is just to cater to one corner case which was may be left in past | 16:34 |
*** belmoreira has quit IRC | 16:34 | |
*** zhipeng has quit IRC | 16:35 | |
openstackgerrit | Accela Zhao proposed openstack/cinder: Dedup code for test_cgsnapshots.py https://review.openstack.org/276338 | 16:35 |
mtanino | sheel: In the past, we could delete metadatas which has same name abc, ABC, Abc etc unexpectedly, and now the bug was fixed in your patch, right? | 16:36 |
sheel | mtanino: yes, exactly | 16:36 |
hemna | patrickeast, congrats! much deserved :) | 16:37 |
*** salv-orlando has quit IRC | 16:37 | |
*** alonma has quit IRC | 16:37 | |
patrickeast | hemna: thanks! | 16:37 |
*** mvk_ has quit IRC | 16:37 | |
mtanino | sheel: I thought it's better to announce this to admin via release note. | 16:38 |
*** baojg has joined #openstack-cinder | 16:38 | |
sheel | mtanino: ok, no issues... | 16:38 |
mtanino | sheel: thank you. | 16:39 |
sheel | matnino: Thanks to you for review | 16:39 |
mtanino | sheel: As for the APIImpact flag, I thought the patch changes existing behaviour even if it's corner case bug, I thought we needed APIImpact flag, but we need opinion from others. please leave it as is. | 16:42 |
*** mylu has quit IRC | 16:42 | |
*** vgridnev has joined #openstack-cinder | 16:43 | |
*** mylu has joined #openstack-cinder | 16:43 | |
*** ndipanov has quit IRC | 16:44 | |
*** baojg has quit IRC | 16:44 | |
sheel | mtanino : yes i too think so | 16:45 |
*** fernnest__ has joined #openstack-cinder | 16:45 | |
*** fernnest__ has quit IRC | 16:46 | |
*** mylu has quit IRC | 16:46 | |
sheel | mtanino: plz remove -1, lets have second opinion. | 16:46 |
*** mylu has joined #openstack-cinder | 16:47 | |
*** mylu has quit IRC | 16:50 | |
*** mylu has joined #openstack-cinder | 16:51 | |
*** mylu has quit IRC | 16:54 | |
*** jgriffith is now known as jgriffith_away | 16:58 | |
*** shyama has quit IRC | 16:59 | |
*** e0ne has quit IRC | 16:59 | |
*** haomaiwa_ has quit IRC | 17:01 | |
*** sgotliv has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-cinder | 17:01 | |
*** mylu has joined #openstack-cinder | 17:02 | |
*** jistr has quit IRC | 17:02 | |
*** dustins_ has quit IRC | 17:03 | |
*** edtubill_ has joined #openstack-cinder | 17:04 | |
*** edtubill has quit IRC | 17:05 | |
*** jordanP has quit IRC | 17:06 | |
*** daneyon has joined #openstack-cinder | 17:06 | |
*** daneyon_ has joined #openstack-cinder | 17:06 | |
*** daneyon has quit IRC | 17:11 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS driver: retry on Connection reset fix https://review.openstack.org/276357 | 17:14 |
*** dustins_ has joined #openstack-cinder | 17:14 | |
*** jistr has joined #openstack-cinder | 17:16 | |
*** jistr has quit IRC | 17:18 | |
*** edtubill_ has quit IRC | 17:19 | |
*** leeantho has joined #openstack-cinder | 17:20 | |
*** mylu has quit IRC | 17:22 | |
*** mudassirlatif has joined #openstack-cinder | 17:23 | |
*** merooney has joined #openstack-cinder | 17:24 | |
*** alonma has joined #openstack-cinder | 17:33 | |
*** kfarr has joined #openstack-cinder | 17:33 | |
openstackgerrit | Merged openstack/cinder: Tests: Strengthen assertFalse assertions https://review.openstack.org/263404 | 17:36 |
*** esker has quit IRC | 17:37 | |
*** esker has joined #openstack-cinder | 17:38 | |
*** alonma has quit IRC | 17:38 | |
openstackgerrit | Merged openstack/cinder: mock time.sleep in Broadcom unit test https://review.openstack.org/275952 | 17:38 |
*** salv-orlando has joined #openstack-cinder | 17:39 | |
*** krtaylor has quit IRC | 17:41 | |
*** alonma has joined #openstack-cinder | 17:42 | |
*** salv-orlando has quit IRC | 17:44 | |
*** diablo_rojo has quit IRC | 17:46 | |
*** alonma has quit IRC | 17:46 | |
*** edtubill has joined #openstack-cinder | 17:46 | |
*** jaqo78 has joined #openstack-cinder | 17:47 | |
*** baojg has joined #openstack-cinder | 17:50 | |
*** alkhodos has joined #openstack-cinder | 17:53 | |
*** diablo_rojo has joined #openstack-cinder | 17:55 | |
*** amit213 has joined #openstack-cinder | 17:55 | |
*** amit213 has quit IRC | 17:55 | |
*** krtaylor has joined #openstack-cinder | 17:56 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS driver: retry on Connection reset fix https://review.openstack.org/276357 | 17:56 |
*** amit213 has joined #openstack-cinder | 17:56 | |
*** erlon has quit IRC | 17:56 | |
*** baojg has quit IRC | 17:59 | |
*** sheel has quit IRC | 17:59 | |
*** daneyon_ has quit IRC | 17:59 | |
*** timcl1 has joined #openstack-cinder | 18:00 | |
*** haomaiwang has quit IRC | 18:01 | |
*** haomaiwang has joined #openstack-cinder | 18:01 | |
*** timcl has quit IRC | 18:03 | |
*** merooney has quit IRC | 18:03 | |
*** merooney has joined #openstack-cinder | 18:03 | |
*** esker has quit IRC | 18:04 | |
*** arch-nemesis has quit IRC | 18:05 | |
*** arch-nemesis has joined #openstack-cinder | 18:06 | |
*** rockeychen has quit IRC | 18:09 | |
*** jungleboyj has quit IRC | 18:09 | |
*** rockeychen has joined #openstack-cinder | 18:09 | |
*** alonma has joined #openstack-cinder | 18:09 | |
*** jdurgin1 has quit IRC | 18:10 | |
*** jdurgin1 has joined #openstack-cinder | 18:11 | |
*** jdurgin1 has quit IRC | 18:13 | |
jbernard | DuncanT: hey, did you have questions about RBD iscsi? | 18:14 |
jbernard | DuncanT: i may be misremembering | 18:14 |
jbernard | DuncanT: but i understadn the approach now | 18:14 |
*** alonma has quit IRC | 18:14 | |
*** dustins_ has quit IRC | 18:15 | |
*** alonma has joined #openstack-cinder | 18:17 | |
*** alonma has quit IRC | 18:21 | |
openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support https://review.openstack.org/255015 | 18:21 |
*** ociuhandu has quit IRC | 18:22 | |
*** alonma has joined #openstack-cinder | 18:24 | |
*** salv-orlando has joined #openstack-cinder | 18:24 | |
*** jungleboyj has joined #openstack-cinder | 18:25 | |
*** apoorvad has joined #openstack-cinder | 18:26 | |
*** salv-orlando has quit IRC | 18:28 | |
*** e0ne has joined #openstack-cinder | 18:28 | |
*** alonma has quit IRC | 18:29 | |
*** lpetrut has quit IRC | 18:30 | |
*** tongli has joined #openstack-cinder | 18:31 | |
*** ChubYann has joined #openstack-cinder | 18:38 | |
*** alonma has joined #openstack-cinder | 18:38 | |
*** diablo_rojo has quit IRC | 18:41 | |
*** alonma has quit IRC | 18:43 | |
*** merooney has quit IRC | 18:43 | |
*** alonma has joined #openstack-cinder | 18:45 | |
*** timcl1 has quit IRC | 18:47 | |
*** rlrossit has quit IRC | 18:47 | |
*** rlrossit has joined #openstack-cinder | 18:48 | |
*** bardia has joined #openstack-cinder | 18:49 | |
*** alonma has quit IRC | 18:50 | |
*** alonma has joined #openstack-cinder | 18:51 | |
*** rlrossit has quit IRC | 18:52 | |
*** rlrossit has joined #openstack-cinder | 18:53 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/275729 | 18:53 |
*** esker has joined #openstack-cinder | 18:54 | |
*** timcl has joined #openstack-cinder | 18:55 | |
*** alonma has quit IRC | 18:55 | |
smcginnis | patrickeast: ping | 18:56 |
patrickeast | smcginnis: pong | 18:56 |
smcginnis | patrickeast: Hey! Question for you - is your FC CI running? | 18:57 |
patrickeast | smcginnis: good question :o | 18:57 |
patrickeast | smcginnis: it died last night for a while | 18:57 |
smcginnis | patrickeast: :) | 18:57 |
patrickeast | smcginnis: i *think* everything is ok now though | 18:57 |
smcginnis | patrickeast: I'm getting some odd failures on mine. | 18:57 |
smcginnis | patrickeast: Seems to only happen for my FC tests. | 18:57 |
patrickeast | smcginnis: oh yea? | 18:57 |
smcginnis | patrickeast: Curious to see if you're seeing it to, or if it's something localized. | 18:57 |
patrickeast | smcginnis: i saw one weird error, but couldn't repro it | 18:57 |
hemna | ! | 18:58 |
* patrickeast digs up the link | 18:58 | |
smcginnis | I'm getting: | 18:58 |
smcginnis | VolumeBackendAPIException: Bad or unexpected response from the storage volume backend API: Validate volume connection failed (error: 'staticmethod' object is not callable) | 18:58 |
patrickeast | oh dang | 18:58 |
hemna | wth | 18:58 |
patrickeast | yea | 18:58 |
patrickeast | http://openstack-logs.purestorage.com/19/275919/1/check/PureFCDriver-tempest-dsvm-volume-multipath/21e6a5e/logs/screen-c-vol.txt.gz?level=TRACE | 18:58 |
patrickeast | i had some "offline" hba's, figured that was the problem | 18:58 |
smcginnis | Which would indicate a wrapper method is incorrectly defined as static. | 18:58 |
patrickeast | but maybe that was just hiding it | 18:58 |
smcginnis | But I don't see it. | 18:58 |
hemna | n-cpu ? | 18:59 |
patrickeast | yea i stared at the code for a while last night and didn't see how it could be working, or any recent changes | 18:59 |
smcginnis | patrickeast: Hah, at least it's not just me. ;) | 18:59 |
hemna | are you guys picking up my brick patch ? | 18:59 |
hemna | https://review.openstack.org/#/c/275943/ | 18:59 |
smcginnis | hemna: I don't _think_ so, but maybe? | 18:59 |
patrickeast | shouldn't be | 18:59 |
* patrickeast checks | 18:59 | |
* smcginnis checks 2 | 18:59 | |
hemna | yah, I'd be concerned if you were for a Cinder CI run | 18:59 |
patrickeast | http://openstack-logs.purestorage.com/19/275919/1/check/PureFCDriver-tempest-dsvm-volume-multipath/21e6a5e/logs/pip-freeze.txt.gz | 19:00 |
hemna | but it sure smells coincidental | 19:00 |
patrickeast | hemna: mine was for a cinder patch | 19:00 |
smcginnis | No, doesn't look like it. I have LIBS_FROM_GIT=os-brick commented out. | 19:00 |
*** jaqo78 has quit IRC | 19:00 | |
hemna | -e git+https://review.openstack.org/p/openstack/os-brick@4b3dbdcb2fb52043b1f8d4df1a3702d4e5319b73#egg=os_brick | 19:00 |
patrickeast | http://git.openstack.org/cgit/openstack/os-brick/commit/?id=4b3dbdcb2fb52043b1f8d4df1a3702d4e5319b73 | 19:01 |
*** haomaiwang has quit IRC | 19:01 | |
smcginnis | Last requirements change was a few days ago. | 19:01 |
hemna | damn release notes killed FC! | 19:01 |
*** haomaiwang has joined #openstack-cinder | 19:01 | |
hemna | I knew it | 19:01 |
smcginnis | Hah! :) | 19:01 |
e0ne | :) | 19:01 |
patrickeast | hmm so i wonder what happened to the fc tests | 19:02 |
hemna | ooh | 19:02 |
*** timcl1 has joined #openstack-cinder | 19:02 | |
hemna | validate_connector | 19:02 |
hemna | that's what is puking | 19:03 |
patrickeast | yea https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L2858 | 19:03 |
smcginnis | Calls self.driver.validate_connector_has_setting that is static, but this looks like a wrapper. | 19:03 |
patrickeast | which is calling some static stuff...but no idea why that would be a problem | 19:03 |
patrickeast | git blame doesn't show any recent changes around here | 19:03 |
*** baumann has quit IRC | 19:03 | |
*** timcl has quit IRC | 19:04 | |
patrickeast | and i didn't see anything suspicious in cinder, brick, or nova | 19:04 |
smcginnis | patrickeast: You're at least confirming what I've seen. :) | 19:04 |
patrickeast | well, in the last few days at least | 19:04 |
hemna | why is validate_Connector_has_setting static ? | 19:04 |
hemna | odd | 19:04 |
patrickeast | doesn't need self anywhere, so why not? | 19:04 |
hemna | maybe unit test? | 19:05 |
hemna | but statics are usually because someone outside the class needs to call it directly w/o instantiating the object | 19:05 |
hemna | meh | 19:05 |
*** dansmith has quit IRC | 19:05 | |
patrickeast | yea | 19:05 |
*** dansmith has joined #openstack-cinder | 19:05 | |
patrickeast | well... if using pycharm it will complain and tell you things should be static if they can be | 19:06 |
smcginnis | Not entirely sure how python does overriding with static methods, but there is another staticmethod validate_connector_has_settings in BaseVD. | 19:06 |
hemna | I wonder if this is why my os-brick FC tests are failing as well | 19:06 |
smcginnis | hemna: I wouldn't be surprised. | 19:06 |
smcginnis | hemna: That patch looked fine to me. | 19:06 |
hemna | pycharm? what's that ? :P | 19:06 |
patrickeast | smcginnis: yea, not sure how that works... but the real confusing part is why it changed recently | 19:06 |
patrickeast | haha | 19:06 |
smcginnis | Yeah, I can't find what triggered this to suddenly start failing. | 19:07 |
smcginnis | That code has been like that for some time. | 19:07 |
smcginnis | So I think the code itself is fine. | 19:07 |
patrickeast | smcginnis: looking at my now-online hba jobs (that aren't failing after like 30 seconds) i think its still hitting the issues | 19:07 |
*** baumann has joined #openstack-cinder | 19:08 | |
smcginnis | osprofiler is in the stack, and that was updated recently, but not that recent. | 19:08 |
*** diablo_rojo has joined #openstack-cinder | 19:08 | |
*** diablo_rojo has left #openstack-cinder | 19:08 | |
*** diablo_rojo has joined #openstack-cinder | 19:08 | |
hemna | so it looks like that will happen if someone uses self inside a static | 19:09 |
* hemna is confused | 19:09 | |
patrickeast | this is the time i wish i had logstash for 3rd party ci's... | 19:10 |
hemna | yah | 19:10 |
patrickeast | could see exactly when it started | 19:10 |
hemna | if that wasn't such a bitch to setup | 19:11 |
*** david-lyle has quit IRC | 19:11 | |
hemna | .... | 19:11 |
smcginnis | I've thought about setting it up locally, but eh | 19:11 |
patrickeast | haha yea seriously | 19:11 |
hemna | I tried once... | 19:11 |
patrickeast | ive got it on my backlog | 19:11 |
*** baumann has left #openstack-cinder | 19:11 | |
patrickeast | but the instructions ive seen are daunting | 19:11 |
hemna | it's almost as bad as common CI itself | 19:11 |
hemna | walk away.....don't look back..... | 19:11 |
patrickeast | it would be cool if we could just point our ci's to push logs to the upstream one | 19:11 |
patrickeast | i'll start setting up a fc devstack to debug this more | 19:13 |
patrickeast | takes a while since i have to steal precious hba's from the ci system without breaking anything | 19:13 |
smcginnis | patrickeast: Yeah, that's my problem too. Not as easy to do a quick test as it is with iSCSI. | 19:14 |
smcginnis | patrickeast: Thanks for doing that. I'll work on it here too and let you know if I find anything. | 19:14 |
openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/276060 | 19:15 |
*** angela-s has joined #openstack-cinder | 19:16 | |
patrickeast | smcginnis: hemna: i filed https://bugs.launchpad.net/cinder/+bug/1541996 to track this | 19:20 |
openstack | Launchpad bug 1541996 in Cinder "FC CI jobs are failing with" error: 'staticmethod' object is not callable"" [Undecided,New] | 19:20 |
hemna | ok | 19:21 |
smcginnis | patrickeast: Started for me noon yesterday from what I can find in my logs. | 19:22 |
hemna | https://review.openstack.org/#/c/275943/ that went up around 2pm yesterday | 19:26 |
hemna | I don't think that's related though | 19:26 |
smcginnis | hemna: Yeah, I'm pretty sure that's not it. And I think it started failing before that for me. | 19:27 |
openstackgerrit | Angela Smith proposed openstack/cinder: Adds support for configuring zoning in a virtual fabric https://review.openstack.org/275407 | 19:28 |
*** dustins has joined #openstack-cinder | 19:29 | |
*** baojg has joined #openstack-cinder | 19:29 | |
e0ne | why do we have 'validate_connector' as static in base driver? | 19:32 |
hemna | e0ne, heh. that's what I was wondering as well | 19:32 |
hemna | someone else outside it must be calling it | 19:32 |
hemna | ? | 19:32 |
e0ne | I didn't find any | 19:33 |
smcginnis | I don't think it needs to be, but I know some folks think if it doesn't need to use anything self.X it should be marked static. | 19:33 |
smcginnis | Eh | 19:33 |
hemna | that's....just....odd | 19:33 |
smcginnis | Seems really weird to me, because other languages don't work right "inheriting" static methods. | 19:34 |
hemna | yah I'm not sure how that works | 19:34 |
e0ne | https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L2858-L2867 - wtf? | 19:36 |
smcginnis | Hmm, oslo.messaging was release around then and we don't have a cap on that requirement so it could have been picked up. | 19:36 |
smcginnis | Nothing had changed in Cinder itself between my last successful run and when the failures started. | 19:37 |
smcginnis | So it's something external it appears. | 19:37 |
e0ne | smcginnis: it doesn't mean that cinder is not broken | 19:37 |
smcginnis | e0ne: True, it could have just made something latent surface. | 19:37 |
smcginnis | hemna: BTW, we have 2 weeks to get anything else in os-brick before we need to do a final non-client library cut for Mitaka. | 19:39 |
smcginnis | I had planned on looking what we have out there before I got pulled in to this. | 19:40 |
hemna | damn 2 weeks | 19:40 |
hemna | ! | 19:40 |
hemna | ok | 19:41 |
smcginnis | hemna: Coming up fast! | 19:41 |
hemna | shit, the privsep stuff needs to get in | 19:41 |
hemna | and tested | 19:41 |
hemna | but all we have is the remotefs patch | 19:41 |
hemna | I'm trying to test that today | 19:41 |
smcginnis | hemna: OK, cool. | 19:41 |
hemna | network seems to be against me though | 19:41 |
smcginnis | Maybe next meeting we should do a call out to anyone willing and able to help with privsep. | 19:42 |
smcginnis | That would be a bummer if we can't get that in. | 19:42 |
hemna | yah, it's a lot of work unforuntately | 19:42 |
hemna | and big risk | 19:42 |
hemna | it needs a lot of testing | 19:42 |
smcginnis | hemna: Yeah, the more runtime on that the better. | 19:42 |
hemna | yup | 19:42 |
*** mragupat has joined #openstack-cinder | 19:48 | |
hemna | migrating my vagrant image builder to using my apt-mirror | 19:49 |
hemna | bandersnatch finished, but I can't get pip to work with my local mirror :( | 19:50 |
smcginnis | hemna: Isn't there some environment variable to tell it where to go? | 19:51 |
hemna | well pip hits it, but it complains about missing stuffs. | 19:51 |
hemna | :( | 19:51 |
*** alonma has joined #openstack-cinder | 19:51 | |
hemna | just haven't messed with it much | 19:51 |
*** merooney has joined #openstack-cinder | 19:54 | |
*** tiki45 has joined #openstack-cinder | 19:54 | |
*** salv-orlando has joined #openstack-cinder | 19:54 | |
*** alonma has quit IRC | 19:55 | |
*** stevemar has quit IRC | 19:56 | |
*** alonma has joined #openstack-cinder | 19:58 | |
*** timcl1 has quit IRC | 20:00 | |
*** rlrossit has quit IRC | 20:00 | |
*** haomaiwang has quit IRC | 20:01 | |
mc_nair | smcginnis: a while back you sent a link to a site that's basically grep OpenStack online... and then life was good. I "misplaced" some bookmarks, do you happen to still have that link handy? | 20:01 |
*** haomaiwang has joined #openstack-cinder | 20:01 | |
*** timcl has joined #openstack-cinder | 20:01 | |
*** jwcroppe has quit IRC | 20:01 | |
*** jwcroppe has joined #openstack-cinder | 20:02 | |
*** alonma has quit IRC | 20:03 | |
*** baojg has quit IRC | 20:03 | |
scottda | mc_nair: this? http://codesearch.openstack.org | 20:04 |
mc_nair | scottda: that's the one :) | 20:05 |
*** alonma has joined #openstack-cinder | 20:05 | |
mc_nair | scottda: thanks! Was driving me nuts trying to remember what it was... | 20:05 |
*** jwcroppe has quit IRC | 20:07 | |
*** alonma has quit IRC | 20:09 | |
*** vgridnev has quit IRC | 20:11 | |
*** sgotliv has joined #openstack-cinder | 20:12 | |
*** andymaier has quit IRC | 20:13 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make validate_connector_has_setting non-static https://review.openstack.org/276425 | 20:14 |
e0ne | patrickeast, smcginnis: could you please run yours CIs against of ^^ | 20:14 |
smcginnis | e0ne: Thanks, I was thinking of doing that. Will be interesting to see if that does make a difference. | 20:14 |
*** alonma has joined #openstack-cinder | 20:14 | |
*** baumann has joined #openstack-cinder | 20:14 | |
patrickeast | e0ne: its in the queue for mine, should have results in a little while | 20:15 |
e0ne | anyway, it should not break anything, just a some cleanup to make code cleaner | 20:16 |
e0ne | patrickeast, smcginnis: thanks | 20:16 |
*** xyang1 has joined #openstack-cinder | 20:16 | |
* e0ne hopes that nobody will ask to write unit-tests fot that patch | 20:16 | |
smcginnis | hehe | 20:17 |
hemna | e0ne, did you find out if anyone external is using that ? | 20:17 |
hemna | statically ? | 20:18 |
e0ne | no:( | 20:18 |
hemna | I guess CI will say | 20:18 |
*** tiki45 has quit IRC | 20:18 | |
e0ne | hemna: TBH. I don't know why CIs fail | 20:18 |
hemna | me either | 20:18 |
*** jgriffith_away is now known as jgriffith | 20:19 | |
smcginnis | Yeah, this one has be really confused. | 20:19 |
*** alonma has quit IRC | 20:19 | |
smcginnis | It has to be something external. | 20:19 |
e0ne | hemna: http://paste.openstack.org/show/486010/ | 20:19 |
smcginnis | But why it makes it fail in this particular way, I have no idea. | 20:19 |
hemna | huh | 20:19 |
hemna | got me | 20:19 |
e0ne | that's why I prefer do not use staticmethods | 20:20 |
hemna | unless it's really static | 20:20 |
hemna | re: needing to be called from outside | 20:20 |
e0ne | they are not needed in the most cases | 20:20 |
e0ne | hemna: make it simple function, noert class memb | 20:20 |
smcginnis | classmethod is another interesting one. Took me a while to figure out the difference and why you would use one or the other. | 20:21 |
e0ne | s/noert class memb/not a class member | 20:21 |
hemna | we like things convoluted and complicated here in Cinder. | 20:21 |
patrickeast | ugh devstack isn't work now, anyone else seeing http://paste.openstack.org/show/486011/ | 20:21 |
smcginnis | ;) | 20:21 |
patrickeast | working* | 20:21 |
smcginnis | patrickeast: Yeah, I was getting that yesterday. | 20:21 |
hemna | I saw that earlier today | 20:21 |
* e0ne needs to take a break | 20:22 | |
patrickeast | any work-around? | 20:22 |
hemna | I just spun up a new vagrant...fixed! | 20:22 |
smcginnis | Er, no something different. SOrry. | 20:22 |
e0ne | bb tomorrow (~in 12 hours) | 20:22 |
smcginnis | e0ne: Have a good one. | 20:22 |
*** alonma has joined #openstack-cinder | 20:22 | |
smcginnis | patrickeast: Oh, no, that is it. | 20:22 |
smcginnis | I was getting that no such option -c thing. | 20:22 |
smcginnis | I think I did several ./clean.sh and apt-get autoremove, apt-get autoclean. | 20:22 |
smcginnis | Ultimately I think I had to blow away by devstack dir and re-git clone it, then it was fine. | 20:23 |
smcginnis | s/by devstack/my devstack/ | 20:23 |
*** rockeychen has quit IRC | 20:26 | |
*** rockeychen has joined #openstack-cinder | 20:26 | |
*** alonma has quit IRC | 20:27 | |
patrickeast | hmm must have been some untracked files, git clean -fdx and re-stacking did the trick | 20:28 |
smcginnis | Well, now you're just getting fancy. Show off. | 20:29 |
smcginnis | :P | 20:29 |
*** alonma has joined #openstack-cinder | 20:30 | |
patrickeast | fancy git is the only git i know | 20:30 |
* smcginnis only know enough git to get by | 20:31 | |
*** mylu has joined #openstack-cinder | 20:31 | |
*** jgriffith is now known as jgriffith_away | 20:31 | |
openstackgerrit | Jacob Gregor proposed openstack/cinder: Reduce redundant code in Storwize SVC tests https://review.openstack.org/276433 | 20:32 |
smcginnis | eharney: Hey. e0ne's patch should probably be marked "TEST" or something. | 20:32 |
smcginnis | eharney: We're trying to figure out what the subtle bug is, or at least get around it. | 20:33 |
eharney | smcginnis: i think the problem is that you aren't printing the whole exception that occurs | 20:33 |
smcginnis | eharney: Yeah, that was going to be my next step. We're swallowing some of the details that might help. | 20:34 |
patrickeast | yea | 20:34 |
eharney | you need a stack trace where it occurs rather than where we repeat it | 20:34 |
patrickeast | ive got a fc devstack almost there (assuming no more suprises from devstack) | 20:34 |
*** TheJulia has quit IRC | 20:34 | |
*** alonma has quit IRC | 20:35 | |
*** dustins has quit IRC | 20:35 | |
*** TheJulia has joined #openstack-cinder | 20:35 | |
*** timcl has quit IRC | 20:36 | |
*** stevemar has joined #openstack-cinder | 20:36 | |
*** fthiagogv_ has joined #openstack-cinder | 20:37 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX-Replacing deprecated API EMCGetTargetEndpoints https://review.openstack.org/244328 | 20:37 |
*** alonma has joined #openstack-cinder | 20:38 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - necessary updates for CG changes https://review.openstack.org/268223 | 20:38 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: TEST: Troubleshoot FC test failures https://review.openstack.org/276436 | 20:39 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/275729 | 20:39 |
smcginnis | eharney, patrickeast: ^^ Thought I might as well get something in the queue to get those exception details. | 20:39 |
smcginnis | It will be a bit before it makes it through my CI's queue. | 20:40 |
patrickeast | same :( | 20:41 |
patrickeast | zuul is saying things are about 6 hours behind on mine | 20:41 |
*** wolsen_ has joined #openstack-cinder | 20:41 | |
smcginnis | I'm about brain dead on this for now anyway. I'll check tonight if there's any useful logging. | 20:42 |
*** alonma has quit IRC | 20:42 | |
*** edtubill has quit IRC | 20:44 | |
*** vgridnev has joined #openstack-cinder | 20:45 | |
*** timcl has joined #openstack-cinder | 20:45 | |
*** briancurtin_ has joined #openstack-cinder | 20:46 | |
*** mattymo_ has joined #openstack-cinder | 20:46 | |
*** fmccrthy_ has joined #openstack-cinder | 20:46 | |
*** charlesr has joined #openstack-cinder | 20:47 | |
*** alonma has joined #openstack-cinder | 20:47 | |
*** mudassirlatif has quit IRC | 20:47 | |
*** crose has quit IRC | 20:47 | |
*** porrua has quit IRC | 20:47 | |
*** isaacb has quit IRC | 20:47 | |
*** briancurtin has quit IRC | 20:47 | |
*** fmccrthy has quit IRC | 20:47 | |
*** fthiagogv has quit IRC | 20:47 | |
*** lennyb has quit IRC | 20:47 | |
*** bswartz has quit IRC | 20:47 | |
*** v12aml has quit IRC | 20:47 | |
*** dulek has quit IRC | 20:47 | |
*** wolsen has quit IRC | 20:47 | |
*** mattymo has quit IRC | 20:47 | |
*** haomaiwang has quit IRC | 20:47 | |
*** fmccrthy_ is now known as fmccrthy | 20:48 | |
*** lennyb has joined #openstack-cinder | 20:48 | |
*** briancurtin_ is now known as briancurtin | 20:48 | |
*** dustins has joined #openstack-cinder | 20:49 | |
*** mylu has quit IRC | 20:49 | |
*** baumann1 has joined #openstack-cinder | 20:49 | |
*** mudassirlatif has joined #openstack-cinder | 20:49 | |
*** porrua has joined #openstack-cinder | 20:49 | |
*** v12aml has joined #openstack-cinder | 20:49 | |
*** dulek has joined #openstack-cinder | 20:49 | |
*** haomaiwang has joined #openstack-cinder | 20:49 | |
*** dulek has quit IRC | 20:50 | |
*** mylu has joined #openstack-cinder | 20:50 | |
*** baumann has quit IRC | 20:50 | |
*** bswartz has joined #openstack-cinder | 20:50 | |
*** mudassirlatif has quit IRC | 20:51 | |
*** porrua has quit IRC | 20:51 | |
*** mudassirlatif has joined #openstack-cinder | 20:52 | |
*** alonma has quit IRC | 20:52 | |
*** stevemar has quit IRC | 20:52 | |
*** mylu has quit IRC | 20:52 | |
*** alonma has joined #openstack-cinder | 20:53 | |
*** mylu has joined #openstack-cinder | 20:54 | |
*** alonma has quit IRC | 20:58 | |
*** merooney has quit IRC | 20:58 | |
*** isaacb has joined #openstack-cinder | 20:59 | |
*** apoorvad has quit IRC | 21:00 | |
*** haomaiwang has quit IRC | 21:01 | |
*** jungleboyj has quit IRC | 21:01 | |
*** haomaiwang has joined #openstack-cinder | 21:01 | |
*** alonma has joined #openstack-cinder | 21:01 | |
*** e0ne has quit IRC | 21:04 | |
patrickeast | got a (slightly) better stacktrace http://paste.openstack.org/show/486015/ | 21:04 |
*** baojg has joined #openstack-cinder | 21:04 | |
patrickeast | an interesting note, i couldn't repro this just by attaching a volume, had to run the DeleteServersTestJSON tempest test | 21:05 |
smcginnis | So that makes it look like it is actually due to calling the static method. Hm | 21:05 |
eharney | i wonder if osprofiler works as expected w/ staticmethods | 21:05 |
smcginnis | eharney: It did make me a little leary seeing that in the stack. | 21:06 |
*** alonma has quit IRC | 21:06 | |
*** raildo is now known as raildo-afk | 21:06 | |
eharney | 'staticmethod' is a decorator, not a method | 21:06 |
patrickeast | applying e0ne's fix and seeing how that helps | 21:06 |
smcginnis | Yeah, it's like something is not decorating the decorator correctly. | 21:06 |
eharney | smcginnis: there's a reason i wrote a patch long ago to make it possible to not have it hooked into everything | 21:07 |
smcginnis | eharney: Oh? What was that? | 21:07 |
*** mylu has quit IRC | 21:07 | |
eharney | smcginnis: made it possible to run Cinder w/o loading it, so we didn't have to sit here wondering about such things when looking at a failure | 21:07 |
smcginnis | eharney: Is that still out there? | 21:07 |
*** alonma has joined #openstack-cinder | 21:08 | |
eharney | smcginnis: it got abandoned because people didn't want to go that route | 21:08 |
smcginnis | Hmm, this is the last commit on osprofiler: | 21:10 |
smcginnis | https://github.com/openstack/osprofiler/commit/395edeaafb96211870521baa76c68552189c1c8f | 21:10 |
*** esp_ has joined #openstack-cinder | 21:10 | |
eharney | hrm | 21:10 |
*** xyang has joined #openstack-cinder | 21:10 | |
patrickeast | last release was 2 days ago https://github.com/openstack/osprofiler/releases | 21:11 |
*** mylu has joined #openstack-cinder | 21:11 | |
patrickeast | timing would be pretty close | 21:11 |
patrickeast | the "fix" of removing the static decorators does stop the problem from happening | 21:12 |
*** alonma has quit IRC | 21:12 | |
xyang | mtanino: ping | 21:13 |
patrickeast | what channel do we bug people in about osprofiler? | 21:13 |
mtanino | xyang: pong | 21:13 |
openstackgerrit | Merged openstack/cinder: Don't create cgsnapshot if cg is empty https://review.openstack.org/276023 | 21:13 |
*** mylu has quit IRC | 21:13 | |
smcginnis | Boris should be online in the morning I think. | 21:13 |
*** esp_ has quit IRC | 21:14 | |
xyang | mtanino: about this bug: https://bugs.launchpad.net/cinder/+bug/1536873 | 21:15 |
openstack | Launchpad bug 1520102 in Cinder "duplicate for #1536873 Migration: additional volume deletion will fail" [Undecided,Fix released] - Assigned to Thang Pham (thang-pham) | 21:15 |
mtanino | xyang: yes, | 21:15 |
xyang | mtanino: Peter re-tested it. the original problem was fixed but he ran into a problem deleting temp volume | 21:15 |
xyang | mtanino: versioned_objects version problem | 21:16 |
patrickeast | looks like sudo pip install 'osprofiler<0.4.0' fixes it too | 21:16 |
xyang | mtanino: have you seen that | 21:16 |
*** alonma has joined #openstack-cinder | 21:16 | |
mtanino | xyang: Did you have a bug# for that new one? I haven't seen it. | 21:16 |
xyang | mtanino: we didn't open a bug yet. I don't know if it is a set up issue. the error is ERROR oslo_messaging.rpc.dispatcher RPCVersionCapError: Requested message version, 1.36 is incompatible. It needs to be equal in major version and less than or equal in minor version as the specified version cap 1.30. | 21:17 |
mtanino | xyang: It's new for me. May be he should set up clean devstack environment and if he still meet same problem, need to report as new bug. | 21:19 |
xyang | mtanino: ya, it could be the installed versioned_objects is too old. I'll ask him to test with clean setup | 21:20 |
mtanino | xyang: I hope so. thanks. | 21:20 |
*** dguryanov has quit IRC | 21:20 | |
xyang | mtanino: thank you | 21:20 |
*** alonma has quit IRC | 21:21 | |
patrickeast | smcginnis: eharney: so as a temporary fix for this (and to make the code less confusing) should we go ahead and remove the static decorators? | 21:21 |
patrickeast | can follow up on osprofiler later, but seems like that is easier than pinning the version on it | 21:21 |
*** timcl has quit IRC | 21:21 | |
smcginnis | patrickeast: Yeah, if e0ne's patch effectively works around it, that's probably the answer for now. | 21:22 |
smcginnis | patrickeast: I'm writing some osprofiler unit tests right now to see if I can get it to repro the failure. | 21:22 |
*** mvk has joined #openstack-cinder | 21:22 | |
smcginnis | patrickeast: If I can get that (not) working I'll see if I can propose a fix. | 21:23 |
eharney | this should be something we can reproduce in cinder unit tests | 21:23 |
smcginnis | osprofiler is disabled in unit tests. | 21:23 |
*** dguryanov has joined #openstack-cinder | 21:23 | |
eharney | smcginnis: well, partially disabled | 21:24 |
eharney | actually maybe it is | 21:24 |
eharney | and i wrote that patch, who knew | 21:24 |
smcginnis | ;) | 21:24 |
*** akerr has quit IRC | 21:26 | |
*** gouthamr_ has quit IRC | 21:28 | |
*** rlrossit has joined #openstack-cinder | 21:32 | |
*** baojg has quit IRC | 21:32 | |
*** zul has joined #openstack-cinder | 21:42 | |
*** cknight has quit IRC | 21:43 | |
*** sgotliv has quit IRC | 21:44 | |
*** fmccrthy has quit IRC | 21:44 | |
*** fmccrthy has joined #openstack-cinder | 21:47 | |
*** mattymo_ has quit IRC | 21:47 | |
*** mattymo has joined #openstack-cinder | 21:49 | |
*** mylu has joined #openstack-cinder | 21:51 | |
*** xyang has quit IRC | 21:51 | |
*** xyang has joined #openstack-cinder | 21:51 | |
*** apoorvad has joined #openstack-cinder | 21:52 | |
*** julim has quit IRC | 21:55 | |
*** julim has joined #openstack-cinder | 21:55 | |
*** mylu has quit IRC | 21:56 | |
*** baumann1 has left #openstack-cinder | 21:56 | |
*** rlrossit has quit IRC | 21:57 | |
*** mylu has joined #openstack-cinder | 21:57 | |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in smbfs volume driver https://review.openstack.org/261551 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in vzstorage volume driver https://review.openstack.org/261550 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in netapp_nfs unit test https://review.openstack.org/261557 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in glusterfs volume driver https://review.openstack.org/261552 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in scality volume driver https://review.openstack.org/261553 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/270908 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in quobyte volume driver https://review.openstack.org/261554 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Use versionedobjects in nfs volume driver https://review.openstack.org/261555 | 21:57 |
hemna | whoa | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Update initialize_connection to use versionedobjects https://review.openstack.org/238894 | 21:57 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: vzstorage: fix creating and deleting snapshots of in-use volumes https://review.openstack.org/276465 | 21:57 |
*** mylu has quit IRC | 21:58 | |
*** kfarr has quit IRC | 21:59 | |
jbernard | smcginnis: heya, im thinking about submitting an outreachy project for cinder | 22:00 |
jbernard | smcginnis: and wondered if you had any bit-sized project that might fit into that format well | 22:00 |
*** haomaiwang has quit IRC | 22:01 | |
*** haomaiwa_ has joined #openstack-cinder | 22:01 | |
jbernard | smcginnis: i spun through the todo list we made last week, but most are a bit more involved | 22:01 |
*** julim has quit IRC | 22:02 | |
*** edtubill has joined #openstack-cinder | 22:02 | |
*** mudassirlatif_ has joined #openstack-cinder | 22:02 | |
*** eharney has quit IRC | 22:03 | |
*** mudassirlatif has quit IRC | 22:04 | |
*** mudassirlatif_ is now known as mudassirlatif | 22:04 | |
smcginnis | jbernard: Yeah, the smaller stuff seems to get snapped up pretty quick. :) | 22:06 |
smcginnis | jbernard: I'll see if I can think of something. | 22:06 |
*** krtaylor has quit IRC | 22:06 | |
smcginnis | jbernard: We could also bring it up in the weekly meeting to see if anyone has any ideas they'd like to hand off. | 22:06 |
jbernard | smcginnis: that's a good call | 22:06 |
smcginnis | jbernard: That would be cool to get something going! | 22:06 |
jbernard | smcginnis: ill add it to the adjenda | 22:06 |
*** tpsilva has quit IRC | 22:06 | |
smcginnis | jbernard: Perfect, thanks! | 22:07 |
jbernard | smcginnis: dont thank me yet ;) | 22:07 |
smcginnis | hehe | 22:07 |
*** jamielennox|away is now known as jamielennox | 22:10 | |
openstackgerrit | Kendall Nelson proposed openstack/cinder: Remove keystonemiddleware endpoint from setup.cfg https://review.openstack.org/274810 | 22:12 |
*** cknight has joined #openstack-cinder | 22:13 | |
*** isaacb has quit IRC | 22:14 | |
*** alonma has joined #openstack-cinder | 22:16 | |
*** vgridnev has quit IRC | 22:17 | |
mtanino | jbernard: Does Transition to LIO means changing the default taget from tgtd to lio? | 22:18 |
jbernard | mtanino: yeah | 22:19 |
mtanino | jbernard: I see. | 22:19 |
jbernard | mtanino: in my mind, it's something we should do at some point | 22:19 |
jbernard | mtanino: so it seems reasonable to raise it now | 22:19 |
mtanino | jbernard: I like LIO, easy to use. | 22:20 |
*** krtaylor has joined #openstack-cinder | 22:20 | |
jbernard | mtanino: changing the default is trivial, i suspect the migration path is where the concern will lie | 22:20 |
*** alonma has quit IRC | 22:20 | |
mtanino | jbernard: yes, I agree on it. We should announce migration path, I guess. | 22:21 |
jbernard | mtanino: im not sure if we can just document what is needed and be done, or if we need to provide as part of the upgrade a tool to move everything over | 22:21 |
jbernard | mtanino: the latter scares me a little ;) | 22:21 |
mtanino | jbernard: upgrade tool... wow.. | 22:22 |
jbernard | mtanino: it's probably something the admin will want to do gradually | 22:22 |
*** alonma has joined #openstack-cinder | 22:22 | |
jbernard | as attached volumes will have to reamin until unneeded | 22:22 |
jbernard | exported targets for attached volumes, rather | 22:23 |
mtanino | jbernard: Does Red Hat have such kind of migration tool from tgtd to LIO already? | 22:23 |
jbernard | mtanino: not that im aware of | 22:23 |
jbernard | mtanino: im really hoping a transition document is sufficient | 22:23 |
jbernard | mtanino: but ive not propsed such a thing before, so im not sure of the requirements | 22:24 |
mtanino | jbernard: better to discuss at the meeting. | 22:24 |
*** diablo_rojo has quit IRC | 22:24 | |
jbernard | mtanino: yeah | 22:24 |
mtanino | jbernard: I got it. Thanks. | 22:25 |
jbernard | mtanino: no problem | 22:25 |
*** jgregor has quit IRC | 22:25 | |
*** alonma has quit IRC | 22:26 | |
*** savihou has joined #openstack-cinder | 22:34 | |
anteaya | savihou: hello | 22:34 |
anteaya | smcginnis: savihou is trying to set up a ci system with cinder | 22:34 |
savihou | anteaya: hello , thanks for your help | 22:34 |
anteaya | savihou: well I haven't helped much since I'm not entirely sure what it is you want to do | 22:34 |
anteaya | but if you want to test cinder things meeting the cinder team is a good place to begin | 22:35 |
anteaya | savihou: so to start I would look in gerrit at current cinder patches | 22:35 |
anteaya | https://review.openstack.org/#/q/project:openstack/cinder | 22:36 |
anteaya | and read the build log for dsvm tempest tests that run on cinder patches | 22:36 |
anteaya | it helps you understand how the build process works | 22:36 |
anteaya | so that when you try to troubleshoot how it is working with your system you know what step it is stuck on | 22:36 |
savihou | anteaya: you tried, that's a start... I believe I'm facing an infra issue here - as im not reaching the cinder phase .. | 22:36 |
anteaya | and then perhaps you can figure out why it is stuck | 22:37 |
anteaya | believe me if you want to test cinder it is best to meet the cinder team | 22:37 |
savihou | anteaya - I'll dig in | 22:37 |
anteaya | thank you | 22:37 |
savihou | anteaya - thanks | 22:37 |
anteaya | many helpful folks in cinder with a whole lot of good experience running ci systems | 22:37 |
anteaya | welcome | 22:38 |
savihou | anteaya: :) | 22:38 |
*** salv-orl_ has joined #openstack-cinder | 22:39 | |
*** salv-orlando has quit IRC | 22:42 | |
*** arch-nemesis has quit IRC | 22:43 | |
*** rockeychen has quit IRC | 22:43 | |
*** mudassirlatif has quit IRC | 22:44 | |
*** rockeychen has joined #openstack-cinder | 22:44 | |
*** mylu has joined #openstack-cinder | 22:44 | |
*** david-lyle has joined #openstack-cinder | 22:48 | |
*** david-lyle has quit IRC | 22:52 | |
*** xyang has quit IRC | 22:53 | |
*** mudassirlatif has joined #openstack-cinder | 22:53 | |
*** mudassirlatif has quit IRC | 22:55 | |
*** angela-s has quit IRC | 22:57 | |
*** charlesr has quit IRC | 22:59 | |
*** mudassirlatif has joined #openstack-cinder | 22:59 | |
*** xyang has joined #openstack-cinder | 23:00 | |
*** haomaiwa_ has quit IRC | 23:01 | |
*** mylu has quit IRC | 23:01 | |
*** haomaiwang has joined #openstack-cinder | 23:01 | |
*** xyang1 has quit IRC | 23:02 | |
*** mudassirlatif has quit IRC | 23:04 | |
*** mylu has joined #openstack-cinder | 23:06 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - get iscsi ip from port in existing MV https://review.openstack.org/245997 | 23:16 |
*** JoseMello has quit IRC | 23:18 | |
*** alonma has joined #openstack-cinder | 23:23 | |
*** alonma has quit IRC | 23:27 | |
*** mudassirlatif has joined #openstack-cinder | 23:27 | |
*** mgarza has quit IRC | 23:27 | |
*** chlong has quit IRC | 23:30 | |
*** dims_ has quit IRC | 23:31 | |
*** jamielennox is now known as jamielennox|away | 23:31 | |
*** dims has joined #openstack-cinder | 23:32 | |
openstackgerrit | Patrick East proposed openstack/cinder: Allow for eradicating Pure volumes on Cinder delete https://review.openstack.org/275919 | 23:37 |
DuncanT | jbernard: I did have questions about rdb iSCSI, starting with a general description of the approach, and a link to any code or tutorial - the point being that I have a way of doing it via modifying the cinder driver, it works but is clunky. It's something I'd like to add as an option to cinder, but I'd like to use best practice not whatever hacky stuff I | 23:37 |
DuncanT | came up with | 23:37 |
openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Operations and timeout issues https://review.openstack.org/272760 | 23:39 |
*** mragupat_ has joined #openstack-cinder | 23:45 | |
*** xyang has quit IRC | 23:46 | |
*** mragupat has quit IRC | 23:49 | |
*** mragupat_ has quit IRC | 23:49 | |
*** gouthamr has joined #openstack-cinder | 23:52 | |
*** daneyon has joined #openstack-cinder | 23:58 | |
*** daneyon_ has joined #openstack-cinder | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!