*** garthb has quit IRC | 00:02 | |
*** akshai has quit IRC | 00:02 | |
openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Add Fujitsu ETERNUS DX Volume Driver (again) https://review.openstack.org/201500 | 00:02 |
---|---|---|
*** barra204 has quit IRC | 00:03 | |
jgriffith | tsekiyama: ping | 00:04 |
tsekiyama | jgriffith: hi | 00:04 |
jgriffith | tsekiyama: hey there | 00:04 |
jgriffith | tsekiyama: I just left a comment, but then thought I should just ping you here :) | 00:04 |
tsekiyama | jgriffith: yeah i'm just reading it :) | 00:05 |
jgriffith | tsekiyama: did the cinder as a glance backend stuff actually get fixed and merge? | 00:05 |
tsekiyama | jgriffith: not yet. I need a review from Cinder cores for it | 00:05 |
tsekiyama | jgriffith: but it works even without the glance side patch, | 00:06 |
tsekiyama | jgriffith: although the image cannot be accessed from nova ... | 00:06 |
jgriffith | tsekiyama: the nova part doesn't bother me so much... as it solves the cache problem at least | 00:07 |
jgriffith | tsekiyama: but I'm confused... so I can load this up with LVM and use it as is right now? | 00:07 |
*** smoriya has joined #openstack-cinder | 00:07 | |
jgriffith | Oh... :) | 00:07 |
*** topshare has quit IRC | 00:07 | |
tsekiyama | jgriffith: yeah glance backend can be used as the image pointer + ACL. just the is only used (read/write) in the cinder | 00:07 |
jgriffith | yeah, ok... I"m back now :) | 00:07 |
jgriffith | Yes, this is what we talked about a few weeks ago | 00:08 |
jgriffith | patrickeast_: ^^ | 00:08 |
patrickeast_ | oh nice | 00:08 |
tsekiyama | jgriffith: my patch for Glance is actually to make it readable/writable from others than cinder | 00:08 |
jgriffith | tsekiyama: That's actually a good bit simpler than I thought it was going to be | 00:08 |
jgriffith | tsekiyama: which is why I thought something was missing... but looking closer I get it | 00:09 |
jgriffith | tsekiyama: yeah, looking at those glance patches now.... | 00:09 |
jgriffith | tsekiyama: good stuff! | 00:09 |
tsekiyama | jgriffith: thanks :) | 00:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 00:09 |
tsekiyama | jgriffith: this makes upload-to-image and create-from-image really quick.. especially on LVM thin backend. Even 100GB images can be created just in a second. :) | 00:11 |
jgriffith | tsekiyama: yes!! I see that, it's way cool! | 00:11 |
jgriffith | tsekiyama: so it just automates a snapshot template and stores the access info in Glance | 00:11 |
tsekiyama | jgriffith: right | 00:12 |
jgriffith | tsekiyama: which is WAY easier than trying to juggle things internally in Cinder | 00:12 |
jgriffith | tsekiyama: I like it a lot, and anybody with a fast clone, or create from snap can benefit from it | 00:12 |
tsekiyama | jgriffith: And it can exploit ACL feature of Glance to sharing the volume data among tenants | 00:12 |
* jgriffith prepares to replace his crazy driver code for Liberty :) | 00:13 | |
mtanino | hahaha | 00:13 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/201821 | 00:13 |
tsekiyama | jgriffith: before writing some unit tests for them, I want to hear comments from other people | 00:14 |
tsekiyama | jgriffith: current problem is Glance side patch, which enables the image accessible from other projects and make this work to put images into the service tenant. | 00:16 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 00:16 |
tsekiyama | jgriffith: Glance side is stuck in reviews now... It is actually a bunch of Cinder API calls so I really hope having reviews from Cinder people to make it forward. | 00:17 |
*** gouthamr has joined #openstack-cinder | 00:19 | |
*** mtanino has quit IRC | 00:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/201821 | 00:20 |
*** tsekiyam_ has joined #openstack-cinder | 00:21 | |
jgriffith | tsekiyama: I'll make sure to put them on my list of reviews when I get descent internet connection | 00:21 |
jgriffith | tsekiyama: I'm in and out of connectivity right now (plane wifi), but I have to say I'm kind of impressed with how well it's worked for most of the trip :) | 00:22 |
*** vokt has quit IRC | 00:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 00:24 |
*** david-lyle has quit IRC | 00:25 | |
*** tsekiyama has quit IRC | 00:25 | |
*** tsekiyam_ has quit IRC | 00:26 | |
*** primechuck has quit IRC | 00:26 | |
*** diogogmt has joined #openstack-cinder | 00:27 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/201821 | 00:28 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/200898 | 00:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 00:31 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/201821 | 00:35 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Add support for driver retype https://review.openstack.org/200765 | 00:36 |
*** Apoorva has quit IRC | 00:39 | |
openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Add Fujitsu ETERNUS DX Volume Driver (again) https://review.openstack.org/201500 | 00:39 |
*** primechuck has joined #openstack-cinder | 00:41 | |
*** hemna is now known as hemnafk | 00:41 | |
*** leeantho has quit IRC | 00:43 | |
*** bitblt has quit IRC | 00:44 | |
*** laughterwym has joined #openstack-cinder | 00:50 | |
openstackgerrit | Raunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver https://review.openstack.org/198853 | 00:51 |
*** topshare has joined #openstack-cinder | 00:53 | |
*** heyun has joined #openstack-cinder | 00:54 | |
*** dims__ has quit IRC | 00:54 | |
*** Yogi1 has joined #openstack-cinder | 01:06 | |
*** lixiaoy1 has joined #openstack-cinder | 01:10 | |
*** Yogi1 has quit IRC | 01:12 | |
*** casusbel_ has joined #openstack-cinder | 01:12 | |
*** sghanekar_ has joined #openstack-cinder | 01:13 | |
*** raunak has joined #openstack-cinder | 01:15 | |
*** casusbelli has quit IRC | 01:15 | |
*** sghanekar has quit IRC | 01:15 | |
*** zhonghua-lee has joined #openstack-cinder | 01:15 | |
*** topshare has quit IRC | 01:18 | |
*** davechen has joined #openstack-cinder | 01:18 | |
*** vilobhmm has quit IRC | 01:20 | |
*** jwang_ has quit IRC | 01:21 | |
*** heyun has quit IRC | 01:22 | |
*** topshare has joined #openstack-cinder | 01:23 | |
*** changbl has joined #openstack-cinder | 01:24 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 01:26 |
*** lixiaoy11 has joined #openstack-cinder | 01:26 | |
tbarron | jgriffith: can you point me to logs? Are you saying that something is not mocked out that should be? | 01:26 |
*** zhonghua-lee has quit IRC | 01:27 | |
tbarron | jgriffith: or possibly that we have some issues that are exposed with the new version of mock? | 01:27 |
*** lixiaoy1 has quit IRC | 01:28 | |
*** primechuck has quit IRC | 01:28 | |
tbarron | jgriffith: I'm running unit tests all day and am not hitting those issues myself, so maybe some diff in environment to chase down. | 01:28 |
*** dims_ has joined #openstack-cinder | 01:28 | |
*** zhonghua-lee has joined #openstack-cinder | 01:30 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 01:30 |
*** kjelly has joined #openstack-cinder | 01:30 | |
*** lixiaoy1 has joined #openstack-cinder | 01:31 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 01:31 |
*** lixiaoy11 has quit IRC | 01:31 | |
tbarron | jgriffith: on another subject, do you intend to pursue https://review.openstack.org/#/c/191983/ further? | 01:33 |
tbarron | jgriffith: I also find those migration logging messages very annonying and I don't think there's a whole | 01:33 |
tbarron | jgriffith: lot more to do to get rid of them. | 01:34 |
tbarron | jgriffith: the -1s on that review are not against the initiative, they just point out a few more details to be covered before merge IMO | 01:34 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 01:34 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/201821 | 01:36 |
openstackgerrit | Kuo-tung Kao proposed openstack/python-cinderclient: Add more details for replication https://review.openstack.org/196562 | 01:37 |
*** raunak has quit IRC | 01:37 | |
*** Apoorva has joined #openstack-cinder | 01:37 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 01:38 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 01:43 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 01:45 |
openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Add Fujitsu ETERNUS DX Volume Driver (again) https://review.openstack.org/201500 | 01:52 |
*** boris-42 has quit IRC | 01:52 | |
*** markvoelker has quit IRC | 01:54 | |
*** markvoelker has joined #openstack-cinder | 02:00 | |
*** heyun has joined #openstack-cinder | 02:08 | |
*** zhenguo has joined #openstack-cinder | 02:09 | |
*** patrickeast_ has quit IRC | 02:10 | |
*** coolsvap|away has quit IRC | 02:10 | |
*** zhonghua-lee has quit IRC | 02:11 | |
*** coolsvap|away has joined #openstack-cinder | 02:12 | |
*** david-lyle has joined #openstack-cinder | 02:12 | |
*** sghanekar_ has quit IRC | 02:14 | |
openstackgerrit | LisaLi proposed openstack/python-cinderclient: Show backup and volume info in backup_restore https://review.openstack.org/199846 | 02:15 |
*** dims_ has quit IRC | 02:17 | |
*** markvoelker has quit IRC | 02:18 | |
anteaya | thingee: I will ignore thanks | 02:18 |
*** mutoulbj has joined #openstack-cinder | 02:22 | |
*** david-lyle has quit IRC | 02:35 | |
*** annashen has joined #openstack-cinder | 02:39 | |
*** davechen has quit IRC | 02:43 | |
*** coolsvap has quit IRC | 02:45 | |
*** davechen has joined #openstack-cinder | 02:46 | |
*** annashen has quit IRC | 02:48 | |
*** hakimo has joined #openstack-cinder | 02:52 | |
*** patrickeast has joined #openstack-cinder | 02:53 | |
*** hakimo_ has quit IRC | 02:54 | |
*** jungleboyj has joined #openstack-cinder | 02:55 | |
*** annashen has joined #openstack-cinder | 02:58 | |
*** davechen1 has joined #openstack-cinder | 03:07 | |
*** haomaiwang has quit IRC | 03:09 | |
*** haomaiwa_ has joined #openstack-cinder | 03:09 | |
*** davechen has quit IRC | 03:10 | |
*** dims_ has joined #openstack-cinder | 03:19 | |
jgriffith | tbarron: FYI, I wasn't referring to that patch, I made those statements prior to submitting that patch | 03:21 |
jgriffith | tbarron: what I was saying that on local master I run to -epy27 in a loop of 10, and get random failures on NetApp tests | 03:22 |
jgriffith | we can chat tomorrow | 03:22 |
jgriffith | tbarron: and yes, I absolutely intend to follow up with that patch to remove the logging | 03:23 |
*** dims_ has quit IRC | 03:23 | |
*** zhonghua-lee has joined #openstack-cinder | 03:25 | |
*** gouthamr has quit IRC | 03:28 | |
*** topshare has quit IRC | 03:28 | |
*** topshare has joined #openstack-cinder | 03:30 | |
*** markvoelker has joined #openstack-cinder | 03:35 | |
*** david-lyle has joined #openstack-cinder | 03:38 | |
*** markvoelker_ has joined #openstack-cinder | 03:38 | |
*** markvoelker has quit IRC | 03:40 | |
*** _cjones_ has joined #openstack-cinder | 03:41 | |
*** _cjones_ has quit IRC | 03:41 | |
*** Lee1092 has joined #openstack-cinder | 03:45 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Add unit tests for cinder.api.v2.volumes https://review.openstack.org/198637 | 03:50 |
*** heyun has quit IRC | 03:55 | |
*** breitz has quit IRC | 03:56 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Raise BadRequest for invalid replication status https://review.openstack.org/198640 | 03:56 |
*** breitz has joined #openstack-cinder | 03:56 | |
*** diogogmt has quit IRC | 03:56 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 03:59 |
*** markvoelker has joined #openstack-cinder | 04:04 | |
*** Apoorva has quit IRC | 04:04 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 04:07 |
*** markvoelker_ has quit IRC | 04:07 | |
*** markvoelker_ has joined #openstack-cinder | 04:08 | |
*** dannywilson has joined #openstack-cinder | 04:08 | |
*** dannywilson has quit IRC | 04:08 | |
*** dannywilson has joined #openstack-cinder | 04:09 | |
*** dannywilson has quit IRC | 04:09 | |
*** markvoelker has quit IRC | 04:09 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 04:10 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 04:14 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add multiple pools support for Huawei driver https://review.openstack.org/188251 | 04:17 |
*** wanghao has quit IRC | 04:17 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver https://review.openstack.org/188732 | 04:20 |
*** coolsvap|away is now known as coolsvap | 04:20 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 04:23 |
*** dannywilson has joined #openstack-cinder | 04:24 | |
*** topshare has quit IRC | 04:26 | |
*** zhonghua-lee has quit IRC | 04:27 | |
*** fifieldt has joined #openstack-cinder | 04:28 | |
*** annashen has quit IRC | 04:29 | |
*** topshare has joined #openstack-cinder | 04:31 | |
*** harlowja_at_home has joined #openstack-cinder | 04:31 | |
*** shyama has joined #openstack-cinder | 04:32 | |
*** danieru has quit IRC | 04:35 | |
*** danieru has joined #openstack-cinder | 04:36 | |
*** pradipta has joined #openstack-cinder | 04:37 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 04:37 |
*** zhonghua-lee has joined #openstack-cinder | 04:37 | |
*** boris-42 has joined #openstack-cinder | 04:43 | |
*** Apoorva has joined #openstack-cinder | 04:45 | |
*** patrickeast has quit IRC | 04:51 | |
openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver https://review.openstack.org/162927 | 05:03 |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 05:06 |
*** markvoelker has joined #openstack-cinder | 05:06 | |
*** markvoelker_ has quit IRC | 05:06 | |
*** markvoelker_ has joined #openstack-cinder | 05:08 | |
*** markvoelker has quit IRC | 05:11 | |
*** annashen has joined #openstack-cinder | 05:14 | |
*** nikeshm has quit IRC | 05:17 | |
*** davechen1 is now known as davechen | 05:18 | |
*** xyang1 has quit IRC | 05:20 | |
*** jungler has joined #openstack-cinder | 05:21 | |
*** vilobhmm has joined #openstack-cinder | 05:26 | |
*** vilobhmm1 has joined #openstack-cinder | 05:27 | |
*** Longgeek has joined #openstack-cinder | 05:28 | |
*** vilobhmm has quit IRC | 05:30 | |
*** harlowja_at_home has quit IRC | 05:30 | |
*** mutoulbj_ has joined #openstack-cinder | 05:31 | |
*** laughterwym_ has joined #openstack-cinder | 05:32 | |
*** ByPasS has joined #openstack-cinder | 05:33 | |
*** mutoulb__ has joined #openstack-cinder | 05:34 | |
*** mutoulbj has quit IRC | 05:34 | |
*** laughterwym has quit IRC | 05:35 | |
*** zhonghua-li has joined #openstack-cinder | 05:36 | |
*** pradipta has quit IRC | 05:37 | |
*** mutoulbj_ has quit IRC | 05:37 | |
*** zhonghua-lee has quit IRC | 05:38 | |
*** topshare has quit IRC | 05:48 | |
*** annashen has quit IRC | 05:48 | |
*** topshare has joined #openstack-cinder | 05:51 | |
*** rhefner has quit IRC | 05:51 | |
openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 05:52 |
*** ankit has joined #openstack-cinder | 05:59 | |
*** vilobhmm1 has quit IRC | 06:06 | |
*** IlyaG has quit IRC | 06:08 | |
*** deepakcs has joined #openstack-cinder | 06:09 | |
*** Apoorva has quit IRC | 06:10 | |
*** annashen has joined #openstack-cinder | 06:10 | |
*** BharatK has joined #openstack-cinder | 06:12 | |
*** belmoreira has joined #openstack-cinder | 06:21 | |
openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO Volume driver Consistency Groups support https://review.openstack.org/189135 | 06:23 |
*** primechuck has joined #openstack-cinder | 06:24 | |
*** tobe has joined #openstack-cinder | 06:28 | |
*** topshare has quit IRC | 06:29 | |
*** afazekas has joined #openstack-cinder | 06:30 | |
*** Apoorva has joined #openstack-cinder | 06:31 | |
*** topshare has joined #openstack-cinder | 06:33 | |
*** spradeepv has joined #openstack-cinder | 06:34 | |
*** nkrinner_afk is now known as nkrinner | 06:35 | |
spradeepv | Anybody facing this issue https://ask.openstack.org/en/question/78501/unable-to-run-tox-tests-due-to-error-with-pip-version/ | 06:37 |
lifeless | upgrade your virtualenv | 06:40 |
spradeepv | lifeless: How do I do it? I removed .tox folder and tried to run the command again, but it was of no use... | 06:43 |
*** wanghao has joined #openstack-cinder | 06:44 | |
*** annashen has quit IRC | 06:46 | |
spradeepv | removed .venv folder and ran ./run_tests.sh and it worked. Thanks lifeless!!! | 06:47 |
*** Miouge has quit IRC | 06:49 | |
*** Miouge has joined #openstack-cinder | 06:50 | |
*** Apoorva has quit IRC | 06:50 | |
*** lpetrut has joined #openstack-cinder | 06:51 | |
*** lpetrut1 has joined #openstack-cinder | 06:54 | |
*** abehl has joined #openstack-cinder | 06:54 | |
*** vilobhmm has joined #openstack-cinder | 06:55 | |
*** lpetrut has quit IRC | 06:58 | |
*** markus_z has joined #openstack-cinder | 06:59 | |
*** dannywilson has quit IRC | 07:05 | |
*** lpetrut1 has quit IRC | 07:05 | |
*** topshare_ has joined #openstack-cinder | 07:07 | |
*** topshare has quit IRC | 07:09 | |
*** laughterwym_ has quit IRC | 07:10 | |
*** mutoulb__ has quit IRC | 07:10 | |
*** mutoulbj has joined #openstack-cinder | 07:11 | |
openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 07:14 |
*** annashen has joined #openstack-cinder | 07:15 | |
*** markus_z has quit IRC | 07:17 | |
*** et10man has quit IRC | 07:17 | |
*** dims_ has joined #openstack-cinder | 07:20 | |
*** primechuck has quit IRC | 07:22 | |
*** zhonghua-li has quit IRC | 07:23 | |
*** Longgeek has quit IRC | 07:23 | |
*** zigo has quit IRC | 07:24 | |
*** zigo has joined #openstack-cinder | 07:25 | |
*** dims_ has quit IRC | 07:26 | |
*** annashen has quit IRC | 07:26 | |
*** vilobhmm has quit IRC | 07:28 | |
openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 07:28 |
*** topshare_ has quit IRC | 07:28 | |
*** aix has joined #openstack-cinder | 07:31 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 07:32 |
*** topshare has joined #openstack-cinder | 07:37 | |
*** e0ne has joined #openstack-cinder | 07:39 | |
*** Miouge has quit IRC | 07:40 | |
*** zhonghua-li has joined #openstack-cinder | 07:47 | |
*** topshare has quit IRC | 07:50 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add smartX support for Huawei volume driver https://review.openstack.org/188732 | 07:51 |
*** lpetrut has joined #openstack-cinder | 07:52 | |
*** ankit_ag has joined #openstack-cinder | 07:55 | |
lifeless | spradeepv: your virtualenv package | 07:55 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 07:55 |
*** topshare has joined #openstack-cinder | 07:56 | |
*** bluex-pl has joined #openstack-cinder | 07:57 | |
*** lpetrut has quit IRC | 07:57 | |
*** ankit has quit IRC | 07:58 | |
*** bluex-pl has quit IRC | 07:58 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 07:58 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:58 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 07:58 |
*** bluex-pl has joined #openstack-cinder | 07:58 | |
*** ronis has joined #openstack-cinder | 08:00 | |
*** Longgeek has joined #openstack-cinder | 08:00 | |
*** leseb has quit IRC | 08:04 | |
*** leseb has joined #openstack-cinder | 08:07 | |
*** lpetrut has joined #openstack-cinder | 08:09 | |
openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: ProtecTIER backup driver implementation https://review.openstack.org/153969 | 08:12 |
*** jistr has joined #openstack-cinder | 08:12 | |
*** boris-42 has quit IRC | 08:12 | |
*** ociuhandu has joined #openstack-cinder | 08:16 | |
*** e0ne is now known as e0ne_ | 08:16 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware:Replace vCenter calls with oslo.vmware calls https://review.openstack.org/201981 | 08:21 |
*** markvoelker_ has quit IRC | 08:21 | |
*** e0ne_ is now known as e0ne | 08:22 | |
*** ndipanov has joined #openstack-cinder | 08:24 | |
*** lpetrut1 has joined #openstack-cinder | 08:24 | |
*** lpetrut has quit IRC | 08:24 | |
openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder: ProtecTIER backup driver implementation https://review.openstack.org/153969 | 08:25 |
*** zhongjun has joined #openstack-cinder | 08:25 | |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 08:25 |
*** liuxg has joined #openstack-cinder | 08:25 | |
liuxg | ping zhongjun | 08:25 |
*** liuxg has left #openstack-cinder | 08:27 | |
*** chenying has joined #openstack-cinder | 08:27 | |
*** liuxg has joined #openstack-cinder | 08:27 | |
*** haypo has joined #openstack-cinder | 08:29 | |
*** topshare has quit IRC | 08:30 | |
*** jdandrea has quit IRC | 08:32 | |
*** yrabl has quit IRC | 08:32 | |
*** yrabl has joined #openstack-cinder | 08:33 | |
*** jordanP has joined #openstack-cinder | 08:34 | |
*** topshare has joined #openstack-cinder | 08:34 | |
*** markvoelker has joined #openstack-cinder | 08:36 | |
openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 08:40 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 08:41 |
haypo | win 7 | 08:42 |
*** vlaza has joined #openstack-cinder | 08:43 | |
*** ronis_ has joined #openstack-cinder | 08:44 | |
*** ronis has quit IRC | 08:45 | |
*** ociuhandu has quit IRC | 08:45 | |
*** markvoelker has quit IRC | 08:45 | |
*** dalgaaf has quit IRC | 08:48 | |
*** lixiaoy1 has quit IRC | 08:50 | |
*** markvoelker has joined #openstack-cinder | 08:51 | |
*** davechen has left #openstack-cinder | 08:55 | |
*** bluex-pl has quit IRC | 08:56 | |
*** markvoelker has quit IRC | 08:56 | |
*** markus_z has joined #openstack-cinder | 09:00 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Raise HTTP exception for backup not found https://review.openstack.org/201999 | 09:01 |
*** markvoelker has joined #openstack-cinder | 09:06 | |
*** markvoelker has quit IRC | 09:10 | |
*** dalgaaf has joined #openstack-cinder | 09:12 | |
*** bluex-pl has joined #openstack-cinder | 09:13 | |
*** e0ne is now known as e0ne_ | 09:15 | |
*** e0ne_ is now known as e0ne | 09:16 | |
*** liuxg has quit IRC | 09:17 | |
*** markvoelker has joined #openstack-cinder | 09:20 | |
*** e0ne has quit IRC | 09:22 | |
*** aarefiev has quit IRC | 09:22 | |
*** yuriy_n17 has quit IRC | 09:23 | |
*** markvoelker has quit IRC | 09:25 | |
*** markvoelker has joined #openstack-cinder | 09:35 | |
*** zhongjun has quit IRC | 09:38 | |
*** markvoelker has quit IRC | 09:40 | |
*** zhonghua-li has quit IRC | 09:41 | |
*** zhongjun has joined #openstack-cinder | 09:44 | |
*** liuxg has joined #openstack-cinder | 09:45 | |
BharatK | DuncanT, Ping | 09:45 |
BharatK | DuncanT, Need your blessings for the patches https://review.openstack.org/#/c/163647/ and https://review.openstack.org/#/c/182254/ | 09:45 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 09:46 |
*** heyun has joined #openstack-cinder | 09:46 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Add unit tests for cinder.api.v2.volumes https://review.openstack.org/198637 | 09:48 |
DuncanT | BharatK: looking | 09:49 |
*** markvoelker has joined #openstack-cinder | 09:49 | |
BharatK | DuncanT, Thank you very much. | 09:49 |
*** liuxg has quit IRC | 09:51 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: Raise BadRequest for invalid replication status https://review.openstack.org/198640 | 09:52 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add hypermetro support for huawei volume driver https://review.openstack.org/202023 | 09:53 |
*** tobe has quit IRC | 09:54 | |
*** markvoelker has quit IRC | 09:54 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 09:55 |
*** zhonghua-li has joined #openstack-cinder | 09:56 | |
*** dims_ has joined #openstack-cinder | 10:03 | |
*** markvoelker has joined #openstack-cinder | 10:04 | |
*** dims_ has quit IRC | 10:05 | |
openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 10:07 |
*** mutoulbj has quit IRC | 10:07 | |
*** markvoelker has quit IRC | 10:08 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Service object https://review.openstack.org/160417 | 10:11 |
*** vlaza has quit IRC | 10:11 | |
*** e0ne has joined #openstack-cinder | 10:12 | |
openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of querying cinder resource detail https://review.openstack.org/202029 | 10:12 |
*** chenying has quit IRC | 10:14 | |
*** tobe has joined #openstack-cinder | 10:17 | |
*** IanGovett has joined #openstack-cinder | 10:17 | |
BharatK | DuncanT, ping | 10:17 |
BharatK | DuncanT, Thanks for +2 | 10:17 |
*** markvoelker has joined #openstack-cinder | 10:19 | |
BharatK | DuncanT, One late thought, sgotliv mentioned that with the patch https://review.openstack.org/#/c/163647/ I am changing NFS backup configuration options. | 10:19 |
BharatK | DuncanT, I am moving NFS backup config options to Posix and changing the names also | 10:19 |
BharatK | DuncanT, Will that affect backward compatibility? | 10:20 |
DuncanT | BharatK: Yes. You can't do that | 10:20 |
DuncanT | o | 10:20 |
BharatK | BharatK, Oh okay, Then do I need to provide deprecated_name='<OLD_NFS_BACKUP_CONFIG_NAME>' | 10:22 |
DuncanT | Looking again, give me a minute | 10:22 |
BharatK | DuncanT, ^^ | 10:22 |
BharatK | DuncanT, okay | 10:22 |
DuncanT | You need to add deprecated option names, if you want to change them. I'm not convinced the new names are any better though | 10:23 |
*** markvoelker has quit IRC | 10:23 | |
BharatK | DuncanT, I will keep the old names then. | 10:24 |
DuncanT | Probably easier | 10:25 |
BharatK | DuncanT, I will modify and send another patch set. Thanks. | 10:27 |
*** vlaza has joined #openstack-cinder | 10:28 | |
*** heyun has quit IRC | 10:29 | |
*** e0ne is now known as e0ne_ | 10:32 | |
*** markvoelker has joined #openstack-cinder | 10:33 | |
openstackgerrit | Xi Yang proposed openstack/cinder: Provide snap copy feature in EMC VNX Cinder driver https://review.openstack.org/184733 | 10:34 |
*** markvoelker has quit IRC | 10:38 | |
*** zhongjun has quit IRC | 10:38 | |
*** mdenny has quit IRC | 10:39 | |
*** e0ne_ is now known as e0ne | 10:39 | |
*** kmartin has quit IRC | 10:40 | |
*** zhonghua-li has quit IRC | 10:40 | |
*** jamielennox is now known as jamielennox|away | 10:41 | |
*** coolsvap is now known as coolsvap|away | 10:41 | |
*** e0ne has quit IRC | 10:46 | |
*** markvoelker has joined #openstack-cinder | 10:48 | |
*** tobe has quit IRC | 10:48 | |
*** spradeepv has left #openstack-cinder | 10:50 | |
*** e0ne has joined #openstack-cinder | 10:50 | |
*** markvoelker has quit IRC | 10:54 | |
*** topshare has quit IRC | 10:54 | |
*** mutoulbj has joined #openstack-cinder | 10:54 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 10:54 |
*** e0ne has quit IRC | 11:00 | |
*** markvoelker has joined #openstack-cinder | 11:02 | |
*** e0ne has joined #openstack-cinder | 11:04 | |
*** afazekas has quit IRC | 11:05 | |
*** markvoelker has quit IRC | 11:07 | |
*** Yogi1 has joined #openstack-cinder | 11:09 | |
*** asselin has quit IRC | 11:09 | |
*** ociuhandu has joined #openstack-cinder | 11:10 | |
*** zhongjun has joined #openstack-cinder | 11:10 | |
*** e0ne has quit IRC | 11:13 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: Posix backup driver https://review.openstack.org/163647 | 11:16 |
*** markvoelker has joined #openstack-cinder | 11:17 | |
*** e0ne has joined #openstack-cinder | 11:18 | |
*** breitz has quit IRC | 11:18 | |
BharatK | DuncanT, Modified the patch as per your comments. Please review it. Thanks. https://review.openstack.org/#/c/163647/ | 11:19 |
openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 11:19 |
*** timcl has joined #openstack-cinder | 11:21 | |
*** breitz has joined #openstack-cinder | 11:21 | |
*** markvoelker has quit IRC | 11:21 | |
*** markvoelker has joined #openstack-cinder | 11:24 | |
*** liuxg has joined #openstack-cinder | 11:26 | |
*** openstack has joined #openstack-cinder | 11:37 | |
*** markvoelker has quit IRC | 11:37 | |
*** skraynev_ has joined #openstack-cinder | 11:37 | |
*** BharatK_ has joined #openstack-cinder | 11:37 | |
*** kambiz_ has joined #openstack-cinder | 11:37 | |
*** cburgess_ has joined #openstack-cinder | 11:37 | |
*** e0ne is now known as e0ne_ | 11:37 | |
*** Anticime1 has joined #openstack-cinder | 11:37 | |
*** Yogi1 has quit IRC | 11:37 | |
*** yrabl_ has joined #openstack-cinder | 11:37 | |
*** kambiz_ is now known as kambiz | 11:37 | |
*** BharatK_ has quit IRC | 11:37 | |
*** BharatK_ has joined #openstack-cinder | 11:37 | |
*** openstackstatus has joined #openstack-cinder | 11:37 | |
*** ChanServ sets mode: +v openstackstatus | 11:37 | |
*** ndipanov has joined #openstack-cinder | 11:38 | |
*** yrabl has quit IRC | 11:39 | |
*** markvoelker has joined #openstack-cinder | 11:39 | |
*** e0ne_ is now known as e0ne | 11:39 | |
*** yrabl has joined #openstack-cinder | 11:40 | |
*** erlon has joined #openstack-cinder | 11:41 | |
*** ociuhandu_ has joined #openstack-cinder | 11:41 | |
*** ociuhandu has quit IRC | 11:41 | |
*** ociuhandu_ is now known as ociuhandu | 11:41 | |
*** yrabl has quit IRC | 11:44 | |
*** markvoelker has quit IRC | 11:44 | |
*** yrabl has joined #openstack-cinder | 11:44 | |
*** dims_ has joined #openstack-cinder | 11:47 | |
*** ganso_ has joined #openstack-cinder | 11:48 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add pagination to snapshots https://review.openstack.org/195071 | 11:49 |
*** pradipta has joined #openstack-cinder | 11:53 | |
*** markvoelker has joined #openstack-cinder | 11:54 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers https://review.openstack.org/197999 | 11:55 |
*** cdelatte has joined #openstack-cinder | 11:56 | |
*** delattec has joined #openstack-cinder | 11:56 | |
*** markvoelker has quit IRC | 11:58 | |
*** zhongjun has quit IRC | 11:59 | |
*** topshare has joined #openstack-cinder | 11:59 | |
*** topshare has quit IRC | 12:00 | |
*** nikeshm has joined #openstack-cinder | 12:01 | |
tbarron | BharatK_: https://review.openstack.org/163647 looks good to my eye, but my eye is fallible | 12:04 |
tbarron | BharatK_: let me pull down the patch today and run some nfs backup/restores, etc. just to make sure. | 12:05 |
BharatK_ | tbarron, Sure. Thank you very much. | 12:05 |
tbarron | BharatK_: given that I missed the compat issue that DuncanT caught ... | 12:05 |
BharatK_ | tbarron, I tested it locally with GlusterFS. | 12:06 |
BharatK_ | tbarron, That should be fine. | 12:06 |
BharatK_ | tbarron, thanks. | 12:06 |
tbarron | BharatK_: good. I will use my existing nfs backup config, etc. | 12:06 |
BharatK_ | BharatK_, Sure. | 12:06 |
DuncanT | tbarron: Actually I missed it, sgotliv caught it | 12:06 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add more Rally scenarios to run https://review.openstack.org/194180 | 12:07 |
tbarron | sgotliv: TY! | 12:07 |
sgotliv | DuncanT, tbarron always glad to help guys | 12:07 |
BharatK_ | sgordon, DuncanT tbarron Thanks everyone. | 12:08 |
*** markvoelker has joined #openstack-cinder | 12:08 | |
*** deepakcs has quit IRC | 12:10 | |
*** gouthamr has joined #openstack-cinder | 12:12 | |
*** topshare has joined #openstack-cinder | 12:12 | |
*** gouthamr has quit IRC | 12:12 | |
*** topshare has quit IRC | 12:12 | |
*** markvoelker has quit IRC | 12:13 | |
*** topshare has joined #openstack-cinder | 12:13 | |
*** crose has joined #openstack-cinder | 12:13 | |
openstackgerrit | Himanshu Dwivedi proposed openstack/python-cinderclient: Adhere to Coding Standards https://review.openstack.org/202069 | 12:13 |
*** tobe has joined #openstack-cinder | 12:15 | |
*** bswartz has quit IRC | 12:17 | |
*** tobe has quit IRC | 12:18 | |
*** tobe has joined #openstack-cinder | 12:19 | |
*** markvoelker has joined #openstack-cinder | 12:23 | |
*** topshare has quit IRC | 12:24 | |
*** tobe has quit IRC | 12:24 | |
*** markvoelker has quit IRC | 12:28 | |
*** lpabon has joined #openstack-cinder | 12:30 | |
*** edmondsw has joined #openstack-cinder | 12:35 | |
*** markvoelker has joined #openstack-cinder | 12:37 | |
*** jordanP has quit IRC | 12:40 | |
*** jordanP has joined #openstack-cinder | 12:40 | |
*** dims_ has quit IRC | 12:41 | |
*** bnemec has joined #openstack-cinder | 12:41 | |
*** markvoelker has quit IRC | 12:42 | |
*** akerr has joined #openstack-cinder | 12:42 | |
*** liuxg has quit IRC | 12:44 | |
*** afazekas has joined #openstack-cinder | 12:44 | |
*** topshare has joined #openstack-cinder | 12:44 | |
*** simondodsley has joined #openstack-cinder | 12:45 | |
*** jordanP has quit IRC | 12:48 | |
*** jordanP has joined #openstack-cinder | 12:48 | |
*** markvoelker has joined #openstack-cinder | 12:52 | |
*** jungleboyj has quit IRC | 12:52 | |
*** eharney has joined #openstack-cinder | 12:55 | |
*** markvoelker has quit IRC | 12:56 | |
*** smoriya has quit IRC | 12:58 | |
*** bswartz has joined #openstack-cinder | 12:58 | |
*** jaypipes has joined #openstack-cinder | 12:59 | |
*** julim has joined #openstack-cinder | 13:00 | |
*** markvoelker has joined #openstack-cinder | 13:01 | |
openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 13:02 |
*** Yogi1 has joined #openstack-cinder | 13:04 | |
*** dims_ has joined #openstack-cinder | 13:06 | |
*** gouthamr has joined #openstack-cinder | 13:09 | |
*** markvoelker has quit IRC | 13:09 | |
*** markvoelker has joined #openstack-cinder | 13:09 | |
*** markvoelker_ has joined #openstack-cinder | 13:10 | |
*** thangp has joined #openstack-cinder | 13:11 | |
*** markvoelker has quit IRC | 13:14 | |
*** akshai has joined #openstack-cinder | 13:15 | |
*** krtaylor has joined #openstack-cinder | 13:16 | |
*** primechuck has joined #openstack-cinder | 13:22 | |
*** topshare has quit IRC | 13:25 | |
*** dustins has joined #openstack-cinder | 13:27 | |
*** primechuck has quit IRC | 13:30 | |
*** timcl has quit IRC | 13:31 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 13:33 |
*** mriedem_away is now known as mriedem | 13:34 | |
openstackgerrit | Thang Pham proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 13:36 |
openstackgerrit | Thang Pham proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 13:36 |
*** diogogmt has joined #openstack-cinder | 13:36 | |
*** sgotliv has quit IRC | 13:37 | |
*** sgotliv has joined #openstack-cinder | 13:37 | |
*** jdandrea has joined #openstack-cinder | 13:37 | |
*** BharatK_ has quit IRC | 13:41 | |
*** jistr is now known as jistr|mtg | 13:49 | |
*** ankit_ag has quit IRC | 13:52 | |
*** jordanP has quit IRC | 13:58 | |
*** rhefner has joined #openstack-cinder | 13:59 | |
jgriffith | tbarron: hey ya | 14:03 |
*** Apoorva has joined #openstack-cinder | 14:03 | |
jgriffith | tbarron: I'll see if I can reproduce the mock things I was talking about yesterday... FWIW, I saw it in other places as well... Gluster | 14:04 |
*** diogogmt has quit IRC | 14:04 | |
jgriffith | tbarron: it very well could be worth me just loading a fresh env and starting from scratch | 14:04 |
jgriffith | tbarron: since it seems to be limited to me :) | 14:04 |
*** annegentle has joined #openstack-cinder | 14:06 | |
*** liewegas has quit IRC | 14:10 | |
*** liewegas has joined #openstack-cinder | 14:11 | |
tbarron | jgriffith: just got out of my daily scrum :-) | 14:11 |
jgriffith | tbarron: good times :) | 14:11 |
tbarron | jgriffith: I'd like to get a copy of your script and environment (OSX, linux, whatever). | 14:12 |
tbarron | jgriffith: I think there are random failures with high concurrency and would expect them to show | 14:12 |
tbarron | jgriffith: in our UTs as well as elsewhere. | 14:12 |
jgriffith | tbarron: hehe... my *script* is the cmd line: "for i in {1..100}; do to -epy27; done" | 14:13 |
e0ne | good script:) | 14:13 |
tbarron | jgriffith: given what you say about gluster mebbe remotefs derivative fs-es hit it | 14:13 |
*** Yogi1 has quit IRC | 14:13 | |
tbarron | jgriffith: that reads a little easier than devstack | 14:13 |
jgriffith | tbarron: pip stdout and stderr though if you ever want to go back and look at anything ;) | 14:13 |
*** primechuck has joined #openstack-cinder | 14:14 | |
tbarron | jgriffith: are you running on your macbook with lots of cores? | 14:14 |
tbarron | jgriffith: concurrency | 14:14 |
jgriffith | tbarron: so yesterday I was running on my macbook, but it only has 2 cores :( | 14:14 |
tbarron | jgriffith: we need to get you a new macbook so you can do proper testing! | 14:15 |
jgriffith | tbarron: one thing I was thinking about... it's possible that my laptop slept and woke up and that may coincide with the failures | 14:15 |
tbarron | jgriffith: hmm, that might be it. | 14:15 |
tbarron | jgriffith: so if you hit the issue again, let's talk. in the mean time | 14:15 |
jgriffith | tbarron: It's AWESOME that tox can/will pause and resume like that, but who knows if it's really reliable | 14:15 |
jgriffith | tbarron: fair enough | 14:15 |
tbarron | I will use your fancy script myself and see what I see. | 14:15 |
jgriffith | tbarron: I just thought I should mention it incase anybody else has seen it, or if it starts showing up in the gate | 14:16 |
tbarron | jgriffith: definitely | 14:16 |
jgriffith | we've seen this movie before ;) | 14:16 |
tbarron | king kong vs godzilla | 14:16 |
jgriffith | hehe | 14:17 |
*** mtanino has joined #openstack-cinder | 14:20 | |
e0ne | jgriffith: hi. i think nobody except you will merge this https://review.openstack.org/#/c/192683/ | 14:23 |
*** topshare has joined #openstack-cinder | 14:25 | |
*** topshare has quit IRC | 14:25 | |
ociuhandu | thingee: hello, you have denied the reply we have sent to third-party-announce? Just got an email on that, but the reference you mention seems to relate to another cinder third-party ci | 14:25 |
*** xyang has joined #openstack-cinder | 14:26 | |
*** mdenny has joined #openstack-cinder | 14:27 | |
openstackgerrit | John Griffith proposed openstack/cinder: Make base driver sane again https://review.openstack.org/201812 | 14:27 |
*** markvoelker has joined #openstack-cinder | 14:28 | |
*** topshare has joined #openstack-cinder | 14:30 | |
*** topshare has quit IRC | 14:31 | |
*** markvoel_ has joined #openstack-cinder | 14:31 | |
*** markvoelker has quit IRC | 14:31 | |
*** abehl has quit IRC | 14:31 | |
*** topshare has joined #openstack-cinder | 14:31 | |
*** markvoelker_ has quit IRC | 14:32 | |
*** boichev has quit IRC | 14:32 | |
*** timcl has joined #openstack-cinder | 14:33 | |
*** jistr|mtg is now known as jistr | 14:33 | |
*** markvoel_ has quit IRC | 14:33 | |
e0ne | jgriffith: great comment about removing ABC. i'm ready to +2 only for it:) | 14:37 |
*** Apoorva has quit IRC | 14:39 | |
jgriffith | e0ne: thanks | 14:40 |
*** rushil has joined #openstack-cinder | 14:40 | |
e0ne | jgriffith: one minor typo only | 14:40 |
guitarzan | the minus lines > the plus lines | 14:40 |
guitarzan | +1 :) | 14:40 |
jgriffith | e0ne: my patches are guaranteed to have plenty of typos :) | 14:40 |
*** agarciam1 has joined #openstack-cinder | 14:40 | |
*** agarciam has quit IRC | 14:40 | |
e0ne | jgriffith: :) | 14:41 |
jgriffith | e0ne: I'll fix it up | 14:41 |
e0ne | jgriffith: thanks | 14:41 |
*** mutoulbj has quit IRC | 14:45 | |
openstackgerrit | John Griffith proposed openstack/cinder: Make base driver sane again https://review.openstack.org/201812 | 14:46 |
jgriffith | e0ne: See.. I found a second one as well :) | 14:46 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 14:46 |
e0ne | jgriffith: before or after new patch was created?:) | 14:46 |
*** mutoulbj has joined #openstack-cinder | 14:47 | |
jgriffith | guitarzan: dang... what have I done, that's certainly not going to help me out when the code contribution counters get started in October :( | 14:47 |
jgriffith | e0ne: HA!! Before, but I make no guarantees that there aren't others waiting | 14:48 |
*** vlaza has quit IRC | 14:49 | |
*** jasondotstar has joined #openstack-cinder | 14:49 | |
smcginnis | There should be special recognition for those that have _removed_ the most lines of code. | 14:49 |
jgriffith | smcginnis: remember what happened when somebody suggest special recognition for -1 :) | 14:49 |
jgriffith | smcginnis: the road to hell is paved with good intentions | 14:49 |
*** ociuhandu_ has joined #openstack-cinder | 14:49 | |
smcginnis | :) | 14:50 |
*** tsekiyama has joined #openstack-cinder | 14:50 | |
*** markvoelker has joined #openstack-cinder | 14:53 | |
*** ociuhandu_ has quit IRC | 14:54 | |
*** markvoelker_ has joined #openstack-cinder | 14:54 | |
*** topshare has quit IRC | 14:56 | |
openstackgerrit | John Griffith proposed openstack/cinder: Handle volume not found on zfssa volume delete https://review.openstack.org/199330 | 14:57 |
*** markvoelker has quit IRC | 14:58 | |
jgriffith | geguileo: ^^ THANK YOU for being reasonable and not doing a -1 for an apostrophe or minor typos | 14:58 |
jgriffith | geguileo: I appreciate rationale feedback | 14:58 |
*** jungleboyj has joined #openstack-cinder | 14:59 | |
geguileo | jgriffith: np XD | 14:59 |
*** topshare has joined #openstack-cinder | 14:59 | |
*** laughterwym has joined #openstack-cinder | 15:04 | |
*** ronis_ has quit IRC | 15:07 | |
*** diogogmt has joined #openstack-cinder | 15:07 | |
mtanino | jungleboyj: Hi, could you see my comment inline? https://review.openstack.org/#/c/200237/ Thanks, | 15:07 |
thingee | e0ne: any movement with v1 removal in rally? | 15:13 |
*** vincent_hou has joined #openstack-cinder | 15:13 | |
*** markus_z has quit IRC | 15:13 | |
vincent_hou | jbernard: Hi. | 15:13 |
jungleboyj | mtanino: Ok, Looking. | 15:14 |
mtanino | jungleboyj: thank you :) | 15:14 |
openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Continue conversion to volume object https://review.openstack.org/201404 | 15:14 |
thingee | scottda: going to ask for an update with the nova <-> cinder api stuff in meeting | 15:14 |
*** Yogi1 has joined #openstack-cinder | 15:14 | |
e0ne | thingee: actually, is't not about v1 removing. it's about suppporting v2 | 15:14 |
e0ne | thingee: boris-42 is working on it. he promised to implement it asap | 15:15 |
thingee | e0ne: and heat? | 15:15 |
*** jordanP has joined #openstack-cinder | 15:16 | |
e0ne | thingee: is should be easy. only constant change. will test it | 15:17 |
*** kmartin has joined #openstack-cinder | 15:17 | |
scottda | thingee: OK, cool | 15:19 |
*** topshare has quit IRC | 15:19 | |
*** topshare has joined #openstack-cinder | 15:20 | |
scottda | thingee: I might be cell-phone only around 1/2 past meeting hour (i.e. 9:30 PDT) so please ask early or later in the hour if you can. | 15:20 |
*** dannywilson has joined #openstack-cinder | 15:22 | |
*** zhenguo has quit IRC | 15:23 | |
*** aix has quit IRC | 15:23 | |
*** dannywilson has quit IRC | 15:24 | |
openstackgerrit | Edwin Wang proposed openstack/cinder: Report capability of multiattach for FlashSystem https://review.openstack.org/199818 | 15:24 |
*** dannywilson has joined #openstack-cinder | 15:24 | |
j_king | anyone remember how to get the cinder service tenant id? | 15:24 |
j_king | working on the trusts patch for backup. not even sure it's going to work since afaik the requesting user of the trusts create endpoint has to be the same as the trustor parameter. | 15:26 |
*** BharatK has joined #openstack-cinder | 15:27 | |
*** jdurgin1 has joined #openstack-cinder | 15:30 | |
Swanson | More consistency groups. On create_volume if the add to a consistency group fails do I fail the volume create or just soldier on. | 15:34 |
Swanson | ? | 15:34 |
j_king | https://bugs.launchpad.net/cinder/+bug/1298135 it's for this silly bug that I keep getting 90% close to fixing and then... have a kid... but now I'm back (again) and want to get it closed up. | 15:34 |
openstack | Launchpad bug 1298135 in Cinder "Cinder should handle token expiration for long ops" [Medium,Confirmed] - Assigned to j_king (james-agentultra) | 15:34 |
*** garthb has joined #openstack-cinder | 15:35 | |
*** e0ne is now known as e0ne_ | 15:36 | |
*** timcl has quit IRC | 15:39 | |
*** hemnafk is now known as hemna | 15:39 | |
*** ig0r__ has quit IRC | 15:39 | |
openstackgerrit | Mathieu Gagné proposed openstack/cinder: Add volume_attachment to volume usage notifications https://review.openstack.org/201811 | 15:40 |
*** ig0r_ has joined #openstack-cinder | 15:40 | |
*** lcurtis has joined #openstack-cinder | 15:43 | |
*** e0ne_ is now known as e0ne | 15:45 | |
*** belmoreira has quit IRC | 15:48 | |
*** Yogi1 has quit IRC | 15:49 | |
*** dustins has quit IRC | 15:50 | |
*** sseverson has joined #openstack-cinder | 15:53 | |
winston-d_ | meeting? | 15:53 |
thingee | jgriffith: are there any replication issues we should try to sort in the meeting today? | 15:53 |
thingee | winston-d_: yes | 15:54 |
thingee | winston-d_: six more minutes | 15:54 |
*** laughterwym has quit IRC | 15:55 | |
*** timcl has joined #openstack-cinder | 15:56 | |
*** patrickeast has joined #openstack-cinder | 15:56 | |
*** hiro__ has joined #openstack-cinder | 15:57 | |
jgriffith | thingee: don't think so... I mean, really it comes down to other peoples objections or approvals | 15:57 |
*** adurbin_ has joined #openstack-cinder | 15:57 | |
thingee | jgriffith: ok, if it's things we can resolve in the meeting, might be worth discussing so you can continue on and not be blocked. | 15:58 |
*** jaypipes has quit IRC | 15:58 | |
jungleboyj | mtanino: There isn't a way to wrap that in () instead of having to use \ ? | 15:59 |
mtanino | jungleboyj: Actually, I'm not sure that way using () | 16:00 |
*** annashen has joined #openstack-cinder | 16:00 | |
*** logan2 has quit IRC | 16:00 | |
mtanino | jungleboyj: Are there reference in current unit tests? | 16:00 |
*** topshare has quit IRC | 16:01 | |
mtanino | jungleboyj: I saw gluster's unit test for a reference. | 16:01 |
jungleboyj | mtanino: Let me grab the patch and try it. Make sure I am not being a pain for no reason. | 16:01 |
*** diogogmt has quit IRC | 16:01 | |
*** diogogmt_ has joined #openstack-cinder | 16:01 | |
mtanino | jungleboyj: thank you for the help. | 16:01 |
jungleboyj | mtanino: Of course. | 16:01 |
*** salv-orl_ has joined #openstack-cinder | 16:01 | |
*** hiro__ has quit IRC | 16:04 | |
*** salv-orlando has quit IRC | 16:05 | |
*** earlephilhower_ has joined #openstack-cinder | 16:05 | |
*** earlephilhower has quit IRC | 16:06 | |
hemna | xyang, the EMC os-brick CI is still failing to even run | 16:06 |
hemna | xyang, http://storage.superna.net/openstack/scaleio/EMC-os-brick-CI/build-44/index.html | 16:06 |
*** leeantho has joined #openstack-cinder | 16:07 | |
*** jdurgin1 has quit IRC | 16:12 | |
*** Apoorva has joined #openstack-cinder | 16:12 | |
*** lpetrut1 has quit IRC | 16:13 | |
*** _cjones_ has joined #openstack-cinder | 16:14 | |
*** crose has quit IRC | 16:15 | |
*** krtaylor has quit IRC | 16:15 | |
*** vokt has joined #openstack-cinder | 16:16 | |
*** bluex-pl has quit IRC | 16:18 | |
openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework the Scality SOFS driver to use the RemoteFS class https://review.openstack.org/200610 | 16:18 |
*** asselin has joined #openstack-cinder | 16:18 | |
xyang | hemna: I'll check. | 16:18 |
*** lpetrut has joined #openstack-cinder | 16:19 | |
*** thangp has quit IRC | 16:20 | |
*** bswartz has quit IRC | 16:20 | |
*** simondodsley has quit IRC | 16:20 | |
*** Anticime1 has quit IRC | 16:20 | |
*** jordanP has quit IRC | 16:20 | |
*** Anticimex has joined #openstack-cinder | 16:20 | |
*** thangp has joined #openstack-cinder | 16:20 | |
*** fron has joined #openstack-cinder | 16:20 | |
*** bswartz has joined #openstack-cinder | 16:20 | |
mriedem | i have a question about cinder multi-attach and how virt drivers in nova handle that, | 16:21 |
mriedem | re: https://review.openstack.org/#/c/197192/ | 16:21 |
mriedem | apparently with ^ cinder multi-attach would be totally busted (not sure it would work today anyway) | 16:21 |
mriedem | since it moves the volume to a different datastore | 16:21 |
mriedem | moves it to wherever the instance resides | 16:22 |
*** fron has quit IRC | 16:22 | |
mriedem | if the volume was attached to an instance in a different ds before, that instance would now be busted | 16:22 |
mriedem | maybe the vmware driver in nova would just have to detect that somehow and fail the attach request? | 16:22 |
*** jgregor has joined #openstack-cinder | 16:25 | |
hemna | mriedem, so, it depends on a few things actually | 16:26 |
*** BharatK has quit IRC | 16:26 | |
hemna | some cinder drivers create a new export for every attachment they do, even to the same host (different vm on the same host) | 16:26 |
hemna | drivers that don't, might have problems | 16:26 |
hemna | afaik, most drivers in Cinder currently aren't reporting they support multi-attach | 16:27 |
*** afazekas has quit IRC | 16:27 | |
*** krtaylor has joined #openstack-cinder | 16:28 | |
guitarzan | there be dragons | 16:28 |
hemna | guitarzan, +1 | 16:29 |
guitarzan | I wish people would start plugging hard drives into two servers at once and see how it goes | 16:30 |
hemna | :) | 16:30 |
hemna | guitarzan, and make sure they use ext4. that'll learn em. | 16:31 |
hemna | just don't complain to me when the fs is corrupt. | 16:31 |
*** liviu__ has joined #openstack-cinder | 16:33 | |
*** liviu__ has quit IRC | 16:33 | |
*** haypo has left #openstack-cinder | 16:36 | |
*** ronis_ has joined #openstack-cinder | 16:37 | |
*** vilobhmm has joined #openstack-cinder | 16:38 | |
*** markvoelker_ has quit IRC | 16:40 | |
*** annashen has quit IRC | 16:42 | |
*** markvoelker has joined #openstack-cinder | 16:42 | |
*** lpetrut has quit IRC | 16:44 | |
*** ndipanov has quit IRC | 16:45 | |
*** annashen has joined #openstack-cinder | 16:46 | |
*** vilobhmm has quit IRC | 16:49 | |
*** Yogi1 has joined #openstack-cinder | 16:50 | |
*** jistr has quit IRC | 16:50 | |
openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 16:50 |
jgriffith | eharney: I'm somewhat offended by your statement that my commit message is "emotional diatribe", I don't feel that anything in my commit message is emotional. | 16:53 |
openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Continue conversion to volume object https://review.openstack.org/201404 | 16:53 |
jgriffith | eharney: I'm more than open to changing the commit message, but I don't appreciate the nasty comments | 16:54 |
jgriffith | eharney: if the commit message comes across that way, then my apologies, and I'll fix it | 16:54 |
flip214 | jgriffith: I guess you'd like the DRBD driver to implement the replication v2 methods, right? I'll try to find some time to look at the spec and the other review link. | 16:54 |
jgriffith | flip214: yes, that wold be great | 16:54 |
flip214 | wait, I'll fetch my dictionary | 16:54 |
flip214 | ah, | 16:56 |
flip214 | : an area of hilly land in the country | 16:56 |
flip214 | ;) | 16:56 |
*** akshai has quit IRC | 16:56 | |
eharney | i don't want to read commit messages full of "what put us in this mess in the first place, but regardless" ... "The only compelling argument... well, so?" ... etc. it's not informative and is just distracting | 16:57 |
*** jgregor has left #openstack-cinder | 16:58 | |
*** akshai has joined #openstack-cinder | 16:58 | |
*** sghanekar_ has joined #openstack-cinder | 16:58 | |
*** patrickeast has quit IRC | 16:59 | |
*** mutoulbj has quit IRC | 16:59 | |
*** bswartz has quit IRC | 16:59 | |
*** qwebirc20432 has joined #openstack-cinder | 17:01 | |
*** timcl1 has joined #openstack-cinder | 17:01 | |
*** Fdaisuke has joined #openstack-cinder | 17:01 | |
*** akerr has quit IRC | 17:02 | |
*** akerr has joined #openstack-cinder | 17:02 | |
*** vincent_hou_ has joined #openstack-cinder | 17:03 | |
jgriffith | eharney: Ok, I'll change that for you | 17:03 |
jgriffith | eharney: no need to be insulting | 17:03 |
*** hemna is now known as hemnafk | 17:04 | |
Fdaisuke | thingee: I was not sure if you read my comment or not. So reposting it here. | 17:04 |
eharney | framing it as "for me" is missing the point | 17:04 |
*** timcl has quit IRC | 17:05 | |
Fdaisuke | thingee: Thanks for the comments. We might sounded like we're not aware of all the announcements in the past, but we're :) Since our driver is not completely new one, we thought we could find a chance to discuss it for Liberty inclusion. | 17:05 |
jgriffith | eharney: Ok... I'll "change it for EVERYBODY" | 17:05 |
*** vincent_hou has quit IRC | 17:05 | |
jgriffith | eharney: is that better? | 17:05 |
*** vincent_hou_ is now known as vincent_hou | 17:05 | |
eharney | jgriffith: yes | 17:05 |
jgriffith | eharney: ok, give me a bit and I'll update the commit message and try to find a way to make it more useful for anybody viewing git log in the future | 17:06 |
jgriffith | eharney: I will likely have to ask for some help from you in terms of what to put in there... because IMO we typically put some description of why we're submitting the patch etc | 17:07 |
jgriffith | eharney: so I'll need some help figuring out how to make the wording more acceptable from a community perspective | 17:07 |
*** bswartz has joined #openstack-cinder | 17:08 | |
*** crose has joined #openstack-cinder | 17:09 | |
thingee | e0ne, jungleboyj, jgriffith, hemnafk https://review.openstack.org/#/c/201736/ | 17:09 |
*** logan2 has joined #openstack-cinder | 17:10 | |
jgriffith | thingee: I don't know what the plan is for numbering, or what our scheme is there... | 17:11 |
jgriffith | thingee: so i.e., who would it be 0.3.0 vs 0.2.2? | 17:11 |
thingee | jgriffith: there was a request from dhellmann in the release manager office | 17:11 |
jgriffith | thingee: not saying I think it should be one or the other, just asking so I *know* | 17:11 |
*** vincent_hou has quit IRC | 17:11 | |
e0ne | thingee, hemnafk: does os-brick really support py33? | 17:12 |
thingee | jgriffith: since we're introducing new feature/drivers, it's a minor I guess. | 17:12 |
e0ne | https://github.com/openstack/os-brick/blob/master/setup.cfg#L21 | 17:12 |
jgriffith | thingee: to use 0.3.0? Totally cool, just making sure I know :) | 17:12 |
jgriffith | thingee: good enough for me ;) | 17:12 |
jgriffith | thingee: thanks | 17:12 |
*** timcl1 has quit IRC | 17:12 | |
jgriffith | thingee: it's probably not an important question... I was just wondering :) | 17:13 |
*** ronis_ has quit IRC | 17:15 | |
Fdaisuke | thingee: We'll prepare our driver and CI for M. Please let us make sure one thing. Maybe it'd be too early to discuss, but can we assume the cut date for merging into M would be M-1? | 17:18 |
*** patrickeast has joined #openstack-cinder | 17:18 | |
openstackgerrit | John Griffith proposed openstack/cinder: Remove logging statements from migrations https://review.openstack.org/191983 | 17:20 |
openstackgerrit | John Griffith proposed openstack/cinder: Fail migration if fk drop fails https://review.openstack.org/202201 | 17:25 |
jgriffith | xyang: winston-d_ tbarron ^^ | 17:25 |
*** timcl has joined #openstack-cinder | 17:26 | |
jgriffith | xyang: winston-d tbarron I think it's more appropriate to log that as a separate bug and fix it independent of the logging changes... agree? | 17:26 |
tbarron | jgriffith: I think that makes sense. Thanks for moving this one along again. | 17:27 |
jgriffith | tbarron: cool | 17:29 |
*** mgarza_ has joined #openstack-cinder | 17:32 | |
jungleboyj | eharney: Which packages provides lioadm in RHEL 7? | 17:33 |
jgriffith | patrickeast: you still around? | 17:33 |
eharney | jungleboyj: well... "lioadm" doesn't actually exist. the target stuff is in the kernel, python-rtslib manages that. | 17:33 |
patrickeast | jgriffith: yep | 17:33 |
*** dustins has joined #openstack-cinder | 17:33 | |
jgriffith | patrickeast: hey... got a quick question for ya | 17:33 |
eharney | jungleboyj: targetcli is the user tool which is in a different package | 17:34 |
patrickeast | jgriffith: sure, whats up? | 17:34 |
jgriffith | patrickeast: on this: https://review.openstack.org/#/c/201812/4/cinder/volume/drivers/pure.py | 17:34 |
*** dustins has quit IRC | 17:34 | |
jungleboyj | eharney: So I need python-rtslib and targetcli installed. | 17:34 |
patrickeast | ah good timing, i just put a comment on there | 17:34 |
*** qwebirc20432 has left #openstack-cinder | 17:34 | |
jgriffith | patrickeast: I just realized the reason that failed and I had to add that... your just using SanDriver, not SanISCSI | 17:34 |
eharney | jungleboyj: you don't need targetcli for Cinder actually | 17:34 |
jgriffith | patrickeast: is that my mix up, or is that intentional? | 17:34 |
jgriffith | OH LOL | 17:34 |
patrickeast | jgriffith: uhhh let me check | 17:34 |
*** dustins has joined #openstack-cinder | 17:34 | |
*** jaypipes has joined #openstack-cinder | 17:35 | |
jungleboyj | eharney: Ok. | 17:35 |
patrickeast | jgriffith: i wanna say we should be using the iscsi one | 17:35 |
jungleboyj | Let me see here. Know anyone who has actually gotten devstack to work on RHEL 7? | 17:35 |
jgriffith | patrickeast: so if we change it to SanISCSI you just get those for free, otherwise it's using the FC version of things | 17:35 |
patrickeast | so the PureISCSIDriver right now inherits from san.SanISCSIDriver | 17:36 |
jgriffith | patrickeast: Ok, I'll modify it with the commit message update and see what you think (and most important what your CI thinks) | 17:36 |
jgriffith | patrickeast: Oh.. maybe I'm missing something... just a sec | 17:36 |
patrickeast | jgriffith: ohh so the base driver does the san.SanDriver | 17:36 |
jgriffith | patrickeast: correct, and that's where you were calling those iscsi methods | 17:37 |
patrickeast | jgriffith: i think becuase we use some config options from that | 17:37 |
jgriffith | patrickeast: oh.. wait... hmmm | 17:37 |
patrickeast | haha now im confused | 17:37 |
jgriffith | patrickeast: ok, hold on... I might have just made a mistake on something here | 17:37 |
jgriffith | Gimmie a minute :) | 17:37 |
*** hemnafk is now known as hemna | 17:38 | |
*** mtanino has quit IRC | 17:39 | |
jgriffith | patrickeast: so you're right... the inheritance on your side is all correct | 17:40 |
jgriffith | patrickeast: so I misread something or my config was crap or something... let me run the unit tests again | 17:40 |
* jgriffith wishes he wasn't running this on his little macbook now :( | 17:41 | |
*** pradipta has quit IRC | 17:43 | |
xyang | jgriffith: that makes sense to fix it separately | 17:43 |
*** sghanekar_ has quit IRC | 17:43 | |
jgriffith | hmm.. anybody else seen this: http://paste.openstack.org/show/378114/ | 17:44 |
patrickeast | yea i got that | 17:46 |
patrickeast | i had to update tox | 17:47 |
patrickeast | and it went away | 17:47 |
hemna | jgriffith, pip install -U pip | 17:47 |
jgriffith | http://paste.openstack.org/show/378115/ | 17:47 |
hemna | that fixed it for me | 17:47 |
jgriffith | Ahh... ok | 17:47 |
jgriffith | LOL | 17:47 |
jgriffith | I've got a 50/50 chance :) | 17:47 |
patrickeast | the problem for me was that the pip version tox was trying to use was too old | 17:47 |
patrickeast | on osx it is apparently different than the system pip | 17:47 |
jgriffith | patrickeast: ok.. perfect | 17:47 |
*** lpetrut has joined #openstack-cinder | 17:48 | |
jgriffith | patrickeast: hemna thanks! this is on Ubuntu, so I'll first try the pip upgrade | 17:48 |
jgriffith | hmm.. that's went terribly | 17:48 |
*** dalgaaf has quit IRC | 17:48 | |
jgriffith | http://paste.openstack.org/show/378126/ | 17:49 |
hemna | ew | 17:49 |
jgriffith | haha | 17:49 |
jgriffith | hemna: turns out easy enough | 17:50 |
jgriffith | apt-get remove python-pip | 17:50 |
*** salv-orl_ has quit IRC | 17:50 | |
lifeless | jgriffith: https://rbtcollins.wordpress.com/2015/04/28/dealing-with-deps-in-openstack/ ? | 17:50 |
jgriffith | actually... update should work. but then there's the whole use easy vs apt | 17:51 |
*** crose has quit IRC | 17:51 | |
jgriffith | lifeless: thanks.. reading now | 17:51 |
lifeless | jgriffith: wait no, never use easy. apt or pip sure, but never easy. | 17:51 |
jgriffith | lifeless: yeah... that's been my rule | 17:51 |
jgriffith | lifeless: sorry, was just throwing it out there as one of the 3 options | 17:52 |
lifeless | easy doesn't track installed files, can't uninstall sanely | 17:52 |
jgriffith | lifeless: the only one I try and never use for anything | 17:52 |
lifeless | not to mention not doing secure ssl, not doing proxies sanely | 17:52 |
jgriffith | lifeless: yes, I learned that the hard way a long time ago | 17:52 |
lifeless | :) | 17:52 |
jgriffith | lifeless: the tracking an uninstall issues | 17:52 |
*** mtanino has joined #openstack-cinder | 17:52 | |
lifeless | sorry, I tend to overreact to the name | 17:52 |
jgriffith | lifeless: I reimaged my workstation because of it | 17:52 |
jgriffith | lifeless: I think it's warranted :) | 17:52 |
jgriffith | lifeless: I think it was your or mordred long ago that had a post to ML like "easy-install is EVIL" or something along those lines | 17:53 |
lifeless | there's been a few such :) | 17:53 |
casusbel_ | Hi! If anybody has ever seen virt-aa-helper being unable to load an apparmor profile in devstack, i't be great if you take a look at https://bugs.launchpad.net/cinder/+bug/1473116 and maybe drop a line of your experience. Thanks! | 17:54 |
openstack | Launchpad bug 1473116 in Cinder "File permission error when running volume snapshot tests" [Undecided,New] - Assigned to Silvan Kaiser (2-silvan) | 17:54 |
nikeshm | hello everyone, to prevent spams, CI is not running for hpmsa, dothill and lenovo, i am working on it to run it multiple times on sandbox and then moved it to cinder before end of this week | 17:55 |
nikeshm | i will update the corresponding wiki page too | 17:56 |
*** akshai has quit IRC | 17:56 | |
mordred | jgriffith: uhoh. someone said my name | 17:56 |
mordred | jgriffith: oh yes - please don't use easy_install | 17:57 |
jgriffith | mordred: LOL.. nah, it was a good thing | 17:57 |
jgriffith | :) | 17:57 |
mordred | \o/ | 17:57 |
jgriffith | This time...... | 17:57 |
jgriffith | :) | 17:57 |
*** vilobhmm has joined #openstack-cinder | 17:58 | |
jgriffith | hemna: patrickeast FYI, I needed both to be updated.. so thanks to both of you! | 17:58 |
*** akshai has joined #openstack-cinder | 17:59 | |
*** mutoulbj has joined #openstack-cinder | 17:59 | |
hemna | sweet. :) | 18:03 |
jgriffith | patrickeast: xyang ok, now I remember this :) | 18:03 |
jgriffith | patrickeast: xyang so the deal was that the Pure driver was the only thing calling ISCSIDriver:_run_iscsiadm_bare | 18:03 |
*** shyama has quit IRC | 18:03 | |
jgriffith | patrickeast: xyang so I nuked it out of the base class and put it in the Pure driver, as it appears to be unique for them anyway | 18:04 |
jgriffith | patrickeast: xyang would you prefer it stay in ISCSIDriver? | 18:04 |
patrickeast | that makes sense | 18:04 |
xyang | jgriffith: I think it makes sense to stay in iscsidriver | 18:04 |
patrickeast | if we are the only ones using it we can keep it in the pure driver | 18:04 |
*** thangp has quit IRC | 18:04 | |
xyang | jgriffith: not are pure specific call | 18:04 |
patrickeast | but if anyone else needs it then having it higher up is better | 18:04 |
jgriffith | xyang: yeah, probably right, I'll put it back and somebody can change later if they want | 18:05 |
jgriffith | patrickeast: xyang Ok. Thanks! | 18:05 |
jgriffith | xyang: patrickeast I think that's the right answer for sure | 18:05 |
patrickeast | like i mentioned in the review, if the alternative iscsi portal stuff is merged in nova we don’t need that anymore at all | 18:05 |
*** nkrinner is now known as nkrinner_afk | 18:05 | |
*** annegentle has quit IRC | 18:06 | |
*** mutoulbj has quit IRC | 18:07 | |
*** thangp has joined #openstack-cinder | 18:15 | |
*** ociuhandu has quit IRC | 18:17 | |
*** dalgaaf has joined #openstack-cinder | 18:19 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use a single base class for the VolumeDriver https://review.openstack.org/201812 | 18:22 |
jgriffith | ^^ Hopefully that commit mess is more appropriate but still provides enough info to be useful | 18:22 |
jgriffith | my apologies to anyone who took issue with my previous commit message, and PLEASE feel free to ask me for details on my motivation and why I may or may not have changed my mind on some of the details regarding our approach on this topic | 18:23 |
*** boris-42 has joined #openstack-cinder | 18:23 | |
*** Yogi1 has quit IRC | 18:23 | |
e0ne | jgriffith: are you going to send mail about it to '[openstack-dev] [Cinder] Implementation of ABC MetaClasses' thread? | 18:24 |
jgriffith | e0ne: I guess I can... sure | 18:25 |
*** markstur has quit IRC | 18:25 | |
jgriffith | I already replied to that thread but happy to do so again, and perhaps that's the appropriate place for the other detail I had in the commit message | 18:25 |
e0ne | jgriffith: you started it. it's only my opinion, that it would be good to publish it there | 18:27 |
*** belmoreira has joined #openstack-cinder | 18:28 | |
*** lpetrut has quit IRC | 18:30 | |
*** annashen has quit IRC | 18:30 | |
jgriffith | e0ne: will do | 18:34 |
jgriffith | e0ne: thanks | 18:34 |
jgriffith | but not until after I eat lunch :)_ | 18:34 |
e0ne | jgriffith: thank you:) | 18:34 |
*** Lee1092 has quit IRC | 18:35 | |
*** annegentle has joined #openstack-cinder | 18:35 | |
*** belmoreira has quit IRC | 18:41 | |
*** annashen has joined #openstack-cinder | 18:42 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 18:50 |
anish | hemna: I'm being lazy about the tests, but I don't know if you got a chance to look at that ^ | 18:50 |
*** timcl has quit IRC | 18:51 | |
*** markvoelker has quit IRC | 18:58 | |
*** annasort has quit IRC | 18:58 | |
*** chenying has joined #openstack-cinder | 19:00 | |
*** zhonghua-li has joined #openstack-cinder | 19:00 | |
*** liuxg has joined #openstack-cinder | 19:01 | |
*** zhongjun has joined #openstack-cinder | 19:01 | |
xyang | hemna: which patch you were looking at? There was a network outage and everything failed. I can do a recheck if you tell me which patch you are looking at | 19:01 |
anteaya | mtanino: this is the cinder channel | 19:02 |
mtanino | anteaya: yeah | 19:02 |
hemna | xyang, https://review.openstack.org/#/c/201736/ | 19:02 |
anteaya | :) | 19:02 |
mtanino | anteaya: I joined Cinder meeting tooo long | 19:02 |
*** Yogi1 has joined #openstack-cinder | 19:03 | |
*** hemna is now known as hemnafood | 19:03 | |
anteaya | ah | 19:04 |
*** e0ne has quit IRC | 19:05 | |
*** crose has joined #openstack-cinder | 19:06 | |
*** annashen has quit IRC | 19:06 | |
openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Continue conversion to volume object https://review.openstack.org/201404 | 19:06 |
*** timcl has joined #openstack-cinder | 19:06 | |
*** Longgeek has quit IRC | 19:08 | |
*** mutoulbj has joined #openstack-cinder | 19:08 | |
*** thangp has quit IRC | 19:09 | |
*** agarciam1 has quit IRC | 19:10 | |
*** thangp has joined #openstack-cinder | 19:10 | |
*** fifieldt_ has joined #openstack-cinder | 19:10 | |
mriedem | wow, VMwareEsxVmdkDriver has been slated for removal since juno but is still in tree | 19:10 |
mriedem | is someone working on that? | 19:10 |
*** jungleboyj has quit IRC | 19:10 | |
*** mgarza_ has quit IRC | 19:11 | |
*** mgarza has joined #openstack-cinder | 19:11 | |
*** fifieldt has quit IRC | 19:13 | |
*** belmoreira has joined #openstack-cinder | 19:13 | |
*** hakimo has quit IRC | 19:13 | |
*** mutoulbj has quit IRC | 19:14 | |
*** hakimo has joined #openstack-cinder | 19:14 | |
thangp | mriedem: but VMwareVcVmdkDriver subclasses VMwareEsxVmdkDriver | 19:15 |
mriedem | thangp: sure | 19:15 |
mriedem | so what? | 19:15 |
thangp | mriedem: I do not believe VMwareEsxVmdkDriver, unless you rebase it or rename it | 19:15 |
mriedem | merge them and remove the esxi support | 19:15 |
thangp | mriedem: yup | 19:15 |
mriedem | we had the same thing with the virt drivers in nova | 19:15 |
mriedem | removed esxi back in juno | 19:15 |
thangp | mriedem: usually Vipin takes care of the vmware stuff | 19:16 |
thangp | mriedem: vbala(?) | 19:16 |
*** liuxg has quit IRC | 19:18 | |
*** zhongjun has left #openstack-cinder | 19:19 | |
*** timcl1 has joined #openstack-cinder | 19:21 | |
*** timcl has quit IRC | 19:21 | |
*** angela-s has joined #openstack-cinder | 19:25 | |
xyang | mriedem: hi, question for you | 19:25 |
*** jungleboyj has joined #openstack-cinder | 19:25 | |
*** bswartz has quit IRC | 19:25 | |
mriedem | xyang: ok | 19:26 |
*** zhonghua-li has quit IRC | 19:26 | |
xyang | mriedem: hemna's os-brick patch is being review and I assume the merge deadline is 7/28, L-2? | 19:26 |
mriedem | 7/30 | 19:26 |
xyang | mriedem: what is the merge deadline for patches dependent on his patch? | 19:26 |
xyang | mriedem: https://review.openstack.org/#/c/194454/ | 19:26 |
*** bswartz has joined #openstack-cinder | 19:26 | |
xyang | mriedem: the shim libvirt volume driver | 19:26 |
mriedem | xyang: didn't you have a nova blueprint for that? | 19:27 |
xyang | mriedem: I do, but that needs to be changed now | 19:27 |
xyang | mriedem: the blueprint has a lot of details that are not needed any more because os-brick has it | 19:28 |
mriedem | xyang: the blueprint or the spec? | 19:28 |
mriedem | i'm just talking about the blueprint in launchpad | 19:28 |
xyang | mriedem: oh, ok, I have a blueprint, let me find it | 19:28 |
mriedem | xyang: you should link your nova bp from lp into the nova change | 19:28 |
mriedem | so it's actually tracked | 19:28 |
mriedem | xyang: having said that, i don't see a problem with this getting into liberty | 19:28 |
xyang | https://blueprints.launchpad.net/nova/+spec/emc-sdc-libvirt-volume-driver | 19:29 |
xyang | mriedem: I wasn't sure it still needs a blueprint now that it depends on os-brick | 19:29 |
xyang | mriedem: I can add that reference in the patch | 19:30 |
xyang | mriedem: that's good know | 19:30 |
xyang | that's good to know | 19:30 |
mriedem | oh i see the spec wasn't approved | 19:31 |
mriedem | https://review.openstack.org/#/c/181941/ | 19:31 |
xyang | mriedem: it was not approved because of os-brick | 19:31 |
*** skylerberg has joined #openstack-cinder | 19:31 | |
mriedem | yeah | 19:31 |
mriedem | so, this is just some weird process limbo | 19:31 |
mriedem | i'd link to the bp to the nova change, the shim | 19:31 |
mriedem | we can work with johnthetubaguy on retro-actively approving the spec - maybe you need to update that it depends on os-brick | 19:32 |
mriedem | process really shouldn't hold this up though | 19:32 |
xyang | mriedem: sure, I'll do that | 19:32 |
xyang | mriedem: thanks for your help! | 19:32 |
mriedem | yw | 19:33 |
mriedem | xyang: did your scaleio stuff get into os-brick? | 19:34 |
xyang | mriedem: yes | 19:34 |
mriedem | ok | 19:34 |
*** markstur has joined #openstack-cinder | 19:34 | |
mriedem | then the nova spec should just say it's in os-brick version whatever, list the dependency on the os-brick change in nova, and we should be good | 19:34 |
*** gman-tx has joined #openstack-cinder | 19:35 | |
xyang | mriedem: the unit test on that patch failed now I think because the os-brick version is out of sync | 19:35 |
xyang | mriedem: ok, let me do that | 19:35 |
earlephilhower_ | xyang, mriedm: Coming to the discussion late, but I got bit by the same thing with new os-brick connector changes -2'd in Nova. | 19:38 |
xyang | earlephilhower_: seems that we need to follow the same process here | 19:39 |
earlephilhower_ | Wish the process was a little clearer. Both the technical and bureaucratic ones...ugh. | 19:40 |
earlephilhower_ | The os-brick change in nova as a depends-on won't make CIs pass, though, as the PIP version of os-brick is installed at 0.2.0 (pre L-1) | 19:41 |
earlephilhower_ | At least, not if you got merged after 0.2.0 | 19:41 |
xyang | earlephilhower_: mriedm said we should list the os-brick version with our connector | 19:42 |
*** mfalatic has left #openstack-cinder | 19:43 | |
*** belmoreira has quit IRC | 19:43 | |
*** markvoelker has joined #openstack-cinder | 19:43 | |
*** markvoelker has quit IRC | 19:45 | |
*** markvoelker has joined #openstack-cinder | 19:46 | |
earlephilhower_ | xyang, thanks. Assume that means we need a requirements.txt change. | 19:47 |
xyang | earlephilhower_: I think so | 19:48 |
earlephilhower_ | mriedem: Howdy. Sorry to sound like an echo but I'm in the same boat as xyang with a BP for a new os-brick shim in Nova that just got -2'd. | 19:50 |
mriedem | earlephilhower_: yeah, i see that | 19:50 |
*** mgarza has quit IRC | 19:50 | |
mriedem | earlephilhower_: so you never got a nova-spec posted right? | 19:50 |
mriedem | i didn't see one linked to your bp anyway | 19:50 |
earlephilhower_ | mriedem, actually I've tried everything. HAd a spec, a BP and a bug. Told the spec was not required, though, so that got axed a while back | 19:50 |
*** mgarza has joined #openstack-cinder | 19:50 | |
mriedem | earlephilhower_: where was the spec? | 19:50 |
earlephilhower_ | mriedem, one sec. Let me find it... | 19:51 |
*** nkrinner_afk is now known as nkrinner | 19:51 | |
*** salv-orlando has joined #openstack-cinder | 19:51 | |
earlephilhower_ | mriedem: The BP is here: https://blueprints.launchpad.net/nova/+spec/add-os-brick-volume-driver-hgst-solutions. Still trying to find way to show abandoned older stuff at reviews.openstack.org...one sec | 19:53 |
*** annegentle has quit IRC | 19:53 | |
*** annegentle has joined #openstack-cinder | 19:53 | |
earlephilhower_ | mriedem: Spec was https://review.openstack.org/#/c/190663/ , abandoned on requests from some Nova folks that it should be a BP | 19:54 |
*** hemnafood is now known as hemna | 19:54 | |
*** salv-orlando has quit IRC | 19:57 | |
*** lpetrut has joined #openstack-cinder | 19:58 | |
mriedem | earlephilhower_: ok, updated the bp in launchpad | 19:59 |
mriedem | i'll add your bp and xyang's to the nova meeting agenda for thursday | 19:59 |
mriedem | earlephilhower_: is your hgst stuff merged in os-brick? | 20:00 |
earlephilhower_ | mriedem, Yes, I got merged in L1. Much appreciate the help! | 20:00 |
mriedem | earlephilhower_: is it in a *released* version of os-brick? | 20:01 |
mriedem | nope https://github.com/openstack/os-brick/commit/dc4628f972ce44ad0e63426b31f012e838760a4a | 20:01 |
*** mgarza has quit IRC | 20:01 | |
*** mgarza has joined #openstack-cinder | 20:02 | |
mriedem | xyang: neither is scaleio https://github.com/openstack/os-brick/commit/2c2b42a307513be87e9faf262cf2394ec346e683 | 20:02 |
earlephilhower_ | mriedem: It's not in 0.2.0 tag. I was under impression that a new tag would be made (0.2.1 or 0.3.0 or whatever) on L1 complete | 20:02 |
mriedem | so, you guys are going to need to get a release of os-brick out the door and then update global-requirements for that minimum version before your changes can land in nova | 20:02 |
Swanson | Consistency group question. Anyone? xyang? Is there a way to get back to the cgsnapshot ID from a volume snapshot ID (if that volume was snapshotted via a cgsnapshot.) | 20:02 |
xyang | mriedem: there will be another release later this week, thingee just announced that | 20:02 |
mriedem | ok | 20:02 |
mriedem | xyang: g-r will need to be updated too | 20:02 |
mriedem | for the min versoin | 20:02 |
mriedem | would be at least 0.3.0 | 20:03 |
xyang | mriedem: ok | 20:03 |
xyang | mriedem: what time is tomorrow's meeting? 14:00 UTC? | 20:03 |
*** Apoorva has quit IRC | 20:03 | |
mriedem | 2100 | 20:03 |
xyang | mriedem: ok | 20:04 |
*** crose has quit IRC | 20:04 | |
xyang | Swanson: that is in snapshot dict | 20:04 |
xyang | Swanson: cgsnapshot_id is in snapshot | 20:04 |
xyang | mriedem: from meeting notes: 16:04:42 <thingee> #info os-brick 0.3.0 will be released this week | 20:06 |
Swanson | xyang: Thanks! | 20:06 |
mriedem | ok | 20:06 |
mriedem | https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 20:06 |
xyang | mriedem: thanks | 20:06 |
earlephilhower_ | mriedem: Thx again. | 20:08 |
mriedem | i'll state first that i can't promise anything | 20:08 |
mriedem | we got into it in the nova meeting last week about non-priority bp freeze | 20:08 |
mriedem | so there could be some pushback on this | 20:08 |
*** Longgeek has joined #openstack-cinder | 20:08 | |
*** mgarza has quit IRC | 20:10 | |
*** mgarza has joined #openstack-cinder | 20:10 | |
earlephilhower_ | mriedem: Understood. Anything you think would help strengthen the case for xyang and me? Unfortunately my time machine's in the shop this week... | 20:10 |
*** Apoorva has joined #openstack-cinder | 20:11 | |
mriedem | not really, the code was up before proposal freeze, it's mechanical, i don't think it's a big deal, but we'll see | 20:11 |
mriedem | it won't help that i was banging the drum last week about now letting in more features for liberty :) | 20:11 |
mriedem | s/now/not/ | 20:12 |
*** mutoulbj has joined #openstack-cinder | 20:12 | |
earlephilhower_ | Maybe we could call it a bug and not a feature? :) | 20:13 |
mriedem | nope | 20:13 |
earlephilhower_ | Wasn't serious, of course. :) | 20:13 |
*** Longgeek has quit IRC | 20:13 | |
xyang | mriedem: our shim drivers are not really features, os-brick already has most of the code | 20:15 |
mriedem | yeah i know | 20:15 |
mriedem | preaching to the choir here | 20:15 |
mriedem | if they don't make it, i'll blame jungleboyj | 20:15 |
xyang | :) | 20:16 |
*** annegentle has quit IRC | 20:18 | |
jungleboyj | What are you blaming me for mriedem ? | 20:18 |
jungleboyj | xyang: you owe me more coffee. | 20:19 |
hemna | mriedem, so I think I need to add some fakes into os-brick, so it makes mocking test in nova easier. | 20:19 |
hemna | but we won't have em for a while, so for now I'll just do local mocks (in my nova patch) | 20:19 |
mriedem | hemna: ok, TODOs in the code are always fine for library things like this | 20:19 |
hemna | jungleboyj, coffee? man you are cheap, beer is my miniumum requirements :) | 20:19 |
hemna | mriedem, ok I'll note that. | 20:20 |
hemna | thanks for the review and help | 20:20 |
*** lpetrut has quit IRC | 20:20 | |
xyang | hemna: jungleboyj realized I didn't buy him any coffee in Vancouver. Now I owe him more | 20:20 |
jungleboyj | xyang: +++ Hold on, wrapping up education session. | 20:21 |
*** crose has joined #openstack-cinder | 20:21 | |
smcginnis | hemna: They must mean something like this: https://www.samueladams.com/craft-beers/black-and-brew-coffee-stout | 20:22 |
jungleboyj | xyang: kjnelson says you owe her coffee too. | 20:22 |
xyang | jungleboyj: is it?:) I'll buy her coffee when I get yours | 20:22 |
hemna | smcginnis, ooh nice yah | 20:23 |
earlephilhower_ | It alternates coffee, beer, and ice cream around here, doesn't it? Should start researching Japanese ones... | 20:24 |
smcginnis | jungleboyj: You're supposed to tell kjnelson that the new folks are supposed to buy the coffee. :) | 20:24 |
xyang | smcginnis: hmm, that is true:) | 20:25 |
*** dustins has quit IRC | 20:28 | |
*** dustins has joined #openstack-cinder | 20:28 | |
*** mutoulbj has quit IRC | 20:28 | |
*** thangp has quit IRC | 20:30 | |
*** timcl1 has quit IRC | 20:35 | |
*** annegentle has joined #openstack-cinder | 20:36 | |
*** jasondotstar has quit IRC | 20:37 | |
*** jungleboyj has quit IRC | 20:39 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 20:39 |
*** krtaylor has quit IRC | 20:40 | |
*** nkrinner is now known as nkrinner_afk | 20:40 | |
*** angela-s is now known as angela-s|afk | 20:43 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/202286 | 20:44 |
anish | tsekiyama: I was looking at your review on https://review.openstack.org/#/c/193451/, I don't see an inline comment | 20:44 |
anish | can you tell me the lino no ? | 20:45 |
anish | *line | 20:45 |
tsekiyama | anish: just a moment please | 20:45 |
anish | sure | 20:45 |
tsekiyama | anish: done | 20:45 |
tsekiyama | anish: I was commenting on the old revision. sorry | 20:46 |
*** jasondotstar has joined #openstack-cinder | 20:46 | |
anish | tsekiyama: I guessed that, but didn't see it on rev 8 or 9 so was confused | 20:47 |
anish | thanks for the code hint, will put it in | 20:48 |
*** mgarza has quit IRC | 20:48 | |
tsekiyama | anish: looks like you have submitted a new one while I was writing the comment :) thanks anyway | 20:48 |
anish | no worries, your comment is still valid | 20:49 |
*** bswartz has quit IRC | 20:49 | |
*** mgarza has joined #openstack-cinder | 20:49 | |
jgriffith | xyang: as far as that doctoring.. maybe; also maybe eharney could just tell us what the intention was: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/remotefs.py#L610 | 20:49 |
* jgriffith should've noted that in the review | 20:49 | |
jgriffith | xyang: eharney I mean... I understand it's a note saying "if you inherit from this, you must implement _local_volume_dir"; at least that's what I thought when I reviewed | 20:50 |
xyang | jgriffith: which comment is this about? | 20:51 |
jgriffith | but it's kind of unclear, because they could just implement _get_moutn_point_for_share | 20:51 |
jgriffith | xyang: your comment in the abc patch I put up | 20:51 |
xyang | jgriffith: ok, let me check | 20:51 |
jgriffith | xyang: You asked if we should file a but because I noted the docstring was somewhat unclear | 20:52 |
*** jungleboyj has joined #openstack-cinder | 20:52 | |
jgriffith | s/but/bug/ | 20:52 |
xyang | jgriffith: I think that is patrickeast | 20:52 |
patrickeast | what’d i do? | 20:52 |
*** krtaylor has joined #openstack-cinder | 20:53 | |
xyang | jgriffith: this one? https://review.openstack.org/#/c/201812/5/cinder/volume/drivers/remotefs.py | 20:53 |
patrickeast | ohh | 20:53 |
jungleboyj | smcginnis: Ok, we can try that. :-) She may fight it though. :-) | 20:53 |
*** mylu has joined #openstack-cinder | 20:53 | |
patrickeast | yea, i just put that comment on there since i feel like that sort of thing would probably just be forgotten about and not fixed | 20:53 |
patrickeast | not sure if it really warrants a bug or anything | 20:53 |
mylu | Hi guys, does anyone know if there's a way to create a cinder client with auth_url and scoped token? | 20:53 |
jgriffith | DERP.. sorry | 20:54 |
xyang | jgriffith: I think my comment was about the version. https://review.openstack.org/#/c/201812/4/cinder/volume/drivers/ibm/ibmnas.py | 20:54 |
jgriffith | jungleboyj: https://review.openstack.org/#/c/202201/1/cinder/db/sqlalchemy/migrate_repo/versions/002_quota_class.py | 20:54 |
jgriffith | jungleboyj: if you note, the parent of this is actually removing all of those LOG statements | 20:54 |
jgriffith | jungleboyj: which I guess means you'll be providing a -1 to that for sure :) | 20:55 |
jungleboyj | hemna: For you the minimum is beer. Since xyang isn't a beer drinker I settle for coffee. :-) | 20:55 |
*** lpabon has quit IRC | 20:55 | |
jungleboyj | jgriffith: Looking. | 20:55 |
jgriffith | obviously I need to fix my pep8 issue | 20:56 |
jgriffith | but may not be any point if this is going to be rejected | 20:56 |
ameade | anyone seen this? openstack Invalid command 'volume type create --property' | 20:56 |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 20:56 |
jgriffith | ameade: what did you run to get that? | 20:56 |
jungleboyj | hemna^^ | 20:57 |
ameade | http://dcf901611175aa43f968-c54047c910227e27e1d6f03bb1796fd7.r95.cf5.rackcdn.com/30/199330/2/check/cdot-FC-tempest-openstack/78e25a5/console.txt | 20:57 |
ameade | "openstack volume type create --property volume_backend_name=fs-fibrechannel fs-fibrechannel" | 20:57 |
ameade | it works sometimes though | 20:57 |
ameade | feels like a really weird race or something within stack.sh | 20:57 |
jgriffith | ameade: huh... I never knew there was a "property" option added :( | 20:57 |
jungleboyj | jgriffith: I am not a fan of having no logging in there. Did this come out of some discussion somewhere? | 20:58 |
jgriffith | jungleboyj: there was some discussion in channel | 20:58 |
jgriffith | jungleboyj: so it doesn't make a ton of sense, we either migrate or we don't | 20:59 |
*** Yogi1 has quit IRC | 20:59 | |
jgriffith | the logs aren't overly useful, you'll get the info from a trace if it raises anyway | 20:59 |
jgriffith | jungleboyj: and it makes a real mess of the unit test outputs | 20:59 |
smcginnis | jgriffith: Did you see my comment on that patch? | 20:59 |
jgriffith | jungleboyj: and finally... we're about the only ones that are doing much on this | 20:59 |
jgriffith | smcginnis: not yet | 20:59 |
smcginnis | jgriffith: Wondering if we can just suppress it for the tests. | 21:00 |
smcginnis | jgriffith: I see some value in logging under normal conditions. | 21:00 |
jgriffith | smcginnis: ok, what's useful about it? | 21:00 |
jgriffith | hemna: same question WRT to you question regarding logging of the exception? | 21:01 |
smcginnis | jgriffith: Being able to monitor what's going on. | 21:01 |
smcginnis | jgriffith: I see what you're saying though. | 21:01 |
smcginnis | jgriffith: But I think that argument could be used for ~90% of the logging we have as a whole. | 21:01 |
jgriffith | smcginnis: well, the migration will pass or fail, there's really no monitoring. And anything that fails is going to give you the exception anyway | 21:01 |
jgriffith | alright | 21:01 |
*** mgarza has quit IRC | 21:01 | |
jgriffith | well, I tried | 21:01 |
jgriffith | never mind then | 21:01 |
*** akerr has quit IRC | 21:01 | |
smcginnis | jgriffith: Not trying to shoot it down. | 21:01 |
smcginnis | jgriffith: Just wondering if we can come to some kind of middle ground. | 21:02 |
*** mgarza has joined #openstack-cinder | 21:02 | |
jungleboyj | So, without the logging it will indicate the migration that it is in and then you will get an exception ... | 21:02 |
jgriffith | smcginnis: sure maybe | 21:02 |
*** hw_ has joined #openstack-cinder | 21:02 | |
jgriffith | jungleboyj: yeah typically that's how it would work. | 21:02 |
smcginnis | jgriffith: So running run_tests or tox I am not able to get that migration output. What's the condition that you get it with testr? | 21:02 |
jgriffith | jungleboyj: frankly the try/except is only useful if you want to "do something" with the exception. Which in this case you can't/don't (in the code) so why bother | 21:03 |
smcginnis | jgriffith: I do agree we shouldn't just swallow failures. | 21:03 |
*** nikeshm has quit IRC | 21:03 | |
jungleboyj | Ok, the reason I liked having the logging in there was it made it clearer which migration really failed as it isn't clear to me the way the output is now which migration is really running. I am more a fan of the verbose then others though. | 21:04 |
hemna | and if something fails, it's not useful to help note what failed ? | 21:05 |
*** chenying has quit IRC | 21:05 | |
hemna | jungleboyj, +1 | 21:05 |
hemna | I'm fully ok with removing the "hey I'm doing this phase now" log messages | 21:06 |
smcginnis | Sorry, just realized I was talking to the Remove Logging patch, but you're talking about the fail migration. | 21:06 |
smcginnis | So that one, the exception will get bubbled up and presumable logged there, right? | 21:07 |
*** dims_ has quit IRC | 21:07 | |
jungleboyj | hemna: Yeah, I would be fine with reducing it to just the exceptions. | 21:08 |
*** jasondotstar has quit IRC | 21:09 | |
*** julim has quit IRC | 21:10 | |
*** mgarza has quit IRC | 21:13 | |
*** mgarza_ has joined #openstack-cinder | 21:13 | |
openstackgerrit | John Griffith proposed openstack/cinder: Remove logging statements from migrations https://review.openstack.org/191983 | 21:14 |
mgagne | jgriffith: would there be anything preventing Cinder from allowing online resize? (from in-use volume) | 21:16 |
mgagne | jgriffith: extend is currently restricted to available volumes | 21:17 |
jungleboyj | mgagne: We have a blueprint proposing that and it hasn't been well received. :-( | 21:17 |
*** dustins has quit IRC | 21:17 | |
jgriffith | mgagne: we don't allow it in the API, mostly because we don't want people to hurt themselves :) | 21:17 |
mgagne | jungleboyj: link? | 21:17 |
*** mylu has quit IRC | 21:17 | |
jungleboyj | mgagne: Getting it. | 21:18 |
mgagne | jgriffith: extend, not shrink ;) | 21:18 |
jgriffith | mgagne: understood | 21:18 |
jgriffith | mgagne: there was also some objection because some backends can't do it non-disruptively | 21:18 |
mgagne | jgriffith: we have been carrying a patch for 2 years now and I'm kind of tired to rebase it against latest version each time ;) | 21:18 |
mgagne | jgriffith: could the driver exposes its capabilities and let the operation if supported? | 21:18 |
jgriffith | mgagne: I'm fine with it :) Maybe as an extension? | 21:18 |
jungleboyj | mgagne: Sorry, it is a spec: https://review.openstack.org/#/c/200627/ | 21:19 |
jungleboyj | We would love to get it in. I believe Gerald has an internal version that they are using but we would like it upstream. | 21:20 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 21:20 |
mgagne | I fixed the "but you have to rescan to see the new space" concern by actually rescanning the iscsi targets with a custom Nova extension. | 21:21 |
jungleboyj | mgagne: Appreciate any updates/info you can add into the spec. | 21:22 |
mgagne | sure, will add my thoughts. I thought it was an old spec from last year but I see it's recent | 21:23 |
jungleboyj | Yeah, Gerald just pushed that up. There are a bunch of old ones out there. | 21:24 |
*** lpetrut has joined #openstack-cinder | 21:25 | |
jgriffith | mgagne: I'm not a fan of doing the capability thing and just figuring it out | 21:26 |
jgriffith | mgagne: I am however a fan of making it configurable, whether that be config option or ideally an extension | 21:27 |
*** xyang has quit IRC | 21:27 | |
*** mutoulbj has joined #openstack-cinder | 21:27 | |
mgagne | jgriffith: AFAIK the validation is done before the call ends up in the driver. And I guess people will want to be able to disable/enable extend on a per driver/backend basis. | 21:28 |
*** crose has quit IRC | 21:28 | |
jgriffith | mgagne: correct | 21:28 |
jungleboyj | Gotta drop off for a while. Back on in a bit. | 21:34 |
jgriffith | mgagne: jungleboyj see my comment here: https://review.openstack.org/#/c/200627/ | 21:36 |
jgriffith | mgagne: unless the libvirt side has been fixed or you have a way around it? | 21:36 |
*** akshai has quit IRC | 21:36 | |
mgagne | jgriffith: good point avec cached info, will have to check | 21:36 |
*** crose has joined #openstack-cinder | 21:36 | |
*** salv-orlando has joined #openstack-cinder | 21:36 | |
mgagne | jgriffith: which problem? | 21:36 |
jgriffith | mgagne: the cached size issue, item 1 | 21:37 |
mgagne | jgriffith: hahaha, forgot that part | 21:37 |
mgagne | jgriffith: hold on | 21:37 |
mgagne | jgriffith: will have to double check | 21:37 |
*** mriedem is now known as mriedem_away | 21:38 | |
*** jungleboyj has quit IRC | 21:39 | |
*** mutoulbj has quit IRC | 21:41 | |
*** vilobhmm has quit IRC | 21:42 | |
*** vilobhmm has joined #openstack-cinder | 21:42 | |
*** bnemec has quit IRC | 21:43 | |
*** mgarza_ has quit IRC | 21:43 | |
*** annegentle has quit IRC | 21:44 | |
*** mgarza has joined #openstack-cinder | 21:44 | |
*** annegentle has joined #openstack-cinder | 21:44 | |
openstackgerrit | John Griffith proposed openstack/cinder: Use a single base class for the VolumeDriver https://review.openstack.org/201812 | 21:44 |
jgriffith | erlon: thanks for the review, updated typos | 21:45 |
mgagne | found that patch: https://lists.gnu.org/archive/html/qemu-devel/2011-01/msg01914.html | 21:45 |
*** jamielennox|away is now known as jamielennox | 21:49 | |
*** annegentle has quit IRC | 21:49 | |
*** lpetrut has quit IRC | 21:55 | |
mgagne | jgriffith: virsh qemu-monitor-command --domain <domain-name> "block_resize drive-virtio-disk0 20" | 21:55 |
mgagne | jgriffith: and didn't have to reboot | 21:55 |
mgagne | jgriffith: will have to check if it makes sense or not to use that command with isci targets | 21:55 |
*** earlephilhower_ has quit IRC | 21:56 | |
*** jaypipes has quit IRC | 21:57 | |
*** Longgeek has joined #openstack-cinder | 21:57 | |
mgagne | jgriffith: looks to work "fine". I called the block_resize command with a bogus size (35 instead of 30) and it just informed the instance about the new size. | 22:01 |
*** mylu has joined #openstack-cinder | 22:01 | |
mgagne | gtg | 22:01 |
anish | is this an internal pep issue ? can't see any indication of it being due to code http://logs.openstack.org/51/193451/9/check/gate-os-brick-pep8/7171d83/console.html | 22:02 |
*** patrickeast has quit IRC | 22:03 | |
*** krtaylor has quit IRC | 22:03 | |
*** earlephilhower has joined #openstack-cinder | 22:03 | |
jgriffith | anish: yes, I believe so | 22:05 |
*** edmondsw has quit IRC | 22:05 | |
*** breitz has quit IRC | 22:06 | |
*** breitz has joined #openstack-cinder | 22:06 | |
*** hakimo has quit IRC | 22:07 | |
*** hakimo has joined #openstack-cinder | 22:08 | |
*** mgarza has quit IRC | 22:11 | |
*** mgarza has joined #openstack-cinder | 22:12 | |
*** patrickeast has joined #openstack-cinder | 22:15 | |
*** krtaylor has joined #openstack-cinder | 22:16 | |
*** Longgeek has quit IRC | 22:16 | |
*** zul has quit IRC | 22:17 | |
*** ganso_ has quit IRC | 22:18 | |
*** lcurtis has quit IRC | 22:19 | |
anish | jgriffith: thanks ! | 22:21 |
*** mylu has quit IRC | 22:22 | |
*** EmilienM is now known as EmilienM|off | 22:22 | |
*** mylu has joined #openstack-cinder | 22:23 | |
*** topshare has joined #openstack-cinder | 22:25 | |
*** akerr has joined #openstack-cinder | 22:27 | |
*** patrickeast has quit IRC | 22:27 | |
*** patrickeast has joined #openstack-cinder | 22:32 | |
*** winston-d_ has quit IRC | 22:33 | |
*** annashen has joined #openstack-cinder | 22:34 | |
*** annashen has quit IRC | 22:35 | |
*** primechuck has quit IRC | 22:36 | |
*** bswartz has joined #openstack-cinder | 22:36 | |
*** rushil has quit IRC | 22:37 | |
*** EmilienM|off is now known as EmilienM | 22:37 | |
openstackgerrit | John Griffith proposed openstack/cinder: Fail migration if fk drop fails https://review.openstack.org/202201 | 22:38 |
*** annashen has joined #openstack-cinder | 22:38 | |
*** mutoulbj has joined #openstack-cinder | 22:41 | |
*** akerr has quit IRC | 22:47 | |
*** dims_ has joined #openstack-cinder | 22:51 | |
*** mutoulbj has quit IRC | 22:51 | |
*** xyang1 has joined #openstack-cinder | 22:52 | |
*** jungleboyj has joined #openstack-cinder | 22:52 | |
*** hw_ has quit IRC | 22:57 | |
*** vilobhmm has quit IRC | 23:02 | |
*** vilobhmm has joined #openstack-cinder | 23:02 | |
*** EmilienM is now known as EmilienM|off | 23:03 | |
*** mgarza has quit IRC | 23:04 | |
*** vilobhmm2 has joined #openstack-cinder | 23:04 | |
*** diogogmt_ has quit IRC | 23:05 | |
*** adurbin_ has quit IRC | 23:06 | |
*** topshare has quit IRC | 23:06 | |
dannywilson | anyone else having devstack problems. | 23:06 |
dannywilson | getting ERROR: openstack Invalid command 'volume type create --property' | 23:06 |
*** gouthamr has quit IRC | 23:07 | |
*** jasondotstar has joined #openstack-cinder | 23:07 | |
*** vilobhmm has quit IRC | 23:07 | |
openstackgerrit | Patrick East proposed openstack/cinder: Prevent missing Purity hosts from raising errors https://review.openstack.org/201393 | 23:07 |
jgriffith | dannywilson: funny, that question was asked earlier today | 23:07 |
dannywilson | yeah, I think it was ameade | 23:07 |
jgriffith | dannywilson: indeed it was | 23:08 |
jgriffith | hmm | 23:08 |
jgriffith | oops | 23:08 |
jgriffith | wrong window :) | 23:08 |
dannywilson | jgriffith: never heard a resolution though | 23:08 |
jgriffith | dannywilson: I never had a chance to look at it, TBH I've never used that command :( | 23:09 |
jgriffith | didn't know it existed TBH | 23:09 |
*** topshare has joined #openstack-cinder | 23:09 | |
jgriffith | If ti's intermittent not sure what might be up | 23:09 |
jgriffith | if it's consistent I'd be suspect of latest client update.. but that's a long shot | 23:09 |
dannywilson | happens for me every time, will look around, thanks | 23:09 |
jgriffith | dannywilson: ahh... in that case :) | 23:09 |
*** jasondotstar has quit IRC | 23:10 | |
jgriffith | dannywilson: is in in your c-api log? | 23:10 |
jgriffith | dannywilson: if you can find where it's getting raised we can probably nail the culprit | 23:10 |
jgriffith | although "Invalid command" is typically on the client side | 23:10 |
*** annegentle has joined #openstack-cinder | 23:10 | |
jgriffith | just a sec... | 23:10 |
jgriffith | dannywilson: that's running "cinder type-create --property" right? | 23:10 |
dannywilson | jgriffith: not sure if it is getting that far, might be on openstack client side, I don't even see a "openstack volume" command | 23:11 |
jgriffith | dannywilson: ah.. on deploy | 23:11 |
jgriffith | dannywilson: so it's devstack trying to set up the default types | 23:11 |
patrickeast | uhh | 23:12 |
*** primechuck has joined #openstack-cinder | 23:12 | |
patrickeast | so the last commit on devstack for me is Merge "Use openstack cli for cinder type creation" | 23:12 |
jgriffith | patrickeast: :) | 23:12 |
patrickeast | that looks like mebbe related | 23:12 |
jgriffith | patrickeast: I do believe you've discovered the problem :) | 23:12 |
patrickeast | im suprised it isn’t happening in the gate though | 23:13 |
jgriffith | patrickeast: indeed, I was just going to say that ;) | 23:13 |
*** topshare has quit IRC | 23:14 | |
jgriffith | Unless it's a specific version of openstack cli in the gate | 23:14 |
jgriffith | pinned maybe? | 23:14 |
patrickeast | maybe | 23:14 |
dannywilson | going to try and uninstall all old pip modules to see if that helps | 23:14 |
*** annegentle has quit IRC | 23:15 | |
openstackgerrit | Cindy Pallares proposed openstack/cinder: Fix multi-line docstrings to meet hacking rules https://review.openstack.org/184820 | 23:16 |
jgriffith | dannywilson: you're probably better off removing that call | 23:17 |
jgriffith | dannywilson: ubuntu@stack-2:~/devstack$ openstack volume type create | 23:17 |
jgriffith | ERROR: openstack Unknown command ['volume', 'type', 'create'] | 23:17 |
jgriffith | ubuntu@stack-2:~/devstack$ | 23:17 |
*** topshare has joined #openstack-cinder | 23:17 | |
ameade | jgriffith, dannywilson: still havent figured it out | 23:17 |
ameade | v2 doesnt have type create | 23:17 |
jgriffith | jamielennox: ping | 23:18 |
jgriffith | ameade: dannywilson FWIW, jamielennox committed that change, so perhaps there's some insight there :) | 23:18 |
jgriffith | https://github.com/openstack-dev/devstack/commit/494f7cdf35d025f73b4cce9a3bde7b433183b1d8 | 23:19 |
ameade | mm | 23:19 |
jgriffith | ameade: dannywilson so v2 auth stuff? | 23:20 |
patrickeast | huh so it worked in gate for that review http://logs.openstack.org/83/186683/14/check/check-tempest-dsvm-full/5c460ca/logs/devstacklog.txt.gz#_2015-07-07_20_20_47_314 | 23:20 |
jgriffith | patrickeast: yeah, and here just now: http://logs.openstack.org/36/201336/12/check/gate-tempest-dsvm-full/12628fb/logs/devstacklog.txt.gz#_2015-07-15_22_45_42_111 | 23:21 |
jgriffith | Ohhh! | 23:21 |
jgriffith | dannywilson: what's your local.con look like? | 23:22 |
dannywilson | jgriffith: I have CINDER_ENABLED_BACKENDS=pure:puredriver-1 | 23:23 |
dannywilson | jgriffith: and enabled_backends and default_volume_type set | 23:23 |
jgriffith | dannywilson: so I wonder if the be_name isn't being picked up | 23:23 |
jgriffith | nah.. that doesn't make sense | 23:23 |
jgriffith | NM | 23:23 |
dannywilson | here is the end of the log from failed devstack run http://paste.openstack.org/show/378831/ | 23:24 |
*** lixiaoy1 has joined #openstack-cinder | 23:25 | |
*** annegentle has joined #openstack-cinder | 23:26 | |
*** zhenguo has joined #openstack-cinder | 23:27 | |
ameade | OS_IDENTITY_API_VERSION=2.0....should this be 3.0? | 23:27 |
jgriffith | ameade: yup | 23:27 |
dannywilson | ameade: where do you set that? | 23:27 |
ameade | dannywilson: looks like you set IDENTITY_API_VERSION as an env before you run devstack (or in the local conf i spose) | 23:28 |
*** mylu has quit IRC | 23:29 | |
jgriffith | ameade: dannywilson IDENTITY_API_VERSION | 23:29 |
ameade | or ENABLE_IDENTITY_V2=False | 23:29 |
ameade | still no idea if that matters | 23:31 |
patrickeast | uh, so looks like there is one more commit that maybe would help with this? https://review.openstack.org/#/c/186684/ | 23:31 |
patrickeast | the next one in the dependencies after the one that switched to this new cli command | 23:31 |
patrickeast | https://review.openstack.org/#/c/186684/10/stack.sh | 23:31 |
patrickeast | is exporting that env variable | 23:32 |
ameade | hmm, i'm hoping this is it | 23:32 |
ameade | i did 'env' and saw that it was 2.0 and that struck me as funny | 23:32 |
*** lixiaoy11 has joined #openstack-cinder | 23:37 | |
*** lixiaoy1 has quit IRC | 23:40 | |
*** lpabon has joined #openstack-cinder | 23:42 | |
*** topshare has quit IRC | 23:43 | |
*** topshare has joined #openstack-cinder | 23:44 | |
*** gouthamr has joined #openstack-cinder | 23:44 | |
*** topshare_ has joined #openstack-cinder | 23:46 | |
*** hemna is now known as hemnafk | 23:47 | |
*** IanGovett has quit IRC | 23:48 | |
*** topshare has quit IRC | 23:49 | |
*** topshare_ has quit IRC | 23:51 | |
*** mutoulbj has joined #openstack-cinder | 23:51 | |
dannywilson | ameade: jgriffith: after setting OS_IDENTITY_API_VERSION=3 I get a new error http://paste.openstack.org/show/378919/ | 23:51 |
dannywilson | probably related to using 3.0 but not having the right endpoint? going to revert back one commit in devstack at this point | 23:52 |
jamielennox | jgriffith: hey | 23:54 |
jgriffith | jamielennox: hey ya | 23:54 |
jgriffith | jamielennox: so some folks were having trouble deploying devstack with the openstack client creating types change | 23:55 |
jgriffith | jamielennox: and ident v3 | 23:55 |
jgriffith | jamielennox: thought you might be much better at helping them out than me | 23:55 |
jgriffith | jamielennox: you can see it's been quite a topic in the back-scroll :( | 23:56 |
jamielennox | jgriffith: so it probably should have but the change you reference above shouldn't affect the auth, just the cli tool used | 23:56 |
jgriffith | jamielennox: but I'm half here, half not, and not overly well versed in Auth stuff :( | 23:56 |
jgriffith | jamielennox: yeah, but what's interesting is that requires auth V3 right? Which is what their devstack deploys don't have | 23:56 |
jgriffith | jamielennox: in my case I use a snapshot of devstack each week... so I haven't hit it yet, and haven't had a chance to deploy and look at it myself | 23:57 |
jamielennox | jgriffith: it shouldn't require v3 auth, at the moment everything in devstack not explicity converted to v3 is still v2 auth | 23:57 |
jgriffith | jamielennox: and a quick glance I couldn't figure out how devstack sets i tup. | 23:57 |
jgriffith | jamielennox: OHHH | 23:57 |
jgriffith | well that's weird then | 23:57 |
jamielennox | patrickeast: do you have an example gat where it failed | 23:58 |
jamielennox | gate | 23:58 |
jgriffith | jamielennox: so oddly I can't get openstack client to work for that cmd either | 23:58 |
patrickeast | jamielennox: nope | 23:58 |
jgriffith | jamielennox: wonder if it's pinned there and not locally? | 23:58 |
patrickeast | it hasn’t seemed to fail in gate | 23:58 |
patrickeast | just on some folks devstacks | 23:58 |
*** annashen has quit IRC | 23:58 | |
jamielennox | that's weird | 23:58 |
*** crose has quit IRC | 23:58 | |
jamielennox | i wonder when the volume type-create command got into devstack, maybe the OSC version is old | 23:58 |
jamielennox | oh - does it say something about unable to authenticate? | 23:59 |
patrickeast | i asked dannywilson to check that, his was the same as what is showing up in the pip freeze from gate | 23:59 |
jgriffith | patrickeast: maybe try adding that to the pip from sources? | 23:59 |
jgriffith | j/me should bug out now | 23:59 |
patrickeast | jamielennox: heres the error output he saw http://paste.openstack.org/show/378831/ | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!