Wednesday, 2018-01-24

*** zhurong has joined #openstack-cinder00:03
openstackgerritKien Ha proposed openstack/cinder master: Update test cases  https://review.openstack.org/53701700:05
*** owalsh_ has joined #openstack-cinder00:05
*** owalsh has quit IRC00:06
*** owalsh_ has quit IRC00:11
openstackgerritlihaijing proposed openstack/cinder master: Fix cinder quota-usage error  https://review.openstack.org/53516700:12
*** owalsh has joined #openstack-cinder00:16
*** zhurong has quit IRC00:20
*** yangyapeng has quit IRC00:25
*** yangyapeng has joined #openstack-cinder00:25
*** wanghao has quit IRC00:28
*** wanghao has joined #openstack-cinder00:29
*** yangyapeng has quit IRC00:30
openstackgerritKien Ha proposed openstack/cinder master: Update python version number to 3.6 for test case  https://review.openstack.org/53702100:33
openstackgerritKien Ha proposed openstack/cinder master: Update python version number to 3.6 for test case  https://review.openstack.org/53702100:35
*** Dinesh_Bhor has joined #openstack-cinder00:38
*** Brin has joined #openstack-cinder00:44
*** Dinesh_Bhor has quit IRC00:49
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/53703100:50
*** Dinesh_Bhor has joined #openstack-cinder00:50
openstackgerritOpenStack Proposal Bot proposed openstack/cinder-tempest-plugin master: Updated from global requirements  https://review.openstack.org/53395700:50
*** wanghao_ has joined #openstack-cinder00:55
*** wanghao has quit IRC00:55
*** zhurong has joined #openstack-cinder00:58
*** nicolasbock has quit IRC00:58
*** kien-ha has quit IRC01:01
*** r-daneel has quit IRC01:01
*** wanghao has joined #openstack-cinder01:07
*** zhaochao has joined #openstack-cinder01:10
*** wanghao_ has quit IRC01:11
*** stakeda has joined #openstack-cinder01:14
*** crushil has joined #openstack-cinder01:15
*** Dinesh_Bhor has quit IRC01:15
*** salv-orlando has joined #openstack-cinder01:15
*** yangyapeng has joined #openstack-cinder01:17
openstackgerritlihaijing proposed openstack/cinder master: Fix cinder quota-usage error  https://review.openstack.org/53516701:18
*** Dinesh_Bhor has joined #openstack-cinder01:19
*** salv-orlando has quit IRC01:20
*** david-lyle has quit IRC01:22
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements  https://review.openstack.org/53713501:23
*** yangyapeng has quit IRC01:26
*** yangyapeng has joined #openstack-cinder01:26
*** dobson has quit IRC01:27
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/53511301:27
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements  https://review.openstack.org/53715801:27
*** dobson has joined #openstack-cinder01:31
*** dalgaaf has quit IRC01:41
*** dalgaaf has joined #openstack-cinder01:42
*** s-shiono has joined #openstack-cinder01:48
openstackgerritOpenStack Proposal Bot proposed openstack/cinder-tempest-plugin master: Updated from global requirements  https://review.openstack.org/53395701:50
*** markstur has quit IRC02:00
*** dikonoor has joined #openstack-cinder02:03
*** gcb has joined #openstack-cinder02:09
*** zhaochao has quit IRC02:11
*** Dinesh_Bhor has quit IRC02:15
*** Dinesh__Bhor has joined #openstack-cinder02:15
*** salv-orlando has joined #openstack-cinder02:17
*** harlowja has quit IRC02:20
*** salv-orlando has quit IRC02:21
*** dikonoo has joined #openstack-cinder02:22
*** dikonoor has quit IRC02:22
*** zhaochao has joined #openstack-cinder02:25
*** kaisers__ has joined #openstack-cinder02:25
*** kaisers_ has quit IRC02:29
*** wxy has joined #openstack-cinder02:33
openstackgerritliuyamin proposed openstack/cinder master: Use constants for cinder-backup and cinder-schedule  https://review.openstack.org/53418102:38
openstackgerritMerged openstack/cinder master: Add contributor notes on how to use pdb with tests  https://review.openstack.org/53381302:40
*** zhurong has quit IRC02:57
*** crushil has quit IRC03:09
*** salv-orlando has joined #openstack-cinder03:17
*** salv-orlando has quit IRC03:22
*** ykarel has joined #openstack-cinder03:23
*** david-lyle has joined #openstack-cinder03:31
*** david-lyle has quit IRC03:32
*** david-lyle has joined #openstack-cinder03:33
*** david-lyle has quit IRC03:39
*** felipemonteiro_ has joined #openstack-cinder03:41
*** felipemonteiro__ has joined #openstack-cinder03:42
*** felipemonteiro_ has quit IRC03:45
*** abhishekk has joined #openstack-cinder03:47
*** ganso has quit IRC03:48
*** psachin` has joined #openstack-cinder03:52
*** psachin has joined #openstack-cinder03:52
*** markstur has joined #openstack-cinder03:55
*** liverpooler has quit IRC04:11
*** salv-orlando has joined #openstack-cinder04:18
*** salv-orlando has quit IRC04:22
*** dave-mccowan has quit IRC04:27
*** markstur has quit IRC04:27
*** markstur has joined #openstack-cinder04:27
*** felipemonteiro__ has quit IRC04:31
*** markstur has quit IRC04:32
*** Dinesh__Bhor has quit IRC04:40
*** Dinesh__Bhor has joined #openstack-cinder04:40
*** masber has joined #openstack-cinder04:42
*** dikonoo has quit IRC05:12
*** salv-orlando has joined #openstack-cinder05:19
*** harlowja has joined #openstack-cinder05:19
openstackgerritMerged openstack/cinder master: Run doc8 first  https://review.openstack.org/53381405:19
*** salv-orlando has quit IRC05:23
*** lkuchlan has joined #openstack-cinder05:27
*** dikonoor has joined #openstack-cinder05:29
*** links has joined #openstack-cinder05:32
openstackgerritGao Fei proposed openstack/cinder-specs master: Replace Chinese punctuation with English punctuation  https://review.openstack.org/53725805:33
*** lkuchlan has quit IRC05:33
*** wanghao_ has joined #openstack-cinder05:37
*** wanghao has quit IRC05:39
*** salv-orlando has joined #openstack-cinder05:45
*** lkuchlan has joined #openstack-cinder05:49
*** dikonoor has quit IRC05:51
*** pgadiya has joined #openstack-cinder05:51
*** armaan has quit IRC05:51
*** dikonoor has joined #openstack-cinder05:52
*** armaan has joined #openstack-cinder05:59
*** Dinesh__Bhor has quit IRC05:59
*** kaisers__ has quit IRC06:02
*** rcernin_ has joined #openstack-cinder06:03
*** rcernin has quit IRC06:03
*** kaisers_ has joined #openstack-cinder06:05
*** lkuchlan has quit IRC06:05
*** dikonoor has quit IRC06:06
*** dikonoor has joined #openstack-cinder06:06
*** Dinesh__Bhor has joined #openstack-cinder06:08
*** kaisers_ has quit IRC06:10
*** dikonoor has quit IRC06:15
*** lkuchlan has joined #openstack-cinder06:17
*** kaisers_ has joined #openstack-cinder06:21
*** gcb has quit IRC06:22
*** kaisers_ has quit IRC06:26
*** zhangbailin_ has joined #openstack-cinder06:28
*** dikonoor has joined #openstack-cinder06:30
*** Brin has quit IRC06:31
*** kaisers_ has joined #openstack-cinder06:37
*** threestrands has joined #openstack-cinder06:39
*** threestrands has joined #openstack-cinder06:39
*** Dinesh__Bhor has quit IRC06:46
*** threestrands has quit IRC06:49
*** Dinesh__Bhor has joined #openstack-cinder06:50
*** Dinesh__Bhor has quit IRC06:52
*** Dinesh__Bhor has joined #openstack-cinder06:52
*** ykarel is now known as ykarel|away07:09
*** pcaruana has joined #openstack-cinder07:10
*** ykarel|away has quit IRC07:11
*** rcernin has joined #openstack-cinder07:12
*** rcernin_ has quit IRC07:12
*** Dinesh__Bhor has quit IRC07:17
*** pcaruana has quit IRC07:21
*** pcaruana has joined #openstack-cinder07:22
*** markvoelker has quit IRC07:25
*** lkuchlan has quit IRC07:25
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: avoid ascii code error for ssh response stderr  https://review.openstack.org/53728607:26
*** lkuchlan has joined #openstack-cinder07:26
*** markvoelker has joined #openstack-cinder07:27
*** jdurgin has quit IRC07:28
*** e0ne has joined #openstack-cinder07:30
*** markvoelker has quit IRC07:32
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: avoid ascii code error for ssh response stderr  https://review.openstack.org/53728607:35
*** armaan has quit IRC07:41
*** armaan has joined #openstack-cinder07:41
*** lkuchlan has quit IRC07:44
*** lkuchlan has joined #openstack-cinder07:44
*** jdurgin has joined #openstack-cinder07:45
*** itlinux has joined #openstack-cinder07:56
*** e0ne has quit IRC08:01
*** pcaruana has quit IRC08:01
*** e0ne has joined #openstack-cinder08:01
*** alexchadin has joined #openstack-cinder08:10
*** AlexeyAbashkin has joined #openstack-cinder08:23
*** itlinux has quit IRC08:24
*** lpetrut has joined #openstack-cinder08:25
*** tesseract has joined #openstack-cinder08:27
*** alexchadin has quit IRC08:28
*** alexchadin has joined #openstack-cinder08:29
*** lpetrut has quit IRC08:32
openstackgerritwanghao proposed openstack/cinder master: Transfer snapshots with volumes  https://review.openstack.org/53356408:34
*** alexchadin has quit IRC08:36
*** alexchadin has joined #openstack-cinder08:36
*** lpetrut has joined #openstack-cinder08:38
*** pcaruana has joined #openstack-cinder08:39
*** alexchad_ has joined #openstack-cinder08:40
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: [1/2] Fix mock for sleep mocks in volume drivers tests  https://review.openstack.org/53136308:41
*** diegows has quit IRC08:41
*** dikonoo has joined #openstack-cinder08:41
*** dikonoor has quit IRC08:42
*** alexchadin has quit IRC08:42
*** rcernin has quit IRC08:44
openstackgerritwanghao proposed openstack/cinder master: Transfer snapshots with volumes  https://review.openstack.org/53356408:44
*** s-shiono has quit IRC08:45
wanghao_eharney: hi, Eric, are you around?08:45
wanghao_eharney: need your help to review this patch: Transfer snapshots with volumes  https://review.openstack.org/533564. I have addressed your concern and need to check if we can remove the -2 and push this forward this week.08:46
*** diegows has joined #openstack-cinder08:46
*** kaisers_ has quit IRC08:48
*** kaisers_ has joined #openstack-cinder08:48
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: [2/2] Fix mock for sleep mocks in volume drivers tests  https://review.openstack.org/53730308:49
*** tesseract has quit IRC08:54
openstackgerritwanghao proposed openstack/cinder master: Support to query volume filter by changes since/before  https://review.openstack.org/53730508:58
*** tesseract has joined #openstack-cinder09:03
*** harlowja has quit IRC09:07
*** freerunner has quit IRC09:08
*** lpetrut_ has joined #openstack-cinder09:08
*** gkadam has joined #openstack-cinder09:10
*** lpetrut has quit IRC09:10
*** itlinux has joined #openstack-cinder09:12
*** freerunner has joined #openstack-cinder09:14
*** kaisers_ has quit IRC09:15
*** alexchad_ is now known as alexchadin09:15
*** kaisers_ has joined #openstack-cinder09:15
*** kaisers_ has quit IRC09:20
*** lkuchlan has quit IRC09:24
*** psachin` has quit IRC09:24
*** lpetrut_ has quit IRC09:25
*** psachin has quit IRC09:25
*** lkuchlan has joined #openstack-cinder09:25
openstackgerritliuyamin proposed openstack/cinder master: Use constants for cinder-backup and cinder-schedule  https://review.openstack.org/53418109:27
*** markvoelker has joined #openstack-cinder09:28
*** wanghao_ has quit IRC09:30
*** lkuchlan has quit IRC09:30
*** wanghao has joined #openstack-cinder09:30
*** lkuchlan has joined #openstack-cinder09:30
*** wanghao has quit IRC09:31
*** lkuchlan has quit IRC09:31
*** wanghao has joined #openstack-cinder09:31
*** lkuchlan has joined #openstack-cinder09:31
*** wanghao has quit IRC09:31
*** wanghao has joined #openstack-cinder09:32
*** chrisyang_0660 has joined #openstack-cinder09:32
*** wanghao has quit IRC09:32
*** wanghao has joined #openstack-cinder09:33
*** psachin has joined #openstack-cinder09:33
*** wanghao has quit IRC09:33
*** wanghao has joined #openstack-cinder09:33
*** wanghao has quit IRC09:34
*** wanghao has joined #openstack-cinder09:34
*** wanghao has quit IRC09:35
*** lkuchlan has quit IRC09:35
*** wanghao has joined #openstack-cinder09:35
*** lkuchlan has joined #openstack-cinder09:35
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-cinder09:36
*** wanghao has quit IRC09:36
*** wanghao has joined #openstack-cinder09:36
*** wanghao has quit IRC09:37
*** wanghao has joined #openstack-cinder09:37
*** wanghao has quit IRC09:38
*** wanghao has joined #openstack-cinder09:38
*** stakeda has quit IRC09:39
*** kaisers_ has joined #openstack-cinder09:46
*** yangyapeng has quit IRC09:50
Roamer`hm, can somebody take a look at http://logs.openstack.org/33/140733/19/check/tempest-full/efcf44f/job-output.txt.gz#_2018-01-24_07_33_00_269858 - tempest.api.volume.admin.test_volumes_backup.VolumesBackupsAdminTest.test_volume_backup_reset_status failed twice in a row for a completely unrelated Nova change09:50
*** kaisers_ has quit IRC09:50
*** yangyapeng has joined #openstack-cinder09:51
*** yangyapeng has quit IRC09:55
*** kaisers_ has joined #openstack-cinder09:59
*** markvoelker has quit IRC10:02
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix mocks for ZFS Storage Appliance unit tests  https://review.openstack.org/53733210:04
*** jcosmao has left #openstack-cinder10:14
*** jcosmao has joined #openstack-cinder10:17
*** lkuchlan has quit IRC10:20
*** dikonoo has quit IRC10:21
*** pgadiya has quit IRC10:22
*** pgadiya has joined #openstack-cinder10:23
*** zhangbailin__ has joined #openstack-cinder10:26
*** zhangbailin_ has quit IRC10:29
*** dcdamien has joined #openstack-cinder10:33
*** dikonoo has joined #openstack-cinder10:33
*** abhishekk has quit IRC10:34
*** rcernin has joined #openstack-cinder10:34
*** yangyapeng has joined #openstack-cinder10:39
*** dikonoo has quit IRC10:48
*** adisky__ has joined #openstack-cinder10:52
*** AlexeyAbashkin has quit IRC10:53
*** AlexeyAbashkin has joined #openstack-cinder10:54
*** alexchadin has quit IRC10:56
*** yangyapeng has quit IRC10:58
*** markvoelker has joined #openstack-cinder10:58
chrisyang_0660Hello guys, I have a question about the volume driver. Please give me some advices.10:59
chrisyang_0660As we know, vendors keep fixing bugs and adding features to their drivers.10:59
chrisyang_0660But there are some patches are not backward-compatible.10:59
chrisyang_0660For example, https://review.openstack.org/#/c/453654/10:59
chrisyang_0660"configuration.SHARED_CONF_GROUP" is not supported in the Ocata version.10:59
chrisyang_0660If users try to use the newer version of driver which contains some backward-incompatible patches in their stable version of cinder, driver may not work as expected.10:59
chrisyang_0660So, if vendors want to ensure backward-compatibility, how should they do?10:59
chrisyang_0660(1)Also contribute patches to the older (stable) cinder branch?10:59
chrisyang_0660(2)Handle situations in the driver code depends on different cinder versions.10:59
chrisyang_0660(3)Just ask users upgrade to compatible cinder version ?10:59
chrisyang_0660(4)Some other way?10:59
*** yangyapeng has joined #openstack-cinder10:59
*** alexchadin has joined #openstack-cinder10:59
*** ganso has joined #openstack-cinder11:03
*** gaoyan has joined #openstack-cinder11:04
*** gaoyan has quit IRC11:06
*** gaoyan_ has joined #openstack-cinder11:06
e0nechrisyang_0660: hi. #3 is the best one. we don't allow to backport features to the stable branches11:17
e0nechrisyang_0660: btw, it's up to distros vendors to backport new drivers to older version on their own risk11:18
*** sapd_ has quit IRC11:19
*** sapd_ has joined #openstack-cinder11:19
*** psachin has quit IRC11:19
*** itlinux has quit IRC11:23
*** gaoyan_ has quit IRC11:23
*** gaoyan_ has joined #openstack-cinder11:23
*** armaan has quit IRC11:23
Roamer`e0ne, I suspect chrisyang_0660 is not even talking about a distro, but a real hardware/software vendor that has written their own OpenStack driver and has possibly deployed it in customer installations11:24
*** armaan has joined #openstack-cinder11:24
*** markvoelker has quit IRC11:32
*** itlinux has joined #openstack-cinder11:38
*** mvenesio has joined #openstack-cinder11:45
chrisyang_0660eOne: Do you mean that vendors should always only fix bugs in the stable branches, and new features are only allowed in the next stable branch?11:46
*** zhangbailin__ has quit IRC11:54
*** zhangbailin__ has joined #openstack-cinder11:55
*** pgadiya has quit IRC12:02
*** dave-mccowan has joined #openstack-cinder12:09
*** zhurong has joined #openstack-cinder12:10
*** armaan has quit IRC12:13
*** pgadiya has joined #openstack-cinder12:14
*** mvk has quit IRC12:19
*** pgadiya_ has joined #openstack-cinder12:19
*** pgadiya has quit IRC12:23
*** lpetrut_ has joined #openstack-cinder12:29
*** markvoelker has joined #openstack-cinder12:29
*** mvk has joined #openstack-cinder12:31
*** dcdamien has quit IRC12:34
*** dcdamien has joined #openstack-cinder12:36
*** stvnoyes has joined #openstack-cinder12:36
*** dcdamien has quit IRC12:42
*** dcdamien has joined #openstack-cinder12:42
*** tpsilva has joined #openstack-cinder12:45
Roamer`ok, this is stupid... the same problem - test_volume_backup_reset_status failing to create a backup - failed on a different job this time, not tempest-full, but one of the legacy cells jobs... This time tempest-full passed, so this is apparently a transient problem12:52
*** mvk has quit IRC12:56
*** dcdamien has quit IRC12:57
*** dcdamien has joined #openstack-cinder12:58
*** markvoelker has quit IRC13:03
*** abishop has joined #openstack-cinder13:04
*** lpetrut_ has quit IRC13:05
*** lpetrut_ has joined #openstack-cinder13:06
*** abishop has quit IRC13:06
*** abishop has joined #openstack-cinder13:06
zhangbailin__Is it possible to sync the latest version of the patch to the lower version? Some colleagues today asked if the latest version of the patch could be updated to a lower version.13:06
*** lpetrut_ has quit IRC13:07
*** lpetrut_ has joined #openstack-cinder13:08
zhangbailin__Because most users are still using stable versions, such as Ocata and Pike, if the problem found in Queens and the solution exist in previous releases, is it possible to update?13:08
zhangbailin__this is my idea.:)13:09
Roamer`zhangbailin__, this is generally done by checking out the stable/pike branch, making the same changes (or cherry-picking the patch from the master branch), then pushing a new change with git-review13:09
Roamer`Gerrit will figure out that this is a different change, even though it has the same change-id, because it's based on a different branch (stable/pike, not master)13:09
*** lpetrut_ has quit IRC13:09
Roamer`but you have to make sure first that the patch actually works for the older release, and (most probably) change it a bit so that it does, since some things have changed between releases13:10
*** lpetrut_ has joined #openstack-cinder13:10
*** lpetrut_ has quit IRC13:11
Roamer`and if you want the patch to be accepted, it will have to be accepted in the master branch first, and only then for the older releases, and only if the releases still accept that kind of patches (some are in maintenance mode where only critical bugfixes are accepted, and some are out of maintenance, so that nothing is officially changed there)13:11
*** lpetrut_ has joined #openstack-cinder13:11
zhangbailin__Yeah, there is a need for a way to test for differences between two large versions.If we can find this way, and curing this update and change, the maintenance of subsequent versions is a big improvement.13:13
Roamer`or maybe I misunderstood exactly what you meant... that happens to me sometimes13:13
zhangbailin__Roamer`: I agree with you that there really will be a big problem.13:14
*** liverpooler has joined #openstack-cinder13:20
zhangbailin__but still hope that the follow-up version can add a mechanism for automatic detection and synchronization.13:21
*** edmondsw has joined #openstack-cinder13:21
*** edmondsw_ has joined #openstack-cinder13:22
openstackgerritErlon R. Cruz proposed openstack/cinder master: Overprovisioning Improvements  https://review.openstack.org/53485413:24
openstackgerritErlon R. Cruz proposed openstack/cinder master: Netapp Ontap: Adds support for auto-max-over-subscription  https://review.openstack.org/53485513:24
*** pgadiya_ has quit IRC13:25
*** zhangbailin__ has quit IRC13:26
*** edmondsw has quit IRC13:26
openstackgerritErlon R. Cruz proposed openstack/cinder master: Overprovisioning Improvements  https://review.openstack.org/53485413:28
openstackgerritErlon R. Cruz proposed openstack/cinder master: Netapp Ontap: Adds support for auto-max-over-subscription  https://review.openstack.org/53485513:28
*** lpetrut_ has quit IRC13:30
*** alexchadin has quit IRC13:31
*** lhx_ has joined #openstack-cinder13:33
*** alexchadin has joined #openstack-cinder13:40
*** alexchadin has quit IRC13:41
*** pchavva has joined #openstack-cinder13:46
*** AlexeyAbashkin has quit IRC13:47
*** rcernin has quit IRC13:47
*** daidv has quit IRC13:50
*** mvk has joined #openstack-cinder13:56
*** alexchadin has joined #openstack-cinder13:57
openstackgerritHamdy Khader proposed openstack/cinder master: Deprecate Cinder target_protocol values (iscsi, iser)  https://review.openstack.org/53739013:58
*** eharney has quit IRC13:58
*** lpetrut_ has joined #openstack-cinder13:59
*** markvoelker has joined #openstack-cinder14:00
*** AlexeyAbashkin has joined #openstack-cinder14:00
mriedemildikov: jgriffith: the novaclient 2.60 change merged yesterday, will be in the final release for queens this week14:01
*** mchlumsky has joined #openstack-cinder14:01
ildikovmriedem: I saw it this morning14:01
ildikovmriedem: thank you!14:02
ildikovmriedem: the cinderclient patch is landed too for the Cinder microversion (3.50) that's needed for the volume_type/multiattach changes14:02
mriedemok14:03
ildikovmriedem: I will look into adding a test to that bug fix patch and look into the other bug when I can finally spend more than 5 minutes with my laptop, prolly tomorrow14:03
*** kaisers_ has quit IRC14:07
*** mchlumsky has quit IRC14:09
*** mchlumsky has joined #openstack-cinder14:11
*** sapd has joined #openstack-cinder14:13
jgriffithmriedem: perfect, tahnks!14:14
*** mchlumsky has quit IRC14:14
*** links has quit IRC14:15
*** mchlumsky has joined #openstack-cinder14:15
*** zhurong has quit IRC14:18
*** markvoelker has quit IRC14:27
jgriffithmriedem: BTW, you know anything about this:  http://paste.openstack.org/show/652153/14:27
*** markvoelker has joined #openstack-cinder14:27
jgriffithmriedem: after stack.sh unable to boot nova servers; *usually* clears up after a while, but this time it's not14:27
jgriffithcould've sworn I opened a bug on this the other week, but not finding it14:27
*** crushil has joined #openstack-cinder14:27
*** kaisers_ has joined #openstack-cinder14:28
jgriffithin fact, just about every bug I've ever logged against Nova is closed as "opinion" or "won't fix" now that I search on thing a bit14:28
mriedemjgriffith: haven't seen that14:28
jgriffithmriedem: hmm14:29
jgriffithok14:29
mriedemubuntu 16.04 LTS?14:29
jgriffithYes14:29
jgriffith16.04.3 to be precise14:29
jgriffithIt does seem like maybe running stack.sh additional times on a system is the culprit, but haven't verified14:31
jgriffithI'll confirm if that's true or not14:31
*** AlexeyAbashkin has quit IRC14:33
*** crushil has quit IRC14:42
*** eharney has joined #openstack-cinder14:44
*** zhaochao has quit IRC14:45
*** armaan has joined #openstack-cinder14:47
*** dcdamien has quit IRC14:52
*** dcdamien has joined #openstack-cinder14:52
*** markvoelker has quit IRC14:55
*** markvoelker has joined #openstack-cinder14:59
*** AlexeyAbashkin has joined #openstack-cinder15:00
jgriffithmriedem: Looks like it may be related to ./unstack.sh; stack.sh sequence.  If I see it again I'll see if I can gather enough enough for a reasonable bug report15:01
*** dcdamien has quit IRC15:02
*** dcdamien has joined #openstack-cinder15:02
jgriffithildikov: what am I missing here:  ERROR (Conflict): Volume b051181d-fa54-47ba-82a8-1096261714fb has 'multiattach' set, which is not supported for this instance.15:03
mriedemi don't try to use unstack15:03
mriedemjgriffith: using the libvirt driver with nova and lvm?15:03
jgriffithmriedem: yes; I do a reboot inbetween to fix up lvm and the loopback files15:03
mriedemis the instance shelved offloaded?15:04
jgriffithmriedem: wait, you're talking about my last question :)15:04
mriedemi'm talking about all of the questions yes15:04
mriedemfor that conflict error,15:04
jgriffithYes, libvirt, nova and lvm, latest novaclient from git15:04
mriedemare you just trying to attach a multiattach volume to an active instance?15:04
jgriffithand issued an `export OS_COMPUTE_API_VERSION=2.60`15:04
jgriffithmriedem: yes15:04
jgriffithbut I suspect my mv setting is not right maybe?15:05
mriedemnovaclient always uses the latest available between the server and client so that env var is unnecessary, but doesn't hurt15:05
jgriffithmriedem: ahh, I tried that when it failed the first time thinking that must be it15:05
mriedemwhat's your nova git hash?15:05
jgriffithwe should make cinder do that15:05
mriedemthe nova api change just landed like 2 days ago15:05
mriedemtbc, nova *CLI* does version negotiation for the latest, the python API bindings are opt-in and default to 2.115:06
jgriffithlast commit commit 87ea686f9f2cc706205d188922bb14272625e7be15:06
mriedemwe don't consider the CLI an API15:06
jgriffithfrom this morning15:06
jgriffithmriedem: that philosophy would solve a multitude of problems15:06
mriedemassuming you restarted the nova-api service?15:06
jgriffithno, I just ran a fresh stack.sh on a fresh Instance15:07
mriedem"we don't consider the API an API" yes that would be easier :)15:07
jgriffithhehe15:07
mriedemhmm, the instance is ACTIVE yes?15:07
mriedemnot shelved offloaded?15:07
jgriffithcorrect15:07
*** kwathore has quit IRC15:07
*** kwathore has joined #openstack-cinder15:08
*** felipemonteiro has joined #openstack-cinder15:08
jgriffithhmmm, my libvirt version is 3.6.015:09
jgriffith:(15:09
mriedemthat's fine, that's the version from the Pike UCA15:09
mriedemwhich devstack sets up if you're using xenial15:09
*** felipemonteiro_ has joined #openstack-cinder15:10
jgriffithwell, the checks in virt driver require >=3.1015:10
mriedemOOOOOOOOO15:10
mriedemyeah...you can't use the Pike UCA15:10
mriedemforgot about that one15:11
mriedemyou need qemu<2.1015:11
jgriffithhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L30415:11
mriedemyup15:11
*** yangyapeng has quit IRC15:11
mriedemyou need the vars from the local.conf in the nova-multiattach job, sec15:11
mriedemhttps://review.openstack.org/#/c/532689/5/playbooks/legacy/nova-multiattach/run.yaml@3215:11
jgriffithexcellent15:11
jgriffithyaml is going to eat the world15:12
*** yangyapeng has joined #openstack-cinder15:12
jgriffithmriedem: and so since you say "don't" unstack/stack that means start over, spawn a new instance, and wait the 45 minutes for devstack?15:13
jgriffith:(15:13
mriedemwell, you can try it, but15:13
jgriffithyeah15:13
*** felipemonteiro has quit IRC15:13
mriedemif you're going from pike UCA to non-UCA, you'll probably still have the UCA packages15:14
mriedemi do'nt think unstack is smart enough to roll that all back15:14
jgriffithmriedem: i'll just save myself the pain and run a new playbook15:14
ildikovjgriffith: I'm mostly not with wifi for the day, but I can see mriedem takes good care of you already :)15:15
jgriffithildikov: how convenient :)15:16
jgriffithI want to be without wifi for the day :)15:16
*** itlinux has quit IRC15:16
ildikovjgriffith: you just need to go up to the mountains :)15:16
*** yangyapeng has quit IRC15:17
ildikovjgriffith: not even far from you :)15:17
mriedemplaybook - who's loving ansible and yaml now15:17
mriedemi am woefully behind knowing ansible, or containers, or anything that's new and sexy in the last 5 years15:17
jgriffithmriedem: meh, you're not missing much15:18
jgriffithmriedem: same old stuff, new vocabulary15:18
mriedemi know enough to find a thing, copy and change it15:18
mriedemthat's called professional software development15:18
*** itlinux has joined #openstack-cinder15:18
*** e0ne has quit IRC15:18
jgriffith+115:18
jgriffithI find yaml to be horribly over used15:18
jgriffithanyway... I'm old and grumpy15:19
*** e0ne has joined #openstack-cinder15:19
ildikovjgriffith: I think you're just grumpy :)15:19
*** alexchadin has quit IRC15:19
jgriffithildikov: hmmm... you could be right15:19
smcginnis(he is getting kind of old)15:20
smcginnis:P15:20
*** e0ne has quit IRC15:20
*** prometheanfire has joined #openstack-cinder15:21
prometheanfirehttp://logs.openstack.org/50/536150/1/check/cross-cinder-py27/057dda2/testr_results.html.gz15:21
prometheanfirelooks like the test needs updating for the new keystoneauth1 bump15:21
*** dave-mccowan has quit IRC15:22
smcginnisprometheanfire: Thanks, I'll take a look at it.15:22
prometheanfiresmcginnis: it's just adding system_scope to the list of options passed15:22
*** stvnoyes has left #openstack-cinder15:23
-openstackstatus- NOTICE: gerrit has been suffering from a full disk, some mails may have been lost in the last couple of hours. we will now restart gerrit to address ongoing slowness, too15:23
openstackgerritJay Bryant proposed openstack/cinder-specs master: Move Generic Backup Spec to Rocky  https://review.openstack.org/53741615:24
smcginnisprometheanfire: Do you recall - did we raise the minimum on keystoneauth1?15:25
smcginnisprometheanfire: Or this failure is on the patch to raise that?15:25
prometheanfiresmcginnis: >=3.3.015:25
prometheanfirethat's current, 2 months ago15:26
smcginnisSo that's just u-c, so we might still end up running with a lib version that does not include that additional value.15:26
jgriffithsmcginnis: careful, IIRC you and are close to the same age!  :)15:27
smcginnisjgriffith: You've got a year or two on me. :)15:27
jgriffithdarn it!15:28
smcginnisI didn't say I wasn't getting old too though. :D15:28
*** mvenesio has quit IRC15:29
jgriffithWell that's inevitable (almost), but the one alternative doesn't really do much good :)15:29
smcginnisVery true15:29
prometheanfiresmcginnis: ya, if something REQUIRES that option then I agree, a gr bump would be needed15:30
smcginnisprometheanfire: Just makes it a bit tricky. I can add that value so it passes with the new lib, but then it will fail where the new lib isn't being used.15:30
smcginnisWhich right now will be everywhere other than that requirements patch.15:31
smcginnisSo just have to look at how to redo this test a bit.15:31
prometheanfiresmcginnis: ya, that's what I hate about this hard locking on libs :P15:32
prometheanfireimo you should be testing lib behavior with your project, only caring about the options you use15:33
*** Guest28399 is now known as mgagne15:33
*** mgagne has joined #openstack-cinder15:33
jungleboyjAnyone else seeing r.o.o as down?15:33
prometheanfireI think that would be just as valid (or maybe more valid) coverage15:33
prometheanfirejungleboyj: related to the gerrit restarts?15:33
jungleboyjGuess so.  *Sadness*15:34
smcginnisprometheanfire: Yeah, I agree. I think this assertion is a little too strict.15:34
*** edmondsw_ is now known as edmondsw15:34
*** armaan has quit IRC15:34
SwansonWhat's the current state of multi attach?15:38
SwansonDrivers need to do something?15:38
jungleboyjSwanson:  Booo yah!  Landed.15:38
jungleboyjSwanson:  Oh, yeah, I believe that drivers need to indicate if they are capable.15:39
prometheanfireoh... that's new in queens I guess?15:39
Swansonconsumer side, too?15:39
mriedemvolume backup tests seem to be timing out at a pretty high rate in the last 24 hours http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22backup%5C%22%20AND%20message%3A%5C%22failed%20to%20reach%20available%20status%20(current%20creating)%20within%20the%20required%20time%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d15:39
jungleboyjSwanson:  There are policies to be set depending on how the consumer wants to use it.15:40
prometheanfirejungleboyj: the normal iscsi driver/provider not good enough?15:40
jungleboyjprometheanfire:  What do you mean?15:40
*** itlinux has quit IRC15:40
mriedemlikely related to https://github.com/openstack/cinder/commit/dd556fa755adca195e7df82477ae6400f693af1415:40
prometheanfirejungleboyj: I'm probably just confused, ignore me :D15:40
jungleboyjprometheanfire:  Ok.  Drivers that have a backend that can export volumes to be attached to multiple hosts need to indicate the support.  We don't assume.15:41
jungleboyjmriedem:  Oy.  Ok.  Thank you.15:42
jungleboyjgeguileo:  ^^  Thoughts?15:42
mriedemjungleboyj: https://bugs.launchpad.net/cinder/+bug/174516815:42
openstackLaunchpad bug 1745168 in Cinder "volume backup tests timing out since 1/23" [Undecided,New]15:42
prometheanfirejungleboyj: of course, you happen to have a list (in docs or something)?15:42
Roamer`hm, but wasn't that "backup in native thread" change supposed to *improve* performance?  But yeah, I've seen three of these failures just on a single patch today15:42
mriedemsmcginnis: let's hold off on the backport for that change also15:42
jungleboyjMaybe we should revert that?15:42
jungleboyjRoamer`:  Yeah ....15:43
geguileojungleboyj: I don't know much about the specific backends15:43
jungleboyjprometheanfire:  Don't have that listed right now.15:43
geguileojungleboyj: I think all backends support multiple connections, right?  I mean via iSCSI you can connect multiple initiators15:43
jungleboyjmriedem:  Adding to our meeting discussion.15:43
ildikovjgriffith: smcginnis: I don't understand why you guys want to be old so much :)15:43
jungleboyjgeguileo:  I was talking about the backup/native thread patch.15:44
geguileolol15:44
geguileoOK, let me check15:44
openstackgerritSean McGinnis proposed openstack/cinder master: Simply keystone password assertions  https://review.openstack.org/53742715:44
Swansonsmcginnis, "Simplify"15:45
*** dave-mccowan has joined #openstack-cinder15:45
SwansonOr is it simply?15:45
SwansonI should look.15:45
openstackgerritSean McGinnis proposed openstack/cinder master: Simplify keystone password assertions  https://review.openstack.org/53742715:46
* jungleboyj laughs15:46
smcginnisSwanson: Simply simplify.15:46
jungleboyjSimply simplify simplest simpletons.15:47
*** dustins has joined #openstack-cinder15:47
tommylikehuhey geguileo can we deploy multiple scheduler instances within the feature of HA support?15:47
geguileojungleboyj: I located the error15:47
jgriffithmriedem: FYI those settings in the local.conf on a fresh stack worked like a charm15:48
geguileojungleboyj: it shouldn't be happening...15:48
jungleboyjgeguileo:  Ok.15:48
geguileojungleboyj: It's an eventlet bug15:48
mriedemjgriffith: great15:48
geguileoJan 24 15:04:16.031978 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: Traceback (most recent call last):15:48
geguileoJan 24 15:04:16.032126 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]:   File "/usr/local/lib/python2.7/dist-packages/eventlet/hubs/hub.py", line 457, in fire_timers15:48
geguileoJan 24 15:04:16.032267 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]:     timer()15:48
geguileoJan 24 15:04:16.032404 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]:   File "/usr/local/lib/python2.7/dist-packages/eventlet/hubs/timer.py", line 58, in __call__15:48
geguileoJan 24 15:04:16.032559 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]:     cb(*args, **kw)15:48
geguileoJan 24 15:04:16.032703 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]:   File "/usr/local/lib/python2.7/dist-packages/eventlet/semaphore.py", line 145, in _do_acquire15:48
geguileoJan 24 15:04:16.032840 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]:     waiter.switch()15:48
geguileoJan 24 15:04:16.032990 ubuntu-xenial-citycloud-sto2-0002119447 cinder-backup[11788]: error: cannot switch to a different thread15:48
geguileojungleboyj: ^ that's happening because a native thread withing a greenthread is logging15:48
geguileojungleboyj: I'll try to figure out now where is this happening15:48
openstackgerritAseel Awwad proposed openstack/cinder master: Add policy check for complete attachment API action  https://review.openstack.org/53611915:49
mriedemgeguileo: nice spot, because this merged the same day https://github.com/openstack/cinder/commit/1a569e15c35d772a8ca236d75cd1840a415983d315:49
mriedemgeguileo: something is logging where it shouldn't be15:49
SwansonSimple simplification of the simplest simpleton simple keystone password assertions.15:49
geguileomriedem: that other is different, that one actually fixes the req-id, so now is easier to follow the logs15:50
geguileomriedem: both patches are mine   XD15:50
*** mriedem has quit IRC15:50
*** david-lyle has joined #openstack-cinder15:50
jungleboyjHe he he.15:51
*** jose-phi_ has joined #openstack-cinder15:51
*** mriedem has joined #openstack-cinder15:51
*** e0ne has joined #openstack-cinder15:52
*** yangyapeng has joined #openstack-cinder15:53
*** jose-phillips has quit IRC15:54
jgriffithmriedem: I'll move to the test next; but manual run of swap seems to work as expected:  http://paste.openstack.org/show/652274/15:55
geguileomriedem: I believe the problem is the swift object writer, that is logging stuff on close15:57
geguileomriedem: that fails because of https://github.com/eventlet/eventlet/issues/43215:57
mriedemyeah was just looking at the swift backup driver15:58
geguileomriedem: I'll submit a patch right away to remove the logging there15:58
mriedemi knew about the logging thing https://review.openstack.org/#/c/526748/15:58
mriedemsimilar issue in nova with guestfs15:58
mriedemlooks like the other chunkeddrivers (google and posix) are clean16:01
*** AlexeyAbashkin has quit IRC16:05
openstackgerritGorka Eguileor proposed openstack/cinder master: Remove logging on Swift backup obj writer  https://review.openstack.org/53743716:06
geguileomriedem: ^16:09
openstackgerritSean McGinnis proposed openstack/cinder master: Simplify keystone password assertions  https://review.openstack.org/53742716:10
mriedemgeguileo: lgtm but left a comment on doc'ing this16:10
mriedem"don't log in specific methods" is generally forgotten if there isn't a warning16:11
geguileomriedem: I mentioned something in https://review.openstack.org/#/c/518316/14/cinder/backup/manager.py@87  but it's true that I was not clear about the logging, I'll add it to this patch16:12
mriedemgeguileo: oh and add bug 1745168 to the commit message16:14
openstackbug 1745168 in Cinder "volume backup tests timing out since 1/23" [High,Confirmed] https://launchpad.net/bugs/1745168 - Assigned to Gorka Eguileor (gorka)16:14
*** lpetrut_ has quit IRC16:17
openstackgerritGorka Eguileor proposed openstack/cinder master: Remove logging on Swift backup obj writer  https://review.openstack.org/53743716:25
mriedemgeguileo: nice lgtm16:25
mriedemsmcginnis: jungleboyj: ^16:26
geguileomriedem: thanks16:26
openstackgerritJohn Griffith proposed openstack/cinder master: Add debug logging around multiattach status changes  https://review.openstack.org/53744416:27
jgriffithmriedem: maybe run with that added and see if it helps give us some leads ^^16:28
mriedemjgriffith: will do in a bit16:28
*** felipemonteiro_ has quit IRC16:33
*** felipemonteiro_ has joined #openstack-cinder16:34
*** pcaruana has quit IRC16:35
openstackgerritMerged openstack/cinder-specs master: Move update cheesecake promotion to Rocky  https://review.openstack.org/53546716:36
*** kaisers_ has quit IRC16:38
*** e0ne has quit IRC16:40
*** e0ne has joined #openstack-cinder16:40
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/53511316:41
*** markstur has joined #openstack-cinder16:45
erlone0ne, hey, do you know if this failure is related to anything the merged recently? http://logs.openstack.org/54/534854/6/check/tempest-full/4cfe7fe/job-output.txt.gz16:48
*** _ix has quit IRC16:49
e0neerlon: nothing I'm familiar with16:50
*** kaisers_ has joined #openstack-cinder16:54
*** crose has joined #openstack-cinder16:55
*** kien-ha has joined #openstack-cinder16:57
*** kaisers_ has quit IRC16:59
*** prometheanfire has left #openstack-cinder16:59
*** _ix has joined #openstack-cinder17:00
openstackgerritMerged openstack/cinder-specs master: Move Generic Backup Spec to Rocky  https://review.openstack.org/53741617:00
*** luizbag has joined #openstack-cinder17:02
jungleboyjerlon:  Which file are you talking about?17:02
*** kaisers_ has joined #openstack-cinder17:07
kien-hajungleboyj: I took a deeper look into tox and trying to get it working with py36. Was going to have you take a look at the test case changes I made and see if I'm on the right track for it but Sean brought up a good point about not everything is ready for py36. Should I leave it until another time17:10
*** kaisers_ has quit IRC17:12
*** kaisers_ has joined #openstack-cinder17:15
jungleboyjkien-ha:  Right.  That isn't a current goal for OpenStack so I don't think that something to focus on right now.17:18
openstackgerritJay Bryant proposed openstack/cinder-specs master: Move Update Backup's Size Spec to Rocky  https://review.openstack.org/53746017:22
*** sapd has quit IRC17:24
*** gkadam has quit IRC17:26
*** felipemonteiro_ has quit IRC17:29
openstackgerritAlan Bishop proposed openstack/cinder master: Fix how backups handle encryption key IDs  https://review.openstack.org/53746217:30
*** lhx_ has quit IRC17:30
eharneyjungleboyj: this is a pretty significant backup fix that we should get on the queens list ^17:33
*** mvk has quit IRC17:35
jungleboyjOk.17:35
jungleboyjeharney:  Good idea actually.  Should start a list of important bugs to address.17:36
*** gouthamr has joined #openstack-cinder17:49
*** gouthamr has quit IRC17:49
*** mvenesio has joined #openstack-cinder17:49
*** Tahvok has quit IRC17:51
*** Tahvok has joined #openstack-cinder17:52
jungleboyjAdded a section here:  https://etherpad.openstack.org/p/cinder-spec-review-tracking17:55
jungleboyjRelease python-cinderclient:  https://review.openstack.org/53746817:56
*** gouthamr has joined #openstack-cinder17:56
jungleboyjRelease python-brick-cinderclient-ext:  https://review.openstack.org/53747017:56
*** kien-ha has quit IRC17:59
*** david-lyle has quit IRC18:04
*** tesseract has quit IRC18:08
erlonjungleboyj, this file: https://github.com/openstack/cinder/blob/master/doc/source/configuration/tables/cinder-common.inc18:12
jungleboyjerlon:  Interesting.18:13
erlon@all, jungleboyj: are you familiar with this error ( http://logs.openstack.org/54/534854/6/check/tempest-full/4cfe7fe/job-output.txt.gz)? tempest-full job is failing on some of my patches: https://review.openstack.org/#/c/534855/, https://review.openstack.org/#/c/534854/18:13
geguileoerlon: I'm familiar18:15
geguileoerlon: there's already a fix18:15
erlongeguileo, hmmm18:16
erlongeguileo, do you have the link?18:16
geguileoerlon: https://review.openstack.org/#/c/537437/18:16
geguileoit's trying ot merge18:17
*** stvnoyes has joined #openstack-cinder18:20
Roamer`geguileo, actually there are six failed tests in erlon's log, and I'm not sure if the sixth one is directly related to that bug (and THANKS for the fix, BTW!); but still it might be related to "the system was in a weird state after the other five failed", so, yeah.18:22
erlongeguileo, nice, but, now how that change will fix the bug? The error is blowing in n-cpu, on qemu-img trying to open an invalid file: http://logs.openstack.org/54/534854/6/check/tempest-full/4cfe7fe/controller/logs/screen-n-cpu.txt.gz?level=ERROR18:22
geguileoerlon: I saw some backup errors, and those were the ones the patch is fixing, if you have more, I don't know...18:23
geguileoerlon: I don't think those nova errors are my fault18:23
*** markstur has quit IRC18:23
geguileolol18:23
erlongeguileo, haha18:24
*** markstur has joined #openstack-cinder18:24
erlongeguileo, so in the error you fix there where ERRORs on c-bak?18:24
Roamer`erlon, the first five failed tests, the ones that mention backups, should probably all be fixed by geguileo's patch18:25
*** imacdonn has joined #openstack-cinder18:26
*** imacdonn has quit IRC18:26
*** imacdonn has joined #openstack-cinder18:26
geguileoRoamer`: +118:26
*** e0ne has quit IRC18:28
*** _ix has quit IRC18:32
erlonjungleboyj, hey, https://review.openstack.org/#/c/534855/ is passing NetApp CI now, just need to pass zuul18:34
imacdonnany CI gurus around? I updated Jenkins for Oracle ZFSSA CI, and the Gerrit Plugin config got lost. I thought I had a backup, but can't find it now. I have it sort of working, but need to refine the triggers. At the bottom of https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers there is a reference to a layout.yaml file, but I can't find that18:35
imacdonnI guess layout.yaml is a zuul thing, not a jenkins thing18:47
openstackgerritMerged openstack/cinder-specs master: Move Update Backup's Size Spec to Rocky  https://review.openstack.org/53746019:00
*** felipemonteiro has joined #openstack-cinder19:00
*** felipemonteiro_ has joined #openstack-cinder19:02
*** felipemonteiro has quit IRC19:06
*** adisky__ has quit IRC19:07
erloneharney, not sure if the encryption patches has something to do with this but, there are some NFS tests failing on the encryption test: http://logs.openstack.org/37/537437/2/check/legacy-tempest-dsvm-full-devstack-plugin-nfs/9fc23b9/logs/screen-c-vol.txt.gz?level=ERROR19:13
*** izombie has joined #openstack-cinder19:15
*** david-lyle has joined #openstack-cinder19:17
*** luizbag has quit IRC19:17
eharneyerlon: the NFS driver doesn't support encryption. need to finish https://review.openstack.org/#/c/523138/ .19:24
*** e0ne has joined #openstack-cinder19:28
*** felipemonteiro_ has quit IRC19:32
*** _ix has joined #openstack-cinder19:32
*** muttley has joined #openstack-cinder19:32
*** AlexeyAbashkin has joined #openstack-cinder19:34
*** e0ne_ has joined #openstack-cinder19:35
*** e0ne has quit IRC19:38
*** harlowja has joined #openstack-cinder19:38
*** AlexeyAbashkin has quit IRC19:38
*** _ix has quit IRC19:39
*** kaisers_ has quit IRC19:40
erloneharney, ow, good, just noticed the failure and was curious about it19:43
stvnoyesjgriffith or ildikov: Can you take a look at this bug when you get a chance? - https://bugs.launchpad.net/cinder/+bug/1745209 thanks...19:47
openstackLaunchpad bug 1745209 in Cinder "multiattach fails with context object has no attribute 'authorize'" [Undecided,New]19:47
*** muttley has quit IRC19:48
*** dave-mccowan has quit IRC19:58
*** erlon has quit IRC20:03
jgriffithstvnoyes: are you running current master?20:15
stvnoyesyes20:15
stvnoyeswith ildoko's patch for setting multiattach on the volume20:16
stvnoyesildilko...20:16
stvnoyeswhich is why I may be hitting it as I have multiattach set on the volume (maybe?)20:16
*** kaisers_ has joined #openstack-cinder20:16
stvnoyesI am posting a review with a fix in a minute or 220:16
stvnoyesdo you agree that this is a bug?20:17
stvnoyeswith the fix it is now working for me20:17
*** izombie has quit IRC20:19
*** e0ne_ has quit IRC20:19
mriedemstvnoyes: she's out basically i think20:19
openstackgerritMatt Riedemann proposed openstack/cinder master: Add back support for the multiattach flag for volume create  https://review.openstack.org/53586320:20
mriedemildikov: ^ added the bug link and a test20:20
openstackgerritMatt Riedemann proposed openstack/cinder master: Add back support for the multiattach flag for volume create  https://review.openstack.org/53586320:21
mriedemstvnoyes: yeah _attachment_reserve is definitely not using the correct variable20:23
mriedemshould be ctxt.authorize20:23
mriedemstvnoyes: push a patch20:23
*** kaisers_ has quit IRC20:24
openstackgerritSteve Noyes proposed openstack/cinder master: Fix mis-named context object  https://review.openstack.org/53764320:25
*** kaisers_ has joined #openstack-cinder20:26
*** gouthamr has quit IRC20:26
jgriffithstvnoyes: Well, considering I just ran it without issue I'm not sure.20:26
stvnoyesyeah, it's seems strange that I seem to be the only hitting it.20:27
stvnoyesmost likely because I made the volume bootable... You wouldn't see this otherwise.20:28
jgriffithstvnoyes: oh!20:29
jgriffithyup20:29
jgriffithThat's exactly why20:29
stvnoyeskk I added you to the review20:29
jgriffithapproved, thanks!20:30
jgriffithso much for the unit test20:30
stvnoyes:-)20:30
*** crose has quit IRC20:31
*** crose has joined #openstack-cinder20:33
mriedemgiven this was broken b/c of a lack of unit test, why not add a unit test now?20:33
*** crose has quit IRC20:33
jgriffithmriedem: my point was there *is* a unit test20:33
mriedemthere can't be20:34
mriedemotherwise you'd have an AttributeError20:34
jgriffithok20:34
mriedemor the test is mocking out the context call20:34
mriedemeither way there is no test, or the test is broken20:34
mriedemif no one wants to write a test for it, i'll pull it down and add that quick20:34
jgriffiththat was my point, the test is not working properly20:35
mriedemok so let's fix that in the same change20:35
jgriffithfine with me; and it is a missing test for the BFV case20:36
mriedemi'm actually not sure how this didn't fail in tempest,20:36
mriedemneed to look but i thought we had a bfv+multiattach test,20:36
mriedembut maybe we don't try to attach that volume to a 2nd server20:36
mriedemyeah that's what it is, it doesn't try to attach that volume to a 2nd server https://review.openstack.org/#/c/266605/32/tempest/api/compute/volumes/test_attach_volume.py@39620:37
mriedemstvnoyes: are you going to fix the unit test in cinder for your patch or want me to?20:37
stvnoyesi'll do thta. I finally got my new env working again. I am moving everything to a laptop as I will be traveling next week but will still be working20:38
mriedemok20:39
mriedemyou didn't have a laptop all these years?20:39
jgriffithstvnoyes: just let me know; I'm happy to add it on since i missed it in the first place20:39
stvnoyesi'll get it...20:39
*** dave-mccowan has joined #openstack-cinder20:41
*** crose has joined #openstack-cinder20:43
*** itlinux has joined #openstack-cinder20:44
openstackgerritMatt Riedemann proposed openstack/cinder master: Fix multiattach policy names in release note  https://review.openstack.org/53765220:47
mriedemjgriffith: smcginnis: ^20:48
openstackgerritMerged openstack/cinder master: Remove logging on Swift backup obj writer  https://review.openstack.org/53743720:50
mriedemwoot ^20:50
smcginnismriedem: There's the logging/threading fix.20:50
mriedemway ahead of you :)20:50
openstackgerritJohn Griffith proposed openstack/cinder master: Add debug logging around multiattach status changes  https://review.openstack.org/53744420:51
smcginnisThen I shall woot your woot.20:51
smcginnisHmm, sounds kind of naughty. Maybe I won't do that.20:51
*** belmoreira has joined #openstack-cinder20:51
mriedemsloppy woots?20:51
* smcginnis is not going to continue this conversation20:51
jungleboyjWoot woot woot!20:51
SwansonEww.20:51
jungleboyjmriedem:  He he he.20:52
mriedemand....i'm spent20:53
* jungleboyj hands mriedem a cigarette 20:53
jungleboyjWas it good for you?20:53
mriedemyou know,20:53
mriedemi've had better20:53
jungleboyjbwah ha ha.20:54
*** e0ne has joined #openstack-cinder21:00
*** liverpooler has quit IRC21:00
*** crose has quit IRC21:07
openstackgerritEric Harney proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest  https://review.openstack.org/53120621:17
openstackgerritEric Harney proposed openstack/cinder-tempest-plugin master: Add LVM+tgt tempest job  https://review.openstack.org/53765821:18
*** crose has joined #openstack-cinder21:19
*** e0ne has quit IRC21:20
openstackgerritEric Harney proposed openstack/cinder-tempest-plugin master: Add LVM+tgt tempest job  https://review.openstack.org/53765821:21
eharneywe need some kind of tox zuul syntax checker there ^21:22
eharneynot sure how to see if the patch is correct w/o just throwing it at the gate :)21:22
smcginniseharney: I think there is an infra zuul linter job.21:23
smcginniseharney: I wonder how hard it would be to reuse that.21:23
eharneysmcginnis: there should be one somewhere, it's done for project-config etc21:23
*** dave-mccowan has quit IRC21:25
eharneysmcginnis: if that runs, it should address your -1 on the patch above it ^21:26
*** pchavva has quit IRC21:33
openstackgerritSean McGinnis proposed openstack/cinder-tempest-plugin master: Correct plugin test location reporting  https://review.openstack.org/53766421:38
smcginniseharney: So now we're going to run two tempests against it?21:39
eharneysmcginnis: should we not?21:39
smcginniseharney: Well, like I commented on there, LIO is currently non-voting on cinder, so it seems odd to me to have that a voting job on cinder-tempest-plugin.21:39
smcginniseharney: But maybe that should change.21:40
eharneysmcginnis: the LIO job should be voting on cinder, since it's the only thing testing barbican21:40
smcginniseharney: Do we expect there to be functional differences that we need to verify in the tempest plugin?21:40
smcginnisI guess I would rather just have LIO then, and not both unless there is significant differences between the two when it comes to these tests. (which I didn't think there were)21:41
*** threestrands has joined #openstack-cinder21:41
eharneyme too21:42
*** threestrands_ has joined #openstack-cinder21:44
*** threestrands_ has quit IRC21:44
*** threestrands_ has joined #openstack-cinder21:44
*** threestrands has quit IRC21:47
openstackgerritOpenStack Release Bot proposed openstack/python-brick-cinderclient-ext master: Update reno for stable/queens  https://review.openstack.org/53767421:52
*** eharney has quit IRC21:54
openstackgerritSean McGinnis proposed openstack/python-brick-cinderclient-ext master: Update reno for stable/queens  https://review.openstack.org/53767421:55
*** kien-ha has joined #openstack-cinder21:55
*** eharney has joined #openstack-cinder21:56
*** dustins has quit IRC21:56
*** eharney has quit IRC21:59
*** belmoreira has quit IRC22:00
openstackgerritOpenStack Release Bot proposed openstack/python-cinderclient master: Update reno for stable/queens  https://review.openstack.org/53768322:02
*** mvenesio has quit IRC22:05
*** tpsilva has quit IRC22:07
openstackgerritAlan Bishop proposed openstack/cinder master: Fix how backups handle encryption key IDs  https://review.openstack.org/53746222:08
*** rmcall has quit IRC22:09
*** dustins has joined #openstack-cinder22:10
*** dustins_ has joined #openstack-cinder22:12
*** abishop has quit IRC22:13
*** rcernin has joined #openstack-cinder22:14
*** dustins has quit IRC22:15
*** kien-ha has quit IRC22:15
*** yangyapeng has quit IRC22:16
*** crose has quit IRC22:20
*** rcernin has quit IRC22:29
*** chkumar246 has joined #openstack-cinder22:30
*** rcernin has joined #openstack-cinder22:31
*** chandankumar has quit IRC22:32
*** chkumar246 has quit IRC22:35
*** chkumar246 has joined #openstack-cinder22:36
*** erlon has joined #openstack-cinder22:43
*** edmondsw has quit IRC22:47
*** abishop has joined #openstack-cinder22:49
openstackgerritSteve Noyes proposed openstack/cinder master: Fix mis-named context object  https://review.openstack.org/53764322:50
ildikovstvnoyes: sorry, I don't have wifi/data most of the day :/22:51
*** itlinux has quit IRC22:51
stvnoyesno problem. Hopefully that will make things less stressful for you :-)22:52
ildikovstvnoyes: well, let's not talk about my stress level... :)22:53
ildikovstvnoyes: jgriffith can fill you in how many times I freak out like on a weekly sometimes daily basis... :/ :)22:54
*** dustins_ has quit IRC22:57
openstackgerritMatt Riedemann proposed openstack/cinder master: Fix mis-named context object  https://review.openstack.org/53764322:59
*** yangyapeng has joined #openstack-cinder23:01
*** itlinux has joined #openstack-cinder23:03
*** eharney has joined #openstack-cinder23:08
*** itlinux has quit IRC23:23

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!