Thursday, 2020-02-27

*** yamamoto has joined #openstack-meeting-400:05
*** yamamoto has quit IRC00:10
*** admcleod has joined #openstack-meeting-400:13
*** yangziguan has quit IRC00:56
*** yamamoto has joined #openstack-meeting-401:56
*** yamamoto has quit IRC02:49
*** diablo_rojo has quit IRC02:57
*** ktibi has joined #openstack-meeting-403:05
*** ktibi has quit IRC03:09
*** lvbin01 has joined #openstack-meeting-403:42
*** yamamoto has joined #openstack-meeting-403:55
*** yamamoto has quit IRC04:00
*** yamamoto has joined #openstack-meeting-404:13
*** abhishekk|away is now known as abhishekk04:49
*** evrardjp has quit IRC05:34
*** evrardjp has joined #openstack-meeting-405:35
*** lvbin01 has quit IRC05:38
*** lvbin01 has joined #openstack-meeting-405:38
*** links has joined #openstack-meeting-405:53
*** kozhukalov has joined #openstack-meeting-406:07
*** gcheresh has joined #openstack-meeting-406:15
*** e0ne has joined #openstack-meeting-407:16
*** e0ne has quit IRC07:23
*** e0ne has joined #openstack-meeting-407:24
*** ralonsoh has joined #openstack-meeting-407:28
*** e0ne has quit IRC07:29
*** e0ne has joined #openstack-meeting-407:48
*** e0ne has quit IRC07:52
*** slaweq has joined #openstack-meeting-407:53
*** e0ne has joined #openstack-meeting-408:06
*** e0ne has quit IRC08:08
*** anastzhyr has quit IRC08:28
*** ktibi has joined #openstack-meeting-409:15
*** k_mouza has joined #openstack-meeting-409:21
*** yamamoto has quit IRC09:35
*** lvbin01 has quit IRC09:36
*** lvbin01 has joined #openstack-meeting-409:36
*** dmellado has quit IRC09:59
*** roman_g has joined #openstack-meeting-411:00
*** kozhukalov has quit IRC11:17
*** kozhukalov has joined #openstack-meeting-411:21
*** ktibi has quit IRC11:22
*** kozhukalov has quit IRC11:35
*** kozhukalov has joined #openstack-meeting-411:35
*** yamamoto has joined #openstack-meeting-411:42
*** kozhukalov has quit IRC11:43
*** yamamoto has quit IRC11:46
*** yamamoto has joined #openstack-meeting-412:25
*** ktibi has joined #openstack-meeting-412:32
*** kozhukalov has joined #openstack-meeting-412:39
*** eharney has quit IRC12:45
*** enriquetaso has joined #openstack-meeting-412:53
*** ktibi has quit IRC13:00
*** ktibi has joined #openstack-meeting-413:03
*** dmellado has joined #openstack-meeting-413:05
*** enriquetaso has quit IRC13:27
*** enriquetaso has joined #openstack-meeting-413:28
*** senrique_ has joined #openstack-meeting-413:30
*** enriquetaso has quit IRC13:34
*** yamamoto has quit IRC13:35
*** e0ne has joined #openstack-meeting-413:35
*** lpetrut has joined #openstack-meeting-413:36
*** e0ne has quit IRC13:46
*** yamamoto has joined #openstack-meeting-413:54
*** eharney has joined #openstack-meeting-413:56
*** e0ne has joined #openstack-meeting-413:57
*** rosmaita has joined #openstack-meeting-413:58
*** nao-shark has joined #openstack-meeting-414:00
abhishekk#startmeeting glance14:00
openstackMeeting started Thu Feb 27 14:00:30 2020 UTC and is due to finish in 60 minutes.  The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
abhishekk#topic roll call14:00
*** openstack changes topic to " (Meeting topic: glance)"14:00
openstackThe meeting name has been set to 'glance'14:00
*** openstack changes topic to "roll call (Meeting topic: glance)"14:00
abhishekk#link https://etherpad.openstack.org/p/glance-team-meeting-agenda14:00
abhishekko/14:00
*** jbernard has joined #openstack-meeting-414:01
rosmaitao/14:01
nao-sharko/14:02
rosmaitaabhishekk: i haven't had time to look at the glanceclient patches this morning, did my comments make sense?14:02
abhishekklets wait 2-3 minutes for jokke_14:02
abhishekkrosmaita, Yes, I have added inline answer to those14:02
rosmaitaok, will take a look14:03
abhishekkIf you still insist to remove those check then I will do it in a separate patch14:03
abhishekklets start14:04
abhishekk#topic release/periodic jobs update14:04
*** openstack changes topic to "release/periodic jobs update (Meeting topic: glance)"14:04
abhishekkWe need to release glanceclient ASAP14:04
abhishekktwo important patches needs to be merged14:05
jokke_o/14:05
abhishekk#link https://review.opendev.org/69965614:05
whoami-rajatHi14:05
abhishekkjokke_, o/14:05
abhishekk#link https://review.opendev.org/70908614:05
abhishekkwhoami-rajat, o/14:05
abhishekkKindly review these patches and get it going14:06
abhishekkPeriodic jobs couple of timeouts due to parser error14:06
abhishekkwhich is our next topic14:06
abhishekk#topic the stestr situation14:06
*** openstack changes topic to "the stestr situation (Meeting topic: glance)"14:07
abhishekkrosmaita, floor is yours14:07
rosmaitayeah, i was looking into this real quickly14:07
rosmaitawas wondering if we could force the issue by going back to testr14:07
rosmaitabut, it looks like that is no longer maintained14:07
rosmaitaplus, the real problem seems to be in subunit14:07
*** e0ne_ has joined #openstack-meeting-414:08
jokke_yeah iirc that's why we moved to stestr14:08
rosmaitaand it looks like stephinfin has a patch up for that14:08
abhishekkrosmaita, https://github.com/testing-cabal/subunit/pull/40 this seems to be solved the parser error14:08
rosmaitalooks like the subunit maintainer is not too keen on that patch14:08
*** e0ne has quit IRC14:08
abhishekkI have applied that in my local environment, and seems to be working14:08
rosmaitaso i wonder whether it would help if abhishekk and efried for nova added comments to that pull request14:08
jokke_is it still the same issue that we're too heavy with logging and overwhelm subunit?14:08
abhishekkjokke_, yes14:09
rosmaitai don't know14:09
abhishekkstephinfin has reproduce for this, https://review.opendev.org/#/c/700522/1/nova/tests/unit/test_utils.py14:09
rosmaitathe consensus seemed to be that there's some kind of mishandling of a file object somewhere14:09
* jokke_ can see os_subunit fork coming soon to your neighbourhood14:09
rosmaitaanyway, i put this on the agenda wondering if there was an alternative testrunner we could use14:10
abhishekkI have applied this patch in local environment without subunit pull changes it is failing with parser error14:10
rosmaitabut it looks like not if we want to have highly parallel tests running14:10
rosmaitayeah, so the situation is that the patch works, but the subunit maintainer thinks it is a hack14:11
abhishekkand with his pull changes it is working14:11
jokke_rosmaita: thanks for looking into this and I truly hope we as community can come together and fix the issues rather than a) forking yet another project or b) start fiddling around and patching yet another test runner. Also like you said the options list is pretty short14:11
abhishekkI can comment on that, but will it help?14:12
rosmaitayeah, i think maybe abhishekk report on the pull request, and maybe we can circle up with stephen for some strategy14:12
jokke_does he has better solution to offer?14:12
rosmaitayeah, fix our tests so that it doesn't happen14:12
rosmaita:P14:12
jokke_oh lol, so no14:13
jokke_yeah lets keep the pressure on14:13
abhishekkrosmaita, cyril is going to drop a mail to him14:13
rosmaitaok, sounds good ... that's all from me14:13
jokke_one can always start flame war on reddit :P14:14
jokke_That helps every time14:14
abhishekk:D14:14
abhishekkwe had lengthy discussion on this today, lets see how this will progress14:15
abhishekkmoving ahead14:15
abhishekk#topic Delete image from single store14:15
*** openstack changes topic to "Delete image from single store (Meeting topic: glance)"14:15
jokke_++14:15
abhishekkjokke_, do you mind if I take this ahead?14:15
abhishekkIMO this will be very helpful feature and don't want to waste our efforts14:16
jokke_I don't care, haven't had time for it and my request for adding hours to a day has not been approved yet :P14:16
rosmaitaabhishek has been disconnected14:18
rosmaitaok, so jokke_ you are ok with abhishekk taking over the patch?14:18
abhishekksorry, I have been disconnected from the network14:18
abhishekkdid I miss something?14:18
*** yamamoto has quit IRC14:18
rosmaitanope14:18
jokke_I don't care, haven't had time for it and my request for adding hours to a day has not been approved yet :P14:19
jokke_^^ that possibly14:19
abhishekkcool14:19
abhishekkok, moving ahead14:19
abhishekk#topic glance cinder store nfs mount issue14:20
*** openstack changes topic to "glance cinder store nfs mount issue (Meeting topic: glance)"14:20
abhishekkwhoami-rajat, stage is yours14:20
whoami-rajatHi14:20
whoami-rajatSo i wanted to discuss regarding my patch that mounts the image-volume in the glance directory rather than the cinder one14:20
*** jvisser has joined #openstack-meeting-414:20
whoami-rajati've faced error regarding that in my environment (Permission Denied) and i think it can cause much bigger issues too14:21
whoami-rajatNova has the same approach of mounting nfs volumes to their own custom path14:21
whoami-rajatand similarly i would like to propose for glance14:21
rosmaitathat seems to make sense14:22
jokke_whoami-rajat: I hope my stance did not come too hars. Happy that you're looking into the problems there. This is just really something os_brick should handle rather than each consumer reinventing the wheel on their own, and that's why os_brick was pushed to us to use in that driver14:22
whoami-rajatjokke_, not at all harsh :)14:23
rosmaitawell, i think that glance_store is supposed to be providing the abstraction layer to glance14:23
eharneyi'd like to understand the objection better... i think there's some misunderstanding about what os-brick does and doesn't do14:23
rosmaitaso the place to do the os-brick mediation would seem to be in the glance_store driver14:23
rosmaitait has suddenly gone quiet in here? or am i disconnected?14:25
jokke_nope, just quiet14:25
abhishekknope14:25
rosmaitaok, thanks14:25
rosmaitaeharney: can you explain why nova takes the approach it does with os-brick?14:26
eharneyos-brick provides an interface to connect to things, but the consumer (glance_store here, or nova) still has to have code that handles block devices (iscsi/fc), nfs volumes, rbd volumes, etc14:26
jokke_So iirc the whole idea of cinder+os-brick was that we get block device from cinder and os-brick provides us access to it so that we do not need to take care of the special sauce of different back-ends in glance14:27
eharneyos-brick will not provide a block device when cinder is serving volumes over nfs14:27
jokke_and yes, we already have special sauce for rbd, but lets be honest they are just special in every way.14:27
eharneyi assume that the glance_store driver for cinder needs similar work for rbd, but that's a whole different project14:28
abhishekkI just have one question, if cinder has configured multiple nfs backends then how this new config option will help?14:28
eharneynfs exports are mounted to different directories under $cinder_nfs_mount_point_base14:29
whoami-rajatjokke_, the code we're trying to implement isn't for each and every driver but the types of driver i.e. iscsi, fc, nfs, rbd so we just need to have code handling these scenarios rather than all the storage drivers14:29
*** senrique__ has joined #openstack-meeting-414:30
whoami-rajateharney++14:31
whoami-rajatthe mount point is a directory with the volume id (IIRC) inside the mount_point_base14:31
eharneyi don't think i answered the why nova does what it does question... some of that predates the existence of cinder14:31
whoami-rajat /var/lib/glance/<vol-id>/14:32
eharneybut the "how" of what it does, is that it determines where it wants to mount nfs exports to consume cinder volumes, and chooses a path to mount them to.  this particular item is one of things missing from glance_store currently14:32
jokke_whoami-rajat: well, that's the thing ... when the cinder driver was introduced the selling point was that we don't need any of that as cinder abstracts the backend from us, now you're saying that we need special sauce for every type of connectivity as os-brick can't handle that and I guess next step is that we need spaghetti for HP, NetAPP, DellEMC, PureStore etc. cause they will need their14:32
jokke_own special treatment14:32
eharneyno, it's not a per-driver issue14:32
eharneyit's a per-protocol issue14:33
abhishekkcurrent glance_store cinder driver has many loop holes14:33
eharneyso, iscsi, fc (which are mostly the same in brick), nfs, ceph, and... other rare ones if someone wants to14:33
*** senrique_ has quit IRC14:33
jokke_so can we give os-brick "base mount path" and it does the right thing there or do we need to bring special setting for each protocol?14:33
eharneybase_mount_path is only relevant for FS protocols (and at this point only NFS is really of interest there, i think)14:34
jokke_cause _that_ is the problem I have with this proposal. If os-brick needs a folder it has permission to do it's thing I'm fine with that. If we need to specify that for each and every thing separately I'm having problem with it14:35
abhishekk+!14:35
abhishekk+114:35
rosmaitawell, it's either fix it inside the glance_store cinder driver, or have a glance_store cinder_nfs, cinder_iscis etc14:35
jokke_eharney: so can we call it "cinder_base_mount_path" and never have this discussion again when there is next special sayce that needs some node local FS trickstery, been that loop mounts, nfs, iscsi fs with volume images etc.14:36
jokke_?14:36
eharneyi guess it can be called that since it's in the glance_store cinder driver, but both nova and cinder have the option named "nfs_mount_point_base"14:37
jokke_cause I know that day will come if we call that nfs specific that next guy doing their thing doesn't want to use it as it's nfs14:37
rosmaitai think it's better to be consistent with cinder and nova14:38
rosmaitasince people from those teams may wind up working on it14:38
eharneythe problem with making it generic is that later when you add support for, i dunno, yzfs, then you need a different option named something else for yzfs mounts14:38
whoami-rajatjokke_, i think the glance cinder store has a generic code that requires some restructuring for some specific cases.14:38
eharneyso i'm not sure it's a good idea to avoid putting "nfs" in the name14:38
abhishekkrosmaita, as per our glance store standard we define each option with store_prefix_option_name14:39
whoami-rajateharney, nova has different mount_point_base names for different drivers like quobyte14:39
eharneywhoami-rajat: right14:39
jokke_eharney: that's exactly what I wan to avoid14:39
rosmaitaok, so i guess if we're arguing over the name of the config opt, then the issue is basically settled?14:39
eharneyi'm not sure it's a good idea to try to avoid that, i think that's a requirement14:40
eharneybut, what rosmaita said14:40
jokke_eharney: so if brick is given mount path it can then do /put/mount/craps/here/[nfs,yfs,psnpfs] and the driver side should never need to know about it14:40
eharneytrue, we could just make it "cinder_mount_point_base" and all nfs mounts go under $cinder_mount_point_base/nfs/<asdf>14:41
jokke_it's just gives brick a path that it has permissions to do what it needs to do14:41
eharneyand yzfs goes under $cinder_mount_point_base/yzfs etc14:41
eharneynot a bad idea14:41
rosmaitawhoami-rajat: that sound OK to you?14:42
jokke_cause there is no reason the consumer should be touching those or know about them, I'm assuming brick gives the consumer just fd anyways14:42
abhishekksounds reasonable to me14:42
whoami-rajatrosmaita, i'm not sure there are 2 discussions going on14:42
abhishekk:D14:43
whoami-rajatrosmaita, if you're talking about making cinder_mount_point_base the generic and all FS driver files go into it then i'm ok with it14:43
jokke_and that would avoid us having extra 300 lines of config options and comments for them in our already mile long config files just because14:43
rosmaitawhoami-rajat: ok, and what issue is still open?14:43
abhishekk+114:43
*** gcheresh has quit IRC14:43
whoami-rajatrosmaita, i think jokke_ is still suggesting it inside os-brick ?14:44
rosmaitano, i thought it was going to be done in the cinder driver14:44
whoami-rajatrosmaita, then i misunderstood the words and i'm clearly ok with everything14:44
abhishekk15 minutes remaining14:45
jokke_the best case scenario is that it's in os-brick, we can pass it when we initiate the connector (regardless if it's needed or not) and brick does the right thing14:45
jokke_it just knows "Hey I have a path here if I need one I have permissions to"14:46
jokke_that would be nice way to consume it :P14:46
whoami-rajatcan we also have the final action items so i don't miss out anything?14:47
whoami-rajati know one for sure to rename the mount base path config option14:48
abhishekkcinder_mount_point_base we can have this option in glance_store cinder driver14:48
jokke_whoami-rajat: that's a good start, I think we need to continue this discussion offline. I need to run in a minute14:49
abhishekkand all nfs mount goes under cinder_mount_point_base/nfs and others under cinder_mount_point_base/14:49
whoami-rajatjokke_, ack, thanks for the discussion14:49
jokke_can't be late twice a day and I joined couple of min late so need to take it back and run couple early :P14:49
abhishekk:D14:49
abhishekkmoving to open discussion14:49
abhishekk#topic Open discussion14:50
*** openstack changes topic to "Open discussion (Meeting topic: glance)"14:50
rosmaitai have something14:50
rosmaita#link http://lists.openstack.org/pipermail/openstack-discuss/2020-February/012883.html14:50
rosmaitaquestion about show_multiple_locations deprecation status14:50
rosmaitai replied, but the author suggested revising the train release notes to say the workaround is still needed14:51
rosmaitaabhishekk: that's up to you14:51
abhishekkI need to do one stable/train release coming week14:51
jokke_I haven't read that but I think we need to lift the deprecation and hopes of getting rid of it due to how popular ceph became and everything is heavily relying on hetting those location uris14:51
jokke_getting14:52
abhishekkrosmaita, If you provide me the wording I will add one for sure14:52
abhishekkI am pretty bad in writing release notes14:52
rosmaitaabhishekk: sure, i think we can just copy the one from stein14:52
rosmaitai will look and see14:52
abhishekkrosmaita, ack,14:53
jokke_Ping me if anything is needed. Now I really need to run (today is last day when my passport is valid and I have appointment to order a new one so can't be late from that) :)14:53
jokke_will be back online in a while14:53
abhishekkjokke_, o/~14:53
rosmaitajokke_: ok, don't get deported14:53
rosmaitaabhishekk: i'll get up a patch for you in a bit14:53
abhishekkwe have new member waiting I guss14:53
abhishekkrosmaita, thanks14:53
abhishekknao-shark, around?14:54
nao-sharkThanks.14:54
nao-sharkI have two questions .14:54
nao-sharkhttps://review.opendev.org/#/c/687390/14:54
nao-sharkThis is my spec for revive the S3 driver.14:54
nao-sharkand this is my patch of S3 driver. https://review.opendev.org/#/c/695844/14:55
nao-sharkI want to edit glance-api.conf to show how to configure the S3 driver.14:55
nao-sharkBut it looks like generated by oslo-config-generator.14:55
nao-sharkSo my first question is Can I make a separate patch for glance-api.conf before my S3 driver patch is merged?14:55
nao-sharkMy concern is that modifications to the S3 driver may directly affect the content of glance-api.conf.14:55
abhishekknao-shark, I will have a look at those and will get back to you14:56
rosmaitanao-shark: what you can do is regenerate the config file and submit it as part of your patch14:56
rosmaitai think we still keep a full config sample? or did we stop doing that?14:56
abhishekknao-shark, could you ping me tomorrow in the morning time?14:56
abhishekkrosmaita, I guess we have stopped doing that14:56
rosmaitaoh, ok14:57
rosmaitanao-shark: best work it out with abhishekk tomorrow14:57
nao-sharkabhishekk OK. thanks14:57
abhishekkcool14:57
abhishekkanything else14:57
nao-sharkMaybe the next question should probably be tomorrow14:58
abhishekkrosmaita, I have replied to your question on client patch14:58
*** danielarthurt has joined #openstack-meeting-414:58
*** andrebeltrami has joined #openstack-meeting-414:58
abhishekknao-shark, yes14:58
abhishekkor you can drop me a mail as well14:58
abhishekkthank you all14:59
nao-sharkabhishekk  OK Thanks for your support !14:59
rosmaitabye14:59
abhishekknao-shark, no worries14:59
abhishekk#endmeeting14:59
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"14:59
openstackMeeting ended Thu Feb 27 14:59:55 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:59
openstackMinutes:        http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-02-27-14.00.html14:59
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-02-27-14.00.txt14:59
openstackLog:            http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-02-27-14.00.log.html15:00
*** nao-shark has quit IRC15:00
*** rosmaita has left #openstack-meeting-415:02
*** abhishekk is now known as abhishekk|away15:03
*** e0ne_ has quit IRC15:04
*** ktibi has quit IRC15:21
*** e0ne has joined #openstack-meeting-415:24
*** ktibi has joined #openstack-meeting-415:46
*** diablo_rojo has joined #openstack-meeting-416:00
*** e0ne has quit IRC16:12
*** links has quit IRC16:14
*** lvbin01 has quit IRC16:20
*** lvbin02 has joined #openstack-meeting-416:21
*** lvbin02 is now known as lvbin0116:23
*** lpetrut has quit IRC16:40
*** dosaboy has quit IRC16:45
*** e0ne has joined #openstack-meeting-416:53
*** dosaboy has joined #openstack-meeting-417:01
*** jbernard has left #openstack-meeting-417:23
*** ktibi has quit IRC17:27
*** evrardjp has quit IRC17:35
*** LiangFang has quit IRC17:35
*** evrardjp has joined #openstack-meeting-417:35
*** Liang__ has joined #openstack-meeting-417:35
*** igordc has joined #openstack-meeting-417:39
*** abhishekk|away is now known as abhishekk17:39
*** k_mouza has quit IRC17:51
*** igordc has quit IRC18:01
*** igordc has joined #openstack-meeting-418:04
*** danielarthurt has quit IRC18:08
*** macz_ has joined #openstack-meeting-418:12
*** macz_ has quit IRC18:14
*** macz_ has joined #openstack-meeting-418:15
*** k_mouza has joined #openstack-meeting-418:21
*** e0ne has quit IRC18:21
*** k_mouza has quit IRC18:25
*** gcheresh has joined #openstack-meeting-418:37
*** e0ne has joined #openstack-meeting-418:57
*** andrebeltrami has quit IRC18:58
*** e0ne has quit IRC18:59
*** ralonsoh has quit IRC19:08
*** e0ne has joined #openstack-meeting-419:10
*** lvbin02 has joined #openstack-meeting-419:11
*** lvbin01 has quit IRC19:13
*** lvbin02 is now known as lvbin0119:13
*** gcheresh has quit IRC19:22
*** abhishekk is now known as abhishekk|away19:27
*** eharney has quit IRC19:43
*** e0ne has quit IRC20:07
*** kozhukalov has quit IRC20:27
*** kozhukalov has joined #openstack-meeting-420:28
*** gcheresh has joined #openstack-meeting-420:31
*** gcheresh has quit IRC20:55
*** jvisser has quit IRC20:56
*** gcheresh has joined #openstack-meeting-421:00
*** kozhukalov has quit IRC21:08
*** gcheresh has quit IRC21:09
*** slaweq has quit IRC21:15
*** ktibi has joined #openstack-meeting-421:27
*** jvisser has joined #openstack-meeting-421:29
*** ktibi has quit IRC21:32
*** senrique__ is now known as enriquetaso21:39
*** kozhukalov has joined #openstack-meeting-421:40
*** slaweq has joined #openstack-meeting-422:11
*** slaweq has quit IRC22:15
*** slaweq has joined #openstack-meeting-422:17
*** jvisser has quit IRC22:21
*** slaweq has quit IRC22:22
*** eharney has joined #openstack-meeting-422:29
*** slaweq has joined #openstack-meeting-423:11
*** slaweq has quit IRC23:16
*** kozhukalov has quit IRC23:35

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