| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266 | 00:12 |
|---|---|---|
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266 | 01:13 |
| *** weshay|ruck is now known as weshay | 01:31 | |
| *** gtema has joined #openstack-ansible-sig | 03:44 | |
| *** gtema has quit IRC | 03:49 | |
| *** evrardjp has quit IRC | 04:36 | |
| *** evrardjp has joined #openstack-ansible-sig | 04:37 | |
| *** spatel has joined #openstack-ansible-sig | 06:20 | |
| *** spatel has quit IRC | 06:25 | |
| *** gtema has joined #openstack-ansible-sig | 06:51 | |
| *** dtantsur|afk is now known as dtantsur | 07:44 | |
| tremble | dtantsur mordred : Are the Ansible sanity tests being run against the OpenStack collection at the minute? | 09:12 |
| tremble | dtantsur mordred : https://review.opendev.org/715266 shouldn't be passing the sanity tests. | 09:13 |
| tremble | ( sshnaidm|afk ^ ) | 09:14 |
| *** sshnaidm|afk is now known as sshnaidm | 09:25 | |
| sshnaidm | tremble, looking.. | 09:26 |
| dtantsur | I'm not sure | 09:26 |
| tremble | Although it's possible I'm used to the Ansible 2.10 sanity tests rather than the 2.9 tests | 09:28 |
| tremble | but the missing "type: dict" in the docs should be picked up by the ansible-test sanity tests. | 09:28 |
| sshnaidm | WARNING: All targets skipped | 09:30 |
| tremble | That would be why... | 09:37 |
| tremble | sshnaidm, Which test are they under? | 09:37 |
| sshnaidm | tremble, it should run in linters.. | 09:38 |
| tremble | sshnaidm, I can have a poke at that if you'd like. I'm a big fan of catching as much of the 'dumb' stuff as possible in automated tests | 09:39 |
| tremble | You'll even find my name in the commits for some of them :) | 09:40 |
| *** holser has joined #openstack-ansible-sig | 09:41 | |
| sshnaidm | tremble, sure, won't take this joy from you :) | 09:42 |
| sshnaidm | tremble, I think I know.. | 09:49 |
| sshnaidm | tremble, installing collection into repo directory was a bad idea | 09:49 |
| sshnaidm | tremble, so you want to fix it? | 09:52 |
| sshnaidm | tremble, most likely we'll need to fix linter issues as well.. :( | 09:53 |
| tremble | Hopefully not, I did a big batch of them | 09:53 |
| tremble | I'm poking it locally to see if I can reproduce the "skipping" | 09:53 |
| tremble | unfortunately the nexus instance I'm using has apparently fallen over so it's slowing things down. | 09:54 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Fix ansible sanity linter tests https://review.opendev.org/716908 | 09:57 |
| sshnaidm | tremble, let's see how it goes ^ | 09:58 |
| tremble | :) | 10:03 |
| tremble | sshnaidm, It would be good to use the 2.10 tests too, but I'm not sure how doable that would be. | 10:04 |
| sshnaidm | tremble, yeah, tbh I'd prefer to use latest ansible-test there as well | 10:05 |
| sshnaidm | ok, now it's running | 10:07 |
| tremble | Well there's a good sign, it failed :) | 10:07 |
| sshnaidm | tremble, https://121c67397e86d1847de4-11eee2f9a8da0aa272aa38b865c1ef08.ssl.cf1.rackcdn.com/716908/1/check/openstack-tox-linters/9c51987/job-output.txt | 10:08 |
| *** spatel has joined #openstack-ansible-sig | 10:09 | |
| tremble | Yeah, it should be '- "Mark Chappell (@tremble) <mchappel@redhat.com>"' | 10:11 |
| tremble | Although it's also not picking up on the ignores file by the looks of it. | 10:13 |
| *** spatel has quit IRC | 10:14 | |
| sshnaidm | tremble, it should pick 2.9 ignore file | 10:15 |
| *** gshippey has joined #openstack-ansible-sig | 11:08 | |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: Fix ansible sanity linter tests https://review.opendev.org/716908 | 11:20 |
| tremble | sshnaidm, Sanity tests fixed up ^ | 11:26 |
| sshnaidm | tremble, thanks! | 11:32 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686 | 11:43 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632 | 11:43 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633 | 11:43 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991 | 11:43 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992 | 11:43 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537 | 11:43 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632 | 11:59 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633 | 11:59 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991 | 11:59 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992 | 11:59 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537 | 11:59 |
| * tremble sniggers at some bad copy&paste comments in .zuul.yaml | 12:16 | |
| tremble | Morning mordred | 12:22 |
| mordred | morning tremble | 12:26 |
| gtema | ok then, it's far away from morning, but still - morning to everyone :D | 12:27 |
| tremble | gtema, You're also in Germany, it's not *that* far away... | 12:31 |
| gtema | well, when your morning starts at 7 - it is | 12:31 |
| gtema | I'm close to EOD already ;) | 12:31 |
| tremble | Unfortunately I work too much with Americans if I started at 7 I'd have pretty much 0 overlap with their 9-5 | 12:32 |
| gtema | sure | 12:32 |
| gtema | I'm "lucky" to work in the opposite direction | 12:33 |
| gtema | "go east" | 12:33 |
| gtema | where my 9 is 17 for some | 12:34 |
| tremble | I have a 4 year old who's an early riser, a 7am start wouldn't be too much of a change for me :) | 12:38 |
| mordred | a few years ago I had a team with folks in New York, Seattle, Spain, UK, Australia and New Zealand - team overlap time was hard ;) | 12:39 |
| gtema | so yeah, let mordred wake up earlier :D | 12:39 |
| mordred | gtema: :) | 12:39 |
| gtema | unrelated: I face lots of timeou failures from files.pythonhosted.org in my zuul. Anybody else seeing that? | 12:39 |
| dtantsur | tremble: where exactly are you based btw? | 12:40 |
| mordred | not I - but I'm still on first coffee | 12:40 |
| gtema | okay | 12:40 |
| tremble | dtantsur, Munich | 12:44 |
| dtantsur | oh lovely! I had plans to pass by in May, not going to happen probably :) | 12:45 |
| dtantsur | maybe one day.. | 12:47 |
| tremble | Yeah, who knows when it'll be 'normal' to travel again. | 12:47 |
| dtantsur | yeah, I'm not even sure when I get to gtema's town which is what, 1.5 hours by an RE? | 12:48 |
| * mordred wonders how long after it's "ok" will he feel ok about it himself | 12:49 | |
| dtantsur | let's live to see it, I guess? | 12:50 |
| mordred | ++ | 12:50 |
| mordred | yes- I look forward to learning the answer to this question | 12:50 |
| tremble | mordred, I chose 'normal' deliberately ;) | 12:54 |
| tremble | I kinda expect we're going to see some form of disruption off and on for the rest of the year at least | 12:55 |
| dtantsur | tremble: how is Munich looking nowadays? I've heard you have a serious lock-down, unlike in NRW? | 12:59 |
| tremble | dtantsur, 'serious'. We're still allowed outside, we can still go shopping, it's mostly just "don't socialise" | 13:01 |
| dtantsur | ah, so more similar to here | 13:01 |
| gtema | I do not think very like here, we still allowed to socialize at max 2, BY not at all | 13:02 |
| tremble | From what I gather we're in a more relaxed state than northern Italy, but they're able to talk about crazy fines/prison time because of the law they've based the lockdown on. | 13:02 |
| gtema | And I know police was really harsh on any groups of people >2 | 13:02 |
| *** weshay is now known as weshay|ruck | 13:55 | |
| *** dtantsur is now known as dtantsur|brb | 14:10 | |
| sshnaidm | mordred, any ideas how I can run tox linters job, but with different ansible version? | 14:22 |
| sshnaidm | mordred, like running 2 jobs actually, each one has its own ansible ver | 14:22 |
| *** sshnaidm is now known as sshnaidm|afk | 14:25 | |
| mordred | sshnaidm|afk: yeah - make two jobs | 14:26 |
| mordred | jobs are cheap :) | 14:26 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Fix ansible sanity linter tests https://review.opendev.org/716908 | 14:30 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686 | 14:35 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632 | 14:42 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633 | 14:42 |
| *** dmellado has quit IRC | 15:14 | |
| *** sshnaidm|afk is now known as sshnaidm | 15:15 | |
| tremble | mordred, Are you likely to have time to get to the last two new modules or should I rebase https://review.opendev.org/#/c/716537 and break the dependency on the Federation Protocol patches? | 15:18 |
| tremble | (I made the mistake of being lazy and dropping the patch at the tail of my chain of new modules | 15:19 |
| tremble | ) | 15:19 |
| mordred | tremble: typo in https://review.opendev.org/#/c/714991/8 | 15:21 |
| mordred | tremble: same in the next one | 15:22 |
| mordred | otherwise they look good | 15:22 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537 | 15:24 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991 | 15:27 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992 | 15:27 |
| *** gtema has quit IRC | 15:28 | |
| * tremble thanks mordred | 15:29 | |
| mordred | thank you - and that's for being patient as we sort through all these new things :) | 15:29 |
| *** dmellado has joined #openstack-ansible-sig | 15:30 | |
| tremble | sshnaidm, Hopefully if you get the 2.10 sanity tests up and running you *shouldn't* see any additional failures, locally I ran against devel... | 15:31 |
| sshnaidm | tremble, great | 15:32 |
| sshnaidm | mordred, seems like need to play with arguments passed to tox-linter job.. | 15:33 |
| openstackgerrit | Mark Chappell proposed openstack/ansible-collections-openstack master: Attempt to run Ansible-test from the devel ansible branch https://review.opendev.org/717062 | 15:34 |
| tremble | sshnaidm, /me wonders if it's as simple as ^ | 15:34 |
| sshnaidm | tremble, that may work! If it installs "siblings" | 15:35 |
| sshnaidm | let's see, thanks for the patch | 15:36 |
| tremble | At the least it should get ansible somewhere we could then pull it from... | 15:38 |
| tremble | Not sure if it likes me messing with .zuul.yaml | 15:40 |
| tremble | mordred, Oh nice, it noticed 716537 was just a rebase and didn't require a full re-review | 15:42 |
| mordred | tremble: that _should_ work | 15:44 |
| *** dtantsur|brb is now known as dtantsur | 16:01 | |
| *** evrardjp has quit IRC | 16:36 | |
| *** evrardjp has joined #openstack-ansible-sig | 16:36 | |
| sshnaidm | dtantsur, so you're leaving the openstack nest? :) | 16:49 |
| dtantsur | wait, am I? | 16:50 |
| sshnaidm | dtantsur, going to independent journey | 16:50 |
| sshnaidm | Ironic I mean | 16:50 |
| dtantsur | well, that's a discussion rather then a plan at this point | 16:50 |
| mordred | I thought dtantsur and ironic were the same thing | 16:50 |
| dtantsur | tssssss! | 16:50 |
| dtantsur | there is also my cat, but she pretends to be Julia | 16:50 |
| sshnaidm | but the modules stay here for a while, right? | 16:51 |
| dtantsur | sshnaidm: let's not talk as if the decision has been made | 16:55 |
| sshnaidm | sure, np | 16:56 |
| dtantsur | until we get a good-to-go from both the ironic team and the TC this is just my speculations | 16:56 |
| dtantsur | even if the PTL warmly supports them | 16:56 |
| mordred | dtantsur: of course you're going to make my brain hurt about what that means for openstacksdk ;) | 16:56 |
| dtantsur | sshnaidm: but feel free to voice the question of modules and SDK as a concern on the ML, if it bothers you | 16:56 |
| dtantsur | mordred: ^^^ :) | 16:57 |
| dtantsur | it hurts my brain already, yes | 16:57 |
| mordred | dtantsur: well - my first thought it "screw it, we'll make an exception" | 16:57 |
| dtantsur | *shrug* it may be the way to go | 16:57 |
| sshnaidm | I'll do whatever mordred does with sdk :D | 16:57 |
| dtantsur | so far the idea hasn't been well received by the TC, so we'll see how it progresses | 16:58 |
| dtantsur | it definitely is not going to happen over weekend | 16:58 |
| dtantsur | even if we get a good-to-go right now (which is not going to happen), it will probably take most of Victoria to do the actual "iron-out" how one person already called it | 16:58 |
| dtantsur | on this positive note I'll call it a day, talk to you tomorrow | 16:59 |
| *** dtantsur is now known as dtantsur|afk | 16:59 | |
| sshnaidm | actually modules are not openstack official as well, so I don't think it may be a concern | 17:02 |
| mordred | well - they're sort of official - the SIG is an openstack sig and we've got delegated authority over the galaxy namespace. that said - I think the bigger can of worms is related to the goal stated by the ironic team balanced against what makes things work better or worse for users | 17:04 |
| mordred | like - if the goal is to create some marketing separation to make it clear to people that the standalone use case is a solidone, then needing to install the openstack.cloud collection and openstacksdk to get the ansible modules might be weird | 17:04 |
| mordred | otoh - ceasing to use those makes the experience harder for the instances where the two are used together still | 17:05 |
| mordred | there isn't an easy answer I don't think | 17:05 |
| mordred | but - fwiw - I would not be personally opposed to continuing to support ironic if it were to move out - if for no other reason than that we would still need to support it for historical reasons until the end of time anyway | 17:06 |
| mordred | so since we _will_ be supporting using ironic from sdk, there's no reason to not just use that still from my pov | 17:06 |
| sshnaidm | I just realized how Ansible went far from "ssh tool", you can actually provision real baremetal with ironic modules | 17:20 |
| mordred | ++ | 17:28 |
| mordred | much to the chagrin of the original ansible author who worked on cobbler and thought the idea of ironic was crazy | 17:28 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add properties for os_project https://review.opendev.org/715266 | 17:41 |
| *** gtema has joined #openstack-ansible-sig | 18:17 | |
| *** sshnaidm is now known as sshnaidm|afk | 18:24 | |
| *** gtema has quit IRC | 18:50 | |
| tremble | sshnaidm|afk mordred, Looks like My change checked out the relevant Ansible branches but didn't install / 'enable' them something to poke at tomorrow | 19:11 |
| sshnaidm|afk | Sibling ansible-base at src/github.com/ansible/ansible | 19:17 |
| sshnaidm|afk | maybe because of "ansible-base" | 19:17 |
| sshnaidm|afk | it doesn't recognize it as sibling to "ansible" | 19:18 |
| sshnaidm|afk | damn.. this broke everything in ansible-devel jobs, they run with 2.9 now | 19:20 |
| sshnaidm|afk | mordred, ^^ | 19:21 |
| mordred | oh right | 19:28 |
| mordred | we actually just had to do something with this in infra production test jobs too | 19:28 |
| mordred | we might have to use more of a hammer here | 19:29 |
| * tremble just causes trouble. Or spots trouble... | 19:30 | |
| mordred | sshnaidm|afk, tremble : ansible-base does exist | 19:30 |
| mordred | in pypi | 19:30 |
| mordred | it doesn't do anything | 19:30 |
| mordred | BUT - if we have the -devel job pip install ansible-devel instead of ansible | 19:31 |
| mordred | gah | 19:31 |
| mordred | BUT - if we have the -devel job pip install ansible-base instead of ansible | 19:31 |
| mordred | then siblings will install the git repo | 19:31 |
| tremble | mordred, Feel free to hijack the change I started | 19:33 |
| sshnaidm|afk | so this will require conditional in test-requirements..? | 19:33 |
| sshnaidm|afk | or.. ansible-base provides ansible-2.9 as well | 19:34 |
| sshnaidm|afk | well, it just does nothing.. | 19:36 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Deal with ansible-base https://review.opendev.org/717108 | 19:37 |
| mordred | sshnaidm|afk: I think that should do it | 19:37 |
| mordred | sshnaidm|afk: yeah - it's just an empty package - but it can be used to trigger the logic in tox-siblings that we want to replace something installed in the venv with something from git | 19:38 |
| mordred | this is ... a bit of a mess ;) | 19:38 |
| sshnaidm|afk | oh, yeah.. | 19:43 |
| sshnaidm|afk | so ansible-base of ansible-devel? https://review.opendev.org/#/c/717108/1/.zuul.yaml | 19:44 |
| tremble | Are vars "merged" between parent and child jobs? | 19:44 |
| sshnaidm|afk | tremble, yes | 19:53 |
| sshnaidm|afk | and override previous with same name | 19:53 |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: os_keystone_mapping: fixup some minor nits found in followup reviews https://review.opendev.org/716537 | 19:57 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Fix ansible-base case for Ansible devel branches https://review.opendev.org/717118 | 20:07 |
| sshnaidm|afk | my try ^^ | 20:07 |
| openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Deal with ansible-base https://review.opendev.org/717108 | 20:09 |
| mordred | sshnaidm|afk: yours is definitely worth a try :) | 20:10 |
| *** gshippey has quit IRC | 20:30 | |
| *** weshay|ruck is now known as weshay | 20:57 | |
| sshnaidm|afk | and seems like it's working: https://review.opendev.org/#/c/717118/ | 21:09 |
| *** openstackstatus has quit IRC | 22:56 | |
| *** openstack has joined #openstack-ansible-sig | 22:59 | |
| *** ChanServ sets mode: +o openstack | 22:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!