| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-07-04 | |||
| 13:14:14 | mordred | morning seanhandley ! | |
| 14:54:21 | openstackgerrit | Javier Peña proposed openstack/python-openstackclient master: Fix man page build https://review.openstack.org/480218 | |
| 17:46:35 | openstackgerrit | OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/480277 | |
| 19:28:41 | openstackgerrit | Merged openstack/python-openstackclient master: Fix man page build https://review.openstack.org/480218 | |
| #openstack-sdks - 2017-07-05 | |||
| 08:54:48 | openstackgerrit | Boris Bobrov proposed openstack/keystoneauth master: Change locations of docs for intersphinx https://review.openstack.org/480447 | |
| 09:05:58 | openstackgerrit | Boris Bobrov proposed openstack/keystoneauth master: Change locations of docs for intersphinx https://review.openstack.org/480447 | |
| 12:29:51 | openstackgerrit | Arundhati Surpur proposed openstack/os-client-config master: There is no documentation for os-client config https://review.openstack.org/480553 | |
| 13:34:34 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add new parameter "is_default" to Network QoS policy. https://review.openstack.org/432260 | |
| 13:43:37 | GreenBlood | Hey guys I think I found a bug in the shade library | |
| 13:44:59 | GreenBlood | In the openstackcloud.py file, in the userdata section (function _encode_server_userdata), the userdata is tested with isinstance() twice, but the second time the variable name is "text" and not "userdata" | |
| 13:45:27 | GreenBlood | and text being not defined, the code crashes. I changed "text" to "userdata" locally and it seems to work | |
| 13:47:06 | GreenBlood | https://github.com/openstack-infra/shade/blob/master/shade/openstackcloud.py#L5618 here | |
| 13:47:32 | mordred | GreenBlood: looking ... | |
| 13:47:44 | GreenBlood | thanks | |
| 13:50:35 | mordred | GreenBlood: yes. that is definitely a bug - you wanna send a patch? or want me to just fix it? | |
| 13:51:34 | GreenBlood | I don't have a github acc at the moment so if you want to handle the stuff go for it | |
| 13:59:02 | mordred | GreenBlood: https://review.openstack.org/480592 patch up - thanks! | |
| 14:00:31 | GreenBlood | neat :) thanks for the quick reply | |
| 14:08:07 | openstackgerrit | Honza Pokorny proposed openstack/python-openstackclient master: Allow objects to be streamed to stdout https://review.openstack.org/476199 | |
| 14:38:41 | openstackgerrit | Honza Pokorny proposed openstack/python-openstackclient master: Allow objects to be streamed to stdout https://review.openstack.org/476199 | |
| 14:50:08 | snecklifter | Hello sdkers, is it possible to get a review on a backport pls https://review.openstack.org/#/c/480595/ | |
| 14:50:10 | snecklifter | thanks | |
| 15:03:57 | seanhandley | snecklifter: I spy an ale fan :-) | |
| 15:04:28 | snecklifter | seanhandley: thank goodness _someone_ recognises it | |
| 15:06:35 | snecklifter | seanhandley: only problem is, they moved production when the town flooded (understandable) and its completely changed character | |
| 15:07:03 | seanhandley | It's a while since I had a Sneck Lifter tbh... | |
| 15:07:06 | snecklifter | seanhandley: much lighter, a shadow of the dark mighty force it used to be | |
| 15:07:07 | seanhandley | don't often see it on the shelves | |
| 15:07:12 | seanhandley | ah that's a shame | |
| 15:07:23 | seanhandley | You can't make the same beer in a different place | |
| 15:07:26 | seanhandley | same with whisky | |
| 15:07:36 | snecklifter | seanhandley: you didnt used to be able to see through it, used the same process as guiness | |
| 15:07:42 | snecklifter | seanhandley: ACK | |
| 15:07:45 | seanhandley | it's just not possible to get every aspect of the process the same between locations | |
| 15:07:59 | seanhandley | ahh and now it's filtered and transparent? | |
| 15:08:14 | snecklifter | not far off, very light | |
| 15:08:41 | snecklifter | but I kept the name as my internet handle because Americans cant pronounce it | |
| 15:09:11 | snecklifter | lolol, jk, because its unique and unlike my actual name I can be tracked down with ease | |
| 15:09:20 | seanhandley | heh | |
| 15:09:26 | seanhandley | things have changed | |
| 15:09:30 | seanhandley | Americans have good beer now | |
| 15:09:42 | snecklifter | seanhandley: now to business, can you +1 my backport? | |
| 15:09:47 | seanhandley | heheh | |
| 15:09:50 | snecklifter | seanhandley: true, we now steal their ideas | |
| 15:09:55 | mordred | seanhandley: actually a surprisingly large amount of it | |
| 15:09:58 | seanhandley | Will review for beer | |
| 15:10:09 | snecklifter | seanhandley: ok, will buy you one at next meetup | |
| 15:10:15 | seanhandley | deal :D | |
| 15:11:31 | mordred | seanhandley: btw- almost done with the service-types publication stuff ... | |
| 15:11:46 | mordred | seanhandley: you can see examples of the files that will be published at: | |
| 15:11:48 | mordred | http://logs.openstack.org/41/478541/2/check/gate-service-types-authority-validate/cad9923/ | |
| 15:11:59 | seanhandley | oh cool - I'll take a look | |
| 15:12:01 | mordred | and they'll be at https://service-types.openstack.org | |
| 15:13:04 | seanhandley | what does the `reverse` key represent mordred? | |
| 15:15:20 | seanhandley | and the `forward` key, come to think of it :D | |
| 15:19:02 | mordred | seanhandley: easy forward and reverse lookups - basically, to consume the raw data in services, pretty much everyone would need to build a lookup table in code - so we just make it at build time :) | |
| 15:19:23 | mordred | seanhandley: forward is "given this official name, give me the list of aliases in preference order" | |
| 15:19:37 | mordred | samueldmq: reverse is "given this alias, give me the official name" | |
| 15:19:41 | mordred | gah | |
| 15:19:47 | mordred | seanhandley: ^^ sorry samueldmq :) | |
| 15:20:15 | mordred | seanhandley: then services is the raw canonical data | |
| 15:28:48 | seanhandley | ahaaa | |
| 15:28:56 | seanhandley | thanks mordred | |
| 15:34:57 | efried | mordred Looks like maybe I'm straddling the 'publish' change here http://logs.openstack.org/56/478556/1/check/gate-service-types-authority-validate/d74f749/console.html | |
| 15:35:07 | efried | I can't seem to rebase. | |
| 15:37:51 | mordred | efried: there's a system bug which will be fixed soon | |
| 15:38:14 | efried | mordred Cool. Was thinking maybe I need to rebase on top of https://review.openstack.org/#/c/478541/ But I'll just wait. | |
| 15:38:26 | mordred | efried: the dns name for service-types.o.o got added before the vhost did - so GET service-types.o.o is returning a documentation not a 404 or an empty value | |
| 15:38:50 | mordred | efried: but you _do_ need to rebase on top of that - because the job changes to assume the publish updates has gone in | |
| 15:39:00 | mordred | efried: just don't worry about the validate job failing for now :) | |
| 15:39:15 | efried | mordred Okay. The failure was on the 'publish' directory not existing. | |
| 15:39:17 | efried | I'll rebase. | |
| 15:39:24 | mordred | ++ | |
| 15:40:40 | openstackgerrit | Eric Fried proposed openstack/service-types-authority master: Improve validation & documentation https://review.openstack.org/478556 | |
| 15:40:52 | openstackgerrit | Eric Fried proposed openstack/service-types-authority master: Remove validate.py main() https://review.openstack.org/478562 | |
| 15:41:09 | efried | cdent mordred ^^ | |
| 15:41:48 | mordred | cool. once the vhost goes live, we can recheck those and be good to go | |
| 15:43:43 | efried | k | |
| 15:56:45 | amotoki | dtroyer: stevemar: could you review the network tag support in OSC? https://review.openstack.org/#/c/461195/ | |
| 15:57:25 | amotoki | it is one of the last two pieces of neutron CLI migration, but I failed to get review for three weeks. | |
| 15:57:45 | stevemar | sorry amotoki been busy :) | |
| 15:57:54 | stevemar | amotoki: i've added it to my queue :) | |
| 15:58:04 | amotoki | no worries. it is better to split into pieces? | |
| 15:58:21 | amotoki | stevemar: if so let me know it. | |
| 16:09:22 | stevemar | amotoki: no, its big but it's do-able | |
| 16:09:48 | amotoki | stevemar: thanks. will keep the current version | |
| 16:10:19 | stevemar | amotoki: thanks for contributing to osc btw :) | |
| 16:13:13 | amotoki | stevemar: I am mainly responsible for CLI side in the neutron team (in addition to neutron core). we've almost migrated to osc so I will continue to keep my eyes on osc. | |
| 16:22:25 | openstackgerrit | Merged openstack/os-client-config master: use openstackdocstheme html context https://review.openstack.org/477005 | |
| 16:25:37 | openstackgerrit | Dean Troyer proposed openstack/cliff master: add --fit-width option to table formatter https://review.openstack.org/471917 | |
| 17:16:02 | openstackgerrit | Honza Pokorny proposed openstack/python-openstackclient master: Allow objects to be streamed to stdout https://review.openstack.org/476199 | |
| 17:42:59 | openstackgerrit | Merged openstack/cliff master: sphinxext: Add 'application' option to the autoprogram directive https://review.openstack.org/469726 | |
| 18:07:24 | dtroyer | dhellmann: do you have an opinion on https://review.openstack.org/#/c/479157/ for cliff? that is the last thing I was wondering about before we do the soon-to-be-stable release for this cycle | |
| 18:13:18 | dtroyer | stevemar: do you have an opinion on https://review.openstack.org/#/c/476523/ in osc-lib? I think I agree with the logic... | |
| 18:14:07 | stevemar | ohhhh find_resource | |
| 18:14:18 | stevemar | last time we poked around there badness happened | |
| 18:14:51 | dtroyer | I remember…that's why I want to think through this well… I think I am onboard but it's been a while | |
| 18:15:14 | dtroyer | we could hold off until after the upcoming stable branch | |
| 18:15:37 | stevemar | we can do it | |
| 18:15:45 | stevemar | looks solid | |