| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-15 | |||
| 17:17:37 | cdent | you happy with it edleafe ? | |
| 17:19:52 | edleafe | Made two micro-nit corrections, and now LGTM | |
| 17:20:14 | cdent | nice | |
| 17:20:45 | cdent | I'm often confused on project names, because back when docs wrote docs they preferred lowercase project names | |
| 18:01:53 | elmiko | cdent: sorry got pulled away | |
| 18:02:02 | cdent | no worries | |
| 18:08:46 | elmiko | ooh, i had not noticed that we are cross-posting those to openstack-sigs now as well. i will update my proceedures | |
| 18:09:33 | cdent | i don't always remember | |
| 18:09:42 | cdent | but I think we said at some point that it was maybe a good idea | |
| 18:13:05 | elmiko | yeah, it is a good idea. i had just forgotten we said that | |
| 18:24:59 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212 | |
| 18:24:59 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356 | |
| 18:25:00 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware https://review.openstack.org/550252 | |
| 18:25:00 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Improve Version.matches() method https://review.openstack.org/550251 | |
| 18:25:01 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Clean up package version and python version support https://review.openstack.org/550264 | |
| 18:25:01 | openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware https://review.openstack.org/550263 | |
| 18:25:14 | cdent | elmiko, edleafe ^ I think I got the things you mentioned | |
| 18:32:13 | elmiko | cdent: ack | |
| 18:39:07 | openstackgerrit | James Penick proposed openstack/python-openstackclient master: Stop prefetching images in server list https://review.openstack.org/553577 | |
| 18:49:53 | edleafe | cdent: will look again soon-ish | |
| 18:50:14 | cdent | thanks edleafe, elmiko | |
| 19:26:19 | edleafe | cdent: Looking over the HTTP Guidelines doc, it looks like it is logically 3 documents: HTTP caching, HTTP response code guidance, and HTTP method guidance | |
| 19:26:56 | cdent | yes, breaking things up is probably warranted | |
| 19:27:06 | edleafe | Perhaps breaking it into 3 separate docs would make them more "discoverable" | |
| 19:27:17 | edleafe | due to the more specific titles | |
| 19:27:37 | edleafe | If you like, I can take a crack at it. | |
| 19:30:07 | elmiko | edleafe: that sounds like a nice idea | |
| 19:30:26 | cdent | edleafe: feel free to go for it if you're feeling motivated | |
| 19:30:41 | cdent | do we want to leave redirects or something? | |
| 19:32:52 | edleafe | not sure - does sound like a good idea just in case someone has bookmarked the current page | |
| 19:33:11 | cdent | it would be very bad form for the API-SIG to break the web | |
| 19:43:14 | elmiko | cdent: ++ | |
| 19:43:18 | elmiko | lol | |
| 19:45:17 | cdent | edleafe: bah, broken client code is broken client code | |
| 19:45:43 | cdent | and in any case when I say "the web" i mean public document urls. I went to library school after all. | |
| 19:45:54 | elmiko | can we just have the page return 4xx, "we're not the problem, you are!" | |
| 19:46:50 | edleafe | elmiko: "Figure it out yourself!" | |
| 19:47:03 | elmiko | so helpful XD | |
| 19:47:08 | edleafe | "We're not your babysitter!" | |
| 19:47:12 | elmiko | LOL | |
| 19:47:24 | elmiko | best.error.message.ever. | |
| 20:31:44 | openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Fix typo in 'floating ip associate' command and doc https://review.openstack.org/553603 | |
| 21:12:44 | EmilienM | osc-functional-devstack failed on my patch, is it accurate to do recheck? I've seen some messages about failures recently | |
| 21:13:01 | EmilienM | failure is in post | |
| 21:13:16 | EmilienM | (job passed all tests but fail after, I think it's infra related) | |
| 23:34:13 | openstackgerrit | Merged openstack/osc-lib master: Allow to use the none auth plugin https://review.openstack.org/512699 | |
| #openstack-sdks - 2018-03-16 | |||
| 04:54:55 | niraj_singh | Note: it is coming while running unit test case. Functionally it is working fine. | |
| 04:54:55 | niraj_singh | i am getting error "AttributeError: Service Descriptors cannot be set" while creating client connection at line https://github.com/openstack/python-openstacksdk/blob/master/openstack/connection.py#L342 | |
| 06:59:49 | openstackgerrit | Hengqing Hu proposed openstack/python-openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404 | |
| 07:38:26 | openstackgerrit | OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902 | |
| 07:43:44 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/553257 | |
| 08:42:47 | EmilienM | hello, could I have some review on https://review.openstack.org/#/c/553374/ ? thanks :-) | |
| 08:59:29 | cdent | EmilienM: no | |
| 09:01:28 | EmilienM | cdent: this patch is solving all world problems, you have no idea | |
| 09:01:41 | cdent | no soup for you! | |
| 09:07:30 | cdent | no soup for cmurphy! | |
| 09:07:39 | cmurphy | T.T | |
| 09:08:03 | cdent | soup for everyone! | |
| 09:08:08 | cmurphy | \o/ | |
| 09:11:48 | EmilienM | you think you can buy me with soup | |
| 09:12:10 | EmilienM | you know only gin tonic work nowadays | |
| 09:12:12 | cdent | surfboards and wetsuits for everyone! | |
| 09:12:23 | cdent | gunpowder gin | |
| 10:17:56 | openstackgerrit | Merged openstack/keystoneauth master: Update links in README https://review.openstack.org/551694 | |
| 14:17:06 | cdent | elmiko, figleaf is my response to gilles okay? If not, can you also respond please? | |
| 14:17:25 | trevormc | hi all o/ I have a question about the cliff sphinx-extension.. Does it support the ArgparseDefaultHelpFormatter? https://docs.python.org/2/library/argparse.html#argparse.ArgumentDefaultsHelpFormatter | |
| 14:19:46 | figleaf | cdent: will read shortly | |
| 14:19:56 | cdent | thanks | |
| 14:26:10 | figleaf | cdent: ok, read it. I think you strike a good balance between "we are open to the idea" and "none of the cores are interested in doing that work" | |
| 14:26:42 | figleaf | The point about emphasizing email over IRC meetings is important | |
| 14:26:45 | cdent | I'm trying to say "there may be people out there who are open to the idea, it's not us" | |
| 14:28:30 | cdent | thanks for checking it figleaf, i'll take your response as "it's fine" | |
| 14:29:40 | figleaf | cdent: a better tone than I would have come up with, for sure :) | |
| 16:25:24 | lkwan | In the stable/queens branch, since openstacksdk0.11.3 and os-service-types1.1.0 are described in openstack's upper-constraints.txt, | |
| 16:25:25 | lkwan | https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L297 | |
| 16:25:25 | lkwan | https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L411 | |
| 16:25:41 | lkwan | if I do > git clone https://git.openstack.org/openstack-dev/devstack -b stable/queens | |
| 16:25:42 | lkwan | We will see it is using openstacksdk-0.12.0 and os_service_types-1.2.0 | |
| 16:25:42 | lkwan | And then stack.sh | |
| 16:26:20 | lkwan | Need some help how to get the older version, how to configure devstack to use openstacksdk===0.11.3 and os-service-types===1.1.0 | |
| 16:26:34 | lkwan | tried to manually install it before stack.sh | |
| 16:27:03 | lkwan | then had a dependency on those two packages. | |
| 16:27:03 | lkwan | indirectly brought in to meet some dependency that didn't get uninstalled, which | |
| 16:27:03 | lkwan | but it seems it's possible there was a package | |
| 16:27:07 | lkwan | Any tip? | |
| 16:31:25 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Fix crashing "console log show" https://review.openstack.org/541609 | |
| 16:35:07 | dhellmann | trevormc : I'm not sure what would be needed to ensure support. How is the output of the sphinx extension different from what you were hoping to see? | |
| 16:40:31 | trevormc | hi dhellman: I think we should allow a custom formatter to be provided for the get_parser method, https://github.com/openstack/cliff/blob/master/cliff/command.py#L158 | |
| 16:40:55 | dhellmann | trevormc : which command are you working on? | |
| 16:42:09 | trevormc | dhellman: not any upstream at the moment :( | |
| 16:42:37 | dhellmann | ok. what is it that you're trying to do? maybe there's another way. | |
| 16:42:56 | dhellmann | I'm not saying no to making that class configurable, just trying to understand the root of the request you have | |
| 16:43:49 | trevormc | dhellman: yeah i understand, thanks for your time. I'm looking to add the ArgumentDefaultsHelpFormatter which will print the default value along with the help message. | |
| 16:44:09 | dhellmann | ok, so you just want the default value to show up in the help text? | |
| 16:44:25 | trevormc | dhellman, yeah it was a small improvement. | |
| 16:44:28 | dhellmann | ok | |
| 16:44:40 | dhellmann | if the string %(default)s appears in your help text, it will be filled in automatically | |
| 16:45:01 | dhellmann | so you can do something like help='this option is boring. It defaults to %(default)s.' | |
| 16:45:17 | trevormc | dhellman, yeah that's another approach, I can use that. Thanks :) | |
| 16:45:37 | dhellmann | that should help in the short term. in the longer term, we could add the default info to what the existing smart help thing does | |