| rpittau | good morning ironic! o/ | 07:00 |
|---|---|---|
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/905812 | 07:44 |
| kubajj | good morning ironic! o/ | 09:07 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Refactor the use of include_vars https://review.opendev.org/c/openstack/bifrost/+/888448 | 09:12 |
| iurygregory | good morning ironic | 11:22 |
| kubajj | good morning iurygregory! | 11:36 |
| opendevreview | Merged openstack/ironic master: [docs] Add procedure to EoL and delete old bugfix branch https://review.opendev.org/c/openstack/ironic/+/926648 | 12:35 |
| TheJulia | good morning | 12:45 |
| cardoe | good morning ironic | 13:02 |
| opendevreview | Merged openstack/ironic master: docs-audit-2024: Labeling references https://review.opendev.org/c/openstack/ironic/+/925691 | 13:14 |
| TheJulia | good morning cardoe | 13:25 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 13:28 |
| cardoe | Just a little open request wrangling.. I think the following are ready for workflow: https://review.opendev.org/c/openstack/sushy/+/909451 & https://review.opendev.org/c/openstack/ironic/+/926685 & https://review.opendev.org/c/openstack/ironic/+/926899 | 13:34 |
| cardoe | Those are 3 with a +2 code review and +1 from zuul with no open comments | 13:36 |
| cardoe | Just figured I'd wrangle a little before the meeting. | 13:37 |
| cardoe | https://review.opendev.org/c/openstack/openstacksdk/+/926414 would be a good one to land before the release. It needs a +2 first. JayF I resolved your open comment because cid handled it in patch 3 and you gave patch 3 a +1 | 13:40 |
| cardoe | if I can get https://review.opendev.org/c/openstack/ironic/+/926227 over the finish line before the release I think it would be a positive one given the release notes talking about deprecating idrac-wsman and pushing people to the redfish | 13:42 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/926939 is another one that's just needing workflow. | 13:56 |
| cardoe | Anyway, I hope this list helps someone. | 13:56 |
| opendevreview | Merged openstack/ironic-inspector master: Follow up to broken Zuul config https://review.opendev.org/c/openstack/ironic-inspector/+/921351 | 14:05 |
| JayF | cardoe: I'll note that while I technically have plus two on openstack SDK as a TC member, I'm not actually a core on that team | 14:12 |
| cardoe | wrt https://review.opendev.org/c/openstack/ironic/+/926227 I'm trying to silently steer people to "redfish" instead of "idrac-redfish" when the interface is now just a no-op | 14:20 |
| cardoe | JayF: ^ | 14:20 |
| opendevreview | Merged openstack/bifrost master: Refactor the use of include_vars https://review.opendev.org/c/openstack/bifrost/+/888448 | 14:28 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 14:37 |
| opendevreview | Merged openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 14:52 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 14:54 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Aug 26 15:00:10 2024 UTC and is due to finish in 60 minutes. The chair is rpittau. 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 |
| dtantsur | o/ | 15:00 |
| masghar | o/ | 15:00 |
| rpittau | let's give a couple of minutes for people to join | 15:00 |
| cid | \o | 15:01 |
| JayF | o/ | 15:01 |
| TheJulia | o/ | 15:01 |
| rpittau | ok, I think we can start :) | 15:02 |
| rpittau | Hello everyone! | 15:02 |
| rpittau | Welcome to our weekly meeting! | 15:02 |
| rpittau | The meeting agenda can be found here: | 15:02 |
| rpittau | https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_August_26.2C_2024 | 15:02 |
| rpittau | #topic Announcements/Reminders | 15:02 |
| rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: | 15:02 |
| rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| rpittau | related to this ^ this week is Feature Freeze week according to the Dalmatian Release schedule | 15:03 |
| rpittau | #link https://releases.openstack.org/dalmatian/schedule.html | 15:03 |
| rpittau | this week is also the Final release for client libraries week | 15:04 |
| rpittau | some releases (non client) are already up for review | 15:04 |
| rpittau | #info PTL nominations period ends August 28 | 15:05 |
| TheJulia | rpittau: which releases are already up? | 15:05 |
| rpittau | TheJulia: bifrost, ipa-builder, ironic-ui, networking-baremetal, networking-generic-switch, and ironic-prometheus-explorer | 15:06 |
| * cid updated https://wiki.openstack.org/wiki/Meetings/Ironic with bug deputy report | 15:06 | |
| rpittau | thanks cid :) | 15:06 |
| TheJulia | Thanks | 15:06 |
| rpittau | ironicclient should come up at some point this week | 15:06 |
| rpittau | and remember that Changes that integrate with other OpenStack projects that follow feature feature must be merged by August 30 | 15:07 |
| rpittau | #info OpenInfra PTG October 21-25, 2024 virtually! | 15:07 |
| rpittau | #link https://ptg.openinfra.dev/ | 15:07 |
| rpittau | add your topics here https://etherpad.opendev.org/p/ironic-ptg-october-2024 | 15:07 |
| rpittau | any questions or anything else to annonce/remind? | 15:08 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 15:09 |
| rpittau | ok, moving on! | 15:09 |
| rpittau | #topic Review Ironic CI status | 15:09 |
| rpittau | anything worth mentioning for CI? | 15:09 |
| rpittau | wow, alright! :D | 15:10 |
| rpittau | onward | 15:10 |
| rpittau | #topic Discussion Topics | 15:11 |
| rpittau | #info ironic dalmatian cycle highlights are up for review | 15:11 |
| rpittau | #link https://review.opendev.org/c/openstack/releases/+/927116 | 15:11 |
| rpittau | please please check the cycle highlights | 15:11 |
| rpittau | deadline for that is September 5 | 15:11 |
| opendevreview | OpenStack Release Bot proposed openstack/tenks stable/2.0: Update .gitreview for stable/2.0 https://review.opendev.org/c/openstack/tenks/+/927159 | 15:11 |
| opendevreview | OpenStack Release Bot proposed openstack/tenks stable/2.0: Update TOX_CONSTRAINTS_FILE for stable/2.0 https://review.opendev.org/c/openstack/tenks/+/927160 | 15:11 |
| opendevreview | OpenStack Release Bot proposed openstack/tenks master: Update master for stable/2.0 https://review.opendev.org/c/openstack/tenks/+/927161 | 15:11 |
| rpittau | ok, next one | 15:12 |
| rpittau | #info I will be out for 3 weeks starting Wednesday August 28 | 15:12 |
| rpittau | I'll check emails and IRc from time to time, but probably won't be able to do much | 15:13 |
| opendevreview | Merged openstack/sushy master: Adds Ethernet Interfaces for manager. https://review.opendev.org/c/openstack/sushy/+/909451 | 15:13 |
| rpittau | please keep an eye on the cycle highlights :D | 15:14 |
| rpittau | also on the releases, but I whould be back before the final ones | 15:14 |
| TheJulia | Worth noting, Monday is a holiday in the US. | 15:14 |
| rpittau | s/whould/should | 15:14 |
| TheJulia | Next monday, that is. | 15:14 |
| iurygregory | o/ | 15:14 |
| * iurygregory is late | 15:14 | |
| rpittau | ok | 15:14 |
| * iurygregory won't be around next week because of the OIS Asia | 15:14 | |
| iurygregory | I would skip next weekly meeting =) | 15:15 |
| rpittau | more than the meeting, releases and highlights are quite important and sensitive | 15:15 |
| kubajj | o/ | 15:15 |
| iurygregory | I added some comments to the highlights an | 15:15 |
| iurygregory | and I'm checking the releases =) | 15:15 |
| rpittau | if I'm not able to update the cycle highlights before Friday, please someone take over in case there are still comments, thanks! | 15:16 |
| cid | CI is having some timeouts, it's not clear to me yet if that's change specific. | 15:16 |
| TheJulia | The highlights LGTM | 15:16 |
| iurygregory | rpittau, ack | 15:16 |
| rpittau | thanks TheJulia iurygregory :) | 15:16 |
| rpittau | any more discussion topics for today? | 15:17 |
| dtantsur | I just want to mention that I"ve created a spec on hardware that cannot be powered off | 15:17 |
| JayF | I wanted to ask how folks would feel about an implementation of firmware interface that used in band steps to execute the firmware upgrade | 15:17 |
| TheJulia | Are we setting aside time for the ptg to discuss that? | 15:17 |
| JayF | If there's general agreement that would fit, I'll pursue writing a spec in the next few weeks | 15:18 |
| dtantsur | JayF: I sorta feel like it's inevitable, it's just that we're focused on Redfish for Metal3/OpenShift | 15:18 |
| dtantsur | TheJulia: "that" = Jay's firmware interface or my NC-SI work? | 15:18 |
| TheJulia | dtantsur: NC-SI fun | 15:19 |
| TheJulia | But, thinking about it, Jay's topic as well | 15:19 |
| rpittau | probably both, yeah | 15:19 |
| iurygregory | JayF, I like the idea of a new implementation ++ | 15:19 |
| dtantsur | TheJulia: whether we have something to discuss or not will be more obvious after the folks read the spec and comment on it :) | 15:19 |
| dtantsur | it could be a "Just Do It" case in the end | 15:19 |
| TheJulia | Indeed, very true | 15:19 |
| TheJulia | But we shouldn't short change our time during the ptg | 15:20 |
| TheJulia | we historically try to squeeze too much into too little time | 15:20 |
| JayF | The best way to fix that is to do more async work ahead of time | 15:20 |
| JayF | by reviewing and iterating on the specs in the month+ we have before the ptg | 15:20 |
| TheJulia | indeed, just trying to make sure we don't shoot ourselves in the foot time wise | 15:21 |
| TheJulia | So we *have* the time if we need it | 15:21 |
| rpittau | problem is currently I think we're more focused on closing and reviewing urgent stuff for the release, so mayb we won't have enough time to proper review new specs | 15:21 |
| TheJulia | This is often the case as well | 15:22 |
| TheJulia | No matter how much we tell ourselves we'll do better, none of us have crystal balls to tell us what the next week entirely holds | 15:22 |
| TheJulia | unless we have plane tickets | 15:22 |
| TheJulia | :) | 15:22 |
| rpittau | I mean, of course if we have the time iterating directly on the specs is ideal before the discussion, but please consider we're at the end of the cycle | 15:22 |
| rpittau | alright, anything else to discuss? | 15:24 |
| rpittau | okey-dokey ! | 15:25 |
| rpittau | #topic Bug Deputy Updates | 15:25 |
| rpittau | cid masghar anything to report ? | 15:25 |
| TheJulia | Just an FYI, the week of October 8th, I'll be traveling to the EU. The week after, I'm planning at being at OpenInfra Days NA | 15:25 |
| rpittau | TheJulia: ack :) | 15:25 |
| cid | rpittau: Nothing at this time. | 15:25 |
| masghar | In the first 3 days of the week, no new bugs | 15:26 |
| rpittau | cool :) | 15:26 |
| rpittau | I guess we need bug deputy for the next 2-3 weeks ! | 15:26 |
| JayF | It's pretty important this time around too, since we'll want to quash any we can before final release | 15:27 |
| rpittau | considering that we're close to final release I invite eveeryone to keep and eye on bugs | 15:27 |
| JayF | cid: if you can, would you mind keeping an eye on this? I suspect cores will be very busy in coming weeks | 15:27 |
| cid | Erm, I don't mind! | 15:27 |
| cid | Sure | 15:27 |
| iurygregory | I will only be back on Sep 24th o/ | 15:27 |
| masghar | I can help around as well, if you dont mind, cid | 15:27 |
| rpittau | cid masghar that sounds great :) | 15:28 |
| rpittau | thanks both | 15:28 |
| cid | masghar: Great! | 15:28 |
| rpittau | alrighty, we don't have new RFEs to review | 15:29 |
| rpittau | last chance for discussion topics! | 15:29 |
| rpittau | 3 | 15:30 |
| rpittau | 2 | 15:30 |
| rpittau | 1 | 15:30 |
| rpittau | thanks everyone! | 15:30 |
| rpittau | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Mon Aug 26 15:30:07 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-08-26-15.00.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-08-26-15.00.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-08-26-15.00.log.html | 15:30 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 15:30 |
| rpittau | iurygregory, cid: updated the cycle highlights, thanks | 15:37 |
| opendevreview | Verification of a change to openstack/ironic master failed: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 15:38 |
| rpittau | bye everyone, see you tomorrow! o/ | 15:41 |
| masghar | o\ | 15:41 |
| cardoe | The failure on https://review.opendev.org/c/openstack/ironic/+/926685 seems unrelated to the change. | 16:00 |
| TheJulia | I would just recheck it then | 16:01 |
| TheJulia | but also dig into the failure to see if it is something else going on | 16:01 |
| opendevreview | Merged openstack/ironic stable/2024.1: [CI][stable only] fix zuul config https://review.opendev.org/c/openstack/ironic/+/926636 | 16:19 |
| opendevreview | Merged openstack/ironic master: Fix versions in release notes https://review.opendev.org/c/openstack/ironic/+/926991 | 16:20 |
| opendevreview | Merged openstack/ironic master: Clarify how to configure for non-master branch https://review.opendev.org/c/openstack/ironic/+/926939 | 16:20 |
| cardoe | Unfortunately I don't know tempest enough to really figure out what happened. It rebooted the box and it expected it to ping at 172.24.5.11 but it didn't. 172.24.5.0/24 was on br-ex | 16:38 |
| cardoe | I dunno how (or if I have the perms) to issue a recheck. | 16:39 |
| TheJulia | cardoe: you just add a comment with "recheck" followed by a reason, for example "recheck failure unrelated" | 16:40 |
| TheJulia | got a link to the failed job? | 16:41 |
| cardoe | https://zuul.opendev.org/t/openstack/build/a2750ce3e78e42b18b8e8122edcf249a | 16:41 |
| JayF | If you see that from the job output, usually my next step is to check conductor and ironic-bm-log logs to see what failed | 16:42 |
| JayF | if it's a pxe boot that failed, sigh deeply and go verify the neutron-dhcp-agent's (q-dhcp) dnsmasq crashed at the moment of truth | 16:43 |
| cardoe | Just trying to help clear ironic-week-prio and whatever might be ready to merge while I had some free cycles this morning. | 16:43 |
| cardoe | There's a few things tagged ironic-week-prio that have conflicts and haven't been touched since May | 16:44 |
| JayF | in those cases, I usually rebase them if they look relevant still, or drop the hashtag and put a comment on it | 16:49 |
| JayF | I'm not sure if non-cores can or should do that, but that's generally how I adjudicate in that case | 16:49 |
| cardoe | I see... | 17:11 |
| cardoe | Aug 26 14:57:43.183134 np0038281099 ironic-conductor[88620]: WARNING ironic.dhcp.neutron [None req-56a309d0-b65d-40cc-83fc-1295541d6fa8 None None] No VIFs found for node 06244832-12d8-4f36-9f02-e873bff9cc27 when attempting to get IP address for port: 09d1c2d6-1df7-4d95-9d33-49431115166b. | 17:12 |
| JayF | I suspect, given nova is planning to deprecate AMI, it would be wise for us to make a docs change and a mention in release notes that IPA ramdisks should not be uploaded as aki/ari | 17:14 |
| TheJulia | JayF: likely, yes | 17:15 |
| TheJulia | btw, I used bare/raw recently and it worked, so if we haven't already, we should change devstack | 17:15 |
| JayF | ack | 17:15 |
| JayF | I'll bug it real quick | 17:16 |
| TheJulia | there is already one in launchpad | 17:16 |
| JayF | https://bugs.launchpad.net/ironic/+bug/2074090 | 17:16 |
| * JayF will make sure 2074090 gets done | 17:18 | |
| opendevreview | Pierre Riteau proposed openstack/tenks master: Bump minimum Python and Ansible versions https://review.opendev.org/c/openstack/tenks/+/921620 | 17:59 |
| opendevreview | Pierre Riteau proposed openstack/tenks master: Bump minimum Python and Ansible versions https://review.opendev.org/c/openstack/tenks/+/921620 | 18:01 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Deduplicate and remove invalid information for steps https://review.opendev.org/c/openstack/ironic/+/919779 | 18:11 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Deduplicate and remove invalid information for steps https://review.opendev.org/c/openstack/ironic/+/919779 | 18:23 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Deduplicate and remove invalid information for steps https://review.opendev.org/c/openstack/ironic/+/919779 | 18:23 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Deduplicate and remove invalid information for steps https://review.opendev.org/c/openstack/ironic/+/919779 | 18:24 |
| opendevreview | Verification of a change to openstack/ironic master failed: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 18:26 |
| cardoe | JayF: so for the iDRAC, it's bios, power, and vendor that are identical to the redfish | 18:46 |
| JayF | It seems to me like your change should potentially me deprecating those noop interfaces in favor of the redfish versions, yeah? | 18:48 |
| JayF | I'm not 100% sure to be clear, just where my head is at | 18:49 |
| cardoe | Yeah I don't disagree. I saw this change https://review.opendev.org/c/openstack/ironic/+/902107 originally which is what inspired me to do it that way. | 18:53 |
| cardoe | All I was trying to originally fix is the fact that idrac inherits from GenericHardware and then you have to keep wrapping and adding each Redfish interface in. | 18:54 |
| JayF | yeah | 18:54 |
| cardoe | Which seems silly when idrac is really redfish but with some special Dell sauce for a handful of interfaces. | 18:54 |
| JayF | let me put it this way, I think I would prefer "all" or "nothing" | 18:54 |
| cardoe | So there's still "idrac-redfish" where it makes sense. | 18:55 |
| JayF | and right now this seems to be in the middle: we change to prefer the redfish-proper class in one case but not all, yeah? | 18:55 |
| * JayF suspects there's a chance he's missing a critical contextual piece | 18:55 | |
| cardoe | In the current idrac code? No. | 18:55 |
| cardoe | So right now every time I've poked you guys about idrac missing something. (e.g. redfish-https). Someone goes in and copies the bit from redfish and calls it "idrac-redfish-https" | 18:56 |
| JayF | Let me ask the most direct question to answer: other than making more redfish-based interfaces available for idrac-redfish, are there any other changes? (including changes to default interfaces) | 18:56 |
| JayF | if the only change is the list now has "n" more entries appended to the end, +2 as is | 18:56 |
| JayF | if we're shuffling priorities, that's operator-facing and we need to do all of it we need to do in one fell swoop | 18:56 |
| cardoe | The only other change is that "redfish-https" exists instead of us adding "idrac-redfish-https" which is the same thing as "redfish-https" | 18:56 |
| JayF | that's what I mean when I say it seems in the middle :) | 18:56 |
| cardoe | So I am shuffling the priority in Patch 7, which you reviewed. | 18:57 |
| JayF | ps 7 ordering is the same, new entries appended-only? | 18:57 |
| cardoe | Patch 3 originally kept the priority the same and just added redfish as append. | 18:57 |
| cardoe | ps 7 shuffles to prefer "redfish" > "idrac-redfish" on 3 interfaces. | 18:58 |
| JayF | yeah I don't have a -1 for shuffling, just more we need to pick a direction and go there: if we prefer !drac redfish interfaces in one place, we should everywhere and deprecate the noop versions of the other | 18:58 |
| JayF | if we don't wanna do that, we should just append imo | 18:58 |
| cardoe | So Dmitry pointed me to https://review.opendev.org/c/openstack/ironic/+/902107 | 18:58 |
| cardoe | Which shuffled priority and removed "idrac-redfish". | 18:59 |
| JayF | and is deprecating the noop classes | 18:59 |
| cardoe | So I'm just trying to come up with a compromise that existing users aren't broken | 18:59 |
| JayF | this is closer to what I was asking for, but they both go halfway afaict | 18:59 |
| cardoe | Well if this is good I'll add a release note and make some words about deprecated. | 19:00 |
| JayF | that Supported=False from dmitry's change | 19:00 |
| JayF | is how you mark as deprecated in code | 19:00 |
| JayF | so you'd mark supported=false in all those places, and a release note saying "they are deprecated but changing is a noop" | 19:00 |
| JayF | and at some future point when we remove it, we'd add a data migration to migrate nodes using the old interface to the new one | 19:01 |
| JayF | I just don't want our DRAC-using operators to have changes to bring that driver more in line trickled in, we need to hit it all in one release if we can :) | 19:01 |
| cardoe | Yeah that's my goal too. | 19:05 |
| cardoe | I figured since there's a change this release wrt wsman in it. People will have to touch idrac this release. | 19:05 |
| opendevreview | cid proposed openstack/ironic master: Replace reference to ari/aki https://review.opendev.org/c/openstack/ironic/+/927202 | 19:13 |
| cardoe | JayF: should I take "idrac-redfish" out of the supported interfaces doc for bios, power and vendor? Or leave it in the doc for now and just mention deprecated below? | 19:14 |
| JayF | if you remove it, it will break existing installs | 19:16 |
| JayF | so you want to leave it as supported and deprecate it | 19:16 |
| JayF | next cycle we could remove them entirely and write an online data migration to ensure opers aren't broken | 19:16 |
| JayF | I'll note, you wanted to get something in by D, you could just do the append-only change now, we land it | 19:17 |
| JayF | you can go less scope for now if you want, I'd just prefer us not change ordering to later deprecate something out of the order (appending is a non-visible change) | 19:17 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 19:29 |
| cardoe | There's my release note. | 19:29 |
| JayF | +2 | 19:35 |
| cardoe | Hang on. I'll do what you last said and won't change the ordering okay? | 19:39 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 19:40 |
| cardoe | Now that just makes "redfish" as available in the list. | 19:40 |
| cid | o/ | 19:55 |
| * cid good night ironic | 19:55 | |
| JayF | \o | 19:58 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: idrac: inherit driver interface from redfish https://review.opendev.org/c/openstack/ironic/+/926227 | 20:12 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 20:41 |
| opendevreview | Graeme Moss proposed openstack/networking-generic-switch master: Add Supermicro switches to allow for supported write config https://review.opendev.org/c/openstack/networking-generic-switch/+/926886 | 20:44 |
| opendevreview | Merged openstack/ironic master: Update network_data.json to include dns nameservers https://review.opendev.org/c/openstack/ironic/+/926685 | 21:05 |
| cardoe | JayF: I suck and had a typo. But now it passes tests. | 22:08 |
| JayF | don't worry, you're at least 2x better than AI and according to 10/10 CEOs that's good enough for software dev ;) | 22:09 |
| JayF | restored +2 | 22:09 |
| cardoe | lol. Thanks. | 22:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!