| TheJulia | Welcome! For what it is worth, most contributors are in the EU and Americas, so it will be a bit quiet until people start their days. | 00:12 |
|---|---|---|
| tony__k | Thanks, Julia! I’m new to IRC and still getting used to things. Could you please let me know who the mentors are here in the Ironic channel? | 00:25 |
| opendevreview | Steve Baker proposed openstack/ironic master: Fixes for console containers functional testing https://review.opendev.org/c/openstack/ironic/+/963323 | 00:41 |
| opendevreview | Steve Baker proposed openstack/ironic master: Improve logging when console unit doesn't start https://review.opendev.org/c/openstack/ironic/+/963324 | 00:41 |
| najuna | Hi, I’m Najuna. I’m an Outreachy applicant and I’ll be contributing to the Ironic project. Looking forward to learning from everyone here! | 01:14 |
| TheJulia | Greetings najuna! It is night time for most contributors right now. Expect people to start waking up in a few hours. | 02:52 |
| najuna | TheJulia: Greetings and thanks for the guidance. Hope I will get to understand and adjust to the perfect hours soon enough | 02:58 |
| TheJulia | Most contributors are in the EU and Americas, so that is this channel's busiest times. As for myself, I'm in North America and it is time to get some sleep. | 02:59 |
| TheJulia | Have a wonderful day! | 02:59 |
| *** ab is now known as Guest28596 | 03:09 | |
| rpittau | good morning ironic! o/ | 06:15 |
| *** mdfr2 is now known as mdfr | 06:51 | |
| *** dnvr_ is now known as denver_brka | 07:01 | |
| zigo | tkajinam: Hi there! I wonder if you could help with https://bugs.debian.org/1117426 . I tried, but it's kind of annoying. The issue behind this bug is that I upgraded pysnmp to version 7+. | 07:11 |
| zigo | I tried, but it seems non-trivial to fix. | 07:11 |
| *** hroy_ is now known as hroy | 07:46 | |
| opendevreview | Mathieu Parent proposed openstack/ironic-python-agent-builder master: Wait up to 30 seconds for config drive https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962673 | 08:27 |
| *** dnvr_ is now known as denver_ba | 09:14 | |
| *** joy is now known as JoyAkinyi | 10:33 | |
| Elvis | Hi, glad to be here. Im Elvis from Outreachy | 10:38 |
| denver_ba | Welcome Elvis | 10:42 |
| pcnerd | hi, my name is Habeeb from Outreachy, I am glad to be here | 10:43 |
| rpittau | hey Elvis pcnerd welcome :) | 10:47 |
| Elvis | Thank you guys, i appreciate :) | 10:48 |
| hroy | hey Elvis pcnerd welcome :) | 10:48 |
| pcnerd | thank you guys for your warm welcoming :} | 10:50 |
| queensly[m] | Hello Elvis, welcome:) | 10:50 |
| pcnerd | I am kinda new to this interface, can I get help navigating myself around here? 🥹 | 10:51 |
| JoyAkinyi | Hi everyone, I’m Joy Akinyi. I just joined the channel and I’m happy to be here...from outreachy | 11:06 |
| Saurabh | Hello, my name is Saurabh Pareek. I am an Outreachy applicant interested in the "Extend Ironic node deployment history api" project. I'm currently setting up my environment and look forward to contributing. | 11:12 |
| queensly[m] | Hey Saurabh, welcome :) | 11:15 |
| queensly[m] | That's great. You can reach out when you need any assistance with your setup. | 11:15 |
| ayo_ | Hi everyone | 11:15 |
| ayo_ | I just joined and I’d like to confirm if this is the right group for the openstack community for outreachy | 11:17 |
| ayo_ | Hello @queensly[m]: | 11:18 |
| Saurabh | Yes it is the right group for Outreachy | 11:18 |
| ayo_ | Hi @Saurabh, thanks for confirming | 11:19 |
| queensly[m] | Hi ayo_ welcome :) | 11:24 |
| ayo_ | Good day everyone, my name is Ayo from outreachy and I’ll be really happy to contributing to this project | 11:26 |
| ayo_ | I’m a bit nervous but I’m overall looking forward to what I can learn and contribute to this community | 11:28 |
| ayo_ | According to the project breakdown there are additional info we should know of, can you please enlighten me @queensly[m]: | 11:30 |
| hroy | welcome everyone :) | 12:07 |
| *** iurygregory_ is now known as iurygregory | 12:08 | |
| iurygregory | good morning ironic | 12:08 |
| dnvr_ | good morning | 12:09 |
| hroy | We are preparing the next set of instructions for outreachy project applicants, will share them with you as soon as possible. | 12:09 |
| pcnerd | thanks hroy | 12:10 |
| egicharu | Thanks Roy :) | 12:11 |
| ayo_ | Thank you | 12:12 |
| *** dnvr_ is now known as denver_ba | 12:13 | |
| opendevreview | Stephen Finucane proposed openstack/ironic master: api: Catch correct exception on schema validation error https://review.opendev.org/c/openstack/ironic/+/963370 | 12:24 |
| opendevreview | cid proposed openstack/ironic-python-agent-builder master: Drop support for TinyIPA https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962490 | 12:26 |
| opendevreview | Stephen Finucane proposed openstack/ironic master: api: Catch correct exception on schema validation error https://review.opendev.org/c/openstack/ironic/+/963370 | 12:36 |
| opendevreview | Stephen Finucane proposed openstack/ironic master: api: Allow bios values to be None https://review.opendev.org/c/openstack/ironic/+/963376 | 12:43 |
| alegacy | In yesterday's adhoc meeting regarding the ngs <--> ironic import issue we discussed a couple of different options. | 13:01 |
| alegacy | 1) move the new ironic-specific ngs entrypoint out of ngs and into ironic as an in-repo driver implementation | 13:01 |
| alegacy | 2) clone the new Ironic switch driver base class into the ngs project and eliminate the need for an import | 13:01 |
| alegacy | Prior to the meeting the option of resurrecting ironic-lib was also tossed around but that seemed to fizzle. | 13:01 |
| alegacy | We deferred the decision until it can be considered for a day or so. | 13:01 |
| alegacy | IMHO, Option 1 is probably the cleanest architecturally but has the drawback that ngs will become an optional dependency of Ironic | 13:01 |
| alegacy | IMHO, Option 2 is not as architecturally clean and would require some awkward driver-specific exception handling in ironic-networking, but may be the simplest option. | 13:01 |
| alegacy | dtantsur: TheJulia: ^^ | 13:01 |
| dtantsur | I'm still slightly leaning towards 1 | 13:02 |
| TheJulia | good morning | 13:10 |
| alegacy | dtantsur: that's my current preference as well. | 13:12 |
| TheJulia | Option 1 is the cleanest I think. It allows us to move forward and refactor later if we want without too much in the way of headaches | 13:14 |
| alegacy | ok, then unless there are other strong opinions about this in the next little bit I will make the necessary modifications. | 13:18 |
| alegacy | thank you to both of you for helping me figure this out. | 13:18 |
| alegacy | I'll abandon the global-requirements patch. I assume no such change is required for having ngs as a driver requirement in Ironic | 13:21 |
| *** JoyAkinyi is now known as Guest28623 | 13:22 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM/Test: debug neutron failures https://review.opendev.org/c/openstack/ironic/+/963342 | 13:28 |
| cloobtech | Hello Everyone, I'm Olamide from Nigeria, an Outreachy applicant interested in the OpenStack Ironic Project. I'm super excited to learn and get started... | 13:30 |
| TheJulia | Greetings cloobtech! | 13:30 |
| TheJulia | So, who are our Outreachy points of contact for this next session? | 13:31 |
| cloobtech | This is actually my first time, still finding my footing around the docs and all... TheJulia | 13:32 |
| TheJulia | cloobtech: no worries, the question was likely for iurygregory dtantsur | 13:33 |
| dtantsur | the points of contacts are rpittau and hroy | 13:34 |
| TheJulia | Neutron folks have proposed a WIP revert for the suspected change which has resulted in unhappy CI | 13:39 |
| TheJulia | so, we'll see if it makes things better in ~1hr | 13:39 |
| *** ab is now known as Guest28625 | 13:41 | |
| cardoe | I've created this in nova's bug tracker but its related to ironic... not sure if I should cross-tag it somehow... https://bugs.launchpad.net/nova/+bug/2127087 | 13:51 |
| JayF | zigo: you need to update the hlapi version as well. Iirc | 14:04 |
| JayF | zigo: pysnmp basically has a partner library that it uses and they have to be upgraded in lockstep. | 14:04 |
| tony_kamande | Have the further instructions been communicated or where can i access the tasks to be done ? any guidance | 14:06 |
| zigo | JayF: What python module is this? | 14:07 |
| opendevreview | Aykut Bulgu proposed openstack/sushy master: Check required credentials in a detailed way https://review.opendev.org/c/openstack/sushy/+/963282 | 14:10 |
| dtantsur | Outreachy folks: it may be easier to drop an email to your mentors, you should have their contacts in the project details. Your messages are easy to lose in IRC. | 14:12 |
| cardoe | Didn't we add a CLI for the image format validator? | 14:16 |
| cardoe | Having an issue where a GPT image is being SafetyCheck rejected as MBR | 14:16 |
| cardoe | Wanted to just iterate the code. | 14:16 |
| JayF | cardoe: it's in Oslo utils. | 14:42 |
| cardoe | ah | 14:43 |
| TheJulia | cardoe: for ironic bugs, it is likely best to cross-tag the bug since we're likely going to need to propose a patch to reclassify the failure conditions in a way which breaks the retry logic around spawn | 14:43 |
| cardoe | well clearly I don't know how cause I moved it from nova to ironic. I agree we'll need to classify that but the code change will happen in nova. | 14:46 |
| cardoe | there fixed my whoopsie. | 14:47 |
| TheJulia | So, we should likely expect neutron to propose a fix or outright revert the ovn break tomorrow | 14:55 |
| TheJulia | looks like the very first patch in the chain was the culprit though | 14:56 |
| dtantsur | nice | 15:39 |
| JayF | zigo: I think it's pyasn1 but not 100% | 15:40 |
| zigo | Right, will look into it. | 15:40 |
| zigo | Thanks. | 15:40 |
| *** sfinucan is now known as stephenfin | 15:44 | |
| stephenfin | I assume you folks are aware of 'ironic-tempest-ovn-uefi-ipmi-pxe' permafailing currently? | 16:08 |
| * stephenfin got kicked from IRC earlier and is missing some logs | 16:09 | |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Check _by_arch values with redfish vmedia https://review.opendev.org/c/openstack/ironic/+/963335 | 16:10 |
| TheJulia | stephenfin: yes, we've identified the likely breaking neutron change, they will likely refert or have a fix tomorrow. | 16:22 |
| stephenfin | ack | 16:22 |
| TheJulia | https://bugs.launchpad.net/ironic/+bug/2127038 | 16:22 |
| JayF | can we get them to cross-test against that job? | 16:25 |
| JayF | even if just for ovn changes | 16:25 |
| dtantsur | cid: hey, are you planning to finish https://review.opendev.org/c/openstack/ironic/+/949521? If not, may I take it over? | 16:28 |
| dtantsur | It's blocking my summit demo :) | 16:28 |
| TheJulia | JayF: already did, jeeze | 16:37 |
| JayF | oh nice! | 16:37 |
| TheJulia | oh | 16:37 |
| JayF | \o/ | 16:37 |
| TheJulia | you mean run our job in their CI | 16:37 |
| JayF | yes | 16:37 |
| TheJulia | That likely won't be hard to do after this | 16:37 |
| TheJulia | I was going to broach it tomorrow with them | 16:38 |
| JayF | Awesome, thank you for owning this | 16:38 |
| JayF | Can Ironic do any kind of access control when used in basic auth mode? | 16:40 |
| dtantsur | IIRC no | 16:40 |
| JayF | Basically curious if there's any way to get the multitenancy standalone ironic goodness around node.owner without running a whole keystone | 16:40 |
| JayF | That's unfortunate; I wonder if there'd be a way to make it work. | 16:41 |
| dtantsur | I've been pondering it too. It should be doable, even if must less flexible. | 16:41 |
| cid | dtantsur, Yes, please go ahead. | 16:41 |
| dtantsur | cool, thx | 16:41 |
| cid | ++ | 16:42 |
| JayF | I'm going ahead with activities to retire metalsmith and inspector now. | 16:45 |
| JayF | putting all relevant changes in this topic: https://review.opendev.org/q/topic:%22ironic-2026.1-retirements%22 | 16:54 |
| opendevreview | Jay Faulkner proposed openstack/metalsmith master: Retire metalsmith https://review.opendev.org/c/openstack/metalsmith/+/963427 | 16:59 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix IPA external inspection callback url override https://review.opendev.org/c/openstack/ironic/+/949521 | 17:01 |
| dtantsur | I think we should be doing this ^^^ | 17:01 |
| dtantsur | (I'll test it tomorrow on my environment) | 17:05 |
| opendevreview | cid proposed openstack/ironic-python-agent-builder master: Drop support for TinyIPA https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962490 | 17:13 |
| dtantsur | I've even tested it today during dinner! :) https://review.opendev.org/c/openstack/ironic/+/949521 works, could someone please take a look soon? I'd really prefer to have it in by the time of my summit demo next week :D | 18:26 |
| dtantsur | (hopefully, OVN stuff will get fixed soon) | 18:26 |
| TheJulia | hopefully tomorrow | 18:26 |
| dtantsur | fingers crossed.. I cannot believe I lost this patch on my radar | 18:27 |
| * dtantsur gets back to dinner now | 18:28 | |
| Saurabh | welcome new commers! | 18:56 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: add context around bmc password changes https://review.opendev.org/c/openstack/ironic/+/963441 | 19:09 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: cover what happens if someone changes iscsi passwords https://review.opendev.org/c/openstack/ironic/+/963442 | 19:09 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector master: Retire ironic-inspector https://review.opendev.org/c/openstack/ironic-inspector/+/963460 | 20:59 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 21:00 |
| JayF | FWIW project-config changes have to land before any of these, I'll tag 'em with -prio when we need them landed | 21:00 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector-specs master: Retire ironic-inspector-specs https://review.opendev.org/c/openstack/ironic-inspector-specs/+/963461 | 21:04 |
| opendevreview | Jay Faulkner proposed openstack/python-ironic-inspector-client master: Retire python-ironic-inspector-client https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/963462 | 21:14 |
| JayF | OK I actually got the ordering wrong; I think we need to zero out the repos before any of the governance can land | 22:03 |
| JayF | but that's not possible until project-config merges | 22:03 |
| JayF | fun-gi just landed the project-config changes, we'll need to land our end as the next step (then governance) | 22:18 |
| opendevreview | Steve Baker proposed openstack/ironic-tempest-plugin master: WIP test graphical console novnc connection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/963152 | 22:37 |
| opendevreview | Steve Baker proposed openstack/ironic master: Fixes for console containers functional testing https://review.opendev.org/c/openstack/ironic/+/963323 | 23:54 |
| opendevreview | Steve Baker proposed openstack/ironic master: Improve logging when console unit doesn't start https://review.opendev.org/c/openstack/ironic/+/963324 | 23:54 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!