Thursday, 2022-04-28

TheJulia\o/ v6 works00:26
*** diablo_rojo is now known as Guest302200:55
*** Guest3022 is now known as diablo_rojo01:03
opendevreviewZhouHao proposed openstack/ironic stable/yoga: [iRMC] Change the way to get irmc-info in raid  https://review.opendev.org/c/openstack/ironic/+/83967501:31
arne_wiebalckGood morning, Ironic!06:32
rpittaugood morning ironic! o/06:49
rpittauTheJulia: great! :)06:49
opendevreviewHarald Jensås proposed openstack/ironic master: Exclude current conductor from offline_conductors  https://review.opendev.org/c/openstack/ironic/+/83970408:26
opendevreviewHarald Jensås proposed openstack/ironic master: Exclude current conductor from offline_conductors  https://review.opendev.org/c/openstack/ironic/+/83970408:28
opendevreviewRiccardo Pittau proposed openstack/ironic-inspector master: Fix lower-constraints job  https://review.opendev.org/c/openstack/ironic-inspector/+/83970809:04
*** diablo_rojo is now known as Guest306209:09
iurygregorymorning Ironic o/12:18
rpittauI have a bad feeling about grenade job in inspector, it's failing bad https://review.opendev.org/c/openstack/ironic-inspector/+/83970812:48
TheJuliagood morning13:51
TheJuliarpittau: is it trying to use ovn or ovs?13:52
TheJuliait says no ports available, which makes me think misconfiguration13:52
rpittauTheJulia: I think it's OVS13:53
rpittauyeah it's OVS13:54
TheJuliahjensas: I feel conflicted w/r/t your conductor exclusion change13:55
TheJuliawe're basically talking an extreme condition which is a sign of a much worse problem13:56
TheJuliaand I guess it is reasonable for the software not to break itself.... but the system is already in a super bad state due to starvation13:56
dtantsurwell, we shouldn't start doing something wild on the resource starvation14:01
TheJuliayeah, and knowing it is starvation is *hard*14:05
TheJuliadownside is if another conductor declares the starved conductor as dead, it can/will lead to issues as well14:06
TheJuliabut... maybe this will be a little less db churn too14:06
TheJuliadunno14:06
TheJuliaI added comments and approved the change, also flagged for backporting14:06
dtantsuryeah, we cannot fix what another conductor does14:06
dtantsurbut taking over itself is pointless either way14:07
TheJuliayeah14:07
TheJuliahjensas: lmk if you have any concerns regarding the change of the v6 job to use dhcpv6-stateless14:08
hjensasTheJulia: I think stateless will not work at all with UEFI, unless it is iPXE. OVMF expect statefull dhcpv614:13
TheJuliaI guess that was why we changed from stateless to stateful14:13
TheJuliahmmmm14:14
TheJuliaThe conundrum is the host ID changes, I guess we're exhausting dhcpv6 record data. One of the prior failed jobs, centos8 booted but never got a v6 address beyond the local link and route advertiser14:14
TheJuliaso it was kind of braindead14:14
TheJuliaand kept trying to dhcp new addresses14:15
hjensasIs it neutron interface or flat?14:15
TheJuliaThe job is set to neutron14:15
TheJuliaugh, ngs... why your unit tests hate me14:16
hjensasSo it should have 4 addresses allocated by default for any provision/clean etc14:16
hjensasoh, so it deployed, and boot a C8 image? Then the instance never get an address?14:17
TheJuliaIPA14:17
TheJuliaIPA never got network connectivity14:17
TheJuliagive me a couple and I'll see if I can find the log14:17
TheJuliafreaky, my ngs fix worked on its own job but not on my multinode grenade14:20
TheJuliaoh... wait14:20
TheJuliamaybe we're not actually upgrading it?!14:20
TheJuliagah14:20
TheJuliaand if grenade is master, not master + current change14:21
TheJuliayeah14:21
TheJuliaugh14:21
TheJuliahjensas: https://1a74a0482f2b4bd22e89-19b00ab3af46f59a1799d97d7ae09a21.ssl.cf1.rackcdn.com/839086/9/check/ironic-tempest-ipxe-ipv6/9db579f/controller/logs/ironic-bm-logs/node-0_console_2022-04-27-17%3A09%3A48_log.txt14:22
hjensasTheJulia: Is it ubuntu focal's dnsmasq?14:44
hjensasTheJulia: I wonder if it does not have the patches to support multiple IPv6 addresses for one dhcp-host entry.14:45
hjensasTheJulia: I don't see -> https://paste.opendev.org/show/bqLr5cWvYqoWg7Dfv61M/ paragraph in the ubuntu focal dnsmasq manual page.14:46
hjensasstack@devstack:~/devstack$ dnsmasq --version14:47
hjensasDnsmasq version 2.80  Copyright (c) 2000-2018 Simon Kelley14:47
hjensasI think we need 2.8214:47
TheJuliayou14:51
TheJuliaerr14:51
TheJuliajoy14:51
TheJuliaso, bios only, we can have a working job for now14:51
TheJuliabut we're going to need to get it to UEFI eventually I guess14:51
TheJuliaI guess we can note that and move on for now?14:51
TheJuliafyi, jammy has 2.8614:52
TheJuliaSo as soon as we upgrade to jammy we will be in a far better state14:52
TheJuliaand also, the firmware default supports other fun mixes14:52
hjensasTheJulia: sounds like a plan, wait for jammy.14:54
hjensasTheJulia: or use Fedora? *hides behid a rock* :D14:54
TheJuliacan't do the ramdisk stuffs on fedora14:54
dtantsurplaying catch-up with fedora versions is not fun14:55
TheJuliathat too14:59
TheJuliait is basically unstable from my point of view14:59
opendevreviewMerged openstack/ironic master: Exclude current conductor from offline_conductors  https://review.opendev.org/c/openstack/ironic/+/83970415:33
TheJuliahjensas: please backport when you get a chance :)15:44
rpittaugood night! o/16:02
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: CI: Fix Multinode ssh key file placement  https://review.opendev.org/c/openstack/networking-generic-switch/+/83964516:21
opendevreviewElod Illes proposed openstack/ironic stable/victoria: [stable-only][CI] Install dependencies for docs target  https://review.opendev.org/c/openstack/ironic/+/83979416:48
opendevreviewHarald Jensås proposed openstack/ironic stable/yoga: Exclude current conductor from offline_conductors  https://review.opendev.org/c/openstack/ironic/+/83973916:57
hjensasTheJulia: ack, yoga backport ^. I'll have to look at xena and wallaby later. Conflicts must be resolved.16:58
opendevreviewDmitry Tantsur proposed openstack/bifrost master: [WIP] Ability to generate fake nodes  https://review.opendev.org/c/openstack/bifrost/+/83979617:00
TheJuliaack, thanks!17:01
* TheJulia wonders how she is going to fix ngs unit tests17:01
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: CI: Fix Multinode ssh key/hostname for unit tests  https://review.opendev.org/c/openstack/networking-generic-switch/+/83964517:26
opendevreviewElod Illes proposed openstack/ironic stable/victoria: [stable-only][CI] Install dependencies for docs target  https://review.opendev.org/c/openstack/ironic/+/83979418:01
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: CI: Fix Multinode ssh key/hostname for unit tests  https://review.opendev.org/c/openstack/networking-generic-switch/+/83964518:10
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: CI: Fix Multinode ssh key/hostname for unit tests  https://review.opendev.org/c/openstack/networking-generic-switch/+/83964518:27
opendevreviewJulia Kreger proposed openstack/networking-generic-switch master: CI: Fix Multinode ssh key/hostname for unit tests  https://review.opendev.org/c/openstack/networking-generic-switch/+/83964520:39

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!