| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-03-18 | |||
| 14:43:18 | openstackgerrit | OpenStack Release Bot proposed openstack/osc-lib master: Update master for stable/stein https://review.openstack.org/644042 | |
| 14:43:43 | efried | mordred: yeah, nice, guess we'll see if it works, I'll recheck the dependents. | |
| 14:43:49 | mordred | cool | |
| 14:44:18 | mordred | efried: once we get a version of it that we're happy with for the downstream patches, I'll work on some unittests for it | |
| 14:45:46 | efried | mordred: any thoughts on the `valid_interfaces` debacle? | |
| 14:46:27 | mordred | efried: you mean the need to support valid_interfaces as an alternate way of saying interface? | |
| 14:47:10 | efried | mordred: Well, ksa adapter conf options have valid_interfaces and deprecate interface. And the thing we're doing here is accepting ksa adapter conf opts. | |
| 14:47:49 | efried | mordred: But the logic that converts the valid_interfaces conf opt into the internal `interface` thingy - which accepts a list IIUC - only lives in the load-from-conf ksa code | |
| 14:47:57 | mordred | heh. well - openstacksdk accepts a list or a scalar for interface | |
| 14:48:12 | efried | which is currently embedded in a factory method that can't easily be invoked from sdk code, from what I can see. | |
| 14:48:13 | mordred | yeah - valid_interfaces is really only a thing in oslo.config files - it's not the actual interface anywhere else | |
| 14:48:36 | efried | Proposal: factor the logic in ksa loading into a helper method that we can call. | |
| 14:49:06 | mordred | we could - except I don't want to deprecate interface in openstacksdk - so we'd have to make sure not to throw any warnings | |
| 14:49:49 | mordred | (interface is the better option name - I think the issue is that oslo.config doesn't support defining a single thing as list-or-single-value and that's why we had to add valid-interfaces right? | |
| 14:49:52 | efried | mordred: I refer to: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/adapter.py#L223-L232 | |
| 14:50:12 | mordred | AH | |
| 14:50:14 | efried | I think it's because we had already defined 'interface' as not a list opt? | |
| 14:50:15 | mordred | that | |
| 14:50:36 | mordred | yeah. and oslo.config can't have an option that's both a listopt and a stringopt | |
| 14:50:36 | efried | yeah, we don't need it to sling a warning. We just need to accept (and in fact prefer) valid_interfaces over interface in oslo conf. | |
| 14:50:43 | mordred | ++ | |
| 14:50:45 | mordred | agree with that | |
| 14:51:24 | efried | rn that ends with load_from_options which instantiates Adapter, which doesn't work in the Connection flow; so we have to pull that chunk into a method. | |
| 14:51:24 | mordred | efried: so yeah - I think extracting that into a helper method/function is a great idea | |
| 14:51:37 | mordred | efried: we should probably include line 220-22 too | |
| 14:51:53 | efried | yup | |
| 14:52:08 | efried | mordred: Okay, I can propose that, will tag ya. | |
| 14:52:39 | mordred | efried: cool | |
| 15:10:38 | dtroyer | brtknr: have you sorted your magnumclient problem yet? I started chasing something last week that led to it failing in some gate tests elsewhere late last week… | |
| 15:14:22 | brtknr | not yet, ive posted a story though: https://storyboard.openstack.org/#!/story/2005257 | |
| 15:28:41 | brtknr | dtroyer: also, it looks like there's already a fix: https://review.openstack.org/#/c/638077/4 | |
| 15:29:05 | brtknr | dtroyer: just tested the same command using the master branch and it seems happy now | |
| 15:37:46 | brtknr | mordred: would you be able to comment on this: https://review.openstack.org/#/c/643195/6 | |
| 15:39:12 | mordred | brtknr: seems fine to me! | |
| 15:39:22 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Do not disregard tags when updating stacks https://review.openstack.org/643195 | |
| 15:39:46 | mordred | brtknr: rebased it on top of the latest from gtema | |
| 15:40:29 | brtknr | mordred: thanks :) | |
| 15:40:48 | brtknr | mordred: didn't realise I was speaking to Monty Taylor | |
| 15:41:02 | brtknr | mordred: I'll be attending your talk at OpenInfra days :) | |
| 15:41:25 | dtantsur | The Monty Taylor. The one and only! | |
| 15:42:00 | mordred | uhoh. that means I might have to say things :) | |
| 15:42:05 | mordred | brtknr: which one? | |
| 15:42:46 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Factor Adapter conf-processing logic into a helper https://review.openstack.org/644251 | |
| 15:43:06 | brtknr | mordred: the workshop! | |
| 15:44:27 | mordred | sweet! well - hopefully that goes well and you don't all end up throwing rotten vegetables at me | |
| 15:45:28 | brtknr | mordred: you'll do fine I'm sure :) | |
| 15:45:53 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Factor Adapter conf-processing logic into a helper https://review.openstack.org/644251 | |
| 15:46:35 | efried | oh look, those get posted here too ^ | |
| 15:46:52 | mordred | efried: it's almost like these pieces all depend on each other | |
| 15:46:58 | efried | heh | |
| 17:49:31 | openstackgerrit | Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601 | |
| 17:49:49 | efried | mordred: Still need to test locally but ^ | |
| 17:50:22 | efried | ahcrap, forgot depends-on... | |
| 17:51:03 | openstackgerrit | Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601 | |
| 17:59:59 | mordred | efried: update looks good | |
| 18:39:43 | efried | mordred: It once again suffers the NoSuchOptError problem, and now I don't have a good way to fix it. Going to reinstate your conditional import thing, unless you think I shouldn't bother and should just trap Exception. | |
| 18:42:18 | mordred | efried: can you not do project_name in config? (I guess that didn't work out?) | |
| 18:43:27 | efried | oslo_config.cfg.NoSuchOptError: no such option valid_interfaces in group [cinder] | |
| 18:43:27 | efried | mordred: that part would work; what breaks is if that group didn't import ksa opts, then you get e.g. | |
| 18:43:28 | mordred | efried: it looks like a ConfigOpts is a collections.Mapping and defines __contains__ | |
| 18:43:39 | mordred | oh! different issue | |
| 18:44:00 | efried | mordred: so I would have to, like, load up the conf opts and query the group whether it has 'em before invoking the helper method, which seemed like horrible man. | |
| 18:45:35 | mordred | efried: we could have the ksa method do a hasattr before every get? is that a terrible idea? | |
| 18:45:36 | efried | I could put something in the helper to convert the exception to something standard, or return a bool, or... but all of that is really ugly too. | |
| 18:45:56 | mordred | efried: or we could just trap Exception in sdk until it becomes an issue | |
| 18:46:04 | efried | yup, that's what I'm setting up now. | |
| 18:50:18 | openstackgerrit | Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601 | |
| 18:50:22 | efried | mordred: thar she blows | |
| 18:50:56 | mordred | efried: I believe in her | |
| 18:51:29 | efried | now dep updates | |
| #openstack-sdks - 2019-03-19 | |||
| 09:51:29 | dtantsur | API SIG people, we seem to have some PTG space on Thu morning (and it does not conflict with ironic) | |
| 10:56:14 | openstackgerrit | Merged openstack/openstacksdk master: Fix wait_for_server docstring https://review.openstack.org/643955 | |
| 11:21:55 | openstackgerrit | Merged openstack/openstacksdk master: Add unit tests for connection.add_service https://review.openstack.org/643604 | |
| 13:37:33 | openstackgerrit | Merged openstack/openstacksdk master: Update master for stable/stein https://review.openstack.org/644016 | |
| 14:13:24 | openstackgerrit | Merged openstack/osc-lib master: Update master for stable/stein https://review.openstack.org/644042 | |
| 14:57:24 | openstackgerrit | Merged openstack/openstacksdk master: added support for binding:profile parameter in create_port/update_port https://review.openstack.org/642418 | |
| 14:57:25 | openstackgerrit | Merged openstack/openstacksdk master: baremetal: implement the correct update of the maintenance_reason field https://review.openstack.org/643639 | |
| 15:54:19 | openstackgerrit | Merged openstack/cliff master: add python 3.7 unit test job https://review.openstack.org/637766 | |
| 19:56:09 | dustinc | mordred: is there a way to define retry policy for sdk proxy? | |
| 19:56:27 | dustinc | related to my question just now in #openstack-ironic | |
| 19:56:46 | dustinc | I didn't see anything when I checked source real fast but might have missed it? 🤞 | |
| 21:50:52 | mordred | dustinc: there is a set of retry support in there that dtantsur|afk added for ironic but works for all the proxies | |
| 21:52:19 | mordred | dustinc: the basic idea is that the proxy knows what status codes are safe to retry, and then it's possible to configure for each service how many retries should be attempted for retriable status codes | |
| 21:52:40 | mordred | baremteal_status_code_retries would be the config setting you'd be looking for for ironic - and it defaults to 5 | |
| 22:05:09 | dustinc | oh thanks, I will go looking for that | |
| 22:06:01 | efried | nice! | |
| 22:06:39 | efried | ...before, that is. | |
| 22:09:50 | efried | dustinc: So you don't have to do a thing - it's already handled and defaulted appropriately. | |
| 22:09:54 | efried | \o/ | |
| 22:10:10 | dustinc | !!! | |
| 22:10:11 | openstack | dustinc: Error: "!!" is not a valid command. | |
| #openstack-sdks - 2019-03-20 | |||
| 13:41:16 | nullscan | hi | |
| 13:42:07 | nullscan | i have been trying to use the python openstack client with osprofiler but when i try to execute any command with --os-profile HMAC_KEY the client responds that the --os-profile switch is invalid | |
| 13:42:19 | nullscan | am i missing something ? | |
| 13:43:18 | nullscan | the client is installed using debian package and the version is 3.16.2 | |
| 13:43:32 | nullscan | the same happens when i try to use the latest version installed via pip | |
| 13:48:15 | dtroyer | nullscan: what version of osc-lib? That is where the —os-profile option is actually declared, although it has been around for a while... | |
| 13:48:52 | nullscan | 1.11.1 | |
| 13:50:53 | dtroyer | That is a global option so it needs to appear on the command line before the actual command: openstack —os-profile qaz server list | |
| 13:51:23 | nullscan | yes, i have actually been testing with openstack --os-profile qwerty image list | |