| *** dims_ has joined #openstack-containers | 00:10 | |
| *** dims has quit IRC | 00:11 | |
| *** hongbin has quit IRC | 00:14 | |
| *** ramishra has joined #openstack-containers | 00:36 | |
| *** sdake has quit IRC | 00:50 | |
| *** sdake has joined #openstack-containers | 00:52 | |
| *** absubram has joined #openstack-containers | 00:55 | |
| *** jperry has quit IRC | 01:07 | |
| *** Syed__ has quit IRC | 01:25 | |
| openstackgerrit | Vijendar Komalla proposed openstack/magnum: Set is_admin flag correctly in RequestContext https://review.openstack.org/427507 | 01:44 |
|---|---|---|
| *** tonanhngo has quit IRC | 01:47 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 01:50 | |
| *** ramishra has quit IRC | 01:53 | |
| *** tonanhngo has joined #openstack-containers | 02:13 | |
| *** tonanhngo has quit IRC | 02:17 | |
| *** yuanying has quit IRC | 02:21 | |
| openstackgerrit | Shu Muto proposed openstack/magnum-ui: Add update action for cluster https://review.openstack.org/424984 | 02:46 |
| openstackgerrit | Merged openstack/magnum-ui: Change cluster_template creating action for using json schema https://review.openstack.org/409107 | 03:01 |
| openstackgerrit | Merged openstack/magnum-ui: Add update action for cluster template https://review.openstack.org/422477 | 03:01 |
| openstackgerrit | Merged openstack/magnum-ui: Change cluster creating action for using json schema https://review.openstack.org/424536 | 03:04 |
| openstackgerrit | Shu Muto proposed openstack/magnum-ui: Add release notes for update actions https://review.openstack.org/425002 | 03:04 |
| openstackgerrit | Merged openstack/magnum-ui: Add update action for cluster https://review.openstack.org/424984 | 03:04 |
| openstackgerrit | Merged openstack/magnum-ui: Add release notes for update actions https://review.openstack.org/425002 | 03:27 |
| *** vimal has joined #openstack-containers | 03:35 | |
| *** mtanino has quit IRC | 03:43 | |
| *** ricolin has joined #openstack-containers | 03:49 | |
| *** absubram has quit IRC | 03:50 | |
| *** sdake has quit IRC | 03:53 | |
| *** vimal has quit IRC | 04:01 | |
| openstackgerrit | Shu Muto proposed openstack/magnum-ui: Fix typo https://review.openstack.org/427546 | 04:04 |
| *** mdnadeem has joined #openstack-containers | 04:10 | |
| *** zenirc369 has joined #openstack-containers | 04:18 | |
| *** vimal has joined #openstack-containers | 04:30 | |
| *** sdake has joined #openstack-containers | 04:45 | |
| *** adrian_otto has joined #openstack-containers | 04:56 | |
| *** janki has joined #openstack-containers | 05:01 | |
| *** sdake_ has joined #openstack-containers | 05:05 | |
| *** sdake has quit IRC | 05:05 | |
| *** dave-mccowan has quit IRC | 05:18 | |
| *** tonanhngo has joined #openstack-containers | 05:22 | |
| *** dave-mccowan has joined #openstack-containers | 05:22 | |
| *** mkrai has joined #openstack-containers | 05:22 | |
| *** mkrai_ has joined #openstack-containers | 05:44 | |
| *** ricolin has quit IRC | 05:54 | |
| *** dave-mcc_ has joined #openstack-containers | 05:59 | |
| *** dave-mccowan has quit IRC | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/magnum-ui: Imported Translations from Zanata https://review.openstack.org/427559 | 06:12 |
| *** zenirc_ has joined #openstack-containers | 06:15 | |
| *** sdake_ is now known as sdake | 06:16 | |
| *** adrian_otto has quit IRC | 06:16 | |
| *** yolanda has quit IRC | 06:23 | |
| openstackgerrit | Merged openstack/magnum-ui: Imported Translations from Zanata https://review.openstack.org/427559 | 06:24 |
| *** hishh has joined #openstack-containers | 06:26 | |
| *** rcernin has joined #openstack-containers | 06:29 | |
| *** hishh1 has joined #openstack-containers | 06:32 | |
| *** hishh has quit IRC | 06:32 | |
| *** mjura has joined #openstack-containers | 06:33 | |
| *** hishh1 is now known as hishh | 06:35 | |
| *** hishh has joined #openstack-containers | 06:38 | |
| *** absubram has joined #openstack-containers | 06:49 | |
| *** absubram has quit IRC | 06:54 | |
| *** absubram has joined #openstack-containers | 07:02 | |
| *** Oku_OS-away is now known as Oku_OS | 07:18 | |
| *** pcaruana has joined #openstack-containers | 07:19 | |
| *** vijaykc4 has joined #openstack-containers | 07:21 | |
| *** yamamoto has quit IRC | 07:25 | |
| *** xek__ is now known as xek | 07:31 | |
| *** vijaykc4 has quit IRC | 07:34 | |
| *** tonanhngo has quit IRC | 07:40 | |
| *** hishh has quit IRC | 07:42 | |
| *** adisky_ has joined #openstack-containers | 07:54 | |
| openstackgerrit | Merged openstack/magnum-ui: Fix typo https://review.openstack.org/427546 | 08:00 |
| *** yolanda has joined #openstack-containers | 08:04 | |
| *** vijaykc4 has joined #openstack-containers | 08:04 | |
| *** slagle has quit IRC | 08:05 | |
| *** hishh has joined #openstack-containers | 08:07 | |
| *** vijaykc4 has quit IRC | 08:14 | |
| *** vijaykc4 has joined #openstack-containers | 08:15 | |
| openstackgerrit | Kevin Lefevre proposed openstack/magnum: Use heat-params in systemd unit https://review.openstack.org/425275 | 08:18 |
| openstackgerrit | Kevin Lefevre proposed openstack/magnum: Fix missing $ in CoreOS proxy conf https://review.openstack.org/427586 | 08:21 |
| *** dave-mcc_ has quit IRC | 08:24 | |
| *** pcastets has quit IRC | 08:27 | |
| *** yuanying has joined #openstack-containers | 08:30 | |
| *** pcastets has joined #openstack-containers | 08:35 | |
| *** jchhatbar has joined #openstack-containers | 08:46 | |
| *** janki has quit IRC | 08:48 | |
| *** vijaykc4 has quit IRC | 08:49 | |
| openstackgerrit | Cristovao Cordeiro proposed openstack/magnum: [swarm] Monitoring with Prometheus and Grafana https://review.openstack.org/407581 | 08:58 |
| *** absubram has quit IRC | 09:04 | |
| openstackgerrit | Kevin Lefevre proposed openstack/magnum: use variables for hyperkube et kube version https://review.openstack.org/427227 | 09:07 |
| *** mdnadeem has quit IRC | 09:24 | |
| *** mdnadeem has joined #openstack-containers | 09:24 | |
| *** mvelten has joined #openstack-containers | 09:27 | |
| *** vijaykc4 has joined #openstack-containers | 09:36 | |
| *** ricolin has joined #openstack-containers | 09:41 | |
| openstackgerrit | Kevin Lefevre proposed openstack/magnum: Improve consistency for SSL PATH accross template https://review.openstack.org/427608 | 09:42 |
| *** prameswar has joined #openstack-containers | 09:48 | |
| *** salmankhan has joined #openstack-containers | 09:50 | |
| *** xiangxinyong has quit IRC | 09:51 | |
| *** xiangxinyong has joined #openstack-containers | 09:52 | |
| openstackgerrit | Kevin Lefevre proposed openstack/magnum: Remove $myip when unnecessary and use KUBE_NODE_IP https://review.openstack.org/427614 | 09:52 |
| openstackgerrit | Kevin Lefevre proposed openstack/magnum: Improve consistency for SSL PATH accross template https://review.openstack.org/427608 | 10:08 |
| *** pcastets has quit IRC | 10:31 | |
| openstackgerrit | Cristovao Cordeiro proposed openstack/magnum: [swarm] Monitoring with Prometheus and Grafana https://review.openstack.org/407581 | 10:36 |
| openstackgerrit | Johannes Grassler proposed openstack/magnum: Fix gate: caused by tempest(removal of "service" param) https://review.openstack.org/427630 | 10:39 |
| *** salmankhan has quit IRC | 10:40 | |
| *** chhavi has joined #openstack-containers | 10:41 | |
| *** mkrai_ has quit IRC | 10:45 | |
| *** salmankhan has joined #openstack-containers | 10:58 | |
| *** salmankhan has quit IRC | 11:05 | |
| *** salmankhan has joined #openstack-containers | 11:05 | |
| *** pcastets has joined #openstack-containers | 11:08 | |
| *** mdnadeem has quit IRC | 11:36 | |
| *** mdnadeem has joined #openstack-containers | 11:42 | |
| *** chhavi_ has joined #openstack-containers | 11:49 | |
| *** chhavi has quit IRC | 11:51 | |
| *** ramishra has joined #openstack-containers | 11:56 | |
| *** salmankhan has quit IRC | 11:59 | |
| *** salmankhan has joined #openstack-containers | 11:59 | |
| *** vijaykc4 has quit IRC | 12:01 | |
| *** v1k0d3n has quit IRC | 12:02 | |
| *** danil has joined #openstack-containers | 12:05 | |
| *** sgordon has joined #openstack-containers | 12:06 | |
| *** shu-mutou is now known as shu-mutou-AWAY | 12:12 | |
| *** chhavi_ has quit IRC | 12:24 | |
| *** catintheroof has joined #openstack-containers | 12:28 | |
| *** prameswar has quit IRC | 12:29 | |
| *** slagle has joined #openstack-containers | 12:29 | |
| *** jchhatbar_ has joined #openstack-containers | 12:31 | |
| *** chhavi_ has joined #openstack-containers | 12:33 | |
| *** jchhatbar has quit IRC | 12:34 | |
| *** vimal has quit IRC | 12:45 | |
| *** mtanino has joined #openstack-containers | 12:56 | |
| *** zenirc369 has quit IRC | 13:02 | |
| *** zenirc_ has quit IRC | 13:02 | |
| *** salmankhan has quit IRC | 13:06 | |
| *** bapalm has quit IRC | 13:07 | |
| *** afranc has quit IRC | 13:09 | |
| *** afranc has joined #openstack-containers | 13:09 | |
| *** mordred has quit IRC | 13:10 | |
| *** bapalm has joined #openstack-containers | 13:12 | |
| *** mtanino has quit IRC | 13:13 | |
| *** yuanying_ has joined #openstack-containers | 13:14 | |
| *** vijaykc4 has joined #openstack-containers | 13:15 | |
| *** askb has quit IRC | 13:18 | |
| *** mordred has joined #openstack-containers | 13:18 | |
| *** vijaykc4 has quit IRC | 13:21 | |
| *** bapalm has quit IRC | 13:24 | |
| *** vijaykc4 has joined #openstack-containers | 13:35 | |
| *** vijaykc4 has quit IRC | 13:36 | |
| *** salmankhan has joined #openstack-containers | 13:46 | |
| *** prameswar has joined #openstack-containers | 13:46 | |
| *** v1k0d3n has joined #openstack-containers | 13:48 | |
| *** danil has quit IRC | 13:55 | |
| *** vijaykc4 has joined #openstack-containers | 13:57 | |
| *** vijaykc4 has quit IRC | 13:58 | |
| *** vijaykc4 has joined #openstack-containers | 14:02 | |
| *** vijaykc4 has quit IRC | 14:02 | |
| *** salmankhan has quit IRC | 14:03 | |
| *** niv_ has joined #openstack-containers | 14:07 | |
| niv_ | hey , im trying to install magnum on openstack newton on ubuntu xenial | 14:08 |
| niv_ | here are missing files in /usr/lib/python2.7/dist-packages/magnum/drivers i copied it manually from github magnum. | 14:08 |
| niv_ | i got the error: ERROR: Failed to validate : resources.network : The Resource Type (Magnum::Optional::Neutron::Net) could not be found. | 14:09 |
| niv_ | Does magnum support xenial and openstack newton? | 14:09 |
| openstackgerrit | Michal Jura proposed openstack/magnum: Add wsgi script to deploy Magnum behind Apache https://review.openstack.org/427723 | 14:14 |
| *** vijaykc4 has joined #openstack-containers | 14:20 | |
| *** vijaykc4 has quit IRC | 14:22 | |
| *** PagliaccisCloud has joined #openstack-containers | 14:37 | |
| *** yuanying_ has quit IRC | 14:37 | |
| *** yuanying_ has joined #openstack-containers | 14:37 | |
| *** prameswar has quit IRC | 14:39 | |
| *** vijaykc4 has joined #openstack-containers | 14:39 | |
| *** vijaykc4 has quit IRC | 14:39 | |
| *** yuanying_ has quit IRC | 14:42 | |
| *** dave-mccowan has joined #openstack-containers | 14:46 | |
| *** vijaykc4 has joined #openstack-containers | 14:47 | |
| *** vijaykc4 has quit IRC | 14:48 | |
| *** zenirc369 has joined #openstack-containers | 14:51 | |
| *** zenirc_ has joined #openstack-containers | 14:52 | |
| *** jasond has joined #openstack-containers | 14:58 | |
| *** zz_dimtruck is now known as dimtruck | 14:59 | |
| *** adrian_otto has joined #openstack-containers | 15:02 | |
| *** salmankhan has joined #openstack-containers | 15:05 | |
| *** salmankhan has quit IRC | 15:05 | |
| *** EricGonczer_ has joined #openstack-containers | 15:05 | |
| *** salmankhan has joined #openstack-containers | 15:07 | |
| *** adrian_otto has quit IRC | 15:08 | |
| *** jperry has joined #openstack-containers | 15:11 | |
| *** vijaykc4 has joined #openstack-containers | 15:11 | |
| openstackgerrit | Spyros Trigazis proposed openstack/magnum: WIP Copy swarm_v1 to contrib/swarm_v2 https://review.openstack.org/427761 | 15:12 |
| apuimedo | gus: I would like to discuss about adding kuryr support to kubernetes/kubernetes openstack cloudprovider for supporting the loadbalancer service type | 15:14 |
| apuimedo | please ping me when you are around | 15:14 |
| apuimedo | I was thinking that maybe we could make the cloud provider configurable | 15:14 |
| apuimedo | to choose between the current operation | 15:14 |
| apuimedo | and kuryr networking | 15:15 |
| *** salmankhan has quit IRC | 15:16 | |
| *** adrian_otto has joined #openstack-containers | 15:16 | |
| *** vijaykc4 has quit IRC | 15:17 | |
| *** hongbin has joined #openstack-containers | 15:19 | |
| *** chhavi_ has quit IRC | 15:20 | |
| *** salmankhan has joined #openstack-containers | 15:20 | |
| *** vijaykc4 has joined #openstack-containers | 15:24 | |
| *** vijaykc4 has quit IRC | 15:24 | |
| *** adrian_otto1 has joined #openstack-containers | 15:27 | |
| *** adrian_otto1 has quit IRC | 15:28 | |
| *** adrian_otto has quit IRC | 15:28 | |
| *** niv_ has quit IRC | 15:29 | |
| *** adrian_otto has joined #openstack-containers | 15:31 | |
| *** mdnadeem has quit IRC | 15:33 | |
| *** chhavi_ has joined #openstack-containers | 15:33 | |
| *** adrian_otto has quit IRC | 15:33 | |
| *** chris_hultin|AWA is now known as chris_hultin | 15:33 | |
| *** EricGonczer_ has quit IRC | 15:34 | |
| *** vijaykc4 has joined #openstack-containers | 15:40 | |
| *** jchhatbar_ has quit IRC | 15:43 | |
| *** vijaykc4 has quit IRC | 15:48 | |
| *** rcernin has quit IRC | 15:51 | |
| *** mtanino has joined #openstack-containers | 15:54 | |
| *** vijaykc4 has joined #openstack-containers | 15:56 | |
| *** EricGonczer_ has joined #openstack-containers | 15:58 | |
| *** randallburt has joined #openstack-containers | 15:59 | |
| *** vijaykc4 has quit IRC | 16:03 | |
| *** randallburt has quit IRC | 16:04 | |
| *** adrian_otto has joined #openstack-containers | 16:04 | |
| *** ramishra has quit IRC | 16:06 | |
| *** ramishra has joined #openstack-containers | 16:08 | |
| *** vijaykc4 has joined #openstack-containers | 16:09 | |
| *** vijaykc4 has quit IRC | 16:10 | |
| jasond | jvgrant: ping | 16:11 |
| jvgrant | jasond: hi | 16:11 |
| jasond | jvgrant: hey, i saw in the git history that you worked on some of the microversioning code | 16:12 |
| jvgrant | yeah, i work on quite a bit of it | 16:12 |
| jasond | jvgrant: can you look at https://bugs.launchpad.net/magnum/+bug/1659431 and confirm that this is unexpected behavior? | 16:12 |
| openstack | Launchpad bug 1659431 in Magnum "Unexpected response from microversioned API" [Medium,Triaged] - Assigned to Jason Dunsmore (jasondunsmore) | 16:12 |
| jvgrant | jasond: yeah, i'll take a look at it | 16:13 |
| jasond | jvgrant: thanks | 16:13 |
| jvgrant | jasond: so the new api certificates doesn't have the microversion decorator on it? | 16:15 |
| jasond | jvgrant: it does | 16:15 |
| jasond | https://github.com/openstack/magnum/blob/master/magnum/api/controllers/v1/certificate.py#L170 | 16:16 |
| *** randallburt has joined #openstack-containers | 16:16 | |
| jasond | jvgrant: but the other certificates apis were affected | 16:17 |
| jasond | jvgrant: now they require the "OpenStack-API-Version: container-infra latest" header to work | 16:18 |
| *** vijaykc4 has joined #openstack-containers | 16:19 | |
| *** hieulq has joined #openstack-containers | 16:19 | |
| jvgrant | jasond: yeah, think there might be a bug in how the version selector works when only part has microversions... thinking it through | 16:19 |
| *** vijaykc4 has quit IRC | 16:19 | |
| *** Drago1 has joined #openstack-containers | 16:20 | |
| jasond | jvgrant: that's what i thought, but when i added @base.Controller.api_version("1.1") to the other methods, i see the same issue | 16:21 |
| *** vijaykc4 has joined #openstack-containers | 16:21 | |
| *** adrian_otto has quit IRC | 16:22 | |
| *** vijaykc4 has quit IRC | 16:22 | |
| jasond | jvgrant: even with @base.Controller.api_version("1.1", "1.5") | 16:23 |
| jvgrant | jasond: that is strange. I'm going to try it out and walk through it | 16:23 |
| *** zenirc_ has quit IRC | 16:24 | |
| *** zenirc369 has quit IRC | 16:24 | |
| jasond | jvgrant: ok, let me know what you find | 16:24 |
| *** Syed__ has joined #openstack-containers | 16:27 | |
| jasond | jvgrant: not sure how this isn't a problem for magnum.api.controllers.v1.cluster.ClustersController | 16:28 |
| *** vijaykc4 has joined #openstack-containers | 16:28 | |
| jvgrant | jasond: seems like we would see it everywhere, so something different must be going on | 16:29 |
| *** vijaykc4 has quit IRC | 16:29 | |
| *** absubram has joined #openstack-containers | 16:31 | |
| jvgrant | jasond: so patch works correctly? fails on microversions below 1.5? | 16:32 |
| jasond | jvgrant: sorry, one minute, in a meeting | 16:34 |
| openstackgerrit | Johannes Grassler proposed openstack/python-magnumclient: Add osc-lib to requirements.txt https://review.openstack.org/427801 | 16:35 |
| *** absubram has quit IRC | 16:36 | |
| *** tonanhngo has joined #openstack-containers | 16:37 | |
| jasond | jvgrant: yep, patch fails with version 1.4 | 16:37 |
| jasond | with a 406 error | 16:38 |
| jvgrant | jvgrant: i'm wondering if it is failing at a different point. The 406 error in the bug report doesn't have the correct error message info. Does the patch fail have the correct version message saying it is not supported? | 16:41 |
| *** tonanhngo has quit IRC | 16:42 | |
| *** vijaykc4 has joined #openstack-containers | 16:44 | |
| jasond | jvgrant: it does "Version 1.4 was requested but the requested API patch is not supported for this version." | 16:44 |
| jvgrant | jasond: that is the message i'd expect | 16:45 |
| *** chhavi_ has quit IRC | 16:45 | |
| jvgrant | jasond: when GET fails we don't get any message with the 406? | 16:45 |
| *** hishh has quit IRC | 16:46 | |
| jasond | jvgrant: trying to test that now | 16:48 |
| openstackgerrit | Johannes Grassler proposed openstack/python-magnumclient: Add osc-lib to requirements.txt https://review.openstack.org/427801 | 16:48 |
| *** vijaykc4 has quit IRC | 16:52 | |
| jasond | jvgrant: it does have the version in the message http://paste.openstack.org/show/597236/ | 16:57 |
| jvgrant | jasond: thanks that helps | 16:57 |
| jasond | jvgrant: (you have to revert fbfdbec60d4 to reproduce that) | 16:57 |
| jasond | ..since a GET work every time, regardless of version, after the first request that contains the "latest" version | 16:58 |
| jasond | jvgrant: ^ which is another weird thing | 16:58 |
| jvgrant | jasond: that is the strange part | 16:59 |
| *** chhavi_ has joined #openstack-containers | 16:59 | |
| jasond | jvgrant: that happens on a per-cluster basis | 17:00 |
| *** yolanda has quit IRC | 17:01 | |
| jvgrant | the decorator just makes a dictionary of any calls that have a microversion, and when the call is made if 1) call is in dictionary 2) dictionary exists then it will check for the correct version | 17:02 |
| jvgrant | jasond: big hint in that message.... Look at the requested API in the message | 17:03 |
| jvgrant | jasond: it thinks it is patch for some reason | 17:03 |
| jvgrant | jasond: not get | 17:03 |
| jasond | jvgrant: ah ha, didn't see that | 17:03 |
| jvgrant | jasond: it is very strange, i can see why it would hit that message if patch was called but i can't see any reason why patch would show up as the call on the get | 17:08 |
| jasond | jvgrant: it is. i noticed if i restart the magnum-api, i can reproduce the issue on the same cluster again | 17:12 |
| jvgrant | jasond: hmmm... does patch get called during the get for some reason the first time? that would makes sense as to why once it passes from setting it to latest you wouldn't hit it again | 17:16 |
| jasond | jvgrant: no, it doesn't | 17:17 |
| *** chhavi_ has quit IRC | 17:17 | |
| *** vijaykc4 has joined #openstack-containers | 17:18 | |
| jvgrant | jasond: hmmm... really strange. it makes sense though that once it gets past the patch call with the latest it doesn't have any issues again | 17:19 |
| jasond | jvgrant: why does that make sense? because of that dictionary? | 17:21 |
| jasond | jvgrant: ok, found the code | 17:22 |
| *** hieulq has quit IRC | 17:22 | |
| jvgrant | jasond: because only patch in the dictionary needs 1.5 and the error message is hit on "patch" not "get". So it is never the "get" version check that fails | 17:22 |
| *** hieulq has joined #openstack-containers | 17:22 | |
| jvgrant | jasond: but it seems like once it calls patch that one time it doesn't do it again. | 17:23 |
| *** EricGonczer_ has quit IRC | 17:23 | |
| jvgrant | jasond: so if you pass the latest version and it passes, you never hit patch on any other call | 17:23 |
| jasond | jvgrant: oh ok. i guess it's just a matter of finding out who is passing the wrong method now | 17:24 |
| *** hieulq has quit IRC | 17:26 | |
| jvgrant | jasond: reproduced it now, walking through the code | 17:29 |
| *** tonanhngo has joined #openstack-containers | 17:31 | |
| *** vijaykc4 has quit IRC | 17:32 | |
| *** vijaykc4 has joined #openstack-containers | 17:33 | |
| *** tonanhngo has quit IRC | 17:36 | |
| jvgrant | jasond: it looks like it is something pecan is doing. it is inspecting each attribute | 17:36 |
| jasond | jvgrant: this is kind of interesting http://paste.openstack.org/show/597253/ | 17:36 |
| jasond | jvgrant: that was for a single GET call | 17:36 |
| jvgrant | jasond: that is expected. you are printing on all get_attribute calls and just showing what is in dict | 17:39 |
| jvgrant | jasond: that looks correct | 17:39 |
| jasond | jvgrant: ah ok | 17:40 |
| jvgrant | jasond: more interesting is to print key when the "if version_method_dict adn key in version_meth_dict" is true | 17:40 |
| *** vijaykc4 has quit IRC | 17:40 | |
| jvgrant | jasond: what i'm seeing is that on the first call to the certificate controller pecan is getting all the attributes, including patch. When it does it fails on version check | 17:41 |
| jvgrant | jasond: but if you make that call with the latest version, then it passes and pecan doesn't get all the attributes again | 17:41 |
| jasond | jvgrant: i see | 17:42 |
| jasond | jvgrant: tried what you suggested and i only get one debug output on a GET call "key: patch" | 17:43 |
| jvgrant | jasond: that is what i'm seeing. That is the error. Looking at the trace it shows that patch was called by pecan | 17:44 |
| jvgrant | jasond: still not sure why pecan wants to call patch | 17:45 |
| jvgrant | jasond: it seems like it is for some initial setup of the pecan controller | 17:45 |
| jasond | jvgrant: or why it doesn't do that on GET /clusters | 17:45 |
| jvgrant | jasond: right... we should see this issue on clusters as well. | 17:46 |
| *** dimtruck is now known as zz_dimtruck | 17:47 | |
| jasond | jvgrant: try doing a getmembers() on the magnum.api.controllers.v1.certificate.CertificateController object | 17:54 |
| jvgrant | jasond: yep, it is setting up the controller object | 17:54 |
| jasond | (Pdb++) getmembers(obj) | 17:55 |
| jasond | *** HTTPNotAcceptable: Version 1.1 was requested but the requested API patch is not supported for this version. | 17:55 |
| jvgrant | jasond: the get_attribute can't tell it is not the patch call, so it checks the version which is 1.1 and fails | 17:55 |
| jvgrant | jasond: once the pecan controller is created(by using 1.5+) then it never calls that again and everything is working fine | 17:55 |
| jasond | jvgrant: is there any way to avoid overriding __getattribute__ in the pecan controller? | 17:56 |
| jvgrant | jasond: you should be able to reproduce this on other controllers as well | 17:56 |
| *** v1k0d3n has quit IRC | 17:56 | |
| jvgrant | jasond: that is what i'm looking into. maybe the version can be set to latest until after the controllers are created. Then set to actual for the calls | 17:57 |
| jasond | jvgrant: ok | 17:57 |
| *** v1k0d3n has joined #openstack-containers | 18:00 | |
| *** sdake_ has joined #openstack-containers | 18:01 | |
| *** prameswar has joined #openstack-containers | 18:02 | |
| *** sdake__ has joined #openstack-containers | 18:04 | |
| jvgrant | jasond: i know why you can't reproduce in other controllers | 18:04 |
| jasond | jvgrant: ? | 18:04 |
| *** ricolin has quit IRC | 18:04 | |
| jvgrant | jasond: it is because the other APIs with microversions are not new | 18:04 |
| *** sdake has quit IRC | 18:04 | |
| *** ricolin has joined #openstack-containers | 18:04 | |
| jvgrant | jasond: they have valid entries for all versions, they are different but there is an entry for all | 18:05 |
| jvgrant | jasond: certificate/patch has nothing for 1.1 - 1.4 so it hits this error | 18:05 |
| jasond | jvgrant: i see | 18:05 |
| *** harlowja_ has joined #openstack-containers | 18:05 | |
| *** sdake_ has quit IRC | 18:06 | |
| *** harlowja has quit IRC | 18:06 | |
| jvgrant | jasond: a quick fix to prove this would be to add another patch method with (1.1, 1.4). It can be empty because it will never be hit | 18:06 |
| jasond | jvgrant: trying it now | 18:07 |
| jasond | jvgrant: yep, that fixed it | 18:08 |
| jvgrant | jasond: this was something i didn't think about when i did this. micro-versions were for differences between versions. A new API really didn't need to separate it self to start at 1.x | 18:08 |
| jasond | jvgrant: yeah, understandable | 18:09 |
| jvgrant | jasond: hmmmm... not sure what to do about that. i can look into a pecan fix, but i'm questioning if a new API needs to have a newer microversion | 18:11 |
| *** ricolin has quit IRC | 18:11 | |
| jasond | jvgrant: makes sense to me. i'll read up on the openstack microversioning guidelines | 18:12 |
| jvgrant | jasond: new APIs are supported as valid in the major X of X.Y. the Y is more for differences within a the major X version | 18:12 |
| jvgrant | jasond: i think I can probably figure out a way to fix it if we really want though | 18:12 |
| jasond | jvgrant: ok, i will make a patch to remove the microversion. thanks a lot for your help with this! | 18:13 |
| jvgrant | jasond: for now my suggestion would be remove the microversion from patch and allow it to be supported for the major version | 18:13 |
| jasond | jvgrant: yeah, an error to prevent the api from starting up would be nice | 18:14 |
| jvgrant | jasond: thanks for bringing it up and looking into it. It was a trickly little bug :) | 18:14 |
| jasond | jvgrant: np | 18:14 |
| *** zz_dimtruck is now known as dimtruck | 18:15 | |
| *** tonanhngo has joined #openstack-containers | 18:20 | |
| *** tonanhngo has quit IRC | 18:23 | |
| *** tonanhngo has joined #openstack-containers | 18:23 | |
| *** pcaruana has quit IRC | 18:28 | |
| *** prameswar has quit IRC | 18:35 | |
| *** adrian_otto has joined #openstack-containers | 18:38 | |
| *** yolanda has joined #openstack-containers | 18:39 | |
| *** adrian_otto has quit IRC | 18:45 | |
| *** EricGonczer_ has joined #openstack-containers | 18:47 | |
| *** mjura has quit IRC | 18:58 | |
| *** yuanying has quit IRC | 19:03 | |
| *** sdake__ is now known as sdkae | 19:03 | |
| *** sdkae is now known as sdake | 19:04 | |
| *** adisky_ has quit IRC | 19:09 | |
| *** harlowja_ has quit IRC | 19:19 | |
| *** salmankhan has quit IRC | 19:20 | |
| *** adrian_otto has joined #openstack-containers | 19:31 | |
| *** salmankhan has joined #openstack-containers | 19:39 | |
| *** adrian_otto has quit IRC | 19:43 | |
| *** yuanying has joined #openstack-containers | 19:49 | |
| *** mtanino has quit IRC | 19:51 | |
| *** yuanying has quit IRC | 19:54 | |
| *** EricGonczer_ has quit IRC | 19:58 | |
| *** mtanino has joined #openstack-containers | 19:59 | |
| *** EricGonczer_ has joined #openstack-containers | 20:03 | |
| openstackgerrit | Jason Dunsmore proposed openstack/magnum: Don't create clusters of an unsupported type https://review.openstack.org/427370 | 20:05 |
| *** salmankhan1 has joined #openstack-containers | 20:07 | |
| *** salmankhan has quit IRC | 20:08 | |
| *** salmankhan1 is now known as salmankhan | 20:08 | |
| *** mtanino has quit IRC | 20:09 | |
| *** mtanino has joined #openstack-containers | 20:12 | |
| *** salmankhan has quit IRC | 20:14 | |
| *** salmankhan has joined #openstack-containers | 20:16 | |
| *** salmankhan has quit IRC | 20:20 | |
| *** salmankhan has joined #openstack-containers | 20:22 | |
| *** salmankhan has quit IRC | 20:29 | |
| openstackgerrit | Jason Dunsmore proposed openstack/magnum: Revert API microversion bump to 1.5 https://review.openstack.org/427907 | 20:34 |
| *** salmankhan has joined #openstack-containers | 20:35 | |
| *** harlowja has joined #openstack-containers | 20:39 | |
| *** salmankhan has quit IRC | 20:40 | |
| *** OGtrilliams has joined #openstack-containers | 20:51 | |
| *** PagliaccisCloud has quit IRC | 20:52 | |
| *** randallburt has quit IRC | 20:59 | |
| *** pcastets has quit IRC | 21:01 | |
| *** Syed__ has quit IRC | 21:05 | |
| *** askb has joined #openstack-containers | 21:09 | |
| *** jasond has quit IRC | 21:25 | |
| *** adrian_otto has joined #openstack-containers | 21:29 | |
| *** EricGonczer_ has quit IRC | 21:34 | |
| *** Jeffrey4l_ has quit IRC | 21:35 | |
| *** salmankhan has joined #openstack-containers | 21:35 | |
| *** Jeffrey4l_ has joined #openstack-containers | 21:36 | |
| *** catintheroof has quit IRC | 21:37 | |
| *** catintheroof has joined #openstack-containers | 21:37 | |
| *** EricGonczer_ has joined #openstack-containers | 21:42 | |
| *** adrian_otto has quit IRC | 21:45 | |
| *** adrian_otto has joined #openstack-containers | 21:56 | |
| *** catintheroof has quit IRC | 21:57 | |
| *** catintheroof has joined #openstack-containers | 21:58 | |
| *** mvelten has quit IRC | 22:10 | |
| *** salmankhan has quit IRC | 22:14 | |
| *** salmankhan has joined #openstack-containers | 22:24 | |
| *** EricGonczer_ has quit IRC | 22:30 | |
| *** catintheroof has quit IRC | 22:35 | |
| *** dimtruck is now known as zz_dimtruck | 22:38 | |
| *** Syed__ has joined #openstack-containers | 22:41 | |
| *** pc_m has quit IRC | 22:49 | |
| *** salmankhan has quit IRC | 22:50 | |
| *** pc_m has joined #openstack-containers | 22:50 | |
| *** chris_hultin is now known as chris_hultin|AWA | 22:58 | |
| *** vijendar_ has joined #openstack-containers | 23:02 | |
| *** yuanying has joined #openstack-containers | 23:07 | |
| *** v1k0d3n has quit IRC | 23:07 | |
| *** jperry has quit IRC | 23:09 | |
| *** zz_dimtruck is now known as dimtruck | 23:11 | |
| *** v1k0d3n has joined #openstack-containers | 23:22 | |
| *** yamamoto has joined #openstack-containers | 23:24 | |
| *** chhavi_ has joined #openstack-containers | 23:38 | |
| *** v1k0d3n has quit IRC | 23:41 | |
| *** chhavi_ has quit IRC | 23:44 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 23:45 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!