| *** mtanino has quit IRC | 00:13 | |
| *** ducttape_ has joined #openstack-glance | 00:20 | |
| *** ducttape_ has quit IRC | 00:31 | |
| *** jose-phillips has quit IRC | 00:38 | |
| dharinic | Wanted to remind on this glance_store backports: | 00:49 |
|---|---|---|
| dharinic | https://review.openstack.org/#/c/417329/ | 00:49 |
| dharinic | https://review.openstack.org/#/c/417197 | 00:49 |
| *** tovin07 has joined #openstack-glance | 00:57 | |
| openstackgerrit | Brian Rosmaita proposed openstack/glance: Add release note for image visibility changes https://review.openstack.org/422897 | 01:10 |
| openstackgerrit | Cao Xuan Hoang proposed openstack/glance_store: Add Python 3.5 classifier and venv https://review.openstack.org/422356 | 01:27 |
| *** jdurgin has quit IRC | 01:34 | |
| *** ducttape_ has joined #openstack-glance | 01:55 | |
| *** ducttape_ has quit IRC | 01:59 | |
| *** ducttape_ has joined #openstack-glance | 02:13 | |
| *** links has joined #openstack-glance | 02:53 | |
| *** ducttape_ has quit IRC | 02:54 | |
| tovin07 | nikhil, could you review this patch https://review.openstack.org/#/c/316799/ it's quite small and being held for a long time... thanks! :D | 03:14 |
| *** itisha has joined #openstack-glance | 03:30 | |
| *** gb21 has joined #openstack-glance | 03:36 | |
| *** nicolasbock has quit IRC | 03:41 | |
| *** flwang1 has quit IRC | 03:50 | |
| *** links has quit IRC | 04:16 | |
| *** links has joined #openstack-glance | 04:18 | |
| *** pdeore has joined #openstack-glance | 04:24 | |
| *** udesale has joined #openstack-glance | 04:25 | |
| *** ratailor has joined #openstack-glance | 04:28 | |
| *** udesale has quit IRC | 04:30 | |
| *** ratailor has quit IRC | 04:50 | |
| *** catinthe_ has quit IRC | 04:54 | |
| *** catintheroof has joined #openstack-glance | 05:02 | |
| *** pdeore has quit IRC | 05:03 | |
| *** gb21 has quit IRC | 05:09 | |
| *** gb21 has joined #openstack-glance | 05:09 | |
| *** udesale has joined #openstack-glance | 05:09 | |
| *** udesale has quit IRC | 05:10 | |
| *** udesale has joined #openstack-glance | 05:11 | |
| *** catintheroof has quit IRC | 05:14 | |
| *** sabari has quit IRC | 05:35 | |
| *** pdeore has joined #openstack-glance | 05:41 | |
| *** sabari has joined #openstack-glance | 05:41 | |
| *** catintheroof has joined #openstack-glance | 05:46 | |
| *** ratailor has joined #openstack-glance | 05:50 | |
| *** pdeore__ has joined #openstack-glance | 06:41 | |
| *** pdeore has quit IRC | 06:42 | |
| *** pdeore__ has quit IRC | 06:52 | |
| *** pdeore__ has joined #openstack-glance | 06:53 | |
| *** pdeore__ has quit IRC | 06:58 | |
| *** pdeore__ has joined #openstack-glance | 07:01 | |
| *** pdeore__ has quit IRC | 07:06 | |
| *** pdeore__ has joined #openstack-glance | 07:07 | |
| *** gabor_antal_ has joined #openstack-glance | 07:13 | |
| *** gabor_antal has quit IRC | 07:14 | |
| *** adisky_ has joined #openstack-glance | 07:17 | |
| *** pdeore__ has quit IRC | 07:23 | |
| *** pdeore__ has joined #openstack-glance | 07:24 | |
| *** pdeore has joined #openstack-glance | 07:30 | |
| *** pdeore__ has quit IRC | 07:30 | |
| *** gabor_antal_ has quit IRC | 07:37 | |
| *** gabor_antal has joined #openstack-glance | 07:37 | |
| *** pcaruana has joined #openstack-glance | 07:43 | |
| *** jokke__ has joined #openstack-glance | 07:46 | |
| *** dosaboy has joined #openstack-glance | 07:47 | |
| *** melwitt_ has joined #openstack-glance | 07:48 | |
| *** flwang has quit IRC | 07:51 | |
| *** dosaboy_ has quit IRC | 07:51 | |
| *** melwitt has quit IRC | 07:51 | |
| *** jokke_ has quit IRC | 07:51 | |
| *** ccard has quit IRC | 07:51 | |
| *** ccard has joined #openstack-glance | 07:52 | |
| *** flwang has joined #openstack-glance | 07:54 | |
| *** mosulica has joined #openstack-glance | 08:02 | |
| *** gb21 has quit IRC | 08:03 | |
| *** tesseract has joined #openstack-glance | 08:05 | |
| *** jlwhite has quit IRC | 08:29 | |
| *** tovin07 has quit IRC | 08:30 | |
| *** tovin07 has joined #openstack-glance | 08:30 | |
| *** jlwhite has joined #openstack-glance | 08:31 | |
| *** mfedosin has joined #openstack-glance | 08:31 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-glance | 09:00 | |
| *** openstackgerrit has quit IRC | 09:02 | |
| *** namnh has joined #openstack-glance | 09:05 | |
| *** johnthetubaguy has quit IRC | 09:16 | |
| *** openstackgerrit has joined #openstack-glance | 09:18 | |
| openstackgerrit | Adam Kijak proposed openstack/glance_store: An unit test passes because is launched as non-root user https://review.openstack.org/422579 | 09:18 |
| *** johnthetubaguy has joined #openstack-glance | 09:19 | |
| *** ratailor has quit IRC | 09:29 | |
| *** timburke has quit IRC | 09:32 | |
| *** timburke has joined #openstack-glance | 09:35 | |
| *** pdeore has quit IRC | 09:37 | |
| tovin07 | mfedosin, hi | 09:41 |
| openstackgerrit | Abhishek Kekane proposed openstack/python-glanceclient: Add request id to returned objects https://review.openstack.org/352892 | 09:42 |
| mfedosin | tovin07: hi :) | 09:44 |
| *** mvk has quit IRC | 09:46 | |
| *** pdeore has joined #openstack-glance | 09:47 | |
| *** pdeore is now known as pdeore|afk | 09:47 | |
| tovin07 | Could you review this one: https://review.openstack.org/#/c/316799/ i saw you +2 before | 09:48 |
| mfedosin | sure, give me a minute | 09:50 |
| tovin07 | oh, thanks, too fast for me to handle this :v | 09:53 |
| tovin07 | nikhil, are you there? | 09:53 |
| *** ratailor has joined #openstack-glance | 09:54 | |
| *** e0ne has joined #openstack-glance | 10:12 | |
| *** gcb has joined #openstack-glance | 10:14 | |
| *** mvk has joined #openstack-glance | 10:17 | |
| *** abhishekk has joined #openstack-glance | 10:18 | |
| *** aarefiev_afk is now known as aarefiev | 10:26 | |
| *** mdbooth has quit IRC | 10:26 | |
| *** mdbooth has joined #openstack-glance | 10:26 | |
| *** namnh has quit IRC | 10:29 | |
| *** gcb has quit IRC | 10:34 | |
| *** mosulica has quit IRC | 10:58 | |
| *** mosulica has joined #openstack-glance | 11:00 | |
| *** ducttape_ has joined #openstack-glance | 11:29 | |
| *** ducttape_ has quit IRC | 11:31 | |
| *** mosulica has quit IRC | 11:32 | |
| *** nicolasbock has joined #openstack-glance | 11:35 | |
| *** mosulica has joined #openstack-glance | 11:38 | |
| *** pdeore|afk has quit IRC | 11:57 | |
| *** pcaruana has quit IRC | 11:59 | |
| *** pcaruana has joined #openstack-glance | 12:06 | |
| *** cdelatte has joined #openstack-glance | 12:08 | |
| *** adisky_ has quit IRC | 12:09 | |
| *** smatzek_ has joined #openstack-glance | 12:24 | |
| *** catinthe_ has joined #openstack-glance | 12:32 | |
| *** catintheroof has quit IRC | 12:33 | |
| *** ratailor has quit IRC | 12:42 | |
| *** links has quit IRC | 12:55 | |
| *** ducttape_ has joined #openstack-glance | 13:11 | |
| sigmavirus | dharinic: when you get in it looks like community images may have broken Glance v1: http://logs.openstack.org/54/423154/1/check/gate-osc-dsvm-functional-ubuntu-xenial/f23fbfa/testr_results.html.gz | 13:12 |
| sigmavirus | cc hemanthm rosmaita ^ | 13:13 |
| rosmaita | NOOOOOOOO!!!! | 13:13 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/glance: Use OS::Nova::HostAggregate instead of OS::Nova::Aggregate https://review.openstack.org/359742 | 13:17 |
| *** gcb has joined #openstack-glance | 13:19 | |
| gcb | glance cores, glance review https://review.openstack.org/#/c/359742/ is needed by horizon review https://review.openstack.org/#/c/367161/, please help review when you're free, thanks | 13:20 |
| *** ducttape_ has quit IRC | 13:21 | |
| *** ducttape_ has joined #openstack-glance | 13:24 | |
| openstackgerrit | Brian Rosmaita proposed openstack/glance: Add release note for image visibility changes https://review.openstack.org/422897 | 13:28 |
| *** ducttape_ has quit IRC | 13:34 | |
| *** dtroyer has joined #openstack-glance | 13:49 | |
| openstackgerrit | Dean Troyer proposed openstack/glance: Revert "Implement and Enable Community Images" https://review.openstack.org/423286 | 14:02 |
| *** Dinesh_Bhor has quit IRC | 14:04 | |
| openstackgerrit | Dean Troyer proposed openstack/glance: Revert "Implement and Enable Community Images" https://review.openstack.org/423286 | 14:13 |
| sigmavirus | gcb: that should have been mentioned at the meeting. We'll add it to our priorities when we next create the list (at next week's meeting) | 14:15 |
| gcb | sigmavirus, thanks | 14:15 |
| *** smatzek_ has quit IRC | 14:31 | |
| *** catintheroof has joined #openstack-glance | 14:33 | |
| *** mosulica has quit IRC | 14:36 | |
| *** catinthe_ has quit IRC | 14:36 | |
| *** mosulica has joined #openstack-glance | 14:37 | |
| *** ducttape_ has joined #openstack-glance | 14:53 | |
| *** ratailor has joined #openstack-glance | 14:55 | |
| *** melwitt_ is now known as melwitt | 14:55 | |
| *** smatzek has joined #openstack-glance | 14:58 | |
| *** ducttape_ has quit IRC | 15:02 | |
| *** gcb has quit IRC | 15:09 | |
| *** mtanino has joined #openstack-glance | 15:10 | |
| *** TravT has joined #openstack-glance | 15:11 | |
| *** david-lyle has joined #openstack-glance | 15:13 | |
| *** udesale has quit IRC | 15:21 | |
| *** udesale has joined #openstack-glance | 15:21 | |
| *** udesale has quit IRC | 15:24 | |
| *** udesale has joined #openstack-glance | 15:25 | |
| *** david-lyle has quit IRC | 15:25 | |
| *** david-lyle has joined #openstack-glance | 15:25 | |
| *** ducttape_ has joined #openstack-glance | 15:29 | |
| *** david-lyle has quit IRC | 15:42 | |
| *** mvk has quit IRC | 15:48 | |
| *** ratailor has quit IRC | 15:52 | |
| *** burgerk has joined #openstack-glance | 15:59 | |
| openstackgerrit | Dean Troyer proposed openstack/glance: Add image update tests for is_public https://review.openstack.org/423344 | 16:01 |
| openstackgerrit | Dean Troyer proposed openstack/glance: Add image update tests for is_public https://review.openstack.org/423345 | 16:02 |
| dtroyer | I am attempting to isolate the change in v1 both with and without https://review.openstack.org/369110 in those two ^^^^^ | 16:02 |
| *** pcaruana has quit IRC | 16:03 | |
| dtroyer | sigmavirus, rosmaita: ^^^ hope that helps narrow it down | 16:03 |
| sigmavirus | dtroyer: thanks =) | 16:04 |
| sigmavirus | we just want to unblock you for now and swarm on making sure we don't break that when we unrevert the change | 16:04 |
| sigmavirus | or revert your revert | 16:04 |
| sigmavirus | or however you want to think about it ;) | 16:05 |
| sigmavirus | Meanwhile I'm tackling the WebOb 1.7 incompatibilities | 16:05 |
| sigmavirus | So I'm not much use yet | 16:05 |
| sigmavirus | hopefully I'm down to the last test | 16:05 |
| sigmavirus | Which if it's a glare test, rosmaita, I'm just going to skip | 16:05 |
| sigmavirus | Because I thought glare was going to be burnt to a crisp this cycle but apparently it's not | 16:05 |
| dtroyer | thanks for that. and the webob bits too, that sounds like a nasty situation | 16:06 |
| sigmavirus | dtroyer: glance's functional tests were what was primarily failing and there's no way to pdb into those subprocesses effectively so it's a whole lot of pausing the test before the temp logs are gone and a lot of log debugging | 16:07 |
| sigmavirus | but I think I'm down to the last failing test | 16:07 |
| *** tovin07 has quit IRC | 16:08 | |
| sigmavirus | It appears, in glance, to be too much to not duplicate logic/code between API versions | 16:08 |
| *** hieulq has quit IRC | 16:09 | |
| *** mosulica has quit IRC | 16:09 | |
| dtroyer | I still get pushback on that in OSC, but this sort of thing is exactly why I believe we need to duplicate it. Plus it makes removing an old API _really_ easy later. | 16:10 |
| sigmavirus | Some of the request deserialization doesn't change though | 16:10 |
| sigmavirus | I also have a feeling our tests would be faster if we could lesson the verbosity of our test suite | 16:12 |
| *** tovin07 has joined #openstack-glance | 16:14 | |
| *** hieulq has joined #openstack-glance | 16:15 | |
| dharinic | sigmavirus: looking. | 16:24 |
| *** e0ne has quit IRC | 16:44 | |
| openstackgerrit | Ian Cordasco proposed openstack/glance: Fix incompatibilities with WebOb 1.7 https://review.openstack.org/423366 | 16:47 |
| *** itisha has quit IRC | 16:52 | |
| *** sigmavirus sets mode: -o sigmavirus | 16:53 | |
| *** _ducttape_ has joined #openstack-glance | 16:59 | |
| *** jose-phillips has joined #openstack-glance | 17:00 | |
| *** udesale has quit IRC | 17:02 | |
| *** ducttape_ has quit IRC | 17:02 | |
| *** jose-phillips has quit IRC | 17:12 | |
| *** nicolasbock has quit IRC | 17:18 | |
| *** nicolasbock has joined #openstack-glance | 17:18 | |
| *** jdillaman has quit IRC | 17:28 | |
| *** jdillaman has joined #openstack-glance | 17:29 | |
| *** mvk has joined #openstack-glance | 17:29 | |
| *** _ducttape_ has quit IRC | 17:35 | |
| *** ducttape_ has joined #openstack-glance | 17:36 | |
| *** marst has joined #openstack-glance | 17:49 | |
| *** hieulq has quit IRC | 17:56 | |
| *** tovin07 has quit IRC | 17:56 | |
| *** e0ne has joined #openstack-glance | 18:06 | |
| *** hieulq has joined #openstack-glance | 18:12 | |
| *** tovin07 has joined #openstack-glance | 18:12 | |
| sigmavirus | dtroyer: rosmaita can one of you make a bug for the v1 regression and tag me in it? | 18:14 |
| sigmavirus | I have suspect I'll have a fix before EOD | 18:14 |
| sigmavirus | rosmaita: did you ever send out a priorities email for this week? | 18:15 |
| *** dalgaaf has joined #openstack-glance | 18:15 | |
| rosmaita | sigmavirus: not yet | 18:19 |
| sigmavirus | okay can you include https://review.openstack.org/423366 like we discussed yesterday | 18:20 |
| rosmaita | will do | 18:20 |
| rosmaita | will also create bug for you | 18:21 |
| rosmaita | sigmavirus: https://bugs.launchpad.net/glance/+bug/1658164 | 18:27 |
| openstack | Launchpad bug 1658164 in Glance "community images breaks Images v1 API" [Critical,Triaged] - Assigned to Ian Cordasco (icordasc) | 18:27 |
| *** mvk has quit IRC | 18:30 | |
| sigmavirus | thanks | 18:33 |
| *** e0ne has quit IRC | 18:35 | |
| *** esmiurium has quit IRC | 18:40 | |
| sigmavirus | dharinic: do you have a public ssh key somewhere? | 18:45 |
| sigmavirus | I'm can add you to this environment and we can dig into this together | 18:45 |
| *** esmiurium has joined #openstack-glance | 18:47 | |
| dharinic | sure sigmavirus | 18:48 |
| *** TravT has quit IRC | 18:59 | |
| *** TravT has joined #openstack-glance | 19:11 | |
| *** TravT_ has joined #openstack-glance | 19:13 | |
| *** TravT has quit IRC | 19:16 | |
| *** david-lyle has joined #openstack-glance | 19:17 | |
| *** david-lyle has quit IRC | 19:24 | |
| *** dalgaaf has quit IRC | 19:29 | |
| *** dalgaaf has joined #openstack-glance | 19:30 | |
| *** tesseract has quit IRC | 19:32 | |
| *** kebray has joined #openstack-glance | 19:32 | |
| openstackgerrit | Ian Cordasco proposed openstack/glance: Fix regression introduced by Community Images https://review.openstack.org/423499 | 19:33 |
| sigmavirus | dtroyer: rosmaita ^ | 19:34 |
| sigmavirus | stevemar: too ^ | 19:35 |
| sigmavirus | and now I'm going to go take a break | 19:35 |
| stevemar | sigmavirus: heyo! | 19:35 |
| rosmaita | sigmavirus: well earned break | 19:35 |
| rosmaita | that was not easy to see | 19:36 |
| sigmavirus | rosmaita: not in the slightest | 19:36 |
| sigmavirus | the func test helped though | 19:36 |
| rosmaita | hooray for tests | 19:36 |
| sigmavirus | I should probably pull in dtroyer's unit test too but | 19:36 |
| stevemar | sigmavirus: func tests usually do :) | 19:36 |
| sigmavirus | we can also just rebase it atop | 19:37 |
| dharinic | sigmavirus: yayy | 19:38 |
| rosmaita | i am just writing up the priorities, what't the procedure on this? | 19:38 |
| sigmavirus | rosmaita: huh? | 19:38 |
| sigmavirus | well we could unapprove dtroyer's patch | 19:38 |
| sigmavirus | no need to go through with a revert when we have a fix | 19:39 |
| sigmavirus | I'd like stevemar or dtroyer to okay that though | 19:39 |
| stevelle | +1 | 19:39 |
| rosmaita | that would be easiest | 19:39 |
| sigmavirus | *patch -> revert | 19:39 |
| sigmavirus | When I get back, I think I might through up a test no-merge patch to osc that depends on our fix | 19:40 |
| sigmavirus | *throw | 19:40 |
| * sigmavirus needs coffee | 19:40 | |
| stevemar | sigmavirus: rosmaita you could ask someone from infra to snipe the revert out of the gate | 19:40 |
| stevemar | not sure if its merged yet | 19:40 |
| sigmavirus | stevemar: wasn't last I checked | 19:40 |
| sigmavirus | rosmaita: can you handle that? | 19:40 |
| rosmaita | sure | 19:41 |
| stevemar | rosmaita: ask fungi hes the guru there | 19:41 |
| fungi | (for some definitions of "guru" anyway) | 19:42 |
| stevemar | i trust sigmavirus to have it fixed up :) | 19:42 |
| stevemar | fungi: can you snipe https://review.openstack.org/#/c/423286/ ? | 19:42 |
| fungi | ahh, you need it to stop running tests in the gate? sure | 19:43 |
| openstackgerrit | Jeremy Stanley proposed openstack/glance: Revert "Implement and Enable Community Images" https://review.openstack.org/423286 | 19:43 |
| fungi | there you go (rebase button in gerrit to the rescue!) | 19:43 |
| stevemar | fungi: yeah, turns out sigmavirus found a solution that doesn't involve undoing everything https://review.openstack.org/#/c/423499/ | 19:44 |
| stevelle | heh | 19:44 |
| stevemar | fungi: i waaaas gonna do that if you didn't get back to me hehe | 19:44 |
| fungi | if a new patchset gets pushed for a change, zuul will immediately remove it from dependent pipelines like the gate | 19:44 |
| fungi | gerrit's server-side rebase is the fastest means of making one of those, unless the change can't be rebased for $reasons | 19:45 |
| fungi | i don't have an super secret admin tool to remove changes from a pipeline. a few people started work on a dequeue subcommand for the zuul rpc cli client but it's never been finished | 19:46 |
| rosmaita | fungi: thanks for the help | 19:46 |
| fungi | and will almost certainly wait for zuul v3 at this stage | 19:46 |
| fungi | any time! | 19:46 |
| stevemar | fungi: if can't rebase, then re-propose with a character change :) | 19:47 |
| fungi | yup | 19:47 |
| fungi | keep in mind that it's far easier to block a change from merging if you don't care that it hangs around in the gate pipeline until it tries and fails to merge. any of the following will get you that: remove all workflow +1 or code-review +2 votes, add a workflow -1 or code-review -2 vote | 19:48 |
| *** jose-phillips has joined #openstack-glance | 19:48 | |
| fungi | also abandoning will do the trick (i don't recall if that also removes it from the gate immediately or simply prevents it from eventually merging) | 19:49 |
| *** catinthe_ has joined #openstack-glance | 19:50 | |
| rosmaita | that's actually a lot of options, funny how in the anxiety of making this patch not merge, i didn't think of any of them! | 19:51 |
| *** catintheroof has quit IRC | 19:53 | |
| dtroyer | rosmaita, sigmavirus: woot! Thanks guys, that fix makes my little unit test happy locally | 20:00 |
| rosmaita | dtroyer: excellent! thanks for checking, and thanks for working with us on this | 20:00 |
| dtroyer | should I clean up my pep8 issues and resubmit the unit tests? | 20:01 |
| rosmaita | stevelle: can you review https://review.openstack.org/#/c/423499/ ? | 20:03 |
| rosmaita | dtroyer: feel free, they would not hurt | 20:03 |
| stevelle | rosmaita: yeah, already on it | 20:03 |
| rosmaita | stevelle: ty ... if it LGTY, then i will give it an anticipatory +A | 20:04 |
| stevelle | rosmaita: want to watch the change or would you like a ping when I'm done? | 20:05 |
| rosmaita | a ping would be helpful | 20:05 |
| stevelle | rosmaita: +2 | 20:09 |
| rosmaita | stevelle: ty | 20:09 |
| openstackgerrit | Dean Troyer proposed openstack/glance: Add image update tests for is_public https://review.openstack.org/423345 | 20:12 |
| sigmavirus | dtroyer: yeah that would be great | 20:13 |
| marst | Hello. I'm having troubles with using "glance image-create --location http://blablabla" with corporate proxy. http_proxy is set in env and "wget -O http://blablabla" works fine. Error message: "The HTTP URL is invalid. _get_size /usr/lib/python2.7/site-packages/glance/api/v1/images.py:758" This is OS liberty and glance-api.conf has "stores=glance.store.http.Store,glance.store.rbd.Store" enabled." Any idea what could be wrong or what to check?? | 20:14 |
| sigmavirus | marst: I assume your glance is configured to allow use of locations? | 20:14 |
| *** esmiurium has quit IRC | 20:15 | |
| marst | sigmavirus: I'd guess so, how do I check? | 20:15 |
| sigmavirus | marst: check glance-api.conf for show_image_direct_url and show_multiple_locations | 20:17 |
| sigmavirus | one will have to be true (iirc) to allow you to set a location | 20:17 |
| sigmavirus | rosmaita: stevemar can you help marst? I have to hop on the VPN for a short bit | 20:19 |
| marst | sigmavirus: show_image_direct_url=True, while show_multiple_locations is commented out. I'll play with those parameters. Thanks a lot! | 20:20 |
| *** ChanServ changes topic to "OpenStack Glance" | 20:23 | |
| *** ChanServ changes topic to "OpenStack Glance | priorities: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110617.html" | 20:23 | |
| *** ChanServ changes topic to "OpenStack Glance | priorities: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110617.html | logs at http://eavesdrop.openstack.org/irclogs/%23openstack-glance/" | 20:23 | |
| *** esmiurium has joined #openstack-glance | 20:31 | |
| *** TravT_ has quit IRC | 20:33 | |
| sigmavirus | marst: could also be that the glance environment that the URL is unreachable | 20:38 |
| sigmavirus | also if glance is behind the same proxy, it needs to be able to detect that | 20:38 |
| sigmavirus | that could be why the URL is unreachable | 20:38 |
| sigmavirus | Glance the API will need to reach the URL and if the Proxy doesn't allow it that could be the reason | 20:39 |
| marst | sigmavirus: checked tcpdump on proxy server, no requests are coming from controller. wget with the same proxy works fine. I guess it's not a proxy problem, after all. | 20:42 |
| sigmavirus | marst: are you sure Glance is using the proxy though? | 20:43 |
| sigmavirus | you might need to turn on debug logging to confirm that | 20:43 |
| marst | sigmavirus: undefined "http_proxy" from env, and changed url to "http://glance_own_ip/test123". Now it says "HTTPBadRequest: 400 Bad Request: External sources are not supported:" | 20:44 |
| marst | sigmavirus: is "glance --debug" enough? | 20:45 |
| marst | or I need to change it glance-api.conf and restart the service? | 20:45 |
| sigmavirus | well that won't tell you if glance (the service) is using a proxy | 20:45 |
| sigmavirus | but --debug will give you info | 20:45 |
| sigmavirus | marst: putting that info into paste.openstack.org would be good | 20:45 |
| marst | I've unset both http_proxy and no_proxy variables. | 20:45 |
| marst | sigmavirus: http://paste.openstack.org/show/595838/ | 20:46 |
| marst | sigmavirus: thanks for the help, btw. Are you going to be in Atlanta? | 20:47 |
| *** david-lyle has joined #openstack-glance | 20:47 | |
| sigmavirus | marst: I won't be, no | 20:47 |
| sigmavirus | Also have to jet so I won't be around after this | 20:51 |
| sigmavirus | Have a good weekend y'all | 20:51 |
| sigmavirus | rosmaita: stevelle ^ marst could still use some help | 20:51 |
| marst | sigmavirus: it's a pity, I was hoping to buy you a beer. | 20:52 |
| marst | sigmavirus: it's still on me though. | 20:52 |
| sigmavirus | marst: no need. I don't drink =D | 20:52 |
| marst | sigmavirus. thanks a lot, anyway! | 20:52 |
| marst | I'll google around, it's sounds like something very simple error in my config | 20:53 |
| stevelle | stay shiny sigmavirus | 20:53 |
| *** thiagolib has quit IRC | 20:58 | |
| *** dalgaaf has quit IRC | 21:00 | |
| *** catinthe_ has quit IRC | 21:02 | |
| *** catintheroof has joined #openstack-glance | 21:03 | |
| marst | stevelle: if you have a minute, can you have a quick look on http://paste.openstack.org/show/595840/ ? Found old bug: https://bugs.launchpad.net/glance-store/+bug/1473009 , but I guess it doesn't apply in my case. | 21:03 |
| openstack | Launchpad bug 1473009 in glance_store "HTTP external sources are not supported" [Undecided,Invalid] | 21:03 |
| stevelle | looking now marst | 21:05 |
| *** catintheroof has quit IRC | 21:07 | |
| stevelle | fair warning: I'm not great with v1 api stuff since I have spent most of my time focused on v2, so I may not get this very fast | 21:10 |
| stevelle | marst: are you sure that you shouldn't have http in the stores list as indicated in the bug comments because that does seem to be straight forward reading | 21:16 |
| marst | stevelle: I have "stores=glance.store.http.Store,glance.store.rbd.Store", while bug description has: "stores = glance.store.filesystem.Store, glance.store.rbd.Store, glance.store.http.Store" | 21:21 |
| stevelle | marst: the suggestion in the comments there was that the issue was resolved by adding ", glance.store.http.Store" to the stores key | 21:21 |
| stevelle | this should allow the glance API to accept locations beginning with http | 21:22 |
| marst | stevelle: I'm not following. I already have http store defined in glance-api.conf: http://paste.openstack.org/show/595840/. Shall I change order? | 21:23 |
| stevelle | comment #4 "resolved it by adding glance.store.http.Store to the stores in glance-api.conf" | 21:23 |
| marst | stevelle: yes! | 21:24 |
| marst | stevelle: I was so happy when I found that solution. and a bit disappointed since I already have it defined in config file | 21:24 |
| stevelle | In this case order should not be the problem. But to be clear, http was already there before you started? | 21:25 |
| marst | stevelle: yes. | 21:27 |
| stevemar | rosmaita: stevelle looks like https://review.openstack.org/#/c/423499/ isn't working as expected :( | 21:27 |
| stevelle | marst: if you do turn on debug logging and restart the service, I'd want you to look for lines featuring "Registering scheme" | 21:27 |
| marst | stevelle: will do. thank you! | 21:28 |
| stevelle | might just check to see what shows up at the last restart if debug was already on | 21:28 |
| *** jamielennox is now known as jamielennox|away | 21:28 | |
| *** mfedosin has quit IRC | 21:29 | |
| *** smatzek has quit IRC | 21:31 | |
| *** flwang1 has joined #openstack-glance | 21:41 | |
| *** david-lyle has quit IRC | 21:45 | |
| stevelle | stevemar: we might have spoken too soon on the community images revert. | 21:46 |
| stevelle | rosmaita: ^ when you're back | 21:46 |
| dharinic | shouldnt have these been caught before +w? | 21:47 |
| dharinic | by jenkins | 21:48 |
| dharinic | or did we not wait for jenkins? | 21:48 |
| dharinic | stevelle^ | 21:48 |
| stevelle | we didn't wait. I ran local suite with and without CI before zuul finished first pass | 21:48 |
| *** _ducttape_ has joined #openstack-glance | 21:48 | |
| dharinic | py27 failed locally for me on that patch. I was actually waiting on Jenkins before +1 ing it | 21:49 |
| *** _ducttape_ has quit IRC | 21:49 | |
| dharinic | okayy | 21:49 |
| stevelle | I probably borked my git history when setting up the test with CI | 21:50 |
| stevelle | that's my best guess | 21:50 |
| *** _ducttape_ has joined #openstack-glance | 21:50 | |
| *** ducttape_ has quit IRC | 21:51 | |
| stevelle | at least I can confirm - Failed: 26 | 21:54 |
| *** dalgaaf has joined #openstack-glance | 21:57 | |
| dharinic | Yes stevelle. I saw that before that patch was +w'd | 21:58 |
| stevelle | doh | 21:58 |
| dharinic | I suspected some issues with my env cos i saw that it was already approved | 21:58 |
| dharinic | hence did not bother to tell it on the channel | 21:58 |
| *** sabari has quit IRC | 22:00 | |
| stevelle | that we don't or can't trust our own envs is probably something to look at later :) | 22:05 |
| stevelle | dharinic: are you digging into the failure too? | 22:10 |
| *** dalgaaf has quit IRC | 22:11 | |
| dharinic | yes stevelle. Not much progress though | 22:11 |
| stevelle | dharinic: so all of the failures seem to be related to functional/test_cache_middleware.py in the process of creating images, so the change fixed updates but breaks a create scenario. | 22:12 |
| stevelle | or at least a sizable portion of them seem to point there | 22:12 |
| dharinic | yeah thats true stevelle | 22:13 |
| stevelle | dharinic: and db/simple/api.py always applies ensure_image_dict_v2_compliant, whereas db/sqlalchemy/api.py conditionally applies it | 22:13 |
| stevelle | that has me suspicious of a logic error, but that's what I have right now | 22:14 |
| dharinic | whats intersting the tests in the same test file of that patch glance.tests.functional.v1.test_api.TestApi. | 22:14 |
| dharinic | yeah. The call that goes form db/simple/api.py for the update methods need to be looked at | 22:14 |
| dharinic | like a similar situtation as in the sqlalchemy/api.py | 22:15 |
| stevelle | marst: yt? | 22:17 |
| marst | stevelle: yes! | 22:18 |
| marst | stevelle: I'm a certified idiot. | 22:18 |
| marst | stevelle: copy-pasted url. it was htttp, instead of http. | 22:18 |
| stevelle | wait, does that mean you fixed it? | 22:18 |
| stevelle | win! | 22:18 |
| stevelle | I was just going to ask you b/c your original comment said: External sources are not supported:" | 22:19 |
| marst | stevelle: sorry for wasting your time | 22:19 |
| stevelle | you didn't paste the actual protocol that wasn't supported | 22:19 |
| marst | I've spent 2 days fighting this. :( | 22:19 |
| stevelle | marst: sorry you wasted your time :(( | 22:19 |
| marst | stevelle: it's ok though. learned quite a bit about glance in the process. :) | 22:20 |
| stevelle | let's call that a win | 22:20 |
| marst | stevelle: thanks a lot! :) | 22:20 |
| marst | stevelle: you have a great weekend! | 22:20 |
| stevelle | cheers | 22:20 |
| rosmaita | so, looks like there's still work to do on CI | 22:22 |
| *** burgerk has quit IRC | 22:25 | |
| stevelle | it shouldn't take much, but yes. the CI patch we merged did have a couple hidden problems. Glad we are finding them now though | 22:32 |
| openstackgerrit | Steve Lewis (stevelle) proposed openstack/glance: Fix regression introduced by Community Images https://review.openstack.org/423499 | 22:37 |
| *** dalgaaf has joined #openstack-glance | 22:39 | |
| *** dalgaaf has quit IRC | 22:43 | |
| *** dalgaaf has joined #openstack-glance | 22:43 | |
| openstackgerrit | Steve Lewis (stevelle) proposed openstack/glance: Add image update tests for is_public https://review.openstack.org/423345 | 22:44 |
| *** wjhofer has quit IRC | 23:01 | |
| *** _ducttape_ has quit IRC | 23:22 | |
| *** kebray has quit IRC | 23:30 | |
| openstackgerrit | Hemanth Makkapati proposed openstack/glance: Port Glance Migrations to Alembic https://review.openstack.org/382958 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!