| opendevreview | Merged openstack/ironic stable/xena: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852507 | 00:00 |
|---|---|---|
| opendevreview | Merged openstack/ironic stable/xena: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852656 | 00:00 |
| opendevreview | Merged openstack/ironic stable/yoga: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852506 | 00:00 |
| opendevreview | Merged openstack/ironic stable/yoga: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852509 | 00:00 |
| TheJulia | JayF: eh... I had once thought of oslo_db getting a backend | 00:46 |
| opendevreview | Merged openstack/ironic stable/wallaby: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852508 | 01:06 |
| opendevreview | Merged openstack/ironic stable/wallaby: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852657 | 01:09 |
| opendevreview | Merged openstack/ironic stable/victoria: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852662 | 01:09 |
| opendevreview | Merged openstack/ironic stable/train: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852678 | 01:09 |
| opendevreview | Merged openstack/ironic master: Log successful clean up in image cache https://review.opendev.org/c/openstack/ironic/+/852603 | 01:10 |
| opendevreview | Merged openstack/ironic stable/ussuri: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852676 | 01:10 |
| opendevreview | Merged openstack/ironic master: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/851802 | 02:11 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: skip header lookup on HTTPS https://review.opendev.org/c/openstack/ironic/+/852694 | 03:22 |
| rpittau | good morning ironic! o/ | 06:45 |
| opendevreview | Vishal Manchanda proposed openstack/ironic-ui master: [DNM] Migrate to Angularjs v1.8.2 https://review.opendev.org/c/openstack/ironic-ui/+/852702 | 07:06 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/yoga: Log successful clean up in image cache https://review.opendev.org/c/openstack/ironic/+/852633 | 08:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/20.2: Log successful clean up in image cache https://review.opendev.org/c/openstack/ironic/+/852634 | 08:16 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/850547 | 08:19 |
| dtantsur | ajya: PTAL ^^ | 08:20 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852635 | 08:29 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852636 | 08:29 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852637 | 08:30 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: Enable skipping disks for cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/850861 | 09:28 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/19.0: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852638 | 09:47 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/yoga: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852639 | 09:47 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/18.1: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852640 | 09:47 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/xena: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852641 | 09:48 |
| opendevreview | Aija Jauntēva proposed openstack/ironic stable/wallaby: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852642 | 09:49 |
| dtantsur | ajya: have we backported ^^ to 20.0 and 20.2 as well? | 10:17 |
| ajya | dtantsur: no. Wanted to asked about those. Is 20.0 still needed? Or only 20.2? | 10:18 |
| dtantsur | ajya: we need to at least backport to everywhere the original patch was backported | 10:19 |
| dtantsur | 20.0.. I don't have strong opinions about it (I do about 20.2) | 10:19 |
| ajya | 20.0 is not listed as supported branch in Ironic WB and original patch is not there | 10:21 |
| ajya | also original patch was not backported to 20.2. If it's needed there, then both (original&followup) need backport | 10:21 |
| dtantsur | I'd appreciate a backport to 20.2 | 10:25 |
| ajya | ok | 10:30 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/20.2: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/852726 | 10:38 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/20.2: Update raid_type handling for Redfish raid_config https://review.opendev.org/c/openstack/ironic/+/852727 | 10:38 |
| opendevreview | Merged openstack/sushy master: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/850547 | 11:32 |
| opendevreview | Merged openstack/ironic bugfix/20.2: Log successful clean up in image cache https://review.opendev.org/c/openstack/ironic/+/852634 | 11:46 |
| iurygregory | good morning Ironic | 11:54 |
| opendevreview | Merged openstack/ironic stable/yoga: Log successful clean up in image cache https://review.opendev.org/c/openstack/ironic/+/852633 | 12:12 |
| TheJulia | good morning | 13:36 |
| * rpittau goes finish packing | 13:46 | |
| * rpittau see you tomorrow o/ | 13:46 | |
| TheJulia | o/ | 13:54 |
| *** mat_fechner is now known as matfechner | 13:58 | |
| sschmitt | Hey all, for some reason I'm seeing VM flavors getting scheduled onto hardware nodes all of a sudden. Any ideas on that? | 14:04 |
| sschmitt | Not only that, its trying to schedule instances on an ironic node that already has an instance running on it | 14:11 |
| TheJulia | Good morning | 14:19 |
| TheJulia | sschmitt: is your scheduling using resource classes? | 14:20 |
| sschmitt | our hardware node flavors do | 14:21 |
| sschmitt | VM flavors do not | 14:22 |
| iurygregory | good morning TheJulia | 14:22 |
| TheJulia | oh... | 14:25 |
| TheJulia | so... I would think VMs would need to since their scheudling could otherwise cross over I guess | 14:26 |
| sschmitt | I mean that makes sense in my mind its just weird we've been doing it without that for a long time and it just now broke | 14:27 |
| sschmitt | Its also failing as its trying to schedule a vm onto a hardware node that already has an instance running on it, so it fails | 14:27 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Correct Image properties lookup for paths https://review.opendev.org/c/openstack/ironic/+/852694 | 14:31 |
| sschmitt | Hah, think I fixed it. Putting the compute nodes into a host aggregate group made it all deploy correctly | 14:32 |
| TheJulia | Yeah, isolate scheduling/resource allocations | 14:32 |
| TheJulia | using a resource class or even just traits would do it as well | 14:32 |
| sschmitt | yeah I'll see about adding resource classes to all our flavors going forward | 14:33 |
| sschmitt | Thank you! | 14:33 |
| TheJulia | No problem, good luck! | 14:36 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 14:50 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/20.2: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852756 | 15:22 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/19.0: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852774 | 15:32 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/18.1: Modify test code to avoid CONF modification affection https://review.opendev.org/c/openstack/ironic/+/852757 | 15:34 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Override external_http_url at node level https://review.opendev.org/c/openstack/ironic/+/852234 | 16:02 |
| opendevreview | Merged openstack/ironic bugfix/20.2: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852635 | 16:18 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852636 | 16:18 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Change molds option to appropriate class https://review.opendev.org/c/openstack/ironic/+/852637 | 16:18 |
| iurygregory | TheJulia, JayF , dtantsur https://review.opendev.org/c/openstack/ironic/+/852234 new version basded on the discussion (the idea to use specific config will be done later so can discuss details on how it should look like etc) | 16:28 |
| JayF | ack, ty for the ping | 16:28 |
| TheJulia | Thanks, I can take a look in a little bit | 17:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Correct Image properties lookup for paths https://review.opendev.org/c/openstack/ironic/+/852694 | 18:09 |
| TheJulia | iurygregory: left a few comments. The reno seems a little awkward | 19:25 |
| TheJulia | and actually, I just realized it is missing anything in docs. | 19:25 |
| iurygregory | TheJulia, ack o/ I'm going to the gym in a few, will take a look when I'm back o/ | 19:26 |
| iurygregory | tks! | 19:26 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Correct Image properties lookup for paths https://review.opendev.org/c/openstack/ironic/+/852694 | 21:16 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Allow project scoped admins to create/delete nodes https://review.opendev.org/c/openstack/ironic/+/852794 | 22:15 |
| TheJulia | dtantsur: iurygregory: JayF: You guys might be interested in ^^^, seems logical to avoid "you must make custom policy files" and also enables the rbac goal pivot compliance. Thoughts? Oh, and actual better delineated including self service ironic... | 22:16 |
| iurygregory | TheJulia, will add to my list to take a look o/ | 22:16 |
| JayF | You do not want me reviewing that right now with the state of my brain, but I added myself as a reviewer and will dedicate some time to Ironic tomorrow | 22:18 |
| JayF | nothing like working on github actions for days straight to make you appreciate the one true overlord, zuul | 22:18 |
| TheJulia | heh | 22:19 |
| TheJulia | "one true overloard, zuul" | 22:19 |
| TheJulia | that seems t-shirt worthy | 22:19 |
| JayF | isn't that like, a thing? | 22:20 |
| JayF | with zuul? | 22:20 |
| JayF | not like, zuul the software but zuul the mythical creature/god/whatever? | 22:20 |
| TheJulia | ironic_tempest_plugin.common.waiters] (BaremetalRedfishIPxeAnacondaNoGlance:test_ip_access_to_server_using_stage2) Node 1a014c9a-a8ba-4a14-bc4e-d280aed9b2a1 failed to reach provision_state=['active'] within the required time (1800 s). <-- sadness | 22:22 |
| TheJulia | dunno. I was always thinking Cthulhu | 22:22 |
| TheJulia | hmm... still in wait_callback :\ | 22:22 |
| iurygregory | TheJulia, for the reno https://paste.opendev.org/show/bvMswO2pbVSzi6nUAo7o/ wdyt? | 22:24 |
| TheJulia | iurygregory: perhaps s/via/via the/ | 22:24 |
| TheJulia | other than that, ship it | 22:25 |
| JayF | ++ | 22:25 |
| JayF | push that now you have both of us here | 22:25 |
| JayF | MERGE PARTY | 22:25 |
| iurygregory | \o/ | 22:25 |
| iurygregory | just waitting for pep8 and releasenote test to finish :D | 22:25 |
| TheJulia | https://ef2ad358dccb56dd5b1d-616db0a32cd5a8455c371f4b45f3133e.ssl.cf5.rackcdn.com/835917/12/check/ironic-standalone-redfish/e5d3b68/controller/logs/ironic-bm-logs/node-1_console_2022-08-10-22%3A10%3A50_log.txt <-- soooooo close | 22:28 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Override external_http_url at node level https://review.opendev.org/c/openstack/ironic/+/852234 | 22:29 |
| TheJulia | https://ef2ad358dccb56dd5b1d-616db0a32cd5a8455c371f4b45f3133e.ssl.cf5.rackcdn.com/835917/12/check/ironic-standalone-redfish/e5d3b68/controller/logs/ironic-bm-logs/node-1_no_ansi_2022-08-10-22%3A10%3A50_log.txt <-- greater sadness, less close, but still closer | 22:29 |
| iurygregory | I got "Not Found" from this url O.o | 22:29 |
| iurygregory | the first one | 22:29 |
| iurygregory | ok, both are giving Not Found here... | 22:30 |
| JayF | iurygregory: +2 | 22:30 |
| iurygregory | JayF, ty! | 22:30 |
| JayF | iurygregory: that's weird, that URL does work for me | 22:30 |
| TheJulia | freaky... | 22:30 |
| TheJulia | they work here | 22:30 |
| iurygregory | O.o | 22:32 |
| iurygregory | let me check in another browser | 22:32 |
| iurygregory | cache?! wtf | 22:34 |
| TheJulia | yay for distributed systems? | 22:34 |
| iurygregory | like, if I open in firefox private window it works | 22:34 |
| TheJulia | and eventual consistency?! | 22:34 |
| TheJulia | cbouchar: o/ so, I'm getting super close to getting anaconda in CI | 22:35 |
| TheJulia | cbouchar: specifically, even though we have 1gb of ram, it looks like we're having network boot sadness. https://ef2ad358dccb56dd5b1d-616db0a32cd5a8455c371f4b45f3133e.ssl.cf5.rackcdn.com/835917/12/check/ironic-standalone-redfish/e5d3b68/controller/logs/ironic-bm-logs/node-1_console_2022-08-10-22%3A10%3A50_log.txt If you could check the command line, I'm wondering if we need to agument the options at all | 22:35 |
| TheJulia | "Initramfs unpacking failed: invalid magic at start of compressed archive" | 22:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Anaconda CI changes https://review.opendev.org/c/openstack/ironic/+/849587 | 22:46 |
| TheJulia | bahahahhahaha | 22:46 |
| TheJulia | so... I suspect it will still run out of memory | 22:47 |
| TheJulia | but... getting there! | 22:47 |
| iurygregory | yeah, sounds like OOM | 22:54 |
| TheJulia | I suddenly want cookies | 22:55 |
| TheJulia | OOM Cookies could be a thing... right?! | 22:57 |
| iurygregory | yes! | 23:05 |
| * TheJulia could use OOM Cookies | 23:08 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!