| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-08-05 | |||
| 21:28:36 | openstackgerrit | Merged openstack/os-client-config master: Remove OSIC https://review.openstack.org/491196 | |
| #openstack-sdks - 2017-08-06 | |||
| 04:22:07 | openstackgerrit | Rajath Agasthya proposed openstack/cliff master: Alias exit to cmd2's quit command to exit interactive shell https://review.openstack.org/491217 | |
| #openstack-sdks - 2017-08-07 | |||
| 00:51:35 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/491294 | |
| 00:53:48 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/490862 | |
| 07:10:08 | openstackgerrit | Lingyong Xu proposed openstack/os-client-config master: Update the documentation link for doc migration https://review.openstack.org/491374 | |
| 07:13:30 | openstackgerrit | Zhaokun Fu proposed openstack/python-openstackclient master: add --password option for openstack server create Closes-Bug:1708570 https://review.openstack.org/490718 | |
| 07:30:05 | openstackgerrit | Thomas Bechtold proposed openstack/cliff master: Make openstackdocstheme an optional doc dependency https://review.openstack.org/491378 | |
| 09:33:27 | openstackgerrit | chenying proposed openstack/python-openstackclient master: Add new commands for karbor osc plugin https://review.openstack.org/491416 | |
| 18:14:35 | openstackgerrit | Rajath Agasthya proposed openstack/python-openstackclient master: Add flavor value options https://review.openstack.org/302795 | |
| 21:41:22 | openstackgerrit | Chris Dent proposed openstack/api-wg master: WIP: Explain, simply, why extensions are bad https://review.openstack.org/491611 | |
| 21:53:00 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Convert network segment functional tests to JSON https://review.openstack.org/491612 | |
| 22:38:41 | openstackgerrit | Joshua Harlow proposed openstack/os-client-config master: Update globals safely https://review.openstack.org/491618 | |
| 22:44:44 | openstackgerrit | Joshua Harlow proposed openstack/os-client-config master: Update globals safely https://review.openstack.org/491618 | |
| 22:47:04 | openstackgerrit | Joshua Harlow proposed openstack/os-client-config master: Update globals safely https://review.openstack.org/491618 | |
| 23:33:40 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add .htaccess for docs migration https://review.openstack.org/491631 | |
| 23:37:17 | openstackgerrit | Morgan Fainberg proposed openstack/keystoneauth master: Remove use of positional decorator https://review.openstack.org/491632 | |
| 23:47:39 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Convert network segment functional tests to JSON https://review.openstack.org/491612 | |
| #openstack-sdks - 2017-08-08 | |||
| 01:17:28 | openstackgerrit | Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/490862 | |
| 04:14:54 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: Allow PD as Subnetpool during Subnet creations https://review.openstack.org/491664 | |
| 12:09:49 | cdent | thanks for the comments on the extensions thing edleafe . That effort was a bit half-hearted: trying to get myself started but not really having enough brain to really get it together. | |
| 12:13:16 | edleafe | Heh, at least in your case "not enough brain" is a temporary state. | |
| 12:16:17 | openstackgerrit | Chris Dent proposed openstack/api-wg master: WIP: Explain, simply, why extensions are bad https://review.openstack.org/491611 | |
| 12:17:10 | cdent | I’m no longer sure. This last year has involved a startling decline. | |
| 12:35:40 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Convert network segment functional tests to JSON https://review.openstack.org/491612 | |
| 14:48:20 | openstackgerrit | Merged openstack/os-client-config master: Update globals safely https://review.openstack.org/491618 | |
| 15:24:25 | efried | Anyone know how/where to open bugs against service-types-authority? | |
| 15:26:38 | cdent | efried: i suspect we never got around to that | |
| 15:27:21 | efried | cdent Since os-service-types is on storyboard, would it be appropriate to create a storyboard... board for it? | |
| 15:27:40 | cdent | I think that would be fine idea | |
| 15:28:04 | efried | cdent Then the question would we where to make note of that. I guess in the sta readme and/or setup.py | |
| 15:28:28 | cdent | you have all the best questions. readme seems good for at least a start | |
| 15:35:58 | cdent | efried: I’ve nominated you for core on that repo as it seems unreasonable for you not to be. we’ll see what sdague and mordred say | |
| 15:36:19 | mordred | cdent: ++ | |
| 15:36:21 | efried | cdent Okay, thanks! | |
| 15:36:30 | mordred | cdent, efried: wait ... | |
| 15:36:57 | mordred | service-types-authority is an API-WG repo, so it should do bugs whereever api-wg does bugs | |
| 15:37:24 | mordred | os-service-types is a lib consuming it that's part of the shade project, so it does bugs where shade does bugs | |
| 15:37:43 | cdent | it’s tc repo, not an api-wg repo, but that would also be fine, there is an openstack-api-wg launchpad presence | |
| 15:37:54 | mordred | that those two places are two different places is a temporary annoyance, but I think we should associated bugs with owning projects | |
| 15:38:05 | mordred | cdent: fair | |
| 15:39:05 | cdent | efried: as a token nearby api-wg person, I hereby authorize you to put bugs in api-wg if you want | |
| 15:39:37 | efried | cdent Wherezat? | |
| 15:39:45 | cdent | https://launchpad.net/openstack-api-wg | |
| 15:39:58 | efried | thx | |
| 15:40:38 | sdague | cdent: ++ | |
| 15:40:42 | efried | cdent mordred btw, the bug I'm going to open is about placement not having an entry. Which started to be addressed by https://review.openstack.org/#/c/462140/ but needs to be driven to closure pretty soon, methinks. | |
| 15:42:00 | efried | https://bugs.launchpad.net/openstack-api-wg/+bug/1709353 | |
| 15:42:01 | openstack | Launchpad bug 1709353 in openstack-api-wg "placement needs an entry in service-types-authority" [Undecided,New] | |
| 15:42:03 | cdent | yeah. that’s currently blocked on docs migration stuff, I think? There’s a bit of blip with regard to getting the right url, which we wouldn’t have until after the current stack of new docs is merged, which is stalled on feature freeze chaos | |
| 15:42:15 | efried | yarks | |
| 15:42:38 | efried | cdent it's not just https://docs.openstack.org/nova/latest/user/placement.html ? | |
| 15:42:47 | cdent | we could just go ahead without the docs ref | |
| 15:42:58 | cdent | no, that’s not the api-ref | |
| 15:43:04 | cdent | we coud use that as a stub for now | |
| 15:43:28 | cdent | I personally think the entry is far more important than whatever the doc link says | |
| 15:43:32 | cdent | we want placement to be consumable | |
| 15:43:41 | cdent | it is and has always been at the service-type placement | |
| 15:45:57 | cdent | efried: you should have +2 now, since mordred and sdague were immediately keen | |
| 15:46:07 | efried | cdent Sweet, thank you. | |
| 15:46:17 | cdent | abuse your power with grace! | |
| 15:46:20 | sdague | cdent: there should be an api-ref url for placement, no? | |
| 15:46:43 | efried | cdent sdague Yeah, the one listed in the change set is live, but actually redirects to the user one today. | |
| 15:46:47 | sdague | or is that not being published? | |
| 15:46:50 | efried | https://docs.openstack.org/nova/latest/placement.html | |
| 15:47:06 | cdent | sdague: there will be eventually but there isn’t yet: we held off on creating the publishing job because the docs migration was in progress | |
| 15:47:08 | sdague | efried: that's not the api-ref | |
| 15:47:19 | efried | bleh, right, developer.o.o? | |
| 15:47:26 | sdague | cdent: ok, so that doesn't really get impacted by docs migration | |
| 15:47:43 | sdague | I'd just do that and land it at developer.openstack.org/api-ref/placement | |
| 15:47:43 | cdent | and we didn’t want to move all the files until after they had all merged | |
| 15:48:02 | sdague | cdent: ok... I guess I'm confused where they are moving to | |
| 15:48:06 | efried | sdague To be clear, land it with a (for now) broken link? | |
| 15:48:44 | cdent | sdague: I’d need to chase some of the conversations with dhellmann and avolkov to really get my memory cear | |
| 15:49:58 | sdague | the api-ref sphinx extension is not guaunteed to not scramble the existing look feel of the main docs | |
| 15:50:04 | sdague | which is why it's in separate repo | |
| 15:50:18 | sdague | and I don't see reasons for that to change | |
| 15:50:27 | sdague | especially as they have a very different target site | |
| 15:52:30 | cdent | part of the issue may have been that having two api-ref-style jobs publishing from the same repo means that the second job needs to have some customizations, and if (I’m not sure if this is even desired?) the placement-api-ref directory was going to move under doc/source we’d wait until after all the pending changes merged before doing that | |
| 15:52:47 | cdent | that was based on the apparently very bad assumption that the pending docs reviews would merge much faster | |
| 15:53:13 | cdent | but mostly I don’t actually know that much of what’s going on: avolkov had that ball and I lost track of it | |
| 15:56:02 | openstackgerrit | Eric Fried proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140 | |
| 15:59:18 | sdague | I think things are going to be really goofy to try to put api-ref under doc/source | |
| 15:59:27 | sdague | and I would generally advise against that | |
| 15:59:37 | sdague | api-guide could probably move | |
| 16:01:05 | cdent | sdague: yeah, I’m really grasping at lose straws of memory here and since I think avolkov may be less around now than he was before, we may be able to just come up with a new correct plan and do whatever. I’m not clear what that should be. | |
| 16:02:05 | sdague | cdent: ok. Well I'm around this week but then gone for 2. So if you want assistance on that bit, getting the publishing patches posted soon would be good | |
| 16:03:54 | cdent | I’m so unanchored that I don’t know where to begin, but can probably resituate myself tomorrow (barring yet more tyre fires in allocations). One concerm was that I thought we didn’t want to publish incomplete docs? | |
| 16:06:34 | sdague | on the api-ref side I kind of thing incomplete is better than none | |
| 16:12:20 | openstackgerrit | Dean Troyer proposed openstack/osc-lib feature/osc4: Refactor initialize_app() to allow overriding https://review.openstack.org/491846 | |
| 16:33:21 | cdent | sdague: I found some of the lost context. In https://review.openstack.org/#/c/480991/ dhellmann says to move the placement-api-ref into docs/source | |
| 16:34:30 | cdent | sdague, dhellmann : however I think much of what is said there is confusion over how placement and nova are nearby | |
| 16:34:57 | sdague | yeh | |
| 16:35:56 | sdague | it also doesn't seem to mention developer.openstack.org, as this would implicitely remove that | |
| 16:36:16 | sdague | however the developer experience folks were very very strongly in the camp that it was important that that was a top level construct | |
| 16:38:42 | sdague | and, actually, that ends up making the api-ref versioned per release, which is something we very much don't want to do | |
| 16:38:55 | cdent | very good point | |
| 16:42:40 | sdague | I left a post hoc comment in https://review.openstack.org/#/c/472275/13 | |
| 16:45:50 | cdent | sdague: cool. I’m going to push up a new gate job, which will make note of some of the conversation here and refer back to the abandoned one | |
| 17:01:29 | cdent | sdague: i made a new job, because i was nearly all the way done before I found the abandoned one: https://review.openstack.org/491860 | |
| 17:03:01 | cdent | thanks | |