| openstackgerrit | Tejdeep Kautharam proposed openstack/cinder master: Rsdlib changed providing_pools interface https://review.opendev.org/670677 | 00:35 |
|---|---|---|
| openstackgerrit | Tejdeep Kautharam proposed openstack/cinder master: extend_volume method fails if volume size is float https://review.opendev.org/674154 | 00:35 |
| *** mriedem has quit IRC | 00:38 | |
| *** markvoelker has joined #openstack-cinder | 00:41 | |
| *** markvoelker has quit IRC | 00:51 | |
| *** markvoelker has joined #openstack-cinder | 01:17 | |
| *** spsurya has joined #openstack-cinder | 01:17 | |
| *** Kuirong has joined #openstack-cinder | 01:28 | |
| *** altlogbot_3 has quit IRC | 01:37 | |
| *** altlogbot_0 has joined #openstack-cinder | 01:38 | |
| *** markvoelker has quit IRC | 01:38 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:48 | |
| *** lixiaoy1 has quit IRC | 01:49 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:56 | |
| *** markvoelker has joined #openstack-cinder | 02:42 | |
| *** whoami-rajat has joined #openstack-cinder | 03:07 | |
| *** markvoelker has quit IRC | 03:15 | |
| *** enriquetaso has quit IRC | 03:56 | |
| *** ircuser-1 has joined #openstack-cinder | 04:34 | |
| *** udesale has joined #openstack-cinder | 04:38 | |
| *** Luzi has joined #openstack-cinder | 05:05 | |
| *** markvoelker has joined #openstack-cinder | 05:28 | |
| *** markvoelker has quit IRC | 05:33 | |
| *** lixiaoy1 has quit IRC | 05:35 | |
| *** lixiaoy1 has joined #openstack-cinder | 05:36 | |
| *** tkajinam has quit IRC | 05:38 | |
| *** tkajinam has joined #openstack-cinder | 05:38 | |
| openstackgerrit | Merged openstack/cinder master: Update Veritas Access in Support Matrix https://review.opendev.org/674654 | 05:48 |
| *** gkadam_ has joined #openstack-cinder | 05:54 | |
| *** gkadam_ has quit IRC | 05:54 | |
| *** gkadam has quit IRC | 05:56 | |
| *** sahid has joined #openstack-cinder | 06:19 | |
| *** vishalmanchanda has joined #openstack-cinder | 06:36 | |
| *** markvoelker has joined #openstack-cinder | 06:36 | |
| openstackgerrit | Sam Wan proposed openstack/cinder master: upload_volume() should accept 'compress' paramter. https://review.opendev.org/674740 | 06:55 |
| *** markvoelker has quit IRC | 07:09 | |
| *** tosky has joined #openstack-cinder | 07:24 | |
| *** tesseract has joined #openstack-cinder | 07:30 | |
| *** rcernin has quit IRC | 08:04 | |
| *** ociuhandu has joined #openstack-cinder | 08:05 | |
| *** ociuhandu has quit IRC | 08:05 | |
| *** tkajinam has quit IRC | 08:06 | |
| *** lpetrut has joined #openstack-cinder | 08:10 | |
| *** helenafm has joined #openstack-cinder | 08:12 | |
| *** ociuhandu has joined #openstack-cinder | 08:17 | |
| *** ociuhandu has quit IRC | 08:30 | |
| *** e0ne has joined #openstack-cinder | 08:35 | |
| *** ociuhandu has joined #openstack-cinder | 08:51 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix upload volume to glance https://review.opendev.org/674758 | 08:55 |
| *** ociuhandu has quit IRC | 09:05 | |
| walshh_ | Is anyone available to look at https://review.opendev.org/#/c/664860 and https://review.opendev.org/#/c/664860. Thank you! | 09:24 |
| *** abishop_ has joined #openstack-cinder | 09:40 | |
| *** abishop has quit IRC | 09:41 | |
| walshh_ | apologies, second review is https://review.opendev.org/#/c/669835 | 09:42 |
| *** ruffian_sheep has joined #openstack-cinder | 09:45 | |
| ruffian_sheep | whoami-rajat:Hi,are u online? | 09:45 |
| *** ociuhandu has joined #openstack-cinder | 09:47 | |
| whoami-rajat | ruffian_sheep: Hey | 09:47 |
| ruffian_sheep | whoami-rajat:Did you have time to discuss the issue with other members yesterday? | 09:48 |
| whoami-rajat | ruffian_sheep: i think sean already replied on your patch | 09:48 |
| whoami-rajat | ruffian_sheep: we don't require a releasenote for your patch | 09:49 |
| *** SrinivasD has joined #openstack-cinder | 10:00 | |
| *** SrinivasD has quit IRC | 10:02 | |
| *** SrinivasD has joined #openstack-cinder | 10:03 | |
| ruffian_sheep | whoami-rajat:Ok, I am correcting it. | 10:05 |
| *** ruffian_sheep has quit IRC | 10:07 | |
| *** yaawang has quit IRC | 10:21 | |
| *** davidsha has joined #openstack-cinder | 10:23 | |
| openstackgerrit | Ivaylo Mitev proposed openstack/cinder master: VMware VMDK: Initial size of volume from image https://review.opendev.org/669892 | 10:38 |
| *** yaawang has joined #openstack-cinder | 10:38 | |
| *** yaawang has quit IRC | 10:44 | |
| *** ociuhandu has quit IRC | 11:13 | |
| *** ociuhandu has joined #openstack-cinder | 11:16 | |
| *** ociuhandu has quit IRC | 11:21 | |
| *** lixiaoy1 has quit IRC | 11:27 | |
| *** SrinivasD has quit IRC | 11:27 | |
| *** ociuhandu has joined #openstack-cinder | 11:35 | |
| *** carloss has joined #openstack-cinder | 11:37 | |
| *** ociuhandu has quit IRC | 11:40 | |
| *** yaawang has joined #openstack-cinder | 11:43 | |
| *** ociuhandu has joined #openstack-cinder | 11:49 | |
| *** brinzhang_ has joined #openstack-cinder | 11:50 | |
| *** brinzhang has quit IRC | 11:50 | |
| *** brinzhang has joined #openstack-cinder | 11:50 | |
| *** dviroel has joined #openstack-cinder | 11:53 | |
| *** brinzhang_ has quit IRC | 11:54 | |
| *** ociuhandu has quit IRC | 12:01 | |
| *** markvoelker has joined #openstack-cinder | 12:04 | |
| *** markvoelker has quit IRC | 12:06 | |
| *** markvoelker has joined #openstack-cinder | 12:06 | |
| *** henriqueof has joined #openstack-cinder | 12:11 | |
| openstackgerrit | Raghavendra Tilay proposed openstack/cinder master: 3PAR: Add Peer Persistence support https://review.opendev.org/674796 | 12:15 |
| *** mvkr has quit IRC | 12:21 | |
| *** udesale has quit IRC | 12:32 | |
| *** udesale has joined #openstack-cinder | 12:32 | |
| *** enriquetaso has joined #openstack-cinder | 12:57 | |
| *** mriedem has joined #openstack-cinder | 13:00 | |
| *** eharney has joined #openstack-cinder | 13:00 | |
| *** enriquetaso has quit IRC | 13:03 | |
| *** enriquetaso has joined #openstack-cinder | 13:08 | |
| *** ociuhandu has joined #openstack-cinder | 13:08 | |
| *** mvkr has joined #openstack-cinder | 13:11 | |
| eharney | https://bugs.launchpad.net/cinder/+bug/1839069 is kind of perplexing, don't we have CI jobs that test this? | 13:18 |
| openstack | Launchpad bug 1839069 in Cinder "Uploading volume to glance is broken" [High,In progress] - Assigned to Gorka Eguileor (gorka) | 13:18 |
| smcginnis | For those looking at py3.7 installation on distros that don't have it, I was just reminded of pyenv - https://amaral.northwestern.edu/resources/guides/pyenv-tutorial | 13:18 |
| smcginnis | eharney: I was thinking the same thing. | 13:18 |
| eharney | probably worth a deep dive into what we're running exactly | 13:20 |
| jungleboyj | smcginnis: Could you add a note about that to the Wiki page rosmaita created? | 13:21 |
| tosky | eharney: do you know which is the patch which introduced the bug ? | 13:22 |
| eharney | tosky: yes, i wrote it | 13:24 |
| eharney | https://review.opendev.org/#/c/668943/ | 13:24 |
| tosky | uhm, weird; that test is executed | 13:28 |
| eharney | maybe it only affects a specific path/config, i didn't look too deep yet | 13:29 |
| smcginnis | jungleboyj: Do you have that link? I don't see it linked off of https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver | 13:31 |
| tosky | geguileo: do you have a link of the failed job? | 13:31 |
| geguileo | tosky: I don't have it around... | 13:33 |
| tosky | geguileo: ok, no rush | 13:33 |
| eharney | maybe it's in one of the dozens of red jobs on the original patch? | 13:34 |
| geguileo | tosky: I was pinged in private and provided the stack trace I posted in the bug | 13:35 |
| jungleboyj | smcginnis: https://wiki.openstack.org/wiki/Cinder/3rdParty-drivers-py3-update | 13:35 |
| geguileo | tosky: I looked at the code and saw the issue | 13:35 |
| tosky | geguileo: uh, just to understand whether something was missing in the tempest test, if there is a specific code path which is not normally triggered; on the other hand, your patch provides a unit test, so it should be fine | 13:36 |
| geguileo | tosky: no, the tempest test caught the issue on the Kaminario backend | 13:36 |
| geguileo | tosky: but some drivers may overwrite the driver method, so it doesn't get called | 13:37 |
| geguileo | and some other CIs may be blacklisting the test... | 13:37 |
| eharney | which method is that? copy_image_to_volume? | 13:37 |
| eharney | er, i meant copy_volume_to_image | 13:38 |
| *** ociuhandu has quit IRC | 13:40 | |
| openstackgerrit | Bhaa Shakur proposed openstack/cinder master: Zadara VPSA: Move to API access key authentication https://review.opendev.org/670715 | 13:45 |
| *** alkhodos has joined #openstack-cinder | 13:45 | |
| openstackgerrit | futaotao proposed openstack/cinder master: Support manage/unmanage in the FusionStorage Cinder Driver https://review.opendev.org/669806 | 13:47 |
| *** lemko has joined #openstack-cinder | 13:48 | |
| geguileo | eharney: yes, and there are some driver overwritting the method | 13:55 |
| geguileo | eharney: LVM is one of them | 13:56 |
| *** Luzi has quit IRC | 13:56 | |
| eharney | ah, i was testing this on Ceph during development, it also is | 13:56 |
| *** abishop_ is now known as abishop | 13:57 | |
| *** dpawlik has quit IRC | 14:10 | |
| *** ociuhandu has joined #openstack-cinder | 14:11 | |
| *** altlogbot_0 has quit IRC | 14:12 | |
| *** altlogbot_1 has joined #openstack-cinder | 14:13 | |
| *** N3l1x has joined #openstack-cinder | 14:14 | |
| *** N3l1x_ has joined #openstack-cinder | 14:14 | |
| *** N3l1x_ has quit IRC | 14:15 | |
| *** ociuhandu has quit IRC | 14:17 | |
| *** brinzhang has quit IRC | 14:24 | |
| *** brinzhang has joined #openstack-cinder | 14:25 | |
| *** brinzhang has quit IRC | 14:26 | |
| *** ociuhandu has joined #openstack-cinder | 14:27 | |
| openstackgerrit | futaotao proposed openstack/cinder master: Support manage/unmanage in the FusionStorage Cinder Driver https://review.opendev.org/669806 | 14:28 |
| *** brinzhang has joined #openstack-cinder | 14:29 | |
| *** brinzhang has quit IRC | 14:29 | |
| *** brinzhang has joined #openstack-cinder | 14:30 | |
| *** KeithMnemonic has joined #openstack-cinder | 14:36 | |
| whoami-rajat | geguileo: Hey, i've a query, fetching from DB works fine now but to fetch an OVO , i tried the following methods [1][2] but while running tests all the db methods inside cinder code gets mocked, not sure why this is happening. | 14:38 |
| whoami-rajat | [1] https://github.com/openstack/cinder/blob/master/cinder/objects/volume_type.py#L159 | 14:38 |
| whoami-rajat | [2] https://github.com/openstack/cinder/blob/master/cinder/objects/volume_type.py#L183 | 14:38 |
| whoami-rajat | geguileo: http://paste.openstack.org/show/755570/ traceback | 14:46 |
| geguileo | whoami-rajat: At some point you are returning a Mock instead of a string | 14:48 |
| geguileo | as the name of a volume type | 14:48 |
| geguileo | and then when the code tries to assign it to the VolumeType OVO 'name' field | 14:48 |
| geguileo | it fails because it does a check against the defined type for the field, which is a string | 14:49 |
| geguileo | whoami-rajat: it looks like it's a testing "issue" | 14:49 |
| geguileo | aka something in the test that is failing needs to be changed | 14:49 |
| whoami-rajat | geguileo: i understand that. but i'm executing | 14:51 |
| whoami-rajat | `self.DEFAULT_TYPE = cinder_objs.VolumeType.get_by_name_or_id(objects.CONTEXT,'__DEFAULT__'))` | 14:51 |
| whoami-rajat | and running predefined dbms tests | 14:51 |
| whoami-rajat | geguileo: and i couldn't find mocking of any db method in cinderlib base tests. | 14:52 |
| geguileo | whoami-rajat: well, the DB persistence code is replacing them iirc | 14:53 |
| whoami-rajat | geguileo: not all the methods right? | 14:53 |
| geguileo | whoami-rajat: https://github.com/openstack/cinderlib/blob/master/cinderlib/persistence/dbms.py#L42 | 14:55 |
| geguileo | whoami-rajat: a bunch of methods are changed there... | 14:55 |
| geguileo | whoami-rajat: is your patch up for review? I could have a look and see what's going on | 14:55 |
| whoami-rajat | geguileo: yep, but not the one i'm using | 14:56 |
| whoami-rajat | geguileo: i can push it but would've errors | 14:56 |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinderlib master: WIP: Add default volume type flow in dbms persistence plugin https://review.opendev.org/674826 | 15:00 |
| geguileo | whoami-rajat: you can either push it to openstack gerrit or push it to a github repo | 15:01 |
| geguileo | whoami-rajat: I would just need to see the code to figure out what's going on | 15:02 |
| whoami-rajat | geguileo: ^^ done | 15:02 |
| *** spatel has joined #openstack-cinder | 15:15 | |
| geguileo | whoami-rajat: unfortunately we need to construct the OVO manually reading from the DB | 15:22 |
| geguileo | whoami-rajat: something similar to this http://paste.openstack.org/show/755576/ | 15:23 |
| *** hoonetorg has quit IRC | 15:24 | |
| whoami-rajat | geguileo: oh i tried that too. but it gave a protected member access error or something. will try again. | 15:25 |
| whoami-rajat | geguileo: could you also explain why we need to do it this way | 15:25 |
| *** hoonetorg has joined #openstack-cinder | 15:25 | |
| whoami-rajat | geguileo: i see OVO fetching being done like this but this didn't work too | 15:26 |
| whoami-rajat | https://github.com/openstack/cinderlib/blob/master/cinderlib/persistence/dbms.py#L129 | 15:26 |
| geguileo | whoami-rajat: because the tests by default mock everything https://github.com/openstack/cinderlib/blob/master/cinderlib/tests/unit/base.py#L38 | 15:27 |
| geguileo | whoami-rajat: and at the point you are trying to read the data the new persistence isn't still in place | 15:27 |
| geguileo | whoami-rajat: so we need to use self.db_instance instead of trying to use db_api or sqla_api | 15:29 |
| geguileo | whoami-rajat: I don't have the cinder patch in place, so I don't know if that code is sufficient or if some modifications are necessary | 15:29 |
| whoami-rajat | geguileo: that worked. | 15:32 |
| geguileo | whoami-rajat: great!! | 15:32 |
| whoami-rajat | geguileo: thanks for the explanation, sometimes it's quite hard to understand the flow between cinder and cinderlib | 15:32 |
| geguileo | whoami-rajat: indeed it is!!! And for the tests it's even worse | 15:33 |
| hemna_ | ™mornin | 15:33 |
| whoami-rajat | geguileo: also i see the ceph initialization steps isn't there in the cinderlib docs but it works great with it (for me). maybe it's quite obvious to set it up, but would you me to add it in the docs. | 15:34 |
| geguileo | whoami-rajat: what initialization steps? | 15:35 |
| *** enriquetaso has quit IRC | 15:35 | |
| whoami-rajat | geguileo: oh, i thought i'm the only one. apologies for bugging everyday :) | 15:35 |
| whoami-rajat | geguileo: here https://docs.openstack.org/cinderlib/latest/topics/backends.html | 15:35 |
| *** ociuhandu has quit IRC | 15:36 | |
| geguileo | whoami-rajat: In theory we should merge this https://review.opendev.org/#/c/651880 | 15:36 |
| geguileo | whoami-rajat: and in there there are explanations for multiple backends | 15:37 |
| geguileo | whoami-rajat: the Ceph contents are similar to what you were looking to add? | 15:37 |
| whoami-rajat | geguileo: more detailed than i thought :) | 15:38 |
| whoami-rajat | geguileo: i will take a look and provide feedback asap | 15:38 |
| geguileo | whoami-rajat: I should update that patch and add a reference from the page you referenced | 15:38 |
| *** alkhodos has quit IRC | 15:39 | |
| whoami-rajat | geguileo: oh yeah, would be great to add basic config steps to the BACKEND page. | 15:39 |
| whoami-rajat | for ceph atleast | 15:40 |
| geguileo | whoami-rajat: I was just going to say, for other backends, check the validated drivers page | 15:40 |
| whoami-rajat | geguileo: that also works for me | 15:41 |
| *** ociuhandu has joined #openstack-cinder | 15:48 | |
| *** helenafm has quit IRC | 15:49 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinderlib master: Add validation and validated documentation https://review.opendev.org/651880 | 15:51 |
| geguileo | whoami-rajat: ^ there it is | 15:51 |
| *** ociuhandu has quit IRC | 16:03 | |
| *** mvkr has quit IRC | 16:20 | |
| *** enriquetaso has joined #openstack-cinder | 16:23 | |
| *** tosky has quit IRC | 16:25 | |
| *** sahid has quit IRC | 16:30 | |
| *** markvoelker has quit IRC | 16:32 | |
| *** enriquetaso has quit IRC | 16:32 | |
| *** davidsha has quit IRC | 16:33 | |
| *** markvoelker has joined #openstack-cinder | 16:44 | |
| *** e0ne has quit IRC | 16:48 | |
| *** N3l1x has quit IRC | 16:49 | |
| *** ociuhandu has joined #openstack-cinder | 16:49 | |
| *** ociuhandu has quit IRC | 16:57 | |
| *** ociuhandu has joined #openstack-cinder | 16:58 | |
| *** alkhodos has joined #openstack-cinder | 16:59 | |
| whoami-rajat | geguileo: great! thanks. | 16:59 |
| *** udesale has quit IRC | 17:00 | |
| *** ociuhandu has quit IRC | 17:22 | |
| *** enriquetaso has joined #openstack-cinder | 17:23 | |
| *** ociuhandu has joined #openstack-cinder | 17:23 | |
| *** qqmber has joined #openstack-cinder | 17:24 | |
| qqmber | Hi! | 17:24 |
| qqmber | How can I move (migrate) a volume from host A, dick i, to host B, disk j? they have different Volume Type (let's call them type-i, type-j). | 17:26 |
| qqmber | I tried as user with "Change Volume Type" --> Migration Policy: "Never" and "On Demand" and no luck... noting happend... | 17:27 |
| *** ociuhandu has quit IRC | 17:27 | |
| qqmber | I also tried as admin "Migrate Volume" -> With and wothout "Force Host Copy"... no luck | 17:28 |
| *** ociuhandu has joined #openstack-cinder | 17:28 | |
| qqmber | with the volume attached to an instance and the opposite... no luck | 17:28 |
| qqmber | How it has to be done? | 17:28 |
| *** senrique_ has joined #openstack-cinder | 17:33 | |
| *** enriquetaso has quit IRC | 17:35 | |
| *** lpetrut has quit IRC | 17:44 | |
| *** ociuhandu has quit IRC | 17:48 | |
| *** sapd1_x has joined #openstack-cinder | 17:51 | |
| *** spsurya has quit IRC | 17:52 | |
| *** ociuhandu has joined #openstack-cinder | 18:02 | |
| *** lpetrut has joined #openstack-cinder | 18:03 | |
| *** qqmber has quit IRC | 18:03 | |
| *** ociuhandu has quit IRC | 18:04 | |
| *** ociuhandu has joined #openstack-cinder | 18:05 | |
| *** ociuhandu has quit IRC | 18:09 | |
| *** e0ne has joined #openstack-cinder | 18:17 | |
| *** sapd1_x has quit IRC | 18:28 | |
| *** e0ne has quit IRC | 18:31 | |
| *** henriqueof has quit IRC | 18:38 | |
| *** senrique_ has quit IRC | 18:41 | |
| jungleboyj | Can't answer your question if you do a drivebye. | 18:42 |
| jungleboyj | *Sigh* | 18:42 |
| smcginnis | Hehe, yep. | 18:45 |
| * jungleboyj yells GET OFF MY LAWN | 18:45 | |
| *** tesseract has quit IRC | 18:52 | |
| jungleboyj | Anyone seen this problem with devstack lately? | 18:56 |
| jungleboyj | ‘/opt/stack/requirements/.venv/bin/pip’: No such file or directory | 18:57 |
| jungleboyj | I thought maybe my ubuntu image was messed up. This is on a fresh image an still this this error. | 18:57 |
| *** jgriffith has joined #openstack-cinder | 19:00 | |
| *** e0ne has joined #openstack-cinder | 19:04 | |
| *** mriedem has quit IRC | 19:08 | |
| *** mriedem has joined #openstack-cinder | 19:09 | |
| *** e0ne has quit IRC | 19:13 | |
| *** e0ne has joined #openstack-cinder | 19:14 | |
| *** e0ne has quit IRC | 19:15 | |
| openstackgerrit | Chris M proposed openstack/os-brick master: Fix bad argument to iscsiadm in iSCSI discovery https://review.opendev.org/674669 | 19:20 |
| openstackgerrit | Jon Bernard proposed openstack/cinder master: Don't allow retype to encrypted+multiattach type https://review.opendev.org/673556 | 19:23 |
| *** lpetrut has quit IRC | 19:39 | |
| *** henriqueof has joined #openstack-cinder | 19:43 | |
| *** e0ne has joined #openstack-cinder | 19:51 | |
| *** senrique_ has joined #openstack-cinder | 19:51 | |
| woojay | jungleboyj: I haven't seen that specific one. I'm on a bit older branch though... | 19:56 |
| jungleboyj | woojay: Thanks. I dug around and ended up just installing the VENV manually. Now things seem to work. I had tried that but then hit other issues. Those must have been due to me reusing the environment. | 19:57 |
| woojay | I can imagine so. Quite a few venvs for devstack and for testing... | 19:58 |
| jungleboyj | *Sigh* | 19:59 |
| woojay | fwiw, my CI is back up, and it's running tests on commit 669b0c5e4eba1a721dc02650e000fb0ac6bfd11b off /stable/stein. | 19:59 |
| woojay | it's a bit old since 4/26 but has been testing pretty well. | 20:00 |
| jungleboyj | woojay: Cool. Thanks for getting it back up and running. | 20:01 |
| woojay | of course. | 20:02 |
| *** lpetrut has joined #openstack-cinder | 20:08 | |
| *** alkhodos has quit IRC | 20:21 | |
| *** eharney has quit IRC | 20:22 | |
| *** alkhodos has joined #openstack-cinder | 20:23 | |
| *** lemko has quit IRC | 20:27 | |
| *** eharney has joined #openstack-cinder | 20:37 | |
| *** mriedem is now known as mriedem_afk | 20:46 | |
| *** e0ne has quit IRC | 20:48 | |
| openstackgerrit | Chris M proposed openstack/cinder master: Additional multiattach fixes for dothill driver https://review.opendev.org/674534 | 20:50 |
| *** whoami-rajat has quit IRC | 20:56 | |
| *** strigazi has joined #openstack-cinder | 21:01 | |
| *** lpetrut has quit IRC | 21:08 | |
| *** spatel has quit IRC | 21:09 | |
| *** senrique_ has quit IRC | 21:15 | |
| *** mriedem_afk is now known as mriedem | 21:20 | |
| *** markvoelker has quit IRC | 21:38 | |
| *** henriqueof has quit IRC | 21:45 | |
| *** hedvig_01 has joined #openstack-cinder | 21:50 | |
| openstackgerrit | Chris M proposed openstack/cinder master: Create Seagate driver from dothill driver https://review.opendev.org/671195 | 21:50 |
| *** Kuirong has quit IRC | 21:51 | |
| openstackgerrit | Merged openstack/cinderlib master: Add validation and validated documentation https://review.opendev.org/651880 | 22:01 |
| hedvig_01 | Hi, We are already in the cinder codebase in stein. We looked at py3.7 installation this morning after the email sent across. The CI setup is py3.7. However, it looks like the deadline was july 22-26 Train-2 milestone which has been missed. Could you please guide us with the future steps? Is qualifying for train out of picture? | 22:37 |
| *** henriqueof has joined #openstack-cinder | 22:43 | |
| smcginnis | hedvig_01: The CI just needs to get updates ASAP to run the tests under py3.7 runtime. | 22:48 |
| hedvig_01 | CI has been updated to run the tests under py3.7 runtime. | 22:48 |
| smcginnis | Oh great. Do you have a link to one I can check out? | 22:49 |
| hedvig_01 | I see a few failures which can be fixed easily | 22:49 |
| hedvig_01 | sure. | 22:50 |
| smcginnis | OK, good. That was the need to get them to run using 3.7. There are a few changes in Python that can cause issues versus 3.5. So just get a patch up to fix up whatever it is and you should be good to go. | 22:50 |
| hedvig_01 | https://review.opendev.org/#/c/671195/ | 22:50 |
| hedvig_01 | this is one of the runs which failed after 3.7 transition | 22:50 |
| hedvig_01 | We can make a fix soon. However, I wanted to know if there is a chance to qualify for train release | 22:51 |
| *** tkajinam has joined #openstack-cinder | 22:51 | |
| smcginnis | Yes, the driver is still in for train. But as per what we agreed on for this cycle, if drivers don't get CIs running, we would need to set the UNSUPPORT flag on them to "deprecate" them in train and remove in the U release. But if you can get those things fixed, then it won't be an issue. | 22:52 |
| hedvig_01 | Awesome thank you! By when do we have to have to get these fixed? Is there a date? | 22:55 |
| smcginnis | Since it's past the initial deadline, I think the soonest you can get things done, the better. Not like you have to drop everything, but at least make it a priority so it doesn't get too much later. | 22:57 |
| hedvig_01 | Sure. Thank you. I've already started to work on it. I'll create a new git branch and will be pushing the fixes soon on that branch for review. | 23:11 |
| *** markvoelker has joined #openstack-cinder | 23:13 | |
| *** markvoelker has quit IRC | 23:18 | |
| *** rcernin has joined #openstack-cinder | 23:26 | |
| *** mriedem has quit IRC | 23:40 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!