| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-09-27 | |||
| 05:06:24 | openstackgerrit | Shogo Saito proposed openstack/openstacksdk master: Adding basic implementation for Accelerator(Cyborg) https://review.opendev.org/679914 | |
| 06:14:01 | openstackgerrit | Merged openstack/openstacksdk master: Make proxy honor raise_exc in REST primitives https://review.opendev.org/684433 | |
| 07:28:40 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | |
| 07:46:29 | openstackgerrit | Shogo Saito proposed openstack/openstacksdk master: Adding basic implementation for Accelerator(Cyborg) https://review.opendev.org/679914 | |
| 08:45:21 | openstackgerrit | Bernhard M. Wiedemann proposed openstack/keystoneauth master: Make tests pass in 2020 https://review.opendev.org/681103 | |
| 10:24:32 | openstackgerrit | Neil Jerram proposed openstack/python-openstackclient master: Allow for router resource without 'routes' attribute https://review.opendev.org/685312 | |
| 10:49:10 | mordred | efried: cool. I'd be happy to backport to train. need to see what cmurphy thinks about backporting the ksa patch | |
| 10:49:23 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for authenticated endpoint_data https://review.opendev.org/685042 | |
| 11:40:09 | mordred | cmurphy, kmalloc: ^^ re that - I was just thinking - instead of that approach - what if we made the discovery call do try: request(auth=False) except 401: request(auth=True) - so that we'll still work with auth-needing discovery docs, but we'll gradually go back to the general noauth case as projects stop requiring it? | |
| 11:40:37 | mordred | it's a little less efficient for the auth requiring case, due to 2 calls | |
| 11:51:54 | efried | mordred: I think that's a reasonable idea. Log a shame warning for the service to unauth their discovery endpoints. Should happen just once per client conn, so hopefully not too intrusive, nah? | |
| 11:52:26 | efried | hacking in new kwargs, we kinda have to support them forever-ish, which is :( | |
| 11:59:13 | openstackgerrit | Bernhard M. Wiedemann proposed openstack/keystoneauth master: Make tests pass in 2020 https://review.opendev.org/681103 | |
| 12:02:09 | openstackgerrit | Stephen Finucane proposed openstack/osc-lib master: Add KeyValueAppendAction to osc-lib https://review.opendev.org/685328 | |
| 12:03:30 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Stop silently ignoring invalid 'server create --hint' options https://review.opendev.org/685329 | |
| 12:03:30 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Use 'KeyValueAppendAction' from osc-lib https://review.opendev.org/685330 | |
| 14:09:54 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Stop silently ignoring invalid 'server create --hint' options https://review.opendev.org/685329 | |
| 14:09:57 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Use 'KeyValueAppendAction' from osc-lib https://review.opendev.org/685330 | |
| 14:11:28 | openstackgerrit | Stephen Finucane proposed openstack/osc-lib master: Add KeyValueAppendAction to osc-lib https://review.opendev.org/685359 | |
| 14:12:36 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting ironic-inspector job https://review.opendev.org/684729 | |
| 14:13:35 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: DNM try breaking ironic-inspector https://review.opendev.org/685360 | |
| 14:23:27 | stephenfin | dtroyer, dtantsur: Could I have you folks look at https://review.opendev.org/#/c/685329/, please? | |
| 14:43:03 | dtantsur | stephenfin: I don't have +2 rights there, so will be of little help | |
| 14:43:22 | stephenfin | dtantsur: Ah, pity. No problem | |
| 16:05:57 | cmurphy | mordred: it's adding a feature so i think it's a no-go on backporting | |
| 16:36:43 | efried | cmurphy: what about the auto-fallback-on-401 option? | |
| 16:47:07 | cmurphy | efried: that might be fine | |
| 20:08:20 | efried | mordred, elmiko: I think I did it https://review.opendev.org/685181 | |
| 20:55:00 | elmiko | \o/ | |
| #openstack-sdks - 2019-09-28 | |||
| 02:38:31 | openstackgerrit | Merged openstack/keystoneauth master: Follow the PTI for docs https://review.opendev.org/685138 | |
| 02:40:11 | openstackgerrit | Merged openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | |
| #openstack-sdks - 2019-09-29 | |||
| 02:26:03 | openstackgerrit | pengyuesheng proposed openstack/os-service-types master: Update the constraints url https://review.opendev.org/685556 | |
| 06:59:32 | openstackgerrit | pengyuesheng proposed openstack/os-client-config master: Update the constraints url https://review.opendev.org/685581 | |
| #openstack-sdks - 2019-09-30 | |||
| 16:39:41 | mordred | efried: I'm just now back in sweden and having fun with timezone shifts, so I'm useless atm. however, when I wake up, I'm going to redo the keystoneauth patch to do the 401 retry | |
| 16:39:55 | efried | mordred: ack, ++ | |
| 16:40:26 | efried | mordred: I've clearly got some more work to do getting nova to allow versioned discovery unauth, but I think I've got it on the run. | |
| 21:10:29 | rm_work | with the SDK when I do: `user_cloud = admin_cloud.connect_as_project(user_project)`, and try to access `user_cloud.identity`, it fails because it can't determine the identity api version, but `admin_cloud.identity` works fine. is something not getting copied over correctly? | |
| 21:10:51 | rm_work | ^^ looking into whether domain isn't getting set properly -- though it's weird that it works on one but not the other | |
| 21:11:36 | efried | rm_work: stack trace or log msg? | |
| 21:12:02 | rm_work | yeah sec, it's via another app using it so i'll dig out the exact response | |
| 21:13:11 | rm_work | trying to trace in | |
| 21:13:56 | rm_work | the high level response is: `NotSupported('The identity service for : exists but does not have any supported versions.',)` | |
| 21:14:59 | rm_work | ok well i did actually find out that i was wrong | |
| 21:15:04 | rm_work | it didn't work on admin_cloud either | |
| 21:15:08 | rm_work | so that makes more sense | |
| 21:15:28 | rm_work | and means it's not an issue with copying something, so much as just not being able to set the version correctly | |
| 21:15:56 | rm_work | which PROBABLY means that I just need to have some other arg set in the config when I pass it to the connection | |
| 21:16:06 | efried | wanna show me your connection init? | |
| 21:16:25 | rm_work | finding it | |
| 21:17:14 | rm_work | it's done via `connection.Connection(config=config.get_one(argparse=options))` | |
| 21:17:56 | rm_work | `config` is an os_client_config.config.OpenStackConfig object | |
| 21:18:13 | rm_work | so, probably "options" needs to have an identity-version set | |
| 21:19:25 | efried | Gotta run for ~20 mins, will check back... | |
| 21:31:43 | rm_work | yeah ok so basically there needs to be a way to specify "--os-identity-api-version" in the argparser that's passed in to the config that's then passed to the SDK | |
| 21:32:07 | rm_work | which wasn't happening (that isn't apparently part of the os_client_config arguments or the auth-module options :/ | |
| 21:32:30 | rm_work | which ... seems like maybe a bug there but for now i'll just add that arg explicitly in the app and that does it | |
| 21:32:51 | rm_work | (allows me to even pass that to the application in question) | |
| 21:33:48 | rm_work | appreciate the prodding, trying to replicate some stuff to paste it to you was exactly what i needed :D | |
| 21:41:23 | Shrews | rm_work: you can set identity api version in your clouds.yaml file, or an environment variable | |
| 21:42:28 | Shrews | there's a small blurb about that in the last paragraph of https://docs.openstack.org/openstacksdk/latest/user/config/configuration.html#site-specific-file-locations | |
| 21:48:55 | efried | rm_work: also noting that os-client-config is deprecated :) | |
| 21:49:19 | efried | or at least superseded | |
| 21:52:07 | dtroyer | yes, o-c-c has been absorbed into the SDK directly. It is now just a wrapper so it should be a good example of how to use SDK directly | |
| 21:53:04 | rm_work | Shrews: yeah but I have to account for the ability for people to not use clouds.yaml | |
| 21:53:08 | rm_work | which is where this came up | |
| 21:53:26 | rm_work | hmmm ok so I should stop importing it | |
| 21:53:32 | rm_work | i'm updating ospurge | |
| 21:54:18 | rm_work | dtroyer: is there an example/doc somewhere about how to replace usage of os_client_config ? | |
| 21:58:13 | dtroyer | that is what I meant about looking at o-c-c itself, it now does exactly that, translate to the SDK bits. Much of it is unchanged just in new module namespaces, but some did and I don't recall what that was offhand | |
| 22:01:11 | rm_work | ah k | |
| 22:04:39 | rm_work | ok yeah so i got rid of the os_client_config import | |
| 22:04:48 | rm_work | replaced it with an import of openstack.config.loader | |
| 22:05:06 | rm_work | and replaced `os_client_config.OpenStackConfig()` with `loader.OpenStackConfig()` | |
| 22:05:10 | rm_work | and it just... works | |
| 23:10:09 | efried | \o/ | |
| 23:32:12 | openstackgerrit | Steve Baker proposed openstack/openstacksdk master: Support vendor data in configdrive building https://review.opendev.org/683842 | |
| #openstack-sdks - 2019-10-01 | |||
| 06:47:56 | bverschueren_ | if someone could have a look at https://review.opendev.org/#/c/682909/ ? thx | |
| 08:38:02 | mordred | rm_work: yay! | |
| 08:41:02 | cdent | I first read that as "rm work ; yay!" which would be pretty yay. | |
| 08:48:01 | dtantsur | rm -rf | |
| 08:51:28 | mordred | ++ | |
| 08:51:51 | mordred | cdent: does the system continue working overall with that? like - if I just rm work, do I still get paychecks? | |
| 08:52:35 | cdent | mordred: in my pretty little mind it does | |
| 08:53:53 | mordred | cdent: I like the workings of that mind | |
| 08:54:16 | cdent | it's a very generous mind | |
| 08:54:22 | dtantsur | just copy paycheck somewhere before rm -rf | |
| 08:54:46 | mordred | dtantsur: ++ | |
| 08:54:53 | cdent | it's a fifo that you can read from as needed | |
| 08:56:00 | mordred | dtantsur: wow. vendor_data2.json | |
| 08:56:04 | mordred | dtantsur: because that's a filename | |
| 08:56:11 | dtantsur | haha | |
| 08:57:30 | mordred | heaven forbid we put any sort of versioning into the file content itself. | |
| 09:12:55 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Fetch discovery documents with auth when needed https://review.opendev.org/685042 | |
| 09:13:17 | mordred | efried, kmalloc, cmurphy: ^^ that's a simpler version and I think should be backportable | |
| 10:58:42 | openstackgerrit | Merged openstack/openstacksdk master: Support vendor data in configdrive building https://review.opendev.org/683842 | |
| 11:06:27 | openstackgerrit | Merged openstack/openstacksdk master: Add a non-voting ironic-inspector job https://review.opendev.org/684729 | |
| 15:15:07 | mordred | efried: you'll be happy to know that the 401 fix for keystoneauth, in additional to uncovering a different issue with swift (easy to fix) also uncovered the fact that, since we weren't ACTUALLY doing version discovery for nova before, we were falling back to 2.1 in sdk in all cases (which was actually our intention - we wanted to opt-in to newer behavior as we supported it) - but we were falling back | |
| 15:15:09 | mordred | to it accidentally- so with discovery fixed, sdk is getting newest rather than oldest microversion by default - and the change to flavor broke sdk | |
| 15:15:11 | mordred | efried: yay for testing | |
| 15:19:33 | mordred | gtema: didn't we have code in sdk that was supposed to cause sdk to skip trying to do version discovery for swift? | |