vanou | good morning ironic | 01:14 |
---|---|---|
rpittau | good morning ironic! o/ | 07:16 |
janders | hey vanou rpittau and Ironic o/ | 08:12 |
vanou | Hi janders | 08:19 |
vanou | Hi rpittau | 08:20 |
frickler | I'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/b1d5058e20cb4ad7baecd3227c02f708 | 08:36 |
rpittau | frickler: I just tested with the old uc and it seems ok, I'm going to do a couple of tests with the new libs | 09:05 |
rpittau | frickler: I think I found the issue, going to do one more test before seding the patch | 09:17 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix double mock call in glance_service test https://review.opendev.org/c/openstack/ironic/+/861956 | 09:39 |
rpittau | frickler: this ^ should fix the issues, at least from local tests it does :) | 09:39 |
frickler | rpittau: thx, I'll watch that and update the reqs patch when it passes | 09:54 |
frickler | on a related note, getting https://review.opendev.org/c/openstack/ironic/+/859040 merged would be good | 09:54 |
rpittau | oh snap forgot to recheck | 09:59 |
* dtantsur sloooowly blinks | 10:18 | |
dtantsur | rpittau: ouch, this test smells.. I wonder why we did it that way | 10:21 |
opendevreview | Jacob Anders proposed openstack/sushy master: Improve resiliency of eTag handling https://review.opendev.org/c/openstack/sushy/+/856123 | 10:22 |
rpittau | I didn't check the blame :P | 10:23 |
dtantsur | rpittau: see my comments, I think we should clear it up rather than repeating the presumably broken pattern | 10:23 |
rpittau | sounds good | 10:23 |
janders | hey dtantsur | 10:23 |
dtantsur | o/ | 10:24 |
janders | hope you got some good sleep | 10:24 |
dtantsur | yeah, but not enough yet :) | 10:24 |
janders | good news = weekend is not far | 10:24 |
dtantsur | true! | 10:24 |
janders | I feel the pain - I got to bed around 130am last night and landscapers rocked up just before 7am | 10:24 |
dtantsur | :( | 10:25 |
dtantsur | janders: I suggest stevebaker[m], you and I meet at some point next week to discuss metal3. wdyt? | 10:25 |
dtantsur | I'm pulling you in since you have the most compatible timezone with Steve :) | 10:25 |
janders | dtantsur 100% | 10:25 |
janders | there'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 discussion | 10:26 |
dtantsur | sure, I'll be there | 10:26 |
dtantsur | now, what's the Steve's timezone again? | 10:27 |
rpittau | NZDT ? | 10:28 |
* rpittau lunch awaits bbl | 10:28 | |
janders | UTC+13 I think | 10:30 |
dtantsur | okay | 10:31 |
janders | (I'm UTC+10) | 10:31 |
dtantsur | stevebaker[m], janders, these times may be a bit of a stretch, but let's see: https://doodle.com/meeting/participate/id/erRQmYWe | 10:32 |
dtantsur | if it does not work, I can go earlier too (doodle no longer allows "if needed" for organizers) | 10:33 |
janders | done. Most of next week works for me, except 0800hrs UTC on Wed - responded accordingly | 10:37 |
janders | dtantsur arne_wiebalck I'm after second review on https://review.opendev.org/c/openstack/sushy/+/856123 (eTags) if you have time | 10:38 |
janders | @ajya would be my other usual suspect to tag but she seems offline | 10:40 |
dtantsur | (still on a meeting, sorry) | 11:20 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix double mock call in glance_service test https://review.opendev.org/c/openstack/ironic/+/861956 | 11:41 |
janders | see you tomorrow Ironic o/ | 11:56 |
rpittau | bye janders :) | 11:57 |
janders | gotta catch up on sleep after the PTG | 11:57 |
janders | see you rpittau | 11:57 |
janders | thanks for your help today! | 11:57 |
iurygregory | morning o/ | 12:05 |
rpittau | hey iurygregory :) | 12:19 |
dtantsur | sleeeeep | 12:21 |
TheJulia | what is sleep?!? | 12:49 |
opendevreview | Merged openstack/ironic master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/ironic/+/859040 | 12:56 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Cross test sushy with python 3.10 https://review.opendev.org/c/openstack/ironic/+/862140 | 13:02 |
* dtantsur is trying to make a person take over TheJulia's kexec patches.. with moderate success so far | 13:12 | |
TheJulia | \o/ | 13:12 |
TheJulia | dtantsur: so... funny thing, is apparently they "just worked" in some testing someone preformed.. but yeah | 13:13 |
dtantsur | TheJulia: the literal thing I've heard is "a few operators have it downstream" | 13:13 |
dtantsur | I'm afraid the ironic patches will need a rewrite after the various changes to deploy interfaces | 13:13 |
TheJulia | yeah | 13:14 |
dtantsur | On the other hand, I think it can be a clean deploy interface now that we've completed the migration to deploy steps. | 13:17 |
dtantsur | just override deploy.tear_down_agent and deploy.boot_instance | 13:18 |
TheJulia | ... likely | 13:33 |
* dtantsur slowly emerges from yet another ptg day | 14:50 | |
TheJulia | I briefly chatted with glance folk, they don't have an answer w/r/t checksum yet, but they are gfoing to get back to me | 15:18 |
dtantsur | great, thank you! | 15:21 |
dtantsur | I'm unable to make my brain collaborate any more, so calling it a day for now | 15:21 |
TheJulia | o/ | 15:22 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Cross test sushy with python 3.10 https://review.opendev.org/c/openstack/ironic/+/862140 | 16:04 |
rpittau | bye o/ | 16:06 |
TheJulia | as far as I can tell, they have no intent on removing image_checksum | 16:22 |
TheJulia | positive ironic words on https://news.ycombinator.com/item?id=33274744 | 16:28 |
TheJulia | overall, FUD and bad feelings are strong there | 16: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 contribute | 16:30 |
JayF | literally the first bulletpoint on the first comment | 16:30 |
JayF | this is going to be my thing after I start inactiving projects | 16:30 |
TheJulia | the CLA aspect is the challenge there, but... maybe that should be a question I guess | 16:33 |
JayF | I think CLA matters less than our tooling being "weird" | 16:33 |
JayF | I was saying this in the TC meeting earlier; when talking about inactive projects | 16:33 |
JayF | basically 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 |
TheJulia | There was *once* talk of building some GH PR -> gerrit change magic | 17:09 |
TheJulia | granted that is a *whole* giant can of worms | 17:10 |
TheJulia | and when I say giant, I mean like... "I'm making pasta sauce for 20 people" industrial can. | 17:10 |
JayF | Gentoo solves this through human-mechanical-turks | 17:10 |
JayF | you submit a PR, it gets reviewed, basically the maintainer is responsible for pulling it down and merging it | 17:10 |
JayF | but 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 trivial | 17:11 |
TheJulia | wheeee https://github.com/yanndegat/ironic | 17:13 |
JayF | please just tell me what that is without making me figure it out myself lol | 17:17 |
ashinclouds[m] | Ovh fork | 17:18 |
JayF | interesting | 17:19 |
* TheJulia goes and gets half and half | 17:24 | |
opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP Add a shard key https://review.opendev.org/c/openstack/ironic-specs/+/861803 | 18:44 |
janders | good morning Ironic o/ | 22:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!