iurygregory | TheJulia, you mean a new provider in https://github.com/openstack/ironic/tree/master/ironic/dhcp ? | 00:04 |
---|---|---|
iurygregory | I like the idea I would say, but would be good to understand the use case to get a bigger picture | 00:05 |
TheJulia | no, for example the redfish httpboot interface amkes an iso and sends the bmc a url to boot from | 00:06 |
TheJulia | the httpboot interfaces mirror out network bootloading | 00:06 |
iurygregory | oh wow | 00:06 |
TheJulia | so they point to grub or ipxe or whatever | 00:06 |
TheJulia | but if you don't have grub or ipxe | 00:06 |
TheJulia | ... what do you do?!? | 00:07 |
opendevreview | Merged openstack/ironic-python-agent stable/2023.2: Call evaluate_hardware_support exactly once per hwm https://review.opendev.org/c/openstack/ironic-python-agent/+/920226 | 00:07 |
iurygregory | I like the idea | 00:07 |
iurygregory | sounds crazy | 00:07 |
iurygregory | and useful :D | 00:07 |
iurygregory | ok, no timeout errors on my tests, but https://paste.opendev.org/show/bUyXbNRkjuqVjEnwInL7/ going to look more tomorrow | 02:11 |
TheJulia | iurygregory: i guess that should kind of be expected... | 03:14 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 03:20 |
opendevreview | cid proposed openstack/ironic master: wip: Provision aarch64 fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic/+/915441 | 05:30 |
dtantsur | JayF: I haven't got to booting IPA yet (only doing this effort in my free time) | 06:44 |
dtantsur | iurygregory: when they don't return a Location header, what do they return at all? Can it be somehow similar to https://review.opendev.org/c/openstack/sushy/+/867969? | 06:47 |
opendevreview | Merged openstack/ironic-tempest-plugin master: Patch to enforce json extension works in existing API behaviour https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/913926 | 06:56 |
opendevreview | Merged openstack/ironic-tempest-plugin master: Additional test state fixes https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918001 | 07:05 |
iurygregory | good morning Ironic | 11:20 |
iurygregory | TheJulia, well not really expected to me (because this is the same machine where I tested the feature, the only error I saw in the past was the timeout <sad>) | 11:21 |
iurygregory | dtantsur, I wish I could check that.. I will try to ask the QE if I can use the R6525 in my setup to have a better debug, I think it only happens in this model =( | 11:30 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-inspector master: [DNM] Instrument to debug memory leaks https://review.opendev.org/c/openstack/ironic-inspector/+/920275 | 12:46 |
opendevreview | cid proposed openstack/ironic master: Fix negative lookahead assertion syntax error https://review.opendev.org/c/openstack/ironic/+/920276 | 12:47 |
opendevreview | cid proposed openstack/ironic master: Fix negative lookahead assertion syntax error https://review.opendev.org/c/openstack/ironic/+/920276 | 12:56 |
iurygregory | ok, since the machine is unhappy, I'm going to test the update via Redfish using curls to see the response with more details :D | 13:14 |
opendevreview | cid proposed openstack/ironic-python-agent master: Fix negative lookahead assertion syntax error https://review.opendev.org/c/openstack/ironic-python-agent/+/920280 | 13:20 |
opendevreview | cid proposed openstack/ironic-inspector master: Fix negative lookahead assertion syntax error https://review.opendev.org/c/openstack/ironic-inspector/+/920281 | 13:36 |
JayF | It's nice to have this problem -- does anyone else wanna look at the changes from cid to fix zuul config, before I land as a single core since it's CI? I normally would JFDI but this is the first time in *years* it might be me landing code for someone at the same company :D | 14:06 |
JayF | I assume that particular worry is likely aged out at his point :D | 14:06 |
JayF | *this | 14:06 |
TheJulia | want to sure, what is driving urgency to do it now | 14:12 |
TheJulia | or this very moment? | 14:12 |
iurygregory | Interesting enough, R6525 doesn't create the Task so we can monitor the update... | 14:12 |
dtantsur | iurygregory: lovely.. is there anything we can track at all? | 14:13 |
iurygregory | dtantsur, well, it doesn't even create a Job in iDRAC... | 14:14 |
iurygregory | I think is just broken | 14:14 |
dtantsur | iurygregory: but does it actually update anything? | 14:14 |
iurygregory | nope | 14:14 |
dtantsur | oookay, so we may simply need a better error message. | 14:14 |
iurygregory | you can't update via SimpleUpdate at least... | 14:14 |
iurygregory | https://paste.opendev.org/show/bQpPkhuM9wzae0CL3lTn/ | 14:15 |
iurygregory | https://opendev.org/openstack/sushy/src/branch/master/sushy/taskmonitor.py#L222 the error we are getting since it couldn't find the Location :D | 14:15 |
JayF | TheJulia: it's spamming the changes, and it's us getting ahead of the problem. My only urgency is once it's done, we don't have to think about it anymore :D -- for box-checking changes like this, I usually want them in asap | 14:15 |
TheJulia | oh yeah, better error message definitely | 14:15 |
JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/912979 is for unmaintained/xena; it removes functional testing as well as updating .gitreview -- functional tests in IPA are borderline-useless anyway, and I felt there was no compelling reason to troubleshoot the failure to install Yappi on jammy to fix the CI :) | 14:16 |
JayF | Do we want unmaintained/ branch commit notifications to come here? they do not currently. | 14:16 |
iurygregory | wondering how it should work in this machine or if shouldn't at all... | 14:16 |
* iurygregory sends email to Dell folks in Brazil to see if they know something | 14:17 | |
TheJulia | JayF: I'm still not sure I follow where this urgency is coming from and also how cid's changes relate to your ipa changes | 14:17 |
TheJulia | JayF: I can look in a little bit, maybe 10-15 minutes | 14:18 |
TheJulia | iurygregory: I guess my confusion yesterday, since my brain is already semi-distracted from the thread on the downstream window at the moment, is that the error message yesterday implies something very different, unfortunately disjointed from what the actual bmc behavior issue is since it replies "OK" bu doesn't actually give you anything useful :( | 14:20 |
JayF | TheJulia: those are two separate items, and the cid thing is less urgent and me looking for blanket "it's OK to single-core-land changes from him that impact CI per our usual policy" | 14:20 |
JayF | since it's literally been like, 6-7 years since I've had the situation of someone working with me who is fixing CI too :D | 14:20 |
dtantsur | I think we've abandoned all "same company" rules loooong ago | 14:20 |
dtantsur | otherwise, Red Hatters would be in trouble :D | 14:20 |
JayF | that was my assumption; but I try to be extremely explicit about things like this | 14:20 |
TheJulia | yeah, basically openstack did | 14:20 |
TheJulia | But really, single core approving in rapid succession when other reviewers are around is really not great either | 14:21 |
TheJulia | there is the needful which needs to be done at times though | 14:21 |
TheJulia | urgency just needs to be understood | 14:21 |
JayF | I'll just let someone else go read those same 2 lines I did on all the various projects then | 14:21 |
TheJulia | are they tagged with ironic-week-prio or is it just the changes the notifier put into irc? | 14:22 |
JayF | not tagged (yet), I will tag em as I go through | 14:22 |
TheJulia | cool, that makes it easy | 14:22 |
TheJulia | thanks! | 14:22 |
iurygregory | TheJulia, you are talking about https://paste.opendev.org/show/bUyXbNRkjuqVjEnwInL7/ right? | 14:25 |
iurygregory | this one is the issue with HPE, I'm planning to patch my bifrost with some code to see if helps... | 14:26 |
TheJulia | iurygregory: no, you had one yesterday about a missing location | 14:30 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-inspector master: [DNM] Instrument to debug memory leaks https://review.opendev.org/c/openstack/ironic-inspector/+/920275 | 15:02 |
iurygregory | TheJulia, oh ok, after talking with the QE I think the firmware 7.10. they shipped is broken you can't even downgrade from the iDRAC UI | 15:12 |
dtantsur | \o/ | 15:25 |
opendevreview | Merged openstack/ironic master: Fix negative lookahead assertion syntax error https://review.opendev.org/c/openstack/ironic/+/920276 | 16:19 |
TheJulia | iurygregory: oh, that is awesome | 16:27 |
TheJulia | and by awesome, I mean sarcastic awesome | 16:27 |
* TheJulia wonders if starting a brewery is in the cards so there can be a beer like "remote procedure call", "semaphore flag", and "burnout flag" | 16:28 | |
iurygregory | TheJulia, yup! | 16:47 |
opendevreview | cid proposed openstack/networking-generic-switch master: WIP: Add VLAN and Port whitelist https://review.opendev.org/c/openstack/networking-generic-switch/+/888047 | 17:14 |
opendevreview | cid proposed openstack/networking-generic-switch master: WIP: Add VLAN and Port whitelist https://review.opendev.org/c/openstack/networking-generic-switch/+/888047 | 17:16 |
iurygregory | I asked QE to add an R640 to the cluster and try to do the upgrade via the CRD to see how it goes \o/ | 17:29 |
JayF | hoping https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/920312 fixes https://review.opendev.org/c/openstack/ironic-python-agent/+/912979 (xena) | 18:51 |
TheJulia | hopefully | 19:34 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic-specs/+/890164 | 20:54 |
JayF | revised ^ for feedback; cid may start implementation of this while I'm in Europe so if folks could prioritize reviewing this, it'd be appreciated | 20:55 |
iurygregory | JayF, ack o/ | 21:01 |
iurygregory | safe travels! | 21:01 |
JayF | absolutely, and I'll probably not entirely disappear from upstream, but it's hard to predict how busy things get when you're visiting :D | 21:02 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic-specs/+/890164 | 21:06 |
TheJulia | Have a good trip | 21:15 |
opendevreview | Jay Faulkner proposed openstack/ironic-specs master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic-specs/+/890164 | 21:30 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Block glean-early.sh from running https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/920183 | 22:02 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Block cloud-init user-data loading https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/920184 | 22:02 |
iurygregory | have a great weekend everyone, see you next week o/ | 22:18 |
JayF | o/ | 22:20 |
opendevreview | Verification of a change to openstack/ironic master failed: Add Node Servicing to api-ref https://review.opendev.org/c/openstack/ironic/+/920152 | 22:25 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: reboot the node in basic ops tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/918462 | 22:28 |
TheJulia | JayF: 6 | 22:28 |
TheJulia | err | 22:28 |
TheJulia | JayF: ^^^ reboots the node now | 22:28 |
JayF | 6 reboots?! that's like 500 calls to evaluate_hardware_support /s /s | 22:29 |
TheJulia | just one | 22:29 |
JayF | +2 | 22:29 |
JayF | I was just making a joke :D | 22:29 |
JayF | Oooh, I should look at gate runtimes since IPA change merged | 22:29 |
TheJulia | 2-5 minutes better maybe? | 22:34 |
JayF | I'll say not enough data yet to tell | 22:36 |
TheJulia | noise wise, we may never know for sure | 22:36 |
JayF | I usually look at min() and max() for the time period | 22:36 |
JayF | and see if there is a notable difference | 22:36 |
JayF | (to a similar length sample before) | 22:37 |
TheJulia | I'd really only do that if I could select down the time windows | 22:37 |
JayF | I am basically doing it by hand in the builds tab at zuul.opendev.org lol | 22:37 |
JayF | codespell update appears to have angered the ironic gate | 22:37 |
JayF | patch incoming | 22:37 |
TheJulia | https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/919762 is also working and addresses one of the fun failure cases we've run into with our tests as of recent | 22:48 |
opendevreview | Jay Faulkner proposed openstack/ironic master: Fix issues found/caused by new codespell https://review.opendev.org/c/openstack/ironic/+/920365 | 22:49 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Fix issues caused/found by new codespell https://review.opendev.org/c/openstack/ironic-python-agent/+/920366 | 22:50 |
JayF | cid: adamcarthur5: ^ Those are fixes for codespell in the two repos that are going to hurt the most from having gates blocked, but can one of you find some time tomorrow to check the rest of the ironic repos for issues with new codespell and fix 'em, please? Slack me if you have any questions | 22:50 |
JayF | TheJulia: +2 on the i-t-p change, if you wanna land those ironic/ipa patches I just put up, it'll unbreak those gates :/ | 22:52 |
JayF | and I can recheck the api-ref change with some spare time tomorrow | 22:52 |
JayF | (I'm not working tomorrow since I'm traveling over the weekend) | 22:52 |
TheJulia | I'm off tomorrow, but I'll have laptops if I'm really needed | 22:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!