| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-03-05 | |||
| 18:39:19 | mordred | brtknr: https://docs.openstack.org/api-ref/container-infrastructure-management/?expanded=list-all-baymodels-detail#id34 | |
| 18:39:46 | mordred | brtknr: GET /baymodels returns {'baymodels': [ {}, {} ] } | |
| 18:39:58 | mordred | so we need the baymodels= there | |
| 18:40:49 | mordred | I agree - inthe response for the POST - there should be no top level baymodel or baymodels | |
| 18:40:56 | mordred | nor in teh request | |
| 18:41:06 | mordred | https://docs.openstack.org/api-ref/container-infrastructure-management/?expanded=create-new-baymodel-detail#id25 | |
| 18:55:25 | brtknr | mordred: nice to see it confirmed | |
| 18:56:32 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix service_type test for magnum in gate https://review.opendev.org/711533 | |
| 18:56:52 | mordred | brtknr, Shrews: ^^ that should fix the magnum tests in the gate | |
| 18:56:52 | openstackgerrit | Bharat Kunwar proposed openstack/openstacksdk master: Normalise create_coe_cluster{,_template} results https://review.opendev.org/711526 | |
| 18:57:54 | brtknr | Wow thats a mouthful, I did not know that | |
| 18:57:55 | mordred | (711533 that is) | |
| 18:58:11 | Shrews | mordred: are we taking bets that the tests still work? ;) | |
| 18:58:21 | mordred | Shrews: nope! | |
| 18:58:35 | mordred | Shrews: but - they should at least RUN now and fail as tests :) | |
| 18:58:40 | mordred | brtknr: https://service-types.openstack.org/service-types.json | |
| 18:59:15 | mordred | or more specifically: https://opendev.org/openstack/service-types-authority/src/branch/master/service-types.yaml#L41 | |
| 18:59:38 | mordred | for normal users it should work to refer by official type or any of the aliases | |
| 18:59:46 | mordred | but we wrote the has_service to be strict :) | |
| 19:08:47 | brtknr | mordred: ouch | |
| 19:30:57 | mordred | yah | |
| 20:56:03 | umbSublime | Hey another quick question related to os-client-config. I just noticed from the README that it was superceded by the sdk, but I'm not sure how I can reproduce even a basic call like `os_client_config.OpenStackConfig().get_all_clouds()` I just want to have an object representing the clouds.yaml without having to parse it myself | |
| 21:02:19 | umbSublime | derr nvm just found it under openstack.config ... | |
| 21:08:14 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Set max_microversion to 2.53 for hypervisors https://review.opendev.org/711294 | |
| 21:09:23 | mordred | umbSublime: \o/ | |
| 21:09:42 | mordred | umbSublime: yeah - os_client_config moved to openstack.config ... perhaps we should update the readme to point more specifically | |
| 21:11:16 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Normalise create_coe_cluster{,_template} results https://review.opendev.org/711526 | |
| 21:12:27 | mordred | Shrews, brtknr: my patch above fixed the magnum functional tests - so I added a depends-on between brtknr's patch and mine so we can make sure that doesnt' break functional tests (I'm pretty sure it's ok - but we have green functional tests - so why not!) | |
| 21:32:22 | umbSublime | tbh If I would've searched 2 minutes more before posting, it would've been fine. It wouldn't hurt though. (btw thanks for updating topic on the chan) | |
| 21:56:55 | umbSublime | is cliff also maintained by the SDK team ? | |
| 22:13:24 | Shrews | no | |
| 22:14:09 | mordred | Shrews: well ... | |
| 22:14:56 | mordred | umbSublime: there is a proposal up to merge the sdk and osc teams - so assuming that goes through (there is currently no dissent) the answer becomes "yes" | |
| 22:15:43 | Shrews | osc team maintained cliff? heh | |
| 22:16:06 | Shrews | TIL | |
| 22:22:32 | brtknr | mordred Shrews: thanks for the merge! | |
| 22:22:44 | brtknr | glad you fixed the ci too! | |
| 22:23:43 | brtknr | What does the magnum job do out of interest? | |
| 22:24:09 | mordred | brtknr: it makes sure that there is a magnum running in the devstack and then runs the openstacksdk functional tests against that devstack | |
| 22:24:24 | mordred | brtknr: so - you can totally add magnum related functional tests and they'll run in that job | |
| 22:25:16 | mordred | obviously nested virt makes doing a _lot_ in functional tests somewhat heavy weight - so I don't think we have many magnum related functional tests today | |
| #openstack-sdks - 2020-03-06 | |||
| 00:23:49 | openstackgerrit | Merged openstack/openstacksdk master: Fix service_type test for magnum in gate https://review.opendev.org/711533 | |
| 00:27:41 | openstackgerrit | Merged openstack/openstacksdk master: Normalise create_coe_cluster{,_template} results https://review.opendev.org/711526 | |
| 05:44:52 | brtknr | mordred: hmm I don’t see any functional tests for magnum, am I looking in the wrong place? | |
| 05:44:52 | brtknr | mordred: hmm I don’t see any functional tests for magnum, am I looking in the wrong place? | |
| 07:04:40 | vishakha | amotoki: Regarding https://review.opendev.org/#/c/697444/. Could you please look over the cmurphy's suggestion, so that I can proceed further on the patch. | |
| 07:42:43 | LarsErikP | Hi guys! Spoke to johnsom over at #openstack-horizon about a thread issue i'm having with octavia-dashboard in stein. He was pretty sure this patch fixed it: https://review.opendev.org/#/c/695795/ | |
| 07:44:05 | LarsErikP | But I'm a little bit confused, because: I have one horizon-server running 16.0.0 (train) and sdk 0.36.0 which does not have the issue; and one server running 15.1.1 and sdk 0.26.0 which has this issue. The confusion is, neither of those SDK versions has the aforementioned patch included... | |
| 07:45:01 | LarsErikP | The issue is basically that leaving a browser open at the list of load-balancers will accumulate threads on the horizon server, and eventually kill the server.. | |
| 07:59:21 | LarsErikP | and; I tried to "ninja-apply" that patch into my horizon-server running 15.1.1 and SDK 0.26.0, but I stil got the issue - so I guess there is more to this? | |
| 09:19:51 | openstackgerrit | Tom Stappaerts proposed openstack/openstacksdk master: Support for stateless security groups https://review.opendev.org/711513 | |
| 10:15:06 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | |
| 10:15:42 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | |
| 10:31:47 | openstackgerrit | Tom Stappaerts proposed openstack/python-openstackclient master: Support for stateless security groups https://review.opendev.org/711515 | |
| 11:16:58 | LarsErikP | did some more testing. Seems like SDK 0.27 acutally will fix the issue. Who should I nudge to get that released in UCA? | |
| 11:22:19 | frickler | LarsErikP: you can create a bug against UCA in lp or ping coreycb+jamespage in #ubuntu-server | |
| 11:29:41 | LarsErikP | frickler: fantastic! thaks :) | |
| 11:29:44 | LarsErikP | thanks* | |
| 11:29:56 | LarsErikP | god damn. Can't even spell easy words now... #friday | |
| 12:28:06 | openstackgerrit | Tom Stappaerts proposed openstack/openstacksdk master: Support for stateless security groups https://review.opendev.org/711513 | |
| 12:54:25 | openstackgerrit | Tom Stappaerts proposed openstack/python-openstackclient master: Support for stateless security groups https://review.opendev.org/711515 | |
| 13:57:38 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Add retired and retired_reason fields to baremetal node https://review.opendev.org/711032 | |
| 13:58:20 | openstackgerrit | James Denton proposed openstack/openstacksdk stable/train: Include "fields" to "SecurityGroup" query parameters https://review.opendev.org/711622 | |
| 14:01:12 | amotoki | vishakha: cmurphy's suggestion works for me. | |
| 14:01:48 | vishakha | amotoki: Thanks for the response. | |
| 14:02:09 | amotoki | vishakha: I originally advised to follow the OSC convention for boolean opts, then you proposed --enable/disable-ignore-foo, I suggested to avoid verb-verb style and finally you proposed the current version. | |
| 14:02:28 | amotoki | vishakha: during this, the route went wrong a bit.... :( | |
| 14:03:06 | vishakha | amotoki: Yes I agree. | |
| 14:03:45 | openstackgerrit | Tom Stappaerts proposed openstack/openstacksdk master: Support for stateless security groups https://review.opendev.org/711513 | |
| 14:16:20 | mordred | brtknr: openstack.tests.functional.cloud.test_cluster_templates.TestClusterTemplate and openstack.tests.functional.cloud.test_magnum_services.TestMagnumServices ... so not many :) | |
| 15:16:57 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: Extract check temp_url_key logic https://review.opendev.org/710973 | |
| 15:16:58 | openstackgerrit | Riccardo Pittau proposed openstack/openstacksdk master: [WIP] Import generate_temp_url from swiftclient https://review.opendev.org/711640 | |
| 16:51:05 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9 https://review.opendev.org/711471 | |
| 16:51:58 | umbSublime | Hi again, bouncing back on the issue we fixed related to microversion on the compute.hypervisor and the work-around of adding `compute_api_version: <target_MV>` in clouds.yaml. I noticed it works as expected with the SDK, but the OSC doesn't seem to pick it up. (I still get id=type(int) and not id=type(uuid)). I only get the header `-H "X-OpenStack-Nova-API-Version: 2.53"` added when I use --os-compute-api-version=2.53 | |
| 16:56:51 | umbSublime | `openstack 5.0.0` | |
| 17:51:44 | mordred | umbSublime: well - joy | |
| 17:53:02 | umbSublime | no biggie, just wanted to point it out | |
| 17:55:15 | mordred | umbSublime: that'll be a little longer to sort out - but it's actually an important one to do. I'm goign to see if there's a quicker way to plumb those through than what my hunch is | |
| 17:56:00 | umbSublime | Yah I figured this one would be a bit trickier. Would you like me to file a bug for tracking this ? | |
| 17:57:49 | mordred | sure! | |
| 17:59:18 | umbSublime | I'll get to it a bit later today ;) | |
| 18:00:48 | umbSublime | I'm building my first cliff app right now, and I'm not sure how to access to apps parser from within a command (or if that's even possible). cmd_obj.get_parser() only returns the parser local to the command. | |
| 18:06:52 | umbSublime | Welp, again I asked a question too quick. I can use cmd_obj.app_args to access the parsed args | |
| 18:08:43 | mordred | woot! | |
| 18:10:24 | umbSublime | I want to centralize all my "operator" scripts under a CLI. I'm still debating if I want a standalone CLI, or an OSC plugin. In any case transitioning for a stand-alone CLI to OSC plugin seems very easy | |
| 18:17:24 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Honor api config version for compute calls https://review.opendev.org/711732 | |
| 18:18:07 | mordred | umbSublime: ^^ that fixes the _specific_ issue - but obviously this is a thing that should be generalized and apply to all of the osc config reading | |
| 18:43:53 | rm_work | mordred: did chances for a release get sidetracked due to a bug yesterday? :D | |
| 18:44:42 | mordred | rm_work: yeah. I think we've landed all the patches now - but I think it'll be first thing monday before the release team will cut a release (they don't tend to release on friday) | |
| 18:44:52 | rm_work | kk, WFM | |
| 18:55:55 | umbSublime | mordred, I applied the patch, but I still get `-H "X-OpenStack-Nova-API-Version: 2.1` | |
| 19:01:19 | mordred | umbSublime: boo. it worked for me locally ... | |
| 19:01:49 | umbSublime | Should we populate L40's _compute_api_version. If not it will always evaluate to None | |
| 19:03:49 | mordred | no - I think it gets populated in check_api_version | |
| 19:04:02 | umbSublime | Is it this call that looks in the config `instance._cli_options.get_default_microversion(API_NAME)`? | |
| 19:05:02 | mordred | yeah - that one and the one before it - get_api_version will get it if you set compute_api_version and get_default_microversion gets compute_default_microversion | |
| 19:05:30 | mordred | I'm curious why it didn't work for you but it did work for me | |
| 19:05:52 | mordred | do you have any env vars set like OS_COMPUTE_API_VERSION or anything? | |
| 19:06:35 | umbSublime | I have this is my config "compute_api_version: 2.53", no Env vars | |
| 19:07:27 | umbSublime | I fall in the `if` on L53 | |