| *** markvoelker has quit IRC | 00:01 | |
| *** crushil has quit IRC | 00:01 | |
| *** crushil has joined #openstack-cinder | 00:02 | |
| *** markvoelker has joined #openstack-cinder | 00:02 | |
| *** markvoelker has quit IRC | 00:04 | |
| *** markvoelker has joined #openstack-cinder | 00:04 | |
| *** liverpooler has quit IRC | 00:08 | |
| *** stakeda has joined #openstack-cinder | 00:09 | |
| *** crushil has quit IRC | 00:32 | |
| *** Julien-zte has joined #openstack-cinder | 00:32 | |
| *** kwathore_ has quit IRC | 00:32 | |
| *** kwathore has joined #openstack-cinder | 00:33 | |
| *** vivsoni has quit IRC | 00:33 | |
| *** vivsoni has joined #openstack-cinder | 00:39 | |
| *** wy has joined #openstack-cinder | 00:40 | |
| *** crushil has joined #openstack-cinder | 00:41 | |
| *** xyang has joined #openstack-cinder | 00:46 | |
| *** wxy| has joined #openstack-cinder | 00:48 | |
| *** crushil has quit IRC | 00:50 | |
| *** xyang has quit IRC | 00:51 | |
| *** wanghao has joined #openstack-cinder | 00:55 | |
| *** Julien-zte has quit IRC | 01:00 | |
| *** Julien-zte has joined #openstack-cinder | 01:01 | |
| *** diablo_rojo has joined #openstack-cinder | 01:01 | |
| *** Julien-zte has quit IRC | 01:05 | |
| *** itlinux has quit IRC | 01:07 | |
| *** harlowja has quit IRC | 01:08 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Add documentation for invalid filter keys https://review.openstack.org/464515 | 01:12 |
|---|---|---|
| *** itlinux has joined #openstack-cinder | 01:16 | |
| *** zhonghua has joined #openstack-cinder | 01:22 | |
| *** oomichi has quit IRC | 01:25 | |
| *** salv-orlando has quit IRC | 01:25 | |
| *** oomichi has joined #openstack-cinder | 01:25 | |
| *** dave-mccowan has quit IRC | 01:28 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Support generalized resource filter in client https://review.openstack.org/465002 | 01:28 |
| *** Apoorva_ has joined #openstack-cinder | 01:29 | |
| *** Apoorva has quit IRC | 01:32 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Roll back snapshot's status when backup failed https://review.openstack.org/457168 | 01:33 |
| *** Apoorva_ has quit IRC | 01:33 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Roll back snapshot's status when backup failed https://review.openstack.org/457168 | 01:37 |
| *** gcb has joined #openstack-cinder | 01:43 | |
| openstackgerrit | Vu Cong Tuan proposed openstack/cinder master: Trivial fix typos https://review.openstack.org/464920 | 01:47 |
| *** zhaochao_ has joined #openstack-cinder | 01:48 | |
| *** zhaochao has quit IRC | 01:50 | |
| *** zhaochao_ is now known as zhaochao | 01:50 | |
| *** dave-mccowan has joined #openstack-cinder | 01:51 | |
| *** itlinux has quit IRC | 01:51 | |
| *** xyang has joined #openstack-cinder | 01:52 | |
| openstackgerrit | Dhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation and unit tests. https://review.openstack.org/433341 | 01:54 |
| *** xyang has quit IRC | 01:55 | |
| *** dhinesh has quit IRC | 01:56 | |
| *** itlinux has joined #openstack-cinder | 02:00 | |
| hemna | smcginnis, jungleboyj ping | 02:03 |
| smcginnis | hemna: howdy | 02:03 |
| hemna | so | 02:04 |
| hemna | I just -1'd a driver review that you had +2'd | 02:04 |
| hemna | and I wanted to chat with you about it. | 02:04 |
| hemna | https://review.openstack.org/#/c/436989/ | 02:04 |
| hemna | that one | 02:04 |
| smcginnis | hemna: Just looking | 02:04 |
| smcginnis | hemna: So that one is technically a new driver, since they are adding iSCSI, right? | 02:05 |
| * smcginnis is pulling it up now | 02:05 | |
| hemna | I've seen a bunch of reviews on drivers where they are making new features or major changes in them and aren't bumping the version # or adding any documentation to the driver (re: internal changelog) | 02:05 |
| hemna | well, it's a grey area | 02:05 |
| hemna | it's the same driver, but now adding support for a different protocol... | 02:05 |
| hemna | is that a new driver? | 02:05 |
| smcginnis | hemna: Oh, their doing the all in one approach. Yeah, I think you're right. | 02:05 |
| hemna | anyway, my beef is the fact that folks aren't -1'ing reviews on drivers due to not changing the version # | 02:06 |
| hemna | big changes, or new features should get a new version # IMHO | 02:06 |
| smcginnis | hemna: Yeah, probably a good policy. | 02:06 |
| hemna | I was pretty anal about that in my 3PAR driver days | 02:06 |
| smcginnis | hemna: I know that was a big thing at HP. | 02:06 |
| smcginnis | I've been a little lax leaving it up to the driver maintainers to decide how they want to handle that. | 02:07 |
| hemna | that version number was really the only way we could tell, what rev/bugfix/featureset of that driver they were using | 02:07 |
| smcginnis | hemna: But I think that's probably a good policy to make sure there's no confusion | 02:07 |
| hemna | Since that number is often reported in bug reports and or deployment issues that come up much later in time | 02:07 |
| hemna | that number is pooped out in the c-vol log at startup and customers usually capture that and include it in defects filed. | 02:08 |
| smcginnis | It gets a little complicated with distro specific backports and things, but I suppose at least there's a better chance to know which version is being used. | 02:08 |
| hemna | yup | 02:08 |
| hemna | in the SVN days, we had that header that SVN would automatically update the hash for the file | 02:09 |
| hemna | so you knew where in time that file was in the repo | 02:09 |
| hemna | I wish we had some automagic way in git to do that with drivers. | 02:09 |
| smcginnis | hemna: Want to bring up that recommendation in the meeting tomorrow? | 02:10 |
| hemna | sure | 02:10 |
| smcginnis | Raise reviewer awareness to think about that. | 02:10 |
| hemna | yah | 02:11 |
| hemna | it's really just good practice | 02:11 |
| hemna | ymmv | 02:11 |
| *** zhaochao has quit IRC | 02:16 | |
| wxy| | Hi Cinder team. this is the link of China Bugsmash: https://etherpad.openstack.org/p/OpenStack-Bug-Smash-Pike-Suzhou-Bug-List | 02:19 |
| *** zhaochao has joined #openstack-cinder | 02:19 | |
| hemna | is that going on now? | 02:19 |
| wxy| | Need your attention and review in the next three days. :) feel free to join us online. | 02:19 |
| wxy| | hemna: yeah. It began just now. | 02:20 |
| hemna | ok coolio | 02:20 |
| *** bkopilov_ has quit IRC | 02:21 | |
| tommylikehu | hey winston-d , smcginnis , could you take a look at this patch again? https://review.openstack.org/#/c/461668/ | 02:22 |
| *** bkopilov has quit IRC | 02:22 | |
| jungleboyj | hemna: What's up? | 02:22 |
| hemna | jungleboyj, ^^ | 02:22 |
| hemna | scroll back | 02:22 |
| jungleboyj | hemna: I agree. I -1'd changes to a EMC driver tonight for the same reason. | 02:24 |
| hemna | ok cool | 02:24 |
| jungleboyj | Sounds like something we should talk about at the meeting tomorrow. | 02:25 |
| hemna | that's kinda what I thought too | 02:25 |
| tommylikehu | wxy|, I guess I can paste my review links over there. Although I can't attend :) | 02:25 |
| *** itlinux has quit IRC | 02:25 | |
| wxy| | tommylikehu: Of cause you can. : ) . Help review or solve bug. Both are welcome. | 02:28 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Rollback snapshot's status when backup failed https://review.openstack.org/457168 | 02:29 |
| tommylikehu | wxy|, great! | 02:30 |
| *** itlinux has joined #openstack-cinder | 02:36 | |
| *** kongwei has joined #openstack-cinder | 02:40 | |
| *** dave-mccowan has quit IRC | 02:42 | |
| *** Shunli has joined #openstack-cinder | 02:44 | |
| *** itlinux has quit IRC | 02:48 | |
| *** nicolasbock has quit IRC | 02:51 | |
| openstackgerrit | Sofie Huang proposed openstack/cinder master: FalconStor: Support generic group https://review.openstack.org/464112 | 02:55 |
| *** crushil has joined #openstack-cinder | 02:55 | |
| jungleboyj | hemna: Are you going to add it to the agenda. | 02:56 |
| *** itlinux has joined #openstack-cinder | 02:57 | |
| *** diablo_rojo has quit IRC | 02:59 | |
| openstackgerrit | Merged openstack/os-brick master: Veritas os-brick connector should use privsep https://review.openstack.org/463607 | 03:07 |
| *** ociuhandu has joined #openstack-cinder | 03:08 | |
| *** ociuhandu has quit IRC | 03:10 | |
| *** oomichi has quit IRC | 03:14 | |
| *** ociuhandu has joined #openstack-cinder | 03:15 | |
| openstackgerrit | Merged openstack/os-brick master: Change log level on _get_hba_channel_scsi_target https://review.openstack.org/424747 | 03:17 |
| *** oomichi has joined #openstack-cinder | 03:19 | |
| *** kongwei has quit IRC | 03:22 | |
| *** itlinux has quit IRC | 03:24 | |
| *** gkadam has joined #openstack-cinder | 03:24 | |
| *** gkadam has quit IRC | 03:24 | |
| *** gkadam has joined #openstack-cinder | 03:24 | |
| *** Poornima has joined #openstack-cinder | 03:28 | |
| openstackgerrit | Merged openstack/cinder master: Add documentation for invalid filter keys https://review.openstack.org/464515 | 03:29 |
| *** crushil has quit IRC | 03:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/464362 | 03:32 |
| *** bkopilov_ has joined #openstack-cinder | 03:39 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/460516 | 03:40 |
| *** dikonoor has joined #openstack-cinder | 03:42 | |
| *** bkopilov has joined #openstack-cinder | 03:42 | |
| *** kfarr has quit IRC | 03:43 | |
| *** links has joined #openstack-cinder | 03:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/464362 | 03:46 |
| *** zsli_ has joined #openstack-cinder | 03:49 | |
| *** links has quit IRC | 03:51 | |
| *** Shunli has quit IRC | 03:52 | |
| openstackgerrit | qtlu proposed openstack/cinder master: Fix the wrong comsumption of thin allocation storage https://review.openstack.org/465282 | 03:53 |
| *** links has joined #openstack-cinder | 03:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/460516 | 03:56 |
| openstackgerrit | Vivek Soni proposed openstack/cinder master: 3PAR: Added volume to vvset in online copy https://review.openstack.org/463474 | 03:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements https://review.openstack.org/464461 | 03:57 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/455988 | 03:57 |
| *** gouthamr has quit IRC | 03:57 | |
| *** wxy| has quit IRC | 04:01 | |
| openstackgerrit | Vu Cong Tuan proposed openstack/cinder master: Trivial fix typos https://review.openstack.org/464920 | 04:02 |
| *** arnewiebalck__ has joined #openstack-cinder | 04:11 | |
| *** links has quit IRC | 04:24 | |
| *** dikonoor has quit IRC | 04:32 | |
| *** adisky_ has joined #openstack-cinder | 04:32 | |
| *** Poornima has quit IRC | 04:33 | |
| *** harlowja has joined #openstack-cinder | 04:33 | |
| *** wanghao has quit IRC | 04:34 | |
| *** markstur has quit IRC | 04:38 | |
| *** markstur has joined #openstack-cinder | 04:39 | |
| *** wxy| has joined #openstack-cinder | 04:40 | |
| *** zhangguoqing has joined #openstack-cinder | 04:40 | |
| *** Julien-zte has joined #openstack-cinder | 04:42 | |
| *** markstur has quit IRC | 04:44 | |
| *** dikonoor has joined #openstack-cinder | 04:50 | |
| *** wy has quit IRC | 04:59 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Add extension to add tenant id to backups https://review.openstack.org/289658 | 05:00 |
| *** ociuhandu has quit IRC | 05:00 | |
| *** links has joined #openstack-cinder | 05:06 | |
| *** wanghao has joined #openstack-cinder | 05:08 | |
| *** ChubYann has quit IRC | 05:11 | |
| *** arnewiebalck__ has quit IRC | 05:17 | |
| *** chhavi has joined #openstack-cinder | 05:22 | |
| chhavi | smcginnis: Please add workflow for this review: https://review.openstack.org/#/c/461850/ | 05:23 |
| *** dikonoor has quit IRC | 05:26 | |
| openstackgerrit | xiaoqin proposed openstack/cinder master: Storwize: add mirrored volume support https://review.openstack.org/463524 | 05:29 |
| *** wanghao has quit IRC | 05:30 | |
| *** salv-orlando has joined #openstack-cinder | 05:32 | |
| *** wy has joined #openstack-cinder | 05:34 | |
| openstackgerrit | Mayur Indalkar proposed openstack/cinder master: Implementation of Cinder driver for Veritas Access https://review.openstack.org/409914 | 05:34 |
| *** wanghao has joined #openstack-cinder | 05:40 | |
| *** pgadiya has joined #openstack-cinder | 05:40 | |
| *** dikonoor has joined #openstack-cinder | 05:43 | |
| *** pgadiya has quit IRC | 05:43 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: LVM: fix max_over_subscription_ratio doesn't work https://review.openstack.org/465409 | 05:45 |
| *** Julien-zte has quit IRC | 05:47 | |
| *** dhinesh has joined #openstack-cinder | 05:47 | |
| *** pcaruana has joined #openstack-cinder | 05:48 | |
| *** dhinesh has quit IRC | 05:50 | |
| *** wanghao has quit IRC | 05:54 | |
| *** harlowja has quit IRC | 05:54 | |
| *** wanghao has joined #openstack-cinder | 05:54 | |
| openstackgerrit | wanghao proposed openstack/cinder master: Fix retype missing in API documentation https://review.openstack.org/465416 | 06:00 |
| *** Julien-zte has joined #openstack-cinder | 06:03 | |
| *** pjm6 has quit IRC | 06:04 | |
| *** pjm6 has joined #openstack-cinder | 06:05 | |
| *** rcernin has joined #openstack-cinder | 06:18 | |
| openstackgerrit | ravi edpuganti proposed openstack/cinder master: Added CG capability to volume group in CoprHD https://review.openstack.org/459610 | 06:21 |
| openstackgerrit | wanghao proposed openstack/cinder master: Fix retype missing in API documentation https://review.openstack.org/465416 | 06:26 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Add Generic Volume Group Into Quota Management https://review.openstack.org/439461 | 06:27 |
| openstackgerrit | qtlu proposed openstack/python-cinderclient master: Fix cinder backup-create force flag detail. https://review.openstack.org/465419 | 06:32 |
| openstackgerrit | wanghao proposed openstack/cinder master: Fix incorrect volume_id request description in api-ref https://review.openstack.org/465421 | 06:44 |
| openstackgerrit | wanghao proposed openstack/cinder master: Fix retype missing in API documentation https://review.openstack.org/465416 | 06:50 |
| kaisers1 | Hi! | 06:51 |
| *** belmoreira has joined #openstack-cinder | 06:51 | |
| *** mgagne has quit IRC | 06:53 | |
| wanghao | hi~ | 06:54 |
| *** mgagne has joined #openstack-cinder | 06:56 | |
| *** mgagne is now known as Guest24103 | 06:56 | |
| *** rcernin has quit IRC | 06:57 | |
| *** shausy has joined #openstack-cinder | 06:59 | |
| *** e0ne has joined #openstack-cinder | 07:02 | |
| *** salv-orlando has quit IRC | 07:09 | |
| *** rcernin has joined #openstack-cinder | 07:12 | |
| *** danpawlik_absent is now known as dpawlik | 07:14 | |
| *** mriedem has quit IRC | 07:27 | |
| *** rcernin has quit IRC | 07:29 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: LVM: update max_over_subscription_ratio help message https://review.openstack.org/465409 | 07:33 |
| *** rcernin has joined #openstack-cinder | 07:35 | |
| *** rcernin has quit IRC | 07:35 | |
| *** xelexin_ has quit IRC | 07:35 | |
| *** rcernin has joined #openstack-cinder | 07:35 | |
| *** xelexin_ has joined #openstack-cinder | 07:36 | |
| *** arnewiebalck__ has joined #openstack-cinder | 07:45 | |
| *** arnewiebalck__ has quit IRC | 07:46 | |
| *** arnewiebalck__ has joined #openstack-cinder | 07:47 | |
| *** markstur has joined #openstack-cinder | 07:48 | |
| *** gkadam is now known as gkadam-afk | 07:52 | |
| openstackgerrit | qtlu proposed openstack/python-cinderclient master: Fix cinder backup-create force flag detail. https://review.openstack.org/465419 | 07:53 |
| *** markstur has quit IRC | 07:53 | |
| *** lpetrut has joined #openstack-cinder | 07:54 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Fix all_tenants doesn't work for volume summary https://review.openstack.org/465442 | 07:58 |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | Dou Rui Yuan proposed openstack/cinder master: Correct sphinx source code syntax https://review.openstack.org/465444 | 08:04 |
| *** zzzeek has joined #openstack-cinder | 08:04 | |
| *** mlakat has joined #openstack-cinder | 08:04 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Clean codes around backup APIs https://review.openstack.org/465448 | 08:07 |
| *** salv-orlando has joined #openstack-cinder | 08:10 | |
| *** salv-orlando has quit IRC | 08:14 | |
| *** isaacb has joined #openstack-cinder | 08:22 | |
| *** dikonoo has joined #openstack-cinder | 08:24 | |
| *** dikonoor has quit IRC | 08:24 | |
| *** shyama has joined #openstack-cinder | 08:39 | |
| *** shausy has quit IRC | 08:41 | |
| *** s-shiono_ has quit IRC | 08:51 | |
| *** aarefiev_afk is now known as aarefiev | 08:51 | |
| openstackgerrit | Dou Rui Yuan proposed openstack/cinder master: Correct sphinx source code syntax https://review.openstack.org/465444 | 08:59 |
| *** salv-orlando has joined #openstack-cinder | 09:00 | |
| *** wangyong__ has joined #openstack-cinder | 09:02 | |
| *** wy has quit IRC | 09:05 | |
| *** salv-orlando has quit IRC | 09:06 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient master: Dynamic log level support https://review.openstack.org/445886 | 09:12 |
| *** reduxio has joined #openstack-cinder | 09:12 | |
| *** wxy| has quit IRC | 09:12 | |
| *** isaacb has quit IRC | 09:16 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Rollback snapshot's status when backup failed https://review.openstack.org/457168 | 09:20 |
| *** isaacb has joined #openstack-cinder | 09:25 | |
| openstackgerrit | Liu Qing proposed openstack/os-brick master: Check host device alive before multipath id discovery https://review.openstack.org/459487 | 09:26 |
| *** reduxio has quit IRC | 09:27 | |
| *** gcb has quit IRC | 09:27 | |
| pjm6 | Good morning all | 09:30 |
| *** zsli_ has quit IRC | 09:30 | |
| *** wanghao has quit IRC | 09:30 | |
| *** lkuchlan has joined #openstack-cinder | 09:30 | |
| *** dalgaaf has quit IRC | 09:33 | |
| *** dalgaaf has joined #openstack-cinder | 09:33 | |
| bhagyashris | smcginnis: Hi, | 09:35 |
| *** stakeda has quit IRC | 09:36 | |
| *** reduxio has joined #openstack-cinder | 09:43 | |
| bhagyashris | smcginnis: I want to discuss regarding the bug: https://bugs.launchpad.net/cinder/+bug/1619647 . In which the main concern is "connector" parameter in _force_detach() method ref: https://github.com/openstack/cinder/blob/master/cinder/api/contrib/admin_actions.py#L209 is mandatory as per the api-ref but it seems that the "connector" parameter is not mandatory and will not raise KeyError because default value will be return as None. So IMO | 09:43 |
| openstack | Launchpad bug 1619647 in Cinder "cinder os-force_detach action api 202 accepted even if required "connector" parameter is not passed in request body" [Low,Invalid] - Assigned to Bhagyashri Shewale (bhagyashri-shewale) | 09:43 |
| bhagyashris | this needs to be fix. | 09:43 |
| *** links has quit IRC | 09:48 | |
| reduxio | Hello, I've tried to update my openstack third party CI environment, and now the 'nodepool image-build' command keeps failing for me and I failed in understanding why: | 09:51 |
| reduxio | I'm getting the following error: /bin/bash: /opt/build_git/openstack-infra/system-config/install_puppet.sh: No such file or directory | 09:51 |
| *** e0ne has quit IRC | 09:55 | |
| openstackgerrit | Alon Marx proposed openstack/cinder master: Fix async mirroring on XIV limited range backends https://review.openstack.org/438199 | 09:56 |
| *** links has joined #openstack-cinder | 10:01 | |
| *** Cibo_ has joined #openstack-cinder | 10:01 | |
| openstackgerrit | ravi edpuganti proposed openstack/cinder master: Added CG capability to volume group in CoprHD https://review.openstack.org/459610 | 10:04 |
| *** johnthetubaguy has left #openstack-cinder | 10:05 | |
| *** ma9_ has joined #openstack-cinder | 10:09 | |
| ma9_ | Hi, I have an openstack installation with several volume types and backends. When on Horizon I choose 'boot form image (create new volume)' it does not allow me to choose which volume type is used. How to I specify the default volume type to use for this? | 10:10 |
| *** salv-orlando has joined #openstack-cinder | 10:10 | |
| *** dikonoo has quit IRC | 10:12 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient master: Dynamic log level support https://review.openstack.org/445886 | 10:12 |
| *** faizy has joined #openstack-cinder | 10:14 | |
| *** salv-orlando has quit IRC | 10:15 | |
| *** Julien-zte has quit IRC | 10:18 | |
| *** Cibo_ has quit IRC | 10:18 | |
| *** Julien-zte has joined #openstack-cinder | 10:18 | |
| openstackgerrit | Isaac Beckman proposed openstack/cinder master: ibm-storage: update version number and history https://review.openstack.org/462496 | 10:24 |
| *** Cibo_ has joined #openstack-cinder | 10:27 | |
| *** sdague has joined #openstack-cinder | 10:29 | |
| *** reduxio has quit IRC | 10:37 | |
| *** reduxio has joined #openstack-cinder | 10:37 | |
| *** shyama has quit IRC | 10:37 | |
| *** bkopilov_ has quit IRC | 10:39 | |
| *** bkopilov has quit IRC | 10:39 | |
| *** reduxio has quit IRC | 10:42 | |
| *** Cibo_ has quit IRC | 10:48 | |
| *** reduxio has joined #openstack-cinder | 10:51 | |
| -openstackstatus- NOTICE: gerrit is being restarted to help stuck git replication issues | 10:53 | |
| isaacb | FYI, gate-cinder-python27-ubuntu-xenial always fail on timeout. infra team is handling this | 10:53 |
| *** nicolasbock has joined #openstack-cinder | 10:54 | |
| *** aspiers has left #openstack-cinder | 11:01 | |
| vivsoni | @isaacb - Thanks for the updates | 11:02 |
| *** Julien-zte has quit IRC | 11:03 | |
| *** Julien-zte has joined #openstack-cinder | 11:04 | |
| *** links has quit IRC | 11:05 | |
| *** Julien-zte has quit IRC | 11:11 | |
| *** salv-orlando has joined #openstack-cinder | 11:11 | |
| *** Julien-zte has joined #openstack-cinder | 11:12 | |
| *** reduxio has quit IRC | 11:16 | |
| *** salv-orlando has quit IRC | 11:16 | |
| *** reduxio has joined #openstack-cinder | 11:16 | |
| *** reduxio has quit IRC | 11:21 | |
| *** links has joined #openstack-cinder | 11:22 | |
| openstackgerrit | Mayur Indalkar proposed openstack/cinder master: Implementation of Cinder driver for Veritas Access https://review.openstack.org/409914 | 11:38 |
| *** reduxio has joined #openstack-cinder | 11:39 | |
| *** zul has quit IRC | 11:39 | |
| *** zul has joined #openstack-cinder | 11:41 | |
| pjm6 | i have a problem in removing a cinder volume | 11:41 |
| pjm6 | but i can create/remove others without a problem | 11:42 |
| pjm6 | i'm using dell eqlx | 11:42 |
| pjm6 | and I don't get any error in the logs | 11:42 |
| pjm6 | the volume or its stuck in " removing..." or " unamanging..." | 11:42 |
| pjm6 | anyone have tips in how can i solve this? | 11:42 |
| pjm6 | without removing it manually from D | 11:42 |
| pjm6 | DB | 11:42 |
| *** Julien-zte has quit IRC | 11:44 | |
| *** Jack_Iv has joined #openstack-cinder | 11:46 | |
| *** e0ne has joined #openstack-cinder | 11:48 | |
| *** abishop has joined #openstack-cinder | 11:49 | |
| *** chlong has quit IRC | 11:57 | |
| kaisers1 | smcginnis: eharney: Regarding Nova assisted snapshot API policy issues, CIs are happy with the revert test you set up: (https://review.openstack.org/#/c/465162) | 11:58 |
| isaacb | smcginnis: who should take care of gate-cinder-python27-ubuntu-xenial. All patches fail on this during jenkins run | 12:03 |
| *** ma9_ has quit IRC | 12:05 | |
| lpetrut | I guess we'll have to update our CI configs, having that patch reverted causes our CI to fail again | 12:07 |
| *** ericyoung has quit IRC | 12:11 | |
| openstackgerrit | Isaac Beckman proposed openstack/cinder master: cleanup: remove DEVSTACK_CINDER_SECURE_DELETE https://review.openstack.org/465531 | 12:12 |
| *** jwcroppe has quit IRC | 12:16 | |
| *** jwcroppe has joined #openstack-cinder | 12:16 | |
| *** mriedem has joined #openstack-cinder | 12:17 | |
| *** mriedem has quit IRC | 12:18 | |
| *** mriedem has joined #openstack-cinder | 12:18 | |
| *** liverpooler has joined #openstack-cinder | 12:19 | |
| *** jwcroppe has quit IRC | 12:20 | |
| *** ma9_ has joined #openstack-cinder | 12:22 | |
| *** ma9_ has left #openstack-cinder | 12:22 | |
| *** gkadam-afk is now known as gkadam | 12:35 | |
| *** ma9_ has joined #openstack-cinder | 12:37 | |
| *** gouthamr has joined #openstack-cinder | 12:39 | |
| *** salv-orlando has joined #openstack-cinder | 12:39 | |
| *** salv-orlando has quit IRC | 12:40 | |
| *** edmondsw has joined #openstack-cinder | 12:41 | |
| *** salv-orlando has joined #openstack-cinder | 12:41 | |
| smcginnis | lpetrut: You mean this one? https://review.openstack.org/465162 | 12:45 |
| *** bhagyashris has quit IRC | 12:45 | |
| lpetrut | yep | 12:45 |
| smcginnis | lpetrut: I don't think that revert is the right (or final, at least) answer. | 12:46 |
| *** chhavi has quit IRC | 12:46 | |
| smcginnis | lpetrut: We need to figure out how to update the client but still support the admin endpoint calls like we had. | 12:46 |
| smcginnis | lpetrut: That was mostly a test to make sure that really was the root cause. | 12:46 |
| lpetrut | smcginnis: makes sense | 12:46 |
| smcginnis | isaacb: eharney put up a patch yesterday to enable logging to help troubleshoot what's causing that. | 12:47 |
| isaacb | smcginnis: ok Thanks. | 12:49 |
| *** chhavi has joined #openstack-cinder | 12:53 | |
| *** zhangguoqing has quit IRC | 12:57 | |
| *** lamt has joined #openstack-cinder | 12:58 | |
| *** xyang1 has joined #openstack-cinder | 12:58 | |
| *** bhagyashris has joined #openstack-cinder | 13:03 | |
| *** eharney has joined #openstack-cinder | 13:03 | |
| *** salv-orlando has quit IRC | 13:03 | |
| *** salv-orlando has joined #openstack-cinder | 13:04 | |
| *** salv-orlando has quit IRC | 13:08 | |
| *** Jack_Iv has quit IRC | 13:12 | |
| *** Jack_Iv has joined #openstack-cinder | 13:12 | |
| *** Jack_Iv has quit IRC | 13:13 | |
| *** Jack_Iv has joined #openstack-cinder | 13:13 | |
| *** Jack_Iv has quit IRC | 13:13 | |
| *** kfarr has joined #openstack-cinder | 13:15 | |
| *** chlong has joined #openstack-cinder | 13:16 | |
| bhagyashris | Hi all, Can anyone please guide me How to check the memory consumption or memory during the process initialization suppose for c-vol service? Is there any command to check this? | 13:17 |
| kaisers1 | lpetrut: Hi! Just now saw your comments here (after answering on the bug at https://bugs.launchpad.net/cinder/+bug/1691362), thanks for that. I'm currently running a test based on your config... | 13:17 |
| openstack | Launchpad bug 1691362 in Cinder "Assisted snapshots API usage broken after nova client update" [Undecided,New] | 13:17 |
| *** mkoderer has joined #openstack-cinder | 13:17 | |
| lpetrut | kaisers1: Hi, glad to know that helped you | 13:21 |
| kaisers1 | lpetrut: still waiting for the results.. ^^ | 13:22 |
| *** Tzure has joined #openstack-cinder | 13:22 | |
| *** Julien-zte has joined #openstack-cinder | 13:22 | |
| kaisers1 | lpetrut: *ick*, nope, same tests failing. :-/ | 13:23 |
| kaisers1 | Could you pls post e.g. a paste link containing your old & new configs into the bug ticket? I can do further config checking later on based on those. | 13:25 |
| *** lhx_ has joined #openstack-cinder | 13:27 | |
| lpetrut | sure | 13:29 |
| *** monjoshi has joined #openstack-cinder | 13:29 | |
| lpetrut | kaisers1: while you're around, have you seen by any chance my comment here? https://review.openstack.org/#/c/461471/ | 13:31 |
| lpetrut | I think you'll end up resizing the wrong image (and the same would apply to the NFS driver) | 13:31 |
| bhagyashris | smcginnis: Hi, | 13:33 |
| *** dustins has joined #openstack-cinder | 13:34 | |
| openstackgerrit | Daniel Pawlik proposed openstack/os-brick master: Changed way of providing RBD keyring from keyring_path to client token https://review.openstack.org/465044 | 13:36 |
| smcginnis | bhagyashris: Hey | 13:36 |
| dpawlik | jbernard, jungleboyj, winston-d: I added tests for os-brick and cinder: https://review.openstack.org/#/c/465044/ and https://review.openstack.org/#/c/456672/ | 13:37 |
| dpawlik | thanks for reviewing | 13:37 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/464362 | 13:38 |
| jbernard | dpawlik: thanks, ill try to test the patches this week | 13:38 |
| dpawlik | jbernard: thanks ;) | 13:41 |
| jungleboyj | dpawlik: Thanks, i'll take a look. | 13:41 |
| dpawlik | I guess cinder CI has some problems ;p | 13:42 |
| dpawlik | locally all tests passed | 13:42 |
| *** zzzeek has quit IRC | 13:43 | |
| *** diablo_rojo has joined #openstack-cinder | 13:43 | |
| *** wangyong__ has quit IRC | 13:43 | |
| *** wangyong__ has joined #openstack-cinder | 13:44 | |
| bhagyashris | smcginnis: I want to discuss regarding the bug: https://bugs.launchpad.net/cinder/+bug/1619647 . In which the main concern is "connector" parameter in _force_detach() method ref: https://github.com/openstack/cinder/blob/master/cinder/api/contrib/admin_actions.py#L209 is mandatory as per the api-ref but it seems that the "connector" parameter is not mandatory and will not raise KeyError because default value will be return as None. So IMO | 13:45 |
| openstack | Launchpad bug 1619647 in Cinder "cinder os-force_detach action api 202 accepted even if required "connector" parameter is not passed in request body" [Low,Invalid] - Assigned to Bhagyashri Shewale (bhagyashri-shewale) | 13:45 |
| bhagyashris | this needs to fix. | 13:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/460516 | 13:47 |
| *** lamt has quit IRC | 13:47 | |
| *** reduxio has quit IRC | 13:48 | |
| *** zzzeek has joined #openstack-cinder | 13:49 | |
| *** reduxio has joined #openstack-cinder | 13:49 | |
| *** ma9_ has left #openstack-cinder | 13:51 | |
| *** johnthetubaguy has joined #openstack-cinder | 13:51 | |
| *** lamt has joined #openstack-cinder | 13:53 | |
| *** reduxio has quit IRC | 13:53 | |
| *** kfarr has quit IRC | 13:54 | |
| *** mriedem is now known as mriedem_afk | 13:54 | |
| openstackgerrit | Arnon Yaari proposed openstack/cinder master: INFINIDAT: add support for generic volume groups https://review.openstack.org/450236 | 13:59 |
| openstackgerrit | Arnon Yaari proposed openstack/cinder master: INFINIDAT: add support for iSCSI https://review.openstack.org/436989 | 13:59 |
| openstackgerrit | Arnon Yaari proposed openstack/cinder master: INFINIDAT: add locking around initialize/terminate connection https://review.openstack.org/422277 | 13:59 |
| openstackgerrit | Arnon Yaari proposed openstack/cinder master: INFINIDAT: Use infinisdk for communication with storage array https://review.openstack.org/418451 | 13:59 |
| openstackgerrit | Arnon Yaari proposed openstack/cinder master: INFINIDAT: delete host objects when detaching the last LUN https://review.openstack.org/465585 | 13:59 |
| *** Cibo_ has joined #openstack-cinder | 13:59 | |
| *** crushil has joined #openstack-cinder | 13:59 | |
| *** salv-orlando has joined #openstack-cinder | 14:00 | |
| *** zhangguoqing has joined #openstack-cinder | 14:02 | |
| *** jwcroppe has joined #openstack-cinder | 14:04 | |
| *** reduxio has joined #openstack-cinder | 14:07 | |
| *** crose has joined #openstack-cinder | 14:08 | |
| smcginnis | pjm6: Should ask over here. | 14:10 |
| smcginnis | pjm6: Those greps are of all cinder logs? | 14:10 |
| pjm6 | smcginnis, yes | 14:10 |
| smcginnis | Hmm, I have to step away for a bit. | 14:10 |
| smcginnis | Anyone here have some experience troubleshooting volume delete? | 14:10 |
| smcginnis | pjm6 has a delete request that seems to disapper. | 14:11 |
| smcginnis | https://pastebin.com/Qu2923xb | 14:11 |
| smcginnis | Only for one specific volume. | 14:11 |
| smcginnis | Be back in a bit. | 14:11 |
| pjm6 | I also try to unmanage but the process is the same: https://pastebin.com/xwbsp991 | 14:11 |
| pjm6 | smcginnis, thanks | 14:11 |
| *** itlinux has joined #openstack-cinder | 14:12 | |
| *** reduxio has quit IRC | 14:17 | |
| *** wangyong__ has quit IRC | 14:18 | |
| *** adisky_ has quit IRC | 14:22 | |
| *** Ted_ has joined #openstack-cinder | 14:24 | |
| *** chhavi_ has joined #openstack-cinder | 14:24 | |
| *** Cibo_ has quit IRC | 14:26 | |
| *** chhavi has quit IRC | 14:26 | |
| *** jwcroppe has quit IRC | 14:28 | |
| *** jwcroppe has joined #openstack-cinder | 14:29 | |
| *** Ted_ has quit IRC | 14:30 | |
| *** Guest24103 is now known as mgagne | 14:32 | |
| *** reduxio has joined #openstack-cinder | 14:32 | |
| *** mgagne has quit IRC | 14:32 | |
| *** mgagne has joined #openstack-cinder | 14:32 | |
| *** jwcroppe_ has joined #openstack-cinder | 14:32 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Test older version of tooz https://review.openstack.org/465624 | 14:32 |
| *** jwcroppe has quit IRC | 14:34 | |
| openstackgerrit | Isaac Beckman proposed openstack/cinder master: Add IBM-Storage replication group support https://review.openstack.org/463314 | 14:34 |
| *** reduxio has quit IRC | 14:36 | |
| eharney | smcginnis: my current theory ^ | 14:37 |
| *** dave-mccowan has joined #openstack-cinder | 14:38 | |
| *** reduxio has joined #openstack-cinder | 14:38 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Mock qemu calls in sheepdog tests https://review.openstack.org/465626 | 14:38 |
| smcginnis | pjm6: No luck yet, huh? | 14:39 |
| pjm6 | smcginnis, nothing yet | 14:39 |
| smcginnis | eharney: Do you know what changed there? | 14:39 |
| *** itlinux has quit IRC | 14:39 | |
| eharney | smcginnis: no, but there was a new release the other day | 14:39 |
| smcginnis | eharney: The logging was useful. I caught that sheepdog issue scanning through the output. | 14:39 |
| eharney | smcginnis: i saw a few odd looking things in there... that's probably something we should have for development in general | 14:40 |
| smcginnis | eharney: Another one of those useful exercises to do from time to time at least. | 14:40 |
| eharney | smcginnis: changing the tooz version also found some buggy unit tests in drivers, trying to patch those up now | 14:41 |
| smcginnis | eharney: What's the reason for the PYTHONHASHSEED=3 change? | 14:41 |
| eharney | smcginnis: uh, i forgot that was in there, that was a separate test | 14:42 |
| smcginnis | eharney: I thought that might be the case. :) | 14:42 |
| eharney | and is probably what is actually revealing buggy tests | 14:42 |
| smcginnis | ;) | 14:42 |
| *** reduxio has quit IRC | 14:43 | |
| eharney | i think i left out a file there too... too early still | 14:44 |
| *** Julien-zte has quit IRC | 14:45 | |
| *** Julien-zte has joined #openstack-cinder | 14:45 | |
| *** Julien-zte has quit IRC | 14:48 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix XIV unit hash randomization failure https://review.openstack.org/465629 | 14:48 |
| *** Julien-zte has joined #openstack-cinder | 14:49 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Test older version of tooz https://review.openstack.org/465624 | 14:50 |
| *** gkadam has quit IRC | 14:50 | |
| *** ociuhandu has joined #openstack-cinder | 14:50 | |
| openstackgerrit | Tzur Eliyahu proposed openstack/cinder master: Ibm storage: added new option for qos type https://review.openstack.org/459649 | 14:52 |
| *** markstur has joined #openstack-cinder | 14:52 | |
| smcginnis | eharney: Isn't there an assertDictEqual or something like that? | 14:53 |
| eharney | smcginnis: this is a list, but, there may be some method like that... | 14:54 |
| smcginnis | eharney: Works fine like this anyway. | 14:54 |
| *** mlavalle has joined #openstack-cinder | 14:54 | |
| eharney | smcginnis: i think we usually sort the list and then do an assertEqual for these cases, as another option | 14:55 |
| smcginnis | Yep, assertListEqual | 14:55 |
| *** links has quit IRC | 14:55 | |
| eharney | that won't work | 14:55 |
| smcginnis | eharney: This won't? https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertListEqual | 14:55 |
| eharney | assertItemsEqual looks good though | 14:55 |
| eharney | assertListEqual is the same as assertEqual and order-dependent | 14:56 |
| smcginnis | Ah, pewp. | 14:56 |
| hemna | @! | 14:56 |
| pewp | hemna (╯°□°)╯︵ ┻━┻ | 14:56 |
| eharney | but in python3 they renamed assertItemsEqual to assertCountEqual... uhh | 14:57 |
| _alastor_ | We're leaving here in a few minutes | 14:57 |
| smcginnis | hemna: ;) | 14:57 |
| _alastor_ | sorry, wrong window :P | 14:57 |
| hemna | my morning flip | 14:57 |
| smcginnis | eharney: Well that's pretty annoying. Oh well, you're approach is good. | 14:58 |
| smcginnis | And doesn't change between python versions. | 14:58 |
| eharney | smcginnis: python six covers this, i'll just update the patchset | 14:58 |
| * eharney really looks forward to py27 going away | 14:59 | |
| *** salv-orlando has quit IRC | 14:59 | |
| *** salv-orlando has joined #openstack-cinder | 15:00 | |
| smcginnis | Me too. | 15:00 |
| chhavi_ | hemna: please review, I have updated it based on your initial comments https://review.openstack.org/#/c/461850/ | 15:00 |
| hemna | chhavi_, np | 15:01 |
| chhavi_ | thanks | 15:01 |
| *** ociuhandu has quit IRC | 15:01 | |
| jungleboyj | smcginnis: eharney Kind of behind this morning. So, the patches from Eric above are to fix the py27 failures? | 15:02 |
| eharney | jungleboyj: still trying to figure out the cause there, my current suspicion is a new tooz release. Most of this discussion is about less important unit test things | 15:03 |
| *** rcernin has quit IRC | 15:04 | |
| *** zhangguoqing has quit IRC | 15:04 | |
| *** salv-orlando has quit IRC | 15:04 | |
| *** zhangguoqing has joined #openstack-cinder | 15:04 | |
| smcginnis | jungleboyj: Yeah, no smoking gun yet, but the investigation is uncovering all kinds of wriggling things. | 15:05 |
| *** zhangguoqing has quit IRC | 15:09 | |
| *** itlinux has joined #openstack-cinder | 15:09 | |
| *** Julien-zte has quit IRC | 15:12 | |
| *** Julien-zte has joined #openstack-cinder | 15:13 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix XIV unit hash randomization failure https://review.openstack.org/465629 | 15:13 |
| *** salv-orlando has joined #openstack-cinder | 15:14 | |
| *** salv-orlando has quit IRC | 15:22 | |
| *** felipemonteiro has joined #openstack-cinder | 15:22 | |
| *** lkuchlan has quit IRC | 15:29 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix Datera hash randomization failure https://review.openstack.org/465635 | 15:31 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Enable hash randomization in unit tests https://review.openstack.org/465636 | 15:31 |
| smcginnis | eharney: You're on a roll. :D | 15:31 |
| smcginnis | eharney: So with that last one - things are working now after those changes? | 15:32 |
| * jungleboyj crosses my fingers | 15:32 | |
| eharney | smcginnis: i think not, just going to keep adding Depends-On to it until we can slip it in | 15:33 |
| eharney | every release i fix some of these, but we add them at around the same rate | 15:33 |
| smcginnis | eharney: OK. Would be good to get rid of that PYTHONHASHSEED, but seems a little early to do so. | 15:34 |
| smcginnis | Would think we would still run into random failures. | 15:34 |
| smcginnis | But then again, now might be the time to do that and work through them before we are too close to the end of the cycle. | 15:34 |
| eharney | i would say it shouldn't be that hard to fix them all, but we're still chasing those wacky UUIDs... | 15:35 |
| smcginnis | Yeah, some of those in the log output too. | 15:36 |
| eharney | some of them show up on your screen under just a normal tox run | 15:36 |
| jungleboyj | eharney: The changes look good to me. | 15:36 |
| eharney | the tricky part, of course, is figuring out which of the hash issues are test bugs vs actual bugs in drivers etc | 15:37 |
| jungleboyj | I can go either way on the PYTHONHASHSEED piece. Seems like there have been numerous issues over time along those lines. | 15:37 |
| eharney | the only reason there are numerous issues over time is because we leave it turned off | 15:37 |
| eharney | but that results in code that passes unit tests and can fail when actually deployed | 15:38 |
| *** psachin has joined #openstack-cinder | 15:38 | |
| jungleboyj | So, because we disabled randomization the unit tests were passing but we hide potential valid bugs? | 15:39 |
| *** reduxio has joined #openstack-cinder | 15:39 | |
| eharney | yes, because it doesn't get disabled when actually running the service | 15:40 |
| jungleboyj | Ah, ok, I had misinterpreted how that actually works. | 15:40 |
| *** reduxio has quit IRC | 15:44 | |
| *** mlakat has quit IRC | 15:48 | |
| tommylikehu | hey smcginnis | 15:52 |
| tommylikehu | I thought I should add a docimpact tag when I writing the api-ref modification patch :) | 15:55 |
| tommylikehu | https://review.openstack.org/#/c/461668/ | 15:55 |
| smcginnis | tommylikehu: Hey | 15:57 |
| smcginnis | tommylikehu: No, DocImpact is if you are making a change that requires a follow up update to documentation. | 15:57 |
| tommylikehu | https://wiki.openstack.org/wiki/Documentation/DocImpact | 15:57 |
| *** Apoorva has joined #openstack-cinder | 15:57 | |
| smcginnis | tommylikehu: Like adding something to openstack-manuals for the config guide or installation guide. | 15:57 |
| tommylikehu | Ok, I will remove this :) | 15:58 |
| tommylikehu | smcginnis: | 15:58 |
| *** gouthamr_ has joined #openstack-cinder | 15:58 | |
| smcginnis | tommylikehu: Cool, thanks! | 15:58 |
| smcginnis | tommylikehu: When a patch merges with that, it creates a bug for the follow up work. | 15:59 |
| *** gouthamr has quit IRC | 15:59 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Add 'resource_filters' API to support generalized filtering https://review.openstack.org/461668 | 16:00 |
| *** isaacb has quit IRC | 16:00 | |
| *** lpetrut has quit IRC | 16:01 | |
| *** Tzure has quit IRC | 16:03 | |
| *** mriedem_afk is now known as mriedem | 16:05 | |
| *** gouthamr2 has joined #openstack-cinder | 16:09 | |
| *** pcaruana has quit IRC | 16:11 | |
| *** reduxio has joined #openstack-cinder | 16:13 | |
| *** ociuhandu has joined #openstack-cinder | 16:16 | |
| *** reduxio has quit IRC | 16:17 | |
| *** monjoshi has quit IRC | 16:17 | |
| *** crose has quit IRC | 16:20 | |
| *** isaacb has joined #openstack-cinder | 16:20 | |
| *** belmoreira has quit IRC | 16:20 | |
| *** reduxio has joined #openstack-cinder | 16:21 | |
| *** ociuhandu has quit IRC | 16:25 | |
| *** reduxio has quit IRC | 16:25 | |
| *** mordred has joined #openstack-cinder | 16:26 | |
| mordred | hey all ... | 16:26 |
| mordred | we got a shade bug report from someone who has issues becaues the response payload had a "volume_links" at the top level beside the volumes dict | 16:27 |
| mordred | we can fix/workaroud - but I'm curious because I dont' see any reference to volume_links anywhere in openstack itself | 16:27 |
| mordred | is volume_links a thing? | 16:27 |
| *** reduxio has joined #openstack-cinder | 16:28 | |
| *** gouthamr_ has quit IRC | 16:28 | |
| *** aarefiev is now known as aarefiev_afk | 16:31 | |
| *** reduxio has quit IRC | 16:32 | |
| *** harlowja has joined #openstack-cinder | 16:32 | |
| pjm6 | smcginnis, if I remove the cinder volume in the database | 16:34 |
| pjm6 | and manually update the quota of the project | 16:34 |
| pjm6 | and remove the volume in the storage | 16:34 |
| pjm6 | it would be a problem? | 16:34 |
| smcginnis | pjm6: Not a problem, just need to be careful not to mess anything else up. | 16:35 |
| pjm6 | smcginnis, so removing in that and updating quota should be ok | 16:39 |
| smcginnis | pjm6: Yeah. Would love to know where this is failing, but tracing through the code I don't see anything jumping out as an obvious failure point. | 16:41 |
| smcginnis | pjm6: So it might be easier just cleaning things up and moving on, unfortunately. | 16:41 |
| pjm6 | smcginnis, me too, I try put debug see why is failing | 16:42 |
| pjm6 | creating new / deletign other volumes | 16:42 |
| pjm6 | and nothing | 16:42 |
| tommylikehu | pjm6: I guess it's better to check the dependencies, such as backup and snapshot | 16:43 |
| pjm6 | tommylikehu, at this mmoment I have snapshot disabled | 16:43 |
| tommylikehu | ok | 16:43 |
| pjm6 | this is very strange, idk why this happen. but if for some reason this happens again | 16:43 |
| pjm6 | I will try to investigate more | 16:43 |
| *** eharney has quit IRC | 16:44 | |
| *** eharney has joined #openstack-cinder | 16:48 | |
| *** lhx_ has quit IRC | 16:54 | |
| *** Cibo_ has joined #openstack-cinder | 17:00 | |
| *** gouthamr has joined #openstack-cinder | 17:02 | |
| *** markstur has quit IRC | 17:03 | |
| *** markstur has joined #openstack-cinder | 17:04 | |
| *** chhavi_ has quit IRC | 17:09 | |
| *** stvnoyes1 has left #openstack-cinder | 17:11 | |
| *** isaacb has quit IRC | 17:12 | |
| *** rods has joined #openstack-cinder | 17:13 | |
| *** jwcroppe_ has quit IRC | 17:13 | |
| pjm6 | it was solved, thanks | 17:17 |
| pjm6 | smcginnis, tommylikehu | 17:18 |
| *** felipemonteiro has quit IRC | 17:18 | |
| mordred | smcginnis: (any idea on the volume_links question above?) | 17:20 |
| *** harlowja has quit IRC | 17:26 | |
| *** gaurangt has joined #openstack-cinder | 17:28 | |
| *** ericyoung has joined #openstack-cinder | 17:28 | |
| *** crushil has quit IRC | 17:37 | |
| openstackgerrit | Merged openstack/os-brick master: Updated from global requirements https://review.openstack.org/460516 | 17:37 |
| *** arnewiebalck__ has joined #openstack-cinder | 17:38 | |
| *** e0ne has quit IRC | 17:39 | |
| openstackgerrit | ravi edpuganti proposed openstack/cinder master: Added CG capability to volume group in CoprHD https://review.openstack.org/459610 | 17:39 |
| *** sivn has quit IRC | 17:39 | |
| *** kaisers has joined #openstack-cinder | 17:39 | |
| *** felipemonteiro has joined #openstack-cinder | 17:39 | |
| *** kfarr has joined #openstack-cinder | 17:40 | |
| *** dhinesh has joined #openstack-cinder | 17:41 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix XIV unit hash randomization failure https://review.openstack.org/465629 | 17:41 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Enable hash randomization in unit tests https://review.openstack.org/465636 | 17:41 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix Datera hash randomization failure https://review.openstack.org/465635 | 17:41 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix VMAX hash randomization failure https://review.openstack.org/465680 | 17:41 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix Blockbridge hash randomization failure https://review.openstack.org/465681 | 17:41 |
| kaisers | lpetrut: Yep, saw your comment. That's why workflowed -1. I want to take a closer look and test things, too. Just not 'nuff time right now... :) | 17:41 |
| kaisers | lpetrut: And thanks for the configs, will check that tomorrow | 17:42 |
| *** jwcroppe has joined #openstack-cinder | 17:43 | |
| *** salv-orlando has joined #openstack-cinder | 17:43 | |
| smcginnis | mordred: Sorry, missed it during our Cinder meeting and then I stepped away for a bit. | 17:44 |
| jungleboyj | mordred: Sean I think is out for a bit. | 17:44 |
| *** crushil has joined #openstack-cinder | 17:44 | |
| jungleboyj | Oh speaking of the devil. | 17:45 |
| smcginnis | mordred: I am not aware of a volume_links attribute. | 17:45 |
| smcginnis | jungleboyj: :) | 17:45 |
| *** Cibo_ has quit IRC | 17:45 | |
| jungleboyj | :-) I just looked through the code and don't see anything. I am not aware of it either. | 17:45 |
| smcginnis | Not sure where that's coming from. | 17:45 |
| *** reduxio has joined #openstack-cinder | 17:45 | |
| smcginnis | jungleboyj: Good - independent varification that I'm not losing it. ;) | 17:45 |
| mordred | jungleboyj, smcginnis: cool. thanks! | 17:45 |
| jungleboyj | :-) Two minds are better than one. | 17:46 |
| eharney | mordred, smcginnis: https://review.openstack.org/#/c/465159/ this patch is related to volume_links, just came up the other day | 17:46 |
| mordred | eharney: oh! so it's a pagination thing | 17:46 |
| eharney | yes i think so | 17:46 |
| mordred | cool. | 17:47 |
| smcginnis | eharney: Oh - thanks! | 17:47 |
| smcginnis | jungleboyj: Aparently three minds is even better. | 17:47 |
| *** ChubYann has joined #openstack-cinder | 17:47 | |
| dhinesh | not able to understand why there are failures here, local unit testing works fine http://logs.openstack.org/41/433341/14/check/gate-cinder-python27-ubuntu-xenial/d19b734/console.html | 17:47 |
| mordred | you know what would be neat? if pagination links worked the same in different projects ... | 17:47 |
| * mordred has such insane dreams | 17:48 | |
| *** salv-orlando has quit IRC | 17:48 | |
| smcginnis | mordred: Always the dreamer... | 17:48 |
| mordred | rods: ^^ see above - volume_links is related to pagination it seems | 17:48 |
| rods | mordred yup, thx | 17:48 |
| smcginnis | mordred: Is there a semi-consistent way that's being handled by other projects? | 17:48 |
| jungleboyj | eharney: Oh, duh, check the client. Thank you sir! | 17:48 |
| eharney | dhinesh: it's an ongoing problem -- https://bugs.launchpad.net/cinder/+bug/1691488 is tracking it, but no real info yet | 17:48 |
| openstack | Launchpad bug 1691488 in Cinder "py27 unit tests failing/hanging" [High,Confirmed] | 17:48 |
| *** felipemonteiro has quit IRC | 17:49 | |
| mordred | smcginnis: dunno. I've only implemented pagination support for list_images so far ... and that is different than this - at least in that it just has a top-level 'next' property | 17:49 |
| openstackgerrit | Merged openstack/cinder master: Rollback snapshot's status when backup failed https://review.openstack.org/457168 | 17:49 |
| mordred | smcginnis: I'll let you know when I discover all of the different ways it's done ;) | 17:49 |
| *** reduxio has quit IRC | 17:49 | |
| *** smcginnis changes topic to "py27 unit tests currently failing, working on a root cause" | 17:49 | |
| smcginnis | mordred: +1 | 17:49 |
| smcginnis | mordred: If we can work towards consistency, I'm up for making changes there. | 17:49 |
| dhinesh | eharney ok thanks | 17:50 |
| *** itlinux has quit IRC | 17:50 | |
| *** felipemonteiro has joined #openstack-cinder | 17:52 | |
| *** kfarr has quit IRC | 17:54 | |
| eharney | smcginnis: my "older version of tooz" test didn't reveal anything, because i don't know how to coax the gate into using an older tooz | 17:54 |
| *** felipemonteiro has quit IRC | 17:56 | |
| *** jamielennox is now known as jamielennox|away | 17:59 | |
| *** arnewiebalck__ has quit IRC | 18:04 | |
| *** jamielennox|away is now known as jamielennox | 18:05 | |
| *** zul has quit IRC | 18:06 | |
| Swimingly | Hi all, would anyone like to review https://review.openstack.org/#/c/464003/4 ? It's a resubmission of something that mostly survived review, so it should be too bad :D | 18:08 |
| *** dhinesh has quit IRC | 18:08 | |
| Swimingly | shouldn't be* | 18:08 |
| openstackgerrit | Merged openstack/os-brick master: Changed way of providing RBD keyring from keyring_path to client token https://review.openstack.org/465044 | 18:09 |
| *** dhinesh has joined #openstack-cinder | 18:11 | |
| *** psachin has quit IRC | 18:23 | |
| *** crushil has quit IRC | 18:27 | |
| *** salv-orlando has joined #openstack-cinder | 18:31 | |
| *** e0ne has joined #openstack-cinder | 18:33 | |
| *** jwcroppe has quit IRC | 18:34 | |
| *** itlinux has joined #openstack-cinder | 18:36 | |
| *** zul has joined #openstack-cinder | 18:37 | |
| *** salv-orlando has quit IRC | 18:39 | |
| *** MVenesio has joined #openstack-cinder | 18:47 | |
| *** salv-orlando has joined #openstack-cinder | 18:48 | |
| *** felipemonteiro has joined #openstack-cinder | 18:49 | |
| *** mtanino_ has joined #openstack-cinder | 18:58 | |
| *** catintheroof has joined #openstack-cinder | 18:59 | |
| *** salv-orlando has quit IRC | 18:59 | |
| openstackgerrit | Merged openstack/cinder master: Remove obsolete Liberty code https://review.openstack.org/458217 | 19:02 |
| *** crushil has joined #openstack-cinder | 19:03 | |
| *** lpetrut has joined #openstack-cinder | 19:03 | |
| *** mordred has quit IRC | 19:05 | |
| *** jwcroppe has joined #openstack-cinder | 19:06 | |
| *** mordred has joined #openstack-cinder | 19:07 | |
| *** crose has joined #openstack-cinder | 19:09 | |
| *** stvnoyes has joined #openstack-cinder | 19:14 | |
| smcginnis | eharney: Maybe base a patch off of a pre-upgrade version? | 19:14 |
| smcginnis | eharney: But I suppose the gate will just rebase it anyway... | 19:14 |
| *** mkoderer has quit IRC | 19:27 | |
| *** jose-phillips has quit IRC | 19:34 | |
| *** clenimar has joined #openstack-cinder | 19:34 | |
| *** harlowja has joined #openstack-cinder | 19:39 | |
| *** xyang1 has quit IRC | 19:43 | |
| *** xyang1 has joined #openstack-cinder | 19:45 | |
| *** jose-phillips has joined #openstack-cinder | 19:46 | |
| *** reduxio has joined #openstack-cinder | 19:46 | |
| *** lpetrut has quit IRC | 19:47 | |
| *** crushil has quit IRC | 19:49 | |
| smcginnis | eharney: testtools update maybe? https://github.com/testing-cabal/testtools/compare/2.2.0...2.3.0 | 19:49 |
| *** jose-phillips has quit IRC | 19:52 | |
| *** harlowja has quit IRC | 19:53 | |
| *** reduxio has quit IRC | 19:56 | |
| *** harlowja has joined #openstack-cinder | 19:57 | |
| *** reduxio has joined #openstack-cinder | 19:57 | |
| *** harlowja has quit IRC | 20:00 | |
| *** reduxio has quit IRC | 20:02 | |
| *** reduxio has joined #openstack-cinder | 20:03 | |
| *** crushil has joined #openstack-cinder | 20:06 | |
| *** reduxio has quit IRC | 20:08 | |
| openstackgerrit | Merged openstack/cinder master: Don't return empty volume_image_metadata when list volume https://review.openstack.org/458008 | 20:08 |
| *** reduxio has joined #openstack-cinder | 20:10 | |
| *** reduxio has quit IRC | 20:14 | |
| *** reduxio has joined #openstack-cinder | 20:16 | |
| *** Julien-zte has quit IRC | 20:19 | |
| *** reduxio has quit IRC | 20:20 | |
| *** reduxio has joined #openstack-cinder | 20:22 | |
| *** Julien-zte has joined #openstack-cinder | 20:23 | |
| *** reduxio has quit IRC | 20:26 | |
| *** reduxio has joined #openstack-cinder | 20:28 | |
| *** lamt has quit IRC | 20:33 | |
| *** reduxio has quit IRC | 20:33 | |
| *** itlinux_ has joined #openstack-cinder | 20:34 | |
| *** reduxio has joined #openstack-cinder | 20:35 | |
| *** lamt has joined #openstack-cinder | 20:35 | |
| *** itlinux has quit IRC | 20:37 | |
| *** jose-phillips has joined #openstack-cinder | 20:39 | |
| *** reduxio has quit IRC | 20:40 | |
| *** reduxio has joined #openstack-cinder | 20:41 | |
| *** gouthamr has quit IRC | 20:44 | |
| *** reduxio has quit IRC | 20:45 | |
| *** reduxio has joined #openstack-cinder | 20:47 | |
| *** reduxio has quit IRC | 20:51 | |
| *** zul has quit IRC | 20:51 | |
| *** chlong has quit IRC | 20:52 | |
| *** catintheroof has quit IRC | 21:01 | |
| *** Apoorva_ has joined #openstack-cinder | 21:01 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick master: Check host device alive before multipath id discovery https://review.openstack.org/459487 | 21:03 |
| *** jwcroppe has quit IRC | 21:03 | |
| *** Apoorva has quit IRC | 21:05 | |
| *** vivsoni has quit IRC | 21:11 | |
| *** vivsoni has joined #openstack-cinder | 21:11 | |
| *** MVenesio has quit IRC | 21:11 | |
| *** itlinux_ has quit IRC | 21:14 | |
| *** gouthamr has joined #openstack-cinder | 21:14 | |
| *** dustins has quit IRC | 21:17 | |
| *** crose has quit IRC | 21:19 | |
| *** sdague has quit IRC | 21:22 | |
| *** edmondsw has quit IRC | 21:27 | |
| *** edmondsw has joined #openstack-cinder | 21:28 | |
| *** edmondsw_ has joined #openstack-cinder | 21:31 | |
| *** edmondsw has quit IRC | 21:32 | |
| *** abishop has quit IRC | 21:33 | |
| *** e0ne has quit IRC | 21:33 | |
| *** e0ne has joined #openstack-cinder | 21:34 | |
| *** e0ne has quit IRC | 21:35 | |
| *** e0ne has joined #openstack-cinder | 21:35 | |
| *** e0ne has quit IRC | 21:36 | |
| *** edmondsw_ has quit IRC | 21:36 | |
| *** e0ne has joined #openstack-cinder | 21:36 | |
| *** e0ne has quit IRC | 21:36 | |
| *** e0ne has joined #openstack-cinder | 21:37 | |
| *** e0ne has quit IRC | 21:37 | |
| *** e0ne has joined #openstack-cinder | 21:37 | |
| *** e0ne has quit IRC | 21:38 | |
| *** mtanino_ has quit IRC | 21:44 | |
| *** reduxio has joined #openstack-cinder | 21:56 | |
| *** xyang1 has quit IRC | 21:58 | |
| *** reduxio has quit IRC | 22:01 | |
| *** reduxio has joined #openstack-cinder | 22:02 | |
| *** harlowja has joined #openstack-cinder | 22:02 | |
| *** Julien-zte has quit IRC | 22:04 | |
| *** Julien-zte has joined #openstack-cinder | 22:04 | |
| *** reduxio has quit IRC | 22:07 | |
| *** reduxio has joined #openstack-cinder | 22:09 | |
| *** reduxio has quit IRC | 22:13 | |
| *** reduxio has joined #openstack-cinder | 22:15 | |
| *** markstur has quit IRC | 22:17 | |
| *** harlowja has quit IRC | 22:18 | |
| *** reduxio has quit IRC | 22:19 | |
| *** reduxio has joined #openstack-cinder | 22:27 | |
| *** markstur has joined #openstack-cinder | 22:27 | |
| *** mlavalle has quit IRC | 22:29 | |
| *** reduxio has quit IRC | 22:31 | |
| *** markstur has quit IRC | 22:32 | |
| *** Julien-zte has quit IRC | 22:32 | |
| *** Apoorva_ has quit IRC | 22:33 | |
| *** Apoorva has joined #openstack-cinder | 22:33 | |
| *** reduxio has joined #openstack-cinder | 22:33 | |
| *** itlinux has joined #openstack-cinder | 22:33 | |
| *** Julien-zte has joined #openstack-cinder | 22:34 | |
| *** reduxio has quit IRC | 22:38 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Don't fail unit test if tooz cleanup fails https://review.openstack.org/465756 | 22:38 |
| *** reduxio has joined #openstack-cinder | 22:40 | |
| *** reduxio has quit IRC | 22:44 | |
| *** reduxio has joined #openstack-cinder | 22:45 | |
| *** gouthamr2 has quit IRC | 22:49 | |
| *** itlinux has quit IRC | 22:49 | |
| *** reduxio has quit IRC | 22:50 | |
| *** eharney has quit IRC | 22:52 | |
| *** abishop has joined #openstack-cinder | 22:54 | |
| *** erhudy has quit IRC | 22:55 | |
| *** lamt has quit IRC | 22:57 | |
| *** itlinux has joined #openstack-cinder | 22:59 | |
| *** liverpooler has quit IRC | 23:08 | |
| *** gouthamr has quit IRC | 23:12 | |
| *** mtanino has joined #openstack-cinder | 23:13 | |
| *** dikonoor has joined #openstack-cinder | 23:19 | |
| *** gouthamr has joined #openstack-cinder | 23:20 | |
| *** seba has quit IRC | 23:24 | |
| *** itlinux has quit IRC | 23:27 | |
| *** Julien-zte has quit IRC | 23:29 | |
| *** stakeda has joined #openstack-cinder | 23:30 | |
| *** crushil has quit IRC | 23:32 | |
| *** seba has joined #openstack-cinder | 23:37 | |
| *** pewp has quit IRC | 23:38 | |
| *** pewp has joined #openstack-cinder | 23:38 | |
| *** mtanino has quit IRC | 23:44 | |
| *** felipemonteiro has quit IRC | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!