| iurygregory | a bug involving nofreecondutor, jesus... D: | 01:25 |
|---|---|---|
| TheJulia | What did somebody do?! | 01:46 |
| iurygregory | a huge deployment... and the virtual bm can't request anything | 02:27 |
| iurygregory | D: | 02:27 |
| iurygregory | funny that BMO gives 502 and Ironic 503 :D | 02:28 |
| opendevreview | Harald Jensås proposed openstack/sushy-tools master: [Libvirt] Support firmware auto-selection https://review.opendev.org/c/openstack/sushy-tools/+/914420 | 03:45 |
| rpittau | good morning ironic! o/ | 07:55 |
| rpittau | JayF: thanks for the emails! | 07:55 |
| rpittau | TheJulia: considering that the secureboot DB Certificates are read only from Redfish schema, I wonder if that is just a link to the certificate collection | 08:05 |
| rpittau | looking again at the redfish schema it seems to suggest that, also consdering that one of the allowed uris for certificatecollection is indeed redfish/v1/Systems/{ComputerSystemId}/SecureBoot/SecureBootDatabases/{DatabaseId}/Certificates | 08:05 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/2024.1: Handle NotAcceptable when Accept-Encoding: identity is not allowed https://review.opendev.org/c/openstack/sushy/+/914442 | 10:04 |
| rpittau | JayF: related to the pypi maintainers cleanup, how's the status for bifrost? we got any answer? | 10:26 |
| iurygregory | good morning Ironic | 11:27 |
| dtantsur | Folks, I know https://review.opendev.org/c/openstack/sushy-tools/+/875366 is huge and somewhat annoying to review, but could we get more eyes on it? | 11:29 |
| opendevreview | Merged openstack/sushy stable/2024.1: Handle NotAcceptable when Accept-Encoding: identity is not allowed https://review.opendev.org/c/openstack/sushy/+/914442 | 12:13 |
| dtantsur | What's the maintenance status of sushy-oem-idrac? rpittau you seem to be the only active core | 12:50 |
| opendevreview | Daniel King proposed openstack/ironic-python-agent master: Fix mocking for TestGenericHardwareManager https://review.opendev.org/c/openstack/ironic-python-agent/+/913208 | 13:18 |
| opendevreview | Daniel King proposed openstack/ironic-python-agent master: Add get_additional_skip_list and get_skip_list https://review.opendev.org/c/openstack/ironic-python-agent/+/913209 | 13:18 |
| dtantsur | JayF: btw, you said in the past you're interested in gophercloud reviews. Here is one https://github.com/gophercloud/gophercloud/pull/3014 :) | 13:22 |
| rpittau | dtantsur: haven't looked at it since a while :/ | 13:27 |
| rpittau | I'll try to get it in shape, I see there's some adjustments to do | 13:27 |
| dtantsur | rpittau: yeah, janders has discovered a bug in one rarely used part | 13:28 |
| rpittau | yeah I bet | 13:28 |
| rpittau | it's really been a while | 13:28 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Update setup and tox config https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914515 | 13:32 |
| rpittau | whoop forgot a part :P | 13:32 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Update setup and tox config https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914515 | 13:34 |
| rpittau | oooook let's see how it goes | 13:34 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Add codespell non voting test https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914517 | 13:39 |
| TheJulia | rpittau: strictly speaking, they are saying the link collection in the schema is read only, I think that is one of the base points where things fall apart | 13:41 |
| TheJulia | rpittau: furthermore, dell's url is something like SecureBoot/DB/Certificates | 13:42 |
| TheJulia | dtantsur: rpittau is there really a need w/r/t sushy-oem-idrac at this point? | 13:43 |
| rpittau | TheJulia: yeah but that could be a Dell only implementation | 13:43 |
| TheJulia | rpittau: exactly, and it all depends on what the underlying links are | 13:43 |
| rpittau | alright, | 13:44 |
| dtantsur | TheJulia: unfortunately, janders and I are even going to add another case for it... | 13:44 |
| rpittau | I see | 13:44 |
| TheJulia | I seriously think they were looking at SecureBoot, hit page down to SecureBootDatabase, and implemented SecureBootDatabase inside SecureBoot | 13:44 |
| TheJulia | dtantsur: le sigh | 13:44 |
| rpittau | TheJulia: it looks like it, it does not look like Redfish standard | 13:44 |
| TheJulia | yeah | 13:45 |
| JayF | rpittau: we're waiting on Monty, that's the project Monty still has access to (pypi username: sdirector). It's in the bottom of that etherpad in the list of unresolved ones | 13:55 |
| rpittau | JayF: thanks! | 13:55 |
| JayF | rpittau: </topic><topic> Before I respond to the mailing list, do you want to do the openinfra live for caracal or should I? | 13:55 |
| JayF | I'm happy to have someone else up there talking about Ironic Caracal, I'm also happy to do it myself | 13:55 |
| rpittau | JayF: you're out next week though? | 13:56 |
| JayF | rpittau: so as I said, since I'll be in Vegas I really think you should tell people how awesome Ironic Caracal is :D | 13:57 |
| JayF | lol | 13:57 |
| rpittau | :D | 13:57 |
| rpittau | sounds good! I'll do it :) | 13:57 |
| JayF | awesome, I'll let you respond on the list then | 13:57 |
| rpittau | cool :) | 13:57 |
| JayF | dtantsur: it's been so long since I've looked at golang! | 13:59 |
| JayF | dtantsur: that's not a no, I appreciate the link, more that I'm surprised how much my ability to read it has atophied :D | 14:00 |
| dtantsur | :D | 14:00 |
| dtantsur | That's your opportunity! | 14:01 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Update setup and tox config https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914515 | 14:09 |
| opendevreview | Riccardo Pittau proposed openstack/sushy-oem-idrac master: Add codespell non voting test https://review.opendev.org/c/openstack/sushy-oem-idrac/+/914517 | 14:09 |
| opendevreview | Stephen Finucane proposed openstack/python-ironicclient master: Remove use of distutils https://review.opendev.org/c/openstack/python-ironicclient/+/914523 | 14:15 |
| opendevreview | Stephen Finucane proposed openstack/python-ironicclient master: trivial: Remove note about requirements ordering https://review.opendev.org/c/openstack/python-ironicclient/+/914524 | 14:15 |
| dking | JayF: I updated https://review.opendev.org/c/openstack/ironic-python-agent/+/913209, but since it's an update of the former commit or maybe because it's the same branch, is there anything that I need to do to have zuul pick it up again? | 14:17 |
| JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/913209/2#message-27e6cd11b7c630af5e7bc1697ba6d3a30becca59 indicates to me it's already churning | 14:17 |
| JayF | You can also go to zuul.opendev.org -> click "status" beside openstack -> put your change number (913209) into search and see CI in progress | 14:18 |
| dking | Oh, thanks. I suppose that I didn't see that somehow. | 14:19 |
| JayF | (if you click in the empty space around your change once doing that, it'll expand to jobs in progress) | 14:19 |
| JayF | so often if I know a specific CI job/test is what I'm trying to pass, I can use that and get early feedback | 14:19 |
| TheJulia | Hey folks, here is a semi-quick review for an issue with sushy-tools which hjensas put forward. https://review.opendev.org/c/openstack/sushy-tools/+/914420 | 14:22 |
| JayF | OK that libvirt features sounds awesome | 14:24 |
| TheJulia | so not awesome when you create a VM, then try to let ironic sync the boot mode again | 14:25 |
| TheJulia | well, without that patch == not awesome | 14:25 |
| JayF | dtantsur: So for the gophercloud patch, is it a good idea to encode force deletion as the default behavior for delete node in gophercloud? | 14:26 |
| JayF | dtantsur: seems like our safeguard requiring maint mode is completely subverted if the client doesn't raise the error? Is there some context I'm missing? | 14:26 |
| TheJulia | I *suspect* there is a programatic view and an operational reality gap this basically highlights | 14:30 |
| JayF | dtantsur: aha, I just saw it's in /acceptance/ -- is it not a go thing to always have test in the filename? | 14:30 |
| JayF | TheJulia: I think it's actually just acceptance test helper code that wasn't labelled how I expected | 14:30 |
| TheJulia | heh | 14:30 |
| TheJulia | okay, I'm less worried now | 14:30 |
| * TheJulia goes back to to the document on the other screen | 14:30 | |
| rpittau | dtantsur: I've given core rights for sushy-oem-idrac to ironic-core, most of the other cores of the project are not active anymore | 15:01 |
| dtantsur | ++ | 15:02 |
| dtantsur | JayF: acceptance seems to be something gophercloud-specific | 15:02 |
| dtantsur | but yeah, you're asking about a test helper | 15:02 |
| JayF | makes sense, I was looking for the blah_test.go moniker and didn't see it and went WTF | 15:02 |
| JayF | but when I looked closely I found what was up | 15:02 |
| opendevreview | Mark Goddard proposed openstack/bifrost master: CI: Modify upgrade jobs to upgrade from 2024.1 (Caracal) https://review.opendev.org/c/openstack/bifrost/+/913679 | 15:07 |
| opendevreview | Merged openstack/sushy-tools master: [Libvirt] Support firmware auto-selection https://review.opendev.org/c/openstack/sushy-tools/+/914420 | 15:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: automatically set the branch for IPA to match TARGET_BRANCH https://review.opendev.org/c/openstack/ironic/+/914250 | 15:43 |
| JayF | TheJulia: quick comment on that, if you address I'll upgrade to +2 | 15:50 |
| TheJulia | JayF: what do you mean by 'this' | 15:52 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/914250/2#message-5e283bd6b7236db9736e55fc06b429b00a4ea248 | 15:52 |
| TheJulia | the line specifically | 15:52 |
| TheJulia | I don't understand your comment | 15:52 |
| JayF | oh, the comment above from jroll | 15:52 |
| JayF | we don't need to update that on branch cut anymore AIUI | 15:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: automatically set the branch for IPA to match TARGET_BRANCH https://review.opendev.org/c/openstack/ironic/+/914250 | 15:53 |
| TheJulia | Nah, not with target branch being around | 15:53 |
| JayF | +2 | 15:53 |
| JayF | thanks :D | 15:53 |
| rpittau | bye everyone see you tomorrow o/ | 16:07 |
| dking | JayF: It looks like the tests completed for https://review.opendev.org/c/openstack/ironic-python-agent/+/913209 | 16:23 |
| JayF | nice, if you make sure it's hashtagged with 'ironic-week-prio' then it'll be in line for a review next time cores do a pass through | 16:24 |
| dking | Is that for the big or for that commit? | 16:25 |
| JayF | for anything that needs review | 16:27 |
| JayF | and is passing CI/generally ready for review | 16:27 |
| dking | JayF: It's been a while. Where would I add that? | 16:27 |
| JayF | top left around where topic is | 16:27 |
| JayF | you have to hit "Show All" | 16:27 |
| dking | Ah! I see it. | 16:29 |
| dking | Done. Sorry, I didn't think about it being hidden and thought that I might be lacking permissions. | 16:30 |
| JayF | not a problem, now it'll show up in our review dashboards | 16:32 |
| opendevreview | Mark Goddard proposed openstack/bifrost master: CI: Add a non-voting SLURP upgrade job for Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/913680 | 16:51 |
| JayF | mgoddard: https://review.opendev.org/c/openstack/ironic/+/848164 looks like a good fix; do you intend on pushing it forward? | 16:52 |
| opendevreview | Jay Faulkner proposed 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 | 17:14 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Special treatment of .json is now disabled for nodes with .json extension. https://review.opendev.org/c/openstack/ironic/+/913467 | 17:16 |
| opendevreview | cid proposed openstack/ironic-inspector master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic-inspector/+/913794 | 18:19 |
| opendevreview | cid proposed openstack/ironic-inspector master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic-inspector/+/913794 | 18:26 |
| opendevreview | cid proposed openstack/ironic master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic/+/913793 | 18:33 |
| opendevreview | Julia Kreger proposed openstack/ironic master: ci: automatically set the branch for IPA to match TARGET_BRANCH https://review.opendev.org/c/openstack/ironic/+/914250 | 20:46 |
| TheJulia | or i had a syntax error.... | 20:47 |
| iurygregory | it happens | 20:47 |
| JayF | TheJulia: https://usercontent.irccloud-cdn.com/file/JM56HyB1/happy.png | 20:48 |
| TheJulia | heh | 20:52 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/zed: CI: Lock Zed CI to Zed IPA https://review.opendev.org/c/openstack/ironic/+/914243 | 21:03 |
| TheJulia | doh! | 21:03 |
| opendevreview | Merged openstack/ironic master: Fix new codespell issues; tweak config https://review.opendev.org/c/openstack/ironic/+/913265 | 22:13 |
| opendevreview | Merged openstack/ironic master: Drop fallback to unmanaged inspection for virtual media and UEFI boot https://review.opendev.org/c/openstack/ironic/+/913523 | 22:19 |
| TheJulia | nice | 22:38 |
| TheJulia | so, question, would it make sense to try and discuss https://review.opendev.org/c/openstack/ironic-specs/+/890164 in advance of the ptg? | 22:41 |
| JayF | if you have time tomorrow or friday, I do | 22:56 |
| JayF | basically clear anytime after 930am tomorrow | 22:56 |
| TheJulia | i should, basically waiting word on my father who is back in the ER at the moment | 22:58 |
| TheJulia | but some hurry up and waiting going on | 22:58 |
| JayF | ack; well just let me know when you get some time free and we can look together | 22:59 |
| JayF | I am trying to use some of this gaping empty time tomorrow/friday to also start on nova guest metadata work | 22:59 |
| JayF | that'll need to be in for me to do the ironic side, and I think it all might come together pretty quickly | 22:59 |
| JayF | nova patch to send metadata to ironic + an ironic patch to use that metadata if it exists and automatic_lessee is enabled | 22:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
