Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-09-25
12:28:44 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting ironic-inspector job https://review.opendev.org/684729
12:28:45 dtantsur mordred: that was my thought as well
12:28:51 dtantsur although I'm not sure how to express it
12:29:00 dtantsur I wonder to run it on e.g. openstack/resource.py
12:29:44 mordred yeah - I think openstack/{connection,resource,proxy}.py should likely run the world - but those should also be rare at this point
12:30:16 mordred fwiw - I'm not opposed to these becomming voting if we figure out the right file matchers
12:30:23 mordred but one step at a time
12:32:21 dtantsur ++
12:32:48 dtantsur to use matchers, do we need to inherit the jobs? or can I do it in project.jobs.<>?
12:33:02 mordred dtantsur, Shrews, gtema_: the new hotness in ansible land is collections, and the intent is to split out modules from ansible core into externally managed collections. these can be hosted anywhere. so - I think we should host the openstack collection in opendev as a deliverable of the sdk project (since the overlap is very high)
12:33:16 dtantsur I very much agree with that
12:33:18 mordred dtantsur: you can do it in project.jobs.<>
12:33:23 dtantsur nice
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 dtantsur they're awkward to say the least
12:56:09 mordred totally
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 mordred: okay, here we go:
15:45:58 efried - Vanilla devstack running nova
15:45:58 efried - Clone cyborg, pull down 682565 (to get proper opt registration) and pip install -e it
15:45:58 efried - This /etc/cyborg/cyborg.conf: http://paste.openstack.org/show/779250/
15:45:58 efried - Run this script: http://paste.openstack.org/show/779251/
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 mordred efried: yeah. like - that is not output we want to see - but it makes me wonder how anything would be working
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: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

Earlier   Later