Friday, 2020-04-17

openstackgerritXuchu Jiang proposed openstack/cinder master: [Storwize] Add an option to specify iSCSI IP version  https://review.opendev.org/69863000:04
openstackgerritMerged openstack/cinder master: Fix missing print format in log messages  https://review.opendev.org/71976000:48
*** ociuhandu has joined #openstack-cinder01:01
*** ociuhandu has quit IRC01:05
*** Liang__ has joined #openstack-cinder01:10
openstackgerritNaoki Saito proposed openstack/cinder stable/queens: NEC driver: fix an undefined variable  https://review.opendev.org/72062501:22
*** yaawang has quit IRC01:46
*** yaawang has joined #openstack-cinder01:47
*** yaawang has quit IRC01:52
*** yaawang has joined #openstack-cinder01:53
*** sapd1 has quit IRC01:53
*** xinliang has joined #openstack-cinder01:55
*** sapd1 has joined #openstack-cinder02:31
*** rcernin has quit IRC03:10
openstackgerritMerged openstack/cinder master: Update hacking to 3.x release  https://review.opendev.org/71515403:11
*** psachin has joined #openstack-cinder03:17
*** rcernin has joined #openstack-cinder03:24
*** andrebeltrami has quit IRC03:39
*** yaawang has quit IRC03:40
*** yaawang has joined #openstack-cinder03:41
*** ociuhandu has joined #openstack-cinder03:44
xinlianghello, please help to review https://review.opendev.org/#/c/720423/ thanks.03:45
*** ociuhandu has quit IRC03:49
*** lvbin02 has joined #openstack-cinder04:05
*** xinliang has quit IRC04:06
*** lvbin01 has quit IRC04:08
*** lvbin02 is now known as lvbin0104:08
*** kevinz has joined #openstack-cinder04:42
*** ykarel|away is now known as ykarel04:51
*** vishakha has quit IRC05:10
*** mnasiadka has quit IRC05:10
*** thgcorrea has quit IRC05:10
*** Anticimex has quit IRC05:10
*** elod has quit IRC05:10
*** vishakha has joined #openstack-cinder05:15
*** mnasiadka has joined #openstack-cinder05:15
*** thgcorrea has joined #openstack-cinder05:15
*** Anticimex has joined #openstack-cinder05:15
*** elod has joined #openstack-cinder05:15
*** udesale has joined #openstack-cinder05:35
*** ociuhandu has joined #openstack-cinder05:37
*** ociuhandu has quit IRC05:47
*** lvbin02 has joined #openstack-cinder05:57
*** lvbin01 has quit IRC05:57
*** lvbin02 is now known as lvbin0105:57
*** udesale has quit IRC06:26
*** udesale has joined #openstack-cinder06:32
*** ociuhandu has joined #openstack-cinder06:37
*** ociuhandu has quit IRC06:43
*** xinliang has joined #openstack-cinder06:55
*** spatel has joined #openstack-cinder06:55
*** spatel has quit IRC07:00
*** ociuhandu has joined #openstack-cinder07:09
*** tosky has joined #openstack-cinder07:30
*** ociuhandu has quit IRC07:30
*** threestrands has quit IRC07:56
*** ociuhandu has joined #openstack-cinder08:04
*** e0ne has joined #openstack-cinder08:13
*** ociuhandu has quit IRC08:23
*** ociuhandu has joined #openstack-cinder08:23
*** ykarel is now known as ykarel|lunch08:26
*** e0ne_ has joined #openstack-cinder08:30
*** e0ne has quit IRC08:30
*** sapd1 has quit IRC08:41
*** ociuhandu has quit IRC08:45
*** snecklifter has joined #openstack-cinder08:48
*** snecker has joined #openstack-cinder08:48
*** tkajinam has quit IRC08:51
openstackgerritXuchu Jiang proposed openstack/cinder master: [Storwize] Add an option to specify iSCSI IP version  https://review.opendev.org/69863008:53
openstackgerritAndreas Jaeger proposed openstack/cinder master: Cleanup py27 support  https://review.opendev.org/72066808:57
openstackgerritHelen Walsh proposed openstack/cinder stable/queens: PowerMax Driver - Legacy volume not found  https://review.opendev.org/71503809:19
*** ykarel|lunch is now known as ykarel09:25
*** ociuhandu has joined #openstack-cinder09:25
*** ociuhandu has quit IRC09:43
*** ociuhandu has joined #openstack-cinder09:49
*** Liang__ has quit IRC09:52
openstackgerritwuqiongdan proposed openstack/cinder master: add docs and release note for sandstone iscsi driver.  https://review.opendev.org/72067809:59
*** n-saito has quit IRC09:59
*** e0ne_ has quit IRC10:21
*** e0ne has joined #openstack-cinder10:22
*** xinliang has quit IRC10:29
*** brinzhang has quit IRC10:35
*** ociuhandu has quit IRC10:48
*** ociuhandu has joined #openstack-cinder10:48
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix rbd attach on python3  https://review.opendev.org/72055310:51
*** ociuhandu has quit IRC10:52
*** spatel has joined #openstack-cinder10:57
openstackgerritRaghavendra Tilay proposed openstack/cinder master: HPE 3PAR: Support duplicated FQDN in network  https://review.opendev.org/71380310:59
*** spatel has quit IRC11:02
*** ociuhandu has joined #openstack-cinder11:23
*** e0ne has quit IRC11:25
*** e0ne has joined #openstack-cinder11:25
openstackgerritSean McGinnis proposed openstack/cinder master: Switch to py38 and update version metadata  https://review.opendev.org/72025511:31
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Handle the case when tempest fails  https://review.opendev.org/72021411:33
*** tkajinam has joined #openstack-cinder11:56
*** udesale_ has joined #openstack-cinder12:02
*** tkajinam has quit IRC12:04
*** udesale has quit IRC12:05
*** m5z has joined #openstack-cinder12:05
*** rcernin has quit IRC12:11
*** sapd1 has joined #openstack-cinder12:16
*** e0ne has quit IRC12:23
*** e0ne has joined #openstack-cinder12:23
*** enriquetaso has joined #openstack-cinder12:24
*** psachin has quit IRC12:26
*** ykarel is now known as ykarel|afk12:35
*** ykarel|afk is now known as ykarel12:39
hemna_mornin12:41
*** snecker has quit IRC12:46
*** snecklifter has quit IRC12:46
openstackgerritBrian Rosmaita proposed openstack/cinder master: Support X-Forwarded-Proto when generating pagination links  https://review.opendev.org/67090312:47
openstackgerritAndreas Jaeger proposed openstack/cinder master: Cleanup py27 support  https://review.opendev.org/72066812:47
*** ociuhandu has quit IRC12:48
*** ociuhandu has joined #openstack-cinder12:49
*** rishabhhpe has joined #openstack-cinder13:12
*** rishabhhpe has quit IRC13:22
*** jv has quit IRC13:28
*** ykarel is now known as ykarel|afk13:31
*** jv has joined #openstack-cinder13:35
openstackgerritDmitry Tantsur proposed openstack/python-cinderclient master: Remove Babel from requirements  https://review.opendev.org/72071313:40
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add glance image colocation feature for Virtuozzo driver  https://review.opendev.org/72072213:59
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - RDF status validation  https://review.opendev.org/72072314:00
toskytwo cinder-tempest-plugin reviews need just another review and approval: https://review.opendev.org/#/c/697291/ and https://review.opendev.org/#/c/652817/14:00
*** ociuhandu has quit IRC14:07
*** ociuhandu has joined #openstack-cinder14:11
whoami-rajattosky, since we've revert to snapshot feature enabled in rbd now https://review.opendev.org/#/c/710334/ , shouldn't we enable the revert test in the ceph job as well?14:11
toskywhoami-rajat: that review enabled it on the ceph job14:12
tosky... in the other repo14:13
toskyok, right14:13
toskyI need to handle that carefully14:14
gansoenriquetaso: Hi! do you know if there is any reason https://review.opendev.org/#/c/705703/ wasn't backported to queens?14:20
enriquetasoganso, I can't remember now... Is the incremental patch available for queens rosmaita ?14:23
rosmaitalooking14:25
*** e0ne has quit IRC14:28
rosmaitai don't remember there being a reason *not* to backport it, just make sure that https://review.opendev.org/#/c/709856/ is also proposed14:30
rosmaitathis isn't a guarantee that they'll be approved, though14:31
rosmaita:)14:31
enriquetasorosmaita++14:33
*** e0ne has joined #openstack-cinder14:34
gansorosmaita: thanks! I am actually working on the fix for that backup bug we discussed in the weekly meeting and I'm assessing the impact, of whether that fix will be necessary or not14:34
rosmaitaok14:34
openstackgerritSofia Enriquez proposed openstack/cinder master: Fix service-get-log to respect server filters  https://review.opendev.org/68600014:55
openstackgerritRodrigo Barbieri proposed openstack/cinder master: [WIP] Fix (a) for bug 1869746  https://review.opendev.org/72074114:55
openstackbug 1869746 in Cinder "Cinder throws error creating incremental backup from parent in another project" [Undecided,Confirmed] https://launchpad.net/bugs/186974614:55
smcginnisHuh, Verizon bought BlueJeans.15:01
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix rbd attach on python3  https://review.opendev.org/72055315:04
*** ykarel|afk is now known as ykarel15:06
openstackgerritSean McGinnis proposed openstack/cinder master: Update HACKING document to match current checks  https://review.opendev.org/71677415:09
*** ociuhandu has quit IRC15:10
*** andrebeltrami has joined #openstack-cinder15:12
*** ociuhandu has joined #openstack-cinder15:13
Xuchuwhoami-rajat: Morning, Could you help to review this patch if you are available? https://review.opendev.org/#/c/715766/15:25
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Enable volume_revert tests when supported  https://review.opendev.org/69729115:33
*** ociuhandu has quit IRC15:40
*** ociuhandu has joined #openstack-cinder15:42
*** ociuhandu has quit IRC15:47
*** dpawlik has quit IRC15:50
*** ykarel is now known as ykarel|away15:57
*** digvijay has joined #openstack-cinder15:59
digvijayjungleboyj, rosmaita: Thank you for blueprint of IBM GPFS Cinder driver.. we've our CI setup almost ready. .there are some glitches for hosting the logs but we've made good progress16:04
digvijayhere are results for recent patch - http://paste.openstack.org/show/792343/16:04
*** e0ne has quit IRC16:04
*** e0ne_ has joined #openstack-cinder16:04
rosmaitadigvijay: that is good to hear!16:04
*** enriquetaso has quit IRC16:04
*** senrique_ has joined #openstack-cinder16:05
*** ociuhandu has joined #openstack-cinder16:05
*** sapd1 has quit IRC16:07
*** ociuhandu has quit IRC16:15
*** ociuhandu has joined #openstack-cinder16:16
smcginnisThis concerns me a little: "^(?=^tempest.api.volume)(?!.*test_volume_from_snapshot_no_size)(?!.*test_volume_from_snapshot).*"16:18
smcginnisShould be .*volume.*, and should be able to create volume from snapshot.16:19
*** sapd1 has joined #openstack-cinder16:19
Xuchusmcginnis: I updated this patch based on your comments, Will you able to review it again? Thanks  https://review.opendev.org/#/c/70787316:19
digvijaysmcginnis: thanks for comment.. I'm no expert in it.. but I saw similar pattern in other driver logs.. will try to change and verify..16:21
*** ociuhandu has quit IRC16:21
*** udesale_ has quit IRC16:23
*** ianychoi_ has joined #openstack-cinder16:29
*** ianychoi has quit IRC16:31
*** yaawang_ has joined #openstack-cinder16:33
*** yaawang has quit IRC16:34
openstackgerritwuqiongdan proposed openstack/cinder master: add docs and release note for sandstone iscsi driver.  https://review.opendev.org/72067816:36
*** kevinz has quit IRC16:40
*** ociuhandu has joined #openstack-cinder16:40
*** digvijay has quit IRC16:50
*** ociuhandu has quit IRC17:03
openstackgerritRodrigo Barbieri proposed openstack/cinder master: [WIP] Fix (a) for bug 1869746  https://review.opendev.org/72074117:06
openstackbug 1869746 in Cinder "Cinder throws error creating incremental backup from parent in another project" [Undecided,Confirmed] https://launchpad.net/bugs/186974617:06
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix rbd attach on python3  https://review.opendev.org/72055317:09
openstackgerritMerged openstack/cinder stable/queens: ChunkedBackupDriver: Freeing memory on restore  https://review.opendev.org/71147917:13
openstackgerritMerged openstack/cinderlib master: DBMS: Fix volume deletion with MySQL  https://review.opendev.org/71396517:13
openstackgerritMerged openstack/python-cinderclient master: Remove Babel from requirements  https://review.opendev.org/72071317:14
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Enable volume_revert tests when supported  https://review.opendev.org/69729117:30
*** ociuhandu has joined #openstack-cinder17:30
*** e0ne_ has quit IRC17:38
*** e0ne has joined #openstack-cinder17:38
*** ociuhandu has quit IRC17:43
*** e0ne_ has joined #openstack-cinder17:49
*** e0ne has quit IRC17:49
*** sapd1 has quit IRC17:58
*** ociuhandu has joined #openstack-cinder18:09
*** ociuhandu has quit IRC18:22
*** ociuhandu has joined #openstack-cinder18:23
*** rosmaita has quit IRC18:25
*** rosmaita has joined #openstack-cinder18:26
*** ociuhandu has quit IRC18:28
openstackgerritBrian Rosmaita proposed openstack/cinder master: Add release note for drivers unsupported in Train  https://review.opendev.org/72080118:30
rosmaitajungleboyj: smcginnis: ^^ could use your feedback ^^18:32
smcginnisrosmaita: Taking a look...18:32
rosmaitathanks!18:33
jungleboyjMe too.18:33
rosmaitasmcginnis: jungleboyj: also this one: https://review.opendev.org/#/c/720502/18:33
rosmaitasmcginnis: jungleboyj: oh yeah, this one too: https://review.opendev.org/#/c/720031/18:37
smcginnisIf you give a rosmaita a review...18:37
rosmaitayeah, i should spread out my requests18:37
smcginnis:)18:38
smcginnisThis works well to get them all done. I have a hard enough time focusing on other work. ;)18:38
*** Lucas_Gray has joined #openstack-cinder18:38
jungleboyjYep.  If you don't get me all at once I don't get to it.18:39
smcginnisGreat, I was going to ask about upgrade checks.18:39
rosmaitasmcginnis: jungleboyj: what do you think would be better on that release note, add the .. note:: or just list the ussuri deprecations there too?18:41
smcginnisI don't have a strong opinion, just through I would mention it.18:41
* smcginnis throws the ball to jungleboyj 18:42
rosmaitamy concern was making sure that a reader didn't mistake that list for *all* the unsupported drivers18:42
rosmaitaso more emphasis would definitely be good18:43
jungleboyjWell, I think that we should change when to if.  So, lets make that change as well.18:43
rosmaitayes, i agree -- i was being too pessimistic18:43
rosmaitathat encryption policy patch you approved has this dependency:18:44
rosmaitahttps://review.opendev.org/#/c/720201/18:44
rosmaitasmcginnis: jungleboyj: ^^18:44
rosmaita(i really am pushing my luck here)18:45
jungleboyjAnything else master?18:49
rosmaitawell, since you asked, this one depends on the encryption policy patch: https://review.opendev.org/#/c/720262/18:49
jungleboyjsmcginnis:  The number of tests being run by the GPFS CI looks kind of low.  Wondering if that is because of what you made note of?18:51
jungleboyjOther CIs Brian and I were looking at were doing 250 to 270 tests.18:54
smcginnisjungleboyj: Yeah, I think so. That job regex is eliminating a lot of tests.18:54
jungleboyjYeah, that was what I was thinking.18:55
smcginnisAnd creating volume from snapshot is pretty fundamental at this point.18:55
*** e0ne_ has quit IRC18:56
*** e0ne has joined #openstack-cinder18:56
openstackgerritBrian Rosmaita proposed openstack/cinder master: Add release note for drivers unsupported in Train  https://review.opendev.org/72080118:57
rosmaitajungleboyj: smcginnis: see what you think of that ^^18:57
jungleboyjI am good.  Want me to +W ?19:02
rosmaitano, let's hold off -- ibm gpfs has until thursday to get the CI running19:03
jungleboyjOkie Dokie.19:03
rosmaitathanks19:03
*** lvbin02 has joined #openstack-cinder19:25
*** e0ne has quit IRC19:26
*** lvbin01 has quit IRC19:28
*** lvbin02 is now known as lvbin0119:28
*** ociuhandu has joined #openstack-cinder19:35
*** ociuhandu has quit IRC19:42
*** flwang1 has joined #openstack-cinder19:43
*** ociuhandu has joined #openstack-cinder20:45
openstackgerritRodrigo Barbieri proposed openstack/cinder master: [WIP] Fix (c) for bug 1869746  https://review.opendev.org/72083320:55
openstackbug 1869746 in Cinder "Cinder throws error creating incremental backup from parent in another project" [Undecided,Confirmed] https://launchpad.net/bugs/186974620:55
*** ociuhandu has quit IRC21:06
*** ociuhandu has joined #openstack-cinder21:08
openstackgerritMerged openstack/os-brick stable/train: rbd: Use showmapped to find the root RBD device during disconnect_volume  https://review.opendev.org/72017821:10
*** ociuhandu has quit IRC21:13
openstackgerritMerged openstack/cinder master: Fix revert snapshot issue  https://review.opendev.org/70787321:37
openstackgerritMerged openstack/cinder master: Update upgrade check removed drivers for ussuri  https://review.opendev.org/72003121:37
openstackgerritMerged openstack/cinder master: Add tests for volume type encryption type policies  https://review.opendev.org/72020121:37
openstackgerritMerged openstack/cinder master: Resolve deprecation of encryption policy target  https://review.opendev.org/72050221:37
openstackgerritMerged openstack/cinder master: Correct description for encryption-type policies  https://review.opendev.org/72026221:38
*** ociuhandu has joined #openstack-cinder22:32
*** Lucas_Gray has quit IRC22:32
*** ociuhandu has quit IRC22:38
*** senrique_ has quit IRC23:13
*** tosky has quit IRC23:21

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