Tuesday, 2019-08-20

*** markvoelker has quit IRC00:01
*** brinzhang has quit IRC00:12
*** brinzhang has joined #openstack-cinder00:13
*** altlogbot_2 has quit IRC00:46
*** altlogbot_2 has joined #openstack-cinder00:47
*** dviroel_ has quit IRC01:42
gmannjungleboyj: smcginnis RE on cinder driver: NEC driver team is working to migrate the driver on py3 and soon will update you on that.01:49
*** markvoelker has joined #openstack-cinder01:51
*** markvoelker has quit IRC01:55
*** n-saito has joined #openstack-cinder02:12
jungleboyjgmann:  Thanks for the update.  Have they updated the Wiki I put together?02:29
*** spsurya has joined #openstack-cinder03:03
gmannjungleboyj: not yet. i will ask driver maintainer team to update the same.03:09
jungleboyjgmann:  Thank you!03:10
gmannjungleboyj: you mean this etherpad right? https://etherpad.openstack.org/p/cinder-py3-ci-review03:14
*** jdillaman has quit IRC03:17
*** ruffian_sheep has joined #openstack-cinder03:19
*** ruffian_sheep has quit IRC03:29
*** gkadam has joined #openstack-cinder03:33
*** gkadam has quit IRC03:40
*** psachin has joined #openstack-cinder03:43
*** udesale has joined #openstack-cinder04:02
*** tkajinam has quit IRC04:09
*** tkajinam has joined #openstack-cinder04:16
*** tkajinam has quit IRC04:17
*** tkajinam has joined #openstack-cinder04:18
*** stakeda has joined #openstack-cinder04:59
*** markvoelker has joined #openstack-cinder05:55
*** markvoelker has quit IRC06:05
*** markvoelker has joined #openstack-cinder06:06
*** sahid has joined #openstack-cinder06:08
*** raghavendrat has left #openstack-cinder06:09
*** markvoelker has quit IRC06:10
*** raghavendrat has joined #openstack-cinder06:11
raghavendrathi whoami-rajat: are you online?06:11
whoami-rajatraghavendrat: Hi06:11
raghavendratwhenever you have some time, can you please have a look at: https://review.opendev.org/#/c/674796/06:12
whoami-rajatraghavendrat: sure06:14
*** udesale has quit IRC06:33
*** udesale has joined #openstack-cinder06:34
*** rcernin has quit IRC06:35
*** udesale has quit IRC07:01
*** udesale has joined #openstack-cinder07:02
*** dpawlik has joined #openstack-cinder07:08
*** trident has quit IRC07:16
*** trident has joined #openstack-cinder07:24
*** psachin has quit IRC07:25
*** ianychoi has quit IRC07:46
*** ianychoi has joined #openstack-cinder07:47
*** ociuhandu has joined #openstack-cinder07:58
*** ociuhandu has quit IRC07:59
*** ociuhandu has joined #openstack-cinder08:00
*** e0ne has joined #openstack-cinder08:26
*** tkajinam has quit IRC08:32
*** ruffian_sheep has joined #openstack-cinder08:50
whoami-rajatruffian_sheep: Hey08:51
ruffian_sheepwhoami-rajat:long time no see08:51
ruffian_sheepwhoami-rajat:Can you help me look at the driver?https://review.opendev.org/#/c/674156/08:52
ruffian_sheepwhoami-rajat:It seems that I have not waited for the follow-up feedback.08:52
whoami-rajatruffian_sheep:  oh yeah, i think my review comments are addressed but the major concerns were raised by hemna  and eharney , i will ask them to take a look at it.08:54
ruffian_sheepwhoami-rajat:At the moment, Sean McGinnis has doubts about the length of response to the CI environment. I have explained it above. I commented out the repetitive process and used the existing successful results to upload. I don't know if such behavior violates the requirements of the community.08:56
whoami-rajatruffian_sheep:  i'm not sure i understand the explanation09:06
whoami-rajatruffian_sheep: if a patch triggers the macrosan CI, the time taken to execute the tests isn't mentioned on gate?09:07
ruffian_sheepwhoami-rajat:Looking at the situation, Sean McGinnis thinks that the length of macrosan ci execution is too short. I explained that because I commented out the process of performing the tempest test again. Because I will perform temepst test verification before uploading the cinder driver, I have already had successful test results. I use this succ09:11
ruffian_sheepessful test result directly. In this case, the duration is mainly consumed in uploading log files. I can also let the ci environment perform the test again. This is no problem.09:11
*** davidsha has joined #openstack-cinder09:12
whoami-rajatruffian_sheep: oh, then there is an issue09:12
whoami-rajatruffian_sheep: if you update your driver with any change then your CI should automatically run all the tests again09:13
ruffian_sheepwhoami-rajat:I can also let him trigger automatic execution if needed. This is no problem. It is just that I think that there are already results of ready-made implementation, and I can save a lot of time.09:14
whoami-rajatruffian_sheep: also on every recheck or 'CI trigger comment' the CI should be running tests again09:14
*** rcernin has joined #openstack-cinder09:15
ruffian_sheepwhoami-rajat:The first few times may be that the network is not good and no information is received. I have not changed the ci environment.09:15
*** ociuhandu_ has joined #openstack-cinder09:15
*** ociuhandu_ has quit IRC09:16
ruffian_sheepwhoami-rajat:Then I will add another supplement to trigger the execution of ci by comment.?09:17
whoami-rajatruffian_sheep: i don't think time is a constraint for CI's and correctness is far more valuable than running time, if the CI fails the first time executing tests and we recheck then the tests should run again to make it pass, i'm not sure why are we manually intervening the CI process that should be automatic.09:18
whoami-rajatruffian_sheep: i think it is already triggered by run-MacroSAN Volume CI or recheck09:18
whoami-rajatso that's not an issue09:19
whoami-rajatdoes your CI run for patches that aren't your driver related?09:19
ruffian_sheepwhoami-rajat:Before I set the +1 comment for zuul, it will start executing for other drivers. In the middle of the period, there may be some problems with the network. I suspect that there is a problem with the environment, and I have cancelled it again. Only retained trigger run-xx-ci09:22
ruffian_sheeplike that09:25
whoami-rajatruffian_sheep: i think running your CI isn't mandatory for non-driver related patches but it sure shows that the CI is healthy09:27
whoami-rajatruffian_sheep: also what's the python version while running CI?09:27
ruffian_sheepThese are all asked once and for all. lollll09:30
whoami-rajatthen the only thing to fix is the rerunning of tests when triggered09:30
ruffian_sheepwhoami-rajat:This is no problem, I will adjust it. Refresh the feedback of ci again. Let him give instant content. Maybe you can look at the driver's place for modification, I can also deal with it.09:32
ruffian_sheepwhoami-rajat:This is originally consistent, but I think that there will be success in the debugging process, and it will take a little unnecessary time to execute it again. No problem, I can change it back.09:33
whoami-rajatruffian_sheep: great. then your CI is in perfect condition :)09:35
*** ociuhandu has quit IRC09:54
*** udesale has quit IRC10:16
*** udesale has joined #openstack-cinder10:17
*** ociuhandu has joined #openstack-cinder10:27
*** rcernin has quit IRC10:28
*** ociuhandu has quit IRC10:35
*** gkadam has joined #openstack-cinder10:38
*** dosaboy has quit IRC10:42
*** ganso has quit IRC10:42
*** tpsilva has quit IRC10:42
*** spatel has joined #openstack-cinder10:45
*** spatel has quit IRC10:50
whoami-rajatruffian_sheep:  one of my queries is unanswered in the review, https://review.opendev.org/#/c/612311/20/cinder/volume/drivers/macrosan/devop_client.py@9310:54
*** gkadam is now known as gkadam-afk11:00
*** tesseract has joined #openstack-cinder11:08
*** ociuhandu has joined #openstack-cinder11:08
*** ociuhandu has quit IRC11:09
*** ociuhandu has joined #openstack-cinder11:09
*** ioni has joined #openstack-cinder11:13
ionihello guys11:13
ionii'm trying to configure cinder volume to use a ceph cluster11:13
ionievery time i try to create a volume, cinder-volume timeouts and sadly, i don't know what service doesn't have acces la ceph mon11:13
hemnaioni can you run ceph -s11:14
hemnasee if your host can talk to the cluster11:15
ionii can11:15
hemnamight have to pass --id <keyring client name>11:15
ionialso i can create volumes into the volumes pool11:15
hemnaseems like a network problem between the c-vol host and the cluster/mon11:16
*** udesale has quit IRC11:18
*** dave-mccowan has joined #openstack-cinder11:20
raghavendratsorry for interrupting in between. Hi hemna: whenever you get time, can you please review: https://review.opendev.org/#/c/674796/11:20
raghavendratzuul & HPE Storage CI have passed11:20
*** sahid has quit IRC11:21
hemnaraghavendrat: you need to remember to add version # bump and history information in your patches11:21
hemnaespecially on bug fixes and new features11:21
hemnawe just reverted a 3par patch kinda due to that11:21
raghavendratyes. saw that. Sean McGinnis had review comments11:22
hemnacan you help with those ci failures so we can get 677232 merged today?11:22
raghavendratok. i will keep in mind about version & history info.11:23
hemnaI wanted to backport that NSP fix to pike, but can't due to the fixes needed11:23
raghavendrati will take a look at CI failure11:23
*** ganso has joined #openstack-cinder11:24
*** tpsilva has joined #openstack-cinder11:25
*** carloss has joined #openstack-cinder11:25
hemnaok thanks.   I'd like to get that NSP patch landed before the peer persistence patch11:26
hemnaso it can get cherry picked easily11:26
raghavendrati had a quick look at CI failure; it seems test_hpe3par.py may require slight modifications.11:28
*** dosaboy has joined #openstack-cinder11:28
raghavendratThree "AssertionError" are observed11:29
raghavendrati think i can clone using "Change-Id". is it possible for me for update test_hpe3par.py and submit it in same code review? i am not sure11:30
hemnayou can do that11:32
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729211:33
hemnagit review -d 67723211:33
hemnavi <file>11:33
hemnagit commit -a && git review11:33
*** markvoelker has joined #openstack-cinder11:35
*** sahid has joined #openstack-cinder11:36
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in template temp name  https://review.opendev.org/67729211:37
*** markvoelker has quit IRC11:40
*** gkadam-afk is now known as gkadam11:42
openstackgerritDavid Hill proposed openstack/cinder master: Use desination id instead of source id in snapshot temp name  https://review.opendev.org/67729211:51
*** ociuhandu has quit IRC11:54
*** ociuhandu has joined #openstack-cinder11:54
*** markvoelker has joined #openstack-cinder11:57
*** henriqueof has quit IRC12:09
*** ociuhandu has quit IRC12:24
ionihemna, https://paste.xinu.at/xgU6Nx/12:27
ionihemna i can use ceph -s from cinder volume12:27
smcginniswhoami-rajat, ruffian_sheep: That CI behavior is very wrong.12:32
*** gkadam has quit IRC12:32
smcginnisTime is not an issue. You should never be uploading the logs of something you had run on your own previously.12:32
smcginnisAnd yes, the CI does need to run against patches that do not modify the driver. The point of the CI is to make sure code changes elsewhere in Cinder do not inadvertently break drivers.12:33
*** dviroel_ has joined #openstack-cinder12:36
whoami-rajatsmcginnis: I have seen many CIs not running on non-driver changes. But yeah it is important. ruffian_sheep agreed to address both the concerns.12:39
*** dviroel_ is now known as dviroel12:39
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Add config for NSP single path attach  https://review.opendev.org/67723212:51
*** ruffian_sheep has quit IRC12:52
*** mriedem has joined #openstack-cinder12:53
raghavendrathi Sean McGinnis, to resolve zuul failure i updated two files and submitted patchset in https://review.opendev.org/#/c/67723212:53
smcginnisraghavendrat: Thanks!12:54
*** henriqueof has joined #openstack-cinder12:56
*** ociuhandu has joined #openstack-cinder12:57
*** ociuhandu has quit IRC12:58
*** ociuhandu has joined #openstack-cinder12:58
*** jdillaman has joined #openstack-cinder13:04
*** spatel has joined #openstack-cinder13:05
*** brinzhang has quit IRC13:21
*** enriquetaso has joined #openstack-cinder13:21
*** brinzhang has joined #openstack-cinder13:21
*** zhubx has quit IRC13:32
*** zhubx has joined #openstack-cinder13:33
*** udesale has joined #openstack-cinder13:35
*** rosmaita has left #openstack-cinder13:36
*** zhubx has quit IRC13:40
*** zhubx has joined #openstack-cinder13:40
*** ociuhandu has quit IRC13:41
*** eharney has joined #openstack-cinder13:42
*** udesale has quit IRC13:42
*** udesale has joined #openstack-cinder13:43
*** ociuhandu has joined #openstack-cinder13:46
*** udesale has quit IRC13:50
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: WIP: Add pdf documentation build in tox  https://review.opendev.org/67730113:51
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick master: WIP Add pdf documentation build in tox  https://review.opendev.org/67730413:52
*** henriqueof has quit IRC14:03
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Docs -  New features and supported software  https://review.opendev.org/67592914:05
openstackgerritIvaylo Mitev proposed openstack/cinder master: VMware VMDK: Initial size of volume from image  https://review.opendev.org/66989214:09
openstackgerritMichael McAleer proposed openstack/cinder master: PowerMax Driver - Debug Metadata Fix  https://review.opendev.org/67747314:11
*** dave-mccowan has quit IRC14:12
walshh_Anyone available to look at https://review.opendev.org/#/c/669866.  Thank you!14:18
*** jmlowe has joined #openstack-cinder14:27
*** dpawlik has quit IRC14:41
*** zhubx has quit IRC14:48
*** zhubx has joined #openstack-cinder14:48
*** zhubx has quit IRC14:50
*** zhubx has joined #openstack-cinder14:51
*** jgriffith has quit IRC14:52
*** jgriffith has joined #openstack-cinder14:52
*** jc__ has joined #openstack-cinder14:57
*** openstackstatus has quit IRC14:58
*** openstack has joined #openstack-cinder15:03
*** ChanServ sets mode: +o openstack15:03
*** spatel has quit IRC15:18
openstackgerritMerged openstack/cinder master: Revert "3PAR: Provide new option to specify NSP for single path attachments"  https://review.opendev.org/67721715:19
openstackgerritDavid Hill proposed openstack/cinder master: Use destination id instead of source id in snapshot temp name  https://review.opendev.org/67729215:20
*** spatel has joined #openstack-cinder15:24
*** spatel has quit IRC15:24
openstackgerritEric Harney proposed openstack/cinder master: Move some code out of utils.py  https://review.opendev.org/67749015:24
openstackgerritEric Harney proposed openstack/cinder master: Log exception info when objects fail to init  https://review.opendev.org/67749115:24
openstackgerritEric Harney proposed openstack/cinder master: Rename volume/utils.py to volume/volume_utils.py  https://review.opendev.org/67749215:24
*** spatel has joined #openstack-cinder15:28
*** spatel has quit IRC15:28
*** zhubx has quit IRC15:29
*** dave-mccowan has joined #openstack-cinder15:32
openstackgerritDavid Hill proposed openstack/cinder master: Use destination id instead of source id in snapshot temp name  https://review.opendev.org/67729215:33
*** baojg has joined #openstack-cinder15:36
*** baojg has quit IRC15:36
*** baojg has joined #openstack-cinder15:37
*** baojg has quit IRC15:37
*** hedvig_01 has quit IRC15:37
*** baojg has joined #openstack-cinder15:38
*** baojg has quit IRC15:38
*** baojg has joined #openstack-cinder15:39
*** baojg has quit IRC15:40
whoami-rajathemna: eharney could you take a look at macrosan driver patch15:48
whoami-rajatand check if your comments were addressed. https://review.opendev.org/#/c/674156/15:48
*** pvradu has quit IRC15:50
whoami-rajatgeguileo: Hey, we're getting a failure[1] for ipv6 job for cinder [2], my guess is that it may be a cause due to non-inclusion of your os-brick patch [3] . Could you take a look and if possible provide feedback on the ML thread [4]15:54
whoami-rajat[1] https://zuul.opendev.org/t/openstack/build/5b7b823d6faa4f5393b4c46d36e15d80/log/controller/logs/screen-n-cpu.txt.gz#273315:54
whoami-rajat[2] https://review.opendev.org/#/c/673266/15:54
whoami-rajat[3] https://review.opendev.org/#/c/667830/15:54
whoami-rajat[4] http://lists.openstack.org/pipermail/openstack-discuss/2019-August/008561.html15:54
*** udesale has joined #openstack-cinder16:00
potshemna: were you ok with the responses to your comment on the seagate driver patch? https://review.opendev.org/#/c/671195/16:09
* hemna looks16:11
hemnahrmm yah that's confusing as can be16:12
potshemna: i'd really like to get that merged since it includes the multiattach fix we need for every CI run16:12
potshemna i think it's only confusing to us, not any actual end users.16:15
potshemna Dot Hill never promoted the driver and we never heard of anyone using it16:16
*** markvoelker has quit IRC16:17
potshemna: thanks!16:18
*** e0ne has quit IRC16:25
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Docs -  New features and supported software  https://review.opendev.org/67592916:31
*** markvoelker has joined #openstack-cinder16:33
*** spsurya has quit IRC16:36
geguileowhoami-rajat: I'm looking at the logs and it's weird that the request for the login is missing... r:-??16:42
*** davidsha has quit IRC16:45
KeithMnemonicsmcginnis is your new patch ready for WF+1 or does it need some more reviews?  https://review.opendev.org/#/c/677232/ I abandoned the cherry-picks and will start fresh once this one is merged16:48
*** udesale has quit IRC17:04
*** tesseract has quit IRC17:10
*** hedvig_01 has joined #openstack-cinder17:14
*** ociuhandu has quit IRC17:18
*** sahid has quit IRC17:26
whoami-rajatgeguileo: oh, there seems to be a login failure log,17:32
whoami-rajatFailed to login iSCSI target iqn.2010-10.org.openstack:volume-d6478b7f-80a2-42d5-8c87-d56ca04dc9fe on portal 2001:4802:7801:103:be76:4eff:fe20:f60:326017:32
geguileowhoami-rajat: but the --login request to iscsiadm is not there17:32
geguileowhoami-rajat: anyway, I think I know the issue, I'm writing a WIP patch and I'll reply to the email thread like you requested17:32
whoami-rajatgeguileo: yep, couldn't find that17:33
whoami-rajatgeguileo: great! thanks for looking into it.17:33
*** boxiang has joined #openstack-cinder17:34
*** enriquetaso has quit IRC17:37
*** tesseract has joined #openstack-cinder17:45
*** enriquetaso has joined #openstack-cinder17:45
*** brinzhang_ has joined #openstack-cinder17:47
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix LVM IPv6 target portals  https://review.opendev.org/67752417:49
geguileowhoami-rajat: ^ I think that fixes it17:50
*** brinzhang has quit IRC17:50
jungleboyjtbarron:  Around?17:52
tbarronjungleboyj: yeah but my son is getting married this weekend and we have a house full of guests17:53
jungleboyjAh, so I am guessing we will not see you this week then?17:53
jungleboyjtbarron:  Congratulations by the way!17:53
tbarronjungleboyj: some might say that argues for me sneaking off to drink beer and play pool with you guys17:53
tbarronjungleboyj: but I'm not sure I can get away with it :)17:54
jungleboyjtbarron:  +++17:54
tbarronjungleboyj: thanks!17:54
jungleboyjBring the guests along!17:54
jungleboyjEarly wedding celebration.  ;-)17:54
tbarronjungleboyj: only if they write specs17:54
enriquetasotbarron++ \o/17:55
whoami-rajatgeguileo: oh, the problem was with the separator between the ip and port. Ah, thanks for the quick fix.17:55
tbarronenriquetaso: o/17:55
openstackgerritMerged openstack/cinder master: Create Seagate driver from dothill driver  https://review.opendev.org/67119518:06
geguileowhoami-rajat: I think it was, not 100% sure, I deducted it from the logs  XD18:06
whoami-rajatgeguileo: hah, that's why the W-1 ?18:07
whoami-rajatThat's for the UT I guess18:10
whoami-rajatHope it works :)18:10
*** jmlowe has quit IRC18:29
*** markvoelker has quit IRC18:34
*** enriquetaso has quit IRC18:51
*** jmlowe has joined #openstack-cinder18:55
*** enriquetaso has joined #openstack-cinder18:56
*** brinzhang_ has quit IRC19:18
*** e0ne has joined #openstack-cinder19:32
hedvig_01Hi @smcginnis: Thank you for helping us out. The CI is now running on python 3.7 and CINDER_BRANCH and CINDER_REPO are pointing to the right location. Ref:https://review.opendev.org/#/c/675499/  http://os-logserver.hedviginc.com/refs/changes/99/675499/3/19:41
hedvig_01Zuul has also given a +119:42
hedvig_01I guess that completes the tasks pending on our side. I'd be happy to address any if there's anything I've missed out on.19:43
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinderlib master: Add support to deal with inline svg images  https://review.opendev.org/67699719:49
*** mriedem has quit IRC19:50
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinderlib master: WIP Add pdf build support  https://review.opendev.org/67699719:51
*** mriedem has joined #openstack-cinder19:54
*** markvoelker has joined #openstack-cinder20:15
*** markvoelker has quit IRC20:16
*** markvoelker has joined #openstack-cinder20:16
*** boxiang has quit IRC20:21
*** zhubx has joined #openstack-cinder20:21
openstackgerritWalter A. Boring IV (hemna) proposed openstack/python-cinderclient master: WIP Add support for building pdf documentation  https://review.opendev.org/67755920:21
*** zhubx has quit IRC20:22
*** enriquetaso has quit IRC20:26
*** enriquetaso has joined #openstack-cinder20:35
*** eharney has quit IRC20:40
*** e0ne has quit IRC20:50
KeithMnemonicsmcginnis: did you see my message earlier about https://review.opendev.org/#/c/677232/ ?20:56
*** jrubenst has quit IRC21:07
*** jrubenst has joined #openstack-cinder21:13
*** jrubenst has quit IRC21:19
*** markvoelker has quit IRC21:22
*** tesseract has quit IRC21:32
*** markvoelker has joined #openstack-cinder21:35
*** markvoelker has quit IRC21:40
*** jrubenst has joined #openstack-cinder21:51
*** rcernin has joined #openstack-cinder22:00
*** trident has quit IRC22:02
*** markvoelker has joined #openstack-cinder22:10
*** trident has joined #openstack-cinder22:11
*** markvoelker has quit IRC22:15
*** carloss has quit IRC22:21
*** markvoelker has joined #openstack-cinder22:29
*** markvoelker has quit IRC22:48
*** tkajinam has joined #openstack-cinder22:55
*** mriedem has quit IRC23:05
*** markvoelker has joined #openstack-cinder23:12
*** markvoelker has quit IRC23:22
*** tejdeep has joined #openstack-cinder23:39
*** brinzhang has joined #openstack-cinder23:51

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