Friday, 2019-10-04

*** brinzhang has joined #openstack-cinder00:01
*** markvoelker has joined #openstack-cinder00:31
*** markvoelker has quit IRC00:35
*** brinzhang_ has joined #openstack-cinder01:00
*** brinzhang has quit IRC01:04
openstackgerritMerged openstack/cinder stable/rocky: Don't allow retype to encrypted+multiattach type  https://review.opendev.org/68205101:06
openstackgerritMerged openstack/cinder stable/queens: Fix DetachedInstanceError for VolumeAttachment  https://review.opendev.org/68597801:06
*** brinzhang has joined #openstack-cinder01:25
*** markvoelker has joined #openstack-cinder01:26
*** dklyle has quit IRC01:27
*** dklyle has joined #openstack-cinder01:28
*** brinzhang_ has quit IRC01:29
*** markvoelker has quit IRC01:30
*** markvoelker has joined #openstack-cinder01:54
*** brinzhang_ has joined #openstack-cinder01:54
*** brinzhang has quit IRC01:58
*** markvoelker has quit IRC01:58
*** markvoelker has joined #openstack-cinder02:05
*** whoami-rajat has joined #openstack-cinder02:09
*** markvoelker has quit IRC02:35
*** markvoelker has joined #openstack-cinder03:06
*** dviroel has quit IRC03:15
openstackgerritBrian Rosmaita proposed openstack/cinder stable/train: NetApp SolidFire: Fix replication  https://review.opendev.org/68651903:20
*** markvoelker has quit IRC03:36
*** markvoelker has joined #openstack-cinder03:36
*** markvoelker has quit IRC04:07
*** gouthamr has quit IRC04:15
*** gouthamr has joined #openstack-cinder04:16
*** markvoelker has joined #openstack-cinder04:35
*** markvoelker has quit IRC04:39
openstackgerritMerged openstack/cinder stable/train: 3PAR: For Peer Persistence, add policy options in RCG  https://review.opendev.org/68646004:50
*** Luzi has joined #openstack-cinder04:58
openstackgerritMerged openstack/cinder stable/train: Add "service token" documentation  https://review.opendev.org/68628605:14
*** markvoelker has joined #openstack-cinder05:19
*** markvoelker has quit IRC05:24
*** markvoelker has joined #openstack-cinder05:28
*** markvoelker has quit IRC05:33
*** sahid has joined #openstack-cinder05:34
*** markvoelker has joined #openstack-cinder05:37
*** markvoelker has quit IRC05:42
*** markvoelker has joined #openstack-cinder05:47
*** markvoelker has quit IRC05:51
openstackgerritRajat Dhasmana proposed openstack/cinder stable/queens: Tests: Fix up migrate notify tests  https://review.opendev.org/68652305:54
*** markvoelker has joined #openstack-cinder05:56
*** markvoelker has quit IRC06:01
*** markvoelker has joined #openstack-cinder06:05
*** markvoelker has quit IRC06:10
*** markvoelker has joined #openstack-cinder06:14
*** markvoelker has quit IRC06:19
*** markvoelker has joined #openstack-cinder06:23
*** markvoelker has quit IRC06:28
*** markvoelker has joined #openstack-cinder06:33
*** spsurya has joined #openstack-cinder06:33
*** markvoelker has quit IRC06:38
*** markvoelker has joined #openstack-cinder06:42
openstackgerritOpenStack Proposal Bot proposed openstack/cinder stable/train: Imported Translations from Zanata  https://review.opendev.org/68652506:44
*** udesale has joined #openstack-cinder06:46
*** markvoelker has quit IRC06:46
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/68652606:48
*** brinzhang has joined #openstack-cinder06:50
*** markvoelker has joined #openstack-cinder06:52
*** brinzhang_ has quit IRC06:53
*** markvoelker has quit IRC06:56
*** markvoelker has joined #openstack-cinder07:01
*** markvoelker has quit IRC07:06
*** brinzhang_ has joined #openstack-cinder07:10
*** markvoelker has joined #openstack-cinder07:11
*** brinzhang has quit IRC07:13
*** pcaruana has joined #openstack-cinder07:15
*** markvoelker has quit IRC07:15
*** tosky has joined #openstack-cinder07:18
*** markvoelker has joined #openstack-cinder07:20
*** e0ne has joined #openstack-cinder07:22
*** e0ne has quit IRC07:22
*** markvoelker has quit IRC07:24
*** markvoelker has joined #openstack-cinder07:47
*** baojg has joined #openstack-cinder07:50
*** brinzhang has joined #openstack-cinder07:51
*** markvoelker has quit IRC07:52
*** brinzhang_ has quit IRC07:55
*** markvoelker has joined #openstack-cinder07:57
*** markvoelker has quit IRC08:01
*** brinzhang has quit IRC08:09
*** rcernin has quit IRC08:15
*** markvoelker has joined #openstack-cinder08:15
*** baojg has quit IRC08:17
*** markvoelker has quit IRC08:20
*** lpetrut has joined #openstack-cinder08:25
*** tkajinam has quit IRC08:27
*** ociuhandu has joined #openstack-cinder08:37
*** davidsha has joined #openstack-cinder08:38
*** markvoelker has joined #openstack-cinder08:43
*** martinkennelly has joined #openstack-cinder08:44
*** markvoelker has quit IRC08:48
*** ociuhandu has quit IRC08:54
*** ociuhandu has joined #openstack-cinder08:54
*** ociuhandu has quit IRC08:59
*** ociuhandu has joined #openstack-cinder08:59
*** e0ne has joined #openstack-cinder08:59
*** markvoelker has joined #openstack-cinder09:02
openstackgerritMerged openstack/cinder stable/queens: Add retry to LVM deactivation  https://review.opendev.org/68645509:06
*** markvoelker has quit IRC09:07
*** markvoelker has joined #openstack-cinder09:21
*** markvoelker has quit IRC09:25
*** markvoelker has joined #openstack-cinder09:39
*** markvoelker has quit IRC09:45
*** markvoelker has joined #openstack-cinder09:58
*** markvoelker has quit IRC10:02
*** markvoelker has joined #openstack-cinder10:07
*** markvoelker has quit IRC10:12
*** ociuhandu has quit IRC10:22
*** ociuhandu has joined #openstack-cinder10:22
*** markvoelker has joined #openstack-cinder10:35
*** markvoelker has quit IRC10:39
*** ociuhandu has quit IRC10:47
*** ociuhandu has joined #openstack-cinder10:48
*** ociuhandu has quit IRC10:52
*** ociuhandu has joined #openstack-cinder10:52
*** markvoelker has joined #openstack-cinder10:53
*** markvoelker has quit IRC10:57
*** markvoelker has joined #openstack-cinder11:02
*** markvoelker has quit IRC11:06
*** ociuhandu has quit IRC11:07
*** ociuhandu has joined #openstack-cinder11:08
*** markvoelker has joined #openstack-cinder11:11
*** ociuhandu has quit IRC11:12
*** ociuhandu has joined #openstack-cinder11:14
*** markvoelker has quit IRC11:16
*** ociuhandu has quit IRC11:18
*** markvoelker has joined #openstack-cinder11:21
*** ociuhandu has joined #openstack-cinder11:25
*** markvoelker has quit IRC11:25
*** markvoelker has joined #openstack-cinder11:39
*** markvoelker has quit IRC11:44
openstackgerritSean McGinnis proposed openstack/cinder stable/pike: Add retry to LVM deactivation  https://review.opendev.org/68657211:46
*** markvoelker has joined #openstack-cinder11:48
*** markvoelker has quit IRC11:49
*** markvoelker has joined #openstack-cinder11:49
*** ociuhandu has quit IRC11:55
*** ociuhandu has joined #openstack-cinder11:57
*** ociuhandu has quit IRC11:59
*** ociuhandu has joined #openstack-cinder11:59
*** enriquetaso has joined #openstack-cinder12:01
*** ociuhandu has quit IRC12:03
*** ociuhandu has joined #openstack-cinder12:04
hemnait's like friday and stuff12:07
*** kukacz has joined #openstack-cinder12:09
*** ociuhandu has quit IRC12:09
*** spsurya has quit IRC12:10
openstackgerritMerged openstack/cinder stable/train: Imported Translations from Zanata  https://review.opendev.org/68652512:10
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/68652612:10
*** lseki has joined #openstack-cinder12:14
*** udesale has quit IRC12:16
*** dviroel has joined #openstack-cinder12:16
*** udesale has joined #openstack-cinder12:16
*** davidsha has quit IRC12:22
*** ociuhandu has joined #openstack-cinder12:34
*** whoami-rajat has quit IRC12:38
*** ociuhandu has quit IRC12:40
*** markvoelker has quit IRC12:41
*** whoami-rajat has joined #openstack-cinder12:42
*** kukacz has quit IRC12:44
*** kukacz has joined #openstack-cinder12:45
*** vesper11 has quit IRC12:53
*** vesper11 has joined #openstack-cinder12:54
*** Luzi has quit IRC12:55
*** udesale has quit IRC12:56
*** markvoelker has joined #openstack-cinder12:58
rosmaitasmcginnis: thanks for hitting the translations ... if you have a minute, can you take a look at https://review.opendev.org/#/c/686525/ ?12:59
*** vesper11 has quit IRC12:59
*** ociuhandu has joined #openstack-cinder13:03
*** ociuhandu has quit IRC13:09
*** psachin has joined #openstack-cinder13:09
*** raghavendrat has joined #openstack-cinder13:15
*** dpawlik has quit IRC13:25
*** vesper11 has joined #openstack-cinder13:27
raghavendratThanks whoami-rajat: rosmaita: @smcginnis: and jungleboyj:13:29
raghavendratReally appreciate quick response from all reviewers and helping me to get patch into master & backporting to Train before deadline.13:29
rosmaitaraghavendrat: hth13:29
*** vesper11 has quit IRC13:31
*** vesper11 has joined #openstack-cinder13:32
*** raghavendrat has left #openstack-cinder13:35
whoami-rajatraghavendrat: np13:35
jungleboyjThanks for getting the translations in.13:50
rosmaitajungleboyj: how much of a faux pas would it be for me to +2A https://review.opendev.org/#/c/686519/ ?  kind of want to get it moving through the gate13:54
jungleboyjrosmaita: I can do it.  It has two cores on it and Rajat.13:55
jungleboyjDone.  :-)13:56
*** sapd1_x has joined #openstack-cinder13:57
*** gmann_afk is now known as gmann13:58
rosmaitaty!13:59
jungleboyjNP.13:59
rosmaitayou were saying yesterday how smooth things were going -- i completely missed that that patch had not been included in RC-114:01
rosmaitaonly noticed when trying to resolve why the other patch couldn't be cherry-picked without conflict resolution14:01
rosmaitabut i think we are back on track now14:01
*** ociuhandu has joined #openstack-cinder14:02
*** davidsha has joined #openstack-cinder14:04
jungleboyjrosmaita:  :-)  No worries.  I would still say it is going smoothly.  :-)14:30
rosmaita:D14:30
jungleboyjsmcginnis: hemna geguileo  Please check out Matt's note to the mailing list about removing the legacy volume attachment code:  http://lists.openstack.org/pipermail/openstack-discuss/2019-October/009946.html14:33
jungleboyjReading through it, it makes sense to me but you guys are the ones who really understand the attach/detach flow.14:34
geguileojungleboyj: I started reading it this morning and got a little headache  XDXD14:34
jungleboyjgeguileo: That wasn't what I wanted to hear. ;-)14:34
geguileojungleboyj: I left it for Monday ;-)14:35
jungleboyjFair enough.14:35
jungleboyjMy brain is feeling quite fried today too.14:35
jungleboyjrosmaita:  Not sure if you saw this on the mailing list:  https://etherpad.openstack.org/p/meet-the-project-leaders14:38
rosmaitait's frieday14:38
jungleboyjI am planning to show up as the out-going PTL.14:39
whoami-rajatgeguileo: Hey, i'm not really familiar with the grenade job ``legacy-grenade-dsvm-cinder-mn-sub-volbak`` , it's failing on my patch [1] saying to look at the relevant logs and I went through almost all of them, could you point me to the exact one?14:39
whoami-rajat[1] https://review.opendev.org/#/c/685914/14:39
rosmaitajungleboyj: i don't get in until late monday afternoon, so probably no monday activities for me14:40
*** ociuhandu has quit IRC14:51
*** ociuhandu has joined #openstack-cinder14:52
*** ociuhandu has quit IRC14:56
geguileowhoami-rajat: I'm not familiar with it eaither, let me have a look15:03
*** ociuhandu has joined #openstack-cinder15:04
geguileowhoami-rajat: looks like a setup problem https://zuul.opendev.org/t/openstack/build/386fdd25dfb64302a730e3048682e711/log/job-output.txt#648315:05
geguileowhoami-rajat: when running the upgrade15:06
toskyhttps://62e3ff9516339057f9a1-e9e87ed9377c28b498dc984ec5d212e1.ssl.cf5.rackcdn.com/685914/1/check/legacy-grenade-dsvm-cinder-mn-sub-volbak/386fdd2/logs/grenade.sh.txt.gz15:06
toskyhere the full log ^^15:06
tosky2019-10-01 16:38:58.327 | Error during database migration: Migration cannot continue until all volumes have been migrated to the api database. Please run `cinder-manage db online_data_migrations' There are still 2 untyped volumes unmigrated.15:06
geguileowhoami-rajat: here's the upgrade issue: https://zuul.opendev.org/t/openstack/build/386fdd25dfb64302a730e3048682e711/log/logs/grenade.sh.txt.gz#6211815:08
*** ociuhandu has quit IRC15:09
whoami-rajattosky: geguileo Thanks a lot, one query, doesn't all the TRAIN online migrations should run before running the db migration script in master? (i think we discussed this previously to include this migration in U release)15:14
toskyuhm, maybe some fixes are needed for grenade at this point15:16
toskynot sure15:16
geguileowhoami-rajat: yes that is correct, the way an upgrade works is first you run the online migrations to ensure everything has been migrated, then you go and run the DB migrations15:17
*** donnyd is now known as donnyd_afk15:19
hemnahrmm, I'll have to look at the api call for attachment delete and see what it does15:19
hemnait's been ages15:19
toskygeguileo, whoami-rajat: maybe grenade needs a fix; compare what cinder does:15:20
toskyhttps://opendev.org/openstack/grenade/src/branch/master/projects/70_cinder/upgrade.sh#L6015:20
toskyand what nova does: https://opendev.org/openstack/grenade/src/branch/master/projects/60_nova/upgrade.sh#L7215:21
toskynova can also call online_data_migrations15:21
toskybut then I'd need a cinder upgrade expert15:21
*** ociuhandu has joined #openstack-cinder15:23
whoami-rajattosky: the cinder upgrade script doesn't contain online migrations15:26
hemnajungleboyj: geguileo so looking at Matt's question regarding the v3 attachments api15:27
hemnaI think it's fine15:27
geguileothanks  :-)15:27
hemnaattachment_delete in the cinder v3 api does everything that we would need to have the backend unexport the volume and have the state move to available15:27
geguileoI'll try to have a proper read on Monday15:27
hemnawhich I think is what his question was15:27
hemnaI just followed the api call chain and it looks good afaik15:28
hemnathe only caveat is that if a host connector isn't provided, then it results in a backend noop15:30
hemnaso the backend could still have an export15:30
*** dviroel has quit IRC15:36
openstackgerritMerged openstack/cinder stable/train: NetApp SolidFire: Fix replication  https://review.opendev.org/68651915:39
*** ociuhandu has quit IRC15:47
*** ociuhandu has joined #openstack-cinder15:48
openstackgerritBrian Rosmaita proposed openstack/cinder stable/train: NetApp SolidFire: Add options for replication mode  https://review.opendev.org/68675415:49
rosmaitalast change for RC-2 ^^15:51
*** ociuhandu has quit IRC15:56
*** donnyd_afk is now known as donnyd16:01
*** dviroel has joined #openstack-cinder16:03
hemnajungleboyj: geguileo I replied to the thread16:03
hemnait'll be a problem I think16:04
hemnaw/o a connector passed in and no connector in the attachment entry.16:04
hemnait'll bypass the driver's chance to terminate_connection/remove_export16:04
hemnaleaving dangling exports16:04
*** tosky has quit IRC16:05
*** sahid has quit IRC16:13
*** sapd1_x has quit IRC16:21
*** martinkennelly has quit IRC16:22
*** davidsha has quit IRC16:28
*** e0ne has quit IRC16:32
*** lpetrut has quit IRC16:41
*** dpawlik has joined #openstack-cinder16:44
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add migration to make volume_type_id non nullable  https://review.opendev.org/68591416:59
whoami-rajattosky geguileo: maybe this will fix it https://review.opendev.org/#/c/686772/ ^^17:00
whoami-rajatThanks for the help!17:00
*** ociuhandu has joined #openstack-cinder17:09
*** lpetrut has joined #openstack-cinder17:15
jungleboyjhemna:  Thanks for looking.17:24
*** psachin has quit IRC17:36
*** lpetrut has quit IRC17:49
*** Swimingly has joined #openstack-cinder17:55
*** martinkennelly has joined #openstack-cinder18:00
*** jdillaman has joined #openstack-cinder18:00
*** ociuhandu has quit IRC18:08
*** ociuhandu has joined #openstack-cinder18:08
*** martinkennelly has quit IRC18:50
*** pcaruana has quit IRC18:51
*** ociuhandu has quit IRC18:51
*** ociuhandu has joined #openstack-cinder18:52
*** ociuhandu has quit IRC18:56
*** CeeMac has quit IRC19:41
*** e0ne has joined #openstack-cinder19:50
*** thgcorrea has joined #openstack-cinder19:53
*** markvoelker has quit IRC19:55
thgcorreais someone available to look at this backport: https://review.opendev.org/#/c/686754/ ?  We've got CI check, thanks19:58
*** e0ne has quit IRC20:09
simondodsleyAnyone going to the shanghai summit?20:24
jungleboyjsimondodsley:  I will be there.20:40
simondodsleyat least one person to drink with then :)20:40
jungleboyjhemna: smcginnis  Can you put another +2 on thgcorrea 's backport?20:40
*** dpawlik has quit IRC20:54
*** enriquetaso has quit IRC21:01
*** whoami-rajat has quit IRC21:18
*** tosky has joined #openstack-cinder21:31
openstackgerritMerged openstack/cinder master: LINSTOR driver update for LINSTOR v0.9.12 with REST API  https://review.opendev.org/66969021:46
woojayrosmaita: smcginnis: Thank you gents.  Only took 5 reboots at the gate, but it's in now.22:00
*** tosky has quit IRC23:39

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