Tuesday, 2019-08-06

openstackgerritTejdeep Kautharam proposed openstack/cinder master: Rsdlib changed providing_pools interface  https://review.opendev.org/67067700:35
openstackgerritTejdeep Kautharam proposed openstack/cinder master: extend_volume method fails if volume size is float  https://review.opendev.org/67415400:35
*** mriedem has quit IRC00:38
*** markvoelker has joined #openstack-cinder00:41
*** markvoelker has quit IRC00:51
*** markvoelker has joined #openstack-cinder01:17
*** spsurya has joined #openstack-cinder01:17
*** Kuirong has joined #openstack-cinder01:28
*** altlogbot_3 has quit IRC01:37
*** altlogbot_0 has joined #openstack-cinder01:38
*** markvoelker has quit IRC01:38
*** lixiaoy1 has joined #openstack-cinder01:48
*** lixiaoy1 has quit IRC01:49
*** lixiaoy1 has joined #openstack-cinder01:56
*** markvoelker has joined #openstack-cinder02:42
*** whoami-rajat has joined #openstack-cinder03:07
*** markvoelker has quit IRC03:15
*** enriquetaso has quit IRC03:56
*** ircuser-1 has joined #openstack-cinder04:34
*** udesale has joined #openstack-cinder04:38
*** Luzi has joined #openstack-cinder05:05
*** markvoelker has joined #openstack-cinder05:28
*** markvoelker has quit IRC05:33
*** lixiaoy1 has quit IRC05:35
*** lixiaoy1 has joined #openstack-cinder05:36
*** tkajinam has quit IRC05:38
*** tkajinam has joined #openstack-cinder05:38
openstackgerritMerged openstack/cinder master: Update Veritas Access in Support Matrix  https://review.opendev.org/67465405:48
*** gkadam_ has joined #openstack-cinder05:54
*** gkadam_ has quit IRC05:54
*** gkadam has quit IRC05:56
*** sahid has joined #openstack-cinder06:19
*** vishalmanchanda has joined #openstack-cinder06:36
*** markvoelker has joined #openstack-cinder06:36
openstackgerritSam Wan proposed openstack/cinder master: upload_volume() should accept 'compress' paramter.  https://review.opendev.org/67474006:55
*** markvoelker has quit IRC07:09
*** tosky has joined #openstack-cinder07:24
*** tesseract has joined #openstack-cinder07:30
*** rcernin has quit IRC08:04
*** ociuhandu has joined #openstack-cinder08:05
*** ociuhandu has quit IRC08:05
*** tkajinam has quit IRC08:06
*** lpetrut has joined #openstack-cinder08:10
*** helenafm has joined #openstack-cinder08:12
*** ociuhandu has joined #openstack-cinder08:17
*** ociuhandu has quit IRC08:30
*** e0ne has joined #openstack-cinder08:35
*** ociuhandu has joined #openstack-cinder08:51
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix upload volume to glance  https://review.opendev.org/67475808:55
*** ociuhandu has quit IRC09: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-cinder09:40
*** abishop has quit IRC09:41
walshh_apologies, second review is https://review.opendev.org/#/c/66983509:42
*** ruffian_sheep has joined #openstack-cinder09:45
ruffian_sheepwhoami-rajat:Hi,are u online?09:45
*** ociuhandu has joined #openstack-cinder09:47
whoami-rajatruffian_sheep:  Hey09:47
ruffian_sheepwhoami-rajat:Did you have time to discuss the issue with other members yesterday?09:48
whoami-rajatruffian_sheep: i think sean already replied on your patch09:48
whoami-rajatruffian_sheep: we don't require a releasenote for your patch09:49
*** SrinivasD has joined #openstack-cinder10:00
*** SrinivasD has quit IRC10:02
*** SrinivasD has joined #openstack-cinder10:03
ruffian_sheepwhoami-rajat:Ok, I am correcting it.10:05
*** ruffian_sheep has quit IRC10:07
*** yaawang has quit IRC10:21
*** davidsha has joined #openstack-cinder10:23
openstackgerritIvaylo Mitev proposed openstack/cinder master: VMware VMDK: Initial size of volume from image  https://review.opendev.org/66989210:38
*** yaawang has joined #openstack-cinder10:38
*** yaawang has quit IRC10:44
*** ociuhandu has quit IRC11:13
*** ociuhandu has joined #openstack-cinder11:16
*** ociuhandu has quit IRC11:21
*** lixiaoy1 has quit IRC11:27
*** SrinivasD has quit IRC11:27
*** ociuhandu has joined #openstack-cinder11:35
*** carloss has joined #openstack-cinder11:37
*** ociuhandu has quit IRC11:40
*** yaawang has joined #openstack-cinder11:43
*** ociuhandu has joined #openstack-cinder11:49
*** brinzhang_ has joined #openstack-cinder11:50
*** brinzhang has quit IRC11:50
*** brinzhang has joined #openstack-cinder11:50
*** dviroel has joined #openstack-cinder11:53
*** brinzhang_ has quit IRC11:54
*** ociuhandu has quit IRC12:01
*** markvoelker has joined #openstack-cinder12:04
*** markvoelker has quit IRC12:06
*** markvoelker has joined #openstack-cinder12:06
*** henriqueof has joined #openstack-cinder12:11
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Add Peer Persistence support  https://review.opendev.org/67479612:15
*** mvkr has quit IRC12:21
*** udesale has quit IRC12:32
*** udesale has joined #openstack-cinder12:32
*** enriquetaso has joined #openstack-cinder12:57
*** mriedem has joined #openstack-cinder13:00
*** eharney has joined #openstack-cinder13:00
*** enriquetaso has quit IRC13:03
*** enriquetaso has joined #openstack-cinder13:08
*** ociuhandu has joined #openstack-cinder13:08
*** mvkr has joined #openstack-cinder13:11
eharneyhttps://bugs.launchpad.net/cinder/+bug/1839069 is kind of perplexing, don't we have CI jobs that test this?13:18
openstackLaunchpad bug 1839069 in Cinder "Uploading volume to glance is broken" [High,In progress] - Assigned to Gorka Eguileor (gorka)13:18
smcginnisFor 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-tutorial13:18
smcginniseharney: I was thinking the same thing.13:18
eharneyprobably worth a deep dive into what we're running exactly13:20
jungleboyjsmcginnis:  Could you add a note about that to the Wiki page rosmaita created?13:21
toskyeharney: do you know which is the patch which introduced the bug ?13:22
eharneytosky: yes, i wrote it13:24
toskyuhm, weird; that test is executed13:28
eharneymaybe it only affects a specific path/config, i didn't look too deep yet13:29
smcginnisjungleboyj: Do you have that link? I don't see it linked off of https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver13:31
toskygeguileo: do you have a link of the failed job?13:31
geguileotosky: I don't have it around...13:33
toskygeguileo: ok, no rush13:33
eharneymaybe it's in one of the dozens of red jobs on the original patch?13:34
geguileotosky: I was pinged in private and provided the stack trace I posted in the bug13:35
jungleboyjsmcginnis: https://wiki.openstack.org/wiki/Cinder/3rdParty-drivers-py3-update13:35
geguileotosky: I looked at the code and saw the issue13:35
toskygeguileo: 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 fine13:36
geguileotosky: no, the tempest test caught the issue on the Kaminario backend13:36
geguileotosky: but some drivers may overwrite the driver method, so it doesn't get called13:37
geguileoand some other CIs may be blacklisting the test...13:37
eharneywhich method is that?  copy_image_to_volume?13:37
eharneyer, i meant copy_volume_to_image13:38
*** ociuhandu has quit IRC13:40
openstackgerritBhaa Shakur proposed openstack/cinder master: Zadara VPSA: Move to API access key authentication  https://review.opendev.org/67071513:45
*** alkhodos has joined #openstack-cinder13:45
openstackgerritfutaotao proposed openstack/cinder master: Support manage/unmanage in the FusionStorage Cinder Driver  https://review.opendev.org/66980613:47
*** lemko has joined #openstack-cinder13:48
geguileoeharney: yes, and there are some driver overwritting the method13:55
geguileoeharney: LVM is one of them13:56
*** Luzi has quit IRC13:56
eharneyah, i was testing this on Ceph during development, it also is13:56
*** abishop_ is now known as abishop13:57
*** dpawlik has quit IRC14:10
*** ociuhandu has joined #openstack-cinder14:11
*** altlogbot_0 has quit IRC14:12
*** altlogbot_1 has joined #openstack-cinder14:13
*** N3l1x has joined #openstack-cinder14:14
*** N3l1x_ has joined #openstack-cinder14:14
*** N3l1x_ has quit IRC14:15
*** ociuhandu has quit IRC14:17
*** brinzhang has quit IRC14:24
*** brinzhang has joined #openstack-cinder14:25
*** brinzhang has quit IRC14:26
*** ociuhandu has joined #openstack-cinder14:27
openstackgerritfutaotao proposed openstack/cinder master: Support manage/unmanage in the FusionStorage Cinder Driver  https://review.opendev.org/66980614:28
*** brinzhang has joined #openstack-cinder14:29
*** brinzhang has quit IRC14:29
*** brinzhang has joined #openstack-cinder14:30
*** KeithMnemonic has joined #openstack-cinder14:36
whoami-rajatgeguileo:  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#L15914:38
whoami-rajat[2] https://github.com/openstack/cinder/blob/master/cinder/objects/volume_type.py#L18314:38
whoami-rajatgeguileo: http://paste.openstack.org/show/755570/ traceback14:46
geguileowhoami-rajat: At some point you are returning a Mock instead of a string14:48
geguileoas the name of a volume type14:48
geguileoand then when the code tries to assign it to the VolumeType OVO 'name' field14:48
geguileoit fails because it does a check against the defined type for the field, which is a string14:49
geguileowhoami-rajat: it looks like it's a testing "issue"14:49
geguileoaka something in the test that is failing needs to be changed14:49
whoami-rajatgeguileo: i understand that. but i'm executing14:51
whoami-rajat`self.DEFAULT_TYPE = cinder_objs.VolumeType.get_by_name_or_id(objects.CONTEXT,'__DEFAULT__'))`14:51
whoami-rajatand running predefined dbms tests14:51
whoami-rajatgeguileo: and i couldn't find mocking of any db method in cinderlib base tests.14:52
geguileowhoami-rajat: well, the DB persistence code is replacing them iirc14:53
whoami-rajatgeguileo: not all the methods right?14:53
geguileowhoami-rajat: https://github.com/openstack/cinderlib/blob/master/cinderlib/persistence/dbms.py#L4214:55
geguileowhoami-rajat: a bunch of methods are changed there...14:55
geguileowhoami-rajat: is your patch up for review? I could have a look and see what's going on14:55
whoami-rajatgeguileo: yep, but not the one i'm using14:56
whoami-rajatgeguileo: i can push it but would've  errors14:56
openstackgerritRajat Dhasmana proposed openstack/cinderlib master: WIP: Add default volume type flow in dbms persistence plugin  https://review.opendev.org/67482615:00
geguileowhoami-rajat: you can either push it to openstack gerrit or push it to a github repo15:01
geguileowhoami-rajat: I would just need to see the code to figure out what's going on15:02
whoami-rajatgeguileo: ^^ done15:02
*** spatel has joined #openstack-cinder15:15
geguileowhoami-rajat: unfortunately we need to construct the OVO manually reading from the DB15:22
geguileowhoami-rajat: something similar to this http://paste.openstack.org/show/755576/15:23
*** hoonetorg has quit IRC15:24
whoami-rajatgeguileo: oh i tried that too. but it gave a protected member access error or something. will try again.15:25
whoami-rajatgeguileo: could you also explain why we need to do it this way15:25
*** hoonetorg has joined #openstack-cinder15:25
whoami-rajatgeguileo: i see OVO fetching being done like this but this didn't work too15:26
geguileowhoami-rajat: because the tests by default mock everything https://github.com/openstack/cinderlib/blob/master/cinderlib/tests/unit/base.py#L3815:27
geguileowhoami-rajat: and at the point you are trying to read the data the new persistence isn't still in place15:27
geguileowhoami-rajat: so we need to use self.db_instance instead of trying to use db_api or sqla_api15:29
geguileowhoami-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 necessary15:29
whoami-rajatgeguileo: that worked.15:32
geguileowhoami-rajat: great!!15:32
whoami-rajatgeguileo: thanks for the explanation, sometimes it's quite hard to understand the flow between cinder and cinderlib15:32
geguileowhoami-rajat: indeed it is!!!  And for the tests it's even worse15:33
whoami-rajatgeguileo: 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
geguileowhoami-rajat: what initialization steps?15:35
*** enriquetaso has quit IRC15:35
whoami-rajatgeguileo: oh, i thought i'm the only one. apologies for bugging everyday :)15:35
whoami-rajatgeguileo: here https://docs.openstack.org/cinderlib/latest/topics/backends.html15:35
*** ociuhandu has quit IRC15:36
geguileowhoami-rajat: In theory we should merge this https://review.opendev.org/#/c/65188015:36
geguileowhoami-rajat: and in there there are explanations for multiple backends15:37
geguileowhoami-rajat: the Ceph contents are similar to what you were looking to add?15:37
whoami-rajatgeguileo: more detailed than i thought :)15:38
whoami-rajatgeguileo: i will take a look and provide feedback asap15:38
geguileowhoami-rajat: I should update that patch and add a reference from the page you referenced15:38
*** alkhodos has quit IRC15:39
whoami-rajatgeguileo: oh yeah, would be great to add basic config steps to the BACKEND page.15:39
whoami-rajatfor ceph atleast15:40
geguileowhoami-rajat: I was just going to say, for other backends, check the validated drivers page15:40
whoami-rajatgeguileo: that also works for me15:41
*** ociuhandu has joined #openstack-cinder15:48
*** helenafm has quit IRC15:49
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add validation and validated documentation  https://review.opendev.org/65188015:51
geguileowhoami-rajat: ^ there it is15:51
*** ociuhandu has quit IRC16:03
*** mvkr has quit IRC16:20
*** enriquetaso has joined #openstack-cinder16:23
*** tosky has quit IRC16:25
*** sahid has quit IRC16:30
*** markvoelker has quit IRC16:32
*** enriquetaso has quit IRC16:32
*** davidsha has quit IRC16:33
*** markvoelker has joined #openstack-cinder16:44
*** e0ne has quit IRC16:48
*** N3l1x has quit IRC16:49
*** ociuhandu has joined #openstack-cinder16:49
*** ociuhandu has quit IRC16:57
*** ociuhandu has joined #openstack-cinder16:58
*** alkhodos has joined #openstack-cinder16:59
whoami-rajatgeguileo: great! thanks.16:59
*** udesale has quit IRC17:00
*** ociuhandu has quit IRC17:22
*** enriquetaso has joined #openstack-cinder17:23
*** ociuhandu has joined #openstack-cinder17:23
*** qqmber has joined #openstack-cinder17:24
qqmberHow 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
qqmberI tried as user with "Change Volume Type" --> Migration Policy: "Never" and "On Demand" and no luck... noting happend...17:27
*** ociuhandu has quit IRC17:27
qqmberI also tried as admin "Migrate Volume" -> With and wothout "Force Host Copy"... no luck17:28
*** ociuhandu has joined #openstack-cinder17:28
qqmberwith the volume attached to an instance and the opposite... no luck17:28
qqmberHow it has to be done?17:28
*** senrique_ has joined #openstack-cinder17:33
*** enriquetaso has quit IRC17:35
*** lpetrut has quit IRC17:44
*** ociuhandu has quit IRC17:48
*** sapd1_x has joined #openstack-cinder17:51
*** spsurya has quit IRC17:52
*** ociuhandu has joined #openstack-cinder18:02
*** lpetrut has joined #openstack-cinder18:03
*** qqmber has quit IRC18:03
*** ociuhandu has quit IRC18:04
*** ociuhandu has joined #openstack-cinder18:05
*** ociuhandu has quit IRC18:09
*** e0ne has joined #openstack-cinder18:17
*** sapd1_x has quit IRC18:28
*** e0ne has quit IRC18:31
*** henriqueof has quit IRC18:38
*** senrique_ has quit IRC18:41
jungleboyjCan't answer your question if you do a drivebye.18:42
smcginnisHehe, yep.18:45
* jungleboyj yells GET OFF MY LAWN18:45
*** tesseract has quit IRC18:52
jungleboyjAnyone seen this problem with devstack lately?18:56
jungleboyj‘/opt/stack/requirements/.venv/bin/pip’: No such file or directory18:57
jungleboyjI 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-cinder19:00
*** e0ne has joined #openstack-cinder19:04
*** mriedem has quit IRC19:08
*** mriedem has joined #openstack-cinder19:09
*** e0ne has quit IRC19:13
*** e0ne has joined #openstack-cinder19:14
*** e0ne has quit IRC19:15
openstackgerritChris M proposed openstack/os-brick master: Fix bad argument to iscsiadm in iSCSI discovery  https://review.opendev.org/67466919:20
openstackgerritJon Bernard proposed openstack/cinder master: Don't allow retype to encrypted+multiattach type  https://review.opendev.org/67355619:23
*** lpetrut has quit IRC19:39
*** henriqueof has joined #openstack-cinder19:43
*** e0ne has joined #openstack-cinder19:51
*** senrique_ has joined #openstack-cinder19:51
woojayjungleboyj: I haven't seen that specific one.  I'm on a bit older branch though...19:56
jungleboyjwoojay:  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
woojayI can imagine so.  Quite a few venvs for devstack and for testing...19:58
woojayfwiw, my CI is back up, and it's running tests on commit 669b0c5e4eba1a721dc02650e000fb0ac6bfd11b off /stable/stein.19:59
woojayit's a bit old since 4/26 but has been testing pretty well.20:00
jungleboyjwoojay: Cool.  Thanks for getting it back up and running.20:01
woojayof course.20:02
*** lpetrut has joined #openstack-cinder20:08
*** alkhodos has quit IRC20:21
*** eharney has quit IRC20:22
*** alkhodos has joined #openstack-cinder20:23
*** lemko has quit IRC20:27
*** eharney has joined #openstack-cinder20:37
*** mriedem is now known as mriedem_afk20:46
*** e0ne has quit IRC20:48
openstackgerritChris M proposed openstack/cinder master: Additional multiattach fixes for dothill driver  https://review.opendev.org/67453420:50
*** whoami-rajat has quit IRC20:56
*** strigazi has joined #openstack-cinder21:01
*** lpetrut has quit IRC21:08
*** spatel has quit IRC21:09
*** senrique_ has quit IRC21:15
*** mriedem_afk is now known as mriedem21:20
*** markvoelker has quit IRC21:38
*** henriqueof has quit IRC21:45
*** hedvig_01 has joined #openstack-cinder21:50
openstackgerritChris M proposed openstack/cinder master: Create Seagate driver from dothill driver  https://review.opendev.org/67119521:50
*** Kuirong has quit IRC21:51
openstackgerritMerged openstack/cinderlib master: Add validation and validated documentation  https://review.opendev.org/65188022:01
hedvig_01Hi, 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-cinder22:43
smcginnishedvig_01: The CI just needs to get updates ASAP to run the tests under py3.7 runtime.22:48
hedvig_01CI has been updated to run the tests under py3.7 runtime.22:48
smcginnisOh great. Do you have a link to one I can check out?22:49
hedvig_01I see a few failures which can be fixed easily22:49
smcginnisOK, 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_01this is one of the runs which failed after 3.7 transition22:50
hedvig_01We can make a fix soon. However, I wanted to know if there is a chance to qualify for train release22:51
*** tkajinam has joined #openstack-cinder22:51
smcginnisYes, 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_01Awesome thank you! By when do we have to have to get these fixed? Is there a date?22:55
smcginnisSince 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_01Sure. 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-cinder23:13
*** markvoelker has quit IRC23:18
*** rcernin has joined #openstack-cinder23:26
*** mriedem has quit IRC23:40

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