| opendevreview | Merged openstack/ironic master: Enable HTTP network boot for Redfish hardware https://review.opendev.org/c/openstack/ironic/+/922160 | 01:27 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] add Targets to firmware.update on multi system BMCs https://review.opendev.org/c/openstack/ironic/+/922438 | 06:44 |
| rpittau | good morning ironic! o/ | 07:08 |
| rpittau | FYI voting for E release is open, deadline is Friday, June 28 #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/33F2I7NRUHGJ5L3A5D66COO26NRJSQJN/ | 07:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Follow up to change on testing CI on ARM https://review.opendev.org/c/openstack/ironic/+/922627 | 08:24 |
| pszczerbik | TheJulia: Hi. I addressed review comments for https://review.opendev.org/c/openstack/ironic/+/922024 Could you take a second look at it? | 08:33 |
| opendevreview | Verification of a change to openstack/ironic master failed: Follow up to change on testing CI on ARM https://review.opendev.org/c/openstack/ironic/+/922627 | 09:03 |
| *** vicent is now known as vicentfg | 10:26 | |
| *** vicentfg is now known as vicent | 10:26 | |
| *** diablo_rojo_phone is now known as Guest10734 | 11:21 | |
| iurygregory | good morning Ironic | 11:25 |
| opendevreview | Merged openstack/ironic master: Follow up to change on testing CI on ARM https://review.opendev.org/c/openstack/ironic/+/922627 | 11:50 |
| rpittau | shajizad: I just ran pep8 test on your patch and it fails locally, so zuul is right :) | 12:26 |
| rpittau | shajizad: interesting though, it passes with python 3.10 but not with 3.11 or 3.12 so definitely a difference in the version as I was saying before | 12:37 |
| rpittau | mmm nvm, when running tox with the latest implementation it fixes the file, so that's why running it on py3.10 after py3.11 it passes | 12:39 |
| shajizad | I probably messed up the commit then | 12:40 |
| shajizad | I'll try again later and submit for review, thank you! | 12:41 |
| iurygregory | yeah, they have pre-commit changes | 12:49 |
| iurygregory | so it corrects | 12:50 |
| dtantsur | ironic_lib/__init__.py:19: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('ironic_lib')`. | 13:04 |
| dtantsur | Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages | 13:04 |
| dtantsur | is it on anyone's radar already? should I file a bug? | 13:04 |
| iurygregory | dtantsur, I never saw this before... | 13:05 |
| iurygregory | lowhangging fruit maybe? | 13:05 |
| dtantsur | yeah | 13:06 |
| dtantsur | I seriously wonder why we even do https://opendev.org/openstack/ironic-lib/src/branch/master/ironic_lib/__init__.py#L16 | 13:06 |
| dtantsur | It does not make any sense to me: ironic-lib is a normal package, not a namespace. | 13:06 |
| iurygregory | ++ | 13:06 |
| iurygregory | I think it was automatically generated =) | 13:06 |
| dtantsur | Possibly | 13:07 |
| iurygregory | I probably have someone who can take this bug, will check with him | 13:08 |
| dtantsur | https://bugs.launchpad.net/ironic-lib/+bug/2070370 | 13:08 |
| iurygregory | I've sent to my friend =) | 13:13 |
| rpittau | should we start using "good first issue" or similar instead of "low-hanging-fruit" ? :) | 13:15 |
| dtantsur | is there a difference? | 13:17 |
| rpittau | I was just thinking if that sounds more explicit as it was mentioned in the docs review | 13:22 |
| rpittau | more clear to a possible new contributor | 13:22 |
| TheJulia | good morning | 13:32 |
| cardoe | So just looking at the Mercury spec and wondering if the idea is to basically just have vendors or deployers have a better networking-generic-switch plugin? | 13:32 |
| TheJulia | Depends on the point of view there, but basically yeah | 13:33 |
| TheJulia | a simplified interface which can be run by another team | 13:33 |
| TheJulia | the biggest problem we tend to see is the "network" team is entirely disjointed from the "cloud infra team" | 13:33 |
| cardoe | That last statement hits too close to home. | 13:34 |
| TheJulia | Sorry, but it is a reality with a ton of shops out there | 13:34 |
| TheJulia | mix in trust issues, and we quickly reach "no, you cannot change my switch config!" | 13:35 |
| cardoe | Oh I was just making a joke. But I figured it was true for others. | 13:36 |
| cardoe | Oh yeah battling that "you cannot touch my switch" all the time. | 13:37 |
| cardoe | I've got jamesdenton | 13:37 |
| cardoe | complaining about netmiko | 13:38 |
| * TheJulia waves at jamesdenton | 13:38 | |
| TheJulia | yes, rightfully so! | 13:38 |
| cardoe | He can't find his IRC client. | 13:38 |
| TheJulia | it *is* awful | 13:38 |
| TheJulia | tell him to launch a new one! | 13:38 |
| TheJulia | or to get irccloud :) | 13:38 |
| cardoe | So essentially we're trying to marry up Nautobot for our DCIM/IPAM and having Ironic be the API to manage that hardware. Just trying to figure out how to best contribute back in a generic fashion that others can use rather than tied to our specific tool selection. | 13:42 |
| cardoe | So my thought has been to engage with the mercury spec. | 13:44 |
| TheJulia | I'm unlikely to make progress on mercury in any finished form in the next few months, and really right now it is just an idea | 13:56 |
| TheJulia | The idea there is that we can have plugins which we then trigger actions, it could be also an IPAM and could be extended through a non-neutron dhcp module | 13:57 |
| cardoe | Of course. Its a slow burn effort to do the right thing. | 13:57 |
| TheJulia | it. always. is. | 13:57 |
| TheJulia | being the chair also doesn't make my time any more free | 13:58 |
| cardoe | I've been difficult (if ya ask the others) and pushing us around a k8s based deployment. The plugins to trigger actions is exactly the model I've been trying to use. | 13:58 |
| TheJulia | with mercury, we wanted something we didn't care how it was run | 13:59 |
| TheJulia | We wanted a trust delineator if that makes sense | 13:59 |
| cardoe | of course. I'm just saying for our efforts. | 13:59 |
| TheJulia | something simple enough trust could be built | 13:59 |
| TheJulia | ack | 13:59 |
| cardoe | So a bit of a Source of Truth which gets updated and triggers an event and later something could monitor to see if said thing misaligned with the Source of Truth and could trigger a clean up / fix up event. | 14:00 |
| TheJulia | yeah, basically | 14:01 |
| TheJulia | or at least, be able to provide a "is it in the state I expect" answer back to the caller | 14:01 |
| cardoe | Exactly | 14:01 |
| TheJulia | at which point we can log, move to an error state, or whatever | 14:01 |
| TheJulia | Fixing is nice, but... sometimes we can't just try and force the will of the software | 14:02 |
| TheJulia | (because it changing might have been for a *really* valid reason | 14:02 |
| TheJulia | ) | 14:02 |
| TheJulia | ((and might have been missed, or noted in an email that nobody read because the subject was talking about murano...)) | 14:03 |
| cardoe | yeah that's a good point | 14:05 |
| TheJulia | Time to go wlak the corgi overlord | 14:07 |
| cardoe | Good call. Keeps the doc monster away too. | 14:10 |
| TheJulia | heh | 15:20 |
| dtantsur | JayF: honestly, I'm not sure which part of https://docs.openstack.org/ironic/latest/contributor/devstack-guide.html makes people assume it's required for unit tests | 15:26 |
| dtantsur | nor does https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#dev-quickstart really do much in terms of clarifying it | 15:26 |
| JayF | I didn't say our docs provide that assumption | 15:26 |
| JayF | only that it preexists | 15:26 |
| dtantsur | if anything, it's having "Integrating Testing Enviroments" before "Unit testing" that is causing this assumption | 15:27 |
| JayF | IME it's more "I'm a *dev* on *openstack* of course step 1 is a devstack to run tests in" | 15:27 |
| JayF | but I can't have this chat now, podcast taping in 3 minutes; but my point is more: peoples' indepedent research on openstack development brings them to this conclusion, not our docs | 15:27 |
| * JayF will read backlog in ~90 minutes | 15:27 | |
| dtantsur | okay, so I remain unconvinced that my patch makes it worse :-/ | 15:28 |
| rpittau | bye everyone, have a nice evening o/ | 15:35 |
| dtantsur | Meanwhile, I'm open to practical suggestions on how to keep or Redfish interoperability profile up-to-date | 15:42 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Update the redfish interoperability profile https://review.opendev.org/c/openstack/ironic/+/920574 | 16:02 |
| dtantsur | Now I only need to figure out what we use for RAID. I think. I hope. | 16:02 |
| opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 20:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!