| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-10-23 | |||
| 20:41:39 | mordred | thx | |
| 22:34:26 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add support for Node tainted field https://review.opendev.org/690755 | |
| 22:48:56 | mordred | Shrews: if you get a sec, https://review.opendev.org/#/c/686765/ - would love to get that in and cut a release | |
| #openstack-sdks - 2019-10-24 | |||
| 01:24:51 | openstackgerrit | Yongli He proposed openstack/python-openstackclient master: Microversion 2.78 - server show topology https://review.opendev.org/680928 | |
| 01:26:01 | yonglihe | added api version check, suggested by Matt , thanks. | |
| 05:27:58 | vishakha | mordred: hey | |
| 05:57:19 | openstackgerrit | jun923.gu proposed openstack/keystoneauth master: Fix trust-scoped token create another token https://review.opendev.org/690812 | |
| 06:20:21 | openstackgerrit | pengyuesheng proposed openstack/osc-lib master: Switch to Ussuri jobs https://review.opendev.org/690839 | |
| 06:45:43 | openstackgerrit | pengyuesheng proposed openstack/os-service-types master: Switch to Ussuri jobs https://review.opendev.org/690859 | |
| 08:19:49 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Re-add functional tests on flavor content https://review.opendev.org/686765 | |
| 08:21:56 | openstackgerrit | jun923.gu proposed openstack/keystoneauth master: Fix trust-scoped token create another token https://review.opendev.org/690812 | |
| 12:49:07 | openstackgerrit | jun923.gu proposed openstack/keystoneauth master: Fix trust-scoped token create another token https://review.opendev.org/690812 | |
| 13:07:34 | elmiko | dtantsur|afk: any chance you could cover the api-sig office hour today? | |
| 14:25:42 | ttx | elmiko: I'm cleaning up the SIG list page... I was wondering if I should update the chair list for the API SIG (removing edleafe maybe?). thoughts? | |
| 14:26:08 | ttx | (page at https://governance.openstack.org/sigs/ for reference) | |
| 14:49:46 | elmiko | ttx: that sounds good, it's basically just myself and dtantsur as the main chairs at this point | |
| 14:49:51 | elmiko | also, thank you =) | |
| 14:50:53 | ttx | ok, so I can update to you and Dmitry | |
| 14:51:10 | ttx | will post review here when I get to it so y'all can +1 | |
| 14:58:25 | elmiko | ttx: thanks! | |
| 14:59:20 | elmiko | _come developers and managers please lend your hand_ XD | |
| 15:21:35 | miloa | Hello | |
| 15:25:01 | miloa | I had a problem with openstack-shade function _get_grant_revoke_params in order to take domain parameter for stable/rocky. I can propose a patch, but as openstack-sdk is the reference, i don't know if it would be interresting to propose this change on shade ? | |
| 15:50:53 | elmiko | miloa: just a heads up, this channel can get quiet sometimes. you might want to ping someone directly from the openstack-shade team | |
| 15:55:02 | mriedem | dtroyer: mordred: any thoughts on how to proceed with this osc bug and novaclient 16.0.0 https://storyboard.openstack.org/#!/story/2006769 ? | |
| 15:59:10 | dtroyer | mriedem: I agree with your assessment that using the SDK is the preferable option here. OSC already uses it anyway | |
| 15:59:20 | miloa | elmiko: thanks :) but i don't know who is in the team :) | |
| 16:06:01 | mriedem | dtroyer: how do i get a handle to the compute v2 sdk proxy from the app clientmanager thing in osc? | |
| 16:06:05 | mriedem | is there an example? | |
| 16:09:14 | dtroyer | the network code has an example but I'm not sure that compute will work exactly the same way. I want to say that there is already a common SDK object in ClientManager to get the proxy from but I don't have this in my head right now, I could probably look at it tomorrow… | |
| 16:11:43 | mriedem | ah i think i see, something like self.app.client_manager.sdk_connection | |
| 16:12:11 | dtroyer | yes | |
| 16:13:11 | efried | mriedem: that looks right to me. You would use ...sdk_connection.compute to get at nova | |
| 16:13:20 | mriedem | yar | |
| 16:13:52 | efried | and then you can cut everything else over to sdk while you're at it. | |
| 16:15:13 | mriedem | i'll cut YOU | |
| 17:39:10 | openstackgerrit | Colleen Murphy proposed openstack/python-openstackclient master: Add support for app cred access rules https://review.opendev.org/677857 | |
| 17:40:50 | cmurphy | mordred: think you could have a look at https://review.opendev.org/669331 ? vishakha's been working on it for a while now | |
| 18:18:10 | openstackgerrit | Matt Riedemann proposed openstack/python-openstackclient master: Use SDK to get compute API extensions https://review.opendev.org/691039 | |
| 18:51:18 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: WIP:Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| 18:52:08 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: WIP:Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| 20:07:43 | mordred | cmurphy, vishakha: lgtm | |
| 20:34:18 | cmurphy | thanks mordred | |
| 20:44:05 | mordred | dtantsur|afk: the sdk ironic and ironic-inspector jobs are failing ... | |
| 20:44:40 | mordred | dtantsur|afk: for ironic: "The /opt/stack/ironic-python-agent-builder project was not found;" | |
| 20:46:09 | mordred | dtantsur|afk: for ironic-inspector, it's a post failure - something about --subunit not being a valid argument to last | |
| 20:46:25 | mordred | I'm going to dig in a little - but if you already know answers, awesome :) | |
| 22:17:28 | openstackgerrit | Merged openstack/openstacksdk master: Re-add functional tests on flavor content https://review.opendev.org/686765 | |
| #openstack-sdks - 2019-10-25 | |||
| 02:24:37 | openstackgerrit | Xuan Yandong proposed openstack/cliff master: Switch to Ussuri jobs https://review.opendev.org/691148 | |
| 02:48:50 | openstackgerrit | Xuan Yandong proposed openstack/openstacksdk master: Switch to Ussuri jobs https://review.opendev.org/691158 | |
| 02:59:10 | openstackgerrit | Xuan Yandong proposed openstack/shade master: Switch to Ussuri jobs https://review.opendev.org/691162 | |
| 04:04:25 | vishakha | mordred: thanks | |
| 07:08:21 | openstackgerrit | zhangboye proposed openstack/keystoneauth master: Switch to Ussuri jobs https://review.opendev.org/691233 | |
| 08:29:38 | yonglihe | zuul failed while in merging. add version check by the way, so, need another round review: https://review.opendev.org/#/c/680928/ | |
| 11:35:58 | ttx | elmiko, dtantsur: SIG chair update proposed at https://review.opendev.org/#/c/691269/ | |
| 14:11:44 | efried | mordred, cmurphy, vishakha: I left some comments in https://review.opendev.org/#/c/669331/ -- if you can assuage my concerns about those two main things, I'll +A. | |
| 15:06:18 | mriedem | can i get a review on this simple osc change to use the sdk to list compute api extensions rather than novaclient? it's holding up using novaclient 16.0.0 in upper-constraints: https://review.opendev.org/#/c/691039/ | |
| 15:27:43 | openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/python-openstackclient master: [DOC]Fix opesntack command for removing router https://review.opendev.org/691375 | |
| 16:08:28 | openstackgerrit | James E. Blair proposed openstack/openstacksdk master: WIP: test extract_name https://review.opendev.org/691386 | |
| 16:11:22 | corvus | Shrews: ^ clarkb and i have been looking into that in -infra | |
| 16:12:37 | corvus | that's a failing unit test, with real data | |
| 16:12:53 | corvus | it produces: b'testtools.matchers._impl.MismatchError: !=:' | |
| 16:12:53 | corvus | b"reference = ['servers']" | |
| 16:12:53 | corvus | b"actual = ['dcaab5e32b234d56b626f72581e3644c', 'a82af292-bb2e-44a4-bf31-fb8145b66e6b']" | |
| 16:13:10 | corvus | and indeed, that matches output we're actually seeing on disk on the graphite server | |
| 16:46:01 | Shrews | corvus: is this a new failure? | |
| 16:46:16 | corvus | Shrews: we're starting to suspect current code may be okay | |
| 16:54:58 | corvus | yeah, it looks like this was probably fixed in 0.36.0 | |
| 17:05:33 | cmurphy | efried: responded | |
| 18:55:20 | mriedem | dtroyer: is it possible to write an osc command which extends command.Lister but doesn't provide output? | |
| 18:55:24 | mriedem | i know that's weird | |
| 18:56:04 | mriedem | trying to figure out how to solve this https://storyboard.openstack.org/#!/story/2006779 - maybe the answer is "add --no-provider to openstack resource provider allocation set" or create a new "openstack resource provider allocation unset --provider" command | |
| 19:22:34 | mriedem | nvm, looks like i'm going with the set --no-provider option | |
| 20:40:28 | openstackgerrit | Merged openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331 | |
| 20:44:35 | efried | cmurphy: that happened ^ | |
| 20:45:28 | cmurphy | \o/ | |
| #openstack-sdks - 2019-10-26 | |||
| 00:27:03 | openstackgerrit | Merged openstack/openstacksdk master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/689393 | |
| 10:36:23 | openstackgerrit | Xuan Yandong proposed openstack/shade master: Stop testing python2.7 https://review.opendev.org/691476 | |
| #openstack-sdks - 2019-10-28 | |||
| 08:42:59 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: CI: add ironic-python-agent-builder to the ironic job https://review.opendev.org/691630 | |
| 08:43:17 | dtantsur | mordred: ^^ | |
| 10:16:32 | openstackgerrit | Shogo Saito proposed openstack/openstacksdk master: Adding basic implementation for Accelerator(Cyborg) https://review.opendev.org/679914 | |
| 10:58:17 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add reset_interfaces argument to patch_node https://review.opendev.org/689414 | |
| 14:15:26 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add reset_interfaces argument to patch_node https://review.opendev.org/689414 | |
| 14:18:34 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Increase dogpile version for Py3.7 compatibility https://review.opendev.org/691680 | |
| 14:58:51 | openstackgerrit | Marc Gariépy (mgariepy) proposed openstack/python-openstackclient master: [DOC]Fix openstack command for removing router https://review.opendev.org/691375 | |
| 16:18:06 | efried | dtroyer: So as I'm working through these plugin docs issues, it has been in the back of my head: | |
| 16:18:34 | efried | Should the plugin projects build osc in a gate job, so they don't break it? | |
| 16:18:57 | efried | or maybe they already do, /me looks... | |
| 16:20:12 | efried | ah, I guess that's what openstackclient-plugin-jobs would be for? ... | |
| 16:22:01 | efried | ugh, that guy doesn't require novaclient or osc-placement. | |
| 16:22:22 | efried | also, it doesn't appear to build docs, which would be a useful add. | |
| 16:22:52 | efried | ...and it's marked as non-voting by default. | |
| 16:27:22 | efried | ...and the only thing it does right now is check for duplicate entry points. | |
| 16:27:38 | efried | okay, so tldr some of the infrastructure is there, but it hasn't really been fleshed out. | |
| 16:28:12 | dtroyer | efried: exactly. that was all supposed to come together in the openstackclient repo and never did | |
| 16:28:46 | dtroyer | IIRC stevemar was organizing all of that when IBM lured him to the dark side… | |
| 16:29:02 | efried | dtroyer: those IBM guys... | |
| 16:29:17 | efried | dtroyer: so what's next here? | |
| 16:30:24 | efried | like, if I want to do a thing (like make sure python-xclient doesn't break python-openstackclient's doc build with a bogus option help) it would be nice if I didn't have to drag that entire openstackclient infra effort into daylight first. | |
| 16:30:39 | dtroyer | efried: I think the cross-project plugin stuff still belongs in openstackclient, its purpose is to provide that integration for users, makes sense for the knowledge of plugins to all be there. | |