| opendevreview | Merged openstack/ironic master: Correct bond_mode enum value for 802.3ad https://review.opendev.org/c/openstack/ironic/+/921340 | 00:08 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: Link MACs with PXE config to pxe_enabled ports https://review.opendev.org/c/openstack/ironic/+/921532 | 02:32 |
| shajizad | Good morning Ironic. I wanted to install devstack to run functional tests against, how much RAM should I allocate to the VM? | 06:31 |
| shajizad | I was told to allocate 8GB for bifrost, just wondering if i'll need the same | 06:32 |
| cid | I think you need a minimum of 4GB | 06:34 |
| cid | Good morning, shajizad | 06:35 |
| cid | So, around the same amount as bifrost is okay. | 06:37 |
| shajizad | I see, thanks! | 06:39 |
| dtantsur | shajizad: yeah, I'd go with at least 8 as well, 4 as a bare minimum | 06:47 |
| rpittau | good morning ironic! o/ | 06:48 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Makes metal3 integration job vote again https://review.opendev.org/c/openstack/ironic/+/921734 | 06:54 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Trivial: fix duplicated docstring https://review.opendev.org/c/openstack/sushy/+/921838 | 08:57 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Decompose the Redfish documentation https://review.opendev.org/c/openstack/ironic/+/921844 | 10:17 |
| dtantsur | FYI iurygregory ^^^ | 10:17 |
| opendevreview | Merged openstack/sushy master: Trivial: fix duplicated docstring https://review.opendev.org/c/openstack/sushy/+/921838 | 10:49 |
| iurygregory | good morning | 11:28 |
| iurygregory | dtantsur, ack will check it | 11:28 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2024.1: Correct bond_mode enum value for 802.3ad https://review.opendev.org/c/openstack/ironic/+/921847 | 11:44 |
| opendevreview | cid proposed openstack/ironic master: Provision ARM (aarch64) fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic/+/915441 | 11:45 |
| TheJulia | good morning | 13:07 |
| opendevreview | cid proposed openstack/ironic master: Provision ARM (aarch64) fake-bare-metal-vms https://review.opendev.org/c/openstack/ironic/+/915441 | 13:42 |
| opendevreview | Merged openstack/ironic master: Follow up to broken Zuul config https://review.opendev.org/c/openstack/ironic/+/921329 | 14:42 |
| JayF | I just workflow'd project mercury spec | 14:43 |
| TheJulia | coooool | 14:43 |
| opendevreview | Merged openstack/ironic master: Makes metal3 integration job vote again https://review.opendev.org/c/openstack/ironic/+/921734 | 14:52 |
| opendevreview | Merged openstack/ironic stable/2024.1: Correct bond_mode enum value for 802.3ad https://review.opendev.org/c/openstack/ironic/+/921847 | 14:52 |
| opendevreview | Merged openstack/bifrost master: Fix codespell reported errors https://review.opendev.org/c/openstack/bifrost/+/920410 | 14:54 |
| opendevreview | Merged openstack/ironic-specs master: Intermediate networking service https://review.opendev.org/c/openstack/ironic-specs/+/916126 | 14:59 |
| opendevreview | Merged openstack/ironic master: docs: add some additional context around iPXE and secure boot https://review.opendev.org/c/openstack/ironic/+/921403 | 15:02 |
| rpittau | need to drop, see you tomorrow! o/ | 15:18 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-inspector master: Cache openstacksdk Connections to avoid leaking memory https://review.opendev.org/c/openstack/ironic-inspector/+/921875 | 15:34 |
| dtantsur | TheJulia: any concerns with this ^^ approach from the OSP (or really any) perspective? | 15:34 |
| TheJulia | seems sane enough to me | 15:35 |
| JayF | do we need to require osdk >=2 with that change, too? | 15:36 |
| JayF | I assume it's backwards compat | 15:36 |
| TheJulia | changing the req would make that impossible to backport | 15:37 |
| JayF | ah, good point too | 15:37 |
| dtantsur | correct | 15:37 |
| dtantsur | also, nothing wrong with this change on earlier versions | 15:37 |
| JayF | yeah, that's what I was thinking -- caching conns on <2 osdk is probably OK too | 15:37 |
| TheJulia | yeah, should be | 15:38 |
| shajizad | Hello ironic, in the baremetal API there are 2 optional arguments for attaching a VIF to a node (port_uuid and portgroup_uuid) | 15:42 |
| shajizad | However in the SDK you cannot specify these parameters in .attach_vif | 15:42 |
| shajizad | Is there a reason for this? | 15:42 |
| shajizad | * Hello ironic, in the baremetal API there are 2 optional arguments for attaching a VIF to a node (port_uuid and portgroup_uuid) | 15:43 |
| shajizad | However in the SDK you cannot specify these parameters in .attach_vif | 15:43 |
| shajizad | Is there a reason for this? | 15:43 |
| dtantsur | I suspect, nobody implemented these | 15:43 |
| JayF | shajizad: if you can file a bug requesting we add that behavior, we can prioritize someone getting it done | 15:44 |
| JayF | I think that makes sense as a thing to do | 15:44 |
| dtantsur | I think shajizad could be this "someone" ;) | 15:45 |
| TheJulia | I forgot https://github.com/openstack/ironic/commit/e8f6fdd56d7eccbdeba3ff3d1101a8dc6a8be72f even happened | 15:47 |
| dtantsur | yeah, it went mostly unnoticed for me as well | 15:48 |
| dtantsur | maybe we just celebrated it too much? | 15:48 |
| * dtantsur upgraded his lab machine to CentOS Stream 9 and now has minus one lab machine :( | 15:49 | |
| TheJulia | yeouch | 15:50 |
| JayF | dtantsur: awesome, yeah, shajizad please do it if you are capable and have time :D would love to review it o/ | 15:50 |
| TheJulia | I took a look at what Mainn did when he did that, he didn't make an sdk change, only python-ironicclient | 15:52 |
| shajizad | <JayF> "dtantsur: awesome, yeah, Salman..." <- Alright, we'll add it to our list of tasks as a team | 16:35 |
| shajizad | Currently we are implementing node history, firmware information and attach/detach virtual media | 16:35 |
| JayF | What team, exactly? | 16:35 |
| * JayF has been in Europe the last two weeks and probably missed a thing or seventeen | 16:36 | |
| shajizad | JayF: Me and some other students are taking a summer course at CMU Qatar in which we contribute towards openstacksdk | 16:37 |
| shajizad | Maybe iurygregory: could tell you more about it | 16:37 |
| JayF | Oh, awesome! I've heard vague rumblings of that. | 16:37 |
| JayF | If there's anything I can do to help, let me know. Happy to be a resource as part of community mentoring. | 16:37 |
| shajizad | Sure, thank you! | 16:39 |
| JayF | Is this course only directed towards openstack? Or are there other teams working on other OSS projects? | 16:40 |
| dtantsur | Could be a question for diablo_rojo, I guess | 16:45 |
| JayF | We have people dedicated to university outreach at GR-OSS who (mostly) work in other communities, I'm happy to make introductions if helpful -- that's all I was fishing for :D | 16:45 |
| shajizad | <JayF> "Is this course only directed..." <- The course is about software engineering in general and each team contributes towards some open source project | 16:47 |
| shajizad | There's 5 teams, we are working on support for ironic in openstacksdk, another team is working on openstack swift | 16:47 |
| shajizad | Some teams are working on some eclipse foundations projects | 16:47 |
| JayF | TIL eclipse has a foundation | 16:47 |
| JayF | it's nice that you have people digging into different communities | 16:48 |
| dtantsur | I'm surprised you missed it, I hear about it quite often in various contexts :) | 16:48 |
| * dtantsur is trying remember which was the last time; happened an eternity (i.e. a few days) ago | 16:49 | |
| iurygregory | shajizad, hey o/ sorry I was having lunch | 16:57 |
| iurygregory | great job finding a bug in the ironic part for openstack sdk :D | 16:59 |
| iurygregory | yeah, probably more a question to diablo_rojo than to me if you want to know more JayF | 16:59 |
| TheJulia | dtantsur: possibly eclipse's outreach regarding guideline building | 17:13 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/896570 this really needs another core review and to be landed | 18:03 |
| JayF | it's relatively complex and it'd be unfortunate if we had to rebase | 18:03 |
| TheJulia | funny enough, I had someone ask for a link on monday because they were pulling it into their fork | 18:19 |
| JayF | Unless there are strong feelings about it, I' | 18:21 |
| opendevreview | Verification of a change to openstack/ironic master failed: minor: additional tests for interaction around node lookup https://review.opendev.org/c/openstack/ironic/+/921549 | 18:21 |
| JayF | **I'd like to make some stable/2023.1 IPA CI jobs nonvoting | 18:21 |
| JayF | mainly because we don't backport much there, so I'm not scared of it, but also am unsure keeping the CI there passing is worth the effort | 18:21 |
| TheJulia | I think the disk image build stuffs are also slight broken and require attention | 18:24 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Fix versions in release notes https://review.opendev.org/c/openstack/ironic-python-agent/+/904104 | 18:26 |
| JayF | Gates are completely broken | 18:27 |
| JayF | TheJulia: yes, I have an idea what the fix is there too | 18:27 |
| JayF | TheJulia: I think we just need to ensure /dev/shm inside the build chroot is a tmpfs | 18:27 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.1: Revert "ci: stable-only: explicitly pin centos build" https://review.opendev.org/c/openstack/ironic/+/921884 | 18:29 |
| JayF | FYI: Do not land anything, gates are broken, see #openstack-infra for more info, I'm working it w/fungi | 18:30 |
| TheJulia | hmm, apparently I was not in there | 18:33 |
| opendevreview | Verification of a change to openstack/ironic master failed: minor: additional tests for interaction around node lookup https://review.opendev.org/c/openstack/ironic/+/921549 | 19:38 |
| shajizad | I sent the VIF attachment fix for review 👍 | 19:45 |
| TheJulia | shajizad: awsome! | 19:50 |
| TheJulia | err, awesome! | 19:50 |
| opendevreview | Merged openstack/ironic master: minor: additional tests for interaction around node lookup https://review.opendev.org/c/openstack/ironic/+/921549 | 19:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!