*** spatel has quit IRC | 00:01 | |
*** n-saito has joined #openstack-cinder | 00:05 | |
*** spatel has joined #openstack-cinder | 00:22 | |
*** brinzhang has joined #openstack-cinder | 00:23 | |
*** spatel has quit IRC | 00:24 | |
*** andrebeltrami has quit IRC | 00:58 | |
*** brinzhang_ has joined #openstack-cinder | 01:11 | |
*** brinzhang has quit IRC | 01:14 | |
*** yaawang has quit IRC | 01:22 | |
*** kevinz has joined #openstack-cinder | 01:23 | |
*** yaawang has joined #openstack-cinder | 01:25 | |
*** rmcall has quit IRC | 01:53 | |
openstackgerrit | Alan Bishop proposed openstack/cinder master: 3PAR - Fix renaming volume after migration https://review.opendev.org/729082 | 02:05 |
---|---|---|
*** sapd1 has joined #openstack-cinder | 02:08 | |
*** sapd1 has quit IRC | 02:22 | |
*** sapd1_x has joined #openstack-cinder | 02:22 | |
*** KeithMnemonic has quit IRC | 02:50 | |
*** psachin has joined #openstack-cinder | 03:39 | |
openstackgerrit | Seema S B proposed openstack/cinder master: Add policy targets to volume-type API https://review.opendev.org/678799 | 03:48 |
*** brinzhang has joined #openstack-cinder | 04:14 | |
*** threestrands has quit IRC | 04:15 | |
*** brinzhang_ has quit IRC | 04:17 | |
*** ociuhandu has joined #openstack-cinder | 04:20 | |
*** ociuhandu has quit IRC | 04:24 | |
*** udesale has joined #openstack-cinder | 04:46 | |
*** vishalmanchanda has joined #openstack-cinder | 04:52 | |
*** ykarel|away is now known as ykarel | 04:57 | |
openstackgerrit | Alan Bishop proposed openstack/cinder master: 3PAR - Fix renaming volume after migration https://review.opendev.org/729082 | 05:01 |
*** vishakha has joined #openstack-cinder | 05:30 | |
*** dpawlik has joined #openstack-cinder | 06:02 | |
*** sapd1_y has joined #openstack-cinder | 06:07 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/729122 | 06:19 |
*** ircuser-1 has joined #openstack-cinder | 06:49 | |
*** xinliang has joined #openstack-cinder | 07:06 | |
*** JaySee has quit IRC | 07:13 | |
*** JaySee has joined #openstack-cinder | 07:13 | |
*** udesale has quit IRC | 07:15 | |
*** udesale has joined #openstack-cinder | 07:15 | |
*** spatel has joined #openstack-cinder | 07:19 | |
*** spatel has quit IRC | 07:23 | |
*** TheJulia has quit IRC | 07:24 | |
*** mnasiadka has quit IRC | 07:25 | |
*** mnasiadka has joined #openstack-cinder | 07:27 | |
*** TheJulia has joined #openstack-cinder | 07:29 | |
*** ociuhandu has joined #openstack-cinder | 07:38 | |
*** vishalmanchanda has quit IRC | 07:39 | |
*** lvbin01 has joined #openstack-cinder | 07:42 | |
*** pcaruana has quit IRC | 07:51 | |
*** tosky has joined #openstack-cinder | 07:56 | |
*** lpetrut has joined #openstack-cinder | 08:01 | |
*** tkajinam has quit IRC | 08:02 | |
*** ociuhandu has quit IRC | 08:03 | |
*** ociuhandu has joined #openstack-cinder | 08:04 | |
*** e0ne has joined #openstack-cinder | 08:06 | |
*** ociuhandu has quit IRC | 08:06 | |
*** ociuhandu has joined #openstack-cinder | 08:06 | |
*** yaawang has quit IRC | 08:08 | |
*** yaawang has joined #openstack-cinder | 08:09 | |
*** martinkennelly has joined #openstack-cinder | 08:14 | |
*** ociuhandu has quit IRC | 08:17 | |
*** ociuhandu has joined #openstack-cinder | 08:18 | |
*** yaawang has quit IRC | 08:19 | |
*** yaawang has joined #openstack-cinder | 08:21 | |
*** ociuhandu has quit IRC | 08:23 | |
e0ne | geguileo: 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 patch | 08:49 |
*** ykarel is now known as ykarel|lunch | 09:07 | |
*** ociuhandu has joined #openstack-cinder | 09:07 | |
geguileo | e0ne: reviewed, one looks good and the other needs some minor changes | 09:19 |
e0ne | geguileo: thanks a lot! will fix it asap | 09:20 |
*** vishalmanchanda has joined #openstack-cinder | 09:24 | |
*** ociuhandu has quit IRC | 09:25 | |
*** ociuhandu has joined #openstack-cinder | 09:26 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder stable/rocky: Fix online data migrations https://review.opendev.org/728478 | 09:27 |
*** spatel has joined #openstack-cinder | 09:39 | |
*** spatel has quit IRC | 09:44 | |
geguileo | e0ne: thank you for the backports :-) | 09:46 |
*** ykarel|lunch is now known as ykarel | 09:53 | |
*** lvbin02 has joined #openstack-cinder | 09:55 | |
*** lvbin01 has quit IRC | 09:57 | |
*** lvbin02 is now known as lvbin01 | 09:57 | |
*** n-saito has quit IRC | 09:59 | |
e0ne | geguileo: you're welcome! | 10:05 |
tosky | geguileo: 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 anyway | 10:16 |
*** xinliang has quit IRC | 10:20 | |
*** tkajinam has joined #openstack-cinder | 10:30 | |
*** johnmp has joined #openstack-cinder | 10:48 | |
*** lvbin01 has quit IRC | 10:54 | |
*** lvbin01 has joined #openstack-cinder | 10:55 | |
openstackgerrit | Helen Walsh proposed openstack/cinder stable/ussuri: PowerMax Driver - Concurrent live migrations failure https://review.opendev.org/729209 | 11:06 |
openstackgerrit | Helen Walsh proposed openstack/cinder stable/ussuri: PowerMax Driver - U4P failover lock not released on exception https://review.opendev.org/729217 | 11:15 |
*** sapd1_x has quit IRC | 11:15 | |
*** eharney has quit IRC | 11:39 | |
*** ociuhandu has quit IRC | 11:40 | |
openstackgerrit | Merged openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/729122 | 11:41 |
*** ociuhandu has joined #openstack-cinder | 11:42 | |
*** ociuhandu has quit IRC | 11:47 | |
*** udesale_ has joined #openstack-cinder | 12:00 | |
openstackgerrit | Helen Walsh proposed openstack/cinder stable/ussuri: PowerMax Driver - Live migrate remove rep vol from sg https://review.opendev.org/729231 | 12:01 |
*** udesale has quit IRC | 12:02 | |
*** enriquetaso has joined #openstack-cinder | 12:05 | |
*** jawad_axd has joined #openstack-cinder | 12:05 | |
*** Xuchu has joined #openstack-cinder | 12:44 | |
openstackgerrit | Andreas Jaeger proposed openstack/python-cinderclient master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728541 | 12:44 |
*** ociuhandu has joined #openstack-cinder | 12:57 | |
*** ociuhandu has quit IRC | 13:06 | |
*** vishalmanchanda has quit IRC | 13:09 | |
openstackgerrit | Ivaylo Mitev proposed openstack/cinder master: VMware VMDK: Initial size of volume from image https://review.opendev.org/669892 | 13:09 |
*** ykarel is now known as ykarel|afk | 13:11 | |
*** eharney has joined #openstack-cinder | 13:13 | |
*** ociuhandu has joined #openstack-cinder | 13:26 | |
*** martinkennelly has quit IRC | 13:27 | |
*** ociuhandu has quit IRC | 13:31 | |
*** psachin has quit IRC | 13:35 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: :PowerMax Driver - Ussuri Documentation https://review.opendev.org/722806 | 13: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 you | 13:45 |
*** ociuhandu has joined #openstack-cinder | 13:48 | |
*** lpetrut has quit IRC | 13:52 | |
*** ociuhandu has quit IRC | 13:54 | |
*** vishakha has quit IRC | 14:10 | |
*** sapd1_x has joined #openstack-cinder | 14:10 | |
*** tkajinam has quit IRC | 14:11 | |
*** tkajinam has joined #openstack-cinder | 14:11 | |
smcginnis | walshh_: Did you see my comment on there? | 14:11 |
*** jawad_axd has quit IRC | 14:12 | |
*** jawad_axd has joined #openstack-cinder | 14:12 | |
*** jawad_axd has quit IRC | 14:18 | |
*** andrebeltrami has joined #openstack-cinder | 14:31 | |
Xuchu | Hi smcginnis: Can you please help to review this code refactor patch : https://review.opendev.org/#/c/723817/ when you are available ? Thanks | 14:39 |
*** ykarel|afk is now known as ykarel | 14:43 | |
*** ociuhandu has joined #openstack-cinder | 14:46 | |
*** jawad_axd has joined #openstack-cinder | 14:49 | |
*** jawad_axd has quit IRC | 14:54 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: Move make_initiator_target_all2all_map out of vutils https://review.opendev.org/729296 | 14:54 |
*** Xuchu has quit IRC | 14:57 | |
*** jawad_axd has joined #openstack-cinder | 15:20 | |
ganso | tosky: 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 |
openstack | Launchpad 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-cinder | 15:21 | |
tosky | ooooh, nice, thanks! | 15:24 |
tosky | I will take a look | 15:24 |
tosky | ganso: oh, quick thing: any special reason to set the volume api to 2? We should use the default | 15:26 |
ganso | tosky: hmm I just copied from another file | 15:27 |
ganso | tosky: I can try removing that and see what changes | 15:28 |
tosky | yes, please; volume API v2 is deprecated | 15:35 |
ganso | tosky: working on it. thanks for the feedback! | 15:38 |
*** ykarel is now known as ykarel|away | 15:38 | |
*** martinkennelly has quit IRC | 15:41 | |
openstackgerrit | Rodrigo Barbieri proposed openstack/cinder-tempest-plugin master: Add test cases for bugs 1869746 and 1873518 https://review.opendev.org/728289 | 15:43 |
openstack | bug 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 |
openstack | bug 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 IRC | 15:55 | |
*** martinkennelly has joined #openstack-cinder | 15:55 | |
*** ociuhandu has joined #openstack-cinder | 15:55 | |
*** jawad_axd has quit IRC | 15:56 | |
*** ociuhandu has quit IRC | 15:59 | |
*** ociuhandu has joined #openstack-cinder | 16:00 | |
*** martinkennelly has quit IRC | 16:01 | |
*** martinkennelly has joined #openstack-cinder | 16:01 | |
*** lpetrut has joined #openstack-cinder | 16:02 | |
*** jawad_axd has joined #openstack-cinder | 16:02 | |
openstackgerrit | Helen Walsh proposed openstack/cinder stable/queens: VMAX Driver Queens - FC live migration failure https://review.opendev.org/723532 | 16:09 |
*** udesale_ has quit IRC | 16:24 | |
simondodsley | could we get some eyes on https://review.opendev.org/#/c/726236/ and https://review.opendev.org/#/c/726417/ please? | 16:30 |
*** dpawlik has quit IRC | 16:37 | |
tosky | simondodsley: 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.conf | 16:42 |
simondodsley | Good question - I'll check | 16:47 |
*** lpetrut has quit IRC | 16:50 | |
*** vishakha has joined #openstack-cinder | 16:51 | |
rosmaita | simondodsley: i looked at the logs, don't think it's enabled | 16:51 |
simondodsley | Whats the key name? I'll get them enabled. | 16:52 |
rosmaita | in [volume-feature-enabled] | 16:55 |
rosmaita | volume_revert = True | 16:55 |
*** udesale has joined #openstack-cinder | 16:55 | |
rosmaita | simondodsley: ^^ | 16:55 |
simondodsley | Got it - thanks | 16:55 |
*** udesale has quit IRC | 17:01 | |
*** ociuhandu has quit IRC | 17:01 | |
*** ociuhandu has joined #openstack-cinder | 17:02 | |
*** ociuhandu has quit IRC | 17:04 | |
*** ociuhandu has joined #openstack-cinder | 17:04 | |
tosky | simondodsley: and you also need to install cinder-tempest-plugin | 17:11 |
*** sapd1_x has quit IRC | 17:11 | |
simondodsley | we have that already i beleive, bu I will check | 17:12 |
tosky | simondodsley: 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 jobs | 17:12 |
tosky | (I found the logs in the meantime) | 17:13 |
*** jawad_ax_ has joined #openstack-cinder | 17:23 | |
*** jawad_axd has quit IRC | 17:27 | |
*** ociuhandu has quit IRC | 17:28 | |
*** ociuhandu has joined #openstack-cinder | 17:29 | |
*** ociuhandu has quit IRC | 17:41 | |
openstackgerrit | Alan Bishop proposed openstack/cinder master: 3PAR - Fix renaming volume after migration https://review.opendev.org/729082 | 17:45 |
*** jawad_ax_ has quit IRC | 17:45 | |
*** jawad_axd has joined #openstack-cinder | 17:45 | |
*** e0ne has quit IRC | 18:02 | |
*** e0ne has joined #openstack-cinder | 18:07 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: Move make_initiator_target_all2all_map out of vutils https://review.opendev.org/729296 | 18:11 |
openstackgerrit | Eric Harney proposed openstack/cinder master: tests: remove self.injected https://review.opendev.org/729343 | 18:11 |
*** martinkennelly has quit IRC | 18:22 | |
*** ociuhandu has joined #openstack-cinder | 18:31 | |
*** rcernin has quit IRC | 18:34 | |
*** jawad_axd has quit IRC | 18:53 | |
*** ociuhandu has quit IRC | 19:09 | |
*** ociuhandu has joined #openstack-cinder | 19:09 | |
*** ociuhandu has quit IRC | 19:15 | |
*** ajitha has joined #openstack-cinder | 19:20 | |
*** ajitha has left #openstack-cinder | 19:30 | |
*** jawad_axd has joined #openstack-cinder | 19:50 | |
*** enriquetaso has quit IRC | 20:10 | |
*** enriquetaso has joined #openstack-cinder | 20:13 | |
*** vishakha has quit IRC | 20:29 | |
*** jawad_axd has quit IRC | 20:44 | |
*** rcernin has joined #openstack-cinder | 22:56 | |
*** andrebeltrami has quit IRC | 23:19 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!