| mordred | hahaha | 00:09 |
|---|---|---|
| mordred | jlk: https://github.com/ansible/ansibullbot/pull/212 :) | 00:11 |
| jlk | mordred: looks like some labeling also happens at https://github.com/ansible/ansibullbot/blob/master/lib/triagers/ansible_ansible_pulltriager.py so I'm confused | 00:13 |
| mordred | sigh | 00:14 |
| jlk | I mean, I think your PR does the right thing, it's just confusing that there is a secondary place where labeling happens | 00:14 |
| jlk | but I think they said a rewrite was in progress, so ¯\_(ツ)_/¯ | 00:15 |
| mordred | I'm honestly a little pissed that there are branches for every cloud BUT openstack | 00:15 |
| mordred | including vmware and digitalocean | 00:15 |
| mordred | but whatev | 00:15 |
| clarkb | mordred: thats because openstack is a fad | 00:16 |
| mordred | jlk: I added a patch to deal with the right openstack things there too | 00:17 |
| *** cdent has quit IRC | 00:27 | |
| *** abregman has joined #openstack-shade | 06:23 | |
| openstackgerrit | Arie Bregman proposed openstack-infra/shade: Add compute usage support https://review.openstack.org/393873 | 07:57 |
| *** jroll has quit IRC | 08:27 | |
| *** abregman has quit IRC | 08:28 | |
| *** abregman has joined #openstack-shade | 08:30 | |
| *** abregman has quit IRC | 08:40 | |
| *** jroll has joined #openstack-shade | 08:41 | |
| *** gouthamr has joined #openstack-shade | 08:45 | |
| *** abregman has joined #openstack-shade | 08:53 | |
| *** gouthamr has quit IRC | 09:05 | |
| *** gouthamr has joined #openstack-shade | 09:14 | |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** openstackgerrit has joined #openstack-shade | 09:48 | |
| *** gouthamr has quit IRC | 10:53 | |
| *** cdent has joined #openstack-shade | 11:12 | |
| *** purplerbot has quit IRC | 11:29 | |
| *** purplerbot has joined #openstack-shade | 11:29 | |
| *** gouthamr has joined #openstack-shade | 11:32 | |
| *** purplerbot has quit IRC | 11:34 | |
| *** purplerbot has joined #openstack-shade | 11:35 | |
| *** gouthamr has quit IRC | 11:47 | |
| *** gouthamr has joined #openstack-shade | 11:52 | |
| *** gouthamr has quit IRC | 12:52 | |
| *** gouthamr has joined #openstack-shade | 13:30 | |
| *** purplerbot has quit IRC | 13:43 | |
| *** abregman has quit IRC | 13:43 | |
| *** purplerbot has joined #openstack-shade | 13:45 | |
| *** cdent_ has joined #openstack-shade | 15:18 | |
| *** cdent has quit IRC | 15:20 | |
| *** cdent_ is now known as cdent | 15:20 | |
| *** abregman has joined #openstack-shade | 16:06 | |
| *** abregman has quit IRC | 16:06 | |
| *** gouthamr has quit IRC | 16:23 | |
| *** cdent_ has joined #openstack-shade | 16:46 | |
| *** cdent has quit IRC | 16:48 | |
| *** cdent has joined #openstack-shade | 16:49 | |
| *** cdent_ has quit IRC | 16:52 | |
| *** cdent has quit IRC | 17:33 | |
| *** cdent has joined #openstack-shade | 17:34 | |
| *** cdent has quit IRC | 18:58 | |
| *** abregman has joined #openstack-shade | 19:34 | |
| mordred | abregman: in answer to your question in infra channel earlier- the normalize functions in _utils just haven't been moved yet | 19:40 |
| mordred | abregman: all of them should be documented, made to support the things that the _normalize ones do (location, properties, strict_mode) and moved to _normalize | 19:41 |
| mordred | abregman: I've been doing them one resource at a time as I can | 19:41 |
| abregman | mordred: ack, I can move them. btw, can you please tell me if this failure is due to my change or infra related issue?: https://review.openstack.org/#/c/393873 | 19:42 |
| abregman | because locally the tests pass for me | 19:42 |
| *** openstackgerrit has quit IRC | 19:48 | |
| *** openstackgerrit has joined #openstack-shade | 19:48 | |
| mordred | abregman: hrm. that's weird ... lemme look at it in just a bit | 19:48 |
| abregman | thanks | 19:51 |
| mordred | I don't see how your patch could have caused most of those errors - but there is one failure related to your patch: | 19:53 |
| mordred | http://logs.openstack.org/73/393873/8/check/gate-shade-dsvm-functional-nova/b38f9c1/console.html#_2016-11-17_17_46_06_193499 | 19:53 |
| mordred | abregman: I wonder if the devstack config for our nova-network test is somehow causing the usage stuff to not show up? | 19:54 |
| abregman | mordred: hmm it started to fail after I updated the change with normalizing the usage data, but I don't see how it could cause this issue | 19:58 |
| mordred | me either | 19:59 |
| *** abregman is now known as abregman|afk | 20:08 | |
| dtroyer | quick question because I'm too lazy to look in the share repo… which Image API version does shade default to? or does it not set a default, letting the app do it? | 21:10 |
| dtroyer | *shade repo | 21:10 |
| mordred | dtroyer: well - | 21:12 |
| mordred | dtroyer: that's in the middle of changing slightly | 21:12 |
| mordred | but at the moment it believes whatever the cloud setting it got from occ | 21:13 |
| dtroyer | occ only has 3 entries in the vendor files for image v1, none for v2 | 21:13 |
| mordred | yah. it defaults to v2 - then we annotate clouds that are v1 | 21:13 |
| dtroyer | ah, ok, I forgot it has opinions on that | 21:13 |
| mordred | in the switch-to-rest patches, I've got the code to do version discovery on the glance endpoint | 21:14 |
| dtroyer | defaults.json, why must you hide from me? | 21:14 |
| dtroyer | the conclusion I am looking for is in changing OSC to default to 2, sounds like it already is for anything using occ that isn't OSC, so we should be good | 21:15 |
| mordred | yah - 2 is pretty widely out there | 21:15 |
| mordred | dtroyer: https://review.openstack.org/#/c/384208 is the patch which switches list_images to rest and also switches it to defaulting to CURRENT version if version discovery works - fwiw | 21:17 |
| morgan_ | mordred: looks like we fixed betamax in ksa. next step task | 21:22 |
| mordred | morgan_: woot | 21:23 |
| *** hoolio has joined #openstack-shade | 22:14 | |
| *** cdent has joined #openstack-shade | 22:15 | |
| mordred | hoolio: hi! what's up? | 22:15 |
| hoolio | hey | 22:15 |
| hoolio | this works; | 22:15 |
| hoolio | http://paste.ubuntu.com/23492642/ | 22:16 |
| hoolio | but if i specify the security group, it errors out | 22:16 |
| hoolio | http://paste.ubuntu.com/23492647/ | 22:16 |
| hoolio | the sec group uid is correct, and i've also tried the sec group by name (SSH) | 22:17 |
| hoolio | if i specify the sec group via name i get "..Unable to find security_group with name 'H'.." | 22:19 |
| mordred | hoolio: (looking at code - one sec) | 22:22 |
| hoolio | k, np | 22:22 |
| *** cdent has quit IRC | 22:23 | |
| mordred | hoolio: I believe security_groups is supposed to be a list | 22:24 |
| mordred | hoolio: we could do a better job in shade of detecting this and fixing it for you ... | 22:25 |
| * hoolio googles "single element list python" | 22:25 | |
| mordred | hoolio: just wrap it in [] | 22:26 |
| mordred | hoolio: but I'll make that better real quick ... | 22:26 |
| hoolio | list3 = ["a"] | 22:26 |
| mordred | yup | 22:26 |
| hoolio | okies | 22:26 |
| hoolio | so i tried pulling from git://git.openstack.org/openstack-infra/shade but it seems it's the same version as pip | 22:27 |
| hoolio | i wondered if there was any latency in propagation between the two | 22:28 |
| mordred | yah - I'll make a patch to make this better for you - but it'll take a little bit | 22:28 |
| hoolio | yeah np. i really wanted to submit a patch myself, but the code is a bit over my head at the moment | 22:28 |
| hoolio | I'm only reasonably new to python | 22:28 |
| hoolio | i should get the o'reilly book ;) | 22:29 |
| hoolio | coz you know, just having it makes you better .. | 22:30 |
| hoolio | it works! | 22:37 |
| hoolio | thx mordred | 22:38 |
| mordred | woot! | 23:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Allow security_groups to be a scalar https://review.openstack.org/399290 | 23:08 |
| mordred | hoolio: ^^ | 23:08 |
| mordred | hoolio: when that lands and we make a new release, just passing a single security group should be fine | 23:09 |
| hoolio | np. will leaving it as a single element list still work? | 23:09 |
| mordred | yup | 23:18 |
| mordred | it'll work both ways | 23:18 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!