Monday, 2019-08-19

*** rcernin has joined #openstack-cinder00:02
*** ircuser-1 has joined #openstack-cinder01:09
*** spsurya has joined #openstack-cinder01:09
*** markvoelker has joined #openstack-cinder01:20
*** markvoelker has quit IRC01:25
*** Kuirong has joined #openstack-cinder01:57
*** boxiang has joined #openstack-cinder01:59
*** redrobot has quit IRC02:23
*** baojg has quit IRC02:25
*** Guest90568 has joined #openstack-cinder02:29
*** Guest90568 is now known as redrobot02:32
*** brinzhang has joined #openstack-cinder02:48
Ben78whoami-rajat: I restarted the host and I lost the VG. I see there are 2 files at /etc/lvm/backup: stack-volumes-lvmdriver-1 and stack-volumes-default. Do you think I can use them to restore the lost VG?02:52
*** boxiang has quit IRC03:12
*** boxiang has joined #openstack-cinder03:13
*** Ben78 has quit IRC03:13
whoami-rajatBen78 : can you try `vgcfgrestore vgcfgrestore stack-volumes-lvmdriver-1` and `vgcfgrestore stack-volumes-default`03:23
*** dave-mccowan has quit IRC03:46
*** dklyle has joined #openstack-cinder03:46
*** dklyle has quit IRC03:52
*** boxiang has quit IRC04:02
*** boxiang has joined #openstack-cinder04:02
*** enriquetaso has quit IRC04:10
*** udesale has joined #openstack-cinder04:30
*** abhishekk has joined #openstack-cinder04:33
*** dpawlik has joined #openstack-cinder05:00
*** trident has quit IRC05:06
*** ociuhandu has joined #openstack-cinder05:15
*** trident has joined #openstack-cinder05:15
*** trident has quit IRC05:21
*** markvoelker has joined #openstack-cinder05:23
*** markvoelker has quit IRC05:27
*** trident has joined #openstack-cinder05:27
*** ociuhandu has quit IRC05:36
*** ociuhandu has joined #openstack-cinder06:02
*** sahid has joined #openstack-cinder06:08
*** n-saito has joined #openstack-cinder06:12
*** threestrands has quit IRC06:34
*** threestrands has joined #openstack-cinder06:35
*** e0ne has joined #openstack-cinder06:37
*** e0ne has quit IRC06:38
*** ociuhandu has quit IRC06:53
*** ociuhandu has joined #openstack-cinder07:00
*** udesale has quit IRC07:05
*** udesale has joined #openstack-cinder07:06
*** ociuhandu has quit IRC07:16
*** rcernin has quit IRC07:17
*** threestrands has quit IRC07:19
*** lemko has joined #openstack-cinder07:21
*** dpawlik has quit IRC07:34
*** takamatsu has joined #openstack-cinder07:37
*** udesale has quit IRC07:43
*** udesale has joined #openstack-cinder07:44
*** trident has quit IRC08:11
*** tkajinam has quit IRC08:18
*** trident has joined #openstack-cinder08:19
*** ociuhandu has joined #openstack-cinder08:37
*** ociuhandu has quit IRC08:41
*** e0ne has joined #openstack-cinder08:42
*** dpawlik has joined #openstack-cinder09:03
*** CeeMac has joined #openstack-cinder09:18
*** davidsha has joined #openstack-cinder09:20
*** markvoelker has joined #openstack-cinder09:35
*** markvoelker has quit IRC09:40
*** ociuhandu has joined #openstack-cinder10:15
*** ociuhandu has quit IRC10:20
*** abhishekk has quit IRC10:26
*** udesale has quit IRC11:15
*** tesseract has joined #openstack-cinder11:15
*** n-saito has quit IRC11:29
*** boxiang has quit IRC11:32
*** zhubx has joined #openstack-cinder11:32
*** markvoelker has joined #openstack-cinder11:36
*** markvoelker has quit IRC11:41
*** markvoelker has joined #openstack-cinder11:57
*** lemko has quit IRC12:00
*** henriqueof has joined #openstack-cinder12:02
*** ociuhandu has joined #openstack-cinder12:07
*** ociuhandu has quit IRC12:11
*** dviroel_ has joined #openstack-cinder12:12
*** raghavendrat has joined #openstack-cinder12:29
*** spsurya has quit IRC12:43
*** mriedem has joined #openstack-cinder12:57
*** esker has joined #openstack-cinder13:00
*** eharney has joined #openstack-cinder13:07
hemnamornin13:18
smcginnisHave fun this week.13:26
*** spatel has joined #openstack-cinder13:28
*** dklyle has joined #openstack-cinder13:29
*** ociuhandu has joined #openstack-cinder13:30
hemnahey guys, can we get some love on some doc reviews here: https://review.opendev.org/#/c/675525/ and https://review.opendev.org/#/c/675526/13:30
smcginnisHmm, not actually cherry-picked right, but probably OK for doc updates.13:31
hemnathanks man13:31
hemnasmcginnis: want me to cherry pick new patches for those?13:32
smcginnisNah, that's OK. Not like a bug we need to track and make sure we've covered everything.13:33
hemnaok coolio13:33
*** spatel has quit IRC13:33
*** ociuhandu has quit IRC13:34
hemnaman already 90F outside at 9:30am13:35
hemnableh13:35
hemnasummer finally showed up13:35
smcginnisHeh, maybe I'm not sad I'm missing things. :D13:35
hemnawx has been crazy in NC the last few days13:35
hemnabig storms, big winds, trees falling n stuffs13:35
*** ociuhandu has joined #openstack-cinder13:58
*** carloss has joined #openstack-cinder14:02
*** ociuhandu has quit IRC14:03
openstackgerritMerged openstack/cinder master: Fixing 404's and broken links  https://review.opendev.org/67551414:04
openstackgerritMerged openstack/cinder stable/stein: Fixing 404's and broken links  https://review.opendev.org/67552514:04
openstackgerritMerged openstack/cinder stable/rocky: Fixing 404's and broken links  https://review.opendev.org/67552614:04
*** esker has quit IRC14:08
*** ociuhandu has joined #openstack-cinder14:11
*** jdillaman has joined #openstack-cinder14:13
*** dpawlik has quit IRC14:20
*** KeithMnemonic has joined #openstack-cinder14:25
*** eharney_ has joined #openstack-cinder14:25
*** eharney has quit IRC14:26
openstackgerritKeith Berger proposed openstack/cinder stable/rocky: 3PAR: Provide new option to specify NSP for single path attachments  https://review.opendev.org/67721514:27
KeithMnemonicjust getting this one ready once the stein one merges14:27
KeithMnemonichttps://review.opendev.org/#/c/676728/1 is ready for WF+1 if there is a core around to review14:28
openstackgerritEric Harney proposed openstack/python-cinderclient master: Autonegotiate API version for shell  https://review.opendev.org/66443014:31
openstackgerritSean McGinnis proposed openstack/cinder master: Revert "3PAR: Provide new option to specify NSP for single path attachments"  https://review.opendev.org/67721714:36
KeithMnemonicsmcginnis thanks! as i did not do the master patch that was HPE, i will let them decide what to do14:38
smcginnisSorry it wasn't an easy one. Tempted to let it go at first, but then there were too many issues. Big one being the release note for me.14:39
smcginnisIf we're going to actually allow backporting the addition of a config option, then I think the release note at least needs to be good.14:39
*** markvoelker has quit IRC14:42
KeithMnemonicsmcginnis no problem, it is surprising how many customers choose not not use FC Zone manager, even though HPE Recommends it14:51
KeithMnemonicfor their FC 3par connections14:52
KeithMnemonicand this results when not doing zoning correctly14:52
*** esker has joined #openstack-cinder14:58
hemnasmcginnis: you are reverting a patch because of an extra space, a \ character and a minor update to a release note?!15:00
hemnathat seems quite a bit harsh.  why not allow me to put up a follow up patch to address those minor issues instead.15:01
smcginnishemna: Not when we want to be able to backport it.15:02
hemnacouldn't we just cherry pick both patches?15:03
hemnaI mean that patch was up for several months w/o reviews15:03
*** markvoelker has joined #openstack-cinder15:03
hemnaseems a bit harsh to revert a patch that fixes major issues, just because of a space and \15:03
smcginnisIt's not the space and \15:05
hemnathe release note change is minor too15:06
hemnabut whatever15:06
smcginnisIt's that along with a bad release note, no driver version bump, and a bunch of little things that should have been addressed in the first place.15:06
hemnaand we all had a say about that during the initial review process, and got no feedback about that then.15:06
hemnathat's quite a bit frustrating15:07
openstackgerritSean McGinnis proposed openstack/cinder master: 3PAR: Add config for NSP single path attach  https://review.opendev.org/67723215:10
hemna:(15:12
*** brinzhang has quit IRC15:15
*** brinzhang has joined #openstack-cinder15:16
*** brinzhang has quit IRC15:16
hemnaso, can we make sure we slam this one through today?15:16
*** brinzhang has joined #openstack-cinder15:16
hemnaI'll ninja the revert as soon as it hits +1 zuul15:16
*** brinzhang has quit IRC15:18
*** brinzhang has joined #openstack-cinder15:18
openstackgerritMerged openstack/cinder master: Tighten unit test dict assertions  https://review.opendev.org/67538715:19
*** brinzhang has quit IRC15:20
openstackgerritMerged openstack/cinder master: Tests: Combine similar encrypted volume tests w/ ddt  https://review.opendev.org/67644315:20
*** eharney_ is now known as eharney15:20
*** brinzhang has joined #openstack-cinder15:20
*** ociuhandu has quit IRC15:37
*** ociuhandu has joined #openstack-cinder15:51
*** henriqueof has quit IRC16:09
*** esker has quit IRC16:29
*** esker has joined #openstack-cinder16:29
*** tesseract has quit IRC16:35
*** Ben78 has joined #openstack-cinder16:36
*** my_nd has joined #openstack-cinder16:45
*** davidsha has quit IRC16:45
*** e0ne has quit IRC17:10
*** enriquetaso has joined #openstack-cinder17:13
*** ociuhandu_ has joined #openstack-cinder17:16
*** henriqueof has joined #openstack-cinder17:17
*** ociuhandu has quit IRC17:19
*** sahid has quit IRC17:19
*** ociuhandu_ has quit IRC17:21
*** esker has quit IRC17:24
*** ociuhandu has joined #openstack-cinder17:24
*** hedvig_01 has joined #openstack-cinder17:27
*** ociuhandu has quit IRC17:29
hedvig_01Hi @smcginnis: SnapshotManageAdminTest and VolumeManageAdminTest used to be skipped earlier. They expect drivers to implement manage unmanage vol snapshot API which are not mandatory. Our CI tests are failing these  2 tests which were skipped. Are these mandatory now?17:40
hedvig_01http://os-logserver.hedviginc.com/refs/changes/99/675499/3/console.html.gz  It's using python3 and the custom branch pushed to openstack for review.17:42
smcginnishedvig_01: Looks like cinder_branch is commented out in local.conf: http://os-logserver.hedviginc.com/refs/changes/99/675499/3/logs/local.conf.txt.gz17:55
smcginnishedvig_01: Those tests should only run if you are setting the tempest flag to run them.17:55
smcginnishttps://opendev.org/openstack/tempest/src/branch/master/tempest/api/volume/admin/test_volume_manage.py#L3217:55
smcginnishttps://opendev.org/openstack/tempest/src/branch/master/tempest/api/volume/admin/test_snapshot_manage.py#L4117:55
smcginnisDefaults to false: https://opendev.org/openstack/tempest/src/branch/master/tempest/config.py#L92117:55
hedvig_01yes, but we are fetching the right branch17:55
smcginnishedvig_01: Outside of devstack?17:56
*** esker has joined #openstack-cinder17:57
smcginnishedvig_01: Looks like you are enabling those tests under the [volume-feature-enabled] section in tempest.conf: http://os-logserver.hedviginc.com/refs/changes/99/675499/3/logs/tempest_conf.txt.gz17:57
hedvig_01in the pre_test_hook, we are fetching the latest branch https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers17:57
smcginnishedvig_01: I thought we talked about that already. Please stop using the pre_test_hook. That's only for before your driver has merged to master.17:58
smcginnishedvig_01: It should now just operate like a normal CI and just use the patch that needs to be tested without manipulating it in any other way.17:59
hedvig_01Agreed. When we tried the CINDER_BRANCH, it did not point to the branch. so we reverted the changes17:59
smcginnisOtherwise you are not actually testing the patch.17:59
smcginnishedvig_01: Then you must have missed something. That's the correct way to do it.17:59
openstackgerritThiago Correa proposed openstack/cinder master: NetApp SolidFire: Fix replication  https://review.opendev.org/67563017:59
hedvig_01okay. In the link I sent earlier, there isa section "How do I configure DevStack so my Driver Passes Tempest?" Would that be the ideal way then ?18:00
hedvig_01https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers18:00
smcginnishedvig_01: That looks right, but the point is it needs to run the code to be tested and run the tests against that. Otherwise the testing isn't valid.18:01
hedvig_01Okay Thank you for your time.18:03
*** Ben78 has quit IRC18:10
*** e0ne has joined #openstack-cinder18:17
potssmcginnis hemna jungleboyj: hi folks, the dothill->seagate driver patch has passed all the CIs, with all the suggested changes--can we go ahead and merge it? https://review.opendev.org/#/c/671195/2018:35
*** strigazi has quit IRC18:37
*** strigazi has joined #openstack-cinder18:38
*** spatel has joined #openstack-cinder18:43
*** spatel has quit IRC18:48
*** ociuhandu has joined #openstack-cinder18:48
*** ociuhandu has quit IRC18:53
*** hedvig_01 has quit IRC19:12
openstackgerritAndreas Jaeger proposed openstack/cinder stable/rocky: Remove experimental openSUSE 42.3 job  https://review.opendev.org/67727419:17
openstackgerritAndreas Jaeger proposed openstack/cinder stable/queens: Remove experimental openSUSE 42.3 job  https://review.opendev.org/67727519:19
*** enriquetaso has quit IRC19:20
openstackgerritAndreas Jaeger proposed openstack/cinder stable/pike: Remove experimental openSUSE 42.3 job  https://review.opendev.org/67727719:20
my_ndHi there, I have issues in creating incremental backups in cinder. I got this error  IndexError: list index out of range  (more info https://pastebin.com/wRBzYSeY ) did anyone see this error before?. I am using GlusterNFS as volume backend and NFS as backup backend in Openstack Pike19:20
*** eharney has quit IRC19:26
*** mriedem has quit IRC19:29
*** mriedem has joined #openstack-cinder19:30
*** brinzhang_ has joined #openstack-cinder20:06
*** brinzhang has quit IRC20:09
*** e0ne has quit IRC20:11
*** eharney has joined #openstack-cinder20:14
*** e0ne has joined #openstack-cinder20:21
*** esker has quit IRC20:27
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729220:30
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729220:34
*** mmethot has quit IRC20:47
*** mmethot has joined #openstack-cinder20:47
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729220:50
*** esker has joined #openstack-cinder20:58
*** brinzhang has joined #openstack-cinder21:07
*** brinzhang_ has quit IRC21:10
*** eharney has quit IRC21:18
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: WIP: Add pdf documentation build in tox  https://review.opendev.org/67730121:26
*** markvoelker has quit IRC21:36
*** esker has quit IRC21:41
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick master: WIP Add pdf documentation build in tox  https://review.opendev.org/67730421:47
*** esker has joined #openstack-cinder22:03
*** trident has quit IRC22:03
*** esker has quit IRC22:04
*** trident has joined #openstack-cinder22:11
*** mriedem has quit IRC22:12
*** carloss has quit IRC22:22
*** dave-mccowan has joined #openstack-cinder22:25
*** dave-mccowan has quit IRC22:32
*** my_nd has quit IRC22:45
*** KeithMnemonic has quit IRC22:49
*** rcernin has joined #openstack-cinder22:57
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729223:03
*** KeithMnemonic has joined #openstack-cinder23:14
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729223:17
*** tkajinam has joined #openstack-cinder23:27
*** e0ne has quit IRC23:40
*** markvoelker has joined #openstack-cinder23:48

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