Friday, 2018-01-19

tonybCan I upload the same image twice with different names? or will glance detect they have the same hash and tell me to go away?00:31
rosmaitatonyb glance would never tell you to go away00:38
rosmaitayou can store as many copies of the same image as you want as long as you're willing to pay for the storage :)00:38
rosmaitaglance does no de-duping00:38
tonybrosmaita: okay cool.  That makes my life slightly easier00:39
tonybrosmaita: life was so much easier when the only OpenStack I dealt with was devstack ;P00:39
rosmaitain fact, i believe you can upload the same image with the same name00:39
rosmaitait will have a different id00:39
tonybrosmaita: Oh wow00:39
rosmaitano uniqueness constraint on names00:39
* tonyb will try not to do that ;P00:40
* tonyb was sure at soem point I got an error like an image with that name already exists but is happy to be wrong00:41
rosmaita+--------------------------------------+--------------------------+00:43
rosmaita| ID                                   | Name                     |00:43
rosmaita+--------------------------------------+--------------------------+00:43
rosmaita| 9460bb68-4f17-4558-bbe9-43f21f576cdc | cirros-0.3.5-x86_64-disk |00:43
rosmaita| 58f1b252-5427-4fb1-935c-3282e2edbb71 | fred                     |00:43
rosmaita| f8d5d240-cbd7-48c2-859c-f76e51e172e8 | fred                     |00:43
rosmaita| f5a2cd19-170c-4e5d-a471-1f40351b4417 | fred                     |00:43
rosmaita| 2ad71d70-9708-421c-8da3-eec88fcc1e63 | 瞰形像案卷實驗           |00:43
rosmaita+--------------------------------------+--------------------------+00:43
rosmaitachallenge accepted!00:43
rosmaitaglance is like George Foreman ... you can name all your sons 'George'00:45
tonybrosmaita: LOL00:51
*** lbragstad has joined #openstack-glance01:09
*** zhurong has joined #openstack-glance01:30
*** dalgaaf has quit IRC01:52
*** dalgaaf has joined #openstack-glance01:53
*** kukacz_ has quit IRC02:00
*** kukacz_ has joined #openstack-glance02:01
*** rosmaita has quit IRC02:25
*** gcb has joined #openstack-glance02:28
*** harlowja has quit IRC02:31
*** namnh has joined #openstack-glance02:40
*** btully has joined #openstack-glance02:43
*** btully has quit IRC02:47
*** threestrands_ has joined #openstack-glance02:49
*** threestrands_ has quit IRC02:49
*** threestrands_ has joined #openstack-glance02:49
*** threestrands has quit IRC02:51
*** lbragstad has quit IRC03:08
*** AlexeyAbashkin has joined #openstack-glance03:44
*** jappleii__ has joined #openstack-glance03:44
*** jappleii__ has quit IRC03:45
*** jappleii__ has joined #openstack-glance03:45
*** jappleii__ has quit IRC03:46
*** threestrands_ has quit IRC03:46
*** lbragstad has joined #openstack-glance03:46
*** jappleii__ has joined #openstack-glance03:47
*** jappleii__ has quit IRC03:48
*** AlexeyAbashkin has quit IRC03:48
*** jappleii__ has joined #openstack-glance03:48
*** rcernin has quit IRC03:49
*** rcernin has joined #openstack-glance03:49
*** lbragstad has quit IRC04:02
*** abhishekk has joined #openstack-glance04:07
*** pdeore has joined #openstack-glance04:18
*** links has joined #openstack-glance04:37
*** namnh has quit IRC04:38
*** zhurong has quit IRC04:39
*** udesale has joined #openstack-glance04:51
*** ratailor has joined #openstack-glance05:06
*** zhurong has joined #openstack-glance05:19
*** harlowja has joined #openstack-glance05:21
openstackgerritDeepak Mourya proposed openstack/glance master: Handle TZ change in iso8601 >=1.12.0  https://review.openstack.org/53560805:47
*** jappleii__ has quit IRC05:52
*** f0o has quit IRC06:07
*** f0o has joined #openstack-glance06:14
*** btully has joined #openstack-glance06:20
*** arcolife has joined #openstack-glance06:24
*** arcolife has quit IRC06:24
*** arcolife has joined #openstack-glance06:24
*** btully has quit IRC06:25
*** harlowja has quit IRC06:29
*** pcaruana has joined #openstack-glance06:44
*** mosulica has joined #openstack-glance06:55
*** arcolife has quit IRC06:55
*** arcolife has joined #openstack-glance06:56
*** namnh has joined #openstack-glance07:03
openstackgerritJohn L. Villalovos proposed openstack/glance master: DNM/WIP: Prove that parent change fixes oslo.serialization issue  https://review.openstack.org/53516807:31
*** rcernin has quit IRC07:43
*** AlexeyAbashkin has joined #openstack-glance08:13
*** markvoelker has quit IRC08:14
*** tesseract has joined #openstack-glance08:16
*** alexchadin has joined #openstack-glance08:16
*** harlowja has joined #openstack-glance08:17
openstackgerritAndreas Jaeger proposed openstack/glance master: Add doc8 to pep8 check for glance project  https://review.openstack.org/52033708:20
openstackgerritAndreas Jaeger proposed openstack/glance master: Add doc8 to pep8 check for glance project  https://review.openstack.org/52033708:20
*** namnh_ has joined #openstack-glance08:26
*** jmccrory has quit IRC08:26
*** namnh has quit IRC08:28
*** jmccrory has joined #openstack-glance08:33
*** harlowja has quit IRC08:37
*** namnh has joined #openstack-glance08:40
*** namnh_ has quit IRC08:43
*** jmccrory has quit IRC08:50
*** jmccrory has joined #openstack-glance08:52
*** namnh has quit IRC08:54
*** e0ne has joined #openstack-glance09:10
*** e0ne has quit IRC09:10
openstackgerritDeepak Mourya proposed openstack/glance master: Handle TZ change in iso8601 >=1.12.0  https://review.openstack.org/53560809:41
*** alexchadin has quit IRC09:46
*** alexchadin has joined #openstack-glance09:47
*** pcaruana has quit IRC09:50
*** pcaruana has joined #openstack-glance09:53
*** pcaruana has quit IRC09:53
*** pcaruana has joined #openstack-glance09:54
*** mosulica has quit IRC09:57
*** markvoelker has joined #openstack-glance10:15
*** alexchadin has quit IRC10:21
*** arcolife has quit IRC10:26
*** arcolife_ has joined #openstack-glance10:26
*** arcolife_ has quit IRC10:27
*** arcolife has joined #openstack-glance10:27
*** udesale_ has joined #openstack-glance10:28
*** udesale has quit IRC10:28
*** alexchadin has joined #openstack-glance10:31
*** e0ne has joined #openstack-glance10:32
*** e0ne has quit IRC10:37
*** zhurong has quit IRC10:41
*** alexchadin has quit IRC10:46
*** markvoelker has quit IRC10:49
*** abhishekk has quit IRC10:53
*** dtantsur|afk is now known as dtantsur10:54
*** e0ne has joined #openstack-glance10:55
*** alexchadin has joined #openstack-glance11:07
*** ratailor has quit IRC11:15
*** mosulica has joined #openstack-glance11:17
*** udesale_ has quit IRC11:30
*** pdeore has quit IRC11:34
*** markvoelker has joined #openstack-glance11:46
*** mosulica has quit IRC11:50
*** mosulica has joined #openstack-glance11:56
*** e0ne has quit IRC12:12
*** markvoelker has quit IRC12:20
*** hogepodge has quit IRC12:26
*** andreykurilin has quit IRC12:27
*** jdurgin has quit IRC12:27
*** arcolife_ has joined #openstack-glance12:27
*** d34dh0r53 has quit IRC12:27
*** gcb has quit IRC12:28
*** d34dh0r53 has joined #openstack-glance12:28
*** arcolife_ has quit IRC12:28
*** arcolife has quit IRC12:28
*** hogepodge has joined #openstack-glance12:28
*** andreykurilin has joined #openstack-glance12:29
*** jdurgin has joined #openstack-glance12:29
*** szaher has quit IRC13:07
*** arcolife has joined #openstack-glance13:12
*** markvoelker has joined #openstack-glance13:17
*** alexchadin has quit IRC13:22
*** MattMan has quit IRC13:22
*** MattMan has joined #openstack-glance13:22
*** alexchadin has joined #openstack-glance13:27
*** links has quit IRC13:28
*** arcolife has quit IRC13:29
*** markvoelker has quit IRC13:29
*** markvoelker has joined #openstack-glance13:29
*** rosmaita has joined #openstack-glance13:38
*** udesale has joined #openstack-glance13:47
*** udesale has quit IRC13:52
*** e0ne has joined #openstack-glance13:55
bhagyashrisrosmaita: Hi, Thanks for keeping patch https://review.openstack.org/#/c/527635 in priorities. abhishekk and nikhil has review patch given +2 but there is one minor comment given by nikhil so i just want to discussed with you should i addressed this or keep as it is?14:01
*** alexchadin has quit IRC14:04
rosmaitabhagyashris i'll be reviewing your patch this morning (New York time), will put comments14:07
*** mosulica has quit IRC14:08
rosmaitabhagyashris i think it is end of day where you are now?14:08
*** lbragstad has joined #openstack-glance14:10
bhagyashrisrosmaita: ok thank you :)14:13
*** Shilpa has quit IRC14:13
bhagyashrisrosmaita: i am working IST time zone in office14:14
rosmaitaso almost 8pm where you are14:14
*** alexchadin has joined #openstack-glance14:14
rosmaitai think that is end of day14:14
rosmaitahave a good weekend!14:14
rosmaitabhagyashris since your patch has 2 +2, if i don't think you need to address nikhil's point right away, i'll just +A14:15
bhagyashrisrosmaita: yes thank you very much :) Happy weekend :)14:17
rosmaitabhagyashris just looked at nikhil's comment, it's a good idea, might as well hit that if you have a minute14:17
rosmaitait's such a minor change that if i don't have any issues during review, i will carry over nikhil & abhishek's +2s14:17
rosmaita(so it won't hold you up)14:18
rosmaita(i mean won't hold up the patch)14:18
bhagyashrisrosmaita: ok i will hit immediately14:18
rosmaitabhagyashris: you are the best! have a good weekend14:18
*** arcolife has joined #openstack-glance14:21
*** arcolife has quit IRC14:21
*** alexchadin has quit IRC14:25
*** alexchadin has joined #openstack-glance14:29
*** jistr is now known as jistr|mtg14:30
openstackgerritBhagyashri Shewale proposed openstack/glance master: Implementation of Inject metadata properties  https://review.openstack.org/52763514:31
bhagyashrisrosmaita: Done.14:32
*** alexchadin has quit IRC14:32
rosmaitabhagyashris: great! enjoy your weekend14:34
bhagyashrisrosmaita: yup :)14:34
*** alexchadin has joined #openstack-glance14:34
*** mosulica has joined #openstack-glance14:39
*** bhagyashris has quit IRC14:46
*** alexchadin has quit IRC14:50
*** jistr|mtg is now known as jistr15:04
*** stephenfin is now known as finucannot15:38
*** kukacz_ has quit IRC15:58
*** kukacz has joined #openstack-glance15:59
*** jlvillal has quit IRC16:04
*** jlvillal has joined #openstack-glance16:14
*** cfriesen has joined #openstack-glance16:16
cfriesenhey all...I've got a question about https://github.com/openstack/glance/blob/master/glance/api/v2/images.py#L316      As described in the last comment in https://bugs.launchpad.net/glance/+bug/1595335 there's a scenario where nova creates an empty image, creates a file directly in ceph, then wants to update the location of the image in glance.16:21
openstackLaunchpad bug 1595335 in Glance "Add image location fails when show_multiple_locations = false" [Undecided,Confirmed]16:21
cfriesenIf CONF.show_multiple_locations is False (the default) that fails.16:21
cfriesenwould it be reasonable to allow setting the location of an rbd image if the location has not yet been set?16:21
*** mosulica has quit IRC16:29
*** AlexeyAbashkin has quit IRC16:35
rosmaitacfriesen: you mean as a way to fix the issue?16:55
*** gyee has joined #openstack-glance16:58
jlvillalrosmaita, I thought I would ping you about: https://review.openstack.org/#/c/535166/17:00
jlvillalrosmaita, That is the patch which fixes the issue when moving to the new version of oslo.serialization17:00
openstackgerritBrianna Poulos proposed openstack/glance master: Update Signature Documentation  https://review.openstack.org/53585617:02
rosmaitajlvillal thanks for the reminder, that's on my list for today ... we have verified that the oslo change doesn't seem to impact glance_store or python-glanceclient17:02
jlvillalrosmaita, Great and thanks.17:03
rosmaita(at least it doesn't break any tests)17:03
cfriesenrosmaita: yes....or if not do you have any other suggestions on how to deal with this scenario?17:05
jungleboyjrosmaita:  Have time for a newbie question?17:08
*** e0ne has quit IRC17:25
*** mvenesio has joined #openstack-glance17:27
*** pcaruana has quit IRC17:36
*** r-daneel has joined #openstack-glance17:59
*** AlexeyAbashkin has joined #openstack-glance18:16
*** AlexeyAbashkin has quit IRC18:20
*** dtantsur is now known as dtantsur|afk18:24
rosmaitacfriesen, jungleboyj just got back18:28
jungleboyjrosmaita:  Hey.  Question for you about how 'architecture' is handled on images.18:29
rosmaitacfriesen best way to deal with it short term is to have a dedicated "service" glance node that only nova (or other services) can access, then you can show locations on that node without exposing them to end users, who have to use a different glance node with show locations off18:29
jungleboyjSeeing a strange issue with RHOSP where if I don't have x86_64 set for the architecture for an image it tells me there are no hosts available.18:29
jungleboyjI can't figure out how to show what is set on an image.18:30
rosmaitaif you don't see 'architecture' in the image-show response, then it's not set18:31
rosmaitait's a "common image property", so optional18:32
rosmaita but its value is enforced by the image schema18:32
rosmaitanot sure that answers your question?18:32
jungleboyjWeird.  Ok.  And I tried doing an explict set of openstack image set --property architecture=x86_6418:32
rosmaitanot sure about the osc, but with glanceclient you do glance image-update --architecture x86_64 <image_id>18:34
jungleboyjrosmaita:  Oh, ok I see it in properties on one that does boot.18:34
rosmaitalooks like osc uses --architecture , too18:35
jungleboyjOk, it did get set so there must be something else wrong with the image.18:35
rosmaitayou'd have to check the nova logs, probably the api if it uses the property for scheduling a build18:36
jungleboyjYeah, I was digging through those and didn't find much helpful but I didn't want to fight with that too much before I learned more.18:38
rosmaitajungleboyj yeah, there's a lot of stuff in those logs ... maybe grepping the image id?18:48
jungleboyjYeah, I will have to go dig into that further.  I am not sure what is going on here.18:51
jungleboyjHave other battles to fight in the mean time.18:51
jungleboyjrosmaita:  Thanks for the pointers.18:51
rosmaitanp18:52
*** btully has joined #openstack-glance19:02
*** btully has quit IRC19:07
*** mvenesio has quit IRC19:08
cfriesenrosmaita: I've got no problems patching glance locally, just wondering whether there's a reasonable code change I could propose upstream to fix the problem.  (Since otherwise we're talking about complicated documentation changes.)19:10
cfriesenrosmaita: if no location has been set yet, it seems not-totally-insane to allow the location to be set by the user.19:11
*** e0ne has joined #openstack-glance19:12
*** e0ne has quit IRC19:17
rosmaitacfriesen: unfortunately, you pretty much never want the location set by an actual end user19:17
rosmaitai think the solution will involve modifying glance to recognize keystone "service tokens" (i believe they are a thing)19:18
rosmaitaglance needs to know that a request is coming from a service before it can allow location mods19:18
*** harlowja has joined #openstack-glance19:31
cfriesenrosmaita: why wouldn't you allow an admin user to upload something directly into ceph, then create a blank image and update the location?19:42
*** AlexeyAbashkin has joined #openstack-glance19:45
*** e0ne has joined #openstack-glance19:47
rosmaitacfriesen: i think that would be ok if the admin user is using the isolated "service" node19:47
cfriesenI think the idea of needing a separate glance node is silly...I do like the idea of using service tokens though.19:48
rosmaitawell, the difference is that you can use a separate node right now, using service tokens requires some code changes19:49
*** AlexeyAbashkin has quit IRC19:49
cfriesenwe patch our stuff locally, so that's no problem.  I'm just looking at what the proper solution would be for upstream.19:50
rosmaitai don't think it will be an easy patch solution.  here's a large part of the problem: https://review.openstack.org/#/c/528021/19:51
rosmaitacfriesen but i don't mean to discourage you from working on this! in fact, i would welcome it immensely.  just want you to understand the scope of the problem19:55
cfriesenrosmaita: hmm...yes that does throw a wrench in the works.  I'm more used to the nova policies.19:55
rosmaitacfriesen yeah, the way glance did it looked like a good idea at the time, but it has caused some ... complications19:56
rosmaitabut i think some combination of policy reform + service tokens (or whatever keystone has on offer like them) is the way to go to fix this19:57
cfriesenrosmaita: although, in this particular case the check is in _do_add_locations() in glance/api/v2/images.py where it explicitly checks CONF.show_multiple_locations.  Would it be that hard to check the service token here?19:58
rosmaitayeah, but the actual location property getter/setters are protected by policies ... somehow the info has to get "down there"20:00
cfriesenrosmaita: could you make a service role, add the role to the nova user, and use that role on the policy for setting location?20:10
cfriesenanother question...the "direct_url" field already seems to show the location of the file in ceph, so what do we gain hiding the location?20:16
*** e0ne has quit IRC20:21
*** AlexeyAbashkin has joined #openstack-glance20:27
rosmaitacfriesen: show_direct_url is False by default20:30
*** AlexeyAbashkin has quit IRC20:31
rosmaitacfriesen: about your nova policy idea ... i'm pretty sure nova uses the user's credentials when it interacts with glance, it's not a special nova user20:32
rosmaitacfriesen: some of the discussion on this review may be helpful: https://review.openstack.org/#/c/444540/20:34
*** e0ne has joined #openstack-glance20:35
*** e0ne has quit IRC20:37
*** mvenesio has joined #openstack-glance20:42
*** mvenesio has quit IRC20:46
*** mvenesio has joined #openstack-glance20:47
cfriesenlooks like nova has the option (which defaults to false) of sending a service token along with the user token20:53
rosmaitacfriesen that's good, then glance needs to be made service-token-aware21:01
*** e0ne has joined #openstack-glance21:04
*** e0ne has quit IRC21:05
*** edmondsw has joined #openstack-glance21:22
*** mvenesio has quit IRC21:29
*** e0ne has joined #openstack-glance21:29
*** e0ne has quit IRC21:33
*** tesseract has quit IRC22:01
*** edmondsw has quit IRC22:37
*** btully has joined #openstack-glance22:40
*** btully has quit IRC22:44
*** AlexeyAbashkin has joined #openstack-glance23:22
*** AlexeyAbashkin has quit IRC23:26
-openstackstatus- NOTICE: Zuul will be offline over the next 20 minutes to perform maintenance; active changes will be reenqueued once work completes, but new patch sets or approvals during that timeframe may need to be rechecked or reapplied as appropriate23:42

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