Thursday, 2022-10-20

vanougood morning ironic01:14
rpittaugood morning ironic! o/07:16
jandershey vanou rpittau and Ironic o/08:12
vanouHi janders08:19
vanouHi rpittau08:20
fricklerI'm seeing this failure on the latest requirements update, can somebody have a look please? I'm not sure whether it is actually related to the patch and if yes, which library https://zuul.opendev.org/t/openstack/build/b1d5058e20cb4ad7baecd3227c02f70808:36
rpittaufrickler: I just tested with the old uc and it seems ok, I'm going to do a couple of tests with the new libs09:05
rpittaufrickler: I think I found the issue, going to do one more test before seding the patch09:17
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix double mock call in glance_service test  https://review.opendev.org/c/openstack/ironic/+/86195609:39
rpittaufrickler: this ^ should fix the issues, at least from local tests it does :)09:39
fricklerrpittau: thx, I'll watch that and update the reqs patch when it passes09:54
frickleron a related note, getting https://review.opendev.org/c/openstack/ironic/+/859040 merged would be good09:54
rpittauoh snap forgot to recheck09:59
* dtantsur sloooowly blinks10:18
dtantsurrpittau: ouch, this test smells.. I wonder why we did it that way10:21
opendevreviewJacob Anders proposed openstack/sushy master: Improve resiliency of eTag handling  https://review.opendev.org/c/openstack/sushy/+/85612310:22
rpittauI didn't check the blame :P10:23
dtantsurrpittau: see my comments, I think we should clear it up rather than repeating the presumably broken pattern10:23
rpittausounds good10:23
jandershey dtantsur10:23
dtantsuro/10:24
jandershope you got some good sleep10:24
dtantsuryeah, but not enough yet :)10:24
jandersgood news = weekend is not far10:24
dtantsurtrue!10:24
jandersI feel the pain - I got to bed around 130am last night and landscapers rocked up just before 7am10:24
dtantsur:(10:25
dtantsurjanders: I suggest stevebaker[m], you and I meet at some point next week to discuss metal3. wdyt?10:25
dtantsurI'm pulling you in since you have the most compatible timezone with Steve :)10:25
jandersdtantsur 100%10:25
jandersthere's still a lot of gaps in my knowledge so it may be good to do a bigger sync but we can start with an ANZ discussion10:26
dtantsursure, I'll be there10:26
dtantsurnow, what's the Steve's timezone again?10:27
rpittauNZDT ?10:28
* rpittau lunch awaits bbl10:28
jandersUTC+13 I think10:30
dtantsurokay10:31
janders(I'm UTC+10)10:31
dtantsurstevebaker[m], janders, these times may be a bit of a stretch, but let's see: https://doodle.com/meeting/participate/id/erRQmYWe10:32
dtantsurif it does not work, I can go earlier too (doodle no longer allows "if needed" for organizers)10:33
jandersdone. Most of next week works for me, except 0800hrs UTC on Wed - responded accordingly10:37
jandersdtantsur arne_wiebalck I'm after second review on https://review.opendev.org/c/openstack/sushy/+/856123 (eTags) if you have time10:38
janders@ajya would be my other usual suspect to tag but she seems offline10:40
dtantsur(still on a meeting, sorry)11:20
opendevreviewRiccardo Pittau proposed openstack/ironic master: Fix double mock call in glance_service test  https://review.opendev.org/c/openstack/ironic/+/86195611:41
janderssee you tomorrow Ironic o/11:56
rpittaubye janders :)11:57
jandersgotta catch up on sleep after the PTG11:57
janderssee you rpittau11:57
jandersthanks for your help today!11:57
iurygregorymorning o/12:05
rpittauhey iurygregory :)12:19
dtantsursleeeeep12:21
TheJuliawhat is sleep?!?12:49
opendevreviewMerged openstack/ironic master: Switch to 2023.1 Python3 unit tests and generic template name  https://review.opendev.org/c/openstack/ironic/+/85904012:56
opendevreviewRiccardo Pittau proposed openstack/ironic master: Cross test sushy with python 3.10  https://review.opendev.org/c/openstack/ironic/+/86214013:02
* dtantsur is trying to make a person take over TheJulia's kexec patches.. with moderate success so far13:12
TheJulia\o/13:12
TheJuliadtantsur: so... funny thing, is apparently they "just worked" in some testing someone preformed.. but yeah13:13
dtantsurTheJulia: the literal thing I've heard is "a few operators have it downstream"13:13
dtantsurI'm afraid the ironic patches will need a rewrite after the various changes to deploy interfaces13:13
TheJuliayeah13:14
dtantsurOn the other hand, I think it can be a clean deploy interface now that we've completed the migration to deploy steps.13:17
dtantsurjust override deploy.tear_down_agent and deploy.boot_instance13:18
TheJulia... likely13:33
* dtantsur slowly emerges from yet another ptg day14:50
TheJuliaI briefly chatted with glance folk, they don't have an answer w/r/t checksum yet, but they are gfoing to get back to me15:18
dtantsurgreat, thank you!15:21
dtantsurI'm unable to make my brain collaborate any more, so calling it a day for now15:21
TheJuliao/15:22
opendevreviewRiccardo Pittau proposed openstack/ironic master: Cross test sushy with python 3.10  https://review.opendev.org/c/openstack/ironic/+/86214016:04
rpittaubye o/16:06
TheJuliaas far as I can tell, they have no intent on removing image_checksum16:22
TheJuliapositive ironic words on https://news.ycombinator.com/item?id=3327474416:28
TheJuliaoverall, FUD and bad feelings are strong there16:28
JayF> - It doesn't follow the "normal" OSS process of submitting pull requests. Their CI/Repos/Bug tracking systems are different enough that a lot of people probably don't bother trying to contribute16:30
JayFliterally the first bulletpoint on the first comment16:30
JayFthis is going to be my thing after I start inactiving projects16:30
TheJuliathe CLA aspect is the challenge there, but... maybe that should be a question I guess16:33
JayFI think CLA matters less than our tooling being "weird"16:33
JayFI was saying this in the TC meeting earlier; when talking about inactive projects16:33
JayFbasically making the point that I think we could have a better "middle ground" for projects people use but don't want to pay "minimum openstack maintenance cost" for (but would, for instance, use it with a github/gitlab PR-style workflow)16:34
TheJuliaThere was *once* talk of building some GH PR -> gerrit change magic17:09
TheJuliagranted that is a *whole* giant can of worms17:10
TheJuliaand when I say giant, I mean like... "I'm making pasta sauce for 20 people" industrial can.17:10
JayFGentoo solves this through human-mechanical-turks17:10
JayFyou submit a PR, it gets reviewed, basically the maintainer is responsible for pulling it down and merging it17:10
JayFbut the big pain there for us would be CI integration; gentoo doesn't really have involed ci (mainly what we'd consider linting) so duplicating it into github is trivial17:11
TheJuliawheeee https://github.com/yanndegat/ironic17:13
JayFplease just tell me what that is without making me figure it out myself lol17:17
ashinclouds[m]Ovh fork17:18
JayFinteresting17:19
* TheJulia goes and gets half and half17:24
opendevreviewJulia Kreger proposed openstack/ironic-specs master: WIP Add a shard key  https://review.opendev.org/c/openstack/ironic-specs/+/86180318:44
jandersgood morning Ironic o/22:00

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!