Tuesday, 2020-01-21

*** zhanglong has quit IRC00:03
*** brinzhang has joined #openstack-cinder00:08
*** brinzhang_ has quit IRC00:12
openstackgerritTakashi Kajinami proposed openstack/python-cinderclient stable/stein: Add missed 'Server ID' output in attachment-list  https://review.opendev.org/70352500:31
openstackgerritTakashi Kajinami proposed openstack/python-cinderclient stable/rocky: Add missed 'Server ID' output in attachment-list  https://review.opendev.org/70352600:32
openstackgerritTakashi Kajinami proposed openstack/python-cinderclient stable/queens: Add missed 'Server ID' output in attachment-list  https://review.opendev.org/70352700:33
*** brinzhang has quit IRC00:39
*** zhanglong has joined #openstack-cinder00:41
*** brinzhang has joined #openstack-cinder00:52
*** brinzhang has quit IRC00:54
openstackgerritzhufl proposed openstack/cinder master: Fix some typos in docs  https://review.opendev.org/70256900:55
*** xiaolin has joined #openstack-cinder01:03
*** brinzhang has joined #openstack-cinder01:07
openstackgerritXuan Yandong proposed openstack/cinder-specs master: Add backup id to volume's metadata  https://review.opendev.org/70097701:12
*** whfnst has joined #openstack-cinder01:12
*** whfnst has quit IRC01:18
*** lseki has quit IRC01:40
*** Lucas_Gray has quit IRC01:49
*** ircuser-1 has joined #openstack-cinder02:02
*** ruffian_sheep has joined #openstack-cinder02:08
ruffian_sheepwhoami-rajat : Hi, are u online?02:09
ruffian_sheepwhoami-rajat : Assuming our storage has different software versions, we also provide different versions of the Cinder driver. How do I submit this change? Currently we have different software versions, all of which are provided for external use, but the new functions are more powerful. I see that huawei provides the cinder driver for v1-5.02:12
*** hoonetorg has quit IRC02:17
*** jvisser has quit IRC02:30
*** hoonetorg has joined #openstack-cinder02:32
*** ociuhandu has joined #openstack-cinder02:47
*** awalende has joined #openstack-cinder02:50
*** ociuhandu has quit IRC02:51
*** awalende has quit IRC02:55
*** zhanglong has quit IRC03:03
*** brinzhang has quit IRC03:27
*** ruffian_sheep has quit IRC04:27
*** udesale has joined #openstack-cinder04:44
*** udesale has quit IRC04:44
*** udesale has joined #openstack-cinder04:44
openstackgerritMerged openstack/cinder master: Add upgrade check for removed Veritas Access driver  https://review.opendev.org/70268905:21
*** ykarel|away is now known as ykarel05:23
*** udesale_ has joined #openstack-cinder05:34
*** udesale has quit IRC05:37
*** rchurch has quit IRC05:52
*** rchurch has joined #openstack-cinder05:54
openstackgerritAbhishek Kekane proposed openstack/cinder master: Support multiple stores of Glance  https://review.opendev.org/66167606:03
*** lpetrut has joined #openstack-cinder06:08
*** lpetrut has quit IRC06:09
*** lpetrut has joined #openstack-cinder06:10
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add Snapshot data integrity test  https://review.opendev.org/70249506:32
openstackgerritRajat Dhasmana proposed openstack/cinder master: Refactor README links  https://review.opendev.org/67959406:36
*** lpetrut has quit IRC06:49
*** do3meli has joined #openstack-cinder07:00
*** cervigni has joined #openstack-cinder07:07
cervigniHello everyone. Unfortunately we cannot still commit patches because of our not poissibility of joining the openstack aggreement07:07
cervigniI noticed though that in manifests/params.pp07:07
cervignithis is the third release rocky / stein / train that the Debian variable ceph_init_override on line 71 is wrong07:08
cervignias in debian should be '/etc/init.d/cinder-volume.override'07:08
cervignianyone in the mood to patch this?07:08
openstackgerritLiang Fang proposed openstack/cinder-specs master: Support volume local cache  https://review.opendev.org/68455607:26
*** ociuhandu has joined #openstack-cinder07:30
*** sahid has joined #openstack-cinder07:32
*** ociuhandu has quit IRC07:35
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add migration to make volume_type_id non nullable  https://review.opendev.org/68591407:36
*** ykarel is now known as ykarel|lunch07:36
*** lpetrut has joined #openstack-cinder07:38
*** awalende has joined #openstack-cinder08:07
*** tkajinam has quit IRC08:10
*** tesseract has joined #openstack-cinder08:20
*** ykarel|lunch is now known as ykarel08:38
*** martinkennelly has joined #openstack-cinder08:49
openstackgerritRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add Snapshot data integrity test  https://review.opendev.org/70249508:50
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add migration to make volume_type_id non nullable  https://review.opendev.org/68591408:53
*** do3meli has quit IRC09:03
*** tosky has joined #openstack-cinder09:07
*** tommylikehu has joined #openstack-cinder09:11
*** brinzhang has joined #openstack-cinder09:16
*** brinzhang has quit IRC09:22
*** brinzhang has joined #openstack-cinder09:23
*** udesale_ has quit IRC09:28
*** udesale_ has joined #openstack-cinder09:28
*** do3meli has joined #openstack-cinder09:31
*** whfnst has joined #openstack-cinder09:39
whfnstHi, Is any core available to look at https://review.opendev.org/#/c/692954/6 , it has +2.  Thank you09:42
*** whfnst has quit IRC09:44
openstackgerritLiang Fang proposed openstack/cinder-specs master: Support volume local cache  https://review.opendev.org/68455609:49
*** sahid has quit IRC09:53
*** whfnst has joined #openstack-cinder09:58
*** sapd1_x has joined #openstack-cinder09:59
*** whfnst has quit IRC10:00
*** ykarel is now known as ykarel|afk10:11
*** openstackgerrit has quit IRC10:12
*** sahid has joined #openstack-cinder10:18
*** e0ne has joined #openstack-cinder10:19
*** davidsha has joined #openstack-cinder10:26
*** sapd1_x has quit IRC10:29
*** CeeMac has joined #openstack-cinder10:43
*** ykarel|afk is now known as ykarel10:43
*** udesale_ has quit IRC10:51
*** awalende has quit IRC11:13
*** awalende has joined #openstack-cinder11:14
*** dviroel has joined #openstack-cinder11:25
*** Lucas_Gray has joined #openstack-cinder11:40
*** zhanglong has joined #openstack-cinder11:44
*** zhanglong has quit IRC11:53
*** jvisser has joined #openstack-cinder12:01
*** ociuhandu has joined #openstack-cinder12:12
*** ociuhandu has quit IRC12:13
*** Lucas_Gray has quit IRC12:13
*** tkajinam has joined #openstack-cinder12:14
*** Lucas_Gray has joined #openstack-cinder12:17
tkajinamhi. I'm now trying to backport a fix in python-cinderclient to some stable branches and noticed that pep8 job for stable/queens is broken because the job runs on py3 instead of py212:26
tkajinamSee https://review.opendev.org/#/c/703527/ to find its detail12:26
tkajinamIIUC, we've not yet migrated to py3 when stable/queens was created, so I guess that it should run on py2. Am I correct ?12:27
tkajinamI'll appreciate any suggestion about the above, thanks12:27
*** rcernin has quit IRC12:28
*** Luzi has joined #openstack-cinder12:28
hemnayah I'm not sure why they can't run the older branches against the older images that still support py212:33
hemnathe cI infrastructure never kept stable branches and images. :(12:33
hemnait sure would help now12:34
*** rcernin has joined #openstack-cinder12:43
*** Liang__ has joined #openstack-cinder12:46
*** dpawlik has joined #openstack-cinder12:46
*** Liang__ is now known as LiangFang12:46
*** ykarel is now known as ykarel|afk12:47
*** rcernin has quit IRC12:47
*** jvisser has quit IRC12:47
*** ociuhandu has joined #openstack-cinder12:52
rosmaitacinder virtual mid-cycle today. 1300-1500 UTC12:55
rosmaitajoin us at https://bluejeans.com/322852897312:55
LiangFango/12:56
rosmaitaetherpad for the virtual mid-cycle: https://etherpad.openstack.org/p/cinder-ussuri-mid-cycle-planning12:56
*** lseki has joined #openstack-cinder12:57
*** sfernand has joined #openstack-cinder12:57
*** do3meli has quit IRC12:57
LiangFangbluejeans still loading12:58
*** udesale has joined #openstack-cinder12:59
hemnamep12:59
*** ociuhandu has quit IRC13:01
jungleboyjHas the meeting started?  I just get the name of the meeting when I log in.13:04
geguileotosky: virtual PTG has started13:05
geguileojungleboyj: https://bluejeans.com/322852897313:05
geguileoeharney: ^13:05
jungleboyjYep.  That is the link I  am using geguileo13:06
geguileoyou should not need anything not even login, just to write your name13:06
jungleboyjTrying restarting my browser.13:06
jungleboyjgeguileo:  All I get is this:13:09
jungleboyjgeguileo:  Nevermind.  I am finally in.13:10
geguileojungleboyj: you are in!!!13:10
jungleboyjIt was being really slow.13:10
*** jvisser has joined #openstack-cinder13:16
toskygeguileo: now?13:17
toskyuh13:17
geguileotosky: yup13:17
toskyI mismarked the time13:17
toskyand I won't be around for a while :/13:17
geguileotosky: it's being recorded13:17
tkajinamhemna, sorry for my late reply because I was on openstack-infra to discuss gate job failures... and agree with you. will look around ci job and will submit a patch related to gate job setup, hopefully13:22
*** whoami-rajat_ has joined #openstack-cinder13:28
*** enriquetaso has joined #openstack-cinder13:29
*** dpawlik has quit IRC13:33
*** ganso has quit IRC13:33
*** ganso has joined #openstack-cinder13:34
*** Liang__ has joined #openstack-cinder13:37
*** LiangFang has quit IRC13:38
*** Lucas_Gray has quit IRC13:42
*** do3meli has joined #openstack-cinder13:47
*** ociuhandu has joined #openstack-cinder13:47
*** ociuhandu has quit IRC13:53
*** bhagyashris_ has joined #openstack-cinder14:09
*** bhagyashris_ has quit IRC14:10
hemnanp14:13
*** abishop_ has joined #openstack-cinder14:14
*** awalende has quit IRC14:16
*** abishop has quit IRC14:16
*** abishop_ is now known as abishop14:33
smcginnisjungleboyj: You're connected twice it seems.14:45
smcginniscervigni: I'm not sure what channel the puppet folks are in. #openstack-puppet maybe?14:46
*** Luzi has quit IRC14:46
jungleboyjsmcginnis:  Yeah, my computer is being a piece of crap so I connected from my phone.14:46
*** jrubenst has joined #openstack-cinder14:49
jungleboyjSorry for all the trouble today.15:05
Liang__I will start vacation from tomorrow, will try to attend the weekly meeting if not blocked on the road to hometown15:09
*** Liang__ is now known as LiangFang15:09
smcginnisLiangFang: Happy new year!15:10
jungleboyjLiangFang:  Happy New Year!15:10
jungleboyjStay healthy.15:10
LiangFangbecause network issue, I get lost for a while just now. so still not clear how cinder know the volume cache mode, and how cinder prevent live migration if it is a cache mode that not supported15:11
LiangFangthanks team15:11
rosmaitaLiangFang: i was not clear on that also, will ask in tomorrow's meeting (even if you are not there)15:12
rosmaitaand happy new year!15:12
LiangFangthanks rosmaita:)15:13
LiangFangI think cache mode is a runtime concept in compute node, and cinder is static15:14
LiangFangI think Nova can do this thing: prevent live migration if cache mode is not suitable15:14
rosmaitathat seems like a good approach15:16
*** tkajinam has quit IRC15:16
smcginnisQuestion for the team to think about - if we are not going to remove drivers anymore and only mark them as unsupported - should we merge back in the ones that have been removed over the last few cycles?15:17
rosmaitagood question15:18
*** LiangFang has quit IRC15:18
jungleboyjsmcginnis:  Yes, that is a good question and one of the ones that we will need to address.15:22
jungleboyjWe are starting a slippery slope here.15:22
*** sahid has quit IRC15:26
smcginnisWheee15:26
*** sahid has joined #openstack-cinder15:27
*** ociuhandu has joined #openstack-cinder15:30
jungleboyjI am surprised you are not fighting this discussion.  :-)15:32
*** sahid has quit IRC15:33
*** sahid has joined #openstack-cinder15:34
smcginnisI do think that slippery slope has oil poured on it, but I am fine going with what the community chooses. ;)15:35
jungleboyjOk.  That makes sense.15:36
smcginnisI have "third party CI fatigue" too.15:36
*** Lucas_Gray has joined #openstack-cinder15:42
*** lpetrut has quit IRC15:46
smcginnisA couple other thoughts about keeping unsupported drivers in-tree.15:48
smcginnisWe may need to move them or something to a place we can ignore with our linting jobs.15:49
smcginnisIt's probably likely that we will start getting py3 changes that will cause failures for older py2 code.15:49
smcginnisAnd requirements/dependencies that are no longer available.15:49
smcginnisSo we could start getting job failures due to bitrotting code being in the tree.15:50
smcginnisWe can probably fix some of the issues by making minor tweaks, but maybe not.15:50
*** ociuhandu has quit IRC15:51
rosmaitasmcginnis: these are all relevant concerns, please keep them coming!15:55
smcginnisThere was some push way way back about having a separate repo for drivers. Maybe we need an x/cinder-third-party-drivers repo that can easily be overlayed onto openstack/cinder.15:57
*** udesale has quit IRC15:59
*** ykarel|afk is now known as ykarel16:20
*** ociuhandu has joined #openstack-cinder16:23
*** ociuhandu has quit IRC16:28
*** tosky has quit IRC16:33
*** eharney has quit IRC16:35
*** ociuhandu has joined #openstack-cinder16:36
*** tommylikehu has quit IRC16:48
*** tesseract has quit IRC17:01
*** do3meli has quit IRC17:04
*** davidsha has quit IRC17:23
*** e0ne has quit IRC17:24
*** openstackgerrit has joined #openstack-cinder17:30
openstackgerritRajat Dhasmana proposed openstack/cinder master: Correct volume transfer filters  https://review.opendev.org/70365817:30
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Add Functional Tests: Show transfer with name  https://review.opendev.org/70295117:43
*** sahid has quit IRC17:44
*** eharney has joined #openstack-cinder17:46
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Add Functional Tests: Show transfer with name  https://review.opendev.org/70295117:47
*** ykarel is now known as ykarel|away17:47
*** ociuhandu_ has joined #openstack-cinder18:13
*** Lucas_Gray has quit IRC18:15
*** ociuhandu has quit IRC18:17
*** ociuhandu_ has quit IRC18:18
*** martinkennelly has quit IRC18:18
*** enriquetaso has quit IRC18:36
openstackgerritMerged openstack/cinder master: remotefs: remove invalid "external" arg on lock method  https://review.opendev.org/69410018:36
*** CeeMac has quit IRC18:51
*** tosky has joined #openstack-cinder19:25
openstackgerritMerged openstack/cinder master: Fujitsu Driver: Change the calculation of TPP's capacity  https://review.opendev.org/69295419:34
openstackgerritMarc Methot proposed openstack/cinder master: Configurable timeout of the QEMU img conversion  https://review.opendev.org/70368219:46
*** abishop_ has joined #openstack-cinder20:15
*** vnayar has joined #openstack-cinder20:15
*** abishop has quit IRC20:17
*** Lucas_Gray has joined #openstack-cinder20:26
*** eharney has quit IRC20:27
vnayarI'd like to add manage/unmanage (manage/unmanage snapshot) support to the Infinidat driver. Should I submit a spec or is a blueprint good enough?20:40
*** rcernin has joined #openstack-cinder20:42
openstackgerritMarc Methot proposed openstack/cinder master: Configurable timeout of the QEMU img conversion  https://review.opendev.org/70368220:42
mmethotsmcginnis, hey pretty green here, I ran "tox -e genopts" but it didnt do anything... I see it properly imported in "cinder/opts.py"20:46
mmethotsmcginnis, am I missing something or is this normal behaviour?20:47
smcginnismmethot: Did it modify the opts.py file?20:48
mmethotnope20:48
smcginnisHuh, I guess you are OK then. I couldn't remember, but I thought you needed to do that to pick up new config options in there.20:49
smcginnisvnayar: Blueprint or just propose the code. Definitely don't need a spec for that.20:49
mmethotsmcginnis, cool thanks for looking into it. From what I was thinking since I was registry under the same image_opts it should grab them the same manner, but idk ;)20:52
smcginnisOh right, it's only if a whole new set of opts is added I think. It's been awhile.20:55
vnayarsmcginnis, thanks20:56
*** whoami-rajat_ has quit IRC20:58
*** jvisser has quit IRC20:58
*** jmlowe has joined #openstack-cinder21:00
openstackgerritMarc Methot proposed openstack/cinder master: Configurable timeout of the QEMU img conversion  https://review.opendev.org/70368221:13
*** jvisser has joined #openstack-cinder21:13
mmethotsmcginnis, decided to add both conversion params to be configurable. Didnt make sense to just leave the one21:14
*** eharney has joined #openstack-cinder21:28
*** rchurch has quit IRC21:57
*** rchurch has joined #openstack-cinder22:00
*** eharney has quit IRC22:06
*** jmlowe has quit IRC22:09
*** jmlowe has joined #openstack-cinder22:13
*** jmlowe has quit IRC22:17
*** jvisser has quit IRC22:25
*** enriquetaso has joined #openstack-cinder22:30
*** tkajinam has joined #openstack-cinder22:57
*** e0ne has joined #openstack-cinder23:00
*** Lucas_Gray has quit IRC23:08
*** Lucas_Gray has joined #openstack-cinder23:09
*** rchurch has quit IRC23:13
*** rchurch has joined #openstack-cinder23:14
*** Liang__ has joined #openstack-cinder23:20
openstackgerritBrian Rosmaita proposed openstack/cinder master: Update reviewing doc with py3 usage guidelines  https://review.opendev.org/70370923:34
*** tosky has quit IRC23:47
*** e0ne has quit IRC23:52
*** dviroel has quit IRC23:57

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