*** anuradha1904 has quit IRC | 00:07 | |
*** akiselev has joined #openstack-ironic | 00:16 | |
*** w14161_ has joined #openstack-ironic | 00:17 | |
openstackgerrit | Merged openstack/ironic master: Allow passing rootfs_uuid for the standalone case https://review.opendev.org/758970 | 00:18 |
---|---|---|
*** w14161 has quit IRC | 00:20 | |
TheJulia | trandles: \o/ | 00:24 |
TheJulia | trandles: interesting considering power status, at least in upstream code points directly to chassis power status | 00:25 |
*** gyee has quit IRC | 00:44 | |
openstackgerrit | likui proposed openstack/ironic-python-agent master: update lower-constraints.txt https://review.opendev.org/759238 | 00:49 |
openstackgerrit | Eric Xie proposed openstack/networking-generic-switch master: Limit netmiko not to use version 3.3.2 https://review.opendev.org/759336 | 01:03 |
*** Goneri has quit IRC | 01:27 | |
*** jawad_axd has joined #openstack-ironic | 01:54 | |
*** jawad_axd has quit IRC | 01:58 | |
*** Qianbiao has joined #openstack-ironic | 02:01 | |
*** jawad_axd has joined #openstack-ironic | 02:15 | |
*** jawad_axd has quit IRC | 02:20 | |
*** jawad_axd has joined #openstack-ironic | 02:35 | |
*** jawad_axd has quit IRC | 02:40 | |
*** rcernin has quit IRC | 02:51 | |
*** jawad_axd has joined #openstack-ironic | 02:56 | |
*** jawad_axd has quit IRC | 03:01 | |
*** jawad_axd has joined #openstack-ironic | 03:17 | |
*** rcernin has joined #openstack-ironic | 03:19 | |
*** rcernin has quit IRC | 03:19 | |
*** rcernin has joined #openstack-ironic | 03:19 | |
*** jawad_axd has quit IRC | 03:21 | |
*** jawad_axd has joined #openstack-ironic | 03:38 | |
*** jawad_axd has quit IRC | 03:42 | |
*** ricolin has joined #openstack-ironic | 03:43 | |
*** jawad_axd has joined #openstack-ironic | 03:58 | |
*** jawad_axd has quit IRC | 04:03 | |
*** pmannidi has joined #openstack-ironic | 04:26 | |
*** sai438 has quit IRC | 04:28 | |
*** jawad_axd has joined #openstack-ironic | 05:00 | |
*** jawad_axd has quit IRC | 05:05 | |
*** bhagyashris is now known as bhagyashris|sick | 05:09 | |
*** happy-contrib-43 has quit IRC | 05:10 | |
*** tzumainn has quit IRC | 05:26 | |
openstackgerrit | Kafilat Adeleke proposed openstack/sushy master: Adds sushy library overview https://review.opendev.org/757907 | 06:13 |
*** yolanda has quit IRC | 06:14 | |
*** smutugrigg has joined #openstack-ironic | 06:19 | |
smutugrigg | hi ironic o/ | 06:19 |
smutugrigg | hi urygregory, I was wondering if I could ask for some assistance with my Outreachy application please? | 06:20 |
smutugrigg | sorry iurygregory | 06:20 |
*** rpittau|afk is now known as rpittau | 06:21 | |
rpittau | good morning ironic! o/ | 06:21 |
rpittau | Happy Friday! | 06:21 |
*** Qianbiao has quit IRC | 06:32 | |
*** jawad_axd has joined #openstack-ironic | 06:40 | |
*** jtomasek has joined #openstack-ironic | 06:41 | |
arne_wiebalck | Good morning, ironic! | 06:48 |
rpittau | hey arne_wiebalck :) | 06:54 |
arne_wiebalck | hey rpittau, good morning! | 06:54 |
janders | good morning rpittau o/ | 06:55 |
janders | Happy Friday! :) | 06:55 |
janders | hey arne_wiebalck | 06:55 |
arne_wiebalck | hey janders o/ | 06:55 |
janders | arne_wiebalck thanks for good questions at yesterday's session! | 06:56 |
arne_wiebalck | janders: did anyone answer already ... I haven't checked yet tbh :) | 06:57 |
*** Qianbiao has joined #openstack-ironic | 06:58 | |
rpittau | hey janders :) | 06:59 |
janders | arne_wiebalck we were answering in near-realtime | 07:00 |
janders | arne_wiebalck will go check if the answers got properly posted | 07:00 |
arne_wiebalck | oh, rly, I had the chat open for 15mins or so but did not see any updated | 07:01 |
arne_wiebalck | updates | 07:01 |
arne_wiebalck | but I remember now someone telling me it sometimes requires to reload the page :-S | 07:01 |
janders | yeah we had to keep doing it to see the questions coming in | 07:02 |
janders | insane | 07:02 |
janders | it looks like some of our answers are still there - but some (by Tim) disappeared (!) | 07:04 |
*** fgofurov has joined #openstack-ironic | 07:04 | |
janders | actually I take this back | 07:04 |
janders | just reshuffled, not disappeared | 07:04 |
*** hoonetorg has quit IRC | 07:07 | |
*** hoonetorg has joined #openstack-ironic | 07:20 | |
iurygregory | good morning Ironic | 07:24 |
iurygregory | smutugrigg, sure | 07:24 |
janders | good morning iurygregory o/ | 07:25 |
iurygregory | hey janders o/ | 07:25 |
rpittau | hey iurygregory :) | 07:26 |
ajya | Good morning, ironic | 07:26 |
iurygregory | rpittau, o/ | 07:26 |
rpittau | hey ajya :) | 07:26 |
iurygregory | hey ajya o/ | 07:26 |
ajya | janders: can you take a look at this story - https://storyboard.openstack.org/#!/story/2004519 I left a comment there | 07:26 |
ajya | hi, rpittau, iurygregory | 07:27 |
rpittau | I thought there were more sessions of open infra today :/ | 07:27 |
iurygregory | same | 07:27 |
rpittau | anyone and everyone, if you have a minute, we really need this -> https://review.opendev.org/757808 | 07:29 |
patchbot | patch 757808 - ironic-python-agent-builder - Use centos element for dib on focal - 2 patch sets | 07:29 |
iurygregory | I thought I had +2 things for ipa-b for focal at least one patch you mentioned =) | 07:30 |
iurygregory | I will look after one more coffee =D | 07:31 |
rpittau | it was probably the one that depends on that | 07:32 |
* iurygregory facepalm hahah | 07:32 | |
openstackgerrit | Emmanouil Bagakis proposed openstack/ironic-python-agent master: Updated Implementation of string interpolation delay on LOG messages https://review.opendev.org/759203 | 07:32 |
iurygregory | rpittau, done | 07:36 |
rpittau | thanks iurygregory :) | 07:36 |
iurygregory | now we just need another core for https://review.opendev.org/#/c/757812/ | 07:36 |
*** tosky has joined #openstack-ironic | 07:36 | |
patchbot | patch 757812 - ironic-python-agent - Run dib ipa src jobs on ubuntu focal - 5 patch sets | 07:36 |
Qianbiao | Hi, may noline give a review on https://review.opendev.org/#/c/752006/ https://review.opendev.org/757606 https://review.opendev.org/757607 | 07:40 |
patchbot | patch 752006 - ironic (stable/ussuri) - opt: Enhance old stable branches to use latest pyt... - 8 patch sets | 07:40 |
patchbot | patch 757606 - ironic (stable/train) - opt: Enhance old stable branches to use latest pyt... - 2 patch sets | 07:40 |
patchbot | patch 757607 - ironic (stable/stein) - opt: Enhance old stable branches to use latest pyt... - 3 patch sets | 07:40 |
*** jtomasek has quit IRC | 07:43 | |
rpittau | it seems we finished the audit \o/ | 07:46 |
iurygregory | beer time | 07:47 |
iurygregory | \o/ | 07:47 |
rpittau | oh I miss beer breakfast | 07:47 |
iurygregory | hahaha this would be normal in CZ XD | 07:47 |
rpittau | :) | 07:48 |
smutugrigg | hi iurygregory, sorry was called away | 07:51 |
iurygregory | smutugrigg, np | 07:51 |
smutugrigg | I am trying to complete my final application and need to understnad if there are additional questions that Ironic would like me to answer? | 07:51 |
iurygregory | I know you need to record your contribution in the website to be eligible | 07:52 |
smutugrigg | Yes I have completed that step | 07:53 |
smutugrigg | the application is now asking if there is any provide any additional information that ironic would like me to probide as part of the application | 07:53 |
*** anuradha1904 has joined #openstack-ironic | 07:54 | |
smutugrigg | It has also asked me to build a Outreachy internship project timeline with a nominated mentor =) | 07:54 |
janders | wrapping up for the day, have a great weekend everyone! o/ | 07:56 |
arne_wiebalck | janders: yes, all answers are there ... seems like I simply missed to refresh the page! | 07:56 |
iurygregory | smutugrigg, let me look here | 07:56 |
janders | arne_wiebalck I really dont think you should need to | 07:56 |
smutugrigg | thanks | 07:56 |
arne_wiebalck | janders: sorry about this, and thanks for the answers! | 07:56 |
janders | arne_wiebalck platform FAIL! | 07:57 |
janders | arne_wiebalck you're most welcome! | 07:57 |
arne_wiebalck | janders: yeah! | 07:57 |
iurygregory | smutugrigg, I would mention how is your skill level for the technologies the project will use, this would help the reviewers | 08:00 |
*** DjeufackZane has joined #openstack-ironic | 08:00 | |
smutugrigg | thanks iurygregory, do I need to create a timeline or is that something to work on if selected? | 08:02 |
iurygregory | I don't think it would be necessary | 08:03 |
iurygregory | I think the projects have already in ther description some timeline idea | 08:04 |
*** jtomasek has joined #openstack-ironic | 08:05 | |
smutugrigg | thank iurygregory, then I might reference the timelines from the documenation provided if that is OK with you? | 08:05 |
iurygregory | yes | 08:05 |
smutugrigg | Great thanks heaps iurygregory, as always, very much appreciated. | 08:06 |
smutugrigg | night ironic o/ | 08:14 |
*** smutugrigg has quit IRC | 08:16 | |
*** jtomasek has quit IRC | 08:19 | |
*** Qianbiao has quit IRC | 08:30 | |
openstackgerrit | huwenhui proposed openstack/ironic master: Fix the bug for updating the dhcp-ops of tenant ports unnecessarily with multiable tenant networks. https://review.opendev.org/759372 | 08:30 |
*** k_mouza has joined #openstack-ironic | 08:31 | |
*** derekh has joined #openstack-ironic | 08:37 | |
*** jawad_axd has quit IRC | 08:37 | |
*** jawad_axd has joined #openstack-ironic | 08:38 | |
anuradha1904 | Hello stevebaker, rpittau: What changes do I need to make in the following https://review.opendev.org/#/c/756682/8/ironic/common/driver_factory.py@131 patch? I am bit confused. Is there anything else to add? | 08:50 |
patchbot | patch 756682 - ironic - Changes to add exception in "default_interface()" ... - 9 patch sets | 08:50 |
*** Qianbiao has joined #openstack-ironic | 08:53 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent-builder failed: Use centos element for dib on focal https://review.opendev.org/757808 | 09:01 |
*** jawad_ax_ has joined #openstack-ironic | 09:13 | |
*** jawad_axd has quit IRC | 09:15 | |
openstackgerrit | Merged openstack/ironic master: Use centos as base element for dib images https://review.opendev.org/757811 | 09:16 |
*** ociuhandu has joined #openstack-ironic | 09:19 | |
arne_wiebalck | I just learnt the hard way that the APIs cache the nodes' conductor affinity ... is there an option to control this (enable/disable, TTL, ...)? | 09:24 |
arne_wiebalck | This is sth to be aware of when you replace controller nodes without restarting all API s. | 09:25 |
*** DjeufackZane has quit IRC | 09:28 | |
iurygregory | arne_wiebalck, ouch! | 09:34 |
arne_wiebalck | It *seems* that nova removes the resource provider if this happens, so it loses memory that this node exists. #bad | 09:41 |
*** ociuhandu has quit IRC | 10:08 | |
rpittau | mmm arne_wiebalck I think that deserves at least a story in storyboard if you have the time | 10:08 |
arne_wiebalck | rpittau: an RFE to make it controllable? | 10:09 |
rpittau | well I was thinking a "report" of the issue and yes, maybe an RFE could be a follow up, or just mix all together | 10:10 |
arne_wiebalck | rpittau: maybe there should also be sth in the admin docs: "Replacing conductor nodes" | 10:10 |
rpittau | arne_wiebalck: yep, that would be nice | 10:10 |
iurygregory | arne_wiebalck, ++ | 10:15 |
arne_wiebalck | https://storyboard.openstack.org/#!/story/2008277 | 10:19 |
rpittau | great, thanks! | 10:20 |
*** k_mouza has quit IRC | 10:23 | |
*** k_mouza has joined #openstack-ironic | 10:23 | |
*** dtantsur|afk is now known as dtantsur | 10:39 | |
dtantsur | morning/afternoon ironic | 10:41 |
dtantsur | wow so much happened in the channel over night | 10:41 |
dtantsur | TheJulia: re tripleo, flat networks and ironic https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/features/baremetal_overcloud.html is what I wrote long ago | 10:42 |
*** k_mouza has quit IRC | 10:43 | |
iurygregory | morning dtantsur o/ | 10:49 |
*** yolanda has joined #openstack-ironic | 10:57 | |
*** Qianbiao has quit IRC | 11:02 | |
*** k_mouza has joined #openstack-ironic | 11:13 | |
*** k_mouza has quit IRC | 11:17 | |
*** k_mouza has joined #openstack-ironic | 11:17 | |
openstackgerrit | Anuradha proposed openstack/ironic master: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python https://review.opendev.org/758762 | 11:24 |
*** Qianbiao has joined #openstack-ironic | 11:25 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 11:43 |
*** thiagop has joined #openstack-ironic | 11:55 | |
*** rh-jelabarre has joined #openstack-ironic | 11:59 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 12:13 |
rpittau | mmm CI is taking friday afternoon off | 12:16 |
arne_wiebalck | TheJulia: I promoted graphical consoles to the agenda for the PTG. | 12:28 |
dtantsur | arne_wiebalck: is there anything to discuss? | 12:33 |
dtantsur | or lemme put it a bit more direct: are you going to volunteer to do the work and need directions? :) | 12:33 |
arne_wiebalck | dtantsur: did you check the item I put on the agenda? | 12:33 |
dtantsur | arne_wiebalck: answer to the question there: yes! | 12:34 |
dtantsur | I don't mind having a discussion too much, but I personally have nothing to add | 12:34 |
rpittau | arne_wiebalck: they are what's been done so far, but there might be some work to refactor the current console as well | 12:35 |
dtantsur | mmm, the CI seems really broken now | 12:36 |
arne_wiebalck | I was thinking it would be good to confirm that the patches are the starting point, and also to discuss initial steps/directions to pick up this work. | 12:36 |
rpittau | dtantsur: yep | 12:36 |
dtantsur | sqlalchemy.exc.ArgumentError: column object 'temp_started_at' already assigned to table 'nodes' | 12:36 |
dtantsur | you're seeing the same ^^? | 12:36 |
rpittau | I'm seeing a lot of errors with mirrors and package installation in general | 12:36 |
arne_wiebalck | And to introduce ebagakis who will work on this. | 12:36 |
arne_wiebalck | It does not have to be long, more so that everyone is aware and on the same page. | 12:37 |
rpittau | arne_wiebalck: that sounds good for a quick talk | 12:37 |
dtantsur | yep | 12:37 |
arne_wiebalck | dtantsur: rpittau: ++ | 12:37 |
dtantsur | rpittau: there was an upper-constraints update https://opendev.org/openstack/requirements/commit/d8d2e848ec057c47963d33dbd44479fc28e6f38b | 12:38 |
dtantsur | it might have broken us | 12:38 |
rpittau | oh didn't see that, I was actually talking about ubuntu and tinycore packages | 12:38 |
*** Qianbiao has quit IRC | 12:38 | |
*** Qianbiao has joined #openstack-ironic | 12:39 | |
rpittau | but yeah I see that could add some entropy to what was supposed to be a quiet friday | 12:39 |
*** k_mouza has quit IRC | 12:39 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 12:44 |
*** Qianbiao has quit IRC | 12:48 | |
*** Qianbiao has joined #openstack-ironic | 12:48 | |
*** akiselev has quit IRC | 12:52 | |
TheJulia | feedback session starting shortly | 12:55 |
TheJulia | Any objections to running the ?monday? session longer? or maybe doing a second session that day? | 12:55 |
*** akiselev has joined #openstack-ironic | 12:58 | |
TheJulia | https://summit.openinfra.dev/a/event/24805 | 12:58 |
TheJulia | iurygregory: rpittau: I'm guessing from the spreadsheet that no oddities were found? | 12:59 |
rpittau | TheJulia: no objections from me, just how much longer ? :) | 12:59 |
rpittau | TheJulia: no weird things except for the twin patches | 13:00 |
TheJulia | There was an interesting thought yesterday plus the ipmi or bmc proxy idea | 13:00 |
TheJulia | rpittau: the merge ones? | 13:00 |
iurygregory | TheJulia, yeah =) | 13:00 |
TheJulia | only on the victoria right? | 13:00 |
rpittau | TheJulia: https://static.opendev.org/project/opendev.org/gerrit-diffs/openstack/python-ironicclient.git/master/0000.txt AND https://static.opendev.org/project/opendev.org/gerrit-diffs/openstack/python-ironicclient.git/master/0002.txt | 13:01 |
rpittau | master on python-ironicclient | 13:01 |
rpittau | they were verified though, there was a quick chat on #opendev about that | 13:02 |
dtantsur | TheJulia: morning. which sessions starts shortly? Oo | 13:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Fix database migrations with SQLAlchemy 1.3.20 https://review.opendev.org/759420 | 13:04 |
dtantsur | rpittau: fun ^^^ | 13:04 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 13:04 |
TheJulia | dtantsur: summit feedback session | 13:05 |
*** Qianbiao has quit IRC | 13:05 | |
rpittau | dtantsur: they changed that in a minor release ? or was it deprecated ? oO | 13:05 |
dtantsur | O_o | 13:05 |
*** Qianbiao has joined #openstack-ironic | 13:05 | |
dtantsur | rpittau: no idea, I'm about to file it | 13:05 |
rpittau | too many oO | 13:05 |
rpittau | ok | 13:05 |
rpittau | not even minor.... that's weird | 13:07 |
*** akiselev has quit IRC | 13:07 | |
iurygregory | omg that was funny | 13:07 |
iurygregory | I'm watching from the session link not the zoom link | 13:08 |
iurygregory | music time lol | 13:09 |
*** jawad_ax_ has quit IRC | 13:10 | |
*** rloo has joined #openstack-ironic | 13:10 | |
*** jawad_axd has joined #openstack-ironic | 13:11 | |
*** jawad_axd has quit IRC | 13:12 | |
iurygregory | crazy people =( bad =( | 13:12 |
*** jawad_axd has joined #openstack-ironic | 13:12 | |
*** jawad_axd has quit IRC | 13:13 | |
iurygregory | dtantsur it's the feedback session | 13:13 |
*** ianychoi_ has joined #openstack-ironic | 13:13 | |
*** jawad_axd has joined #openstack-ironic | 13:13 | |
TheJulia | so we got trolled | 13:14 |
TheJulia | becasue I tweeted the zoom link | 13:14 |
TheJulia | they are getting a new url | 13:14 |
dtantsur | sigh | 13:14 |
rpittau | :/ | 13:14 |
TheJulia | https://zoom.us/j/95495383034?pwd=b1FtQnNrY3h4eXo0RlhVKzdJQTVZQT09 | 13:15 |
dtantsur | okay, a report has been sent: https://github.com/sqlalchemy/sqlalchemy/issues/5669 | 13:16 |
*** ianychoi has quit IRC | 13:16 | |
iurygregory | yeah =( | 13:17 |
dtantsur | on the bright side: it seems to be reasonably doable to test secure boot on VMs | 13:18 |
dtantsur | ("reasonably", "UEFI", hahaha) | 13:18 |
*** akiselev has joined #openstack-ironic | 13:20 | |
iurygregory | hahaha | 13:20 |
dtantsur | but according to the internet it boils down to which NVRAM variables you supply to the VM | 13:21 |
dtantsur | and newer libvirt makes it easy to use, but RHEL 8 doesn't have newer libvirt.. | 13:21 |
*** akiselev has quit IRC | 13:24 | |
rpittau | This problem sounds familiar.... | 13:24 |
*** priteau has quit IRC | 13:28 | |
*** akahat has quit IRC | 13:29 | |
*** stevebaker has quit IRC | 13:29 | |
*** k_mouza has joined #openstack-ironic | 13:29 | |
*** akahat has joined #openstack-ironic | 13:31 | |
*** Qianbiao has quit IRC | 13:32 | |
*** Qianbiao has joined #openstack-ironic | 13:33 | |
TheJulia | :( | 13:42 |
*** outbrito has joined #openstack-ironic | 13:42 | |
openstackgerrit | Anuradha proposed openstack/ironic master: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python https://review.opendev.org/758762 | 13:43 |
*** thiagop has quit IRC | 13:44 | |
*** tzumainn has joined #openstack-ironic | 13:48 | |
openstackgerrit | Aija Jaunteva proposed x/sushy-oem-idrac master: [WIP] Update export, import configuration https://review.opendev.org/759425 | 13:50 |
*** k_mouza has quit IRC | 13:51 | |
dtantsur | UEFI in bifrost: VM boots and does exactly nothing. zero. no output, no actions. any ideas? | 13:51 |
TheJulia | dtantsur: the firmware being used? | 13:53 |
TheJulia | ethernet interface type? | 13:53 |
dtantsur | TheJulia: identical to what we do in devstack | 13:53 |
dtantsur | same | 13:53 |
TheJulia | :\ | 13:54 |
dtantsur | TheJulia: generated VM XML if you want to check: https://69f02ed4a1f3a2992e8c-851002f2a2f4fd257d2d88d1bd1bf1ab.ssl.cf2.rackcdn.com/744434/13/check/bifrost-integration-redfish-uefi-ubuntu-focal/cb7f2ab/logs/testvm1_dump.xml | 13:54 |
TheJulia | :( | 13:54 |
dtantsur | maybe 512M is too little? | 13:54 |
TheJulia | well, it was an idea | 13:54 |
TheJulia | oh, yeah, that is | 13:54 |
dtantsur | also pc-1.0 vs just pc, dunno | 13:54 |
dtantsur | (this is re machine type) | 13:54 |
TheJulia | pc-1.0, aiui, is just an alias | 13:54 |
dtantsur | trying | 13:58 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 13:58 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: [WIP] Add import, export configuration to idrac-redfish https://review.opendev.org/759428 | 14:00 |
*** jawad_axd has quit IRC | 14:00 | |
fgofurov | dtantsur: there is a teams problem while we are admitting you to join the meeting | 14:02 |
*** k_mouza has joined #openstack-ironic | 14:03 | |
dtantsur | fgofurov: yep, seems so | 14:03 |
fgofurov | Can you please join metal3 community meeting | 14:03 |
fgofurov | Here is the link: https://zoom.us/j/97255696401?pwd=ZlJMckNFLzdxMDNZN2xvTW5oa2lCZz09#success | 14:03 |
TheJulia | reviews on trying to fix uefi via ipmi would be appreciated | 14:04 |
TheJulia | https://review.opendev.org/#/c/757198/ | 14:04 |
patchbot | patch 757198 - ironic - IPMI: Handle vendor set boot device differences - 4 patch sets | 14:04 |
* iurygregory looks | 14:04 | |
dtantsur | rpittau, iurygregory: if you're curious, the meeting above is about conductor scaling with metal3 | 14:05 |
iurygregory | joined | 14:05 |
fgofurov | yeah, welcome whoever interested | 14:05 |
rpittau | I actually need to leave now, have a great weekend everyone o/ | 14:05 |
*** rpittau is now known as rpittau|afk | 14:06 | |
TheJulia | erbarr: is dell ci happier today? | 14:09 |
*** k_mouza has quit IRC | 14:14 | |
*** jawad_axd has joined #openstack-ironic | 14:20 | |
*** k_mouza has joined #openstack-ironic | 14:23 | |
Qianbiao | TheJulia I have a quick review with https://review.opendev.org/#/c/757198/, is the patch need a special code from every vendor? | 14:32 |
patchbot | patch 757198 - ironic - IPMI: Handle vendor set boot device differences - 4 patch sets | 14:32 |
Qianbiao | like Huawei will have a special code too? Should I provide a code for that. | 14:33 |
TheJulia | I don't think so unless there is a special need | 14:33 |
TheJulia | Supermicro is the one that we *know* | 14:33 |
TheJulia | cisco MAY be another, but I don't have enough info | 14:33 |
Qianbiao | aha, ok. Only special vendor need a special code. | 14:33 |
TheJulia | yup | 14:33 |
Qianbiao | gotcha ty | 14:34 |
*** Goneri has joined #openstack-ironic | 14:42 | |
dtantsur | TheJulia: (re UEFI) increased RAM, set machine_type=pc, no luck | 15:04 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent master: Allow manual setting of Ironic API Version https://review.opendev.org/759451 | 15:05 |
* dtantsur is very suspicious of ^^ | 15:06 | |
JayF | dtantsur: is that a reference to my patch :D I'm happy to chat about it | 15:07 |
dtantsur | JayF: I haven't looked at it yet, but manually setting microversions is one of the things that are considered no-no in most of cases, hence my suspicions | 15:08 |
JayF | We have a use case for this -- a given node's API access to Ironic in our environment is restricted by an intermediate proxy. Enabling "/" on that proxy makes it significantly more discoverable that an Ironic API lives behind it. | 15:09 |
JayF | So by allowing this value to be set manually -- and disable autodetection -- we avoid having to open up more API surface area to the networks that our nodes are on. | 15:09 |
JayF | FWIW, TheJulia was on board with this feature yesterday. I'm happy to have a more in-depth discussion about it though. | 15:11 |
dtantsur | oh hmm | 15:12 |
TheJulia | yeah | 15:12 |
dtantsur | yeah, a bit exotic case, but I guess it's justified here | 15:12 |
TheJulia | super high security environments are not going to want any unauthenticated discovery | 15:12 |
dtantsur | note that you can also get microversions from headers | 15:13 |
JayF | I also discovered that IRONIC_MIN_API_VERSION == the API version I run IPA against :D | 15:13 |
dtantsur | heh | 15:13 |
JayF | so I hope Julia never gets around to that TODO to bump it, at least not until I bump my whole environment :D | 15:13 |
dtantsur | but yeah, GET /v1/lookup will likely give you microversions too | 15:13 |
TheJulia | me, get back to TODOs... HAHAHAHAHAHAHA | 15:13 |
JayF | That was one of the things I considered when thinking about patching it downstream, but I thought this solution is cleaner | 15:14 |
JayF | especially since if set with the kernel CLI, it's trivial to bump it simultaneous with an Ironic upgrade | 15:14 |
JayF | TheJulia: todo(eventually): ... | 15:14 |
dtantsur | JayF: what exactly are you trying to hide? won't /v1/lookup expose it just as well? | 15:14 |
JayF | dtantsur: first of all, I like the way julia put it | 15:14 |
JayF | > super high security environments are not going to want any unauthenticated discovery | 15:14 |
dtantsur | yep, but what can you discover there that you cannot discover through /v1/lookup? | 15:15 |
JayF | secondly, assuming a case with a bad actor on the network, scanning for interesting services/apis | 15:15 |
JayF | they won't even *know* to go to /v1/lookup, or that an Ironic API lives somewhere behind that IP | 15:15 |
JayF | it's about discoverability | 15:15 |
dtantsur | so you can just bind ironic API to /banana/v1 :) | 15:15 |
JayF | if you say "hey, I'm an Ironic API of Ocata version", then the attacker knows, for absolutely free, that they can hit /v1/lookup and get node information | 15:15 |
dtantsur | I'm not going to block the patch, just seems like you can achieve it easier by binding ironic API to some weird root | 15:16 |
JayF | dtantsur: that's kinda fair, but I don't think this patch is too crazy, and you know as well as I do that moving my Ironic API to /banana/ is the sort of pain I won't be able to endure | 15:16 |
dtantsur | heh | 15:16 |
dtantsur | right, okay, I'm fine with that as long as it carries a warning "do not use unless you really know what you're doing" | 15:16 |
JayF | The real fix for this problem is https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/no-ipa-to-conductor.html | 15:17 |
JayF | Which we just don't have time to implement right now, but remains on the radar generally here | 15:17 |
JayF | dtantsur: ++ I can put a warning label on it, that's sensible | 15:17 |
dtantsur | cool | 15:18 |
dtantsur | for those curious: lookup does expose microversions: http://paste.openstack.org/show/799324/ | 15:18 |
arne_wiebalck | bye everyone, have a good weekend o/ | 15:19 |
JayF | you know, there is an alternate fix for this then | 15:19 |
JayF | /v1/lookup has not changed payload in all our versions | 15:19 |
JayF | we could just always do lookup at MIN_IRONIC_VERSION, and glean the api microversion without an additional request | 15:19 |
JayF | but it's /probably/ bad to assume that /v1/lookup won't ever change | 15:19 |
dtantsur | we can think about it when it does | 15:20 |
dtantsur | or use the try-fallback approach | 15:20 |
JayF | I'm a big fan of my current approach for one big reason: it's already done, with a unit test, just needs a reno and it's done. | 15:20 |
dtantsur | heh, okay | 15:21 |
JayF | but I could see a strong argument that doing our detection against /v1/lookup (or circumventing that detection altogether as described above) would be a better answer | 15:21 |
dtantsur | at the very least, an ability to hardcode a version is useful for testing | 15:21 |
JayF | ++ | 15:21 |
JayF | dtantsur: is this a sufficient "stay away" in the reno? >Deployers in highly-secure environments can now manually set Ironic API version instead of relying on unauthentication autodetection via ipa-ironic-api-version on the kernel command line. This is not a reccomended configuration. | 15:24 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent master: Allow manual setting of Ironic API Version https://review.opendev.org/759451 | 15:26 |
openstackgerrit | Merged openstack/python-ironicclient master: Update create node from file example https://review.opendev.org/759124 | 15:29 |
*** Qianbiao has quit IRC | 15:30 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using deprecated iPXE parameters https://review.opendev.org/759458 | 15:34 |
dtantsur | JayF: probably fine, yes | 15:34 |
JayF | Thanks! I wouldn't have put the warning in there. | 15:34 |
JayF | I always assumed, I guess maybe a bad assumption, that most deployers don't go looking for IPA config options unless they know what they need. | 15:34 |
JayF | But I have absolutely zero perspective on what it's like to be a non-ironic-dev deploying Ironic :D | 15:34 |
dtantsur | some people like trying random stuff in hope it fixes their vaguely related problems | 15:35 |
JayF | makes sense | 15:35 |
JayF | I will say, in my case, the fallback 100% worked | 15:35 |
JayF | I just couldn't leave that traceback on every request. It would've been cruel to the ops who would've chased that wild goose every time. | 15:35 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 15:36 |
dtantsur | maybe UEFI doesn't work because we have displays and VNC in testing VMs? | 15:38 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use centos element for dib on focal https://review.opendev.org/757808 | 15:38 |
dtantsur | I'm fully in the wild uninformed guesses territory | 15:38 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent master: Allow manual setting of Ironic API Version https://review.opendev.org/759451 | 15:38 |
dtantsur | brb, thanks Intel for DSP drivers that panic every week at least once | 15:41 |
JayF | dtantsur: Has someone just tried spinning up a devstack and getting one of those VMs running UEFI manually? | 15:41 |
dtantsur | JayF: it Works In DevStack already | 15:42 |
JayF | ah, trying to do it in standalone? | 15:42 |
dtantsur | I'm trying the seemingly same thing in Bifrost now | 15:42 |
JayF | I see | 15:42 |
dtantsur | and yeah, I'll poke it locally now | 15:42 |
dtantsur | after the reboot (grrrr) | 15:42 |
* JayF knows very little about UEFI, just trying to rubber-duck | 15:43 | |
*** DjeufackZane has joined #openstack-ironic | 15:44 | |
*** gyee has joined #openstack-ironic | 15:48 | |
TheJulia | JayF: dtantsur: I have, but ages ago :( | 15:57 |
dtantsur | Unknown error when attempting to call Galaxy at 'https://galaxy.ansible.com/api/': The read operation timed out | 16:01 |
dtantsur | oh ffs | 16:01 |
dtantsur | "msg": "Request failed: <urlopen error timed out>", "url": "https://download.cirros-cloud.net/0.5.1/cirros-0.5.1-x86_64-disk.img" | 16:04 |
* dtantsur flips table and leaves | 16:04 | |
clarkb | we should cache the cirros image on our test nodes fwiw | 16:08 |
dtantsur | we do, it's local | 16:08 |
*** w14161_ has quit IRC | 16:09 | |
dtantsur | aaand same story locally: a completely empty console log. nothing. no traces. | 16:09 |
*** w14161 has joined #openstack-ironic | 16:10 | |
* dtantsur is seriously lost | 16:14 | |
TheJulia | :( | 16:33 |
TheJulia | calling and being reliant upon galaxy being up is sligtly alarming :( | 16:39 |
*** k_mouza has quit IRC | 16:40 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using deprecated iPXE parameters https://review.opendev.org/759458 | 16:49 |
*** akiselev has joined #openstack-ironic | 16:49 | |
*** arne_wiebalck_ has joined #openstack-ironic | 16:50 | |
*** phrobb_ has joined #openstack-ironic | 16:50 | |
*** DjeufackZane has quit IRC | 16:51 | |
*** ianw has quit IRC | 16:51 | |
*** persia_ has joined #openstack-ironic | 16:51 | |
*** jawad_axd has quit IRC | 16:52 | |
*** persia has quit IRC | 16:57 | |
*** arne_wiebalck has quit IRC | 16:57 | |
*** phrobb has quit IRC | 16:57 | |
*** patchbot has quit IRC | 16:57 | |
*** ajya has quit IRC | 16:57 | |
*** arne_wiebalck_ is now known as arne_wiebalck | 16:57 | |
*** phrobb_ is now known as phrobb | 16:57 | |
*** patchbot has joined #openstack-ironic | 16:58 | |
dtantsur | another issues in the inspector CI \o/ | 17:02 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 17:02 |
dtantsur | if somebody who still has sanity wants to lose it:https://zuul.opendev.org/t/openstack/build/84fb9312c09e4048b53985d381c4c017/log/controller/logs/screen-ironic-inspector-api.txt#724 | 17:10 |
dtantsur | ironic-inspector-api cannot talk to memcached | 17:10 |
dtantsur | on this positive note I'm wishing you a great weekend | 17:12 |
*** dtantsur is now known as dtantsur|afk | 17:12 | |
TheJulia | have a wonderful weekend dtantsur|afk | 17:12 |
NobodyCam | Good Morning Ironic folks | 17:13 |
TheJulia | NobodyCam: I think it is post-summit nap time, tbh | 17:13 |
NobodyCam | Have a good weekend dtantsur|afk | 17:13 |
NobodyCam | :) | 17:13 |
TheJulia | I kind of really want to, I have 3 hours until my next meeting | 17:13 |
NobodyCam | oh yea I was a down circuit issue until like 3 am, so I would vote for nap time | 17:14 |
*** ianw has joined #openstack-ironic | 17:23 | |
TheJulia | ugh | 17:26 |
*** derekh has quit IRC | 17:33 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Tear down raid sets https://review.opendev.org/757141 | 17:55 |
openstackgerrit | Merged openstack/ironic stable/train: DRAC: Fix RAID create_config clean step https://review.opendev.org/750605 | 18:15 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 18:16 |
*** fgofurov has quit IRC | 18:22 | |
*** outbrito has quit IRC | 18:30 | |
*** ociuhandu has joined #openstack-ironic | 18:36 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 18:37 |
anuradha1904 | Hi TheJulia: For the patch https://review.opendev.org/#/c/758762/ , the CI build for requirements is failing even when after locally testing, the local-constraints are successful, I am doing the local test using the command `tox -r -vv -e lower-constraints`, what can possibly be the problem here? | 18:46 |
anuradha1904 | I have fixed the pep8 issues that were earlier mentioned in the review. Thank you very much for your help. | 18:46 |
patchbot | patch 758762 - ironic - Rewrite devstack/tools/ironic/scripts/create-node.... - 5 patch sets | 18:46 |
anuradha1904 | lower-constraints* | 18:47 |
clarkb | anuradha1904: I think you need to drop the comments in the lower-constraints file so it is just the package name and version specifier | 18:59 |
clarkb | https://zuul.opendev.org/t/openstack/build/e85909deade341b8a19707179055447c/log/job-output.txt#814-817 shows you that the comparison doesn't match because of the extra info there | 19:00 |
akiselev | Hi TheJulia: I added two stories on logs filename format as we discussed yesterday: https://storyboard.openstack.org/#!/story/2008280 Add node name to ironic-inspector ramdisk log filename and https://storyboard.openstack.org/#!/story/2008281 Add node name to ironic-conductor ramdisk log filename | 19:03 |
anuradha1904 | okay, got it, will remove it then. Is there however any way to test the changes, because while testing using `tox -r -vv -e lower-constraints`, I did not face any problem locally ? | 19:03 |
anuradha1904 | clarkb: Thank you very much for your help. | 19:04 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 19:06 |
clarkb | anuradha1904: https://zuul.opendev.org/t/openstack/build/e85909deade341b8a19707179055447c/console#2/0/15/ubuntu-bionic that shows you the command that was run: `src/opendev.org/openstack/requirements/playbooks/files/project-requirements-change.py src/opendev.org/openstack/ironic master` | 19:07 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Run dib ipa src jobs on ubuntu focal https://review.opendev.org/757812 | 19:13 |
*** ociuhandu has quit IRC | 19:14 | |
anuradha1904 | clarkb: I am sorry, can you elaborate a bit, how to run the command locally? I did not understand the command structure. Is there documentation to refer to this information? Thank you. | 19:16 |
clarkb | anuradha1904: this is a centralized job to ensure consistency with the openstack/requirements project. So what it does is runs the playbooks/files/project-requirements-change.py script from the openstack/requirements project | 19:21 |
clarkb | the requirements project may have docs on running thsi locally, I'm not sure | 19:21 |
anuradha1904 | clarkb: oh got it now, thank you very much. | 19:22 |
iurygregory | anuradha1904, after you clone the requirements repository if you just need to do the following | 19:24 |
iurygregory | http://paste.openstack.org/show/799339/ | 19:25 |
anuradha1904 | iurygregory: Thank you very much. | 19:26 |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: move node from rebuild 'deploy failed' to 'active' https://review.opendev.org/759511 | 19:27 |
TheJulia | rloo: my brain just twitched a little | 19:37 |
TheJulia | rloo: fwiw, I think it is needed | 19:39 |
TheJulia | I'm just worried about misuse | 19:39 |
*** rcernin has quit IRC | 19:55 | |
openstackgerrit | Anuradha proposed openstack/ironic master: Changes to add exception in "default_interface()" function https://review.opendev.org/756682 | 20:01 |
rloo | TheJulia: yeah. i dunno. we had discussed at last PTG and I felt the need to get that written before this upcoming PTG cuz... shame on me for volunteering and not doing it :-( | 20:11 |
rloo | TheJulia: i figure if nova has such a thing, then it would be ok in ironic. but we can discuss :) | 20:12 |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: move node from rebuild 'deploy failed' to 'active' https://review.opendev.org/759511 | 20:15 |
TheJulia | I swear I did that in the early TripleO days... | 20:39 |
TheJulia | at least, used it that way | 20:39 |
TheJulia | but it is all a blur | 20:40 |
*** vesper has joined #openstack-ironic | 20:44 | |
*** bhagyashris has joined #openstack-ironic | 20:47 | |
*** ianw has quit IRC | 20:50 | |
*** patchbot has quit IRC | 20:50 | |
*** Goneri has quit IRC | 20:50 | |
*** bfournie has quit IRC | 20:50 | |
*** bhagyashris|sick has quit IRC | 20:50 | |
*** vesper11 has quit IRC | 20:50 | |
*** sdanni has quit IRC | 20:50 | |
*** ianw has joined #openstack-ironic | 20:50 | |
*** patchbot has joined #openstack-ironic | 20:50 | |
*** Goneri has joined #openstack-ironic | 20:50 | |
*** bfournie has joined #openstack-ironic | 20:50 | |
*** sdanni has joined #openstack-ironic | 20:50 | |
akiselev | TheJulia: PTAL https://storyboard.openstack.org/#!/story/2008280 https://storyboard.openstack.org/#!/story/2008281 | 20:50 |
*** sdanni has quit IRC | 20:53 | |
*** ociuhandu has joined #openstack-ironic | 21:14 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: move node from rebuild 'deploy failed' to 'active' https://review.opendev.org/759511 | 21:16 |
*** ociuhandu has quit IRC | 21:19 | |
TheJulia | akiselev: lgtm, Have a good weekend | 21:20 |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: update lower-constraints.txt https://review.opendev.org/759238 | 21:21 |
*** rajinir has quit IRC | 21:25 | |
akiselev | TheJulia: Thank you! Have a nice weekend! | 21:29 |
*** bdodd_ has quit IRC | 21:44 | |
*** rcernin has joined #openstack-ironic | 22:00 | |
*** akiselev has quit IRC | 22:09 | |
*** rloo has quit IRC | 22:14 | |
*** anuradha1904 has quit IRC | 22:39 | |
*** k_mouza has joined #openstack-ironic | 22:42 | |
*** k_mouza has quit IRC | 22:46 | |
*** rcernin has quit IRC | 22:59 | |
*** bdodd has joined #openstack-ironic | 23:45 | |
*** tosky has quit IRC | 23:46 | |
*** rcernin has joined #openstack-ironic | 23:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!