| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Allow config of simulated switch https://review.opendev.org/c/openstack/networking-generic-switch/+/942942 | 00:21 |
|---|---|---|
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: docs: add status for force10 os https://review.opendev.org/c/openstack/networking-generic-switch/+/943070 | 00:21 |
| dtantsur | Amarachi_O: please contact rpittau and masghar, the mentors of the project | 07:17 |
| dtantsur | JayF: do you by chance know how to increase the message backlog size in IRCCloud? After migration, I no longer have access to some messages. For example, I have no idea if cid and you answered me anything.. | 07:19 |
| frickler | the topic should actually mention that this channel is getting logged. if someone wants to formulate an update, I'm happy to actually execute the change | 07:45 |
| frickler | and yes, I understand that having it directly in the tool that you use would be nicer | 07:46 |
| jrosser | I think that in the case of IRCCloud it is a feature to notice if you are mentioned even when you don’t have the client open | 07:50 |
| jrosser | the permanent connection is only part of the paid service though | 07:54 |
| rpittau | good morning ironic! o/ | 08:02 |
| masghar | Good morning! | 08:53 |
| buhman | JayF: do you plan on making a series of OpenStack ebuilds for gentoo? | 08:56 |
| masghar | If anybody has any docs nits or bugs they would like fixed, please report them to Launchpad :) | 09:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-inspector/+/944843 | 09:38 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-inspector/+/944844 | 09:38 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-inspector/+/944845 | 09:38 |
| Nayohmee | Quick question: How do i see messages that were already sent in channel before i logged in? I already asked my question about the Ironic task for outreachy applicants, but i logged out and now i can not see previous messages anymore | 09:50 |
| opendevreview | Merged openstack/ironic-inspector stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-inspector/+/944843 | 09:54 |
| opendevreview | Merged openstack/ironic-inspector master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-inspector/+/944845 | 09:54 |
| frickler | Nayohmee: see https://meetings.opendev.org/irclogs/%23openstack-ironic/ | 09:59 |
| opendevreview | Merged openstack/ironic-inspector stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-inspector/+/944844 | 10:00 |
| rpittau | Nayohmee: for anything outreachy related you can contact masghar or myself directly or by email, especially during the first contribution phase :) | 10:00 |
| opendevreview | cid proposed openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 10:03 |
| opendevreview | cid proposed openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 10:07 |
| ps_adavize | Good morning everyone , I'm an outreachy contributor and I intend to learn about and contribute to ironic project. @masghar and @rpittau | 10:28 |
| opendevreview | Merged openstack/networking-generic-switch master: Allow config of simulated switch https://review.opendev.org/c/openstack/networking-generic-switch/+/942942 | 10:32 |
| Nayohmee | Alright thanks alot @frickler and @rpittau | 10:51 |
| masghar | Hello and welcome ps_adavize! If you have any questions, please feel free to ask. | 11:22 |
| opendevreview | Verification of a change to openstack/ironic master failed: Install and run sushy-tools within a virtualenv https://review.opendev.org/c/openstack/ironic/+/943580 | 11:25 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: CI: Dial back the non-voting jobs https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/942846 | 12:18 |
| ps_adavize | Thank you masghar, from the contribution guide we are asked to write a 1-2 page explaining our understanding about the project. where should this be submitted? | 12:23 |
| Richard-Akin_ | Hello, good day everyone, My name is Richard Akintola, one of the recent Outreachy applicants. I look forward to collaborating, honing my skills, and learning/relearning. I would appreciate guidance on how to set up and get started. Thank you for having me. | 12:41 |
| rpittau | Richard-Akin_: welcome! I suggest to read the project page to get info on the first contribution task | 12:55 |
| rpittau | ps_adavize: feel free to send your essay by email to masghar and myself, you can find our emails in the project page | 12:55 |
| dtantsur | rpittau: can we still fit https://review.opendev.org/c/openstack/ironic/+/939217? I've just +2'ed it. | 12:59 |
| rpittau | dtantsur: yep | 12:59 |
| cid | Welcome to the community, ps_adavize, Richard-Akin_, Nayohmee, Amarachi_O, ... o/ | 13:07 |
| * cid felt like I tagged an old member, welcome all the same :D | 13:08 | |
| rpittau | :) | 13:08 |
| cid | By the way, thanks everyone for the reviews on inspection rules. I'm always here for any follow-ups. | 13:11 |
| cid | I should start working on the tempest at some point. Also, masghar, documentation for inpection rules might be an interesting task for your interns: Re^^ to your earlier ask. | 13:12 |
| cardoe | creating some functional examples and not just the non-functional ones in the spec would be a great start | 13:13 |
| opendevreview | Merged openstack/networking-generic-switch master: docs: add status for force10 os https://review.opendev.org/c/openstack/networking-generic-switch/+/943070 | 13:15 |
| cid | cardoe, functional? . The examples in the spec are real rule examples | 13:17 |
| cardoe | okay. real world / complex vs strawman? | 13:19 |
| cardoe | The ones in the spec just change one or two fields and don't combine operations. | 13:20 |
| rpittau | cid: that's probably a bit too much for someone who's approaching ironic for the first time :) | 13:21 |
| cid | cardoe: Yeah. We should add more meaningful examples in the documentation. | 13:21 |
| cardoe | I've just had a hard time explaining to my team what we can do with it. | 13:22 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: inspection: define inventory shape with dataclasses https://review.opendev.org/c/openstack/ironic/+/944868 | 13:22 |
| cid | rpittau: True :D | 13:22 |
| cardoe | ^ is just an idea. Wanted some feedback on doing something like that and later enforcing it with mypy | 13:23 |
| cardoe | The shape of that data is very... fluid... but certain pieces of code relies on specific shapes. | 13:23 |
| TheJulia | good morning | 13:35 |
| JayF | buhman: very no I don't | 13:44 |
| JayF | dtantsur: if you're on a free account, it times you out after 3 hours or something like that | 13:44 |
| dtantsur | I'm on paid account | 13:44 |
| opendevreview | Merged openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 14:00 |
| JayF | dtantsur: then I haven't seen what you're talking about. Are you just closing the app and not quitting/closing? | 14:01 |
| dtantsur | I'm not even closing the app, just going into standby by closing the lid :) | 14:02 |
| JayF | I wonder if it's as simple as the app not loading in old context | 14:03 |
| * TheJulia needs more coffee | 14:03 | |
| dtantsur | That's not relevant to the backlog *size* though | 14:03 |
| JayF | I can scroll back days and days and days | 14:03 |
| dtantsur | It's a configuration option in ZNC, which I guess they don't expose? | 14:03 |
| JayF | oooh | 14:03 |
| JayF | yeah I had trouble with the ZNC backlog stuff | 14:04 |
| JayF | it does a bad job of knowing what's arleady been seen | 14:04 |
| JayF | just opening the mobile app once would make the ZNC think those were all old/read messages | 14:04 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: network simulator support for Cisco Nexus 9k https://review.opendev.org/c/openstack/ironic/+/944873 | 14:07 |
| TheJulia | One aspect worth also noting, irccloud, in general can be a little weird if your backend server is down which handles log storage. It still works through the mobile and web clients, but you get some weird behavior where logs might not be present. | 14:18 |
| masghar | (Bit late to the convo, but yes, inspection rules docs is a more advanced ask for someone just starting out with the project) | 14:40 |
| opendevreview | Merged openstack/ironic master: Install and run sushy-tools within a virtualenv https://review.opendev.org/c/openstack/ironic/+/943580 | 16:19 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/944894 | 17:12 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/944895 | 17:12 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/944896 | 17:12 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/944897 | 17:14 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/944898 | 17:14 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/944899 | 17:14 |
| *** dmellado0755393733 is now known as dmellado075539373 | 17:14 | |
| freemanboss[m] | Hi everyone. I'm Freeman an outreachy internship applicant. I'm glad to be here and looking forward to working with the team and making my contributions | 17:16 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/bifrost/+/944900 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/bifrost/+/944901 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/bifrost/+/944902 | 17:21 |
| freemanboss[m] | Hello mentors and everyone in the house, please I hope this is the group to join for the third project? | 17:21 |
| freemanboss[m] | "Set bare metal host's clock remotely using Reddish"? | 17:21 |
| freemanboss[m] | And I'll appreciate any resources to go through that will simplify the best way for me to get started | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/944903 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/944904 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/944905 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/944906 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/944907 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/944908 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-ui/+/944909 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-ui/+/944910 | 17:22 |
| dtantsur | freemanboss[m]: please contact rpittau and masghar here or per email | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-ui/+/944911 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/networking-baremetal/+/944912 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/networking-baremetal/+/944913 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/networking-baremetal/+/944914 | 17:22 |
| ManideepMaddileti[m] | Hello all, | 17:23 |
| ManideepMaddileti[m] | I am Manideep, an Outreachy Applicant. I have completed setting up the development environment. Could you please guide me on where to submit the document containing a short (1-2 page) summary of my understanding of Ironic and its main components and workflows? | 17:23 |
| ManideepMaddileti[m] | Thank you! masghar rpittau | 17:23 |
| freemanboss[m] | freemanboss[m]: > <@freemanboss:matrix.org> Hello mentors and everyone in the house, please I hope this is the group to join for the third project?... (full message at <https://matrix.org/oftc/media/v1/media/download/AV_EPcOyo8y1o8B9rU7yNUF4Agn9yLhD_OtAzbxt3MSDFzqPHddTFjNa0f-2i1bndJaSbPzXt4a9oURRb_O6wHBCeV8mGqJAAG1hdHJpeC5vcmcvallGVHZvR1lnbWNDYU1BS0NhdkdnY0xN>) | 17:24 |
| freemanboss[m] | dtantsur: Alright thank you | 17:25 |
| dtantsur | ManideepMaddileti[m]: same answer for you: emailing the mentors is probably better because your messages won't get lost in the IRC logs | 17:30 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: Missing 'interface' in Clean Step https://review.opendev.org/c/openstack/ironic-python-agent/+/942152 | 17:48 |
| iurygregory | ok this is interesting, i don't see the variable max_overflow / max_pool_size in https://opendev.org/openstack/ironic/src/branch/master/ironic/conf/database.py , but they are in our docs https://docs.openstack.org/ironic/latest/configuration/config.html#database <confused> | 17:53 |
| iurygregory | https://github.com/metal3-io/ironic-image/pull/644 | 17:53 |
| JayF | https://github.com/openstack/oslo.db/blob/master/oslo_db/options.py | 17:58 |
| JayF | iurygregory: ^ | 17:58 |
| jayanta | /msg Jayanta REGISTER <80165556> <pradhanjayanta91@gmail.com> | 17:59 |
| iurygregory | JayF, tks! totally forgot about checking oslo_db | 18:00 |
| dtantsur | iurygregory: good that you raised my intention. Could you please block attempts to introduce new variables just to expose them in ironic.conf? We already have syntax for that. | 18:00 |
| iurygregory | dtantsur, ack o/ | 18:01 |
| cardoe | JayF, TheJulia: do ya hate https://review.opendev.org/c/openstack/ironic/+/944868 ? | 18:02 |
| JayF | so cardoe, here's my thoughts | 18:03 |
| JayF | like, that seems cool | 18:03 |
| JayF | I'm kinda onboard | 18:04 |
| JayF | EXCEPT it seems like that should be a pattern of change and not just one thing | 18:04 |
| JayF | like, why is it limited to inspection inventory? Why not other things? Also why not jsonschema like we use in the API? | 18:04 |
| cardoe | So two different things from jsonschema | 18:05 |
| cardoe | So today the hooks and rules modify the inventory data by just walking into the dict and fiddling however it wishes. | 18:05 |
| jayanta | /msg Jayanta REGISTER 80165556 pradhanjayanta91@gmail.com | 18:06 |
| JayF | jayanta: that did come across the public channel so if that is a common password you should rotate it | 18:06 |
| cardoe | The point here was these aren't dicts but are objects that you'd modify the defined fields of and then when we saved the inventory I'd use "todict()" of dataclasses to convert it back for the DB | 18:06 |
| dtantsur | cardoe: side note: plugins modify plugin_data, not the inventory | 18:06 |
| dtantsur | (if they modify inventory, we should probably fix it) | 18:07 |
| dtantsur | At this point, we should probably start microversioning inventory... | 18:07 |
| * dtantsur leaves for a day before cardoe has a chance of throwing a wet cat at him | 18:08 | |
| JayF | we're getting real good at this microversioning thing | 18:08 |
| cardoe | I'm not opposed to that but then we need to define the layout and not have the docs say "read the IPA source" | 18:08 |
| JayF | we'd better make sure 1.100 is something cooler than "added network device type to inventory" ;) | 18:08 |
| cardoe | I've created the stub of a doc defining it in ironic by calling out the top-level keys. | 18:09 |
| cardoe | JayF: I'm not actually wanting that patch to be mergeable. I'm just floating the idea of having objects that we interact with instead of raw dicts. | 18:10 |
| cardoe | It's related to the Glance issue I fixed as well. | 18:11 |
| cardoe | Where we just convert objects to dicts and then do dict.get("some_field") and oh whoopsie someone did dict.get("somefield") in one place. | 18:11 |
| JayF | cardoe: I just wanna be sure we're looking at all the places that might have that pattern | 18:14 |
| JayF | cardoe: that's all I'm sayin' :) | 18:14 |
| cardoe | The reason I did the inspection inventory is because IPA actually does this by using objects and then having it's own serialization to a dict to transmit it over the wire. I pulled an adamcarthur5 and took those source files and fed them to Claude and told it to convert them to dataclasses. And that's the patch you see. | 18:14 |
| cardoe | I agree. I can throw it on the PTG agenda as well. | 18:14 |
| cardoe | My question is "how do we feel about doing this as a pattern going forward" | 18:15 |
| cardoe | https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/encoding.py fwiw that's part of it. | 18:16 |
| cardoe | That's basically a Python 2 compatible version of dataclasses. | 18:17 |
| jayanta | /msg NickServ REGISTER 801655 pradhanjayanta91@gmail.com | 18:17 |
| JayF | jayanta: again, you prefixed with a space and so it came to the public channel. that password is now also compromised | 18:18 |
| JayF | cardoe: your WIP needs a Generated-By: then :D | 18:18 |
| TheJulia | brraaaains | 18:38 |
| TheJulia | cardoe: hate, never :) | 18:39 |
| TheJulia | Assuming that also you'll heavily modify the resulting content and all ;) | 18:40 |
| TheJulia | cardoe: also, wet cat deployment mechanisms should be an RPC endpoint ;) | 18:40 |
| kulsooms | Hi everyone, this is kulsoom. I'm new here. I found this channel through Outreachy's project list. I can't see the previous discussions? (I'm also new to IRC). | 19:09 |
| freemanboss[m] | kulsooms: You're welcome kulsooms: yeah that's how it's, you can't view previous messages before you join the channel. | 19:11 |
| JayF | kulsooms: https://meetings.opendev.org/irclogs/%23openstack-ironic/ most openstack channels do have public logs | 19:17 |
| kulsooms | ok thankyou, how can I tag mentors? or is it just "name:" ? | 19:22 |
| JayF | it's customary to address people similar to how I addressed you; however if you ping Riccardo and Masnoor today you're going to be lost in a sea of other applicants doing so. Might be a good idea to email instead. | 19:25 |
| cardoe | TheJulia: that should be the unknown state for nodes "WET_CAT" | 19:26 |
| iurygregory | kulsooms, Riccardo and Mahnoor nicks are rpittau and masghar | 19:34 |
| kulsooms | JayF: ok thankyou also, got my answer from the public log :)) | 19:34 |
| opendevreview | Merged openstack/ironic master: Remove IPA build on cleanup https://review.opendev.org/c/openstack/ironic/+/943987 | 20:04 |
| TheJulia | cardoe: wet cats *might*, require a physical origin of a solar powered catamaran. mordred might have opinions, or not. | 20:52 |
| TheJulia | The wet cat origin theory, having been possibly started in the openstack context by mordred, may require it. ;) | 20:53 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Ironic Config Intergration for IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/944951 | 21:15 |
| cardoe | TheJulia: so feels on us using objects for data instead of dicts? | 23:22 |
| Annosha | Hello everyone, I am Annosha an Outreachy contributor. Anyone here can assist in figuring out the chat for Outreachy round? | 23:33 |
| Annosha | I see no prior history of chats here. | 23:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!