| *** enriquetaso has quit IRC | 00:14 | |
| *** brinzhang0 has joined #openstack-cinder | 00:19 | |
| *** brinzhang_ has quit IRC | 00:22 | |
| *** eharney has joined #openstack-cinder | 00:36 | |
| *** spatel has joined #openstack-cinder | 00:57 | |
| *** spatel has quit IRC | 01:33 | |
| *** sapd1_x has joined #openstack-cinder | 02:02 | |
| *** rcernin has quit IRC | 02:27 | |
| *** rcernin has joined #openstack-cinder | 02:30 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder stable/queens: VNX: delete the LUN from VNX backend https://review.opendev.org/744591 | 02:45 |
|---|---|---|
| *** yaawang has quit IRC | 02:47 | |
| *** yaawang has joined #openstack-cinder | 02:48 | |
| *** psachin has joined #openstack-cinder | 03:37 | |
| *** vishalmanchanda has joined #openstack-cinder | 03:42 | |
| *** dave-mccowan has quit IRC | 04:02 | |
| *** sapd1_x has quit IRC | 04:05 | |
| *** hamalq has joined #openstack-cinder | 04:09 | |
| *** hamalq has quit IRC | 04:09 | |
| *** hamalq has joined #openstack-cinder | 04:10 | |
| *** hamalq has quit IRC | 04:17 | |
| *** Lucas_Gray has quit IRC | 04:36 | |
| *** m75abrams has joined #openstack-cinder | 04:46 | |
| *** xiaolin has quit IRC | 05:10 | |
| *** udesale has joined #openstack-cinder | 05:36 | |
| *** benfelin has joined #openstack-cinder | 05:38 | |
| *** ircuser-1 has joined #openstack-cinder | 05:59 | |
| *** whoami-rajat__ has joined #openstack-cinder | 06:23 | |
| *** eharney has quit IRC | 06:35 | |
| *** devfaz_ is now known as devfaz | 06:39 | |
| *** e0ne has joined #openstack-cinder | 06:47 | |
| *** eharney has joined #openstack-cinder | 06:53 | |
| openstackgerrit | katari manoj kumar proposed openstack/cinder master: [Storwize]:delete_group_snapshot doesn't handle flash copy consistency group cleanup properly. In case of multiple snapshots in the group,existing code exits if any one snapshot deletion fails, but it should update error state and continue with deleting o https://review.opendev.org/744652 | 06:58 |
| *** dsariel has joined #openstack-cinder | 07:14 | |
| *** rcernin has quit IRC | 07:22 | |
| *** sapd1_x has joined #openstack-cinder | 07:23 | |
| *** e0ne has quit IRC | 07:27 | |
| *** e0ne has joined #openstack-cinder | 07:32 | |
| *** rcernin has joined #openstack-cinder | 07:48 | |
| *** tosky has joined #openstack-cinder | 07:48 | |
| *** rcernin has quit IRC | 07:53 | |
| *** martinkennelly has joined #openstack-cinder | 07:54 | |
| *** rcernin has joined #openstack-cinder | 08:26 | |
| *** Lucas_Gray has joined #openstack-cinder | 08:30 | |
| *** rcernin has quit IRC | 08:31 | |
| *** priteau has joined #openstack-cinder | 08:43 | |
| *** rcernin has joined #openstack-cinder | 08:58 | |
| *** baba-tunde has quit IRC | 09:02 | |
| *** baba-tunde has joined #openstack-cinder | 09:04 | |
| *** lpetrut has joined #openstack-cinder | 09:12 | |
| *** rcernin has quit IRC | 09:16 | |
| openstackgerrit | katari manoj kumar proposed openstack/cinder master: [Storwize]:delete_group_snapshot doesn't handle flash copy consistency group cleanup properly. In case of multiple snapshots in the group,existing code exits if any one snapshot deletion fails, but it should update error state and continue with deleting o https://review.opendev.org/744652 | 09:19 |
| openstackgerrit | Girish Chilukuri proposed openstack/cinder master: [Storwize]:check_vdisk_fc_mappings method is not deleting the clone fc maps, if the fc status is in copying and autodelete is set to off. when multiple clones and snapshots are created using common source volume, and deleting the source volume is not remo https://review.opendev.org/744664 | 09:20 |
| *** ociuhandu_ has joined #openstack-cinder | 09:21 | |
| *** ociuhandu has quit IRC | 09:21 | |
| *** yaawang has quit IRC | 09:23 | |
| *** yaawang has joined #openstack-cinder | 09:26 | |
| geguileo | tosky: ping | 09:52 |
| *** yaawang has quit IRC | 10:04 | |
| *** yaawang has joined #openstack-cinder | 10:05 | |
| *** udesale_ has joined #openstack-cinder | 10:12 | |
| *** tkajinam has quit IRC | 10:12 | |
| *** udesale has quit IRC | 10:14 | |
| *** deiter has joined #openstack-cinder | 10:21 | |
| deiter | Hello, could you please review https://review.opendev.org/#/c/743040 and https://review.opendev.org/741543 Thank you very much! | 10:23 |
| *** sapd1_x has quit IRC | 10:24 | |
| *** ociuhandu_ has quit IRC | 10:30 | |
| *** ociuhandu has joined #openstack-cinder | 10:31 | |
| *** sapd1_x has joined #openstack-cinder | 10:36 | |
| *** ociuhandu has quit IRC | 10:36 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Update to oslo.privsep 2.3.0 https://review.opendev.org/744483 | 10:37 |
| *** rcernin has joined #openstack-cinder | 10:55 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinderlib master: Fix cinderlib-lvm-functional https://review.opendev.org/744680 | 11:04 |
| *** eharney has quit IRC | 11:13 | |
| *** jv_ has joined #openstack-cinder | 11:15 | |
| *** Luzi has joined #openstack-cinder | 11:20 | |
| *** tosky_ has joined #openstack-cinder | 11:20 | |
| *** tosky has quit IRC | 11:20 | |
| *** tosky_ is now known as tosky | 11:20 | |
| *** ociuhandu has joined #openstack-cinder | 11:23 | |
| *** ociuhandu has quit IRC | 11:24 | |
| *** ociuhandu has joined #openstack-cinder | 11:25 | |
| *** jv__ has joined #openstack-cinder | 11:26 | |
| *** udesale has joined #openstack-cinder | 11:27 | |
| *** jv_ has quit IRC | 11:27 | |
| *** udesale_ has quit IRC | 11:27 | |
| *** ociuhandu has quit IRC | 11:30 | |
| *** udesale_ has joined #openstack-cinder | 11:31 | |
| *** udesale has quit IRC | 11:32 | |
| *** eharney has joined #openstack-cinder | 11:32 | |
| openstackgerrit | Venkata krishna Thumu proposed openstack/cinder master: [Storwize]: Option to retain the auxiliary volume in a mirror relationship on delete of primary volume or When moving the primary volume to non-mirror https://review.opendev.org/744683 | 11:36 |
| *** sapd1_x has quit IRC | 11:36 | |
| *** udesale_ has quit IRC | 11:38 | |
| *** ociuhandu has joined #openstack-cinder | 11:40 | |
| *** udesale has joined #openstack-cinder | 11:41 | |
| *** rcernin has quit IRC | 11:53 | |
| *** brinzhang_ has joined #openstack-cinder | 11:57 | |
| *** brinzhang0 has quit IRC | 12:00 | |
| *** takamatsu has joined #openstack-cinder | 12:03 | |
| *** tosky has quit IRC | 12:14 | |
| *** tosky_ has joined #openstack-cinder | 12:14 | |
| *** tosky_ is now known as tosky | 12:15 | |
| smcginnis | rosmaita: When you get a chance, it would be good to have you take another look at https://review.opendev.org/#/c/706780/ | 12:15 |
| geguileo | smcginnis: I investigated a bit more the lower constraints issue and sent a mail to the list | 12:31 |
| geguileo | smcginnis: but I haven't been able to figure out hwere to fix the issue :-( | 12:31 |
| smcginnis | geguileo: There have been a few of us looking, and no luck so far. Thanks for digging in to it. Something really strange must have changed, because nothing is too obvious. | 12:38 |
| smcginnis | The root issue seems to be during the phase of the tox run where it does a local install of the project (pip install -e .). | 12:38 |
| geguileo | yeah, with --exists-action w | 12:39 |
| smcginnis | Normally we first run "tox -e lower-constraints --notest" to get all of the constrained requirements installed. That phase looks normal and everything is installed with the right versions. | 12:39 |
| smcginnis | Then the next phase it attempts to install the local project. During that process, it appears certain packages are upgraded to the latest versions, not using any constraints at all. | 12:40 |
| smcginnis | It didn't used to behave that way. | 12:40 |
| smcginnis | So at least for now, it looks like the master branch is passing, but mostly because it hasn't diverged much from the latest dependencies. | 12:40 |
| smcginnis | But stable branches have a lot of issues because of this. | 12:40 |
| geguileo | smcginnis: the latest of the brocade patches cannot merge because of this issue | 12:41 |
| geguileo | s/latest/last | 12:41 |
| smcginnis | Hah, now I see that Jeremy already responded to the ML thread with basically all of that. | 12:42 |
| fungi | yup | 12:42 |
| smcginnis | There was a tox release last week. I expected that to be the issue, but looking at the last patch that passed last Friday and the first that failed, they were both using the latest tox. | 12:42 |
| fungi | that's as far as i got looking into it yesterday | 12:42 |
| fungi | this does seem to have started the day after the last tox release, so *could* be related (but the changelog for it is really small and didn't mention anything suspicious in that regard) | 12:43 |
| smcginnis | fungi: Just read your latest response. Good idea with changing the install_command sounds promising. | 12:44 |
| smcginnis | Though that would be a major pain. | 12:44 |
| fungi | well, the irony is that we used to set constraints in install_command rather than deps | 12:44 |
| fungi | i haven't gone spelunking to find out/remember why we switched to doing it in deps | 12:44 |
| smcginnis | I think that was considered the "more correct" way of doing it, since you may want to override the install to use something other than pip. Or wrap pip with a script with additional logic or something. | 12:45 |
| fungi | you can see examples in some projects which haven't gotten around to updating their tox.ini yet: https://opendev.org/openstack/blazar-nova/src/branch/master/tox.ini#L10 | 12:45 |
| fungi | yeah, looks like it's projects still using a tools/tox_install.sh wrapper script | 12:46 |
| fungi | so maybe this changed as part of erradicating those wrappers? | 12:46 |
| smcginnis | Let's hope we don't have to go back to that everywhere. | 12:46 |
| smcginnis | Most projects removed that quite a while ago. | 12:46 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder stable/ussuri: DNM: Test constrained installs https://review.opendev.org/744698 | 12:47 |
| smcginnis | We can see if that looks any different. ^ | 12:47 |
| fungi | well, we could set something like "install_command = pip install -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master} {opts} {packages}" | 12:48 |
| smcginnis | fungi: Great idea. :D | 12:48 |
| fungi | no no wrapper script | 12:48 |
| smcginnis | fungi: Take a look at the DNM patch I put up. ;) | 12:48 |
| fungi | yeah, i figured that's why you called the idea "great" ;) | 12:49 |
| smcginnis | Guess we'll see in a couple hours. | 12:49 |
| *** yaawang has quit IRC | 12:50 | |
| *** yaawang has joined #openstack-cinder | 12:51 | |
| smcginnis | Ah, now I am reminded from the conversation in -infra why that was removed from the install_command. That breaks the lower-constraints job by baking in the upper-constraints into the install_command. | 12:52 |
| *** raghavendrat has joined #openstack-cinder | 12:52 | |
| *** rpittau has joined #openstack-cinder | 12:52 | |
| *** eharney has quit IRC | 12:55 | |
| *** eharney has joined #openstack-cinder | 12:56 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder stable/ussuri: DNM: Test constrained installs https://review.opendev.org/744698 | 12:56 |
| raghavendrat | hi e0ne: and geguileo: this is regarding https://review.opendev.org/#/c/730870/ | 12:57 |
| raghavendrat | 3PAR: Set the right minimum client version | 12:57 |
| raghavendrat | waiting for conclusion ... about release notes. | 12:58 |
| geguileo | e0ne: do you think we should add a release note? | 12:58 |
| smcginnis | FWIW, I don't think we typically have added release notes for things like that. | 12:59 |
| geguileo | smcginnis: thanks, that was my thought as well, but I don't have a strong opinion either way | 13:00 |
| *** enriquetaso has joined #openstack-cinder | 13:00 | |
| smcginnis | Yeah, doesn't hurt to make sure operators are aware of changes like that. I guess I'm fine either way as well. | 13:01 |
| e0ne | geguileo, smcginnis: sounds reasonable. I just want to confirm that release not is not required in this case | 13:09 |
| raghavendrat | thanks Gorka, Sean & Ivan for your thoughts. thanks Ivan for approval. | 13:11 |
| *** raghavendrat has quit IRC | 13:16 | |
| *** raghavendrat has joined #openstack-cinder | 13:19 | |
| *** baba-tunde has quit IRC | 13:24 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Replacing generations with snap_ids https://review.opendev.org/740433 | 13:24 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Modify default/delete volume type logic https://review.opendev.org/741498 | 13:25 |
| *** baba-tunde has joined #openstack-cinder | 13:26 | |
| *** sapd1_x has joined #openstack-cinder | 13:26 | |
| *** pcaruana has quit IRC | 13:27 | |
| *** hoonetorg has quit IRC | 13:36 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Legacy volumes fail to live migrate https://review.opendev.org/735348 | 13:37 |
| *** hoonetorg has joined #openstack-cinder | 13:39 | |
| *** raghavendrat has quit IRC | 13:41 | |
| *** pcaruana has joined #openstack-cinder | 14:04 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Uncomment psycopg2 in test-requirements.txt https://review.opendev.org/744721 | 14:05 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder master: Add ceph iscsi volume driver https://review.opendev.org/662829 | 14:19 |
| *** Luzi has quit IRC | 14:27 | |
| *** psachin has quit IRC | 14:33 | |
| *** udesale_ has joined #openstack-cinder | 14:35 | |
| openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Failover abilities promotion https://review.opendev.org/741439 | 14:36 |
| *** udesale has quit IRC | 14:38 | |
| *** donnyd has joined #openstack-cinder | 14:40 | |
| *** benfelin has quit IRC | 14:43 | |
| *** brinzhang_ has quit IRC | 14:47 | |
| *** benfelin has joined #openstack-cinder | 14:50 | |
| *** dave-mccowan has joined #openstack-cinder | 14:54 | |
| *** jv__ has quit IRC | 14:58 | |
| *** dave-mccowan has quit IRC | 14:59 | |
| *** jv__ has joined #openstack-cinder | 15:10 | |
| *** lpetrut has quit IRC | 15:15 | |
| *** openstackgerrit has quit IRC | 15:20 | |
| *** jv__ has quit IRC | 15:22 | |
| *** openstackgerrit has joined #openstack-cinder | 15:32 | |
| openstackgerrit | katari manoj kumar proposed openstack/cinder master: [Storwize]:delete_group_snapshot doesn't handle flash copy consistency group cleanup properly. In case of multiple snapshots in the group,existing code exits if any one snapshot deletion fails, but it should update error state and continue with deleting o https://review.opendev.org/744652 | 15:32 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax docs - incorrect property set on volume group https://review.opendev.org/744738 | 15:41 |
| *** lpetrut has joined #openstack-cinder | 15:43 | |
| *** e0ne has quit IRC | 16:00 | |
| *** m75abrams has quit IRC | 16:04 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: PowerMax Docs - Incorrect property set on volume group https://review.opendev.org/744738 | 16:06 |
| *** vishakha has joined #openstack-cinder | 16:15 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Clarify image_conversion_*_limit option descriptions https://review.opendev.org/744746 | 16:17 |
| *** lpetrut has quit IRC | 16:21 | |
| *** ociuhandu has quit IRC | 16:27 | |
| *** hamalq has joined #openstack-cinder | 16:38 | |
| *** gregwork has joined #openstack-cinder | 16:40 | |
| *** hamalq_ has joined #openstack-cinder | 16:41 | |
| *** hamalq has quit IRC | 16:44 | |
| *** benfelin has quit IRC | 16:49 | |
| *** priteau has quit IRC | 16:51 | |
| *** priteau has joined #openstack-cinder | 16:51 | |
| *** udesale_ has quit IRC | 17:16 | |
| *** Lucas_Gray has quit IRC | 17:22 | |
| *** priteau has quit IRC | 17:33 | |
| tosky | so, virtualenv was the culprit of the lower-requirements failure - see the latest discoveries here: http://lists.openstack.org/pipermail/openstack-discuss/2020-August/016295.html | 17:34 |
| smcginnis | That was a fun one. | 17:36 |
| *** sapd1_x has quit IRC | 17:45 | |
| eharney | looking at https://review.opendev.org/#/c/744721/ re: psycopg2 as a test-requirement... does listing this in test-reqs actually accomplish anything? | 17:47 |
| eharney | i don't see where it enables additional testing | 17:47 |
| *** lpetrut has joined #openstack-cinder | 17:49 | |
| clarkb | eharney: https://opendev.org/openstack/cinder/src/branch/master/tools/test-setup.sh#L35-L54 configures the postgres db for unittesting, then you likely have db migration tests that check myslq and postgres | 17:49 |
| *** lpetrut has quit IRC | 17:50 | |
| *** jv__ has joined #openstack-cinder | 17:51 | |
| eharney | clarkb: ah, it looks like it lets cinder.tests.unit.db.test_migrations.TestPostgresqlMigrations run, thanks | 17:51 |
| openstackgerrit | Merged openstack/cinder master: Uncomment psycopg2 in test-requirements.txt https://review.opendev.org/744721 | 18:33 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Modify default/delete volume type logic https://review.opendev.org/741498 | 18:37 |
| *** markvoelker has joined #openstack-cinder | 18:56 | |
| *** vishalmanchanda has quit IRC | 19:12 | |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder master: [WIP] Sizing encrypted volumes https://review.opendev.org/743869 | 20:17 |
| *** baba-tunde has quit IRC | 20:19 | |
| *** baba-tunde has joined #openstack-cinder | 20:23 | |
| *** jv__ has quit IRC | 20:29 | |
| *** vishakha has quit IRC | 20:34 | |
| *** baba-tunde has quit IRC | 20:34 | |
| *** baba-tunde has joined #openstack-cinder | 20:37 | |
| *** baba-tunde has quit IRC | 20:38 | |
| *** baba-tunde has joined #openstack-cinder | 20:40 | |
| *** baba-tunde has quit IRC | 20:42 | |
| *** baba-tunde has joined #openstack-cinder | 20:42 | |
| *** baba-tunde has quit IRC | 20:44 | |
| *** baba-tunde has joined #openstack-cinder | 20:46 | |
| *** whoami-rajat__ has quit IRC | 20:59 | |
| *** markvoelker has quit IRC | 21:30 | |
| openstackgerrit | Lucio Seki proposed openstack/cinder master: [NetApp] Fix HTTPS connection for python 3.7 https://review.opendev.org/732704 | 21:41 |
| *** martinkennelly has quit IRC | 21:44 | |
| *** dsariel has quit IRC | 21:48 | |
| *** Lucas_Gray has joined #openstack-cinder | 22:01 | |
| *** ociuhandu has joined #openstack-cinder | 22:15 | |
| *** ociuhandu has quit IRC | 22:20 | |
| *** enriquetaso has quit IRC | 22:42 | |
| *** tosky has quit IRC | 22:50 | |
| *** tobberydberg_ has quit IRC | 22:55 | |
| *** sorrison has quit IRC | 22:55 | |
| *** dansmith has quit IRC | 22:55 | |
| *** frickler has quit IRC | 22:55 | |
| *** Corwin has quit IRC | 22:55 | |
| *** tkajinam has joined #openstack-cinder | 22:55 | |
| *** sorrison has joined #openstack-cinder | 22:57 | |
| *** tobberydberg_ has joined #openstack-cinder | 22:57 | |
| *** dansmith has joined #openstack-cinder | 22:57 | |
| *** frickler has joined #openstack-cinder | 22:57 | |
| *** Corwin has joined #openstack-cinder | 22:57 | |
| *** Lucas_Gray has quit IRC | 23:10 | |
| *** Lucas_Gray has joined #openstack-cinder | 23:14 | |
| *** hamalq_ has quit IRC | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!