Wednesday, 2020-06-17

« Tuesday, 2020-06-16 Index Thursday, 2020-06-18 »
*** n-saito has joined #openstack-cinder00:01
*** boxiang_ has joined #openstack-cinder00:15
*** hamalq has quit IRC00:15
*** brinzhang has joined #openstack-cinder00:16
*** boxiang has quit IRC00:16
*** Liang__ has joined #openstack-cinder01:15
*** spatel has joined #openstack-cinder01:21
*** lvbin02 has joined #openstack-cinder01:31
*** lvbin01 has quit IRC01:32
*** lvbin02 is now known as lvbin0101:32
*** lvbin02 has joined #openstack-cinder01:43
*** lvbin01 has quit IRC01:44
*** lvbin02 is now known as lvbin0101:44
*** TusharTgite has joined #openstack-cinder01:50
*** Lucas_Gray has quit IRC02:12
*** psachin has joined #openstack-cinder02:48
*** rcernin has quit IRC02:49
*** rcernin has joined #openstack-cinder02:54
*** TusharTgite has quit IRC03:03
*** TusharTgite has joined #openstack-cinder03:07
*** rcernin has quit IRC03:08
*** boxiang_ has quit IRC03:13
*** boxiang has joined #openstack-cinder03:13
*** eharney has quit IRC03:30
*** psachin has quit IRC03:31
*** psachin has joined #openstack-cinder03:32
*** rcernin has joined #openstack-cinder03:44
*** rcernin has quit IRC03:45
*** rcernin has joined #openstack-cinder03:50
*** eharney has joined #openstack-cinder03:52
*** ykarel|away is now known as ykarel04:08
*** brinzhang0 has joined #openstack-cinder04:11
*** brinzhang has quit IRC04:14
*** Liang__ has quit IRC04:19
*** hamalq has joined #openstack-cinder04:22
*** Liang__ has joined #openstack-cinder04:23
*** hamalq has quit IRC04:23
*** vishakha has joined #openstack-cinder04:24
*** markvoelker has joined #openstack-cinder04:27
*** markvoelker has quit IRC04:33
*** vishalmanchanda has joined #openstack-cinder04:42
*** hamalq has joined #openstack-cinder04:44
*** hamalq has quit IRC04:49
*** andrebeltrami has quit IRC04:49
*** abdysn has joined #openstack-cinder05:04
*** hamalq has joined #openstack-cinder05:21
*** hamalq has quit IRC05:26
*** threestrands has joined #openstack-cinder05:27
*** spatel has quit IRC05:29
*** threestrands has quit IRC05:33
*** udesale has joined #openstack-cinder05:33
*** m75abrams has joined #openstack-cinder06:00
openstackgerritXuan Yandong proposed openstack/cinder-specs master: Support revert any snapshot to the volume  https://review.opendev.org/73611106:39
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/73611206:42
*** ircuser-1 has joined #openstack-cinder06:55
*** hamalq has joined #openstack-cinder06:58
*** hamalq has quit IRC07:02
*** psachin has quit IRC07:15
*** psachin has joined #openstack-cinder07:17
*** tosky has joined #openstack-cinder07:36
*** hamalq has joined #openstack-cinder07:43
*** rcernin has quit IRC07:47
*** hamalq has quit IRC07:48
openstackgerritXuan Yandong proposed openstack/cinder-specs master: Support revert any snapshot to the volume  https://review.opendev.org/73611107:52
*** Liang__ has quit IRC08:01
*** Liang__ has joined #openstack-cinder08:02
*** ykarel is now known as ykarel|lunch08:13
*** e0ne has joined #openstack-cinder08:15
*** hamalq has joined #openstack-cinder08:16
*** martinkennelly has joined #openstack-cinder08:16
*** hamalq_ has joined #openstack-cinder08:18
*** hamalq has quit IRC08:21
*** hamalq_ has quit IRC08:23
*** Lucas_Gray has joined #openstack-cinder08:24
*** hamalq has joined #openstack-cinder08:34
*** Lucas_Gray has quit IRC08:35
*** Lucas_Gray has joined #openstack-cinder08:36
*** hamalq has quit IRC08:38
*** priteau has joined #openstack-cinder08:49
*** ociuhandu has quit IRC08:51
*** tkajinam has quit IRC08:57
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/73611209:02
*** e0ne has quit IRC09:02
*** ykarel|lunch is now known as ykarel09:03
*** e0ne has joined #openstack-cinder09:08
*** Xuchu has joined #openstack-cinder09:09
*** Liang__ has quit IRC09:33
openstackgerritSean McGinnis proposed openstack/cinder master: Drop one more use of mock lib  https://review.opendev.org/73614309:34
*** yaawang_ has quit IRC09:40
*** e0ne has quit IRC09:47
*** e0ne_ has joined #openstack-cinder09:47
*** e0ne_ has quit IRC09:51
*** e0ne has joined #openstack-cinder09:52
*** n-saito has quit IRC10:01
*** yaawang_ has joined #openstack-cinder10:17
*** hamalq has joined #openstack-cinder10:20
*** hamalq has quit IRC10:25
*** TusharTgite has quit IRC10:26
walshh_Hi all, is anyone available to review the following documentation patch https://review.opendev.org/#/c/722806/.  Thank you10:30
*** psachin has quit IRC10:31
*** Xuchu has quit IRC10:36
*** hamalq has joined #openstack-cinder10:37
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add privsep support  https://review.opendev.org/73593510:40
*** hamalq has quit IRC10:42
*** e0ne has quit IRC10:42
*** e0ne has joined #openstack-cinder10:43
*** e0ne has quit IRC10:44
*** e0ne has joined #openstack-cinder10:45
*** e0ne has quit IRC10:46
*** e0ne has joined #openstack-cinder10:46
*** Lucas_Gray has quit IRC10:49
*** Xuchu has joined #openstack-cinder10:58
*** psachin has joined #openstack-cinder11:02
*** ociuhandu has joined #openstack-cinder11:12
openstackgerritGorka Eguileor proposed openstack/cinder-specs master: Default volume type overrides  https://review.opendev.org/73355511:21
geguileowhoami-rajat: ^ updated, and changed from tenant to project like you suggested11:22
*** martinkennelly has quit IRC11:35
*** martinkennelly has joined #openstack-cinder11:35
*** tosky has quit IRC11:56
*** psachin has quit IRC11:58
*** boxiang_ has joined #openstack-cinder11:58
*** tosky has joined #openstack-cinder11:59
*** martinkennelly has quit IRC11:59
*** psachin has joined #openstack-cinder11:59
*** martinkennelly has joined #openstack-cinder12:00
*** boxiang has quit IRC12:02
*** Lucas_Gray has joined #openstack-cinder12:02
*** Lucas_Gray has quit IRC12:16
*** Lucas_Gray has joined #openstack-cinder12:19
whoami-rajatthanks a lot geguileo12:23
geguileothank your for the review12:23
*** Wryhder has joined #openstack-cinder12:23
*** andrebeltrami has joined #openstack-cinder12:24
*** Lucas_Gray has quit IRC12:24
*** martinkennelly has quit IRC12:24
*** martinkennelly has joined #openstack-cinder12:28
*** Wryhder has quit IRC12:28
*** jmlowe has quit IRC12:29
*** spatel has joined #openstack-cinder12:31
*** jmlowe has joined #openstack-cinder12:32
*** Lucas_Gray has joined #openstack-cinder12:32
*** spatel has quit IRC12:36
*** martinkennelly has quit IRC12:58
*** martinkennelly has joined #openstack-cinder12:58
*** martinkennelly has quit IRC13:01
openstackgerritRafael Weingärtner proposed openstack/cinder master: Introduces MV to add volume type ID in volumes details  https://review.opendev.org/66688613:07
*** jbalciunas has joined #openstack-cinder13:07
*** udesale_ has joined #openstack-cinder13:26
*** udesale has quit IRC13:28
*** e0ne has quit IRC13:37
*** e0ne has joined #openstack-cinder13:37
*** priteau has quit IRC13:40
*** CeeMac has joined #openstack-cinder13:44
*** kberger_ has joined #openstack-cinder13:45
*** kberger_ has quit IRC13:46
hemna_morning13:46
*** kberger_ has joined #openstack-cinder13:46
hemna_do you guys understand why we don't have the AZ in the request spec to the scheduler during retype ?13:47
hemna_I see in master there is an extraction of the AZ if it exists in the volume type, but not the original volume.13:47
hemna_I would think a retype should keep the volume in the same AZ13:47
*** e0ne has quit IRC13:48
*** e0ne_ has joined #openstack-cinder13:48
*** KeithMnemonic has quit IRC13:49
hemna_https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1738-L174313:49
hemna_that does not contain the AZ of the originating volume13:49
hemna_master has the ability to add the AZ if and only if it exists in the new volume type13:50
hemna_but it seems we should have the AZ in the request spec by default from the originating volume13:50
hemna_I'm seeing an error in my deployment where the scheduler is picking a backend from a different AZ than the AZ of the volume itself, and then nova complains that the new retyped volume is in a different AZ than the vm13:51
hemna_(this is for an attached volume retype)13:51
*** Liang__ has joined #openstack-cinder13:52
*** Liang__ is now known as LiangFang13:53
*** enriquetaso has joined #openstack-cinder13:55
*** lpetrut has joined #openstack-cinder13:56
*** priteau has joined #openstack-cinder13:56
*** LiangFang has quit IRC13:57
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:59
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso ^^13:59
enriquetasothanks!14:00
jungleboyjThanks.14:00
geguileorosmaita: thanks14:01
*** martinkennelly has joined #openstack-cinder14:03
*** throne82_ has joined #openstack-cinder14:04
*** abdysn has quit IRC14:07
*** vishakha has quit IRC14:12
*** rchurch has quit IRC14:14
*** enriquetaso has quit IRC14:15
*** rchurch has joined #openstack-cinder14:15
*** knomura has joined #openstack-cinder14:16
*** enriquetaso has joined #openstack-cinder14:17
openstackgerritSean McGinnis proposed openstack/cinder master: Add generated doc output path to doc8 ignore list  https://review.opendev.org/73623714:23
*** boxiang has joined #openstack-cinder14:30
*** boxiang_ has quit IRC14:32
*** psachin has quit IRC14:34
*** raghavendrat has joined #openstack-cinder14:40
raghavendrathi reviewers, this is regarding https://review.opendev.org/#/c/73083014:41
raghavendratzuul & CI have passed. it would be great if someone can have a look & share feedback. Thanks.14:42
*** m75abrams has quit IRC14:56
smcginnisCores - would be nice to get this through so we can get the requirements update for oslotest through - https://review.opendev.org/73614315:00
smcginnisWe have one more hidden use of mock.15:00
rosmaitasmcginnis: dang, i thought we got all those15:01
*** boxiang has quit IRC15:01
jungleboyjGood catch.  Thanks.15:01
*** senrique_ has joined #openstack-cinder15:02
*** boxiang has joined #openstack-cinder15:02
hemna_done15:02
smcginnisThanks!15:02
rosmaitafor anyone who wasn't here earlier, hemna_ 's question about the AZs is here: http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2020-06-17.log.html#t2020-06-17T13:46:4015:03
eharneysmcginnis: d'oh.. sorry for just writing that the other day :)15:03
hemna_There was a patch added a while back to add support for setting AZ in volume types and those get passed into the request spec15:03
hemna_but, by default there is no honoring the volume's existing AZ if the volume type doesn't contain any AZs specified15:04
e0ne_rosmaita, hemna_: sound like we need to fix it15:04
hemna_which results in the scheduler picking a different AZ15:04
hemna_and this fails especially bad for attached volumes15:04
hemna_https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1738-L174315:05
*** enriquetaso has quit IRC15:05
hemna_so if someone has the availability zone filter set in cinder.conf, it's useless15:05
hemna_for retype in this case15:05
hemna_is this a bug that's been in there forever, or was there some reasoning behind ignoring the AZ  during retype?15:06
smcginniseharney: What was that?15:07
eharneysmcginnis: mock.mock15:07
smcginniseharney: Oh, you saw that too?15:07
smcginnisLuckily an easy fix.15:07
smcginnisJust had to locate where it was being set. ;)15:08
hemna_ok I guess I'll just file a bug and put a patch up to fix it, if nobody objects15:09
rosmaitahemna_: i don15:09
rosmaitat think there was a reason for ignoring AZ during retype15:10
rosmaitai mean, i do not think that15:10
hemna_the existing AZ filter will still act the same way, if the volume type has AZ's specified those will be used first15:10
rosmaitai think that sounds reasonable15:10
*** knomura has quit IRC15:10
hemna_I think for the logest time, cinder just ignored AZ15:10
rosmaitai mean, i think your proposal sounds reasonable15:10
hemna_and that this might be an artifact of that15:10
rosmaitai agree15:10
rosmaitai think it was just an oversight15:11
hemna_ok cool, I just keep hitting it in our deployment, and get a boatload of exceptions and happened to see it15:11
*** Lucas_Gray has quit IRC15:12
*** knomura has joined #openstack-cinder15:13
hemna_https://bugs.launchpad.net/cinder/+bug/188392815:16
openstackLaunchpad bug 1883928 in Cinder "Cinder ignores AZ during retype" [Undecided,New] - Assigned to Walt Boring (walter-boring)15:16
*** knomura has quit IRC15:17
*** hamalq has joined #openstack-cinder15:17
*** hamalq_ has joined #openstack-cinder15:19
*** knomura has joined #openstack-cinder15:19
rosmaitajungleboyj: when you have a minute, lmk what you think about smcginnis's comment here: https://review.opendev.org/#/c/735989/6/releasenotes/notes/bug-1883654-cc069892496bd4a8.yaml@1815:20
*** hamalq has quit IRC15:23
*** knomura has quit IRC15:23
*** knomura has joined #openstack-cinder15:24
smcginnisrosmaita: I do think it's fine, it was just something that made me stop and think as I was reading through it.15:24
rosmaitasmcginnis: i just wanted to see if jungleboyj could think of a better way to say it15:25
rosmaitabecause you are right, it is a very broad statement!15:25
* jungleboyj looks.15:25
*** knomura_ has joined #openstack-cinder15:26
jungleboyjWe could scope it a little better by saying 'If you are running OpenStack on Python 3 you should  not be impacted by this bug.' ?15:27
jungleboyjThat way we are still leaving the decision to the Operator.15:27
hamalq_hi rosmaita: smcginnis: https://review.opendev.org/#/c/736059/ is this change Abandoned ?15:27
jungleboyjThat statement gave me pause too.  Didn't know what to say though.15:27
smcginnishamalq_: Yes, as I said, it needs to merge to more recent branches first, then be properly backported.15:27
hamalq_the code is different in rocky we cant do the backport15:28
rosmaitasmcginnis: it's a weird situation, because of the history of that connector, the original patch wasn't a backport either15:29
hamalq_from train15:29
*** knomura has quit IRC15:29
smcginnisWe still need to fix in the more recent branches first, then backport and modify as needed from there.15:29
rosmaitathat part i agree with15:29
hamalq_agree, do u mean train or another branch15:30
jungleboyjsmcginnis: ++15:30
rosmaitawe can't do rocky until train and stein have merged15:30
smcginnisWe need it to merge to train. Then once that is done, you can backport to stein, then to rocky, and so on.15:30
rosmaitai tested stein locally, it will be a clean backport from train15:30
hamalq_rosmaita: thats great news15:31
*** knomura_ has quit IRC15:31
hamalq_smcginnis: can u please re-open https://review.opendev.org/#/c/736059/ so i can update code review changes15:32
hamalq_can i do that?15:33
*** raghavendrat has quit IRC15:33
hamalq_am not sure how to do that :)15:33
smcginnishamalq_: It doesn't need to be reopened. The change has not merged to train or stein yet.15:34
hamalq_am just preparing for the change to come and then we can merge when ever u guys ready15:35
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/train: Fix "Remove VxFlex OS credentials" regression  https://review.opendev.org/73598915:36
rosmaitahamalq_: thanks, the gate is supposed to be working again, so hopefully we can get train and stein merged today15:37
*** e0ne_ has quit IRC15:40
hamalq_rosmaita: thanks ok i will replicate the changes u do in train for release notes and anything needed to rocky locally and wait for the merge15:40
smcginnisstein was the release before train.15:41
rosmaitahamalq_: rocky and queens are in "extended maintenance" mode, so we don't release from them any more15:41
rosmaitaso i don't think you need a release note with the rocky patch, but we will need to make sure the commit message is clear15:42
smcginnisIf the cherry-pick contains a release note, we can still backport that even if we aren't going to do a release of it.15:43
rosmaitasmcginnis: the difference between stein and rocky is so big, that i'm not sure it makes sense to cherry pick -- there's a file in stein that's not in rocky, and the config file is accessed a completely different way15:44
rosmaitathe code change is still basically a 1 liner15:45
hamalq_rosmaita: that true15:45
smcginnisWe still want the fix tracked through.15:45
rosmaitabut the unit test is a lot different15:45
*** e0ne has joined #openstack-cinder15:46
rosmaitacan we use the same changeid without a cherry-pick?15:46
rosmaitaeither that or a big "conflicts" statement in the commit message, i guess15:47
smcginnisYes, the conflicts statement should be included in the commit message.15:48
smcginnishttps://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes15:48
*** martinkennelly has quit IRC15:51
*** udesale_ has quit IRC15:53
hamalq_smcginnis: but even if we do that the conflict lines will not be there since the file does not exist am not sure though15:57
hamalq_how that will go15:57
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/train: Fix "Remove VxFlex OS credentials" regression  https://review.opendev.org/73598916:00
*** e0ne has quit IRC16:01
rosmaitahamalq_: as a test, you can try a cherry-pick of the train patch into rocky and see what happens16:02
hamalq_rosmaita: will do16:02
hamalq_am trying git cherry-pick -x Ie2db587c3bc379acd53cfd449788d171ae58dec5 fatal: bad revision do u need till it merges?16:15
*** senrique_ has quit IRC16:15
hamalq_that should not be the case16:15
smcginnishamalq_: Looks like you are using the Gerrit change ID. That actually takes the git commit hash.16:16
hamalq_that true16:17
hamalq_and the commit hash will be available when it gets merged right?16:21
smcginnishamalq_: It should be the commit hash of the current train patch.16:25
smcginnishamalq_: Keep in mind that this would only be a test. It needs to ultimately be cherry-picked from stein.16:25
openstackgerritMerged openstack/cinder master: 3PAR: Fix live migration  https://review.opendev.org/73083016:26
rosmaitahamalq_: you can do git review -d 73598916:27
rosmaitathat will give you a local branch with the latest patch in it16:27
hamalq_great thanks16:27
rosmaitathen you can cherry-pick -x 12d252db9cb9deffea3c87b86ea71b3013d93892 into your local stable/rocky16:27
rosmaitabut remember what smcginnis said, this is only a test!16:28
*** enriquetaso has joined #openstack-cinder16:28
openstackgerritSean McGinnis proposed openstack/os-brick master: Add doc linting to pep8 target  https://review.opendev.org/72829016:32
*** ykarel is now known as ykarel|away16:33
openstackgerritMerged openstack/cinder master: Drop one more use of mock lib  https://review.opendev.org/73614316:34
kberger_geguileo: does your 3par fix effect earlier releases as well. ie do i need to try and patch that to Rocky for our customers on 3par16:34
geguileokberger_: I believe the 3PAR live migration has always been broken16:35
geguileoso we'll need to backport the fix as far as we can :-(16:35
kberger_nice, thanks were you planning on doing the backports?16:35
geguileokberger_: I can do them, but if someone else does, then I can do the reviewing16:37
*** ociuhandu_ has joined #openstack-cinder16:38
*** ociuhandu has quit IRC16:42
*** ociuhandu_ has quit IRC16:43
hamalq_smcginnis: rosmaita: no conflict its just try adding new files16:46
hamalq_deleted by us:      os_brick/privileged/scaleio.py16:46
hamalq_new file:   releasenotes/notes/bug-1883654-cc069892496bd4a8.yaml16:47
*** ociuhandu has joined #openstack-cinder16:47
hamalq_i think back-porting does not make sense in this case since it does not fix anything16:48
smcginnisIt tracks the fix. Please follow stable policy procedures as documented.16:48
hamalq_smcginnis: oh ok if that is the case i will wait till this get merged and add my fix after cherry pick is that ok?16:51
smcginnishamalq_: Yep, that's what we're saying. Just need to be patient.16:51
hamalq_am not good at that but i will try :)16:51
smcginnis;)16:51
*** ociuhandu has quit IRC16:51
*** baba-tunde has quit IRC17:02
*** lpetrut has quit IRC17:02
*** pcaruana has quit IRC17:03
*** pcaruana has joined #openstack-cinder17:03
*** e0ne has joined #openstack-cinder17:03
*** baba-tunde has joined #openstack-cinder17:08
*** spatel has joined #openstack-cinder17:16
openstackgerritKeith Berger proposed openstack/cinder stable/ussuri: 3PAR: Fix live migration  https://review.opendev.org/73630217:16
openstackgerritKeith Berger proposed openstack/cinder stable/train: 3PAR: Fix live migration  https://review.opendev.org/73630317:17
kberger_i did the first two, the others have a merge conflict but i can look at them once these clear17:17
*** kberger_ has quit IRC17:20
smcginnisPeople are just trolling me now I think. :)17:20
*** knomura has joined #openstack-cinder17:25
*** KeithMnemonic has joined #openstack-cinder17:25
*** baba-tunde has quit IRC17:26
KeithMnemonicsmcginnis: sorry for queuing up two there. TBH though some projects are fine queuing them up so sometime i forgot who allows and who does not17:26
*** KeithMnemonic has quit IRC17:27
*** knomura_ has joined #openstack-cinder17:27
smcginnisWith my stable team hat on, I can say none should.17:27
smcginnisUnless they are some of the few that are not asserting stable:follows-policy17:27
smcginnishttps://governance.openstack.org/tc/reference/tags/stable_follows-policy.html#application-to-current-deliverables17:28
*** knomura__ has joined #openstack-cinder17:29
*** knomura has quit IRC17:29
*** baba-tunde has joined #openstack-cinder17:30
*** knomura__ has quit IRC17:31
*** knomura has joined #openstack-cinder17:31
*** baba-tunde has quit IRC17:31
*** knomura_ has quit IRC17:31
*** baba-tunde has joined #openstack-cinder17:32
*** knomura_ has joined #openstack-cinder17:35
*** knomura has quit IRC17:35
*** knomura_ has quit IRC17:35
*** knomura has joined #openstack-cinder17:36
*** KeithMnemonic has joined #openstack-cinder17:37
*** knomura has quit IRC17:37
*** knomura has joined #openstack-cinder17:38
*** knomura_ has joined #openstack-cinder17:42
*** knomura has quit IRC17:42
*** e0ne has quit IRC17:43
*** knomura_ has quit IRC17:43
*** knomura has joined #openstack-cinder17:43
*** KeithMnemonic has quit IRC17:43
*** knomura has quit IRC17:44
*** knomura has joined #openstack-cinder17:44
*** knomura has quit IRC17:49
*** knomura has joined #openstack-cinder17:51
*** knomura has quit IRC17:54
*** knomura has joined #openstack-cinder17:55
*** knomura has quit IRC18:00
*** throne82_ has quit IRC18:13
*** spatel has quit IRC18:37
*** spatel has joined #openstack-cinder18:39
*** hamalq_ has quit IRC18:50
*** hamalq has joined #openstack-cinder18:53
*** hamalq_ has joined #openstack-cinder18:53
*** hamalq has quit IRC18:57
*** spatel has quit IRC19:08
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Add test for check dependencies between incr backups  https://review.opendev.org/65277119:18
*** knomura has joined #openstack-cinder19:22
*** knomura has quit IRC19:27
*** knomura has joined #openstack-cinder19:38
*** vishalmanchanda has quit IRC19:38
*** knomura has quit IRC19:39
*** knomura has joined #openstack-cinder19:39
*** spatel has joined #openstack-cinder19:40
*** knomura has quit IRC19:44
*** knomura has joined #openstack-cinder19:44
*** knomura has quit IRC19:45
*** knomura_ has joined #openstack-cinder19:47
*** knomura_ has quit IRC19:48
*** knomura_ has joined #openstack-cinder19:50
*** knomura_ has quit IRC19:51
*** knomura has joined #openstack-cinder19:52
*** ociuhandu has joined #openstack-cinder19:52
*** knomura_ has joined #openstack-cinder19:53
*** knomura has quit IRC19:56
*** knomura_ has quit IRC19:57
*** knomura has joined #openstack-cinder20:01
*** knomura has quit IRC20:03
*** CeeMac has quit IRC20:04
*** knomura_ has joined #openstack-cinder20:05
*** knomura_ has quit IRC20:06
*** knomura has joined #openstack-cinder20:06
*** senrique_ has joined #openstack-cinder20:08
*** knomura has quit IRC20:08
*** knomura has joined #openstack-cinder20:08
*** senrique_ has quit IRC20:09
*** enriquetaso has quit IRC20:10
*** Lucas_Gray has joined #openstack-cinder20:11
*** tobberydberg_ has joined #openstack-cinder20:12
*** tosky_ has joined #openstack-cinder20:12
*** knomura has quit IRC20:13
*** tosky has quit IRC20:13
*** tosky_ is now known as tosky20:13
*** simondodsley_ has joined #openstack-cinder20:14
*** CeeMac has joined #openstack-cinder20:15
*** brinzhang0 has quit IRC20:18
*** brinzhang0 has joined #openstack-cinder20:18
*** irclogbot_2 has quit IRC20:18
*** tobberydberg has quit IRC20:18
*** simondodsley has quit IRC20:18
*** masayukig has quit IRC20:18
*** simondodsley_ is now known as simondodsley20:18
*** brinzhang0 has quit IRC20:19
*** irclogbot_0 has joined #openstack-cinder20:20
*** masayukig has joined #openstack-cinder20:24
*** enriquetaso has joined #openstack-cinder20:26
rosmaitaneed a cinder-stable-maint core to take a look at the os-brick patch for stable/train that we talked about at the meeting today20:28
rosmaitahttps://review.opendev.org/#/c/735989/820:28
rosmaitae0ne eharney geguileo jungleboyj ^^20:28
*** lxkong has quit IRC20:35
*** dviroel has quit IRC20:39
*** knikolla has quit IRC20:40
*** knikolla has joined #openstack-cinder20:42
*** patrickeast has quit IRC20:43
*** dviroel has joined #openstack-cinder20:44
*** sri_ has quit IRC20:44
*** ociuhandu has quit IRC20:48
*** sri_ has joined #openstack-cinder20:48
*** PrinzElvis has quit IRC20:49
*** priteau has quit IRC20:49
*** thgcorrea has quit IRC20:50
*** knikolla has quit IRC20:52
*** PrinzElvis has joined #openstack-cinder20:52
*** patrickeast has joined #openstack-cinder20:53
*** dviroel has quit IRC20:55
*** sri_ has quit IRC20:56
*** dviroel has joined #openstack-cinder20:57
*** PrinzElvis has quit IRC20:57
*** patrickeast has quit IRC21:00
*** patrickeast has joined #openstack-cinder21:01
*** sri_ has joined #openstack-cinder21:01
*** dviroel has quit IRC21:03
*** e0ne has joined #openstack-cinder21:03
*** patrickeast has quit IRC21:05
*** sri_ has quit IRC21:06
*** lxkong has joined #openstack-cinder21:09
*** dviroel has joined #openstack-cinder21:09
*** sri_ has joined #openstack-cinder21:11
openstackgerritSean McGinnis proposed openstack/os-brick master: Add doc linting to pep8 target  https://review.opendev.org/72829021:11
*** noonedeadpunk has quit IRC21:12
*** spatel has quit IRC21:13
*** lxkong has quit IRC21:16
*** dviroel has quit IRC21:17
*** ociuhandu has joined #openstack-cinder21:19
openstackgerritSean McGinnis proposed openstack/os-brick master: Add doc linting to pep8 target  https://review.opendev.org/72829021:22
*** ociuhandu has quit IRC21:24
*** sri_ has quit IRC21:26
*** sri_ has joined #openstack-cinder21:29
*** lxkong has joined #openstack-cinder21:34
*** PrinzElvis has joined #openstack-cinder21:35
*** spatel has joined #openstack-cinder21:38
*** knikolla has joined #openstack-cinder21:40
*** patrickeast has joined #openstack-cinder21:40
*** dviroel has joined #openstack-cinder21:41
*** spatel has quit IRC21:43
*** spatel has joined #openstack-cinder21:45
*** tosky has quit IRC21:56
*** tosky has joined #openstack-cinder21:56
*** spatel has quit IRC22:02
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Cleanups  https://review.opendev.org/73447122:05
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Stop to use the __future__ module.  https://review.opendev.org/73291622:05
*** e0ne has quit IRC22:19
*** spatel has joined #openstack-cinder22:29
*** spatel has quit IRC22:32
*** knomura has joined #openstack-cinder22:35
*** knomura has quit IRC22:39
openstackgerritMerged openstack/os-brick stable/train: Fix "Remove VxFlex OS credentials" regression  https://review.opendev.org/73598922:40
*** rcernin has joined #openstack-cinder22:42
openstackgerrithamza proposed openstack/os-brick stable/stein: Fix "Remove VxFlex OS credentials" regression  https://review.opendev.org/73639522:47
hamalq_smcginnis: ^^22:48
smcginnishamalq_: Thanks!22:48
hamalq_any time22:49
hamalq_smcginnis: can implent to rocky from train or should i wait for stein?22:50
smcginnishamalq_: Wait for stein, then cherry-pick from stein to stable/rocky to pick up the cherry-pick info in the commit message footer.22:51
hamalq_ok cool22:52
*** tkajinam has joined #openstack-cinder22:54
hamalq_smcginnis: sorry for the too much question but it seems for stein its failing the CI (i think an related errors) should i update something22:59
*** Lucas_Gray has quit IRC23:07
*** tosky has quit IRC23:09
*** hoonetorg has quit IRC23:10
*** Lucas_Gray has joined #openstack-cinder23:22
*** hoonetorg has joined #openstack-cinder23:23
*** CeeMac has quit IRC23:34
smcginnishamalq_: Things look happy so far.23:47
smcginnishamalq_: You (mostly) only need to pay attention to the CI results posted by Zuul.23:48
smcginnisI do see some third party CI failures on the patch, but those are likely issues with those vendors' CI systems, not the patch.23:48
smcginnisThey historically have not been very good about running tests right on stable branches.23:48
smcginnishamalq_: According to https://zuul.opendev.org/t/openstack/status you have approximately 1 hour yet before the jobs are expected to finish.23:49
hamalq_thats good to know so i just have to run re-check untill they pass23:50
smcginnishamalq_: So far it does not look like you will need to recheck.23:50
hamalq_cool great23:50
*** andrebeltrami has quit IRC23:53
*** n-saito has joined #openstack-cinder23:54
« Tuesday, 2020-06-16 Index Thursday, 2020-06-18 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!