Monday, 2018-07-23

*** btully has joined #openstack-glance00:50
*** btully has quit IRC00:54
*** edmondsw has joined #openstack-glance01:11
*** edmondsw has quit IRC01:15
*** edmondsw has joined #openstack-glance02:59
*** edmondsw has quit IRC03:04
*** pdeore has joined #openstack-glance03:37
*** dalgaaf has quit IRC03:54
*** udesale has joined #openstack-glance03:55
*** dalgaaf has joined #openstack-glance03:56
openstackgerritwangxiyuan proposed openstack/glance master: Register keystoneauth options in [keystone] group  https://review.openstack.org/58471204:15
*** btully has joined #openstack-glance04:27
*** btully has quit IRC04:31
*** edmondsw has joined #openstack-glance04:47
*** edmondsw has quit IRC04:52
*** ratailor has joined #openstack-glance05:21
*** abhishekk has joined #openstack-glance05:29
*** links has joined #openstack-glance05:38
*** udesale_ has joined #openstack-glance05:42
*** udesale has quit IRC05:45
*** kushalAgrawal has joined #openstack-glance05:48
kushalAgrawalhello folks, task API in glance is failing while using https url for import because of this https://github.com/openstack/glance/blob/master/glance/common/scripts/utils.py#L14105:50
kushalAgrawalwhile creating url connection, there is no way remote certificates can be used or ignored.05:52
kushalAgrawalis there any plan to enhance it ?05:52
abhishekkkushalAgrawal, as of now we are busy in completing the work for the milestone 3 release of glance05:55
abhishekkif you want to work for it, feel free to report a bug and submit a patch05:56
*** zhurong has joined #openstack-glance05:58
*** edmondsw has joined #openstack-glance06:35
*** edmondsw has quit IRC06:39
*** links has quit IRC06:41
*** udesale__ has joined #openstack-glance06:43
*** udesale_ has quit IRC06:45
*** dtantsur|afk is now known as dtantsur06:45
*** links has joined #openstack-glance07:00
*** alexchadin has joined #openstack-glance07:00
*** pcaruana has joined #openstack-glance07:02
*** Luzi has joined #openstack-glance07:05
*** evrardjp_ is now known as evrardjp07:09
*** pooja_jadhav has joined #openstack-glance07:10
*** tesseract has joined #openstack-glance07:21
*** zhurong has quit IRC07:24
*** tshefi has joined #openstack-glance07:34
*** links has quit IRC07:34
*** AlexeyAbashkin has joined #openstack-glance07:42
*** links has joined #openstack-glance07:47
*** zhurong has joined #openstack-glance07:49
*** btully has joined #openstack-glance08:05
*** btully has quit IRC08:09
*** dtantsur is now known as dtantsur|brb08:18
*** btully has joined #openstack-glance08:20
*** edmondsw has joined #openstack-glance08:23
*** edmondsw has quit IRC08:28
*** zhurong has quit IRC08:47
*** btully has quit IRC08:56
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: Add support for hide old images  https://review.openstack.org/57911709:00
*** alexchadin has quit IRC09:17
*** alexchadin has joined #openstack-glance09:23
*** jistr|off is now known as jistr09:29
*** links has quit IRC09:34
*** dtantsur|brb is now known as dtantsur09:38
*** links has joined #openstack-glance09:41
*** finucannot has quit IRC09:53
*** stephenfin has joined #openstack-glance09:55
*** rosmaita has joined #openstack-glance10:00
rosmaita#startmeeting glance_bs10:02
openstackMeeting started Mon Jul 23 10:02:01 2018 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.10:02
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.10:02
*** openstack changes topic to " (Meeting topic: glance_bs)"10:02
openstackThe meeting name has been set to 'glance_bs'10:02
rosmaita#link https://etherpad.openstack.org/p/glance-bug-squad-meeting10:02
rosmaitaunless I'm reading the list incorrectly, no new glanceclient bugs since February10:03
abhishekko/10:04
rosmaita\o10:04
abhishekkyes no new bug since feb10:04
rosmaitalooks like the newest patches are yours10:05
rosmaita:)10:06
rosmaita#link https://review.openstack.org/#/q/status:open+project:openstack/python-glanceclient10:06
abhishekkrosmaita, yes10:07
rosmaitai'm looking at wxy's patch https://review.openstack.org/#/c/583468/10:07
abhishekkit will print os_hash_algo and os_hash_value only if image is active (similar to checksum) otherwise it will show None10:08
rosmaitai think it makes sense if the multihash behaves like the checksum10:08
abhishekkrosmaita, yes10:09
*** edmondsw has joined #openstack-glance10:11
*** edmondsw has quit IRC10:16
rosmaitaabhishekk: looking at your multihash patch, your test_do_image_create_with_multihash is just checking that *if* the os_has_* fields are in the image response, they are printed by the client shell10:23
rosmaitasorry, of course it is ... it's a bit early for me here10:23
abhishekkrosmaita, :D10:23
rosmaitathat patch looks good, one nit, but it's not your fault, since the image schema change hasn't been done in glance yet10:24
abhishekkrosmaita, thank you, will fix it asap, right now working on hidden images comments10:26
rosmaitanp, am just looking at that one10:26
rosmaitai think erno is right, we should use os_hidden everywhere10:26
rosmaitai'm going to end the bug squad meeting and we can discuss the multihash, multi-backend, and hidden patches10:29
rosmaitasince they're not technically bugs10:29
rosmaitaactually, not bugs at all10:29
rosmaita#endmeeting10:29
*** openstack changes topic to "OpenStack Glance | priorities: http://lists.openstack.org/pipermail/openstack-dev/2018-March/128739.html | logs at http://eavesdrop.openstack.org/irclogs/%23openstack-glance/"10:29
openstackMeeting ended Mon Jul 23 10:29:20 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)10:29
openstackMinutes:        http://eavesdrop.openstack.org/meetings/glance_bs/2018/glance_bs.2018-07-23-10.02.html10:29
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/glance_bs/2018/glance_bs.2018-07-23-10.02.txt10:29
openstackLog:            http://eavesdrop.openstack.org/meetings/glance_bs/2018/glance_bs.2018-07-23-10.02.log.html10:29
*** alexchadin has quit IRC10:30
abhishekkrosmaita, yes, I am fixing that only now10:31
*** alexchadin has joined #openstack-glance10:36
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: Add support for multihash  https://review.openstack.org/58464210:39
*** alexchadin has quit IRC10:41
*** btully has joined #openstack-glance10:42
*** links has quit IRC10:44
*** alexchadin has joined #openstack-glance10:45
*** btully has quit IRC10:46
*** links has joined #openstack-glance10:48
abhishekkrosmaita, still around?10:58
rosmaitayes10:58
rosmaitalooking at your multibackend patch10:58
rosmaitafor glanceclient10:58
*** pdeore has quit IRC10:58
abhishekkso only column name should be hidden right, for everything else we should use os_hidden correct?10:59
abhishekkregarding your comment on the specs ^^10:59
rosmaitai was thinking column name, too10:59
rosmaitadoes that complicate things?11:00
abhishekkrosmaita, no11:00
jokke_I think only thing we really want to keep just 'hidden' is the cli command to set it11:00
rosmaitajokke_ i wonder even there11:01
jokke_as in --hidden True instead of --os-hidden True11:01
jokke_it would be just so much more intuitive and user friendly11:01
rosmaitaexcept that what would you use for the filtering command?11:01
abhishekkthat will be difficult to implement as if we pass --hidden from the cli then it goes hidden in request body and no as os_hidden11:02
jokke_abhishekk: nope, that's just software we need to take care of ;)11:02
abhishekkrigtht now i am using --os-hidden true|false for filtering and --os-hidden for create and update11:02
abhishekkjokke_, you are right but the code around it is complicated in client11:03
abhishekkI will have a look though11:03
jokke_but filtering wise can we easily distinquish the difference between "hidden":True and "hidden":"true"?11:03
rosmaitaisn't it --property hidden=true  ?11:04
rosmaitaso it think we should stick with os_hidden for the glance "core" property11:04
rosmaitas/it/i/11:05
abhishekkrosmaita, makes sense11:05
abhishekkso should I rename the table column to os_hidden too?11:05
rosmaitai really think so, keep everything consistent11:06
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: Add support for multihash  https://review.openstack.org/58464211:07
jokke_rosmaita: my point indeed ... we have the os- prefix in none of our client commands11:07
jokke_I don't think we should do it now either11:07
jokke_even if it means that we need to put bit of logic to deal with it in the client11:08
abhishekkok will push modified specs  + patches by tomorrow morning my time11:08
abhishekk(or late in the evening)11:08
rosmaitajokke_ ok, but it will introduce an asymmetry in API vs CLI, which so far, we've mostly avoided11:09
rosmaitai would prefer keeping 'hidden' in API also in that case11:09
abhishekkSo should I revert it back to original patch or add new one with os_hidden ;)11:10
rosmaita(i know, sorry abhishekk !!)11:11
abhishekkrosmaita, no issues11:11
jokke_rosmaita: so if we do that do we have mechanism to distinquish hidden as property and hidden as base property?11:11
rosmaitajokke_ pretty sure we do not11:12
abhishekkwe do not11:12
jokke_because in client that difference comes with the way we present it --property vs --hidden11:12
jokke_but in API we do not call it differently11:12
jokke_and overloading hidden is just so damn easy11:12
rosmaitabut no matter what we all it, we have the "masking" problem11:13
rosmaitas/all/call/11:13
*** links has quit IRC11:13
abhishekkindeed11:14
rosmaitai'm not going to put up a big fight about client vs API11:15
*** links has joined #openstack-glance11:15
rosmaitain the API, though, I think we need to be consistent about os_hidden everywhere11:15
jokke_rosmaita: agreed11:15
jokke_I do prefer that ... and I'm not planning to make big fight about it either ... for me it's just matter of user experience11:15
abhishekkso I will change the column name to os_hidden at api side everywhere in specs I will use os_hidden and only command line argument I will keep --hidden instead of --os-hidden11:16
abhishekkI need to tweak a client code to do this11:16
jokke_and I think worst case scenario would be us calling it plain hidden everywhere and encourage the collision bitween this and any possible "hidden" properties people have been using over the years we have been telling them that it would be the way to do it11:16
jokke_between11:17
abhishekkright now I am ready with column name hidden other os_hidden everywhere including command line params11:17
rosmaitawell, in large part this will come down to what the openstack cli people decide to do :)11:17
jokke_rosmaita: but that's not our fight to pick :P11:18
rosmaita:)11:18
jokke_they do what they feel to do and most likely that will be something between 20 and 120 characters11:18
abhishekkjokke_, rosmaita I am heading back to home, will come online after 5 hours from now11:18
jokke_with prompt if user really wants to do it :P11:18
rosmaitaabhishekk thanks!11:18
jokke_abhishekk: cool ... safe commute11:18
rosmaitahave a good travel home11:18
abhishekkthank you11:19
*** abhishekk has quit IRC11:19
rosmaitai am going to disappear, owe the dog a walk this morning11:19
jokke_fair enough11:19
jokke_I'll try to review some code unless something catches fire ;)11:20
*** ratailor has quit IRC11:20
*** udesale__ has quit IRC11:25
*** kushalAgrawal has quit IRC11:39
*** mvk_ has quit IRC11:44
*** rosmaita has quit IRC11:56
*** MattMan has quit IRC12:08
*** MattMan has joined #openstack-glance12:08
*** edmondsw has joined #openstack-glance12:13
*** aniketh has joined #openstack-glance12:38
*** jistr is now known as jistr|mtg12:41
*** mvk_ has joined #openstack-glance12:51
*** udesale has joined #openstack-glance12:56
*** alexchadin has quit IRC13:21
*** alexchadin has joined #openstack-glance13:26
openstackgerritAbhishek Kekane proposed openstack/glance-specs master: Modify operator image workflow specs  https://review.openstack.org/58056813:30
*** lbragstad has joined #openstack-glance13:31
openstackgerritAbhishek Kekane proposed openstack/glance master: Hide old images  https://review.openstack.org/57875513:33
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: Add support for hide old images  https://review.openstack.org/57911713:35
*** belmoreira has joined #openstack-glance13:38
*** alexchadin has quit IRC13:40
*** alexchadin has joined #openstack-glance13:41
*** alexchadin has quit IRC13:41
*** alexchadin has joined #openstack-glance13:42
*** alexchadin has quit IRC13:42
*** alexchadin has joined #openstack-glance13:43
*** alexchadin has quit IRC13:43
*** alexchadin has joined #openstack-glance13:43
*** alexchadin has quit IRC13:44
*** r-daneel has joined #openstack-glance13:49
*** jose-phillips has joined #openstack-glance13:54
*** alexchadin has joined #openstack-glance14:02
*** alexchadin has quit IRC14:08
*** Luzi has quit IRC14:11
*** jistr|mtg is now known as jistr14:13
*** btully has joined #openstack-glance14:19
*** links has quit IRC14:22
openstackgerritMerged openstack/glance_store master: cinder: Support os-brick privsep filters  https://review.openstack.org/41520914:23
*** btully has quit IRC14:24
*** alexchadin has joined #openstack-glance14:26
*** AlexeyAbashkin has quit IRC14:59
*** AlexeyAbashkin has joined #openstack-glance14:59
*** belmoreira has quit IRC15:12
*** AlexeyAbashkin has quit IRC15:16
*** AlexeyAbashkin has joined #openstack-glance15:16
*** csatari has quit IRC15:19
*** clarkb has quit IRC15:19
*** mancdaz has quit IRC15:19
*** d34dh0r53 has quit IRC15:20
*** lamt has quit IRC15:20
*** lamt has joined #openstack-glance15:22
*** csatari has joined #openstack-glance15:23
*** lamt is now known as Guest2358115:24
*** d34dh0r53 has joined #openstack-glance15:25
*** mancdaz has joined #openstack-glance15:27
*** clarkb has joined #openstack-glance15:33
*** imacdonn has quit IRC15:34
*** imacdonn has joined #openstack-glance15:38
*** itlinux has joined #openstack-glance15:44
*** alexchadin has quit IRC15:44
*** abhishekk has joined #openstack-glance15:46
*** tshefi has quit IRC15:47
*** gyee has joined #openstack-glance15:51
*** udesale has quit IRC15:56
*** martin__ has joined #openstack-glance16:01
openstackgerritAbhishek Kekane proposed openstack/glance master: Hide old images  https://review.openstack.org/57875516:07
*** AlexeyAbashkin has quit IRC16:20
*** pcaruana has quit IRC16:47
*** tesseract has quit IRC17:02
*** mvk_ has quit IRC17:05
*** tshefi has joined #openstack-glance17:19
*** dtantsur is now known as dtantsur|afk17:24
*** mvk_ has joined #openstack-glance17:44
openstackgerritRick Bartra proposed openstack/glance master: Add Policy enforcement for several Metadata Definition delete APIs  https://review.openstack.org/58453017:53
*** btully has joined #openstack-glance17:57
abhishekkjokke_, around?17:57
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: Add multi-store support  https://review.openstack.org/57537018:00
openstackgerritAbhishek Kekane proposed openstack/python-glanceclient master: Unit tests for multi-store support  https://review.openstack.org/57711518:00
*** btully has quit IRC18:01
*** itlinux has quit IRC18:05
*** r-daneel has quit IRC18:10
*** AlexeyAbashkin has joined #openstack-glance18:25
*** abhishekk has quit IRC18:26
*** AlexeyAbashkin has quit IRC18:45
*** lbragstad_ has joined #openstack-glance18:47
*** lbragstad has quit IRC18:48
*** lbragstad_ is now known as lbragstad18:51
*** jose-phillips has quit IRC18:54
*** jose-phillips has joined #openstack-glance18:54
*** jose-phillips has quit IRC18:55
*** arvindn05 has joined #openstack-glance19:03
*** arvindn05 has quit IRC19:27
*** btully has joined #openstack-glance19:46
*** r-daneel has joined #openstack-glance19:47
*** btully has quit IRC19:51
*** edmondsw has quit IRC20:09
*** itlinux has joined #openstack-glance20:36
*** r-daneel_ has joined #openstack-glance20:44
*** r-daneel has quit IRC20:44
*** r-daneel_ is now known as r-daneel20:44
*** edmondsw has joined #openstack-glance20:56
*** itlinux has quit IRC21:12
*** rcernin has joined #openstack-glance21:26
*** edmondsw has quit IRC22:04
*** hoonetorg has quit IRC22:31
*** hoonetorg has joined #openstack-glance22:48
*** itlinux has joined #openstack-glance22:53
*** martin__ has quit IRC22:55
*** aniketh has quit IRC23:06
*** r-daneel has quit IRC23:08
*** btully has joined #openstack-glance23:23
*** btully has quit IRC23:28
*** threestrands has joined #openstack-glance23:30
*** edmondsw has joined #openstack-glance23:38
*** edmondsw has quit IRC23:43

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