| opendevreview | Julia Kreger proposed openstack/ironic master: Add service steps call to agent logic https://review.opendev.org/c/openstack/ironic/+/891229 | 00:24 |
|---|---|---|
| stevebaker[m] | z0x!c.Zv | 01:54 |
| stevebaker[m] | omfg | 01:54 |
| * stevebaker[m] runs passwd and restores previous lock screen settings | 01:56 | |
| TheJulia | C’est la vie | 02:05 |
| JayF | congratulations on using a secure password | 03:01 |
| JayF | or wait, I think the line is supposed to be "I only saw ********" | 03:01 |
| stevebaker[m] | It could be longer, not a complete embarrassment | 06:03 |
| rpittau | good morning ironic! o/ | 06:10 |
| iurygregory | good morning Ironic | 09:58 |
| rpittau | hey iurygregory :) | 09:59 |
| iurygregory | o/ | 09:59 |
| rpittau | iurygregory: I take advantage of your presence, I proposed the release of the bugfix branches, please check them when you have a look :) | 10:00 |
| rpittau | https://review.opendev.org/c/openstack/releases/+/892291 | 10:00 |
| rpittau | https://review.opendev.org/c/openstack/releases/+/892292 | 10:00 |
| rpittau | https://review.opendev.org/c/openstack/releases/+/892293 | 10:00 |
| rpittau | s/look/moment | 10:00 |
| rpittau | I really need vacation... | 10:01 |
| iurygregory | rpittau, ++ for vacation | 10:01 |
| iurygregory | ack let me take a look | 10:01 |
| iurygregory | rpittau, before adding +1 to inspector I think it would be good to check the current state of the repo, given I saw failures in the functional tests on the inspector client, wdyt? | 10:05 |
| iurygregory | see https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/891976 | 10:06 |
| rpittau | iurygregory: as you wish, but if it's just tests I think we'll have to backport the fix anyway | 10:10 |
| rpittau | that's on the client though, not on inspector itself | 10:12 |
| iurygregory | don't we run the functional tests also in ironic-inspector? | 10:12 |
| iurygregory | maybe I'm still sleeping lol | 10:12 |
| * iurygregory grabs a second mug of coffee | 10:13 | |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: [DNM] Testing Gate https://review.opendev.org/c/openstack/ironic-inspector/+/840891 | 10:16 |
| iurygregory | just to be sure that the CI is not broken :D | 10:17 |
| iurygregory | rpittau, +1 =) | 10:41 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Add inspection hooks https://review.opendev.org/c/openstack/ironic/+/890817 | 10:57 |
| rpittau | iurygregory: we don't run the client job in the inspector CI | 11:19 |
| rpittau | an thanks :) | 11:20 |
| iurygregory | yw o/ | 11:20 |
| hjensas | TheJulia: "ip link set mtu 1400 br-ex" <- solves the OVN stalled transfer issue. In dmesg prior to lowering br-ex MTU I saw "ovs-node-0i1: dropped over-mtu packet: 1500 > 1400" and "net_ratelimit: 42 callbacks suppressed" | 12:47 |
| rpittau | bye everyone, see you tomorrow! o/ | 12:50 |
| TheJulia | hjensas: I feel the need to table flip because it literally is not in the kernel log of my test vm :( | 13:02 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Add inspection hooks https://review.opendev.org/c/openstack/ironic/+/890817 | 13:19 |
| JayF | TheJulia: so hilariously, in #openstack-infra they were digging AFS problems today/yesterday | 14:06 |
| JayF | TheJulia: guess what? OVN MTU shenanigans. | 14:06 |
| TheJulia | wheeeeeee | 14:07 |
| JayF | TheJulia: https://bugs.launchpad.net/neutron/+bug/2032817 sounds like the desc could cover what you say but idk | 14:54 |
| TheJulia | wheeeeeee | 15:29 |
| -opendevstatus- NOTICE: Gerrit is going to be restarted to pick up a small config update. You will notice a short outage of the service. | 15:33 | |
| iurygregory | wheeee for the bug | 16:23 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM - Debug logging for keepalive heartbeat https://review.opendev.org/c/openstack/ironic/+/892567 | 16:25 |
| * TheJulia thanks ovirt for loosing her entire morning | 16:47 | |
| iurygregory | I'm not going to ask what you were doing with ovirt, I just feel your pain | 16:48 |
| TheJulia | so long story short, the ovirt sdk has *no* request timeout on connections, so if the ovirt service engine VM starts going haywire, bad things can happen to the conductor | 16:51 |
| iurygregory | ouch | 16:55 |
| JayF | we should blow up ovirt, with some kind of massive stellar explosion | 16:56 |
| JayF | and then where ovirt was, there is now only a nova | 16:56 |
| JayF | ;) | 16:56 |
| TheJulia | kubevirt will come along as a replicator ship | 16:59 |
| JayF | then folks will migrate to it, they'll fix their own versions of every nova bug since 2015, and then at some point gorpyvirt or whatever is next down the line will come and repeat the process ;) | 17:02 |
| TheJulia | I believe that is when the Borg invades | 17:02 |
| JayF | ah, then we all become single-instance hypervisors | 17:02 |
| TheJulia | so worse, timeout is set to 0 explicitly | 17:04 |
| iurygregory | Borg from Start Trek? | 17:06 |
| TheJulia | Star Trek | 17:07 |
| JayF | Yeah, and the evil ones, not the [spoiler] collective from Picard | 17:07 |
| iurygregory | XD | 17:07 |
| TheJulia | .... hi nordix jenkins, can we give you caffeine ?! | 17:10 |
| iurygregory | I wish we could | 17:11 |
| iurygregory | maybe we can give zuul cookies to nordix jenkins? :D | 17:11 |
| JayF | don't mix pantheons of mythologies | 17:12 |
| JayF | they don't like admitting the others exist | 17:12 |
| TheJulia | so it looks like restricting the retry interval way down actually breaks things faster | 17:16 |
| TheJulia | which... I guess... okay | 17:16 |
| TheJulia | but still a bit fatal | 17:16 |
| TheJulia | doesn't look like tenacity is actually retrying | 17:20 |
| TheJulia | ... much | 17:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Revamp Ironic dev-quickstart documentation https://review.opendev.org/c/openstack/ironic/+/891912 | 18:13 |
| JayF | ^ is ready for your review love, please friends | 18:13 |
| JayF | I'm pretty explicit in commit message where it is; I just want to land the good bits and the reorg without having to test every possible permutation of dev env config we have documented :| | 18:14 |
| JayF | TheJulia: https://github.com/ovn-org/ovn/blob/main/TODO.rst | 18:52 |
| JayF | MTU handling is explicitly a to-do. Search the documentation for fragmentation | 18:52 |
| TheJulia | omfg | 18:53 |
| JayF | This is just a proxy to you from another channel, I can't take credit for finding it | 18:53 |
| JayF | Please just put down some padding before you flip your table 😂 | 18:54 |
| TheJulia | I think i might have discovered our held db interaction | 18:54 |
| * TheJulia runs tests | 19:20 | |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Revamp Ironic dev-quickstart documentation https://review.opendev.org/c/openstack/ironic/+/891912 | 19:32 |
| JayF | Heads up: ironic bugfix/18.0 branch was recrated in error, it's been long retired | 19:46 |
| JayF | I'm going to be using the big hammer to clean that up | 19:46 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DB: Un-nest some returns from dbapi https://review.opendev.org/c/openstack/ironic/+/892621 | 19:48 |
| TheJulia | boom | 19:49 |
| JayF | oh that lazy loading list comprehension return | 19:50 |
| JayF | ouch | 19:50 |
| JayF | amazing how the bugs scream so loudly when someone points them out :P | 19:50 |
| TheJulia | And it is a stupid easy thing to do because on a run of the mill object that has not lazy loaded, it is fine | 19:52 |
| JayF | yeah I have a comment on that change right now | 19:54 |
| JayF | validating my understanding of when things get let go of lol | 19:54 |
| TheJulia | replied to your comment already | 19:55 |
| TheJulia | all() trigers the execution and the return of a tuple in sqlalchemy 1.4/2.0 | 19:55 |
| TheJulia | well, a list of tuples | 19:56 |
| JayF | that's what I thought, just asking to verify assumptions :D | 19:57 |
| JayF | Another one: ironic-python-agent bugfix/8.0 branch was recrated in error, it's been long retired, am deleting | 20:00 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fail on node lookup if it is locked https://review.opendev.org/c/openstack/ironic/+/892414 | 20:18 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient stable/yoga: Fix zuul config error https://review.opendev.org/c/openstack/python-ironicclient/+/892622 | 20:28 |
| JayF | TheJulia: dtantsur: Please review the mail I just sent to the ML (https://lists.openstack.org/pipermail/openstack-discuss/2023-August/034854.html) or ensure someone on your respective teams who cares about what branches exist do :) | 20:29 |
| TheJulia | I think we could likely close out every one in the list, but I'll sync up with folks | 20:30 |
| TheJulia | I'm going to step away for a little while to take a break... just spoons | 20:31 |
| JayF | That's fine, there's no rush on that just wanted to make sure I had let you all know via all possible mechanisms | 20:32 |
| JayF | OK; contributor docs updated pending review; zuul config errors touched on and now gotta wait; | 20:32 |
| JayF | oh no, I'm finally at 'play with nova' on my list :) | 20:32 |
| * JayF gotta test https://review.opendev.org/c/openstack/nova/+/888290 & https://review.opendev.org/c/openstack/nova/+/887151 | 20:33 | |
| JayF | frickler: ty for the review; will fix many of the items you pointed out; but some I'm considering out of scope (my primary goal: fix devstack configs, ensure they're linked from a central place, and findable). I will continue iterating to improve these, but mainly focusing on the other sections now :D | 21:48 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Revamp Ironic dev-quickstart documentation https://review.opendev.org/c/openstack/ironic/+/891912 | 21:56 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Revamp Ironic dev-quickstart documentation https://review.opendev.org/c/openstack/ironic/+/891912 | 22:12 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Revamp Ironic dev-quickstart documentation https://review.opendev.org/c/openstack/ironic/+/891912 | 22:14 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient stable/yoga: [stable-only] Fix zuul config; comment failing CI https://review.opendev.org/c/openstack/python-ironicclient/+/892622 | 23:03 |
| iurygregory | JayF, quick question, how can bugfix branches be re-created by mistake? .-. | 23:30 |
| JayF | iurygregory: it was a bug in the release team automation that was fixed. I think I at some point lost the action item to re-delete them before I executed it. | 23:31 |
| JayF | either way, it's fixed now, and it was easy to fix because we already fixed the ACLs the first time retirements were done | 23:31 |
| JayF | You know what, this is an intern project right here | 23:32 |
| JayF | fixing release automation for bugfix branches, to properly support cutting releases and retiring them | 23:32 |
| iurygregory | I see, thanks! | 23:35 |
| iurygregory | makes sense to me (re intern project) | 23:36 |
| *** dtantsur_ is now known as dtantsur | 23:58 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!