| rpittau | good morning ironic! o/ | 07:28 |
|---|---|---|
| *** tkajinam is now known as tkajinam|away | 08:33 | |
| dtantsur | good morning folks | 09:03 |
| dtantsur | lost scrollback over the weekend, please repeat any pings | 09:03 |
| * dtantsur is trying Stream 9 for his bifrost VMs now that his lab machine got restarted anyway | 09:18 | |
| rpittau | I'm not alone in the lab restarting madness \o/ | 10:07 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 10:58 |
| iurygregory | good morning Ironic | 11:12 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 12:28 |
| TheJulia | good morning | 12:58 |
| TheJulia | iurygregory: any word on multipath? | 13:05 |
| iurygregory | TheJulia, a lot .... | 13:05 |
| iurygregory | but they haven't tested yet.. | 13:05 |
| iurygregory | because of many different problems | 13:05 |
| TheJulia | oh joy | 13:06 |
| TheJulia | I guess my only concern is it is CR-2 at present | 13:06 |
| iurygregory | I can give some details in the downstream channel | 13:06 |
| TheJulia | And.. I'm not sure it is not a generally good chagne, but agree we need to be sure about it | 13:06 |
| rpioso | Good morning, ironic :) | 13:07 |
| TheJulia | The only major difference though is we use device mapper names when detected, I think. | 13:08 |
| TheJulia | But I guess to "truly" test it in CI, we need multipath-tools/device-mapper-multipath installed | 13:08 |
| * TheJulia wonders if there *is* any way to do multipath simulation in CI | 13:08 | |
| hjensas | TheJulia: https://sharkcz.livejournal.com/12846.html | 13:11 |
| TheJulia | muahahahahaha | 13:12 |
| TheJulia | I was thinking of trying exactly that | 13:12 |
| TheJulia | Maybe something to do once I try to get my new devstack VM working | 13:12 |
| TheJulia | Silly... new ubuntu version | 13:12 |
| TheJulia | good morning tzumainn | 13:14 |
| tzumainn | TheJulia, hi! | 13:14 |
| TheJulia | devstack's random IO this morning is thrashing one of my disks :( | 13:44 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 13:48 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 14:17 |
| *** tkajinam|away is now known as tkajinam | 14:42 | |
| iurygregory | TheJulia, summary from the PTG on the ML | 14:43 |
| iurygregory | \o/ | 14:43 |
| iurygregory | really sorry for the delay (the downstream case was driving all my energy in the last two weeks...) | 14:43 |
| TheJulia | no worries | 14:48 |
| opendevreview | Merged openstack/ironic master: Fixes log formatiing string. https://review.opendev.org/c/openstack/ironic/+/839132 | 14:50 |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Apr 25 15:00:00 2022 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| iurygregory | Hello ironicers! | 15:00 |
| erbarr | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| iurygregory | Welcome to our weekly meeting! | 15:00 |
| dtantsur | o/ | 15:00 |
| hjensas | o/ | 15:00 |
| TheJulia | o/ | 15:00 |
| ajya | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| iurygregory | the agenda for our meeting can be found in the wiki | 15:00 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meetin | 15:00 |
| stendulker | o/ | 15:01 |
| iurygregory | ops wrong link | 15:01 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| iurygregory | #topic Announcements / Reminder | 15:01 |
| iurygregory | #info Zed PTG Summary is the ML | 15:02 |
| iurygregory | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-April/028293.html | 15:02 |
| rloo | o/ | 15:02 |
| kamlesh6808c | o/ | 15:02 |
| iurygregory | sorry about the delay on this and also on the patch with the priorities for the cycle, last two weeks were a bit complicated downstream | 15:03 |
| iurygregory | #info First bugfix branch to be created next week | 15:04 |
| TheJulia | I guess I'm curious, first bugfix like 4 weeks into the new cycle? | 15:04 |
| rloo | thx for the PTG summary iurygregory! | 15:04 |
| iurygregory | TheJulia, yeah... | 15:04 |
| rloo | what, there's a bug? :D | 15:04 |
| TheJulia | There are always bugs | 15:04 |
| TheJulia | They shall rule the world one day! | 15:04 |
| rpioso | More, now that it's spring :) | 15:04 |
| TheJulia | rpioso: exactly! | 15:04 |
| TheJulia | iurygregory: just every six week timing from the last? | 15:05 |
| iurygregory | some information to help on that front.. downstream we consume bugfix and stable branches, and we had some changes in the calendar | 15:06 |
| rpittau | I think we're in between 5 and 7 weeks, so we should be good? Next week is 5 weeks from the branch cut | 15:06 |
| iurygregory | yeah we will be 5 weeks I think | 15:06 |
| TheJulia | Okay | 15:06 |
| iurygregory | it would help so we don't have to do downstream backports from a feature we had included in *zed* to *yoga* downstream | 15:07 |
| TheJulia | ugh | 15:07 |
| TheJulia | fun! | 15:07 |
| TheJulia | Anyway, onward | 15:08 |
| iurygregory | ok o/ | 15:08 |
| iurygregory | #topic Review action items from previous meeting | 15:08 |
| iurygregory | well I don't think I added as action item, but it's one.. I had to push the summary + the priorities patch so people cold review | 15:09 |
| iurygregory | I've done the summary with some delay, the patch with priorities will be up after my lunch today | 15:09 |
| iurygregory | #Review subteam status reports | 15:10 |
| iurygregory | we will likely skip this week and get back to it next week, thoughts? | 15:10 |
| iurygregory | ok, moving on =) | 15:12 |
| TheJulia | yeah, move on | 15:12 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:12 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:12 |
| TheJulia | I'm thinking of putting multipath patches up... if I do so, any objection if I add them to the prio review list? | 15:13 |
| iurygregory | Does anyone have topics that we should review? we only have 4, I know the tempest-plugin one have been open for a while (I will take a look, I was quite busy past 3 weeks) | 15:13 |
| iurygregory | TheJulia, ++ to adding | 15:13 |
| iurygregory | rpittau, since you have a -2 on it, what are your thoughts? | 15:14 |
| TheJulia | So looks like I could add the auto-add lessee id field feature. two minor things to fix and it shoudl be good to go | 15:14 |
| TheJulia | I'm thinking so we actually have a disk representing it, although it might break our RAID testing. Would be interesting to see! | 15:15 |
| rpittau | iurygregory: let's add that to priority, my -2 was related to the testing part | 15:15 |
| TheJulia | I'm also focused on trying to get our v6 job sorted | 15:15 |
| iurygregory | i would just try to keep as simple as possible so we can backport without many problems TheJulia =D | 15:15 |
| TheJulia | but I think the issue is in devstack at the moment | 15:15 |
| iurygregory | rpittau, ack =) | 15:16 |
| TheJulia | iurygregory: well, at least on master we can likely start testing it fairly easily | 15:16 |
| TheJulia | I'll add related patches once they are up | 15:16 |
| iurygregory | makes sense to me | 15:16 |
| iurygregory | I will also add the grenade skip one after looking at the feedback (tks TheJulia and dtantsur ) | 15:18 |
| iurygregory | and ofc the one with the priorities I will push =D | 15:18 |
| iurygregory | ok, moving on | 15:19 |
| iurygregory | #topic Open discussion | 15:19 |
| iurygregory | we have one topic today \o/ | 15:19 |
| iurygregory | #info Transition Victoria to EM | 15:20 |
| TheJulia | skipping rfe review? | 15:20 |
| iurygregory | rfe is after sig | 15:20 |
| TheJulia | ahh, open discussion should be at the end | 15:20 |
| iurygregory | at least in the order I see in the agenda... | 15:20 |
| TheJulia | Sorry | 15:20 |
| TheJulia | since open ended discussions can end the meeting | 15:20 |
| TheJulia | Anyway, ignore me | 15:20 |
| iurygregory | no worries! | 15:20 |
| TheJulia | I think i figured out why our v6 job is broken | 15:20 |
| iurygregory | #link https://review.opendev.org/c/openstack/releases/+/837937 | 15:21 |
| iurygregory | I'm wondering if we want to push some releases before tagging victoria-em | 15:22 |
| TheJulia | seems reasonable if they are out of date | 15:22 |
| iurygregory | #action iury to check if we have releases in victoria before moving to em | 15:22 |
| iurygregory | #topic Baremetal SIG | 15:23 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:23 |
| iurygregory | #info Recording of the last SIG meeting - Manuel Holtgrewe on "Bare Metal for Health - Using OpenStack Ironic for HPC at Berlin Institute of Health" | 15:24 |
| iurygregory | #link https://youtu.be/5yJvXFOqzSI | 15:24 |
| iurygregory | arne_wiebalck, anything to add for the SIG? | 15:24 |
| iurygregory | ok, I think Arne is not around today =) | 15:26 |
| iurygregory | moving on | 15:26 |
| iurygregory | #topic RFE review | 15:26 |
| iurygregory | #info Prevent mass instance deletions/rebuilds | 15:27 |
| iurygregory | #link https://storyboard.openstack.org/#!/story/2010007 | 15:27 |
| iurygregory | TheJulia, o/ | 15:27 |
| TheJulia | So! | 15:27 |
| TheJulia | I proposed two RFE's based upon discussions during the PTG | 15:27 |
| TheJulia | The first is a basic mechanism to allow preventing mass deletions of nodes | 15:28 |
| TheJulia | The idea being lightweight, and simple to implement, and I believe fairly easy for us to wire in. I'd appreciate any feedback. | 15:28 |
| TheJulia | The latter is regarding Agent power savings | 15:29 |
| TheJulia | #link https://storyboard.openstack.org/#!/story/2010008 | 15:29 |
| TheJulia | I've posted a WIP of this and did some local experimentation, just to keep it simple | 15:29 |
| TheJulia | #link https://review.opendev.org/c/openstack/ironic-python-agent/+/839095 | 15:29 |
| TheJulia | basically, just changes the governor and tries to invoke intel's internal pstate selector if present | 15:30 |
| TheJulia | Please let me know if there are any thoughts or concerns, otherwise I'll proceed with as I've started | 15:30 |
| iurygregory | this is related to the safeguards we talked at the PTG right? | 15:31 |
| TheJulia | yes | 15:31 |
| iurygregory | ok, in my mind it was only for cleaning, but the idea does make a lot of sense after reading what you wrote in the RFE | 15:32 |
| rpittau | sorry, I need to drop, I'll check the backlog later o/ | 15:32 |
| iurygregory | bye rpittau =) | 15:32 |
| TheJulia | I'm thinking upstream can carry a reasonable default, and folks like Arne can tune the setting down to match their environment's usage | 15:33 |
| iurygregory | ++ yeah | 15:33 |
| TheJulia | and actually, that reasonable default *could* just default on startup to number of conductors * threads too | 15:34 |
| TheJulia | maybe that is not reasonable | 15:34 |
| TheJulia | Anyway, just thoughts | 15:34 |
| TheJulia | oh, we should add https://review.opendev.org/c/openstack/ironic/+/818299 to the list for reviews | 15:34 |
| iurygregory | interesting idea (# conductors * threads) | 15:35 |
| TheJulia | In similar vain of protections, I posted another WIP to get an idea out of my head | 15:35 |
| TheJulia | https://review.opendev.org/c/openstack/ironic-python-agent/+/839084 | 15:35 |
| TheJulia | Which also kind of starts building a groundwork an intern can carry forward | 15:35 |
| iurygregory | TheJulia, I just noticed you mention in the RFE *This spec proposes* | 15:36 |
| iurygregory | in the end you think it requires a spec? | 15:36 |
| TheJulia | oh, old habits maybe? | 15:36 |
| iurygregory | maybe =) | 15:36 |
| iurygregory | I just wanted to double check, because after reading I don't think it would require one... | 15:37 |
| iurygregory | but maybe is just me :D we need more feedback to make a final decision | 15:37 |
| iurygregory | to me it does make sense, no objections | 15:38 |
| TheJulia | I did go through and look for common shared disk/lun block filesystems | 15:38 |
| TheJulia | and added the 3 to that patch that made sense | 15:38 |
| dtantsur | I don't think we should add all filesystems | 15:38 |
| TheJulia | maybe not for GPFS, but I found a shared block device example in IBM's docs | 15:38 |
| dtantsur | only that magical ones that can wipe the cluster via iBFT or whatever | 15:38 |
| TheJulia | yeah, and those are these filesystems | 15:38 |
| dtantsur | I highly doubt GFS2 is one | 15:38 |
| TheJulia | According to docs, it is, but I'm happy to remove it. It also doesn't use partitions or a table, which caused me to raise an eyebrow some | 15:39 |
| TheJulia | s/table/mbr or gpt table/ | 15:39 |
| dtantsur | I mean... there is a legitimate case of wiping members of distributed file systems | 15:39 |
| dtantsur | think, decomissioning of a storage node | 15:39 |
| TheJulia | absolutely! | 15:39 |
| dtantsur | I don't disagree that people should do a clean-up first, but it may be impossible e.g. if the OS has died | 15:39 |
| TheJulia | and distributed filesystems in network distributed mode, absolutely | 15:39 |
| TheJulia | agreed, which is why I've also though to fan easy node and global level knob | 15:40 |
| TheJulia | "smash it all, I don't care" sort of button | 15:40 |
| TheJulia | but maybe not a button | 15:40 |
| TheJulia | TBD | 15:40 |
| hjensas | GFS2 is for shared access to the same device. So I think it is something we want to protect? | 15:40 |
| dtantsur | being a distributed fs is not enough | 15:40 |
| TheJulia | but shared block to same device is kind of it, Distributed as in Ceph doesn't use shared block devices | 15:41 |
| dtantsur | we're talking about some $magic that will let the cluster notice (and get broken) even if the cluster software is no longer running | 15:41 |
| TheJulia | indeed, and GFS2 is one of those | 15:41 |
| dtantsur | so, some ring -1 level magic | 15:41 |
| TheJulia | it is not magic actually | 15:41 |
| TheJulia | it is range locking instead of whole device locking | 15:41 |
| TheJulia | for IO at least | 15:41 |
| dtantsur | range locking that survives reboot? | 15:42 |
| TheJulia | Depends on the SAN! | 15:42 |
| * TheJulia has had to reboot a SAN controller due to it keeping a range lock in RAM | 15:42 | |
| TheJulia | With VMFS actually | 15:42 |
| TheJulia | I couldn't vmotion the VMs off the dead hypervisor | 15:42 |
| dtantsur | isn't GFS a software thing? | 15:42 |
| TheJulia | because the block ranges were locked | 15:42 |
| TheJulia | No, its a kernel module | 15:42 |
| dtantsur | still, you're talking about a SAN | 15:43 |
| TheJulia | GFS as in Google Filesystem.. .is unrelated and is software AIUI | 15:43 |
| TheJulia | gfs2 is like any other filesystem, it just does range locking instead of whole device locking | 15:43 |
| TheJulia | specifically Red Hat GFS2 | 15:44 |
| TheJulia | Anyway, we can move on | 15:44 |
| TheJulia | got our v6 devstack scripting past Neutron \o/ | 15:44 |
| iurygregory | yay | 15:45 |
| iurygregory | ok, next topic o/ | 15:45 |
| iurygregory | #topic Who is going to run the next meeting? | 15:45 |
| iurygregory | do we have any volunteers? | 15:45 |
| iurygregory | I will run the next meeting, thanks everyone! | 15:46 |
| iurygregory | #endmeeting | 15:46 |
| opendevmeet | Meeting ended Mon Apr 25 15:46:48 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-04-25-15.00.html | 15:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-04-25-15.00.txt | 15:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-04-25-15.00.log.html | 15:46 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: v6/grenade multinode jobs https://review.opendev.org/c/openstack/ironic/+/839086 | 15:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: v6/grenade multinode jobs https://review.opendev.org/c/openstack/ironic/+/839086 | 15:59 |
| TheJulia | Odds are high that shoudl work | 15:59 |
| ftarasenko | Hi, Ironic! | 16:02 |
| ftarasenko | ajya: Hi! Regarding race condition while cleaning with RAID creation, I have conductor logs uploaded here https://drive.google.com/file/d/1gF7IKTjVP7skO1HQpQuy-S3pr6X5gEdg/view?usp=sharing I think that IPA logs might not help you cause nothing actually happens on IPA. Also I've noticed that after changing to idrac-redfish raid interface additional 50% of nodes succeeded to create RAID. But I still have 2-3 | 16:02 |
| ftarasenko | nodes in each deployment which do not clean with any RAID interface. I can try to create debug logs if required. | 16:02 |
| ajya | ftarasenko: thanks, will take a look tomorrow and see if I can spot anything unusual | 16:04 |
| ftarasenko | ajya: Sure. Thanks. I have couple of deployments to run additional test at any time if required. Also want to notice that today it was R340 Server and last week it were R650. | 16:06 |
| TheJulia | iurygregory: so looks like I can get the v6 job working again, looks like Neutron's agent has some issue right now or on the newest ubuntu | 16:25 |
| TheJulia | rpittau: Just fyi, got past enrolling baremetal nodes, and getting networking up with one of the minor devstack fixes under discussion, so things should be fairly good. At least using sushy-tools. Virtualbmc on the other hand... | 16:25 |
| dtantsur | we don't *have* to use IPMI everywhere | 16:46 |
| iurygregory | TheJulia, ack | 16:47 |
| TheJulia | dtantsur: agree 100000% | 16:47 |
| TheJulia | unfortunately we'll still want at least one ipmi job | 16:47 |
| dtantsur | well, but not necessarily with v6, right? | 16:48 |
| TheJulia | dtantsur: the issue is python 3.10 incompatability with virtualbmc | 16:50 |
| dtantsur | .... | 16:50 |
| dtantsur | cause pyghmi, right? | 16:50 |
| dtantsur | as a workaround, add dead snakes, use venv :) | 16:50 |
| TheJulia | actually, at a glance it was not pyghmi | 16:51 |
| dtantsur | (if someone does not know what dead snakes is: https://launchpad.net/~deadsnakes/+archive/ubuntu/ppa) | 16:52 |
| TheJulia | neat | 16:52 |
| dtantsur | new ubuntu = jammy? | 16:53 |
| TheJulia | yup | 16:54 |
| * TheJulia thinks she sees why the multitenacy tests are unhappy | 16:54 | |
| TheJulia | ewww, and we run it twice by accident | 16:57 |
| * dtantsur has just updated his bifrost VMs to CS9 | 16:58 | |
| dtantsur | see you tomorrow folks o/ | 16:58 |
| TheJulia | o/ | 17:13 |
| TheJulia | hmm.... more than one issue on multinode | 17:32 |
| rpioso | TheJulia: Regarding https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/826646/2..3/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_cleaning.py#b175, are you suggesting the following? | 17:56 |
| rpioso | TheJulia: credentials = ['primary', 'system_admin'] | 17:57 |
| TheJulia | so 'primary', 'admin', 'system_admin' I think | 17:58 |
| TheJulia | at least, until I strip the legacy auth stuff out in aa cycle | 17:58 |
| rpioso | TheJulia: Isn't that already done by its base class, BaremetalStandaloneManager -- https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/scenario/baremetal_standalone_manager.py#L34? | 18:01 |
| TheJulia | yup | 18:02 |
| TheJulia | in which case, because the test is derived from that, it doesn't need to be defined on that test | 18:02 |
| rpioso | TheJulia: Thank you for clarifying :) | 18:02 |
| rpioso | TheJulia: In contrast, do we prefer the min_microversion be explicitly set in the derived class? | 18:03 |
| TheJulia | The test should be mapped to a version, where appropriate of course, most aligning the minimum version of ironic which supports the functionality | 18:04 |
| TheJulia | you don't want to invoke the test and fail when the system doesn't actually support it | 18:04 |
| TheJulia | hopefully that makes sense | 18:04 |
| rpioso | TheJulia: Got that :) | 18:05 |
| rpioso | TheJulia: Similar to credentials, min_microversion is set to the derived test's needed value by its base class -- https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/scenario/baremetal_standalone_manager.py#L37 | 18:06 |
| TheJulia | Yes, however if your clean step is exercising something that didn't exist until say, 1.46 internally, you'll want to set that as the minimum version | 18:16 |
| TheJulia | since the tempest plugin is branchless | 18:16 |
| rpioso | TheJulia: s/min_microversion/api_microversion/ | 18:20 |
| TheJulia | min microversion is the minimum version the test can be successful at | 18:20 |
| rpioso | TheJulia: What's api_microversion? | 18:22 |
| TheJulia | Are you asking what is an API microversion? | 18:27 |
| rpioso | TheJulia: I'm trying to understand how the test variables min_microversion and api_microversion differ? | 18:29 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 18:33 |
| TheJulia | I'm not sure there is an explicit api_microversion setting | 18:33 |
| TheJulia | hmm th ere is | 18:34 |
| TheJulia | rpioso: it sets the explicit version to use | 18:35 |
| TheJulia | which is a bad habit of ours | 18:35 |
| TheJulia | we should be using min and max | 18:35 |
| *** rcastillo_ is now known as rcastillo | 19:02 | |
| * rpioso was in a downstream meeting. Out now :) | 19:04 | |
| rpioso | TheJulia: The new class BaremetalIdracManagementCleaning's base classes set those. | 19:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: v6/grenade multinode jobs https://review.opendev.org/c/openstack/ironic/+/839086 | 19:06 |
| rpioso | TheJulia: BaremetalStandaloneManager: min_microversion = '1.28' and BaremetalStandaloneScenarioTest: api_microversion = '1.28'. | 19:07 |
| TheJulia | That is really an antipattern, tbh | 19:07 |
| TheJulia | min/max microversion allow client negotiation to occur | 19:07 |
| rpioso | TheJulia: Should BaremetalIdracManagementCleaning explicitly set either of them? | 19:08 |
| TheJulia | just min_microversion I think | 19:08 |
| rpioso | TheJulia: ack re: antipattern and negotiation. | 19:08 |
| TheJulia | One more meeting and then I think I'm going to lay down for a little while | 19:09 |
| admiyo | When last we left out heroes...I was able to boot using bifrost and the snponly.efi on AARCH 64. I want to craft this into a patch for Bifrost, but I am not certain when I would need to go snponly (which works for me) and when to go for something else. | 19:09 |
| admiyo | Fun fact. SNP stands for Simple Network Protocol. It is not a Network Protocol. It is an API for talking to network cards. I've been told that this is a Microsoftism. | 19:10 |
| rpioso | TheJulia: The already existing cleaning standalone tests explicitly set only api_microversion: https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_cleaning.py. | 19:10 |
| TheJulia | admiyo: so I think it is part of the UEFI standard, but maybe not by that exact name | 19:11 |
| TheJulia | rpioso: :( | 19:11 |
| admiyo | Pretty sure it is that exact name | 19:11 |
| admiyo | I looked in the EUFI standard | 19:11 |
| admiyo | UEFI. Man, I need to stop assuming I can edit my chats. | 19:12 |
| TheJulia | admiyo: So.... we've basically found that for UEFI, snp only is the only way | 19:12 |
| TheJulia | at least, in more recent firmware | 19:12 |
| TheJulia | and by that... new in the last 4-5 years | 19:12 |
| TheJulia | s/UEFI/EFI/ | 19:12 |
| TheJulia | it is not universal | 19:12 |
| admiyo | OK, so I can make snponle.efi the default, at lease for arm64? | 19:15 |
| admiyo | TheJulia, the other change I would need to make is that for aarch6, get the firmware from a subdir: https://boot.ipxe.org/arm64-efi/ so a little switch logic based on arch | 19:17 |
| admiyo | other than that it should be minimal. Which makes for faster patch reviews and acceptance. | 19:18 |
| TheJulia | yeah, that should be reasonable | 19:18 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 19:25 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: v6/grenade multinode jobs https://review.opendev.org/c/openstack/ironic/+/839086 | 19:44 |
| rpioso | TheJulia: Thank you for your assistance. It was very helpful. I summarized my understanding in comments I posted to the review. | 19:47 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 19:48 |
| TheJulia | rpioso: happy to help | 20:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!