| opendevreview | Verification of a change to openstack/ironic master failed: Add node auto-discovery support for in-band inspection https://review.opendev.org/c/openstack/ironic/+/906376 | 00:15 |
|---|---|---|
| *** dtantsur_ is now known as dtantsur | 01:50 | |
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/906935 | 02:21 |
| Kangie | JayF: Well, you got me. When I manage to get a console to new VM I'll run the bifrost playbooks and have a fiddle with some virtual nodes. | 06:12 |
| rpittau | good morning ironic! Happy Friday! o/ | 08:01 |
| dtantsur | C901 'lookup_node' is too complex (21) | 08:09 |
| dtantsur | oh ffs | 08:09 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add node auto-discovery support for in-band inspection https://review.opendev.org/c/openstack/ironic/+/906376 | 08:25 |
| dtantsur | rpittau, iurygregory, could you re-review please ^^^? The only difference is splitting lookup_node because of the complexity requirements | 08:25 |
| iurygregory | good morning Ironic | 11:19 |
| iurygregory | dtantsur, sure! | 11:19 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector bugfix/12.0: Update .gitreview for bugfix/12.0 https://review.opendev.org/c/openstack/ironic-inspector/+/907541 | 13:14 |
| JayF | Kangie: nice, good luck 🤞 | 13:20 |
| opendevreview | Merged openstack/ironic bugfix/24.0: Update .gitreview for bugfix/24.0 https://review.opendev.org/c/openstack/ironic/+/907410 | 13:59 |
| opendevreview | Merged openstack/ironic-inspector bugfix/12.0: Update .gitreview for bugfix/12.0 https://review.opendev.org/c/openstack/ironic-inspector/+/907541 | 13:59 |
| opendevreview | Alexander Dibbo proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 14:07 |
| opendevreview | Takashi Kajinami proposed openstack/networking-baremetal master: Bump hacking to 6.1.0 https://review.opendev.org/c/openstack/networking-baremetal/+/907552 | 14:10 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Ignore generated config/policy file https://review.opendev.org/c/openstack/ironic/+/907556 | 14:20 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Use .coveragerc file https://review.opendev.org/c/openstack/ironic/+/907557 | 14:20 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: Ignore generated config/policy file https://review.opendev.org/c/openstack/ironic-inspector/+/907559 | 14:22 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: tox: Drop envdir https://review.opendev.org/c/openstack/ironic/+/907560 | 14:27 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: tox: Drop envdir https://review.opendev.org/c/openstack/ironic/+/907560 | 14:29 |
| opendevreview | Merged openstack/ironic master: Add node auto-discovery support for in-band inspection https://review.opendev.org/c/openstack/ironic/+/906376 | 15:04 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/zed: Fix system scoped manageable node network failure https://review.opendev.org/c/openstack/ironic/+/907564 | 15:10 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/yoga: Fix system scoped manageable node network failure https://review.opendev.org/c/openstack/ironic/+/907565 | 15:10 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: [stable-only] Fix tempest checkout for older versions https://review.opendev.org/c/openstack/ironic/+/907610 | 15:14 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/xena: Fix system scoped manageable node network failure https://review.opendev.org/c/openstack/ironic/+/907566 | 15:15 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Fix system scoped manageable node network failure https://review.opendev.org/c/openstack/ironic/+/907567 | 15:16 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: [stable-only] Fix tempest checkout for older versions https://review.opendev.org/c/openstack/ironic/+/907610 | 15:23 |
| iurygregory | TheJulia, you need to change a little bit the syntax ^ | 15:26 |
| iurygregory | - name: openstack/ironic-tempest-plugin | 15:27 |
| TheJulia | tried that first :( | 15:27 |
| TheJulia | err, maybe not | 15:27 |
| iurygregory | override-checkout: <> | 15:27 |
| iurygregory | https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/zuul.d/xena_jobs.yaml#L8 | 15:28 |
| TheJulia | oh, I think I see what I did | 15:28 |
| TheJulia | yup, okay | 15:28 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: [stable-only] Fix tempest checkout for older versions https://review.opendev.org/c/openstack/ironic/+/907610 | 15:28 |
| TheJulia | Well, it seems to be thinking so that is a good thing! | 15:41 |
| iurygregory | \o/ | 15:41 |
| TheJulia | iurygregory: since your around, if you have some time, eyes on https://review.opendev.org/c/openstack/ironic/+/900965 would be appreciated | 15:48 |
| iurygregory | TheJulia, ack will look at it | 15:49 |
| iurygregory | going to grab lunch be back in about 30min | 15:49 |
| TheJulia | ok | 15:55 |
| rpittau | bye everyone, have a great weekend! o/ | 15:58 |
| TheJulia | o/ | 15:59 |
| TheJulia | so here is an idea for the weekly meeting, why don't we revisit a few hold bugs each meeting? | 16:36 |
| TheJulia | Would make sense, almost like RFE review, for things like https://bugs.launchpad.net/ironic/+bug/1461140 | 16:37 |
| samcat116 | Has anyone hit this bug when using Ironic with OVN and assigning FIPs to baremetal nodeS? | 16:39 |
| samcat116 | https://bugs.launchpad.net/neutron/+bug/1995078 | 16:39 |
| TheJulia | oh jeeze that is a narly one | 16:41 |
| TheJulia | If there were not already fixes posted, I'd say add to https://docs.openstack.org/ironic/latest/admin/ovn-networking.html | 16:42 |
| samcat116 | My coworker just spend the better part of 3 days identifying that | 16:42 |
| samcat116 | It seems like a fix just got in at the beginning of the month | 16:43 |
| TheJulia | :( | 16:44 |
| TheJulia | Another fun one: https://bugs.launchpad.net/ironic/+bug/1500891 <-- what do we do with it | 16:46 |
| iurygregory | TheJulia, +2 I think the comments can be addressed in a follow-up | 17:08 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 17:10 |
| TheJulia | iurygregory: quick question on one of those items, does "tftp server service" frame it better, or not | 17:41 |
| iurygregory | in my mind it doesn't | 17:44 |
| iurygregory | I would prefer to keep tftp server only | 17:44 |
| TheJulia | adamcarthur5: just fyi, it looks like you have two accounts in gerrit. Not sure if that is intentional | 17:47 |
| adamcarthur5 | TheJulia it is definitely not... I imagine I accidentally uses my GitHub username a few times | 17:48 |
| adamcarthur5 | I'll try and figure out how to resolve that | 17:48 |
| TheJulia | ack, you might not be able to in the grand scheme of things other than just consistency moving forward | 17:49 |
| TheJulia | (and yes, this is an easy situation to sort of fall into with multiple distinct email addresses) | 17:49 |
| clarkb | and not unique to us. There is a recent ish thread on the pypa mailing list about someone losing access to 2fa accounts necessary to log into pypi because the credentials were stored under an identity associated with their employer | 17:50 |
| clarkb | In general I think a good practice when interacting with open source is to either A) treat your employment identities as distinct and don't be surprised if you occasionally have to create a new identity or B) use a consistent identity tied to yourself and not your employer | 17:51 |
| clarkb | then associated employment to that identity as necessary to make employers happy | 17:51 |
| opendevreview | Merged openstack/bifrost master: Switch the dibipa jobs to Redfish https://review.opendev.org/c/openstack/bifrost/+/905114 | 17:55 |
| TheJulia | fun! One of the best bits of advice I ever got was basically, don't use work email associated accounts for OSS stuffs, at least directly. There are downsides to that though. | 18:02 |
| opendevreview | Merged openstack/ironic-python-agent stable/2023.2: Also retry inspection on HTTP CONFLICT https://review.opendev.org/c/openstack/ironic-python-agent/+/907041 | 18:18 |
| TheJulia | iurygregory: ack, I think one thing that is difficult here is which word is right. In my mind tftp server is right because that is also the name of the field in the response population when using tftp | 18:25 |
| TheJulia | *but* I'm "old". Somewhere there is a cane if I need to shake it at the sky | 18:25 |
| JayF | TheJulia: I worked at a place once that insisted it was OK they forced you to use your purple email address to do OSS, because *github* was smart enough to associate it with your account. I then mentioned to the OSPO leadership that connection gets lost unless I leave an email address I no longer control attached to my account after I quit. All the sudden, "it's OK because X" turned into "do it this way because we say so" | 18:30 |
| JayF | I ended up ignoring that directive for the most part, although it looks like I did around a dozen commits with my official email (I audited this recently to ensure my affiliations/emails were accurate) | 18:30 |
| TheJulia | heh, yeah | 18:31 |
| JayF | TheJulia: INVALID; it's a hardware bug if we are asking for hard reboot and getting soft | 18:32 |
| JayF | TheJulia: re: 1500891 | 18:32 |
| JayF | especially for one from 5 years ago :shrug | 18:33 |
| TheJulia | yeah | 18:33 |
| TheJulia | good point | 18:33 |
| JayF | not every piece of hardware bad behavior is our problem | 18:33 |
| JayF | especially now that there's a happy redfish path | 18:33 |
| TheJulia | heh | 18:37 |
| TheJulia | So, I think we technically already took care of https://bugs.launchpad.net/ironic/+bug/1526451 | 18:39 |
| TheJulia | just... our own organic way | 18:39 |
| TheJulia | but it depends on your process launch model for the api as well | 18:39 |
| iurygregory | TheJulia, yeah, tftp server sounds better | 18:40 |
| JayF | ++ | 18:44 |
| JayF | If we want to actually get in a chat for a quick sync bug jam I have time | 18:44 |
| TheJulia | I actually really like that idea | 18:44 |
| TheJulia | https://meet.google.com/ogc-mdbu-cen | 18:45 |
| JayF | adamcarthur5: if you're working now, ^ is a good oppo :) | 18:48 |
| iurygregory | I would join, but now my mind is focused on writing the controller for BMO to handle HostFirmwareComponents Updates | 18:50 |
| adamcarthur5 | JayF if I can just listen on a phone? | 19:08 |
| JayF | up to you, you manage when you work, just saying it's a good chance to see some context of the project | 19:08 |
| JayF | I don't want you to be distracted from something that needs your full attention (I've had this question asked and the person came up while highway driving LOL) | 19:08 |
| TheJulia | ugh, we need to revisit https://review.opendev.org/c/openstack/ironic/+/336055 | 19:56 |
| JayF | https://bugs.launchpad.net/ironic-python-agent/+bug/2050013 is a fun one | 20:04 |
| JayF | if anyone wants to be nerd sniped by partitioning code | 20:04 |
| TheJulia | I'll note ironic is now down to 199 open bugs | 20:20 |
| JayF | IPA has had 190 bugs filed all-time. | 20:20 |
| NobodyCam | Good afternoon Ironic folks! | 21:03 |
| NobodyCam | TGIF ... ;p | 21:03 |
| NobodyCam | off the wall question, any thing preventing use of latest sushy with a older version like ussuri? | 21:04 |
| JayF | I can't point to a single specific thing .... but I would not expect it to work | 21:05 |
| JayF | if for nothing else, incompatible requirements will be a thing most likely | 21:05 |
| JayF | but I have a hunch it would be a *lot* of breakage | 21:05 |
| TheJulia | ussuri upper-constraint was sushy 3.2.3, were at 4.8.0 now | 21:06 |
| TheJulia | 3.x was the end of pre-enums | 21:07 |
| TheJulia | 4.x uses enums, so yeah, expect breakage | 21:07 |
| NobodyCam | +++ thank you | 21:11 |
| TheJulia | seriously though, consider upgrading | 21:11 |
| TheJulia | ussuri is so far back in time it gets nothing() fix/patch wise | 21:11 |
| NobodyCam | we are rolling out Antelope bit that will take a year ;) | 21:34 |
| NobodyCam | s/bit/but/ | 21:35 |
| *** priteau_ is now known as priteau | 21:44 | |
| opendevreview | Merged openstack/ironic stable/2023.2: Fix service role support https://review.opendev.org/c/openstack/ironic/+/907268 | 21:45 |
| JayF | I have a +2 on the 2023.1 version of ^ if someone wants to land it -> https://review.opendev.org/c/openstack/ironic/+/907269 | 21:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!