openstackgerrit | Merged openstack/glance: Fix typos in glance/doc/source/configuring.rst https://review.openstack.org/172922 | 00:07 |
---|---|---|
openstackgerrit | Merged openstack/glance: Remove non-ascii characters in glance/doc/source/architecture.rst https://review.openstack.org/172929 | 00:08 |
*** mfedosin has quit IRC | 01:21 | |
*** mfedosin has joined #openstack-glance | 01:30 | |
*** julim has quit IRC | 01:38 | |
*** mfedosin has quit IRC | 01:42 | |
*** mfedosin has joined #openstack-glance | 01:50 | |
openstackgerrit | Fei Long Wang proposed openstack/glance: Fix image owner can't be changed issue in v2 https://review.openstack.org/154695 | 01:51 |
*** diegows has quit IRC | 01:54 | |
*** bkopilov has quit IRC | 02:22 | |
*** panbalag has quit IRC | 02:22 | |
*** dims_ has quit IRC | 02:28 | |
*** spzala has quit IRC | 02:41 | |
*** annegentle has joined #openstack-glance | 02:46 | |
*** annegentle has quit IRC | 02:55 | |
*** wolsen_ has joined #openstack-glance | 03:01 | |
*** wolsen has quit IRC | 03:06 | |
*** tristanC has quit IRC | 03:06 | |
*** tristanC has joined #openstack-glance | 03:13 | |
*** ayoung has quit IRC | 03:14 | |
*** buzztroll has quit IRC | 03:23 | |
*** buzztroll has joined #openstack-glance | 03:24 | |
*** buzztroll has quit IRC | 03:28 | |
*** dims has joined #openstack-glance | 03:28 | |
*** cpallares has joined #openstack-glance | 03:32 | |
*** dims has quit IRC | 03:33 | |
*** ajayaa has joined #openstack-glance | 03:44 | |
*** TravT has quit IRC | 03:45 | |
*** TravT has joined #openstack-glance | 03:49 | |
*** harlowja is now known as harlowja_away | 03:58 | |
*** bkopilov has joined #openstack-glance | 04:02 | |
*** ajayaa has quit IRC | 04:21 | |
*** cpallares has quit IRC | 04:29 | |
*** krtaylor has quit IRC | 04:35 | |
*** buzztroll has joined #openstack-glance | 04:38 | |
*** krtaylor has joined #openstack-glance | 04:38 | |
*** mfedosin has quit IRC | 04:41 | |
*** TravT has quit IRC | 04:50 | |
*** aswadr has joined #openstack-glance | 04:53 | |
*** TravT has joined #openstack-glance | 04:54 | |
*** TravT has quit IRC | 05:00 | |
*** buzztroll has quit IRC | 05:04 | |
*** aswadr has quit IRC | 05:06 | |
*** aswadr has joined #openstack-glance | 05:07 | |
*** ajayaa has joined #openstack-glance | 05:08 | |
*** ishant has joined #openstack-glance | 05:14 | |
*** TravT has joined #openstack-glance | 05:18 | |
*** TravT has quit IRC | 05:22 | |
*** groen692 has joined #openstack-glance | 05:33 | |
*** mfedosin has joined #openstack-glance | 05:39 | |
*** sgotliv has joined #openstack-glance | 05:46 | |
*** Longgeek has joined #openstack-glance | 05:53 | |
*** mfedosin has quit IRC | 06:00 | |
*** sgotliv has quit IRC | 06:05 | |
*** krtaylor has quit IRC | 06:11 | |
*** TravT has joined #openstack-glance | 06:19 | |
*** krtaylor has joined #openstack-glance | 06:22 | |
*** TravT has quit IRC | 06:23 | |
*** nlevinki has joined #openstack-glance | 06:26 | |
*** abhishekk has joined #openstack-glance | 06:28 | |
*** buzztroll has joined #openstack-glance | 06:30 | |
*** mfedosin has joined #openstack-glance | 06:43 | |
*** mfedosin has quit IRC | 06:50 | |
*** jamielennox is now known as jamielennox|away | 07:02 | |
*** arnaud___ has joined #openstack-glance | 07:23 | |
*** chlong has quit IRC | 07:27 | |
*** jistr has joined #openstack-glance | 07:29 | |
*** krykowski has joined #openstack-glance | 07:33 | |
*** buzztroll has quit IRC | 07:40 | |
krykowski | does the blueprint for glanceclient requires a spec for it? | 07:43 |
*** markus_z has joined #openstack-glance | 07:44 | |
*** e0ne has joined #openstack-glance | 07:58 | |
*** e0ne is now known as e0ne_ | 07:58 | |
openstackgerrit | Michal Dulko proposed openstack/python-glanceclient: Add unit tests for log_curl_request https://review.openstack.org/105464 | 08:01 |
*** dims has joined #openstack-glance | 08:07 | |
*** e0ne_ has quit IRC | 08:08 | |
*** TravT has joined #openstack-glance | 08:08 | |
*** e0ne has joined #openstack-glance | 08:08 | |
*** dims has quit IRC | 08:12 | |
*** TravT has quit IRC | 08:13 | |
kragniz | krykowski: yeah, it does | 08:23 |
krykowski | kragniz: so should it be put into glance-specs as I haven't seen a glanceclient-specs repo anywhere? | 08:25 |
*** ivasilevskaya has quit IRC | 08:26 | |
*** arnaud___ has quit IRC | 08:26 | |
*** sgotliv has joined #openstack-glance | 08:26 | |
kragniz | krykowski: yeah | 08:26 |
kragniz | that's specs for the whole glance project | 08:26 |
kragniz | (so glance_store, also) | 08:27 |
*** tshefi has joined #openstack-glance | 08:27 | |
krykowski | kragniz: thanks! its clear to me now | 08:28 |
kragniz | no problem! | 08:29 |
*** sgotliv has quit IRC | 08:36 | |
*** sgotliv has joined #openstack-glance | 08:36 | |
*** e0ne is now known as e0ne_ | 08:45 | |
*** pkoniszewski has joined #openstack-glance | 08:51 | |
*** aix has quit IRC | 08:53 | |
*** e0ne_ is now known as e0ne | 08:55 | |
*** frickler_ is now known as frickler | 09:06 | |
*** hassa-kisar has quit IRC | 09:15 | |
*** pkoniszewski has quit IRC | 09:16 | |
*** asrangne has joined #openstack-glance | 09:18 | |
*** aswadr has quit IRC | 09:18 | |
*** aix has joined #openstack-glance | 09:20 | |
*** e0ne is now known as e0ne_ | 09:40 | |
*** lifeless1 has joined #openstack-glance | 09:54 | |
*** TravT has joined #openstack-glance | 09:57 | |
*** e0ne_ is now known as e0ne | 09:58 | |
*** TravT has quit IRC | 10:01 | |
*** bkopilov has quit IRC | 10:02 | |
*** lifeless has quit IRC | 10:02 | |
*** harlowja_away has quit IRC | 10:02 | |
openstackgerrit | Merged openstack/glance_store: Only warn on duplicate path on fs backend https://review.openstack.org/157013 | 10:22 |
*** ishant has quit IRC | 10:50 | |
*** dims has joined #openstack-glance | 10:54 | |
*** dims has quit IRC | 10:58 | |
*** panbalag has joined #openstack-glance | 11:15 | |
*** asrangne__ has joined #openstack-glance | 11:28 | |
*** asrangne has quit IRC | 11:32 | |
*** aix has quit IRC | 11:32 | |
*** aix has joined #openstack-glance | 11:33 | |
*** e0ne is now known as e0ne_ | 11:34 | |
*** asrangne__ has quit IRC | 11:37 | |
*** TravT has joined #openstack-glance | 11:46 | |
*** jistr is now known as jistr|class | 11:47 | |
*** TravT has quit IRC | 11:50 | |
*** e0ne_ is now known as e0ne | 11:52 | |
*** sgotliv has quit IRC | 12:02 | |
*** sgotliv has joined #openstack-glance | 12:15 | |
*** flaper87 has quit IRC | 12:24 | |
*** flaper87 has joined #openstack-glance | 12:24 | |
*** mfedosin has joined #openstack-glance | 12:32 | |
rosmaita | flwang: pong | 12:42 |
*** MVenesio has joined #openstack-glance | 12:42 | |
*** ajayaa has quit IRC | 12:45 | |
*** stpierre has joined #openstack-glance | 12:45 | |
*** dims has joined #openstack-glance | 12:52 | |
*** cpallares has joined #openstack-glance | 12:55 | |
*** mitz has quit IRC | 13:01 | |
*** cpallares has quit IRC | 13:01 | |
*** jistr|class is now known as jistr | 13:03 | |
*** mitz has joined #openstack-glance | 13:04 | |
*** e0ne is now known as e0ne_ | 13:06 | |
*** e0ne_ is now known as e0ne | 13:08 | |
*** peristeri has joined #openstack-glance | 13:10 | |
*** changbl has quit IRC | 13:12 | |
*** dosaboy has quit IRC | 13:14 | |
*** dosaboy has joined #openstack-glance | 13:14 | |
*** julim has joined #openstack-glance | 13:17 | |
*** haomaiwang has joined #openstack-glance | 13:21 | |
rosmaita | flwang: left a comment for you on https://review.openstack.org/#/c/172781/ | 13:26 |
*** sgotliv has quit IRC | 13:30 | |
*** sgotliv has joined #openstack-glance | 13:30 | |
*** dims has quit IRC | 13:31 | |
*** TravT has joined #openstack-glance | 13:31 | |
*** nlevinki has quit IRC | 13:32 | |
*** dims has joined #openstack-glance | 13:32 | |
*** ajayaa has joined #openstack-glance | 13:33 | |
*** tshefi has quit IRC | 13:33 | |
*** MattMan has joined #openstack-glance | 13:34 | |
*** TravT has quit IRC | 13:36 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:37 | |
*** nlevinki has joined #openstack-glance | 13:48 | |
*** tshefi has joined #openstack-glance | 13:48 | |
*** ajayaa has quit IRC | 13:49 | |
*** r-daneel has joined #openstack-glance | 13:52 | |
*** ayoung_ has joined #openstack-glance | 13:53 | |
sigmavirus24 | I'm back and looking at mclaren's patch | 13:55 |
kragniz | sigmavirus24: welcome back! | 13:55 |
sigmavirus24 | thanks | 13:55 |
*** dims has quit IRC | 14:00 | |
*** dims has joined #openstack-glance | 14:00 | |
*** ajayaa has joined #openstack-glance | 14:02 | |
*** ayoung_ has quit IRC | 14:04 | |
*** cpallares has joined #openstack-glance | 14:05 | |
*** jaypipes has joined #openstack-glance | 14:06 | |
*** abhishekk has quit IRC | 14:12 | |
* flaper87 sprints on sprinting | 14:13 | |
kragniz | is there a project called marathon? | 14:13 |
flaper87 | no idea | 14:14 |
kragniz | it would be fun to sprint on it just for the puns | 14:14 |
* flaper87 can tell sigmavirus24 is in his twitter break | 14:14 | |
flaper87 | kragniz: lol | 14:14 |
cpallares | kragniz: are you *glancing* the sprints? | 14:14 |
sigmavirus24 | heh | 14:14 |
sigmavirus24 | having twitter open in a separate app means there are no breaks flaper87 =P | 14:14 |
kragniz | cpallares: hehe | 14:16 |
flaper87 | sigmavirus24: I do the same... btw, I found out this morning I was not following you. #bad | 14:16 |
kragniz | glanceclient gate is unblocked! | 14:16 |
sigmavirus24 | kragniz: awesome | 14:16 |
flaper87 | cpallares: WHERE ARE YOU? >.> | 14:16 |
flaper87 | kragniz: +2 | 14:16 |
kragniz | (thank mclaren) | 14:16 |
flaper87 | (tell him to come on-line) | 14:17 |
flaper87 | :P | 14:17 |
* cpallares hides | 14:17 | |
sigmavirus24 | flaper87: kragniz which patch is unbreaking the gate? | 14:18 |
kragniz | sigmavirus24: the worst sort of unbreak patch | 14:19 |
kragniz | sigmavirus24: https://review.openstack.org/#/c/172999/ | 14:19 |
*** ayoung_ has joined #openstack-glance | 14:19 | |
*** diegows has joined #openstack-glance | 14:31 | |
*** zz_jgrimm is now known as jgrimm | 14:32 | |
nikhil_k | sigmavirus24: flaper87 kragniz ativelkov mfedosin flwang zhiyan : rc2 backports needed are at https://trello.com/c/CoZ9g13d/45-kilo-rc2 | 14:33 |
nikhil_k | and plan is to cut rc2 by the end of the week else ttx will go with rc1 | 14:33 |
nikhil_k | (that was the info update to help plan this week a bit) | 14:34 |
ativelkov | They have to land in master first, right? | 14:34 |
nikhil_k | ativelkov: yes | 14:34 |
nikhil_k | Another updates: | 14:36 |
nikhil_k | everyone - if not seen this already, please look at http://lists.openstack.org/pipermail/openstack-dev/2015-April/061167.html | 14:36 |
nikhil_k | that has some info for the slot allocation we are looking for Glance (too) | 14:36 |
kragniz | what exactly is a fishbowl? | 14:37 |
flaper87 | kragniz: the sessions as we know them | 14:37 |
nikhil_k | if anyone needs changes please do let me know soon, there isn't much scope for getting more slots however, we can try if needed | 14:37 |
kragniz | flaper87: okay, thanks | 14:37 |
*** sgotliv has quit IRC | 14:41 | |
*** bkopilov has joined #openstack-glance | 14:43 | |
*** cpallares1 has joined #openstack-glance | 14:44 | |
*** smcginni1 is now known as smcginnis | 14:44 | |
*** cpallares has quit IRC | 14:47 | |
*** cpallares1 is now known as cpallares | 14:50 | |
*** cpallares has quit IRC | 14:51 | |
*** mclaren has joined #openstack-glance | 14:51 | |
mclaren | sigmavirus24: ping | 14:51 |
*** sgotliv has joined #openstack-glance | 14:53 | |
*** e0ne is now known as e0ne_ | 14:55 | |
sigmavirus24 | mclaren: pong | 14:56 |
sigmavirus24 | mclaren: I'm testing your change locally | 14:56 |
mclaren | sigmavirus24: hey, thanks for your review. I wasn't 100% sure that I hadn't re-broken things | 14:56 |
*** e0ne_ is now known as e0ne | 14:56 | |
sigmavirus24 | mclaren: yeah it's more of a problem for versions of requests older than (2.5.2 or 2.5.1) that are packaged by debian/fedora/etc | 14:57 |
mclaren | we need something that fixes both 1396550 and 1442664. I'm all ears :-) | 14:57 |
sigmavirus24 | So if you're just using vanilla requests you wouldn't notice it | 14:57 |
sigmavirus24 | It's fairly pernicious the bug/review I linked | 14:57 |
mclaren | yeah, we've hit that ourselves at some point I think | 14:57 |
sigmavirus24 | Also, we always have zigo yelling at us that we shouldn't be permitting imports from the vendored version of urllib3 even though in 2.5.(1|2) onwards that will resolve to the system version of urllib3 if it isn't actually vendored | 14:58 |
sigmavirus24 | so | 14:58 |
mclaren | I'm not sure how to reproduce bug 140151, so I'm not sure how best to update the patch | 15:05 |
*** ayoung_ has quit IRC | 15:13 | |
sigmavirus24 | mclaren: looking at it again though, I think yours doesnt' revert my fix and just adds back the old logic to cover multiple cases | 15:15 |
* sigmavirus24 is also helping openstackclient people with interesting glanceclient behaviour at the moment | 15:16 | |
*** groen692 has quit IRC | 15:16 | |
mclaren | sigmavirus24: do you have a reproducer to test if that is the case? | 15:17 |
sigmavirus24 | mclaren: not yet | 15:17 |
mclaren | will you have a reproducer to test if that is the case? :-) | 15:18 |
*** TravT has joined #openstack-glance | 15:19 | |
*** TravT has quit IRC | 15:20 | |
mclaren | (eg, were you able to test your original change?) | 15:20 |
*** TravT has joined #openstack-glance | 15:20 | |
sigmavirus24 | Yes I was but I didn't test it from nova (hence this bug) | 15:20 |
sigmavirus24 | I can't reproduce that original bug | 15:20 |
*** pdb has quit IRC | 15:20 | |
sigmavirus24 | Also you're not removing https://review.openstack.org/#/c/140151/4/glanceclient/common/https.py,cm | 15:20 |
*** pdb has joined #openstack-glance | 15:21 | |
*** jgrimm has quit IRC | 15:21 | |
mclaren | ah, ok | 15:22 |
*** flaper87 has quit IRC | 15:23 | |
*** cpallares has joined #openstack-glance | 15:24 | |
*** flaper87 has joined #openstack-glance | 15:25 | |
*** flaper87 has quit IRC | 15:25 | |
*** flaper87 has joined #openstack-glance | 15:25 | |
*** ayoung_ has joined #openstack-glance | 15:26 | |
mclaren | sigmavirus24: does it affect ubuntu or just debian? (I'll have a go at reproducing) | 15:27 |
sigmavirus24 | mclaren: yeah I think your patch is okay. I don't like toying with module globals in third party dependencies but it should be fine, especially if it fixes it for y'all | 15:28 |
sigmavirus24 | mclaren: it should affect both | 15:28 |
*** TravT has quit IRC | 15:28 | |
mclaren | ok, I'll have a go at reproducing so. It would give me that warm feeling :-) | 15:28 |
*** TravT has joined #openstack-glance | 15:28 | |
*** diegows has quit IRC | 15:28 | |
*** Longgeek has quit IRC | 15:29 | |
*** krykowski has quit IRC | 15:31 | |
sigmavirus24 | heh | 15:32 |
sigmavirus24 | mclaren: the feeling of a warm cache? | 15:32 |
sigmavirus24 | =P | 15:32 |
panbalag | Hi, Is there any specific parameter that I need to configure in order to enable SSL? | 15:33 |
mclaren | heh | 15:33 |
mclaren | cert_file and key_file for starters | 15:34 |
panbalag | mclaren, I did change the file location ca_file, cert_file etc. But still glance seems to use http and not https. | 15:34 |
*** ayoung_ is now known as ayoung | 15:34 | |
mclaren | can you paste the relevant part of your glance-api.conf? | 15:35 |
panbalag | mclaren, wondering if there is anything like "use_ssl=True" or "enable_ssl=True" ? | 15:35 |
panbalag | ok | 15:35 |
*** stevemar has joined #openstack-glance | 15:35 | |
stevemar | it's one of the few channels i don't lurk in | 15:35 |
mclaren | if you're using devstack you can just put USE_SSL=True in your localrc and all services will be spun up with ssl -- may be a useful reference system | 15:35 |
*** changbl has joined #openstack-glance | 15:36 | |
sigmavirus24 | so the openstackclient is trying to figure out how to get all of the images from Glance (like glance image-list does) but the v2.Images/v1.Images APIs dont' seem to provide a way for that since page_size is always an integer (defaulting to DEFAULT_PAGE_SIZE) and limit is unconditionally set to page_size in the filters (at least in v2). cc flaper87 | 15:36 |
*** tshefi has quit IRC | 15:37 | |
mclaren | > limit is unconditionally set to page_size in the filters (at least in v2). cc flaper87 | 15:37 |
flaper87 | sigmavirus24: that was done on purpose to avoid accidental (?) full scans on the database | 15:38 |
mclaren | I seem to remember putting a bug in for that? | 15:38 |
panbalag | mclaren, I see the parameters in two files glance-api and glance-registry...Do I need to update location on both files? looks like I have done only on glance-registry.conf | 15:38 |
mclaren | panbalag: both | 15:38 |
mclaren | kragniz: didn't you fix the limit issue? ^ | 15:38 |
flaper87 | afaict, we recommend paginating for now. That'd be, more requests that return page_size rather than 1 request that will slow down the database and produce a HUGE response | 15:38 |
mclaren | v2 limit bug https://bugs.launchpad.net/python-glanceclient/+bug/1429088 | 15:39 |
openstack | Launchpad bug 1429088 in python-glanceclient "v2 image-list: '--limit' doesn't work" [Medium,Fix released] - Assigned to Louis Taylor (kragniz) | 15:39 |
*** nlevinki has quit IRC | 15:40 | |
stevemar | flaper87, sigmavirus24 the osc bug is here: https://bugs.launchpad.net/python-openstackclient/+bug/1443089 | 15:40 |
openstack | Launchpad bug 1443089 in python-openstackclient "image list displays a maximum of 25 images" [High,Confirmed] | 15:40 |
sigmavirus24 | flaper87: so the thing is that limit is always the page_size https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/images.py#L129 | 15:40 |
sigmavirus24 | and if limit is None, we don't update the URL https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/images.py#L94 | 15:41 |
stevemar | flaper87, check out comment #5 -- `glance image-list` returns 103 images. | 15:41 |
flaper87 | sigmavirus24: yeah, but that's a bug in the client, right ? (the one mclaren just pasted) | 15:41 |
sigmavirus24 | so unless openstackclient passes some arbitrarily large number (that will always be wrong for people with arbitrarily larger numbers of images) there doesn't seem to be a way to just list all images with pagination | 15:41 |
flaper87 | stevemar: looking | 15:42 |
sigmavirus24 | flaper87: that's related, but this is talkign about doing v2.Images.list | 15:42 |
sigmavirus24 | flaper87: openstackclient uses the API in the client, not the shell | 15:42 |
*** ajayaa has quit IRC | 15:42 | |
flaper87 | sigmavirus24: yup | 15:43 |
mclaren | the latest cli works for listing v2 all/paginating, isn't it just driving the client library? | 15:43 |
*** jgrimm has joined #openstack-glance | 15:44 | |
mclaren | not specifying a limit should give everything? | 15:44 |
* mclaren thinks he's missing something | 15:45 | |
*** ajayaa has joined #openstack-glance | 15:46 | |
flaper87 | this line might be buggy https://github.com/openstack/glance/blob/master/glance/api/v2/images.py#L104 | 15:46 |
stevemar | mclaren, right, not specifying a limit should give everything, but the glance api is setting 'limit' to page size by default | 15:46 |
flaper87 | stevemar: v2 has 2 options. 1 for the default limit and another one for the maximum page_size allowed | 15:47 |
stevemar | yep | 15:47 |
flaper87 | giving everything back is not good, imho. | 15:47 |
mclaren | agreed | 15:47 |
flaper87 | stevemar: does it work with v1 ? | 15:47 |
stevemar | i'm not sure :) | 15:48 |
flaper87 | LOL | 15:48 |
mclaren | stevemar: where in the code does the client set limit to page size? | 15:48 |
stevemar | it's in the bug report, 1 sec | 15:48 |
mclaren | or are we talking about a server side limit? | 15:49 |
*** stevemar has quit IRC | 15:49 | |
flaper87 | LOL | 15:49 |
flaper87 | he literally went to the bug | 15:49 |
flaper87 | he'll be back | 15:49 |
flaper87 | :P | 15:49 |
*** stevemar has joined #openstack-glance | 15:49 | |
flaper87 | mclaren: I believe the server side limit is the one causing problems | 15:49 |
* flaper87 checking glanceclient now | 15:49 | |
* flaper87 memories are blurry | 15:49 | |
*** arnaud___ has joined #openstack-glance | 15:50 | |
stevemar | mclaren, actually, it's in the patch for osc -- https://review.openstack.org/#/c/172763/ -- the last comment | 15:50 |
stevemar | theres a few links in the last comment that i made | 15:50 |
mclaren | flaper87: but v1 does that too. That's a per page limit by the server. | 15:50 |
sigmavirus24 | yeah that's why I stopped trying to help stevemar on my own. I'm too tired | 15:50 |
flaper87 | mclaren: https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/images.py#L88 | 15:50 |
mclaren | thx | 15:50 |
sigmavirus24 | flaper87: that was my review comment :P | 15:50 |
*** jistr has quit IRC | 15:51 | |
mclaren | flaper87: that's the page size, not the 'limit'? | 15:51 |
flaper87 | sigmavirus24: >.> | 15:51 |
sigmavirus24 | mclaren: later on, filters['limit'] = page_size | 15:51 |
sigmavirus24 | flaper87: yeah I told you to write that line that way | 15:51 |
sigmavirus24 | flaper87: I remember telling you to do that =P | 15:51 |
flaper87 | sigmavirus24: I remember that too... >.> | 15:52 |
*** e0ne is now known as e0ne_ | 15:52 | |
sigmavirus24 | lol | 15:52 |
mclaren | filters['limit'] = page_size -> but that's setting up the pagination limit, so the server will only give back that number each page list? | 15:53 |
mclaren | which I think is ok | 15:53 |
flaper87 | I don't have an env set up here to give this a try :/ | 15:54 |
stevemar | mclaren, doesn't explain how in the bug comment etoews got back 103 images :\ | 15:55 |
stevemar | by just doing 'glance image-list' | 15:55 |
mclaren | that was v1 | 15:56 |
mclaren | stevemar: I'm saying I don't see a bug here yet, that that filter limit looks ok to me | 15:57 |
*** buzztroll has joined #openstack-glance | 15:57 | |
mclaren | $ glance --os-image-api-version 2 image-list | wc 47 223 3525 | 15:57 |
mclaren | works for me (tm) | 15:57 |
*** krtaylor has quit IRC | 15:57 | |
*** tshefi has joined #openstack-glance | 16:00 | |
sigmavirus24 | mclaren: so maybe the better question is: How is osc misusing glanceclient? | 16:00 |
*** Longgeek has joined #openstack-glance | 16:00 | |
stevemar | sigmavirus24, ++ | 16:01 |
*** Longgeek has quit IRC | 16:01 | |
*** spzala has joined #openstack-glance | 16:01 | |
stevemar | we're just calling client.images.list() (for the most part) | 16:01 |
*** Longgeek has joined #openstack-glance | 16:01 | |
sigmavirus24 | https://github.com/openstack/python-openstackclient/blob/master/openstackclient/image/v2/image.py#L159 | 16:02 |
mclaren | there may be still an issue with glance here | 16:03 |
mclaren | if the v1 registry is configured with api_limit_max=10 | 16:03 |
*** e0ne_ is now known as e0ne | 16:03 | |
mclaren | $ glance --os-image-api-version 1 image-list | wc -l | 16:03 |
mclaren | 14 | 16:03 |
stevemar | sigmavirus24, ugh right, we're doing our own API call here..... | 16:04 |
stevemar | we're not using the glance libs | 16:04 |
sigmavirus24 | stevemar: ugh | 16:04 |
stevemar | https://github.com/openstack/python-openstackclient/blob/master/openstackclient/api/image_v1.py#L66 | 16:04 |
sigmavirus24 | I was just looking at openstack/api | 16:04 |
sigmavirus24 | heh | 16:04 |
stevemar | this was part of the work ... yeah, you found it | 16:04 |
sigmavirus24 | I was about to ask what that was and if it was used | 16:04 |
sigmavirus24 | yeah that's not doing pagination | 16:04 |
mclaren | makes sense | 16:05 |
sigmavirus24 | sorry mclaren flaper87 | 16:05 |
* sigmavirus24 is still too tired to be working | 16:05 | |
mclaren | np | 16:05 |
stevemar | this was part of the "the SDK isn't ready yet, lets make use our own API" | 16:05 |
stevemar | gahhhhh | 16:05 |
stevemar | time to yell at dtroyer in -sdk :P | 16:05 |
mclaren | I do think that the v1 server will, out of the box, truncate listings to max 1000 and the client isn't handling that | 16:05 |
mclaren | it assumes that since it got less than the page size there are no more images | 16:06 |
*** Longgeek has quit IRC | 16:06 | |
mclaren | actually it will only happen in the corner case that api_limit_max < page_size | 16:07 |
* flaper87 back... | 16:08 | |
flaper87 | sorry, had to go downstairs and get cookies... hungry flaper87 hates happy flaper87 | 16:08 |
*** cpallares has quit IRC | 16:09 | |
openstackgerrit | Merged openstack/python-glanceclient: Add SSL cert verification regression tests https://review.openstack.org/115958 | 16:19 |
*** ajayaa has quit IRC | 16:20 | |
*** tshefi has quit IRC | 16:22 | |
*** obedmr has joined #openstack-glance | 16:23 | |
obedmr | Hi there, does anyone knows if there's a guide for setting HTTPD + WSGI + Glance ? | 16:24 |
mclaren | https://bugs.launchpad.net/glance/+bug/1444056 | 16:24 |
openstack | Launchpad bug 1444056 in Glance "v1 client may truncate listings" [Undecided,New] | 16:24 |
*** panbalag has left #openstack-glance | 16:24 | |
mclaren | obedmr: not sure. Last I heard there were some issues with using apache, but that was a fairly long time ago | 16:25 |
obedmr | thanks mclaren, I just want to enable SSL in the front of Glance API, do you know if there's a good practice for this? I was trying with ProxyReverse but not sure if that's the best approach | 16:26 |
mclaren | obedmr: I suggest try devstack setting USE_SSL=True or the +=tlsxxx option. google devstack ssl. That will give you a reference | 16:27 |
obedmr | mclaren: excellent, thanks for that, I'll try with devstack | 16:27 |
mclaren | np. it's 'tls-proxy' | 16:28 |
obedmr | ok, cool, thanks | 16:29 |
*** annashen has joined #openstack-glance | 16:30 | |
*** arnaud___ has quit IRC | 16:40 | |
*** sgotliv has quit IRC | 16:49 | |
sigmavirus24 | obedmr: F5's work well especially if you need load-balancing. Alternatively, Apache and Nginx have fairly well documented ways of turning on TLS. I suggest reading https://hynek.me/articles/hardening-your-web-servers-ssl-ciphers/ | 16:49 |
sigmavirus24 | but that's generic, not specific to openstack | 16:49 |
*** markus_z has quit IRC | 16:49 | |
obedmr | sigmavirus24: thanks, that definitely helps | 16:50 |
mclaren | sigmavirus24: not having much luck reproducing on ubuntu 14.10, going to call it a day. Let me know if you've any ideas | 16:56 |
sigmavirus24 | mclaren: yeah I don't think our patches are actually complimentary | 16:56 |
sigmavirus24 | sorry | 16:57 |
sigmavirus24 | I *do* think they are | 16:57 |
sigmavirus24 | I don't think you've caused a regression | 16:57 |
* sigmavirus24 needs to sleep more | 16:57 | |
mclaren | ok, if at all possible showing it would be great. Tied various combinations but without much luck | 16:58 |
*** harlowja has joined #openstack-glance | 16:59 | |
*** aswadr has joined #openstack-glance | 17:03 | |
*** e0ne has quit IRC | 17:08 | |
*** annashen has quit IRC | 17:08 | |
*** ericpeterson has joined #openstack-glance | 17:11 | |
sigmavirus24 | mclaren: Yeah I can't find my notes on how I last reproducedit | 17:13 |
sigmavirus24 | but I think everything is okay | 17:13 |
*** e0ne has joined #openstack-glance | 17:13 | |
*** e0ne has quit IRC | 17:14 | |
*** aix has quit IRC | 17:18 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 17:30 | |
*** boris-42 has quit IRC | 17:38 | |
*** ajayaa has joined #openstack-glance | 17:40 | |
*** diegows has joined #openstack-glance | 17:48 | |
*** aswadr has quit IRC | 17:59 | |
*** ajayaa has quit IRC | 18:04 | |
*** jamielennox|away is now known as jamielennox | 18:05 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 18:16 | |
*** TravT has quit IRC | 18:20 | |
*** tellesnobrega has quit IRC | 18:25 | |
*** tellesnobrega has joined #openstack-glance | 18:25 | |
*** tellesnobrega_ has joined #openstack-glance | 18:27 | |
*** tellesnobrega_ has quit IRC | 18:27 | |
*** tellesnobrega has quit IRC | 18:29 | |
*** annegentle has joined #openstack-glance | 18:50 | |
*** gberginc has joined #openstack-glance | 18:51 | |
*** annashen has joined #openstack-glance | 18:56 | |
*** TravT has joined #openstack-glance | 18:59 | |
*** e0ne has joined #openstack-glance | 19:00 | |
*** TravT has quit IRC | 19:00 | |
*** smc7 has joined #openstack-glance | 19:03 | |
*** TravT has joined #openstack-glance | 19:06 | |
*** ericpeterson has quit IRC | 19:09 | |
*** ericpeterson has joined #openstack-glance | 19:10 | |
*** ericpete_ has joined #openstack-glance | 19:11 | |
*** ericpeterson has quit IRC | 19:14 | |
*** diegows has quit IRC | 19:16 | |
*** sgotliv has joined #openstack-glance | 19:21 | |
*** annashen has quit IRC | 19:22 | |
*** aix has joined #openstack-glance | 19:23 | |
*** ericpeterson has joined #openstack-glance | 19:42 | |
*** ericpeterson has quit IRC | 19:42 | |
*** ericpeterson has joined #openstack-glance | 19:43 | |
*** ericpete_ has quit IRC | 19:46 | |
*** jdurgin has joined #openstack-glance | 20:01 | |
*** ayoung has quit IRC | 20:05 | |
*** jgrimm is now known as zz_jgrimm | 20:11 | |
*** ayoung has joined #openstack-glance | 20:17 | |
*** annashen has joined #openstack-glance | 20:18 | |
*** cpallares has joined #openstack-glance | 20:24 | |
*** peristeri has quit IRC | 20:29 | |
*** changbl has quit IRC | 20:37 | |
*** lifeless1 is now known as lifeless | 20:37 | |
*** TravT has quit IRC | 20:52 | |
*** ayoung has quit IRC | 20:55 | |
*** gberginc has quit IRC | 20:59 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 21:03 | |
*** obedmr has left #openstack-glance | 21:06 | |
*** TravT has joined #openstack-glance | 21:06 | |
*** TravT has quit IRC | 21:07 | |
*** TravT has joined #openstack-glance | 21:07 | |
*** ayoung has joined #openstack-glance | 21:11 | |
*** annegentle has quit IRC | 21:14 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 21:17 | |
*** annashen has quit IRC | 21:21 | |
*** annashen has joined #openstack-glance | 21:22 | |
*** stpierre has quit IRC | 21:31 | |
*** sgotliv has quit IRC | 21:31 | |
*** boris-42 has joined #openstack-glance | 21:39 | |
*** annashen has quit IRC | 21:40 | |
*** smc7 has quit IRC | 21:46 | |
*** TravT has quit IRC | 21:46 | |
*** annegentle has joined #openstack-glance | 21:51 | |
*** julim has quit IRC | 21:52 | |
*** e0ne has quit IRC | 21:56 | |
*** smc7 has joined #openstack-glance | 22:03 | |
*** ericpeterson has quit IRC | 22:05 | |
*** cpallares has quit IRC | 22:08 | |
*** annegentle has quit IRC | 22:12 | |
*** annashen has joined #openstack-glance | 22:18 | |
*** wokuma has joined #openstack-glance | 22:41 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:59 | |
*** TravT has joined #openstack-glance | 23:02 | |
*** chlong has joined #openstack-glance | 23:10 | |
*** TravT has quit IRC | 23:25 | |
*** flwang1 has joined #openstack-glance | 23:31 | |
flwang1 | nikhil_k: ping | 23:31 |
*** ayoung has quit IRC | 23:33 | |
flwang1 | flaper87: ping | 23:33 |
*** ericpeterson has joined #openstack-glance | 23:34 | |
flaper87 | flwang1: pong | 23:35 |
*** ericpeterson has quit IRC | 23:38 | |
*** chlong has quit IRC | 23:43 | |
flwang1 | flaper87: a quick quesiton | 23:44 |
flwang1 | so what's our current support api version? | 23:44 |
flwang1 | I assume it should be v1 | 23:44 |
flwang1 | given v2 is not really used anywhere, right? | 23:44 |
*** chlong has joined #openstack-glance | 23:45 | |
flaper87 | flwang1: got disconnected, so re-pong | 23:47 |
flwang1 | (11:44:20 AM) flwang1: flaper87: a quick quesiton. (11:44:31 AM) flwang1: so what's our current support api version? (11:44:37 AM) flwang1: I assume it should be v1 (11:44:56 AM) flwang1: given v2 is not really used anywhere, right? | 23:47 |
flwang1 | sorry, the question should be reworded, 'what's our CURRENT api version?' | 23:48 |
flwang1 | in the response of glance server, it's v1 | 23:48 |
flwang1 | but on api-ref page of openstack, it's v2 | 23:48 |
nikhil_k | it's v2 | 23:48 |
flaper87 | flwang1: Actually, it should be v2 and people using v1 should move to it | 23:48 |
flwang1 | i just want to confirm before submitting a patch | 23:49 |
flaper87 | you know, like py3k and py27 ;) | 23:49 |
nikhil_k | (def core has passed for v2) | 23:49 |
*** annashen has quit IRC | 23:49 | |
nikhil_k | and what flaper87 said | 23:49 |
flwang1 | nikhil_k: flaper87: thanks for the confirmation | 23:49 |
flaper87 | flwang1: np | 23:49 |
flwang1 | I will submit a patch to fix that | 23:49 |
flaper87 | flwang1: thanks | 23:49 |
nikhil_k | np | 23:49 |
nikhil_k | flwang1: ah, is it a RC2 blocker? | 23:50 |
flwang1 | nikhil_k: maybe it's not a bug | 23:50 |
flwang1 | https://github.com/openstack/glance/blob/master/glance/api/versions.py#L67 | 23:51 |
flwang1 | I'm shame as a glance core, I even don't know this config :) | 23:51 |
flwang1 | i'm wondering what happened if both the two config are enabled | 23:51 |
nikhil_k | huh, that's a good call. With DefCore wanting us to go with v2 we should change it | 23:52 |
flaper87 | I think it's a blocker, indeed. | 23:53 |
nikhil_k | flwang1: both versions work in that case | 23:53 |
flaper87 | I don't think we need to disable it, though. | 23:54 |
flwang1 | yep, i know | 23:54 |
nikhil_k | ok | 23:54 |
flaper87 | We should probably just move it to SUPPORTED | 23:54 |
flaper87 | I guess... | 23:54 |
nikhil_k | yeah, I agree. Though Nova not using v2 might make this tricker in some deployments | 23:54 |
nikhil_k | trickier* | 23:54 |
flwang1 | nikhil_k: I mean, should we change it? even if the enable_v1_api=True, the v1 api should still be marked at SUPPORT, instead of current | 23:55 |
flaper87 | right but we're just marking it as SUPPORTED. That means we'll still fix bugs, etc. | 23:55 |
flaper87 | However, we're not stating it'll receive new features (which is what current implies, IMHO) | 23:55 |
nikhil_k | flwang1: Please submit the patch and one of us and -Workflow it until further confirmation | 23:56 |
flwang1 | from the operator perspective, it's intuitive to enable both versions, i think | 23:56 |
nikhil_k | s/and/can/ | 23:56 |
flwang1 | nikhil_k: cool, do it right now | 23:57 |
nikhil_k | thanks | 23:58 |
nikhil_k | flwang1: nice catch ;) | 23:58 |
flwang1 | nikhil_k: how can I reach Venkatesh Sampath ? | 23:59 |
nikhil_k | about what? | 23:59 |
flwang1 | about the deleted images list bug in v1 | 23:59 |
nikhil_k | flwang1: I say because he has moved away from the project/company | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!