| vanou | good morning ironic | 02:40 |
|---|---|---|
| TheJulia | Good morning from UTC-2 | 02:45 |
| TheJulia | Err, +2 | 02:55 |
| * TheJulia tries to get more sleep | 02:55 | |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Change boot_interface order of iRMC driver https://review.opendev.org/c/openstack/ironic/+/862616 | 02:56 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Align iRMC driver with Ironic's default boot_mode https://review.opendev.org/c/openstack/ironic/+/862552 | 02:57 |
| janders | hey TheJulia o/ | 04:01 |
| janders | where are you off to? | 04:01 |
| janders | I sense a vibration in the Force that has a nice hops flavour in it | 04:01 |
| arne_wiebalck | Good morning janders TheJulia and Ironic! | 07:17 |
| opendevreview | Merged openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/863730 | 08:02 |
| janders | hey arne_wiebalck o/ | 08:49 |
| rpittau | good morning ironic! o/ | 09:09 |
| opendevreview | Jakob Meng proposed openstack/bifrost master: Adapt role bifrost-cloud-config to Ansible OpenStack Col. >=2.0.0 https://review.opendev.org/c/openstack/bifrost/+/863816 | 09:11 |
| dtantsur | good morning folks, good european morning TheJulia :) | 09:16 |
| TheJulia | Thanks dtantsur | 09:16 |
| TheJulia | janders: I'm in Brno this week | 09:16 |
| dtantsur | how was your trip? | 09:16 |
| TheJulia | not too bad, 27-ish hours door to door | 09:16 |
| * dtantsur has a different definition of "not too bad" :D | 09:17 | |
| TheJulia | returning... it will be brutal | 09:17 |
| dtantsur | oh? | 09:18 |
| TheJulia | I think 30-ish hours with a 7+ hour layover in Salt Lake City. And I'm absolutely going to leave the airport, get drunk go shopping, and then go back to the airport | 09:18 |
| dtantsur | makes a lot of sense | 09:18 |
| TheJulia | and that is the second day of travel | 09:19 |
| TheJulia | Train from Brno to Vienna is the day before | 09:19 |
| dtantsur | well, at least Vienna is not the worst place to stay overnight | 09:20 |
| TheJulia | I'm staying at the airport hotel, but there is the express into the city center | 09:20 |
| TheJulia | the express starts at like 5:30 AM, my flight departs at like 6:25 AM | 09:21 |
| TheJulia | JayF: so was thinking about md5.... and I looked at the code while traveling. I'm wondering if we should short circuit *All* md5 usage in the code base which would impact some of the hardware interaction. | 09:26 |
| opendevreview | Hervé Beraud proposed openstack/virtualbmc master: remove python-dev from bindep https://review.opendev.org/c/openstack/virtualbmc/+/863818 | 09:35 |
| opendevreview | Hervé Beraud proposed openstack/virtualbmc master: remove python-dev from bindep https://review.opendev.org/c/openstack/virtualbmc/+/863818 | 09:36 |
| opendevreview | Hervé Beraud proposed openstack/virtualbmc master: remove python-dev from bindep https://review.opendev.org/c/openstack/virtualbmc/+/863818 | 09:37 |
| janders | ajya thank you for subsequent review of https://review.opendev.org/c/openstack/sushy/+/856597. W/r/ https://review.opendev.org/c/openstack/sushy/+/856597/comments/c31df16d_4b60e003 Would you be able to share a copy of Settings body from the iDRAC in question? For now I will edit the Lenovo one to test the idea but if you have it and are happy to | 09:38 |
| janders | share I reckon using a genuine file would be better. | 09:38 |
| janders | s/Lenovo/Nokia - will make an easier job | 09:39 |
| vanou | TheJulia: Oh, you are so early to wake up :o Good morning! | 09:44 |
| ajya | Hi, janders , see https://paste.opendev.org/show/biVl3U0ldYReLFF0WZAT/ | 09:48 |
| janders | ajya thank you, this is great! | 09:49 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Increase server side retries https://review.opendev.org/c/openstack/sushy/+/863828 | 09:50 |
| rpittau | good old continent morning TheJulia :) | 09:51 |
| rpittau | ajya: thanks again for your help, I simply increased the retries from 5 to 7, should cover 21 secs now | 09:52 |
| ajya | rpittau: ok, then try couple of times to see if there are no exceptional peaks, otherwise increase for additional buffer. I'll see if I can get some expected times on this. | 10:02 |
| TheJulia | JayF: the conundrum is largely things such as doing checksums to call a bmc to supply the checksum it needs for an action | 10:26 |
| janders | ajya with the sample you provided, it seems to me we may need to change the logic of https://review.opendev.org/c/openstack/sushy/+/856597 a bit - this .get() will return False cause "BootSourceOverrideMode": null in the sample ( ref https://review.opendev.org/c/openstack/sushy/+/856597/20/sushy/resources/system/system.py#292 ) | 10:28 |
| janders | with the other machines there are non-null values | 10:28 |
| janders | I wonder 1) how to do this in a slick way and 2) whether this can cause any problems | 10:28 |
| janders | what are your thoughts? | 10:30 |
| ajya | janders: it's null because there are no pending values submitted, not sure what other systems display, current? Anyway, I think it is is necessary to test for existence of key, not their value - check `key in 'Boot'` | 10:36 |
| Kirill_ | Hi, i'm working on vnc console for ironic. And already implemented it in my project. This work is based on approved spec:https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/vnc-graphical-console.html. SPEC is about new graphical console interface and vnc is a part of it. I've got 2 commits. first one adds vnc console to ironic, second - creats new graphical console interface. I seperate it onto 2 parts, because it will be | 11:00 |
| Kirill_ | easier to discuss each feature (1 commit - 1 feature). For fist part(vnc in ironic) i've already created mr in gerrit: https://review.opendev.org/c/openstack/ironic/+/860689. Who can help in reviewing and ready for discussion?) | 11:00 |
| iurygregory | good morning Ironic | 11:13 |
| dtantsur | Kirill_: hi, will you stay for the meeting today? you may announce your work in the open discussion. | 11:21 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Increase server side retries https://review.opendev.org/c/openstack/sushy/+/863828 | 11:22 |
| Kirill_ | hi, when do you have a meeting today? | 11:26 |
| rpittau | Kirill_: in 3 hours | 11:29 |
| rpittau | and a half :) | 11:29 |
| Kirill_ | got it, yep i'll be there) | 11:29 |
| dtantsur | 15:00 UTC to be precise | 11:32 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Increase server side retries https://review.opendev.org/c/openstack/sushy/+/863828 | 11:51 |
| opendevreview | Riccardo Pittau proposed openstack/sushy master: Increase server side retries https://review.opendev.org/c/openstack/sushy/+/863828 | 11:52 |
| janders | ajya yeah from what I see other systems display current value (but that sample is so far limited just to Nokia machine) | 12:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 12:41 |
| opendevreview | Jakob Meng proposed openstack/bifrost master: Adapt role bifrost-cloud-config to Ansible OpenStack Col. >=2.0.0 https://review.opendev.org/c/openstack/bifrost/+/863816 | 12:41 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Fix setting boot related attributes https://review.opendev.org/c/openstack/sushy/+/856597 | 12:45 |
| janders | ^ ajya I think I addressed all the comments now - when you have time let me know what you think. Thank you for continuing oversight - very much appreciated and your perspective is invaluable! | 12:46 |
| ajya | janders: thanks, will take a look | 12:47 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 12:52 |
| timeu | janders: I am wondering how your patch is related to the issue I observed on recent Lenovo SD530 firmware: https://storyboard.openstack.org/#!/story/2010161. I submitted a patch (that I need to revise) to expose the BootSourceOverrideEnabled@Redfish.AllowedValues so that ironic can handle the edge case where Continious is not supported | 12:55 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 13:00 |
| dtantsur | rpittau: trying something here ^^ | 13:01 |
| janders | timeu from a quick glance I don't think they are strictly related, but they may complement each other. My patch revises logic introduced in https://review.opendev.org/c/openstack/sushy/+/840653/1/sushy/resources/system/system.py and expands it so that we handle a wider range of hardware using SettingsURI (the initial patch was very tailored to | 13:05 |
| janders | Nokias and may have introduced problems on some Lenovos) | 13:05 |
| janders | the exact issue I'm trying to fix manifests itself with attempts to PATCH a */Pending URL (which is the SettingsURI in the System) and failing out with a 400 as this SettingsURI is not writeable | 13:06 |
| janders | (at least not writable for the parameter we're trying too set) | 13:06 |
| janders | does this explanation help? | 13:06 |
| timeu | yes thanks for the explanation. I will try to finish my patch this week and maybe test it together with yours on our Lenovo hardware | 13:07 |
| timeu | and see how it behaves | 13:07 |
| janders | timeu great, sounds good. I'm wrapping up for the night (2307hrs here in AUS) but will check messages later | 13:07 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 13:21 |
| opendevreview | Merged openstack/bifrost master: Remove pymysql from system dependencies https://review.opendev.org/c/openstack/bifrost/+/861537 | 13:23 |
| opendevreview | Merged openstack/bifrost master: Install passlib to venv (htpasswd) https://review.opendev.org/c/openstack/bifrost/+/861536 | 13:23 |
| rpittau | dtantsur: that looks promising, probably need to run ansible with sudo | 13:43 |
| dtantsur | nah, it was about the apt command | 13:43 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 13:43 |
| rpittau | ag yeah, just that should work | 13:44 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 14:06 |
| opendevreview | Merged openstack/networking-baremetal master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/networking-baremetal/+/857314 | 14:17 |
| *** tzumainn_ is now known as tzumainn | 14:47 | |
| JayF | The meeting is in 11 minutes, isn't it? | 14:49 |
| * JayF getting DST'd | 14:49 | |
| JayF | TheJulia: I mean, that sounds important, I'd be careful not to mix scopes. I think we can land the image-based stuff separate, and then play w/hardware | 14:49 |
| JayF | #startmeeting Ironic | 15:00 |
| opendevmeet | Meeting started Mon Nov 7 15:00:44 2022 UTC and is due to finish in 60 minutes. The chair is JayF. 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 |
| JayF | Good morning; expecially early morning to those of us in the US after DST | 15:00 |
| rpittau | o/ | 15:01 |
| ajya | o/ | 15:01 |
| ebbex | o/ | 15:01 |
| matfechner | o/ | 15:01 |
| Kirill_ | o/ | 15:01 |
| rloo | o/ | 15:01 |
| JayF | #topic Announcements/Reminders | 15:01 |
| JayF | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:01 |
| iurygregory | o/ | 15:01 |
| JayF | Reminder to contributors and reviewers, please review things hashtagged #ironic-week-prio and mark patches you need reviewed with that hashtag | 15:02 |
| JayF | #note Announcement: Be aware that almost all upstream meetings are UTC-based, US-based contributors may wanna check their calendar. | 15:02 |
| JayF | Anyone else with an announcement or reminder before we move on? | 15:03 |
| JayF | #topic Review action items from last meeting | 15:03 |
| arne_wiebalck | o/ | 15:04 |
| JayF | Looks like dtantsur arne_wiebalck and I were looking at BM SIG quarterly meeting ideas; we need to continue that and get something done-done soon | 15:04 |
| arne_wiebalck | JayF: ack | 15:04 |
| JayF | That's the only action item from last meeting; I reopened that etherpad and will stare at it some when we're done :) | 15:05 |
| JayF | #topic Review Ironic CI Status & update whiteboard if needed | 15:05 |
| JayF | TheJulia and I have been working, generally, on trying to get older CI things working | 15:05 |
| JayF | I specifically noted here that there's a hard failure for stable/wallaby ironic-prometheus-exporter that appears based in ansible version | 15:05 |
| JayF | This is problematic because I think that job uses the ansible zuul uses to setup the job | 15:07 |
| JayF | We're going to end up having to either refactor or remove a large number of jobs (and maybe even branch support on some of these) in order to get CI freed | 15:07 |
| JayF | What do folks think of retiring just some repositories older stable branches? e.g. in this case, ironic-prometheus-exporter. We've never backported anything of substance to that repo, stable/wallaby branch, for instance | 15:08 |
| iurygregory | JayF, today I can check the ironic prometheus exporter CI | 15:10 |
| JayF | I mean, I think my question is still worth consideration: is it even worth it | 15:10 |
| JayF | if we haven't backported stuff to this repo; it seems like a bad use of anyones' time to fix it rather than retiring it | 15:10 |
| JayF | There's clearly minimal engagement about this right now, I'll take an action to email the list about it | 15:13 |
| iurygregory | I can help without problems to fix it | 15:13 |
| JayF | I mean, I appreciate the help I just question the value | 15:13 |
| iurygregory | not sure what other folks think about | 15:13 |
| JayF | I'd rather you be making other things work better than keeping CI alive for the hell of it on branches we literally have never backported something to in 3 years | 15:13 |
| JayF | but either way, the status is out there, I'm going to move on | 15:14 |
| JayF | #topic 2023.1 workstreams in progress | 15:14 |
| JayF | Reminder we're keeping track of status for those who want to here: https://etherpad.opendev.org/p/IronicWorkstreams2023.1 | 15:14 |
| JayF | Also a reminder, please review the workstreams doc here: https://review.opendev.org/c/openstack/ironic-specs/+/862538 -- we should try to get this landed in the next week or two | 15:15 |
| JayF | Any questions about those before we move on? | 15:16 |
| JayF | There is nothing for RFE Review this week; going to skip that. | 15:16 |
| JayF | #topic Open Discussion | 15:16 |
| JayF | I believe Kirill_ had something they wanted to mention about console support? | 15:16 |
| Kirill_ | yep | 15:17 |
| Kirill_ | HI | 15:17 |
| Kirill_ | i've added vnc console to ironic | 15:17 |
| Kirill_ | #link https://review.opendev.org/c/openstack/ironic/+/860689 | 15:17 |
| Kirill_ | it is a first stap to new graphical console interface | 15:18 |
| JayF | Nice work; there's a couple of things -- if you don't mind, can you document you're working on this feature by opening an issue on https://storyboard.openstack.org | 15:18 |
| Kirill_ | it also requires some changes on nova side. but lets focus on ironic) | 15:18 |
| JayF | For those patches; I'll take a look at them, but I can tell you right now you'll likely also need to add more detailed commit messages (see: https://wiki.openstack.org/wiki/GitCommitMessages) and a release note | 15:19 |
| JayF | We have some contributors on weird schedules this week; but I'll try to make sure you get some feedback this week. | 15:19 |
| JayF | Kirill_: Is there a particular reason you came to contribute this? It's exciting to get a new person involved! | 15:20 |
| Kirill_ | we have this spec: https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/vnc-graphical-console.html | 15:20 |
| Kirill_ | i've done it for my company | 15:20 |
| JayF | Oh, perfect. | 15:20 |
| Kirill_ | also new graphical console interface is done on my side | 15:20 |
| JayF | Can you just put a link to that bug in the commit message? | 15:20 |
| JayF | We usually do storyboard stuff now, but we're migrating back | 15:21 |
| JayF | so the launchpad bug will be perfect | 15:21 |
| Kirill_ | sure, i'll add a link to massage | 15:21 |
| JayF | Thanks, and good work o/. Glad that you're having a chance to contribute. If there's anything you get stuck on or have questions about feel free to ask. | 15:22 |
| JayF | Is there anything else for Open Discussion today? "Have a free feature" is a hard act to follow :D | 15:22 |
| Kirill_ | so to be on a same page:i need to add link to spec, and wait for your review? | 15:22 |
| JayF | Just indicate it belongs to that bug should be good enough to get it linked to the spec | 15:23 |
| JayF | you'll need a release note for it at some point too | 15:23 |
| JayF | but you can wait for a real-review on the PR to worry about either of those things | 15:23 |
| JayF | I'm making a note to review it today; if I don't get to it I'll do it tomorrow during my office hours | 15:23 |
| Kirill_ | got it. Thanks | 15:24 |
| Kirill_ | also ping me for any questions) | 15:25 |
| JayF | Last call for more items for open discussion before closing out the meeting | 15:25 |
| JayF | I'll be here next week to run the next meeting. | 15:27 |
| JayF | Thanks everyone o/ | 15:27 |
| JayF | #endmeeting Ironic | 15:27 |
| opendevmeet | Meeting ended Mon Nov 7 15:27:47 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-11-07-15.00.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-11-07-15.00.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-11-07-15.00.log.html | 15:27 |
| opendevreview | Kirill proposed openstack/ironic master: add vnc console https://review.opendev.org/c/openstack/ironic/+/860689 | 15:50 |
| opendevreview | Kirill proposed openstack/ironic master: add vnc console https://review.opendev.org/c/openstack/ironic/+/860689 | 15:50 |
| dtantsur | JFYI JayF, I'm working on an early prototype for a metal3 job in https://review.opendev.org/c/openstack/ironic/+/863873, while rpittau is figuring building Ironic from source in it | 16:42 |
| JayF | nice; if there's anything specific I can help with on that effort lmk | 16:42 |
| dtantsur | I will, but so far I just need to figure out a lot of glueing parts | 16:42 |
| ashinclouds[m] | Awesome | 16:44 |
| dtantsur | Waiting for deployment "baremetal-operator-controller-manager" rollout to finish: 0 of 1 updated replicas are available... | 16:45 |
| dtantsur | meh | 16:45 |
| dtantsur | Y U NO ROLLOUT | 16:47 |
| rpittau | bye! o/ | 17:17 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 17:26 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 17:44 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 17:57 |
| opendevreview | Merged openstack/bifrost master: Adapt role bifrost-cloud-config to Ansible OpenStack Col. >=2.0.0 https://review.opendev.org/c/openstack/bifrost/+/863816 | 18:19 |
| *** tzumainn_ is now known as tzumainn | 20:01 | |
| JayF | FYI: I put up https://review.opendev.org/c/openstack/project-config/+/863931 to enable cores for all ironic projects to be able to toggle WIP state on a PR | 20:42 |
| JayF | I think this is relatively non-controversial change; but at least one or two other ironic folks looking at it would make me feel better :) | 20:43 |
| JayF | iurygregory: dtantsur: TheJulia, others ^ | 20:48 |
| opendevreview | Ebbex proposed openstack/bifrost master: Use a more traditional ansible approach to include_vars https://review.opendev.org/c/openstack/bifrost/+/855806 | 21:28 |
| opendevreview | Ebbex proposed openstack/bifrost master: Refactor use of include_vars https://review.opendev.org/c/openstack/bifrost/+/855807 | 21:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!