| opendevreview | wangjiaqi proposed openstack/python-ironicclient master: remove unicode from code https://review.opendev.org/c/openstack/python-ironicclient/+/854477 | 02:25 |
|---|---|---|
| opendevreview | wangjiaqi proposed openstack/python-ironic-inspector-client master: remove unicode from code https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/854479 | 02:38 |
| *** mat_fechner is now known as matfechner | 04:39 | |
| opendevreview | wangjiaqi proposed openstack/networking-generic-switch master: remove unicode from code https://review.opendev.org/c/openstack/networking-generic-switch/+/854515 | 05:49 |
| opendevreview | wangjiaqi proposed openstack/networking-baremetal master: remove unicode from code https://review.opendev.org/c/openstack/networking-baremetal/+/854516 | 05:53 |
| arne_wiebalck | Good mornikng, Ironic! | 06:42 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 07:03 |
| opendevreview | Michal Nasiadka proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 07:16 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 07:35 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 07:36 |
| opendevreview | Radosław Piliszek proposed openstack/tenks master: Add support for EL9 https://review.opendev.org/c/openstack/tenks/+/854550 | 07:37 |
| mgoddard | stevebaker[m], TheJulia thanks for the reviews! will get it fixed up | 08:16 |
| mgoddard | has anyone seen IPA "certificate not yet valid" issues? | 08:22 |
| mgoddard | my bifrost server is in CEST timezone, IPA probably coming up in UTC | 08:23 |
| rpittau | good morning ironic! o/ | 08:23 |
| mgoddard | morning rpittau | 08:24 |
| rpittau | hey mgoddard :) | 08:24 |
| rpittau | mgoddard: btw that looks like an issue with time difference | 08:24 |
| mgoddard | rpittau: yeah | 08:24 |
| mgoddard | allowed clock skew is 1 hour | 08:25 |
| mgoddard | which should be enough | 08:25 |
| mgoddard | maybe I just need NTP | 08:25 |
| mgoddard | looks like ipa-ntp-server is my friend | 08:27 |
| mgoddard | rpittau: worked :D | 10:22 |
| rpittau | awesome :) | 10:32 |
| iurygregory | good morning Ironic | 11:00 |
| *** tosky_ is now known as tosky | 12:25 | |
| smachi | Hello ironic | 12:48 |
| smachi | TheJulia : I have added a query in comment section for patch https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/848709 | 12:50 |
| smachi | can you please look into it | 12:50 |
| TheJulia | Sure, just waking up and making coffee first | 12:54 |
| opendevreview | Merged openstack/ironic-python-agent master: remove unicode prefix from code https://review.opendev.org/c/openstack/ironic-python-agent/+/854160 | 13:01 |
| TheJulia | lovely they /quit | 13:04 |
| TheJulia | *sigh* | 13:04 |
| TheJulia | Good morning! | 13:19 |
| dtantsur | hello from the overheated side of the globe | 14:18 |
| TheJulia | ugh, how bad? | 14:21 |
| dtantsur | around +33 quite consistently | 14:28 |
| dtantsur | +29 in the room where I work | 14:28 |
| dtantsur | oh sorry, +30 already \o/ that's why I'm sweating so much.. | 14:28 |
| rpittau | well it's officially hotter than where I am now, which is very unsettling | 14:33 |
| dtantsur | especially given the lack of the sea :( | 14:33 |
| dtantsur | iurygregory: I'm updating the sushy release patch for Zed. do you plan on having anything else there? | 14:36 |
| iurygregory | dtantsur, we can go ahead (facing some issues with the tests) | 14:37 |
| dtantsur | yeah, tests are fine to backport | 14:37 |
| iurygregory | I can update later today also (it's on my list) | 14:37 |
| dtantsur | I've just updated Elod's patch with 4.3.0 | 14:38 |
| dtantsur | waiting for one patch to merge, then Yoga too | 14:38 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852797 | 14:38 |
| dtantsur | iurygregory: actually, if I'll call it a day already, could you propose a yoga release once https://review.opendev.org/c/openstack/sushy/+/854456/ merges | 14:38 |
| iurygregory | added +1 | 14:38 |
| iurygregory | dtantsur, sure! | 14:39 |
| dtantsur | we need the etag fix | 14:39 |
| iurygregory | ack | 14:39 |
| TheJulia | +33 is a bit much | 14:48 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix software raid output poisoning https://review.opendev.org/c/openstack/ironic-python-agent/+/854437 | 14:54 |
| *** tosky is now known as Guest1129 | 15:12 | |
| *** tosky_ is now known as tosky | 15:12 | |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Trivial: Fix reno for software raid fix https://review.opendev.org/c/openstack/ironic-python-agent/+/854636 | 15:16 |
| kamlesh6808c | TheJulia,iurygregory : would you please help to review https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/853621 | 15:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent bugfix/8.6: Fix software raid output poisoning https://review.opendev.org/c/openstack/ironic-python-agent/+/854639 | 15:25 |
| TheJulia | kamlesh6808c: looking now | 15:29 |
| TheJulia | kamlesh6808c: 3 comments posted. Specifically curious regarding the rollback. is it *really* needed? Is there a reason to do it, or to skip it, or...?! | 15:31 |
| opendevreview | Jay Faulkner proposed openstack/sushy stable/xena: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/854640 | 15:32 |
| opendevreview | Jay Faulkner proposed openstack/sushy stable/victoria: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/854641 | 15:34 |
| kamlesh6808c | Julia, In this we case would like to bring bm on the the same firmware what is was earlier .This is kind of functionality test where we would like to bring bm how it was before execution of test. | 15:40 |
| opendevreview | Merged openstack/sushy stable/yoga: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/854456 | 15:42 |
| kamlesh6808c | Julia, should we not move it back to the firmware version what it was before in cleanup? | 15:46 |
| opendevreview | Merged openstack/sushy stable/wallaby: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/854454 | 15:48 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852797 | 15:50 |
| TheJulia | kamlesh6808c: just thinking about it, it seems like there are "real world" implications to the hardware | 15:51 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/852797 | 15:52 |
| rpittau | bye everyone, see you tomorrow o/ | 16:02 |
| kamlesh6808c | TheJulia, Yeah. Should I remove rollback then ? If user really want to move it back, he would run it again with older firmware. | 16:05 |
| TheJulia | kamlesh6808c: I don't really know. How many times *can* it be updated before the chip can't be flashed anymore? | 16:06 |
| TheJulia | kamlesh6808c: I guess, I *really* don't have an opinion either way, but it seems risky, which is why I raise the concern | 16:08 |
| TheJulia | kamlesh6808c: we actually had someone go manually update firmware on some systems supporting a 3rd party CI once... which actually failed on the servers effectively destroying them. :\ | 16:09 |
| TheJulia | by that, I mean they found a thumb drive in the data center | 16:09 |
| TheJulia | still in one machine... | 16:09 |
| TheJulia | A bit different in this case, but I'm mostly erroring on a side of caution. | 16:10 |
| TheJulia | Which is why I raise the concern to at least promote some thought | 16:10 |
| kamlesh6808c | I really don't know the answer about count that chip can sustain flashing. I am seeing it better not to flash until and unless its really needed. | 16:12 |
| TheJulia | Yeah, and to be clear, that is my *only* hesitation. I think it is going to vary a lot, so maybe we don't need the rollback. If the bmc firmware de-duplicates it, then \o/ | 16:13 |
| TheJulia | but changing version to version... might be pretty hard on it, at least in CI | 16:14 |
| TheJulia | Really, we're testing the interface doesn't explode | 16:14 |
| kamlesh6808c | TheJulia ,Yeah will discard rollback as discussed. | 16:14 |
| TheJulia | okay, cool! | 16:14 |
| TheJulia | Thanks! | 16:14 |
| kamlesh6808c | Thanks for review ! | 16:15 |
| JayF | I'd be extremely careful with flash cycles on those chips | 17:01 |
| JayF | We had the numbers on some of that for devices we had in OnMetal, and chose not to reflash but instead do verification of firmwares because of the ridiculously low write cycles | 17:01 |
| TheJulia | JayF: I seem to remember you or jim mentioning something to that effect at one point, which strengthens my worry :) | 17:49 |
| JayF | some of the hardware they wouldn't tell us the number | 17:49 |
| JayF | only responded telling us pretty ... urgently to not do it lol | 17:49 |
| TheJulia | not surprising | 17:49 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Redfish: Consider password part of the session cache https://review.opendev.org/c/openstack/ironic/+/850259 | 18:06 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Redfish: Consider password part of the session cache https://review.opendev.org/c/openstack/ironic/+/850259 | 18:08 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add docs for VMware deployment https://review.opendev.org/c/openstack/ironic/+/853549 | 18:19 |
| TheJulia | JayF: Can you revisit your -1's on https://review.opendev.org/q/I4bd8b1b4ea974834f1149fcaa79de85d24f5f7d1 | 18:28 |
| TheJulia | they were all dependent upon another patch merging which has since merged | 18:28 |
| opendevreview | Merged openstack/networking-generic-switch master: remove unicode from code https://review.opendev.org/c/openstack/networking-generic-switch/+/854515 | 18:42 |
| opendevreview | Merged openstack/networking-generic-switch master: Try fixing a non-deterministic test failure on the CI https://review.opendev.org/c/openstack/networking-generic-switch/+/848161 | 18:42 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Docs: Add considerations to anaconda docs https://review.opendev.org/c/openstack/ironic/+/854045 | 18:45 |
| JayF | on it | 19:09 |
| TheJulia | muchas gracias | 19:38 |
| opendevreview | Merged openstack/networking-baremetal master: remove unicode from code https://review.opendev.org/c/openstack/networking-baremetal/+/854516 | 20:29 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/851681 how have I not seen this until now. stevebaker[m] I <3 it, reviewing now | 21:33 |
| stevebaker[m] | sweet :) | 21:33 |
| stevebaker[m] | JayF: bifrost switches to using it here https://review.opendev.org/c/openstack/bifrost/+/851682 | 21:34 |
| JayF | I'm not sure I'm up to speed enough on bifrost to review those yet | 21:34 |
| JayF | especially since I'm trying to get up to speed on sushy/ilo/redfish driver stuff now too | 21:34 |
| JayF | but I'll review the ironic side :) | 21:35 |
| stevebaker[m] | ok no problem | 21:35 |
| opendevreview | Merged openstack/networking-generic-switch master: Trim spaces in ngs_trunk_ports and ngs_physical_networks https://review.opendev.org/c/openstack/networking-generic-switch/+/845774 | 21:42 |
| JayF | TheJulia: what should I do about https://review.opendev.org/c/openstack/ironic-python-agent/+/854639 since you just pushed an update to that reno | 21:55 |
| JayF | TheJulia: JFDI fix in that commit? | 21:55 |
| TheJulia | with reno's design, I don't think you need to do anything | 21:55 |
| TheJulia | but you can pull that in | 21:56 |
| TheJulia | up to you :) | 21:56 |
| TheJulia | master branch's content is always viewed as the correct content for renos | 21:56 |
| TheJulia | fwiw | 21:56 |
| JayF | ack; I guess if that ends up being not-true it's not hard to rebase the trivial fix :D | 21:57 |
| opendevreview | Merged openstack/ironic master: Add kickstart template 'url' option https://review.opendev.org/c/openstack/ironic/+/853368 | 22:00 |
| opendevreview | Merged openstack/ironic-python-agent master: Trivial: Fix reno for software raid fix https://review.opendev.org/c/openstack/ironic-python-agent/+/854636 | 22:00 |
| *** tosky_ is now known as tosky | 22:06 | |
| *** dmellado_ is now known as dmellado | 22:37 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!