Tuesday, 2020-05-19

*** spatel has quit IRC00:01
*** n-saito has joined #openstack-cinder00:05
*** spatel has joined #openstack-cinder00:22
*** brinzhang has joined #openstack-cinder00:23
*** spatel has quit IRC00:24
*** andrebeltrami has quit IRC00:58
*** brinzhang_ has joined #openstack-cinder01:11
*** brinzhang has quit IRC01:14
*** yaawang has quit IRC01:22
*** kevinz has joined #openstack-cinder01:23
*** yaawang has joined #openstack-cinder01:25
*** rmcall has quit IRC01:53
openstackgerritAlan Bishop proposed openstack/cinder master: 3PAR - Fix renaming volume after migration  https://review.opendev.org/72908202:05
*** sapd1 has joined #openstack-cinder02:08
*** sapd1 has quit IRC02:22
*** sapd1_x has joined #openstack-cinder02:22
*** KeithMnemonic has quit IRC02:50
*** psachin has joined #openstack-cinder03:39
openstackgerritSeema S B proposed openstack/cinder master: Add policy targets to volume-type API  https://review.opendev.org/67879903:48
*** brinzhang has joined #openstack-cinder04:14
*** threestrands has quit IRC04:15
*** brinzhang_ has quit IRC04:17
*** ociuhandu has joined #openstack-cinder04:20
*** ociuhandu has quit IRC04:24
*** udesale has joined #openstack-cinder04:46
*** vishalmanchanda has joined #openstack-cinder04:52
*** ykarel|away is now known as ykarel04:57
openstackgerritAlan Bishop proposed openstack/cinder master: 3PAR - Fix renaming volume after migration  https://review.opendev.org/72908205:01
*** vishakha has joined #openstack-cinder05:30
*** dpawlik has joined #openstack-cinder06:02
*** sapd1_y has joined #openstack-cinder06:07
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/72912206:19
*** ircuser-1 has joined #openstack-cinder06:49
*** xinliang has joined #openstack-cinder07:06
*** JaySee has quit IRC07:13
*** JaySee has joined #openstack-cinder07:13
*** udesale has quit IRC07:15
*** udesale has joined #openstack-cinder07:15
*** spatel has joined #openstack-cinder07:19
*** spatel has quit IRC07:23
*** TheJulia has quit IRC07:24
*** mnasiadka has quit IRC07:25
*** mnasiadka has joined #openstack-cinder07:27
*** TheJulia has joined #openstack-cinder07:29
*** ociuhandu has joined #openstack-cinder07:38
*** vishalmanchanda has quit IRC07:39
*** lvbin01 has joined #openstack-cinder07:42
*** pcaruana has quit IRC07:51
*** tosky has joined #openstack-cinder07:56
*** lpetrut has joined #openstack-cinder08:01
*** tkajinam has quit IRC08:02
*** ociuhandu has quit IRC08:03
*** ociuhandu has joined #openstack-cinder08:04
*** e0ne has joined #openstack-cinder08:06
*** ociuhandu has quit IRC08:06
*** ociuhandu has joined #openstack-cinder08:06
*** yaawang has quit IRC08:08
*** yaawang has joined #openstack-cinder08:09
*** martinkennelly has joined #openstack-cinder08:14
*** ociuhandu has quit IRC08:17
*** ociuhandu has joined #openstack-cinder08:18
*** yaawang has quit IRC08:19
*** yaawang has joined #openstack-cinder08:21
*** ociuhandu has quit IRC08:23
e0negeguileo: hi. could you please take a look on these backports https://review.opendev.org/#/q/status:open+project:openstack/cinder++topic:bug/1837703? I'm not sure which branches we need to backport your patch08:49
*** ykarel is now known as ykarel|lunch09:07
*** ociuhandu has joined #openstack-cinder09:07
geguileoe0ne: reviewed, one looks good and the other needs some minor changes09:19
e0negeguileo: thanks a lot! will fix it asap09:20
*** vishalmanchanda has joined #openstack-cinder09:24
*** ociuhandu has quit IRC09:25
*** ociuhandu has joined #openstack-cinder09:26
openstackgerritIvan Kolodyazhny proposed openstack/cinder stable/rocky: Fix online data migrations  https://review.opendev.org/72847809:27
*** spatel has joined #openstack-cinder09:39
*** spatel has quit IRC09:44
geguileoe0ne: thank you for the backports :-)09:46
*** ykarel|lunch is now known as ykarel09:53
*** lvbin02 has joined #openstack-cinder09:55
*** lvbin01 has quit IRC09:57
*** lvbin02 is now known as lvbin0109:57
*** n-saito has quit IRC09:59
e0negeguileo: you're welcome!10:05
toskygeguileo: have you seen https://review.opendev.org/#/c/728279/ ? Without that fix, future reviews may fail, but it seems a few fixes may be needed anyway10:16
*** xinliang has quit IRC10:20
*** tkajinam has joined #openstack-cinder10:30
*** johnmp has joined #openstack-cinder10:48
*** lvbin01 has quit IRC10:54
*** lvbin01 has joined #openstack-cinder10:55
openstackgerritHelen Walsh proposed openstack/cinder stable/ussuri: PowerMax Driver - Concurrent live migrations failure  https://review.opendev.org/72920911:06
openstackgerritHelen Walsh proposed openstack/cinder stable/ussuri: PowerMax Driver - U4P failover lock not released on exception  https://review.opendev.org/72921711:15
*** sapd1_x has quit IRC11:15
*** eharney has quit IRC11:39
*** ociuhandu has quit IRC11:40
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.opendev.org/72912211:41
*** ociuhandu has joined #openstack-cinder11:42
*** ociuhandu has quit IRC11:47
*** udesale_ has joined #openstack-cinder12:00
openstackgerritHelen Walsh proposed openstack/cinder stable/ussuri: PowerMax Driver - Live migrate remove rep vol from sg  https://review.opendev.org/72923112:01
*** udesale has quit IRC12:02
*** enriquetaso has joined #openstack-cinder12:05
*** jawad_axd has joined #openstack-cinder12:05
*** Xuchu has joined #openstack-cinder12:44
openstackgerritAndreas Jaeger proposed openstack/python-cinderclient master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72854112:44
*** ociuhandu has joined #openstack-cinder12:57
*** ociuhandu has quit IRC13:06
*** vishalmanchanda has quit IRC13:09
openstackgerritIvaylo Mitev proposed openstack/cinder master: VMware VMDK: Initial size of volume from image  https://review.opendev.org/66989213:09
*** ykarel is now known as ykarel|afk13:11
*** eharney has joined #openstack-cinder13:13
*** ociuhandu has joined #openstack-cinder13:26
*** martinkennelly has quit IRC13:27
*** ociuhandu has quit IRC13:31
*** psachin has quit IRC13:35
openstackgerritHelen Walsh proposed openstack/cinder master: :PowerMax Driver - Ussuri Documentation  https://review.opendev.org/72280613:43
walshh_Is there anyone available to have a look at a Queens patch.  It is applicable to Queens only.  https://review.opendev.org/#/c/723532/ Thank you13:45
*** ociuhandu has joined #openstack-cinder13:48
*** lpetrut has quit IRC13:52
*** ociuhandu has quit IRC13:54
*** vishakha has quit IRC14:10
*** sapd1_x has joined #openstack-cinder14:10
*** tkajinam has quit IRC14:11
*** tkajinam has joined #openstack-cinder14:11
smcginniswalshh_: Did you see my comment on there?14:11
*** jawad_axd has quit IRC14:12
*** jawad_axd has joined #openstack-cinder14:12
*** jawad_axd has quit IRC14:18
*** andrebeltrami has joined #openstack-cinder14:31
XuchuHi smcginnis: Can you please help to review this code refactor patch : https://review.opendev.org/#/c/723817/ when you are available ? Thanks14:39
*** ykarel|afk is now known as ykarel14:43
*** ociuhandu has joined #openstack-cinder14:46
*** jawad_axd has joined #openstack-cinder14:49
*** jawad_axd has quit IRC14:54
openstackgerritEric Harney proposed openstack/cinder master: Move make_initiator_target_all2all_map out of vutils  https://review.opendev.org/72929614:54
*** Xuchu has quit IRC14:57
*** jawad_axd has joined #openstack-cinder15:20
gansotosky: hi! based on our previous meeting discussion about bug https://bugs.launchpad.net/cinder/+bug/1869746, I wrote test cases for it https://review.opendev.org/#/c/728289 which are passing after setting the depends-on. Those are all of the use cases I could think of. Could you please take a look when you have some time?15:21
openstackLaunchpad bug 1869746 in Cinder "Cinder throws error creating incremental backup from parent in another project" [Undecided,In progress] - Assigned to Rodrigo Barbieri (rodrigo-barbieri2010)15:21
*** martinkennelly has joined #openstack-cinder15:21
toskyooooh, nice, thanks!15:24
toskyI will take a look15:24
toskyganso: oh, quick thing: any special reason to set the volume api to 2? We should use the default15:26
gansotosky: hmm I just copied from another file15:27
gansotosky: I can try removing that and see what changes15:28
toskyyes, please; volume API v2 is deprecated15:35
gansotosky: working on it. thanks for the feedback!15:38
*** ykarel is now known as ykarel|away15:38
*** martinkennelly has quit IRC15:41
openstackgerritRodrigo Barbieri proposed openstack/cinder-tempest-plugin master: Add test cases for bugs 1869746 and 1873518  https://review.opendev.org/72828915:43
openstackbug 1873518 in Cinder "DB method backup_get_all_by_volume only work for admin" [Undecided,In progress] https://launchpad.net/bugs/1873518 - Assigned to Rodrigo Barbieri (rodrigo-barbieri2010)15:43
openstackbug 1869746 in Cinder "Cinder throws error creating incremental backup from parent in another project" [Undecided,In progress] https://launchpad.net/bugs/1869746 - Assigned to Rodrigo Barbieri (rodrigo-barbieri2010)15:43
*** ociuhandu has quit IRC15:55
*** martinkennelly has joined #openstack-cinder15:55
*** ociuhandu has joined #openstack-cinder15:55
*** jawad_axd has quit IRC15:56
*** ociuhandu has quit IRC15:59
*** ociuhandu has joined #openstack-cinder16:00
*** martinkennelly has quit IRC16:01
*** martinkennelly has joined #openstack-cinder16:01
*** lpetrut has joined #openstack-cinder16:02
*** jawad_axd has joined #openstack-cinder16:02
openstackgerritHelen Walsh proposed openstack/cinder stable/queens: VMAX Driver Queens - FC live migration failure  https://review.opendev.org/72353216:09
*** udesale_ has quit IRC16:24
simondodsleycould we get some eyes on https://review.opendev.org/#/c/726236/ and https://review.opendev.org/#/c/726417/ please?16:30
*** dpawlik has quit IRC16:37
toskysimondodsley: I'm not a core but I have a question about the second change (revert-to-snapshot): does your CI run the revert-to-snapshot tempest tests? They are defined inside cinder-tempest-plugin and require a configuration key to be enabled in tempest.conf16:42
simondodsleyGood question - I'll check16:47
*** lpetrut has quit IRC16:50
*** vishakha has joined #openstack-cinder16:51
rosmaitasimondodsley: i looked at the logs, don't think it's enabled16:51
simondodsleyWhats the key name? I'll get them enabled.16:52
rosmaitain [volume-feature-enabled]16:55
rosmaita volume_revert = True16:55
*** udesale has joined #openstack-cinder16:55
rosmaitasimondodsley: ^^16:55
simondodsleyGot it - thanks16:55
*** udesale has quit IRC17:01
*** ociuhandu has quit IRC17:01
*** ociuhandu has joined #openstack-cinder17:02
*** ociuhandu has quit IRC17:04
*** ociuhandu has joined #openstack-cinder17:04
toskysimondodsley: and you also need to install cinder-tempest-plugin17:11
*** sapd1_x has quit IRC17:11
simondodsleywe have that already i beleive, bu I will check17:12
toskysimondodsley: I see a pure_tempest_plugin, but no tests  from it; maybe it's just a matter of tuning the regular expression which trigger the jobs17:12
tosky(I found the logs in the meantime)17:13
*** jawad_ax_ has joined #openstack-cinder17:23
*** jawad_axd has quit IRC17:27
*** ociuhandu has quit IRC17:28
*** ociuhandu has joined #openstack-cinder17:29
*** ociuhandu has quit IRC17:41
openstackgerritAlan Bishop proposed openstack/cinder master: 3PAR - Fix renaming volume after migration  https://review.opendev.org/72908217:45
*** jawad_ax_ has quit IRC17:45
*** jawad_axd has joined #openstack-cinder17:45
*** e0ne has quit IRC18:02
*** e0ne has joined #openstack-cinder18:07
openstackgerritEric Harney proposed openstack/cinder master: Move make_initiator_target_all2all_map out of vutils  https://review.opendev.org/72929618:11
openstackgerritEric Harney proposed openstack/cinder master: tests: remove self.injected  https://review.opendev.org/72934318:11
*** martinkennelly has quit IRC18:22
*** ociuhandu has joined #openstack-cinder18:31
*** rcernin has quit IRC18:34
*** jawad_axd has quit IRC18:53
*** ociuhandu has quit IRC19:09
*** ociuhandu has joined #openstack-cinder19:09
*** ociuhandu has quit IRC19:15
*** ajitha has joined #openstack-cinder19:20
*** ajitha has left #openstack-cinder19:30
*** jawad_axd has joined #openstack-cinder19:50
*** enriquetaso has quit IRC20:10
*** enriquetaso has joined #openstack-cinder20:13
*** vishakha has quit IRC20:29
*** jawad_axd has quit IRC20:44
*** rcernin has joined #openstack-cinder22:56
*** andrebeltrami has quit IRC23:19

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