| *** goldyfruit_ has joined #openstack-sdks | 00:07 | |
| *** goldyfruit_ has quit IRC | 00:33 | |
| *** goldyfruit has joined #openstack-sdks | 00:35 | |
| openstackgerrit | guang-yee proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | 00:52 |
|---|---|---|
| *** mriedem has quit IRC | 01:00 | |
| *** goldyfruit has quit IRC | 01:00 | |
| *** goldyfruit has joined #openstack-sdks | 01:26 | |
| *** factor has joined #openstack-sdks | 02:07 | |
| *** ricolin has joined #openstack-sdks | 02:47 | |
| *** black_ant has joined #openstack-sdks | 03:13 | |
| *** goldyfruit has quit IRC | 04:24 | |
| *** enriquetaso has joined #openstack-sdks | 04:38 | |
| *** enriquetaso has quit IRC | 04:57 | |
| *** zbr|ruck has quit IRC | 05:43 | |
| *** zbr has joined #openstack-sdks | 05:45 | |
| openstackgerrit | Shogo Saito proposed openstack/openstacksdk master: Adding basic implementation for Accelerator(Cyborg) https://review.opendev.org/679914 | 05:56 |
| *** e0ne has joined #openstack-sdks | 06:16 | |
| *** enriquetaso has joined #openstack-sdks | 06:32 | |
| *** slaweq has joined #openstack-sdks | 06:55 | |
| *** e0ne has quit IRC | 06:56 | |
| *** gtema has joined #openstack-sdks | 07:11 | |
| *** enriquetaso has quit IRC | 07:34 | |
| *** enriquetaso has joined #openstack-sdks | 07:35 | |
| *** gtema_ has joined #openstack-sdks | 07:38 | |
| *** e0ne has joined #openstack-sdks | 07:39 | |
| *** jpena|off is now known as jpena | 07:46 | |
| *** ralonsoh has joined #openstack-sdks | 07:51 | |
| *** jpich has joined #openstack-sdks | 08:05 | |
| *** jangutter has joined #openstack-sdks | 08:20 | |
| *** cdent has joined #openstack-sdks | 08:20 | |
| *** e0ne has quit IRC | 08:28 | |
| *** zbr is now known as zbr|ruck | 08:40 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/keystoneauth master: Generate pdf documentation https://review.opendev.org/682272 | 09:02 |
| *** gtema has quit IRC | 09:04 | |
| *** gtema has joined #openstack-sdks | 09:07 | |
| *** dtantsur has joined #openstack-sdks | 09:12 | |
| *** gtema has quit IRC | 09:12 | |
| *** ttsiouts has joined #openstack-sdks | 09:12 | |
| *** e0ne has joined #openstack-sdks | 09:31 | |
| *** gtema_ has quit IRC | 09:33 | |
| *** gtema has joined #openstack-sdks | 09:33 | |
| *** gtema_ has joined #openstack-sdks | 09:51 | |
| *** gtema_ has quit IRC | 09:59 | |
| *** enriquetaso has quit IRC | 10:02 | |
| *** gtema_ has joined #openstack-sdks | 10:26 | |
| *** enriquetaso has joined #openstack-sdks | 10:30 | |
| *** gtema_ has quit IRC | 10:39 | |
| *** cdent has quit IRC | 10:41 | |
| *** ttsiouts has quit IRC | 11:01 | |
| mordred | efried: no - i version discovery endpoints should never require auth | 11:21 |
| mordred | gtema: nice photo :) | 11:22 |
| *** ttsiouts has joined #openstack-sdks | 11:22 | |
| *** jpena is now known as jpena|lunch | 11:25 | |
| frickler | mordred: iiuc efried was talking about the versioned endpoint. the discovery endpoint would seem to be just http://192.168.218.28/compute/ in his example | 11:31 |
| *** gtema_ has joined #openstack-sdks | 11:32 | |
| *** goldyfruit has joined #openstack-sdks | 11:32 | |
| *** gtema_ has quit IRC | 11:32 | |
| *** cdent has joined #openstack-sdks | 11:37 | |
| *** gtema_ has joined #openstack-sdks | 11:38 | |
| mordred | frickler: yeah- but that versioned endpoint is still a version discovery endpoint (it's where the versioned discovery doc would be located) - and should never require auth | 11:39 |
| mordred | efried: ^^ | 11:39 |
| frickler | mordred: efried: hmm, then at least cinder, glance and nova are broken in that respect | 11:48 |
| frickler | the only service where I see this working is identity | 11:49 |
| cdent | placement does no auth for the version doc | 11:50 |
| cdent | or if it doesn't, then something is broken | 11:51 |
| frickler | cdent: placement seems special in that it runs directly under the main endpoint afaict, i.e. there is no /placement/v1.0/ path | 11:55 |
| cdent | that's correct | 11:55 |
| * cdent wishes everything else was the same | 11:55 | |
| cdent | version numbers ins URLs are icky | 11:56 |
| mordred | frickler: *awesome* | 11:58 |
| mordred | frickler: well, I guess it probably doens't break sdk most of the time because we're usually set up with auth anyway | 11:58 |
| mordred | but in a perfect world no version discovery docs should be auth protected - as they do not present any user-specific or special info | 11:59 |
| mordred | they should also be naturally aggressively cacheable | 11:59 |
| cdent | i was surprised to discover on the ML recently that some entries in the service catalog still have project ids? | 12:01 |
| cdent | (in the urls) | 12:01 |
| mordred | yeah. some people still haven't gotten rid of those | 12:11 |
| mordred | keystoneauth version discovery knows how to strip them and add them back when it's looking for version discovery docs - so I stopped caring, even though it's ultimately a tragedy | 12:11 |
| *** goldyfruit has quit IRC | 12:16 | |
| dtantsur | mordred: an interesting topic came up on #release. with more and more projects relying on openstacksdk for x-project interaction, how do we prevent breakages? | 12:21 |
| dtantsur | we cannot reasonably run so many jobs, can we? | 12:21 |
| *** dave-mccowan has joined #openstack-sdks | 12:22 | |
| frickler | I think we can and we should. possibly most of them non-voting | 12:23 |
| dtantsur | okay, I'll add an ironic-inspector job then | 12:25 |
| *** jpena|lunch is now known as jpena | 12:26 | |
| *** dave-mccowan has quit IRC | 12:26 | |
| mordred | dtantsur: I think we might want to do some file matchers ... | 12:28 |
| *** dave-mccowan has joined #openstack-sdks | 12:28 | |
| mordred | dtantsur: you probably don't need to run ironic-inspector jobs on patches to openstack/image/* - for instance :) | 12:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting ironic-inspector job https://review.opendev.org/684729 | 12:28 |
| dtantsur | mordred: that was my thought as well | 12:28 |
| dtantsur | although I'm not sure how to express it | 12:28 |
| dtantsur | I wonder to run it on e.g. openstack/resource.py | 12:29 |
| *** gtema_ has quit IRC | 12:29 | |
| mordred | yeah - I think openstack/{connection,resource,proxy}.py should likely run the world - but those should also be rare at this point | 12:29 |
| mordred | fwiw - I'm not opposed to these becomming voting if we figure out the right file matchers | 12:30 |
| *** gtema_ has joined #openstack-sdks | 12:30 | |
| mordred | but one step at a time | 12:30 |
| dtantsur | ++ | 12:32 |
| dtantsur | to use matchers, do we need to inherit the jobs? or can I do it in project.jobs.<>? | 12:32 |
| 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 |
| dtantsur | I very much agree with that | 12:33 |
| mordred | dtantsur: you can do it in project.jobs.<> | 12:33 |
| dtantsur | nice | 12:33 |
| mordred | I'll get some patches up today for project creation and initial content import and whatnot | 12:34 |
| dtantsur | great! then we can add a bifrost job to verify ironic bits | 12:34 |
| *** black_ant has quit IRC | 12:35 | |
| *** black_ant has joined #openstack-sdks | 12:35 | |
| *** cdent has quit IRC | 12:36 | |
| *** gtema_ has quit IRC | 12:37 | |
| *** dave-mccowan has quit IRC | 12:42 | |
| *** gtema has quit IRC | 12:44 | |
| *** gtema has joined #openstack-sdks | 12:46 | |
| *** mriedem has joined #openstack-sdks | 12:47 | |
| mordred | dtantsur: it's going to be some amount of fun for bifrost - because the module names are going to change | 12:55 |
| *** nicholas has joined #openstack-sdks | 12:55 | |
| dtantsur | mordred: can we also re-engineer ironic modules then? | 12:56 |
| dtantsur | they're awkward to say the least | 12:56 |
| mordred | totally | 12:56 |
| *** gtema has quit IRC | 12:56 | |
| dtantsur | and we should be doing os_baremetal_node, not os_ironic, etc, etc | 12:56 |
| 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 |
| dtantsur | \o/ | 12:56 |
| mordred | new names will be like 'openstack.cloud.ironic_node' instead of os_ironic_node - fwiw | 12:57 |
| dtantsur | openstack.cloud.baremetal_node ;) | 12:57 |
| *** gtema has joined #openstack-sdks | 12:57 | |
| mordred | ++ | 12:58 |
| gtema | oh cool. I like the idea. Let's do this | 12:59 |
| *** slaweq has quit IRC | 13:54 | |
| *** slaweq has joined #openstack-sdks | 13:57 | |
| 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:02 |
| efried | in that it *seems* as though it's not using the Connection's auth when querying that URI | 14:03 |
| efried | because I'm getting a 401 | 14:03 |
| *** cdent has joined #openstack-sdks | 14:13 | |
| *** gtema has quit IRC | 14:13 | |
| *** slaweq has quit IRC | 14:15 | |
| *** gtema has joined #openstack-sdks | 14:16 | |
| *** goldyfruit has joined #openstack-sdks | 14:16 | |
| *** slaweq has joined #openstack-sdks | 14:16 | |
| *** gtema has quit IRC | 14:22 | |
| *** goldyfruit has quit IRC | 14:46 | |
| *** enriquetaso has quit IRC | 14:51 | |
| *** e0ne has quit IRC | 14:59 | |
| *** e0ne has joined #openstack-sdks | 14:59 | |
| *** dtantsur is now known as dtantsur|afk | 15:00 | |
| *** ttsiouts has quit IRC | 15:13 | |
| *** ttsiouts has joined #openstack-sdks | 15:15 | |
| *** ttsiouts has quit IRC | 15:15 | |
| *** goldyfruit has joined #openstack-sdks | 15:18 | |
| *** ttsiouts has joined #openstack-sdks | 15:20 | |
| *** openstackgerrit has quit IRC | 15:21 | |
| *** ttsiouts has quit IRC | 15:24 | |
| *** gtema has joined #openstack-sdks | 15:26 | |
| *** openstackgerrit has joined #openstack-sdks | 15:33 | |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Add collections metadata info https://review.opendev.org/684787 | 15:33 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Import modules from ansible/ansible https://review.opendev.org/684788 | 15:33 |
| mordred | mnaser, dtantsur|afk: ^^ | 15:33 |
| mordred | efried: ah. hrm. maybe there is an issue there we need to fix :( | 15:34 |
| efried | mordred: let me resurrect the repro steps from yesterday... | 15:34 |
| mordred | mnaser, dtantsur|afk: we obviously need to add some zuul jobs and stuff | 15:34 |
| cdent | "add some zuul jobs and stuff" is my mantra | 15:36 |
| Shrews | mordred: what is the proposal for keeping the modules in sync (for the time being) between a-c-o and ansible/ansible? | 15:39 |
| 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:41 |
| mordred | Shrews: and basically stopping accepting PRs in devel branch or something | 15:42 |
| mnaser | i think it's also neat if we added some sort of warning to repoint people too | 15:42 |
| mnaser | ^ yeah and we should kill all issues/prs/etc | 15:42 |
| Shrews | are collections compatible with older versions of ansible? | 15:43 |
| mnaser | nope :< | 15:43 |
| Shrews | then i don't see how you can just tell people to install the collections | 15:43 |
| mnaser | well i guess moving forwards the os_server will be frozen | 15:43 |
| mnaser | but yeah i guessif someone jumps from 2.5 to 2.10 | 15:43 |
| mnaser | it would magically disappear | 15:43 |
| Shrews | and not accept bug fixes? i don't see that going over well | 15:44 |
| mordred | Shrews: I think it's that older versions of ansible already have the built in modules in them | 15:44 |
| 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:44 |
| *** gtema has quit IRC | 15:45 | |
| mordred | Shrews: I think accepting stable branch bugfixes is totally a legit thing to do | 15:45 |
| efried | mordred: okay, here we go: | 15:45 |
| efried | - Vanilla devstack running nova | 15:45 |
| efried | - Clone cyborg, pull down 682565 (to get proper opt registration) and pip install -e it | 15:45 |
| efried | - This /etc/cyborg/cyborg.conf: http://paste.openstack.org/show/779250/ | 15:45 |
| efried | - Run this script: http://paste.openstack.org/show/779251/ | 15:45 |
| efried | - Observe this output: http://paste.openstack.org/raw/779252/ | 15:45 |
| mordred | efried: uh... | 15:46 |
| Shrews | mordred: so i return to my original question... how are we going to keep those fixes in sync in both places? | 15:46 |
| efried | mordred: just skip to the last paste :) | 15:47 |
| 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:47 |
| *** gtema has joined #openstack-sdks | 15:47 | |
| mordred | Shrews: magic? | 15:48 |
| 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:48 |
| mordred | efried: yeah. like - that is not output we want to see - but it makes me wonder how anything would be working | 15:49 |
| 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 |
| efried | mordred: The most obvious guess is that version discovery is not using the auth I pass in. | 15:49 |
| efried | it's using... some other auth, or no auth. | 15:49 |
| efried | that would cause this behavior. | 15:49 |
| efried | edit: this particular version discovery path | 15:50 |
| mordred | yeah. something seems very strange there. I need to dig in to that - but it's lunchtime | 15:51 |
| efried | ack | 15:51 |
| mordred | that is definitely not behavior I expect :( | 15:52 |
| *** gtema has quit IRC | 15:52 | |
| efried | I'll open a story for this, I suppose. | 15:53 |
| mordred | Shrews: it's a topic we definitely need to wrap our heads around | 15:53 |
| *** gtema has joined #openstack-sdks | 16:03 | |
| *** jpich has quit IRC | 16:08 | |
| efried | mordred: https://storyboard.openstack.org/#!/story/2006614 | 16:12 |
| *** goldyfruit_ has joined #openstack-sdks | 16:12 | |
| *** efried is now known as efried_rollin | 16:14 | |
| *** goldyfruit has quit IRC | 16:15 | |
| *** gtema has quit IRC | 16:35 | |
| *** gtema has joined #openstack-sdks | 16:38 | |
| *** gtema has quit IRC | 16:41 | |
| *** jangutter has quit IRC | 16:47 | |
| *** jpena is now known as jpena|off | 16:57 | |
| *** e0ne has quit IRC | 16:59 | |
| *** cdent has quit IRC | 17:24 | |
| *** e0ne has joined #openstack-sdks | 18:25 | |
| *** jkulik has quit IRC | 18:30 | |
| *** jkulik has joined #openstack-sdks | 18:31 | |
| *** ricolin has quit IRC | 18:55 | |
| *** factor has quit IRC | 18:59 | |
| *** efried_rollin is now known as efried | 19:21 | |
| *** e0ne has quit IRC | 19:23 | |
| *** cdent has joined #openstack-sdks | 19:41 | |
| *** ralonsoh has quit IRC | 20:30 | |
| *** gtema has joined #openstack-sdks | 20:37 | |
| *** black_ant has quit IRC | 20:55 | |
| *** slaweq has quit IRC | 20:55 | |
| *** cdent has quit IRC | 21:12 | |
| 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:13 |
| efried | uhm, then again... I suppose that doesn't really help us unless osc is going to re-release for train, huh? | 21:14 |
| 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 |
| efried | okay, never mind then | 21:19 |
| efried | <waves hand> I was never here | 21:20 |
| dtroyer | but that really only matters for distro packagers… everyone else should be using current anyway :) | 21:20 |
| *** gtema_ has joined #openstack-sdks | 21:20 | |
| efried | psshhht, distros <rolls eyes> | 21:21 |
| *** gtema has quit IRC | 21:59 | |
| *** gtema_ has quit IRC | 22:07 | |
| *** gtema has joined #openstack-sdks | 22:19 | |
| *** gtema has quit IRC | 22:34 | |
| *** mriedem has quit IRC | 22:53 | |
| *** goldyfruit_ has quit IRC | 23:12 | |
| *** dave-mccowan has joined #openstack-sdks | 23:28 | |
| *** dave-mccowan has quit IRC | 23:33 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!