| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-09-25 | |||
| 12:34:02 | mordred | I'll get some patches up today for project creation and initial content import and whatnot | |
| 12:34:29 | dtantsur | great! then we can add a bifrost job to verify ironic bits | |
| 12:55:19 | mordred | dtantsur: it's going to be some amount of fun for bifrost - because the module names are going to change | |
| 12:56:03 | dtantsur | mordred: can we also re-engineer ironic modules then? | |
| 12:56:09 | mordred | totally | |
| 12:56:09 | dtantsur | they're awkward to say the least | |
| 12:56:27 | dtantsur | and we should be doing os_baremetal_node, not os_ironic, etc, etc | |
| 12:56:38 | mordred | I'm going to start by importing the existing modules as they are - but I figure since the modules names are changing anyway, it's a good time to rework stuff we're annoyed by | |
| 12:56:51 | dtantsur | \o/ | |
| 12:57:03 | mordred | new names will be like 'openstack.cloud.ironic_node' instead of os_ironic_node - fwiw | |
| 12:57:38 | dtantsur | openstack.cloud.baremetal_node ;) | |
| 12:58:07 | mordred | ++ | |
| 12:59:30 | gtema | oh cool. I like the idea. Let's do this | |
| 14:02:51 | efried | mordred: Given that nova's versioned endpoint is requiring auth (I can fix that, but it's going to take time before we can rely on it), I'm seeing odd behavior in _validate_proxy. | |
| 14:03:26 | efried | in that it *seems* as though it's not using the Connection's auth when querying that URI | |
| 14:03:31 | efried | because I'm getting a 401 | |
| 15:33:24 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Add collections metadata info https://review.opendev.org/684787 | |
| 15:33:25 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Import modules from ansible/ansible https://review.opendev.org/684788 | |
| 15:33:34 | mordred | mnaser, dtantsur|afk: ^^ | |
| 15:34:06 | mordred | efried: ah. hrm. maybe there is an issue there we need to fix :( | |
| 15:34:33 | efried | mordred: let me resurrect the repro steps from yesterday... | |
| 15:34:39 | mordred | mnaser, dtantsur|afk: we obviously need to add some zuul jobs and stuff | |
| 15:36:27 | cdent | "add some zuul jobs and stuff" is my mantra | |
| 15:39:22 | Shrews | mordred: what is the proposal for keeping the modules in sync (for the time being) between a-c-o and ansible/ansible? | |
| 15:41:42 | mordred | Shrews: mnaser and I were discussing just not doing that. saying that the a/a modules are good for 2.9 - and starting with 2.10 we just recommend people install the collection | |
| 15:42:11 | mordred | Shrews: and basically stopping accepting PRs in devel branch or something | |
| 15:42:14 | mnaser | i think it's also neat if we added some sort of warning to repoint people too | |
| 15:42:23 | mnaser | ^ yeah and we should kill all issues/prs/etc | |
| 15:43:00 | Shrews | are collections compatible with older versions of ansible? | |
| 15:43:07 | mnaser | nope :< | |
| 15:43:26 | Shrews | then i don't see how you can just tell people to install the collections | |
| 15:43:44 | mnaser | well i guess moving forwards the os_server will be frozen | |
| 15:43:53 | mnaser | but yeah i guessif someone jumps from 2.5 to 2.10 | |
| 15:43:58 | mnaser | it would magically disappear | |
| 15:44:04 | Shrews | and not accept bug fixes? i don't see that going over well | |
| 15:44:08 | mordred | Shrews: I think it's that older versions of ansible already have the built in modules in them | |
| 15:44:32 | mordred | so we don't really need to do anything with those - it's only for ansible 2.10 and greater, which support collections, that anything would happen to the modules in a/a | |
| 15:45:41 | mordred | Shrews: I think accepting stable branch bugfixes is totally a legit thing to do | |
| 15:45:58 | efried | - Run this script: http://paste.openstack.org/show/779251/ | |
| 15:45:58 | efried | - This /etc/cyborg/cyborg.conf: http://paste.openstack.org/show/779250/ | |
| 15:45:58 | efried | - Clone cyborg, pull down 682565 (to get proper opt registration) and pip install -e it | |
| 15:45:58 | efried | - Vanilla devstack running nova | |
| 15:45:58 | efried | mordred: okay, here we go: | |
| 15:45:59 | efried | - Observe this output: http://paste.openstack.org/raw/779252/ | |
| 15:46:51 | mordred | efried: uh... | |
| 15:46:55 | Shrews | mordred: so i return to my original question... how are we going to keep those fixes in sync in both places? | |
| 15:47:17 | efried | mordred: just skip to the last paste :) | |
| 15:47:57 | Shrews | i'm guessing this is a problem all external collections are going to have. just wondering if there was talk about that process | |
| 15:48:03 | mordred | Shrews: magic? | |
| 15:48:28 | mordred | Shrews: I think we should be able to make a zuul job that runs on changes to those that proposes patches to gerrit | |
| 15:49:12 | efried | mordred: the last ten lines or so of that output show the 401 for auth required for the versioned endpoint, which shouldn't be happening because a) endpoint shouldn't require auth, but even if it does, b) we're using a proper auth to set up the Connection (which can be proven by using strict_proxies=False and using the resulting proxy to do real things). | |
| 15:49:12 | mordred | efried: yeah. like - that is not output we want to see - but it makes me wonder how anything would be working | |
| 15:49:44 | efried | mordred: The most obvious guess is that version discovery is not using the auth I pass in. | |
| 15:49:51 | efried | it's using... some other auth, or no auth. | |
| 15:49:59 | efried | that would cause this behavior. | |
| 15:50:16 | efried | edit: this particular version discovery path | |
| 15:51:28 | mordred | yeah. something seems very strange there. I need to dig in to that - but it's lunchtime | |
| 15:51:41 | efried | ack | |
| 15:52:02 | mordred | that is definitely not behavior I expect :( | |
| 15:53:12 | efried | I'll open a story for this, I suppose. | |
| 15:53:21 | mordred | Shrews: it's a topic we definitely need to wrap our heads around | |
| 16:12:56 | efried | mordred: https://storyboard.openstack.org/#!/story/2006614 | |
| 21:13:23 | efried | dtroyer: Whom can we bug to get https://review.opendev.org/#/c/664393/ merged? It's a dep of a couple of nova doc patches that would be nice to get into rc1 | |
| 21:14:04 | efried | uhm, then again... I suppose that doesn't really help us unless osc is going to re-release for train, huh? | |
| 21:19:33 | dtroyer | efried: right, I was holding off on new features until I was reasonably sure we didn't need a 4.0.1. But that means 4.0.0 is stable/train | |
| 21:19:55 | efried | okay, never mind then | |
| 21:20:01 | efried | <waves hand> I was never here | |
| 21:20:14 | dtroyer | but that really only matters for distro packagers… everyone else should be using current anyway :) | |
| 21:21:46 | efried | psshhht, distros <rolls eyes> | |
| #openstack-sdks - 2019-09-26 | |||
| 05:31:49 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | |
| 05:37:14 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | |
| 05:50:06 | openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | |
| 05:54:48 | openstackgerrit | pengyuesheng proposed openstack/shade master: Update the constraints url https://review.opendev.org/684929 | |
| 10:51:13 | mordred | efried: I have reproduced your issue against vexxhost | |
| 11:28:38 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for authenticated endpoint_data https://review.opendev.org/685042 | |
| 11:29:45 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use auth for discovery https://review.opendev.org/685043 | |
| 11:31:18 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use auth for discovery https://review.opendev.org/685043 | |
| 11:31:33 | mordred | efried: ^^ those two should fix the issue | |
| 11:32:03 | mordred | although I think there's an improvement I want to make ... | |
| 11:38:14 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use auth for discovery https://review.opendev.org/685043 | |
| 11:38:21 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for authenticated endpoint_data https://review.opendev.org/685042 | |
| 11:38:41 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Send auth tokens for discovery https://review.opendev.org/685043 | |
| 11:39:23 | mordred | efried, dtantsur, cmurphy, kmalloc: ^^ that should be ready for review now and should fix efried's issue | |
| 11:39:44 | dtantsur | okie, will try to get to it (release time in the ironic world) | |
| 11:40:56 | mordred | dtantsur: it's ... you're going to enjoy it I'm certain | |
| 11:41:08 | mordred | and by enjoy, I think I mean "will be sad" | |
| 11:41:11 | dtantsur | I've been enjoying it for a few weeks already :D | |
| 11:41:45 | mordred | yay for enjoyment! :) | |
| 13:42:08 | efried | mordred: Thanks! I'll take a look at that very soon. | |
| 13:42:32 | efried | mordred: were you planning to cut another Train release? | |
| 13:43:09 | efried | I guess that's a backportable bugfix in any case. (Is SDK on a cycle release model anyway?) | |
| 15:01:47 | brtknr | mordred: would you mind blessing this: https://github.com/ansible/ansible/pull/62874 | |
| 15:02:28 | brtknr | or anyone else for that matter | |
| 15:54:32 | efried | mordred: Do we have a doc somewhere containing the edict that discovery endpoints should not require auth? | |
| 15:55:06 | efried | elmiko: perhaps that's something you would also know? | |
| 16:01:00 | elmiko | API SIG office hour is now starting | |
| 16:01:00 | cdent | it may be in that discovery doc that languished without merging for so long | |
| 16:01:08 | cdent | not sure if it ever did | |
| 16:01:41 | elmiko | efried: i feel like there /may/ be some guidance about that, but iirc it is in the languishing doc that cdent is referring to | |
| 16:01:52 | elmiko | at the least, i think it came up for discussion | |