*** gyee has quit IRC | 00:24 | |
*** seongsoocho has quit IRC | 00:43 | |
*** seongsoocho has joined #openstack-ironic | 00:43 | |
*** stevebaker has quit IRC | 01:03 | |
*** Goneri has quit IRC | 01:04 | |
*** Qianbiao has joined #openstack-ironic | 01:22 | |
TheJulia | good morning | 01:37 |
---|---|---|
TheJulia | erbarr: looks like dell jobs are failing with "/tmp/molteniron_checkin.sh: not found" | 01:41 |
erbarr | TheJulia, yea, there's an issue with the DNS, I tried switching to another one but seems no good either | 01:43 |
TheJulia | erbarr: dns howso? | 01:43 |
erbarr | internal changes | 01:44 |
TheJulia | ahh :( | 02:05 |
TheJulia | Okay, hopefully that gets resolved quickly :) | 02:05 |
*** Qianbiao has quit IRC | 02:06 | |
*** Qianbiao has joined #openstack-ironic | 02:07 | |
openstackgerrit | QianBiao Ng proposed openstack/ironic stable/train: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/757606 | 02:09 |
openstackgerrit | Armstrong Liu proposed openstack/ironic-python-agent master: Add an option about collecting network information https://review.opendev.org/758950 | 02:12 |
*** mcarden has quit IRC | 02:18 | |
*** rh-jelabarre has quit IRC | 02:20 | |
*** rh-jelabarre has joined #openstack-ironic | 02:21 | |
openstackgerrit | Merged openstack/ironic master: Change auth type to none in dev-quickstart https://review.opendev.org/758476 | 02:24 |
*** tzumainn has quit IRC | 02:27 | |
*** larsks has quit IRC | 02:36 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Add documentation about the available Elements https://review.opendev.org/758508 | 02:37 |
openstackgerrit | Merged openstack/bifrost master: Force local connection for localhost https://review.opendev.org/758814 | 02:37 |
openstackgerrit | Merged openstack/bifrost master: Set safe version of hacking https://review.opendev.org/758771 | 02:37 |
*** larsks has joined #openstack-ironic | 02:50 | |
*** rcernin has quit IRC | 02:52 | |
*** rcernin has joined #openstack-ironic | 02:56 | |
*** larsks has quit IRC | 02:59 | |
TheJulia | Calling it a night, ironic on the spreadsheet just needs a second set of eyes | 03:01 |
*** larsks has joined #openstack-ironic | 03:02 | |
*** larsks has quit IRC | 03:05 | |
*** larsks has joined #openstack-ironic | 03:08 | |
*** larsks has quit IRC | 03:09 | |
*** larsks has joined #openstack-ironic | 03:12 | |
Qianbiao | TheJulia need an eye for what, I can become the eye :) | 03:43 |
TheJulia | Qianbiao: audit git changes going back to October 1. A few of the cores have done an awesome job already of digging through the spreadsheet I sent to the mailing list, adding entries and checking if everything looks good | 03:48 |
Qianbiao | I see, TheJulia. | 03:51 |
Qianbiao | It seems only networking-baremetal and networking-generic-switch is not confirmed. | 03:51 |
TheJulia | Qianbiao: I don't think they had any commits in the window of concern | 04:12 |
TheJulia | Anyway, I'm going to get some sleep. I have been getting up at 5 AM local all week. :( | 04:12 |
Qianbiao | sure, good sleep | 04:12 |
*** rh-jelabarre has quit IRC | 06:05 | |
*** rh-jelabarre has joined #openstack-ironic | 06:06 | |
*** jtomasek has joined #openstack-ironic | 06:20 | |
openstackgerrit | Pierre Riteau proposed openstack/bifrost stable/victoria: Force local connection for localhost https://review.opendev.org/759188 | 06:25 |
openstackgerrit | Armstrong Liu proposed openstack/ironic-python-agent master: Add an option about collecting network information https://review.opendev.org/758950 | 06:45 |
iurygregory | good morning Ironic! | 06:48 |
janders | good morning iurygregory | 06:49 |
*** uzumaki has joined #openstack-ironic | 06:51 | |
*** rpittau|afk is now known as rpittau | 06:54 | |
rpittau | good morning ironic! o/ | 06:54 |
*** jawad_axd has joined #openstack-ironic | 06:54 | |
rpittau | happy thursday! | 06:54 |
janders | good morning rpittau | 06:55 |
rpittau | hey janders :) | 06:55 |
iurygregory | morning rpittau o/ | 06:58 |
rpittau | hey iurygregory :) | 06:58 |
* iurygregory have the feeling that arne_wiebalck is late today... | 06:58 | |
arne_wiebalck | Good morning, ironic! | 07:02 |
* arne_wiebalck is late today :-D | 07:02 | |
janders | good morning arne_wiebalck | 07:02 |
arne_wiebalck | Hey janders iurygregory rpittau o/ | 07:02 |
rpittau | hey arne_wiebalck :) | 07:02 |
* iurygregory used his power and arne_wiebalck appeared | 07:02 | |
iurygregory | *magic* | 07:02 |
* arne_wiebalck would rather like if iurygregory would use this power to magically make some holidays appear | 07:03 | |
iurygregory | I did | 07:03 |
iurygregory | next week there is 1 holiday for me | 07:04 |
iurygregory | =D | 07:04 |
rpittau | :) | 07:04 |
iurygregory | I'm sorry I can't share lol | 07:04 |
rpittau | my holiday was eaten by the bad sunday monster :/ | 07:04 |
iurygregory | ouch =( | 07:04 |
rpittau | luckily there's another 10 days later :D | 07:05 |
janders | I wish the miracle of compound interest applied to accumulated leave | 07:06 |
* iurygregory has 1 next wed and one on 17th nov... | 07:11 | |
*** smutugrigg has joined #openstack-ironic | 07:24 | |
*** DjeufackZane has joined #openstack-ironic | 07:24 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Set safe version of hacking https://review.opendev.org/758764 | 07:26 |
*** uzumaki has quit IRC | 07:29 | |
*** uzumaki has joined #openstack-ironic | 07:33 | |
smutugrigg | Hi ironic o/ | 07:35 |
uzumaki | hey there! smutugrigg o/ | 07:36 |
uzumaki | Morning iurygregory arne_wiebalck janders o/ | 07:36 |
smutugrigg | hi uzumaki o/ | 07:37 |
uzumaki | how you doing smutugrigg ? | 07:38 |
*** ociuhandu has joined #openstack-ironic | 07:41 | |
*** fgofurov has joined #openstack-ironic | 07:42 | |
smutugrigg | trying to work out some changes I need to make but would really like some guidance as I don't want to make a mistake | 07:42 |
*** rcernin has quit IRC | 07:42 | |
smutugrigg | sorry for uzumaki | 07:43 |
*** tosky has joined #openstack-ironic | 07:44 | |
smutugrigg | hi iurygregory, could I please request some assitance if you are there please? | 07:44 |
smutugrigg | or TheJulia? | 07:45 |
*** rcernin has joined #openstack-ironic | 07:48 | |
*** alexmcleod has quit IRC | 07:50 | |
*** alexmcleod has joined #openstack-ironic | 07:50 | |
iurygregory | smutugrigg, good morning o/ | 07:52 |
iurygregory | sure! | 07:52 |
*** rcernin has quit IRC | 07:54 | |
*** anuradha1904 has joined #openstack-ironic | 07:57 | |
openstackgerrit | Emmanouil Bagakis proposed openstack/ironic-python-agent master: Updated Implementation of string interpolation delay on LOG messages https://review.opendev.org/759203 | 07:58 |
smutugrigg | good morning iurygregory, thank you! | 07:59 |
smutugrigg | Does the comment "doc/source/admin/node-deployment.rst | 08:00 |
smutugrigg | Line 106: | 08:00 |
smutugrigg | huh, this file has a different convention, $node_ident instead of <node> everywhere else. If there is a another revision to this change it would be good to make this consistent" require any action from me? | 08:00 |
iurygregory | let me check the patch | 08:00 |
smutugrigg | Thank you | 08:00 |
*** rcernin has joined #openstack-ironic | 08:00 | |
iurygregory | smutugrigg, oh Julia already +2 I just saw. We can merge your change and you can push a new patch with the suggestions if you can =) | 08:02 |
smutugrigg | Oh? I made all the other changes ;) | 08:02 |
iurygregory | you can submit normally, but instead of "git commit --amend" | 08:03 |
smutugrigg | I would really like it if the original change could be merged and happy to work on a new change to make all the suggested changes | 08:03 |
iurygregory | you can just add the changes and do git commit | 08:03 |
iurygregory | so it will generate a new one | 08:03 |
smutugrigg | perfect | 08:04 |
smutugrigg | will hold the changes for now then? | 08:04 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Set safe version of hacking https://review.opendev.org/758766 | 08:04 |
iurygregory | it should be fine to send the new patch if you already did the changes =) | 08:04 |
smutugrigg | Do I need to follow up with Kaifeng on their review notes on 756708 before the change is ready to submit? | 08:05 |
iurygregory | let me check | 08:06 |
smutugrigg | sure | 08:09 |
iurygregory | smutugrigg, you mean to switch places to <node> ? | 08:09 |
*** lucasagomes has joined #openstack-ironic | 08:10 | |
smutugrigg | iurygregory, I was meaning are they happy with all the changes or are your and TheJulia's review sufficient? | 08:11 |
*** rcernin has quit IRC | 08:12 | |
iurygregory | smutugrigg, it should be fine =) try to address the comments on the last patch in a new one | 08:12 |
smutugrigg | Great, I will do. Shall I create a new task in https://storyboard.openstack.org/#!/story/2008194 to track the changes and commit to it? | 08:13 |
iurygregory | you can =) | 08:14 |
iurygregory | or you can just re-use the same task number | 08:15 |
iurygregory | and explain in the commit message | 08:15 |
smutugrigg | Ok happy to do that | 08:15 |
smutugrigg | iurygregory, who needs to complete the verify the check to allow me to submit? | 08:17 |
iurygregory | you can submit normally, people will try to review when they have time =) | 08:17 |
smutugrigg | https://review.opendev.org/#/c/756708/5 is still in status "Needs verified Label" | 08:19 |
patchbot | patch 756708 - ironic - Update .rst files - 5 patch sets | 08:19 |
smutugrigg | Does someone need to verify check before I am able to submit? | 08:19 |
iurygregory | oh this is done by zuul | 08:20 |
iurygregory | after the patch has Workflow +1 | 08:21 |
smutugrigg | Ah, sorry was not aware of that. | 08:21 |
smutugrigg | So I just wait until Zuul has completed and will be able to submit? | 08:21 |
*** rcernin has joined #openstack-ironic | 08:21 | |
iurygregory | if you look at https://zuul.openstack.org/status and you add 756708 to the search field on the left of the page you can see =) | 08:21 |
smutugrigg | thank you | 08:23 |
*** rcernin has quit IRC | 08:26 | |
*** sshnaidm|afk is now known as sshnaidm|rover | 08:29 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Set safe version of hacking https://review.opendev.org/758764 | 08:30 |
*** DjeufackZane has quit IRC | 08:33 | |
*** derekh has joined #openstack-ironic | 08:38 | |
openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Set safe version of hacking https://review.opendev.org/758789 | 08:46 |
openstackgerrit | Merged openstack/ironic-python-agent master: Set safe version of hacking https://review.opendev.org/758765 | 08:46 |
openstackgerrit | Merged openstack/ironic master: Update .rst files https://review.opendev.org/756708 | 08:46 |
*** ddurst has quit IRC | 08:48 | |
*** ddurst has joined #openstack-ironic | 08:55 | |
*** dtantsur|afk is now known as dtantsur | 08:56 | |
dtantsur | morning ironic | 08:56 |
*** iurygregory has quit IRC | 08:57 | |
rpittau | hey dtantsur :) | 08:58 |
*** uzumaki has quit IRC | 09:10 | |
*** uzumaki has joined #openstack-ironic | 09:10 | |
*** rcernin has joined #openstack-ironic | 09:21 | |
*** rcernin has quit IRC | 09:25 | |
*** iurygregory has joined #openstack-ironic | 09:28 | |
fgofurov | Hello folks, good morning/afternoon! I have a question and highly appreciate if someone can help/guide me through this | 09:36 |
fgofurov | The problem we are facing is with ironic states. For example when the image server does not answer, then the adoption process just hanging forever. We know that it hangs when ironic tries to validate the image usually, but apparently this is not always and sometimes we don't know at what stage it gets stuck/fails silently without changing the state. | 09:37 |
fgofurov | We want to ensure that all operations done for the adoption have a timeout and that adoption would always fail if something hangs for too long, and not get stuck in "adopting" state. We can recover from an "adopt failed" state, but the problem for us is recovering from "adopting" state as it does not fail and it just hangs. Any suggestions/pointers appreciated. | 09:38 |
fgofurov | As far as I understood, we can retry only if it is in adoption failed state, but to go to this state "adopting" should fail(normally), but in our case it is not failing, it just stuck/hangs in "adopting".. | 09:41 |
*** k_mouza has joined #openstack-ironic | 09:44 | |
openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files https://review.opendev.org/759236 | 09:51 |
openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files https://review.opendev.org/759236 | 09:53 |
smutugrigg | hi iurygregory, just to let you know, I have now committed the suggested additional changes under change https://review.opendev.org/#/c/759236/ but still pointing to task 40964 | 09:54 |
patchbot | patch 759236 - ironic - Update .rst files - 2 patch sets | 09:54 |
iurygregory | smutugrigg, cool tks! | 09:55 |
smutugrigg | I will try to make an additional update before the deadline if I have time with class. | 09:55 |
smutugrigg | iurygregory, thanks for all your help. | 09:55 |
iurygregory | np | 09:56 |
smutugrigg | bye ironic o/ | 09:56 |
dtantsur | fgofurov: hi. Mmm, we need to make sure that all calls to all services have an HTTP timeout. | 09:56 |
dtantsur | we've started down that path a while ago, but I guess never completed it. | 09:56 |
*** smutugrigg has quit IRC | 09:57 | |
fgofurov | dtantsur: thanks for reply. Do you mean there is an ongoing work? or should I open a ticket, could you please elaborate more a bit? | 10:00 |
rpittau | fgofurov: maybe opening a story in storyboard with some details would help | 10:02 |
openstackgerrit | likui proposed openstack/ironic-python-agent master: update lower-constraints.txt https://review.opendev.org/759238 | 10:03 |
fgofurov | rpittau: sure, definitely, I will do it. Just wondering if we can complete the work regarding this that was not completed as dtantsur was mentioning and and how can we contribute to this from metal3 side | 10:07 |
dtantsur | fgofurov: I guess it requires some code on the ironic side | 10:11 |
dtantsur | although, there may be just configuration options to change that; maybe we need better defaults | 10:11 |
rpittau | fgofurov: not sure there's anything that can be done from pure metal3 side | 10:11 |
dtantsur | mmm, I see, there should be a configurable timeout here: https://opendev.org/openstack/ironic/src/branch/master/ironic/common/image_service.py#L101 | 10:12 |
rpittau | mmm I just noticed a little issue with one of the ipa jobs, we need to approve this https://review.opendev.org/757812 ASAP | 10:19 |
patchbot | patch 757812 - ironic-python-agent - Run dib ipa src jobs on ubuntu focal - 5 patch sets | 10:19 |
rpittau | the ipa-tempest-bios-ipmi-direct-src doesn't always run DIB | 10:19 |
rpittau | if it gets rax hosts will run tinyipa | 10:19 |
rpittau | on bionic | 10:19 |
rpittau | failing because the kernel is too old for the chroot | 10:20 |
iurygregory | rpittau, +2 | 10:20 |
rpittau | thanks | 10:20 |
iurygregory | funny | 10:20 |
iurygregory | :D | 10:20 |
rpittau | heh | 10:20 |
fgofurov | dtantsur: yes, seems either timeout needed for "adopting" or we can have a timeout for retrieval of disk images as you pointed out | 10:20 |
iurygregory | old kernel etc | 10:20 |
rpittau | iurygregory: yeah, it's because we're running tinycore 11 now with kernel 5.x | 10:21 |
rpittau | bionic has kenerl 4.x | 10:21 |
rpittau | or kernel even | 10:21 |
iurygregory | yeah I'm aware of that =D | 10:21 |
rpittau | oh and this too please https://review.opendev.org/757811 | 10:26 |
patchbot | patch 757811 - ironic - Use centos as base element for dib images - 2 patch sets | 10:26 |
dtantsur | fgofurov: I can look into that if you're not planning to | 10:30 |
iurygregory | dtantsur, I have the feeling openstacksdk CI hates your patch :X | 10:31 |
dtantsur | CI often hates my patches | 10:31 |
iurygregory | =( | 10:32 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM try to inject the correct grub location https://review.opendev.org/758809 | 10:37 |
*** rcernin has joined #openstack-ironic | 10:43 | |
fgofurov | dtantsur: I am happy with that, please do. I will be around and let me know please for coming fixes.. Thanks a lot:) | 10:44 |
fgofurov | dtantsur: I will open a story a paste a link here to track it | 10:45 |
fgofurov | *and ^^ | 10:45 |
*** rcernin has quit IRC | 10:48 | |
fgofurov | dtantsur, rpittau: the story opened for adoption failing here: https://storyboard.openstack.org/#!/story/2008269 | 10:49 |
rpittau | it's probably my paranoia that speaks but don't the two commits https://opendev.org/openstack/python-ironicclient/commit/c09eec0025044b4fe359d39cbcded328f484f56a and https://opendev.org/openstack/python-ironicclient/commit/0e4a23393f8e4b24fa9da078dfe0af24e48088c2 look exactly the same ? | 10:50 |
rpittau | thanks fgofurov | 10:51 |
dtantsur | rpittau: ehhhmmmm | 10:54 |
rpittau | am I being paranoid ? | 10:55 |
rpittau | https://review.opendev.org/#/c/751649/ = https://review.opendev.org/#/c/751649/ | 10:55 |
patchbot | patch 751649 - python-ironicclient - Remove install unnecessary packages (MERGED) - 1 patch set | 10:55 |
patchbot | patch 751649 - python-ironicclient - Remove install unnecessary packages (MERGED) - 1 patch set | 10:55 |
rpittau | whoops | 10:55 |
dtantsur | https://review.opendev.org/#/c/753922/ | 10:55 |
patchbot | patch 753922 - python-ironicclient - Remove install unnecessary packages (MERGED) - 1 patch set | 10:55 |
rpittau | yeah | 10:56 |
dtantsur | but yeah, the reviews also look nearly identical: the same reviewers in the same order | 10:56 |
rpittau | yeeep | 10:56 |
* rpittau paranoia intensifies | 10:56 | |
dtantsur | I mean, it's possible in a normal flow | 10:56 |
dtantsur | the changes are identical, the 2nd one will result in an empty merge conflict | 10:57 |
rpittau | mmmm | 10:57 |
dtantsur | the reviewers matching looks weird, but also not impossible | 10:57 |
dtantsur | ugh s/merge conflict/merge commit/ | 10:57 |
rpittau | they are 4 days apart, the merge I mean | 10:57 |
dtantsur | it's probably worth reporting to the admins for double-checking, but I support it's just a funny coincidence | 10:58 |
dtantsur | (resulting from a lot of people using the same scripts for auto-generating tons of patches) | 10:58 |
iurygregory | zuul wouldn't go crazy? | 10:58 |
iurygregory | O.o | 10:58 |
iurygregory | omg | 10:59 |
dtantsur | merging them will make one normal merge commit, then one empty | 10:59 |
dtantsur | you get a conflict only when changes are different | 10:59 |
dtantsur | you can try it locally to confirm | 10:59 |
rpittau | dtantsur: how do we report that ? | 11:00 |
dtantsur | rpittau: no idea, try #opendev? | 11:00 |
rpittau | ack | 11:00 |
iurygregory | I do remember reviewing the at least one of the patches... | 11:05 |
iurygregory | but its a bit strange... | 11:06 |
rpittau | I posted on #opendev let's see what they say | 11:07 |
*** jawad_axd has quit IRC | 11:11 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add timeout to image operations in the direct deploy https://review.opendev.org/759245 | 11:17 |
dtantsur | fgofurov: ^^ | 11:17 |
fgofurov | dtantsur: thanks a lot for your quick response | 11:30 |
openstackgerrit | Emmanouil Bagakis proposed openstack/ironic-python-agent master: This is a follow up to https://review.opendev.org/#/c/756300/ https://review.opendev.org/759246 | 11:37 |
*** k_mouza_ has joined #openstack-ironic | 11:43 | |
*** k_mouza has quit IRC | 11:43 | |
*** k_mouza has joined #openstack-ironic | 11:44 | |
*** k_mouza_ has quit IRC | 11:47 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Run bifrost integration job on focal https://review.opendev.org/758198 | 11:48 |
*** sai438 has quit IRC | 11:55 | |
*** pmannidi has joined #openstack-ironic | 11:57 | |
*** k_mouza_ has joined #openstack-ironic | 12:07 | |
*** Goneri has joined #openstack-ironic | 12:08 | |
*** k_mouza has quit IRC | 12:10 | |
*** uzumaki has quit IRC | 12:12 | |
*** rloo has joined #openstack-ironic | 12:12 | |
*** uzumaki has joined #openstack-ironic | 12:12 | |
*** k_mouza has joined #openstack-ironic | 12:13 | |
*** rcernin has joined #openstack-ironic | 12:17 | |
*** k_mouza_ has quit IRC | 12:17 | |
*** jawad_axd has joined #openstack-ironic | 12:19 | |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 12:19 |
*** rcernin has quit IRC | 12:22 | |
*** k_mouza has quit IRC | 12:29 | |
*** k_mouza has joined #openstack-ironic | 12:29 | |
TheJulia | good morning | 12:36 |
iurygregory | good morning TheJulia =) | 12:36 |
dtantsur | TheJulia: morning! | 12:36 |
iurygregory | corgi overlord will join the sessions today also? \o/ | 12:37 |
TheJulia | dunno | 12:38 |
TheJulia | Reminder, in about ?22? minutes, we're going to be revisiting https://summit.openinfra.dev/a/event/24757 because of things kind of cratering on Monday | 12:38 |
TheJulia | trandles: janders: https://summit.openinfra.dev/a/event/24539 in 45 minutes | 12:39 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM try to inject the correct grub location https://review.opendev.org/758809 | 12:40 |
iurygregory | TheJulia, the spaghetti session will be on zoom right? | 12:51 |
dtantsur | anyone has had success with devstack on centos8? is it worth trying? | 12:51 |
TheJulia | iurygregory: yes, but hopefully we will have streaming working | 12:52 |
iurygregory | TheJulia, cool | 12:52 |
rpittau | dtantsur: I did make it work a while ago, if memory serves the changes on devstack itself are not that much but it took some effort to make ironic run | 12:55 |
dtantsur | sigh.. | 12:55 |
dtantsur | I'm starting to think it's easier to make UEFI work on bifrost | 12:56 |
rpittau | heh :/ | 12:57 |
dtantsur | .. or switch to ubuntu :( | 12:57 |
janders | TheJulia trandles fingers crossed the platform stands up to the challange this time :) | 13:02 |
openstackgerrit | Emmanouil Bagakis proposed openstack/ironic-python-agent master: Updated Implementation of string interpolation delay on LOG messages https://review.opendev.org/759203 | 13:05 |
*** uzumaki has quit IRC | 13:06 | |
rpioso | Good morning, ironic | 13:07 |
rpioso | A new patch set was uploaded yesterday for https://review.opendev.org/#/c/740721/. ajya and I would appreciate folks reading it before the PTG discussion on Tuesday. Review comments are welcome, too :-) | 13:09 |
patchbot | patch 740721 - ironic-specs - Pre-defined system hw configuration in single step - 17 patch sets | 13:09 |
rpioso | dtantsur, TheJulia: ^^^ | 13:09 |
*** jawad_axd has quit IRC | 13:28 | |
*** ayoung has quit IRC | 13:41 | |
*** rcernin has joined #openstack-ironic | 13:41 | |
*** tzumainn has joined #openstack-ironic | 13:42 | |
*** rcernin has quit IRC | 13:45 | |
janders | see you tomorrow Ironic o/ | 13:48 |
rpittau | bye janders :) | 13:49 |
arne_wiebalck | Thanks for the presentation janders, TheJulia, trandles ! | 13:49 |
TheJulia | Hopefully it was informative and didn't seem silly?! | 13:50 |
TheJulia | That was the 3rd or 4th take on the second day | 13:50 |
*** sshnaidm|rover is now known as sshnaidm|afk | 13:54 | |
arne_wiebalck | It definitely was (informative, not silly ;) | 13:55 |
*** k_mouza has quit IRC | 14:01 | |
*** weshay|ruck has left #openstack-ironic | 14:03 | |
Qianbiao | Hello, Ironic team. | 14:05 |
Qianbiao | I got this when visit docs https://pasteboard.co/JwPCdjQ.png | 14:05 |
Qianbiao | It may cause a security issue (display folder) | 14:05 |
Qianbiao | Not sure, just a remind | 14:06 |
trandles | thanks arne_wiebalck | 14:07 |
*** k_mouza has joined #openstack-ironic | 14:07 | |
dtantsur | Qianbiao: wow | 14:13 |
dtantsur | Qianbiao: could you tell it to #openstack-infra with a link to where you see it? | 14:13 |
Qianbiao | ok | 14:13 |
rpittau | yeah this is not good -> https://docs.openstack.org/diskimage-builder/latest/elements/ | 14:14 |
iurygregory | yeah | 14:15 |
iurygregory | https://docs.openstack.org/diskimage-builder/latest/elements.html is ok | 14:16 |
TheJulia | I guess it is because there is no index.rst file in the elements folder pointing to ../elements.rst | 14:18 |
Qianbiao | TheJulia hmm, i think if there are no elements.rst, it should be 404. | 14:20 |
Qianbiao | maybe "index on" option is set in the static file server(like apache or nginx) | 14:21 |
Qianbiao | is the Anaconda in our talking today the https://www.anaconda.com/? | 14:21 |
Qianbiao | I can not image how it has connection with deploy interface. | 14:21 |
dtantsur | Qianbiao: no, the installer usually used in Red Hat systems | 14:21 |
dtantsur | https://fedoraproject.org/wiki/Anaconda | 14:22 |
Qianbiao | aha ty dtantsur | 14:22 |
TheJulia | possibly, the tempalte to content generation behavior is a little weird and ath eserver settings may be a little weird | 14:22 |
TheJulia | I agree, indexes it shouldn't be doing | 14:22 |
Qianbiao | TheJulia yeah | 14:23 |
Qianbiao | I have no idea with a lot of words during the talking, google will be buzy now.. :) | 14:23 |
TheJulia | heh | 14:24 |
TheJulia | oddly enough, I felt the revisited topic, while kind of hard to keep us moving forward we seemed to actually have more thoughts/ideas/comments raise to the surface | 14:27 |
openstackgerrit | QianBiao Ng proposed openstack/ironic stable/stein: opt: Enhance old stable branches to use latest python-ibmcclient https://review.opendev.org/757607 | 14:33 |
TheJulia | before I pull it up, how does the spreadsheet look for auditing the reviews/changes that got approved? | 14:39 |
rpittau | TheJulia: not bad, we're still missing some | 14:41 |
*** sshnaidm|afk is now known as sshnaidm|rover | 14:48 | |
TheJulia | awesome | 14:53 |
* TheJulia wonders if rover means a mars rover is in use | 14:53 | |
*** jawad_axd has joined #openstack-ironic | 14:55 | |
Qianbiao | maybe rover in dream :) | 14:57 |
Qianbiao | need coffee | 14:57 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM try to inject the correct grub location https://review.opendev.org/758809 | 14:59 |
TheJulia | I made fresh coffee a little while ago and I don't remember what I did with it | 15:03 |
TheJulia | #badsign | 15:03 |
*** jawad_axd has quit IRC | 15:05 | |
trandles | TheJulia: I did the exact same thing this morning. For some reason I took it with me into the garage when I was putting stuff in the recycling bin. | 15:07 |
trandles | Found it sitting on the top of my wife's car... | 15:08 |
Qianbiao | need to you, have a good night, Ironic and everyone :) | 15:08 |
*** Qianbiao is now known as Qianbiao|afk | 15:09 | |
*** Qianbiao|afk has quit IRC | 15:09 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 15:09 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add oem_vendors from resource links https://review.opendev.org/755323 | 15:23 |
openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Add oem_vendors from resource links https://review.opendev.org/755323 | 15:24 |
*** gregwork has joined #openstack-ironic | 15:25 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 15:26 |
openstackgerrit | Anuradha proposed openstack/ironic master: Changes to add exception in "default_interface()" function https://review.opendev.org/756682 | 15:31 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Add CI jobs with Redfish and UEFI https://review.opendev.org/744434 | 15:36 |
*** rcernin has joined #openstack-ironic | 15:41 | |
*** rcernin has quit IRC | 15:46 | |
*** DjeufackZane has joined #openstack-ironic | 15:48 | |
rpittau | Good night! o/ | 15:57 |
*** rpittau is now known as rpittau|afk | 15:57 | |
arne_wiebalck | bye everyone o/ | 15:59 |
*** lucasagomes has quit IRC | 16:00 | |
*** w14161 has quit IRC | 16:02 | |
*** w14161 has joined #openstack-ironic | 16:02 | |
*** k_mouza has quit IRC | 16:06 | |
*** k_mouza_ has joined #openstack-ironic | 16:06 | |
*** k_mouza_ has quit IRC | 16:10 | |
dtantsur | tzumainn: hey! you're referring to a Ceph iSCSI patch in your talk, do you have a link for it? | 16:17 |
dtantsur | (and great talk btw) | 16:17 |
tzumainn | dtantsur, thanks! and yes I do: https://review.opendev.org/#/c/662829/ | 16:17 |
patchbot | patch 662829 - cinder - Add ceph iscsi volume driver - 39 patch sets | 16:17 |
tzumainn | we've tested it a bit, and it seeeeems to work; the author says they're just waiting to set up CI to merge it | 16:17 |
dtantsur | tzumainn: very nice! could you let us know when it merges (and works)? it's a highly demanded thing for ironic. | 16:18 |
dtantsur | (dunno if you're using it in the ironic context or not) | 16:18 |
tzumainn | dtantsur, we are using it in an ironic context, and I'll definitely let you know! | 16:19 |
dtantsur | thx! | 16:19 |
dtantsur | re simplified CLI, I'd love to have `baremetal port list` list node UUIDs | 16:19 |
dtantsur | contributions welcome :) | 16:19 |
tzumainn | dtantsur, there is one odd behavior, and I don't know if it's because I set the networking up weirdly, but - if the ceph iscsi gateway is on the provisioning network, and we start by attaching our node to the provisioning network before deploying | 16:20 |
tzumainn | then ironic will detach the provisioning network from the node and delete the neutron port and then fail | 16:20 |
tzumainn | that doesn't happen if we use the non-provisioning network | 16:21 |
dtantsur | yeah, ironic cleans up everything on the provisioning network before deploying | 16:21 |
dtantsur | to make sure there's no artefacts from previous deployments left | 16:21 |
dtantsur | how critical is it for you? | 16:21 |
tzumainn | gotcha - so should we just not use the provisioning network for the ceph iscsi gateway? | 16:21 |
tzumainn | well, we worked around it for purposes of testing the patch | 16:22 |
tzumainn | and I don't think we *have* to use the provisioning network in that way - I'll talk to the moc folks | 16:22 |
dtantsur | tzumainn: I'd avoid doing that if possible | 16:23 |
tzumainn | dtantsur, okay, I'll pass along the recommendation - it'll be stronger with you guys backing me up :) | 16:24 |
tzumainn | thanks! | 16:24 |
tzumainn | dtantsur, so regarding the CLI, would you guys also be interested in stuff like a more streamlined way of attaching a network to a node, or is that sort of stuff too much? | 16:25 |
dtantsur | tzumainn: it may be problematic to cross the service boundary (this is why metalsmith exists) | 16:27 |
tzumainn | dtantsur, yeah, I suspected as much - makes sense! | 16:28 |
*** ociuhandu_ has joined #openstack-ironic | 16:28 | |
dtantsur | maybe extending metalsmith may work.. | 16:28 |
dtantsur | just food for thought :) | 16:28 |
* dtantsur needs actual food now | 16:28 | |
fgofurov | Bye everyone o/ | 16:30 |
*** fgofurov has quit IRC | 16:30 | |
*** dtantsur is now known as dtantsur|afk | 16:31 | |
dtantsur|afk | o/ | 16:31 |
*** ociuhandu has quit IRC | 16:32 | |
*** ociuhandu_ has quit IRC | 16:32 | |
*** valleedelisle has joined #openstack-ironic | 16:39 | |
*** DjeufackZane has quit IRC | 16:51 | |
*** derekh has quit IRC | 17:00 | |
*** jtomasek has quit IRC | 17:07 | |
*** valleedelisle has quit IRC | 17:13 | |
TheJulia | Anyone know of any good tripleo examples that cover bare metal as a service use cases? | 17:16 |
*** jawad_axd has joined #openstack-ironic | 17:17 | |
*** valleedelisle has joined #openstack-ironic | 17:24 | |
*** jawad_axd has quit IRC | 17:32 | |
*** mrquincy has joined #openstack-ironic | 17:40 | |
mrquincy | re: OpenStack Summit, I was hoping to catch the Ironic presentations after the fact but the platform seems kinda broken. Anyone have working links to the presentations: "Deployment with bare metal system configuration molds in Ironic" and "Plans and Directions for the Bare Metal SIG"? | 17:43 |
dking | Does the ironic-api service respond to any environment varables? | 17:44 |
clarkb | mrquincy: I've asked people who should know. Will let you know what they say | 17:44 |
mrquincy | Thanks, I appreciate it | 17:45 |
*** sshnaidm|rover is now known as sshnaidm|off | 17:45 | |
dking | For instance, is there any way to be able to set, for example, the database connection in an environment variable before running /usr/local/bin/ironic-api ? | 17:48 |
clarkb | mrquincy: https://summit.openinfra.dev/a/event/24761 and https://summit.openinfra.dev/a/event/24754 are links to the two sessions you asked about. I'm told they are working on the video uploads as we speak for forum sessions. Sounds like they hope to have them up by tomorrow | 17:52 |
mrquincy | oh ok, that's good that the videos exist somewhere. I'll check back later. | 17:53 |
mrquincy | Thanks agian | 17:53 |
TheJulia | dking: not _really_ the expectation is to use the configuration file. I guess patches to oslo_config may be appreciated in that regard | 17:54 |
*** ricolin has quit IRC | 17:54 | |
openstackgerrit | Anuradha proposed openstack/ironic master: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python https://review.opendev.org/758762 | 18:19 |
*** KurtB has quit IRC | 18:35 | |
openstackgerrit | Merged openstack/ironic master: Run bifrost integration job on focal https://review.opendev.org/758198 | 18:39 |
*** happy-contrib-43 has joined #openstack-ironic | 18:41 | |
dking | TheJulia: Thank you. I just wanted to confirm. I'm back to trying to put Ironic in a container. At the moment, I'm doing some hacks with the entrypoint to modify the config with the variables. I'll make a note to check with oslo_config later. | 18:55 |
happy-contrib-43 | I'm looking to extend sushy-tools to support sessions. Is that something you guys would be interested in including in the project? | 19:04 |
*** eandersson has quit IRC | 19:10 | |
*** ccstone has quit IRC | 19:10 | |
*** ccstone has joined #openstack-ironic | 19:11 | |
*** eandersson has joined #openstack-ironic | 19:11 | |
openstackgerrit | Merged openstack/virtualbmc master: Use wallaby jobs https://review.opendev.org/758767 | 19:13 |
*** gyee has joined #openstack-ironic | 19:26 | |
*** ayoung has joined #openstack-ironic | 19:35 | |
ayoung | I registered 3 nodes with ironic, and IPMI does not power cycle them. THe difference between a working impitool command and the one generated by ironic is the work chassis. The machines are dell i610s. Should I have created the nodes inside a chassis structure in my nodes.json file? | 19:36 |
*** stevebaker has joined #openstack-ironic | 19:36 | |
*** rcernin has joined #openstack-ironic | 19:43 | |
*** rcernin has quit IRC | 19:48 | |
ayoung | Hmmm, nope. Seems like it still does not put the chassis in the command. | 19:57 |
ayoung | Is that a driver config option? | 19:59 |
TheJulia | blarg | 20:08 |
TheJulia | okay, re: audit, bifrost, metalsmith, pyghmi and ironic-staging-drivers only need a second set of eyes | 20:09 |
TheJulia | otherwise we're done and good it looks like | 20:09 |
TheJulia | happy-contrib-43: I'm sure we would accept the patches to sushy-tools | 20:09 |
TheJulia | ayoung: huh, could you share an example of what your seeing? | 20:10 |
*** akiselev has joined #openstack-ironic | 20:10 | |
ayoung | IN the conductor log I see the following command | 20:11 |
ayoung | Command: ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -f /tmp/tmpxbp36v09 power status | 20:11 |
ayoung | I run that (with my own password file) and I get | 20:11 |
ayoung | Error: Unable to establish IPMI v2 / RMCP+ session | 20:12 |
ayoung | but add the word chassis like this | 20:12 |
ayoung | ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -U root -f ~/ipmi.password chassis power status | 20:12 |
ayoung | and it works | 20:12 |
* TheJulia looks for her jaw on the floor | 20:14 | |
trandles | on my testbed it's `chassis power status` otherwise I get an error (regardless of password) | 20:17 |
TheJulia | I wonder if it is a version/fork issue | 20:18 |
TheJulia | because on my debian machine, "power status" works just fine | 20:18 |
TheJulia | and in the help it says it is just as hortcut to power status command | 20:19 |
mrquincy | Can I run Metal3 over ironic-standalone? | 20:22 |
TheJulia | mrquincy: functionally metal3 runs ironic-standalone, there is apparently a way to run it separately | 20:24 |
TheJulia | trandles: ayoung: ipmitool version, package version, and distribution please! | 20:24 |
*** yolanda has quit IRC | 20:24 | |
* TheJulia has started the great hunt through code and now the hunt through downstreams | 20:24 | |
*** yolanda has joined #openstack-ironic | 20:24 | |
ayoung | I'm on a RHEL 8 box...so.... | 20:25 |
ayoung | $ rpmquery -f `which ipmitool ` | 20:26 |
ayoung | ipmitool-1.8.18-14.el8.x86_64 | 20:26 |
ayoung | [ayoung@nuzleaf nodes]$ ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -U root -f ~/ipmi.password chassis power status | 20:27 |
ayoung | Chassis Power is off | 20:27 |
ayoung | [ayoung@nuzleaf nodes]$ ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -U root -f ~/ipmi.password power status | 20:27 |
ayoung | Chassis Power is off | 20:27 |
ayoung | wtf | 20:27 |
ayoung | [ayoung@nuzleaf nodes]$ ipmitool -I lanplus -H 192.168.123.10 -L ADMINISTRATOR -R 1 -N 1 -f ~/ipmi.password power status | 20:27 |
ayoung | Error: Unable to establish IPMI v2 / RMCP+ session | 20:27 |
ayoung | aH | 20:27 |
ayoung | I bet it is not that....sorry, I read that line 30 times and just now noticed the -U option | 20:28 |
TheJulia | heh | 20:28 |
ayoung | NOT the chassis thing | 20:28 |
trandles | oh no, I haven't been on my testbed since Monday and they did work in the data center...I can't reach it. As soon as it's back online I'll get you the data TheJulia | 20:28 |
TheJulia | ayoung: so you should be loggintg the username, so if you could provide the "baremetal node show" output | 20:29 |
TheJulia | that would be useful | 20:29 |
TheJulia | since usernames/passwords in ipmi are optional | 20:29 |
TheJulia | *bangs head into desk* | 20:29 |
TheJulia | trandles: no worries | 20:29 |
* ayoung knows what he did wrong | 20:30 | |
ayoung | I'll confirm but | 20:30 |
ayoung | (reverse-i-search)`for': for UUID in `openstack baremetal node list -f json | jq -r '.[] | .UUID' ` ; do openstack baremetal node set $UUID --driver-info ipmi_password=`cat ~/ipmi.password` --driver-info ipmi_user=admin ; done | 20:30 |
ayoung | yep | 20:30 |
ayoung | admin should be root | 20:30 |
TheJulia | ipmi_username | 20:30 |
ayoung | trying that now | 20:32 |
ayoung | | driver_info | {'ipmi_address': '192.168.123.10', 'ipmi_password': '******', 'ipmi_user': 'root', 'ipmi_username': 'root'} | 20:33 |
ayoung | That worked. I can hear the fans spinning up. Thanks | 20:33 |
NobodyCam | Good Morning Ironic folks | 20:34 |
ayoung | It does not show the command in the log if it succeeded, just if it failed, but I bet the -L ADMINISDTAOR was due to me not sending the proper ipmi_username parameter. | 20:34 |
akiselev | Hi All! I just joined your channel following dtantsur's guidance. I have a question on Ironic ramdisk logs filename format. Would it be possible to add an actual node name in addition to the default UUID? That nodename may be optional and not affecting the default. The code for ironic-inspector I am talking about: https://opendev.org/openstack/ironic-inspector/src/branch/master/ironic_inspector/conf/processing.py#L87-L94 | 20:35 |
ayoung | Oh yeah...its getting noisy in here. | 20:35 |
TheJulia | ayoung: \o/ | 20:35 |
TheJulia | ayoung: this is why my supermicro boxes are in the garage | 20:36 |
TheJulia | o/ NobodyCam | 20:36 |
TheJulia | akiselev: actual node name as in what dhcp told the machine? | 20:36 |
ayoung | \m/ d(>_<)b <\m/ | 20:37 |
NobodyCam | :) o/ | 20:37 |
ayoung | My garage leaks. I am on my third iteration of digging up the foundation | 20:38 |
akiselev | Yes, I think it is where we get in initially. And later it can be mapped with: ironic node-show --fields name <node UUID> | 20:38 |
anuradha1904 | Hi TheJulia, sorry to disturb you considering you are highly busy. However, if you have some time today, can you check this https://review.opendev.org/#/c/756565/ patch, it has been already approved by iurygregory and Steve Baker, Can we get it merged? Thank you very much. | 20:38 |
patchbot | patch 756565 - ironic-inspector - Replaced openstack baremetal to baremetal in the d... - 2 patch sets | 20:38 |
TheJulia | akiselev: so is it that you want discovered node names to be populated? | 20:39 |
TheJulia | ultimately? | 20:39 |
akiselev | TheJulia: Correct | 20:39 |
TheJulia | I say go for it! | 20:39 |
TheJulia | create an entry against ironic-inspector on storyboard.openstack.org | 20:40 |
TheJulia | and send me the link after describing what you want to do and I'll ack that its a good idea | 20:40 |
akiselev | What is your estimate for this story - how big is the change? | 20:41 |
TheJulia | simple really, you'd have to extract or record it if it is not already record it, and then change ironic-inspector to post a name as well. | 20:42 |
TheJulia | on the node create, it shouldn't be that hard | 20:43 |
akiselev | Is that still true for ironic as well: https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/utils.py#L266-L282 | 20:45 |
TheJulia | that would be like 3 lines of code and maybe 30 lines of testing | 20:45 |
TheJulia | since it would already be on the node object at that point if posted | 20:46 |
akiselev | Great, thank you! | 20:47 |
* TheJulia looks around for spoons | 20:48 | |
ayoung | for UUID in `openstack baremetal node list -f json | jq -r '.[] | .UUID' ` ; do openstack baremetal node power off $UUID ; done | 20:53 |
ayoung | Much quieter in here now. | 20:53 |
* ayoung happy | 20:53 | |
TheJulia | heh | 20:54 |
trandles | I've just been informed my rack PDU didn't survive the data center work /o\ | 21:03 |
TheJulia | :( | 21:05 |
TheJulia | https://giphy.com/gifs/harry-potter-pottermore-M28rUlcjueKUE | 21:06 |
trandles | I should have known. This week's been going pretty well. | 21:07 |
TheJulia | :( | 21:07 |
trandles | The good news is the electricians have been deployed. Hopefully I'm back online in a day or two. | 21:08 |
TheJulia | was that via a module?!? | 21:17 |
TheJulia | :) | 21:17 |
openstackgerrit | Merged openstack/ironic-inspector master: Replaced openstack baremetal to baremetal in the documentation Changed osc_plugin_cli.html to index.html https://review.opendev.org/756565 | 21:23 |
JayF | Would folks be OK with a patch to IPA to enable manually setting Ironic API version and disabling negotiation? Use case would be locked down API access from node side, and not wanting to enable / as it makes the API more discoverable. | 21:31 |
TheJulia | yeah, I think that is fine | 21:32 |
JayF | ack; I'll have a patch for it today or early tomorrow | 21:32 |
trandles | w00t! The electrician found a breaker tripped on the PDU that my Ops guy missed. I'm back online! | 21:35 |
trandles | TheJulia: I know what it was with ipmitool and power state. The testbed is a old greenblade system. The blade _chassis_ required 'chassis power status' but the individual nodes are fine with just 'power status' | 21:36 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: IPMI: Handle vendor set boot device differences https://review.opendev.org/757198 | 21:44 |
*** sai438 has joined #openstack-ironic | 21:52 | |
*** pmannidi has quit IRC | 21:53 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Tear down raid sets https://review.opendev.org/757141 | 22:21 |
*** rloo has quit IRC | 22:28 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig https://review.opendev.org/752308 | 22:28 |
*** DjeufackZane has joined #openstack-ironic | 22:28 | |
*** akiselev has quit IRC | 22:29 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 22:29 |
*** rcernin has joined #openstack-ironic | 22:30 | |
*** rcernin has quit IRC | 22:49 | |
*** rcernin has joined #openstack-ironic | 22:51 | |
*** tosky has quit IRC | 22:54 | |
*** DjeufackZane has quit IRC | 22:58 | |
*** rh-jelabarre has quit IRC | 23:15 | |
*** mrquincy has quit IRC | 23:17 | |
*** benj_ has quit IRC | 23:21 | |
*** benj_ has joined #openstack-ironic | 23:26 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!