Thursday, 2019-08-01

*** lseki has quit IRC00:09
*** brinzhang has joined #openstack-cinder00:17
*** brinzhang_ has quit IRC00:20
*** lixiaoy1 has joined #openstack-cinder00:25
*** enriquetaso has quit IRC00:27
*** tejdeep has quit IRC00:38
*** bhagyashris has joined #openstack-cinder00:51
*** tejdeep has joined #openstack-cinder01:07
*** boxiang has joined #openstack-cinder01:38
*** boxiang has quit IRC01:39
*** boxiang has joined #openstack-cinder01:39
*** boxiang has quit IRC01:42
brinzhangThis API "Show API v3 details", is this need to auth for user?01:47
brinzhanganyone can provide some suggestion?01:47
brinzhangI found it has a "403 - Forbidden", -- Policy does not allow current user to do this operation.01:48
brinzhangWhat does this *current user*  mean?01:49
*** baojg has joined #openstack-cinder02:03
*** spatel has joined #openstack-cinder02:07
*** spatel has quit IRC02:09
*** ruffian_sheep has joined #openstack-cinder02:12
ruffian_sheepwhoami-rajat:Hi do u know how to use it?
openstackLaunchpad bug 1837920 in Cinder "MacroSAN driver has numerous issues that need to be addressed" [Undecided,New]02:21
ruffian_sheepwhoami-rajat:I see someone below it seems to be adding a comment. Am I just adding a comment?02:21
ruffian_sheepwhoami-rajat:As long as the response has been resolved, or do I need to provide something extra?02:33
*** tejdeep has quit IRC02:44
openstackgerritpengyuesheng proposed openstack/cinderlib master: Bump the openstackdocstheme extension to 1.20
*** ruffian_sheep has quit IRC02:51
openstackgerritrenminmin proposed openstack/cinder master: Correct new_type type in VolumeManager retype method
*** Kuirong has joined #openstack-cinder03:25
*** gkadam has joined #openstack-cinder03:29
*** gkadam has quit IRC03:34
*** gkadam has joined #openstack-cinder03:35
*** whoami-rajat has joined #openstack-cinder03:41
*** udesale has joined #openstack-cinder03:46
*** psachin has joined #openstack-cinder03:55
*** Luzi has joined #openstack-cinder04:25
*** n-saito has joined #openstack-cinder04:45
*** bhagyashris has quit IRC04:52
*** n-saito has quit IRC05:09
*** ociuhandu has joined #openstack-cinder05:22
*** dpawlik has joined #openstack-cinder05:26
*** dansmith has quit IRC05:26
*** ociuhandu has quit IRC05:27
*** dansmith has joined #openstack-cinder05:28
*** raghavendrat has joined #openstack-cinder05:29
raghavendrathi whoami-rajat: are you online?05:30
whoami-rajatraghavendrat: hey05:38
raghavendratthis is regarding; got +2 from Walter Boring. whenever you get time, can please provide your feedback.05:39
openstackgerritfengjiankui proposed openstack/cinder master: query information of volume according to volume id or user id
whoami-rajatraghavendrat: Done05:58
raghavendratthanks a lot whoami-rajat: for quick response06:00
whoami-rajatraghavendrat: np06:00
*** bhagyashris_ has joined #openstack-cinder06:03
*** takamatsu has joined #openstack-cinder06:13
*** sahid has joined #openstack-cinder06:14
*** henriqueof has joined #openstack-cinder06:17
*** Kuirong has quit IRC06:31
*** e0ne has joined #openstack-cinder06:57
*** e0ne has quit IRC06:59
*** pcaruana has quit IRC07:12
*** tesseract has joined #openstack-cinder07:20
*** ociuhandu has joined #openstack-cinder07:39
*** boxiang has joined #openstack-cinder07:39
*** boxiang has quit IRC07:39
*** boxiang has joined #openstack-cinder07:40
openstackgerritMerged openstack/cinder master: 3PAR: Provide new option to specify NSP for single path attachments
*** pcaruana has joined #openstack-cinder07:42
*** jsquare has quit IRC07:44
*** lemko has joined #openstack-cinder07:48
*** jsquare has joined #openstack-cinder07:49
*** e0ne has joined #openstack-cinder07:53
*** tosky has joined #openstack-cinder07:54
*** ociuhandu has quit IRC07:56
*** boxiang has quit IRC07:59
*** boxiang has joined #openstack-cinder07:59
*** lpetrut has joined #openstack-cinder08:03
*** lpetrut has quit IRC08:04
*** lpetrut has joined #openstack-cinder08:04
*** boxiang has quit IRC08:17
*** tkajinam has quit IRC08:27
*** lpetrut has quit IRC08:29
*** raghavendrat has quit IRC08:36
brinzhangwhoami-rajat: please look at this patch
brinzhangwhoami-rajat: there is an issue need to talk about the list versions API auth.08:37
*** minmin has joined #openstack-cinder08:45
whoami-rajatbrinzhang: i'm not sure why are we doing this. AFAIK show version is just a subset of list versions and they show the same response for the particular version.08:50
brinzhangwhoami-rajat: Yeah, but the List All Api Version is not required auth policy check,,list-all-api-versions-detail#list-all-api-versions08:52
brinzhangso is this allow anyone(user) to request?08:53
*** lixiaoy1 has quit IRC08:53
whoami-rajatbrinzhang: it does require authentication08:56
*** psachin has quit IRC09:00
brinzhangwhoami-rajat: from the keystone authentication? That's administrator or member identify?09:00
whoami-rajatbrinzhang: it requires authentication from keystone.09:10
whoami-rajatbrinzhang: i don't think it depends on the type of user (admin or non-admin)09:11
brinzhangso, if this is deal with the temptest, just for cinder, not care the authentication?09:12
toskygeguileo: as a initial review, what do you think about cinder-tempest-plugin-lvm-lio-barbican from (and its long chain of dependencies)?09:13
brinzhangwhoami-rajat: can you reply on that patch? I think that deal with is ok, not need the auth. thanks09:13
geguileotosky: I'm looking at and wondering, do we want to joing all subunits into a single file?  Wouldn't it be better to have them separated?09:17
toskygeguileo: you said you just needed all the results :) but joking aside, that's what I proposed originally, let me grab the logs09:18
whoami-rajatbrinzhang: if you check the endpoint list, /v2 and /v3 volume APIs require project_id which incase requires authentication from keystone. also you can try out the curl command without the X-auth-token then too it will return 401 unauthorized,09:18
whoami-rajatbrinzhang: i think i agree with felipe there, request should be used instead of raw_request, will leave my reply on the patch09:19
geguileotosky: I've used the subunits in cinder uts a lot of times to reproduce flaky tests locally09:19
brinzhangwhoami-rajat: thanks, I will have a look.09:19
geguileotosky: I don't see cinderlib functional tests in logs from
toskygeguileo: -> cinderlib.tests.functional.test_basic.BackendFunctBasic09:31
toskygeguileo: and regarding my discussion on combine vs multiple files:
*** e0ne_ has joined #openstack-cinder09:36
*** e0ne has quit IRC09:37
geguileotosky: oooooooh, I'm dumb... I'm complaining about devstack-plugin-ceph-tempest not having the cinderlib tests, when we actually run them in a different job (cinder-plugin-ceph-tempest) defined in Cinder09:38
geguileotosky: I guess I'll have to figure out how to split cinderlib and cinder tests with subunit-filter  XD09:41
toskygeguileo: so you would prefer to have two different files? That's fine, I may propose another patch to Zuul people, but it depends also on their feedback09:41
toskyI'd need that feature anyway09:41
geguileotosky: no, don't worry, single file is fine09:42
toskygeguileo: as you can see, in that job I managed to use the 'functional' tox environment, without additional configuration tasks09:43
geguileotosky: I think the role is looking good!!09:50
geguileotosky: will you be also changing the cinder-plugin-ceph-tempest to replace devstack-plugin-ceph-tempest and using the role as well?09:50
toskygeguileo: oh, that depends on the general feedback; my initial goal is to replace the legacy jobs09:51
geguileotosky: ok09:52
toskybut I can follow-up, sure09:52
geguileotosky: because I just noticed that I did it wrong... I placed the ceph job in Cinder's zuul instead on the plugin, and I even called it "plugin"09:52
* geguileo facepalms09:53
toskygeguileo: oh, that's easy, the job can simply renamed cinder-ceph-tempest or so09:54
toskybut that job already uses playbooks/tempest-and-cinderlib-run.yaml, so the collection of the logs should work09:55
geguileotosky: but maybe it should be in the cinder tempest plugin repo?09:55
geguileotosky: I don't know where things should go  XD09:55
toskygeguileo: I think it's fine where it is, because it extends devstack-plugin-ceph-tempest and it does not run tests from cinder-tempest-plugin09:56
*** ociuhandu has joined #openstack-cinder09:57
geguileotosky: ok09:57
toskyI'm a bit confused about legacy-tempest-dsvm-lvm-multibackend, because it is used also by other repositories (devstack and tempest); is it more a cinder job, or should it be defined inside tempest, as integration job? I guess I will ask around09:57
*** sapd1_x has joined #openstack-cinder09:59
*** ociuhandu has quit IRC10:01
*** lemko has quit IRC10:11
*** bhagyashris_ has quit IRC10:16
*** tejdeep has joined #openstack-cinder10:37
*** tejdeep has quit IRC10:38
*** dpawlik has quit IRC10:58
*** rcernin has quit IRC10:58
*** dpawlik has joined #openstack-cinder11:06
*** ociuhandu has joined #openstack-cinder11:20
*** ociuhandu has quit IRC11:21
*** dpawlik has quit IRC11:27
*** Conqueror has quit IRC11:27
*** usr2033 has quit IRC11:28
*** e0ne has joined #openstack-cinder11:29
*** e0ne_ has quit IRC11:30
*** udesale has quit IRC11:49
*** udesale has joined #openstack-cinder11:50
*** davidsha has joined #openstack-cinder11:54
*** dpawlik has joined #openstack-cinder11:57
*** ociuhandu has joined #openstack-cinder12:01
*** dpawlik has quit IRC12:01
*** jcosmao has joined #openstack-cinder12:04
*** ociuhandu has quit IRC12:05
*** gkadam has quit IRC12:10
*** jrubenst has joined #openstack-cinder12:10
*** ociuhandu has joined #openstack-cinder12:15
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: WIP Update cinder-tempest-plugin-lvm-lio (revamp)
openstackgerritLuigi Toscano proposed openstack/cinder master: WIP Port several legacy tempest jobs to Zuul v3
openstackgerritLuigi Toscano proposed openstack/cinder master: WIP Port several legacy tempest jobs to Zuul v3
*** ociuhandu has quit IRC12:25
*** ociuhandu has joined #openstack-cinder12:27
*** dpawlik has joined #openstack-cinder12:29
*** ociuhandu has quit IRC12:31
*** jrubenst has quit IRC12:31
*** ociuhandu has joined #openstack-cinder12:31
*** sahid has quit IRC12:54
*** Luzi has quit IRC12:55
gansoHi folks! Does anybody know if there is a way to limit concurrent image conversions when creating volume from images? Like, do one conversion at a time when there are multiple requests13:00
*** sahid has joined #openstack-cinder13:01
*** tosky has quit IRC13:10
*** tosky has joined #openstack-cinder13:10
*** enriquetaso has joined #openstack-cinder13:13
*** mchlumsky has joined #openstack-cinder13:13
*** enriquetaso has quit IRC13:15
*** ociuhandu has quit IRC13:26
*** ociuhandu has joined #openstack-cinder13:27
*** mriedem has joined #openstack-cinder13:27
*** mchlumsky has quit IRC13:28
*** mchlumsky has joined #openstack-cinder13:30
*** ociuhandu has quit IRC13:31
*** lseki has joined #openstack-cinder13:33
*** carloss has joined #openstack-cinder13:36
*** enriquetaso has joined #openstack-cinder13:42
*** eharney has joined #openstack-cinder13:46
*** lbragstad has joined #openstack-cinder13:59
*** spatel has joined #openstack-cinder14:06
*** ociuhandu has joined #openstack-cinder14:07
*** lemko has joined #openstack-cinder14:08
*** ociuhandu has quit IRC14:16
*** ociuhandu has joined #openstack-cinder14:19
*** ociuhandu has quit IRC14:23
*** dpawlik has quit IRC14:43
*** thgcorrea has joined #openstack-cinder15:00
*** ociuhandu has joined #openstack-cinder15:10
toskygeguileo: re, I tried adding cinderlib functional tests to the new multibackend job but they are failing - are they supposed to work with that configuration?
*** throne82 has joined #openstack-cinder15:16
geguileotosky: theoretically it should support multi-backend15:16
geguileotosky: but there's something weird... it is trying to use privsep  :-O15:17
toskyuhm, configuration issue?15:18
geguileotosky: no, the problem is that it's using tgt as target instead of lio15:19
geguileotosky: and tgt uses privsep... and cinderlib doesn't support privsep15:19
geguileotosky: so I'll have to add support for privsep in cinderlib before that can work15:19
toskygeguileo: so no cinderlib tests for the multibackend job for now?15:20
toskyI created that new job using the same configuration of the legacy one15:20
geguileotosky: I know that eharney wants me to add it for his patch to move LIO to privsep15:20
geguileotosky: yeah, but the legacy uses LIO instead of tgt15:20
toskygeguileo: no, it does not - or at least it does not set any backend:
toskyor at least it does not look like it uses LIO15:22
toskylet me double check the logs15:22
geguileotosky: what I mean is that the other one uses LIO15:22
geguileotosky: this one doesn't set anything, and the default is tgt15:22
toskyso what do you think I should do now? Remove the cinderlib tests from this job, or given that I need to wait for 3 other patches to merge, wait also for the privsep support?15:23
toskygeguileo: ^15:30
geguileotosky: remove cinderlib tests, seems more realistic15:31
toskygeguileo: ack, thanks15:31
openstackgerritLuigi Toscano proposed openstack/cinder master: WIP Port several legacy tempest jobs to Zuul v3
*** e0ne has quit IRC15:37
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Mark HPE LeftHand driver as unsupported
jungleboyjwoojay:  Your log server is timing out.15:42
jungleboyjfor your CI.15:42
woojayyeah the disk died.  Gotta rebuild 'em... 8-(  Will have it in a few days for sure.15:45
woojayI saw your CI notes on etherpad, and I knew it was my turn next.  8-P15:46
*** ociuhandu has quit IRC15:47
*** whoami-rajat has quit IRC15:51
*** dpawlik has joined #openstack-cinder15:52
*** jcosmao has left #openstack-cinder15:53
jungleboyjwoojay:  Ok.  Thanks for being on top of it.15:55
jungleboyjYou are running py3.7 right?15:55
woojayyeah, sorry it's been out for a few days, but I will get it up soon.  Yes.  3.7.15:55
jungleboyjwoojay:  You knew if failed.  That is half the battle.15:56
jungleboyjwoojay:  Are you coming to the mid-cycle?15:56
*** gkadam has joined #openstack-cinder15:57
jungleboyjI know I have asked before, but I can always harass you again.15:57
woojayI'm a bit worried if I will be useful... 8-)15:57
*** dpawlik has quit IRC15:59
*** gkadam has quit IRC16:01
jungleboyj:-)  I always have that worry about myself.  ;-)16:02
woojayok, I will ask, for sure, this time...  8-)16:04
*** yaawang has quit IRC16:09
*** yaawang has joined #openstack-cinder16:13
jungleboyj:-)  Ok.  Sounds good.16:24
*** sahid has quit IRC16:25
*** davidsha has quit IRC16:28
*** lbragstad has quit IRC16:29
*** yaawang has quit IRC16:30
*** yaawang has joined #openstack-cinder16:32
*** lemko has quit IRC16:47
*** enriquetaso has quit IRC16:48
*** sapd1_x has quit IRC16:49
*** udesale has quit IRC17:14
*** whoami-rajat has joined #openstack-cinder17:15
jungleboyjTeam, I have completed my review of the 3rd Party CI systems.  Between non-compliance of py3 and systems that just aren't running anymore, I am looking at removing nearly half the drivers:
smcginnisThe Great Massacre of '1917:35
eharneythat.. can't be good17:35
jungleboyjIf you are a vendor owner please look at the list and respond if possible in the notes for your driver.  I will be sending out a formal note to the mailing list tomorrow.17:35
jungleboyjI just can't right now.17:35
jungleboyjYeah, 16 of the drivers are not running py3.17:36
jungleboyjAll the IBM drivers are out of compliance.17:36
jungleboyjAnd 7 drivers just aren't running CI anymore.17:36
openstackgerritEric Harney proposed openstack/cinder stable/rocky: Prevent double-attachment race in attachment_reserve
jungleboyjI guess my legacy coming out of my time as PTL will be killing the project.17:37
jungleboyjThough not my fault.  We have had these rules in place for many years.17:37
* jungleboyj is sick to his stomach.17:38
smcginnisWe ca17:38
smcginnisWe can't force vendors to support their products.17:38
smcginnisIt's not on us to be their driver maintainers.17:38
smcginnisIf they are getting paid by customers using OpenStack, then they need to use some of that money to make sure it works for those customers.17:39
smcginnisIf they aren't getting paid, well then there's probably no need to stress about having a driver that supports their storage then.17:39
*** sfernand has joined #openstack-cinder17:43
eharneyand both FC zone managers, huh17:44
*** ociuhandu has joined #openstack-cinder17:44
*** enriquetaso has joined #openstack-cinder17:44
*** senrique_ has joined #openstack-cinder17:47
*** ociuhandu has quit IRC17:48
*** enriquetaso has quit IRC17:49
jungleboyjI was going to ping some of them individually.17:59
*** senrique_ has quit IRC17:59
*** tosky has quit IRC18:01
*** eharney has quit IRC18:32
*** eharney has joined #openstack-cinder18:36
*** thgcorrea has quit IRC18:40
rosmaitajungleboyj: half the drivers does indeed seem like a massacre ... i would like to propose that in your ML note tomorrow, you outline the situation and say that the cinder team will review driver compliance at the midcycle and we'd like at least an action plan or notification if they intend to have CI running at all by then18:47
rosmaitaon our side, i guess y'all are sick of this, but i'm pretty new here, it would be good to review our requirements and see if there's anything we can change18:48
rosmaitai don't think we are being unreasonable, but half the drivers out of compliance is really bad18:48
rosmaita(though i don't have to tell you that)18:49
jungleboyjrosmaita:  :-)  Yeah, I think we will need to discuss things.  I already have added that to the mid-cycle discussion.18:49
jungleboyjAs far as reviewing the requirements.  We have done it so many times but it would be good to do it with a fresh perspective.18:50
rosmaitayeah, i am sure you older cinder folks are sick & tired of the discussion18:51
smcginnisWe did loosen the requirements that it's OK to not run on *every* patch.18:52
smcginnisBut they do still need to run at least once in the window of compliance so there is some sign of coverage.18:52
rosmaitayeah, i really don't think we are unreasonable ... but half the community not participating either indicates that they don't care, or that there's something else going on18:53
rosmaitacould be just a bookkeeping problem, that CI isn't being included as part of driver maintenance18:54
rosmaitai will draft a short google survey that we could send out, i got some good feedback occasionally doing that with glance ... maybe it would be good to ask for feedback about what the noncompliance blockage is before announcing that drivers are about to get dropped18:59
*** senrique_ has joined #openstack-cinder19:17
*** tesseract has quit IRC19:23
*** dpawlik has joined #openstack-cinder19:32
*** senrique_ has quit IRC20:03
*** eharney has quit IRC20:03
*** Luzi has joined #openstack-cinder20:06
*** tosky has joined #openstack-cinder20:17
jungleboyjrosmaita: the bulk of the drivers are ones that are not on Python 37. I don't know how we get the word out better that there is a need to make an update.20:24
jungleboyjI will maybe send a Twitter update as well.20:24
rosmaitayeah, it's not like the openstack-wide python 3 effort has been a secret20:32
*** e0ne has joined #openstack-cinder20:40
*** spatel has quit IRC20:41
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Mark HPE LeftHand driver as unsupported
jungleboyjpots: just had a good idea. It's about having a mailing list for communicating to the vendors.20:45
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Publish backup capabilities to a scheduler
jungleboyjnot sure that I might create a mailing list but I will take the time to go through the different cis and create a mailing list.  send them all a note about what's happening as well as one of the mailing list.20:46
jungleboyjYou shouldn't have to go to the effort of individually pinging all of them but I feel like it's the next step I should take.20:46
potsa mailing list might help the vendors help themselves more easily, too.  Nobody is using the #openstack-third-party-ci channel.20:59
*** senrique_ has joined #openstack-cinder21:18
woojayjungleboyj: See you in a few weeks...  8-)21:33
jungleboyjwoojay: nice! It'll be nice to have a vendor that help give input.21:34
*** Luzi has quit IRC21:46
openstackgerritMerged openstack/cinder master: NetApp ONTAP: Fix JSON serialization error on EMS logs
openstackgerritMerged openstack/cinder master: Fix DetachedInstanceError for VolumeAttachment
*** mriedem has quit IRC21:53
*** tosky has quit IRC22:07
*** henriqueof has quit IRC22:07
*** henriqueof has joined #openstack-cinder22:07
*** senrique_ has quit IRC22:10
*** jbernard_ has joined #openstack-cinder22:29
*** patrickeast_ has joined #openstack-cinder22:29
*** mnasiadka_ has joined #openstack-cinder22:29
*** patrickeast has quit IRC22:37
*** jbernard has quit IRC22:37
*** mnasiadka has quit IRC22:37
*** patrickeast_ is now known as patrickeast22:37
*** mnasiadka_ is now known as mnasiadka22:37
*** e0ne_ has joined #openstack-cinder22:45
*** e0ne has quit IRC22:46
*** e0ne has joined #openstack-cinder22:47
*** e0ne_ has quit IRC22:50
*** tkajinam has joined #openstack-cinder22:51
*** e0ne has quit IRC22:52
*** whoami-rajat has quit IRC22:55
*** ianychoi has quit IRC23:12
*** lixiaoy1 has joined #openstack-cinder23:14
*** rcernin has joined #openstack-cinder23:21
*** sfernand has quit IRC23:51
*** trident has quit IRC23:54
*** trident has joined #openstack-cinder23:59

Generated by 2.15.3 by Marius Gedminas - find it at!