| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-03-18 | |||
| 14:50:15 | mordred | that | |
| 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:36 | mordred | yeah. and oslo.config can't have an option that's both a listopt and a stringopt | |
| 14:50:43 | mordred | ++ | |
| 14:50:45 | mordred | agree with that | |
| 14:51:24 | mordred | efried: so yeah - I think extracting that into a helper method/function is a great idea | |
| 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: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 | mordred: that part would work; what breaks is if that group didn't import ksa opts, then you get e.g. | |
| 18:43:27 | efried | oslo_config.cfg.NoSuchOptError: no such option valid_interfaces in group [cinder] | |
| 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 | |
| 13:51:27 | nullscan | but still no luck | |
| 13:51:52 | dtroyer | hmmm… it's giving me trouble too (I've never actually used it before) | |
| 13:52:13 | dtroyer | oh, duh, liiking at the code... | |
| 13:52:14 | nullscan | openstack: '--os-profile qqqq image list' is not an openstack command. See 'openstack --help'. Did you mean one of these? | |
| 13:52:17 | dtroyer | *looking | |
| 13:52:18 | nullscan | that is what i am getting | |
| 13:52:25 | dtroyer | do you have osprofiler installed? | |
| 13:52:32 | nullscan | yes i do | |
| 13:52:48 | dtroyer | if impporting osprofiler.profiler fails the option isn't defined | |
| 13:52:56 | nullscan | but from what i understand osprofiler is used only to get information from trackIDs | |
| 13:53:26 | nullscan | and the actual documentation does not list any other steps, just enable profiling in the services i want and run any command with --os-profile switch | |
| 13:53:35 | dtroyer | so if OSc is in a venv that venv also needs to have osprofiler in it, etc | |
| 13:54:37 | nullscan | no its not in venv, its installed along with the rest of my system | |
| 13:56:20 | dtroyer | what do you get with: python -m osprofiler | |
| 13:56:21 | nullscan | ah i just found it | |
| 13:56:40 | nullscan | nevemind, i had the python3 version of osc-lib installed instead of the python v2 ones | |
| 13:56:41 | nullscan | :) | |