| *** k_mouza has quit IRC | 00:04 | |
| *** Goneri has quit IRC | 00:15 | |
| *** k_mouza has joined #openstack-ironic | 00:22 | |
| *** k_mouza has quit IRC | 00:26 | |
| *** k_mouza has joined #openstack-ironic | 00:32 | |
| *** k_mouza has quit IRC | 00:36 | |
| *** JamesBenson has quit IRC | 00:42 | |
| *** JamesBenson has joined #openstack-ironic | 00:43 | |
| *** JamesBenson has quit IRC | 00:47 | |
| *** larsks has quit IRC | 00:50 | |
| *** larsks has joined #openstack-ironic | 00:54 | |
| *** zzzeek has quit IRC | 01:10 | |
| *** zzzeek has joined #openstack-ironic | 01:11 | |
| *** k_mouza has joined #openstack-ironic | 01:13 | |
| *** outbrito has joined #openstack-ironic | 01:17 | |
| *** k_mouza has quit IRC | 01:17 | |
| *** JamesBenson has joined #openstack-ironic | 01:20 | |
| *** JamesBenson has quit IRC | 01:24 | |
| *** outbrito has quit IRC | 01:25 | |
| *** k_mouza has joined #openstack-ironic | 01:26 | |
| *** k_mouza has quit IRC | 01:30 | |
| *** Qianbiao has joined #openstack-ironic | 01:33 | |
| openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Corrected merge conflicts https://review.opendev.org/756942 | 01:43 |
|---|---|---|
| *** smutugrigg has joined #openstack-ironic | 01:46 | |
| *** k_mouza has joined #openstack-ironic | 01:57 | |
| smutugrigg | hi, Michelle (Outreachy applicant here) have just committed a update on a last commit as there were conflicts with another commit. Would anyone be able to have a look and let me know if I have resolved the issues? https://review.opendev.org/#/c/756942/ | 02:01 |
| patchbot | patch 756942 - ironic - Corrected merge conflicts - 1 patch set | 02:01 |
| *** k_mouza has quit IRC | 02:01 | |
| *** smutugrigg has quit IRC | 02:04 | |
| janders | @smutugrigg thank you for the patch Michelle. This is continuation of https://review.opendev.org/#/c/756708/ correct? | 02:13 |
| patchbot | patch 756708 - ironic - Update .rst files to remove --os-baremetal-api-ver... - 1 patch set | 02:13 |
| janders | It seems to me it came through as a new change | 02:15 |
| janders | if I understand correctly it should have been another revision of the same change | 02:17 |
| janders | @smutugrigg have a look at https://docs.opendev.org/opendev/infra-manual/latest/sandbox.html and https://gerrit-review.googlesource.com/Documentation/intro-gerrit-walkthrough.html#_reworking_the_change - "git commit --amend" should allow you to submit it as a revision of the existing change | 02:17 |
| janders | @smutugrigg it may be worthwhile walking through this tutorial https://docs.opendev.org/opendev/infra-manual/latest/sandbox.html start to finish, I think it gives a good overview of the process | 02:19 |
| janders | hope this helps :) | 02:22 |
| *** k_mouza has joined #openstack-ironic | 02:23 | |
| *** k_mouza has quit IRC | 02:27 | |
| *** JamesBenson has joined #openstack-ironic | 02:30 | |
| *** JamesBenson has quit IRC | 02:34 | |
| *** k_mouza has joined #openstack-ironic | 02:51 | |
| *** k_mouza has quit IRC | 02:55 | |
| *** JamesBenson has joined #openstack-ironic | 03:05 | |
| *** JamesBenson has quit IRC | 03:10 | |
| *** rcernin has quit IRC | 03:17 | |
| *** Qianbiao has quit IRC | 03:29 | |
| *** smutugrigg has joined #openstack-ironic | 03:39 | |
| *** rcernin has joined #openstack-ironic | 03:55 | |
| *** rcernin has quit IRC | 03:56 | |
| *** ricolin_ has joined #openstack-ironic | 04:01 | |
| *** rcernin has joined #openstack-ironic | 04:03 | |
| *** uzumaki has quit IRC | 04:05 | |
| *** tzumainn has quit IRC | 04:12 | |
| *** pmannidi has quit IRC | 04:16 | |
| *** mpsairam has joined #openstack-ironic | 04:16 | |
| openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 04:19 |
| openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 04:21 |
| *** Qianbiao has joined #openstack-ironic | 04:21 | |
| openstackgerrit | Manuel Schönlaub proposed openstack/sushy master: Adds basic support for TaskService to retrieve task information. https://review.opendev.org/756906 | 04:23 |
| *** zzzeek has quit IRC | 05:02 | |
| *** smutugrigg has quit IRC | 05:04 | |
| *** zzzeek has joined #openstack-ironic | 05:05 | |
| *** Qianbiao has quit IRC | 05:18 | |
| *** zzzeek has quit IRC | 05:42 | |
| *** zzzeek has joined #openstack-ironic | 05:43 | |
| *** rpittau|afk is now known as rpittau | 05:43 | |
| rpittau | good morning ironic! o/ | 05:43 |
| *** kafilat has joined #openstack-ironic | 05:46 | |
| *** sshnaidm is now known as sshnaidm|off | 06:06 | |
| *** zzzeek has quit IRC | 06:11 | |
| *** zzzeek has joined #openstack-ironic | 06:12 | |
| arne_wiebalck | Good morning, rpittau and ironic! | 06:34 |
| *** tkajinam has quit IRC | 06:42 | |
| *** tkajinam has joined #openstack-ironic | 06:42 | |
| *** Akki58 has joined #openstack-ironic | 06:55 | |
| Akki58 | Hello All, | 06:55 |
| Akki58 | and goes into second time booting and gets stuck there in "Probing" state ( Seen on node's console) BUT according to openstack horizon, it is up and running and according to "openstack baremetal node show", it is in "Active" state. | 06:55 |
| Akki58 | again and then comes up, the "spawning" state on openstack horizon goes into ERROR. | 06:55 |
| Akki58 | for callback for node 75210cc4-ad98-442d-ace1-89ce69467580" | 06:55 |
| *** jtomasek has joined #openstack-ironic | 06:59 | |
| rpittau | hey arne_wiebalck :) | 07:00 |
| rpittau | Akki58: hey, good morning, so is it just a problem on horizon ui ? or you can't access the node? you should check the console logs if possible, and also conductor logs | 07:03 |
| Akki58 | Hi rpittau, Good morning. Thank you for your reply. When it is up on horizon, the baremetal console is stuck on "Probing EDD (edd=off to disable) ... ok | 07:07 |
| rpittau | Akki58: would be also useful to have a look at the ironic-python-agent ramdisk logs, check all of those I mentioned, you will probably get some hints on what's going on | 07:08 |
| *** uzumaki has joined #openstack-ironic | 07:10 | |
| *** smutugrigg has joined #openstack-ironic | 07:11 | |
| iurygregory | good morning arne_wiebalck rpittau and Ironic | 07:22 |
| iurygregory | happy friday | 07:22 |
| rpittau | hey iurygregory :) | 07:22 |
| rpittau | Happy Friday! | 07:22 |
| *** Qianbiao has joined #openstack-ironic | 07:31 | |
| Akki58 | rpittau, I searched for logs and found no errors in ironic-conductor: | 07:36 |
| Akki58 | According to ironic-conductor it says: | 07:37 |
| Akki58 | INFO ironic.conductor.deployments [req-6c388902-2862-4756-a1cb-b67984962205 - - - - -] Successfully deployed node 75210cc4-ad98-442d-ace1-89ce69467580 with instance 712633e5-9631-41df-af0f-beedade68a36. | 07:37 |
| smutugrigg | good morning iurygregory | 07:37 |
| Akki58 | Also i can't find ironic-python-agent ramdisk logs. Can you please guide me how to get those? | 07:41 |
| smutugrigg | iurygregory, could I request some assistance please? | 07:43 |
| iurygregory | smutugrigg, morning, sure | 07:44 |
| smutugrigg | I had merge conflicts in my commit, that I think I have resolved, but recieved a build failed from Dell | 07:44 |
| smutugrigg | How do I work out if I have corrected the merge conflicts or not? | 07:45 |
| *** rcernin has quit IRC | 07:45 | |
| iurygregory | oh you submitted another change correct? | 07:46 |
| *** tosky has joined #openstack-ironic | 07:46 | |
| smutugrigg | I thought I amended, but I may have submitted a second change | 07:46 |
| iurygregory | before was https://review.opendev.org/#/c/756708/ and you sent another patch https://review.opendev.org/#/c/756942/ because of the conflict you had | 07:46 |
| patchbot | patch 756708 - ironic - Update .rst files to remove --os-baremetal-api-ver... - 1 patch set | 07:46 |
| patchbot | patch 756942 - ironic - Corrected merge conflicts - 1 patch set | 07:46 |
| iurygregory | no worries =) | 07:46 |
| smutugrigg | :) | 07:47 |
| iurygregory | let me check here | 07:47 |
| smutugrigg | thanks | 07:47 |
| iurygregory | smutugrigg, seems like you fixed the merge conflicts you had, I'm adding a few comments that you will need address ok? | 07:57 |
| smutugrigg | fantastic - thank you | 07:57 |
| rpittau | well inspector grenade is broken | 08:00 |
| *** lucasagomes has joined #openstack-ironic | 08:10 | |
| iurygregory | smutugrigg, I've posted the comments =) | 08:14 |
| smutugrigg | Thanks for the detail iurygregory, much appreciated. Will work on this. | 08:17 |
| iurygregory | smutugrigg, if you have any doubts on how to ammend the change feel free to ask | 08:17 |
| smutugrigg | I think I might need some help to make sure I do it correctly | 08:18 |
| smutugrigg | Will come back to the chat once I am there | 08:18 |
| iurygregory | cool =) | 08:18 |
| *** k_mouza has joined #openstack-ironic | 08:20 | |
| *** derekh has joined #openstack-ironic | 08:28 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE for grenade https://review.opendev.org/756705 | 08:28 |
| *** k_mouza_ has joined #openstack-ironic | 08:28 | |
| *** k_mouza has quit IRC | 08:31 | |
| iurygregory | rpittau, I loved the "recheck sqlalchemy shenanigan" | 08:40 |
| rpittau | heh | 08:42 |
| *** dtantsur|afk is now known as dtantsur | 08:44 | |
| smutugrigg | iurygregory, hello o/ | 08:51 |
| *** ociuhandu has joined #openstack-ironic | 08:51 | |
| iurygregory | smutugrigg, o/ | 08:51 |
| smutugrigg | I think I have now completed the updates and merged all the chnages from the master into my branch | 08:51 |
| smutugrigg | I have also commited using git commit -a --amend | 08:52 |
| smutugrigg | whichc shows me openstack-ironic-task-40964 197e3c051] Update .rst files remove -os-baremetal-api-version | 08:52 |
| smutugrigg | Date: Fri Oct 9 21:47:05 2020 +1300 | 08:52 |
| *** Akki58 has quit IRC | 08:52 | |
| smutugrigg | Should I be OK to submit git review? | 08:52 |
| smutugrigg | qit = git | 08:53 |
| iurygregory | have you added Task and Story in the commit message? | 08:53 |
| smutugrigg | yes I have | 08:53 |
| iurygregory | cool | 08:53 |
| iurygregory | go ahead and do git review | 08:53 |
| smutugrigg | same message as from 756708 with minor additions | 08:53 |
| iurygregory | ok =) | 08:53 |
| dtantsur | morning ironic | 08:55 |
| iurygregory | morning dtantsur | 08:55 |
| rpittau | hey dtantsur :) | 08:55 |
| iurygregory | standalone job hate us all | 08:59 |
| *** ricolin_ has quit IRC | 09:04 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Do not silently swallow errors in the write_image deploy step https://review.opendev.org/757037 | 09:24 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: DNM break prepare_image https://review.opendev.org/757038 | 09:25 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Replace retrying with tenacity https://review.opendev.org/376574 | 09:26 |
| *** jtomasek has quit IRC | 09:29 | |
| *** fmuyassarov has joined #openstack-ironic | 09:33 | |
| smutugrigg | hi iurygregory o/ | 09:36 |
| fmuyassarov | Hi @dtantsur, I have asked earlier a way to avoid disk cleaning as part of deprovisioning process. And I remember you suggested to disable it by setting erase_devices_metadata_priority and erase_devices_priority to zero. And my question is, can I avoid disk cleaning by setting automated_clean to False instead of setting 0 values for erase* fields? Thanks. | 09:40 |
| iurygregory | smutugrigg, hi | 09:42 |
| Qianbiao | Hello ironic. | 09:48 |
| dtantsur | fmuyassarov: yes, you can | 09:48 |
| fmuyassarov | Thanks! | 09:50 |
| Qianbiao | hello, is there a command can transfer node status from error to manageable again? | 09:51 |
| dtantsur | Qianbiao: check here https://docs.openstack.org/ironic/latest/contributor/states.html | 09:52 |
| openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files remove -os-baremetal-api-version https://review.opendev.org/756708 | 09:53 |
| smutugrigg | iurygregory, so I think I have completed the update | 09:54 |
| smutugrigg | but only to https://review.opendev.org/756708 | 09:54 |
| patchbot | patch 756708 - ironic - Update .rst files remove -os-baremetal-api-version - 2 patch sets | 09:54 |
| smutugrigg | I have merged the requested changes from https://review.opendev.org/756942 as well | 09:54 |
| patchbot | patch 756942 - ironic - Corrected merge conflicts - 1 patch set | 09:54 |
| smutugrigg | had a bit of trouble with rebasing ;) | 09:55 |
| Qianbiao | dtantsur i try delete, but it's not allowed. | 09:55 |
| dtantsur | Qianbiao: you mean, undeploy? | 09:55 |
| Qianbiao | yes | 09:56 |
| Qianbiao | http://paste.openstack.org/show/798901/ ^^ | 09:56 |
| *** lucasagomes has quit IRC | 09:56 | |
| *** lucasagomes has joined #openstack-ironic | 09:57 | |
| *** lucasagomes has quit IRC | 09:57 | |
| *** lucasagomes has joined #openstack-ironic | 09:58 | |
| iurygregory | smutugrigg, cool! I will look at it | 09:58 |
| smutugrigg | thanks heaps iurygregory, very much appreciated | 09:59 |
| Qianbiao | dtantsur sorry, my bad, there has a "undeploy" command, i was using delete. | 09:59 |
| dtantsur | delete literally deletes the node, it's not quite what you want | 10:00 |
| Qianbiao | in the status machine picture, it show delete :) | 10:00 |
| Qianbiao | so. i try to run delete command. | 10:00 |
| fmuyassarov | @dtantsur, sorry I forgot to mention one thing.In our setup erase-devices-metadata-priority will have some non-zero value. So, even though erase-devices-metadata-priority has a value greater than zero, I still can avoid disk cleaning by having false for automated-clean right? | 10:02 |
| dtantsur | fmuyassarov: yep, automated_clean=False makes the whole cleaning process completely no-op | 10:02 |
| fmuyassarov | Great. Thank you very much! | 10:02 |
| *** fmuyassarov has quit IRC | 10:06 | |
| rpittau | fixed inspector grenade -> https://review.opendev.org/756705 | 10:13 |
| patchbot | patch 756705 - ironic-inspector - Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE for ... - 3 patch sets | 10:13 |
| openstackgerrit | likui proposed openstack/ironic-inspector master: bump py37 to py38 in tox.ini https://review.opendev.org/757069 | 10:15 |
| iurygregory | rpittau, awesome | 10:16 |
| iurygregory | smutugrigg, only a few more things and it should be good =) | 10:19 |
| iurygregory | if you have problems running pep8, docs locally let me know | 10:20 |
| smutugrigg | thanks iurygregory for your patience, I will get there in the end | 10:23 |
| iurygregory | smutugrigg, yw! good luck =) | 10:23 |
| openstackgerrit | likui proposed openstack/ironic-inspector master: bump py37 to py38 in tox.ini https://review.opendev.org/757069 | 10:31 |
| iurygregory | the commit message doesn't make sense .-. | 10:33 |
| *** Qianbiao has quit IRC | 10:36 | |
| smutugrigg | hi iurygregory o/ | 10:48 |
| iurygregory | smutugrigg, o/ | 10:48 |
| smutugrigg | for the pep8 | 10:48 |
| iurygregory | sure | 10:48 |
| smutugrigg | is the command 'tox -epep8 docs'? | 10:48 |
| iurygregory | tox -r -vv -e pep8 | 10:49 |
| smutugrigg | thank you | 10:49 |
| iurygregory | and for docs is tox -r -vv -e docs | 10:49 |
| smutugrigg | ah ha, thank you | 10:49 |
| *** priteau has joined #openstack-ironic | 10:49 | |
| *** rcernin has joined #openstack-ironic | 10:50 | |
| *** zzzeek has quit IRC | 10:52 | |
| openstackgerrit | Merged openstack/bifrost master: Use upper constraints when installing Bifrost requirements https://review.opendev.org/756754 | 10:53 |
| *** zzzeek has joined #openstack-ironic | 10:54 | |
| *** Qianbiao has joined #openstack-ironic | 10:56 | |
| smutugrigg | hi iurygregory, could I request some help with some diagnosis please? | 10:56 |
| iurygregory | smutugrigg, sure | 10:56 |
| smutugrigg | pep8 is passing, however docs is erroring | 10:56 |
| smutugrigg | InvocationError for command /home/michelle/openstack/ironic/.tox/docs/bin/sphinx-build -b html -W doc/source doc/build/html (exited with code 2) | 10:56 |
| iurygregory | can you paste the error in http://paste.openstack.org/ | 10:56 |
| iurygregory | oh seems like you are missing some packages | 10:57 |
| smutugrigg | pasted to URL as well | 10:57 |
| smutugrigg | did I miss a command during setup? | 10:57 |
| iurygregory | maybe, what OS are you using? | 10:58 |
| smutugrigg | ubuntu 20.04.1 | 10:58 |
| iurygregory | https://www.sphinx-doc.org/en/master/usage/installation.html | 10:59 |
| iurygregory | try installing python3-sphinx | 10:59 |
| smutugrigg | ok | 10:59 |
| iurygregory | and re run the tox for docs | 10:59 |
| smutugrigg | hi iurygregory, same error :( | 11:04 |
| smutugrigg | Also says | 11:04 |
| smutugrigg | RSVG converter command 'rsvg-convert' cannot be run. Check the rsvg_converter_bin setting | 11:04 |
| dtantsur | smutugrigg: you need to install it, it's librsvg2-tools on Fedora, for example | 11:05 |
| smutugrigg | Ah ha, thank you dtantsur, will attempt | 11:06 |
| iurygregory | yeah | 11:06 |
| iurygregory | librsvg package probably | 11:06 |
| dtantsur | iurygregory: could you check https://review.opendev.org/#/c/757037/ please? it's nasty enough to require a new V release. | 11:10 |
| patchbot | patch 757037 - ironic-python-agent - Do not silently swallow errors in the write_image ... - 1 patch set | 11:10 |
| iurygregory | dtantsur, wow | 11:10 |
| smutugrigg | thank you kindly iurygregory & dtantsur, it was librsvg2-bin | 11:11 |
| iurygregory | smutugrigg, cool! | 11:11 |
| smutugrigg | all tox commands now green with cngrats | 11:11 |
| smutugrigg | yah! | 11:11 |
| smutugrigg | small wins | 11:11 |
| iurygregory | \o/ | 11:11 |
| iurygregory | dtantsur, lgtm +2 (will wait for the CI) | 11:12 |
| * iurygregory goes for lunch | 11:12 | |
| *** zzzeek has quit IRC | 11:17 | |
| *** zzzeek has joined #openstack-ironic | 11:18 | |
| arne_wiebalck | FYI reducing the IPA image size seems to address some OOM situations in grub (this is with UEFI bootmode). YMMV as the grub file we use is not brand new. | 11:26 |
| *** jtomasek has joined #openstack-ironic | 11:31 | |
| *** rcernin has quit IRC | 11:31 | |
| *** shreya11 has quit IRC | 11:34 | |
| *** dking has quit IRC | 11:34 | |
| *** bdodd has quit IRC | 11:34 | |
| *** jamesdenton has quit IRC | 11:34 | |
| *** rajinir has quit IRC | 11:34 | |
| *** shreya11 has joined #openstack-ironic | 11:34 | |
| *** outbrito has joined #openstack-ironic | 11:38 | |
| *** bdodd has joined #openstack-ironic | 11:38 | |
| *** dking has joined #openstack-ironic | 11:40 | |
| *** jamesdenton has joined #openstack-ironic | 11:40 | |
| *** rajinir has joined #openstack-ironic | 11:40 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Use direct as IRONIC_DEFAULT_DEPLOY_INTERFACE for grenade https://review.opendev.org/756705 | 11:47 |
| openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Corrected merge conflicts https://review.opendev.org/756942 | 11:48 |
| openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files https://review.opendev.org/756708 | 11:48 |
| smutugrigg | hi iurygregory o/ | 11:49 |
| smutugrigg | for some reason I could not abandon the https://review.opendev.org/756942 commit and commited it with https://review.opendev.org/756708 | 11:50 |
| patchbot | patch 756942 - ironic - Corrected merge conflicts - 2 patch sets | 11:50 |
| patchbot | patch 756708 - ironic - Update .rst files - 3 patch sets | 11:50 |
| smutugrigg | I hope this is OK. All tox unit tests passing | 11:51 |
| iurygregory | smutugrigg, it's because you are using https://review.opendev.org/#/c/756942/ as base for https://review.opendev.org/#/c/756708/ | 11:55 |
| patchbot | patch 756942 - ironic - Corrected merge conflicts - 2 patch sets | 11:55 |
| patchbot | patch 756708 - ironic - Update .rst files - 3 patch sets | 11:55 |
| iurygregory | to fix this in the branch you are working on https://review.opendev.org/#/c/756708 you need to run git rebase -i origin/master | 11:56 |
| patchbot | patch 756708 - ironic - Update .rst files - 3 patch sets | 11:56 |
| iurygregory | and fix any problems it shows when doing the rebase | 11:56 |
| rpittau | arne_wiebalck: that's probably because of the reduced memory need | 12:10 |
| arne_wiebalck | rpittau: Yes. But newer versions of grub or different f/w may handle this differently. | 12:12 |
| rpittau | true | 12:12 |
| arne_wiebalck | rpittau: And I a not sure there is an actual limit of the amount of RAM. Seems like grub does not have such a limit. | 12:12 |
| arne_wiebalck | rpittau: All this may also have a completely different cause as I observed the same node with the same image sometimes working, sometimes failing. | 12:13 |
| arne_wiebalck | rpittau: Could be sth with the PXE server or some memory on the node that is not freed properly ... | 12:14 |
| rpittau | yeah, I was not thinking of a limit on grub side, but more on hwo the image is uncompressed and transfered | 12:14 |
| *** Qianbiao has quit IRC | 12:22 | |
| openstackgerrit | Michelle (Shelly) Mutu-Grigg proposed openstack/ironic master: Update .rst files https://review.opendev.org/756708 | 12:22 |
| *** Qianbiao has joined #openstack-ironic | 12:22 | |
| smutugrigg | hi iurygregory, I think I finally have it....hopefully | 12:24 |
| *** smutugrigg has quit IRC | 12:29 | |
| *** vilitho has joined #openstack-ironic | 12:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent stable/victoria: Do not silently swallow errors in the write_image deploy step https://review.opendev.org/757091 | 12:34 |
| *** jtomasek has quit IRC | 12:44 | |
| Qianbiao | Hello, when undeploy and redeploy node frequency, the arp table is not refresh, is this normal. | 12:46 |
| Qianbiao | ^^^ http://paste.openstack.org/show/798909/ | 12:46 |
| *** lmcgann has joined #openstack-ironic | 12:50 | |
| rpittau | closing early today, have a great weekend everyone! o/ | 13:03 |
| *** rpittau is now known as rpittau|afk | 13:03 | |
| arne_wiebalck | bye rpittau|afk o/ | 13:03 |
| *** outbrito has quit IRC | 13:09 | |
| *** uzumaki has quit IRC | 13:18 | |
| *** tzumainn has joined #openstack-ironic | 13:19 | |
| iurygregory | bye rpittau|afk | 13:24 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Do not silently swallow errors in the write_image deploy step https://review.opendev.org/757037 | 13:31 |
| vilitho | Hello everyone, my name is Valerie from Nigeria and I'm an Outreachy applicant. For my first task, I picked #40969. Its the one that requires that I validate bifrosts installation documentation. I am doing this using via the WSL feature on windows and i have run into a libvirt problem. From browsing online i realize its very likely because i am running ubuntu virtually but so far i haven't been able to fix this problem and i'd really | 13:35 |
| vilitho | appreciate any help /pointers. Thanks :D | 13:35 |
| *** Goneri has joined #openstack-ironic | 13:36 | |
| vilitho | I am doing this using ubuntu* | 13:37 |
| Qianbiao | <vilitho> wsl2 or wsl? | 13:38 |
| Qianbiao | wsl is not a original ubuntu i think. | 13:38 |
| Qianbiao | Some package is not supported on wsl | 13:39 |
| vilitho | wsl2 | 13:39 |
| vilitho | Right. But if i were to now install ubuntu on vmware for example, that should fix it right? | 13:44 |
| vilitho | I'll just test that out and see. | 13:46 |
| *** ociuhandu has quit IRC | 14:11 | |
| *** ociuhandu has joined #openstack-ironic | 14:12 | |
| *** jtomasek has joined #openstack-ironic | 14:12 | |
| *** jtomasek has quit IRC | 14:12 | |
| *** vilitho has quit IRC | 14:16 | |
| *** vilitho has joined #openstack-ironic | 14:16 | |
| iurygregory | Hi vilitho welcome! | 14:25 |
| *** k_mouza_ has quit IRC | 14:25 | |
| iurygregory | I would say it would be more easy if you have a machine with Ubuntu / Fedora / CentOS to validate | 14:26 |
| *** k_mouza has joined #openstack-ironic | 14:26 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Log the ports we bind in flat https://review.opendev.org/757115 | 14:28 |
| * TheJulia does the kahn type scream because we don't log the vifs in rescue | 14:29 | |
| openstackgerrit | Merged openstack/ironic stable/ussuri: Increase memory of tinyipa vms https://review.opendev.org/747633 | 14:34 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/victoria: Use upper constraints when installing Bifrost requirements https://review.opendev.org/757116 | 14:37 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/victoria: Use upper constraints when installing Bifrost requirements https://review.opendev.org/757116 | 14:38 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/ussuri: Use upper constraints when installing Bifrost requirements https://review.opendev.org/757117 | 14:39 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/train: Use upper constraints when installing Bifrost requirements https://review.opendev.org/757118 | 14:41 |
| openstackgerrit | Mark Goddard proposed openstack/bifrost stable/ussuri: Use upper constraints when installing Bifrost requirements https://review.opendev.org/757117 | 14:41 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: docs: Add information on post-branch release tasks for bifrost https://review.opendev.org/757120 | 14:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Do not use RPC continue_node_deploy/clean in agent_base https://review.opendev.org/756354 | 14:51 |
| *** k_mouza has quit IRC | 14:52 | |
| *** k_mouza has joined #openstack-ironic | 15:04 | |
| TheJulia | did we ever validate if cirros _could_ boot to a raidset? | 15:05 |
| TheJulia | I'm guessing lots of nope | 15:05 |
| dtantsur | TheJulia: you mean, software RAID? did not work the last time I tried. | 15:06 |
| TheJulia | that is what I figured | 15:07 |
| TheJulia | I think I know why rescue keeps failing on victoria | 15:07 |
| dtantsur | do I want to know? :) | 15:07 |
| TheJulia | the deploy is ending up on the raidset | 15:07 |
| * TheJulia is tryign to figure out why it is sitll around or still being used | 15:07 | |
| dtantsur | yeah, we should be removing it | 15:07 |
| * dtantsur looks at agent_base.py and wants to scream | 15:08 | |
| TheJulia | looks like we're orphaning raid_config :\ | 15:11 |
| dtantsur | we what? Oo | 15:11 |
| TheJulia | And I don't think we're cleaning so it stays around :( | 15:12 |
| * dtantsur ponders declaring wallaby a technical debt cycle | 15:12 | |
| TheJulia | I was already thinking along those lines, tbh | 15:12 |
| dtantsur | except that everyone wants active steps.. which will make everything a much bigger mess | 15:12 |
| TheJulia | ugh | 15:12 |
| dtantsur | I'll probably keep refactoring the guts of the agent deploy so that it's not so bad in the end | 15:13 |
| dtantsur | sad we cannot remove the iscsi deploy right now | 15:13 |
| TheJulia | ugh, yeah, cleaning is off | 15:14 |
| * TheJulia looks at the tempest plugin | 15:14 | |
| * dtantsur wonders how many more regressions he'll introduce in the process | 15:15 | |
| TheJulia | heh, I have a note saying we should do this in the tempest job | 15:17 |
| iurygregory | hehehe | 15:18 |
| * TheJulia turns on cleaning | 15:19 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Do not use RPC continue_node_deploy/clean in agent_base https://review.opendev.org/756354 | 15:27 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: CI: Turn on cleaning on standalone jobs https://review.opendev.org/757132 | 15:27 |
| TheJulia | ^ -- is likely the source of many of our standalone failures... if not ALL of ours present standalone failures | 15:28 |
| *** k_mouza has quit IRC | 15:28 | |
| iurygregory | yay | 15:30 |
| *** zaneb has quit IRC | 15:32 | |
| *** zaneb has joined #openstack-ironic | 15:33 | |
| TheJulia | even _IF_ we explicitly called for the raidset to be nuked from high orbit by the tempest test, cleaning would still need to be enabled for it to work. We may need to change it so it is just a metadata erase though | 15:34 |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Do not silently swallow errors in the write_image deploy step https://review.opendev.org/757091 | 15:34 |
| TheJulia | I _think_ that nukes everything with arne_wiebalck's recent chagnes | 15:34 |
| *** vilitho has quit IRC | 15:35 | |
| *** k_mouza has joined #openstack-ironic | 15:35 | |
| * arne_wiebalck wakes up | 15:35 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: docs: Add information on post-branch release tasks for bifrost https://review.opendev.org/757120 | 15:35 |
| * TheJulia puts coffee in front of arne_wiebalck | 15:35 | |
| iurygregory | arne_wiebalck will send cookies \o/ | 15:35 |
| TheJulia | wait, is he baking cookies?!? | 15:35 |
| arne_wiebalck | such a nice community! | 15:35 |
| iurygregory | he will \o/ | 15:35 |
| iurygregory | arne_wiebalck is a nice person =D | 15:35 |
| TheJulia | ++ | 15:35 |
| arne_wiebalck | ha, iurygregory, getting cookies is not that easy! ;) | 15:36 |
| arne_wiebalck | my recent changes broke sth? | 15:36 |
| dtantsur | TheJulia: if you have a second: https://review.opendev.org/#/c/757114/ | 15:36 |
| iurygregory | arne_wiebalck, you are only going to share the cookies with zuul?! | 15:36 |
| patchbot | patch 757114 - x/ironic-staging-drivers (stable/victoria) - Prepare stable/victoria - 1 patch set | 15:36 |
| iurygregory | =OO | 15:36 |
| arne_wiebalck | zuul tests my patches | 15:36 |
| TheJulia | arne_wiebalck: no, Your change likely reduced the risk of a failure, we have a race condition in the standalone job sequence where we're creating a raid set in one job, not running cleaning, and then provisioning the next job on the machine | 15:38 |
| TheJulia | err, same node | 15:38 |
| TheJulia | which ends up choosing the already created and present raid set | 15:38 |
| dtantsur | TheJulia: automated cleaning does not remove software RAID (by design) | 15:39 |
| dtantsur | but I think we have some code to run manual cleaning to wipe it? | 15:39 |
| *** k_mouza has quit IRC | 15:39 | |
| TheJulia | errr | 15:40 |
| TheJulia | well this stinks | 15:40 |
| dtantsur | heh | 15:40 |
| arne_wiebalck | would it make sense to make RAID cleaning part of automatic cleaning? | 15:40 |
| TheJulia | the tempest test likely needs to explicitly tear it down | 15:40 |
| arne_wiebalck | we discussed this twice at least ... | 15:40 |
| TheJulia | I'm going to move the raid test to the very last test | 15:41 |
| TheJulia | as a first step | 15:41 |
| *** k_mouza has joined #openstack-ironic | 15:41 | |
| arne_wiebalck | that sounds like a good work around | 15:41 |
| TheJulia | That should help get CI more reliable in the mean time | 15:41 |
| TheJulia | dtantsur: are software raid sets still being preserved if target_raid_config == {} | 15:42 |
| arne_wiebalck | TheJulia: yes | 15:42 |
| arne_wiebalck | TheJulia: as the cleaning step is not run | 15:42 |
| dtantsur | TheJulia: just like hardware RAID - yes | 15:42 |
| arne_wiebalck | yeah, this ^^ is the reason for what we do, analogy with h/w RAID | 15:43 |
| arne_wiebalck | maybe it is time to break this | 15:43 |
| TheJulia | no, tempest needs to ask for the teardown | 15:43 |
| arne_wiebalck | break the analogy, heal the code | 15:43 |
| TheJulia | err, its a separate file :\ | 15:45 |
| arne_wiebalck | re scale tests: it seems I am not able to drive the conductor into an OOM situtation in our current setup (this is without the RAM guard patch, 3 controllers with 8GB each) | 15:48 |
| arne_wiebalck | I just launched 200 nodes in parallel and the conductors were not impressed | 15:48 |
| arne_wiebalck | the only way to squeeze it now, is to go from 3 controllers to 1 ... | 15:49 |
| * TheJulia pokes and prods tempest chagnes | 15:49 | |
| TheJulia | conductor: "Foolish mortals! I shall deploy quickly!" | 15:50 |
| arne_wiebalck | conductor: "Doing tests, huh? No problem for me! But wait when you do this in prod again .." (evil laugh) | 15:51 |
| * arne_wiebalck thinks it is time to call it a week when you make your processes speak ... | 15:52 | |
| *** k_mouza has quit IRC | 15:55 | |
| *** lucasagomes has quit IRC | 15:56 | |
| dtantsur | arne_wiebalck: make processes speak? easy! https://pypi.org/project/jazzit/ | 15:56 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP Tear down raid sets https://review.opendev.org/757141 | 15:57 |
| arne_wiebalck | dtantsur: ha ha ha, this is hilarious! | 15:57 |
| *** k_mouza has joined #openstack-ironic | 15:57 | |
| dtantsur | right? we should totally add it to unit tests! | 15:57 |
| TheJulia | I think we need like little concurency tracking raspberry pi's with LCD screens that then start laughing evilly | 15:57 |
| arne_wiebalck | there was a proposal some time ago to translate log messages into noises | 15:58 |
| TheJulia | that is kind of amusing | 15:58 |
| arne_wiebalck | normal operation would be some background humming | 15:58 |
| arne_wiebalck | when sth happens the frequency would change | 15:58 |
| arne_wiebalck | this way, you can monitor things with your ears | 15:58 |
| TheJulia | ++ | 15:58 |
| arne_wiebalck | and use your eyes to continue writing code :-D | 15:59 |
| TheJulia | or just wire it to play things like the Red Alert klaxon | 15:59 |
| dtantsur | on my previous job someone installed a buzzer that went off when a build was broken | 15:59 |
| dtantsur | it did not survive for long :) | 15:59 |
| TheJulia | heh | 16:00 |
| TheJulia | dtantsur: something like https://www.youtube.com/watch?v=kHyWYA1Nvq8 ? | 16:01 |
| arne_wiebalck | yeah, no real alarms, just some background humming with occasional knock tones | 16:01 |
| TheJulia | knock tones?!? like "*whiirrrrrrlllll* *knock knock knock* | 16:01 |
| TheJulia | " | 16:01 |
| JayF | That episode of enterprise was triggering | 16:01 |
| JayF | "That new thing is annoying, shut it up, make it go away forever" | 16:02 |
| JayF | [5 minutes later] | 16:02 |
| dtantsur | TheJulia: louder :) | 16:02 |
| JayF | "Wow, the new thing saved us!" | 16:02 |
| dtantsur | more like an emergency vehicle thing :) | 16:02 |
| JayF | it's like every ops or security person in an IT shop when their "annoying processes" finally pay off and prevent shenanigans :D | 16:02 |
| dtantsur | JayF: was about to say "security in IT" :) | 16:02 |
| *** priteau has quit IRC | 16:06 | |
| *** k_mouza has quit IRC | 16:06 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Fix inspection for idrac https://review.opendev.org/755972 | 16:16 |
| openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Add a Centos8 UEFI based job https://review.opendev.org/748315 | 16:20 |
| TheJulia | surprising zuul doesn't fail jobs that are missing referenced yaml files.... | 16:20 |
| *** ociuhandu has quit IRC | 16:21 | |
| *** Qianbiao has quit IRC | 16:25 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't run os-prober from grub2-mkconfig https://review.opendev.org/752308 | 16:26 |
| *** dtantsur is now known as dtantsur|afk | 16:26 | |
| dtantsur|afk | have a great weekend folks | 16:26 |
| TheJulia | o/ dtantsur|afk have a wonderful weekend | 16:29 |
| *** gyee has joined #openstack-ironic | 16:32 | |
| *** tosky has quit IRC | 16:35 | |
| TheJulia | arne_wiebalck: replied to your comment on efi content preservation. It might be good to refactor, but I think right now the best thing to do is to focus on the partition image raidless case and then refactor from the basis of trying to support all the cases a bit more cleanly | 16:35 |
| TheJulia | I think if we can get into a place where we consider it a bad image if we're running grub-install, then we're doing good | 16:35 |
| *** priteau has joined #openstack-ironic | 16:41 | |
| arne_wiebalck | TheJulia: yes ... my point was mostly we that dive into that function for md devices, just to (hopefully) do nothing and I thought it might be more readable/clearer if we could make the 'if' clearly state that this is for partition images only ... from what I see, the code should work as is, also in the case of s/w RAID. | 16:43 |
| TheJulia | unfortunately I don't think we know the image type | 16:43 |
| TheJulia | hitting there in this case would almost exclusively be a partition image case | 16:44 |
| TheJulia | or a whole disk that is really broken | 16:44 |
| *** ViniciusCoelho has joined #openstack-ironic | 16:46 | |
| arne_wiebalck | but we could always add "and not hardware.is_md_device(device)", for instance | 16:48 |
| TheJulia | that is exactly what I was thinking of doing | 16:48 |
| TheJulia | I've only gotten derailed nicely this morning :) | 16:49 |
| TheJulia | 3 people messaging me with-in 10 minutes... seems... suspect :) | 16:49 |
| arne_wiebalck | they plot sth ... | 16:49 |
| arne_wiebalck | bye everyone, have a good week-end o/ | 16:52 |
| TheJulia | Enjoy! | 16:53 |
| TheJulia | have a wonderful weekend | 16:53 |
| *** derekh has quit IRC | 16:55 | |
| dking | For DIB elements, I notice that a lot of them set their variables in scripts inside of "environment.d" directories. However, I don't see that documented as a valid directory. Am I missing the documentation, or is there a better place to set variables? | 17:00 |
| dking | I started looking because I wanted to be sure that a variable was set in time to be used by source-repositories. I see that runs a script in extra-data.d. | 17:02 |
| dking | Nevermind. I see that environment.d is mentioned further down in the documentation. Sorry. | 17:03 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Support partition image efi contents https://review.opendev.org/748049 | 17:14 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 17:14 |
| TheJulia | dking: I think it is loaded, or at least was loaded, it is not afaik part of the process of building and the defined directories for such | 17:16 |
| TheJulia | oh, okay | 17:16 |
| TheJulia | :) | 17:16 |
| arne_wiebalck | TheJulia: For the fstab entry, I think the UUID for the ESP from the image and the one we create for md devices is the same. I can verify, but I think this is why the newly created EFI partitions are mounted into /boot/efi. So, I *think* we would end up with twice the same line (which does not hurt, it is just not very useful). | 17:42 |
| *** priteau has quit IRC | 17:58 | |
| *** priteau has joined #openstack-ironic | 18:03 | |
| *** priteau has quit IRC | 18:08 | |
| TheJulia | it _should_ be fine, I think. | 18:16 |
| TheJulia | I guess I could always check the contents and replace the line :\ | 18:17 |
| TheJulia | anyway, CI should run and it should pull in centos8.2 when it finally gets to that job and we will either see success or great sadness | 18:17 |
| *** tkajinam has quit IRC | 18:25 | |
| *** tzumainn has quit IRC | 18:31 | |
| *** tosky has joined #openstack-ironic | 18:32 | |
| *** k_mouza has joined #openstack-ironic | 18:32 | |
| *** tzumainn has joined #openstack-ironic | 18:34 | |
| arne_wiebalck | TheJulia: I just checked, on our nodes with s/w RAID, the UUID of the ESP that comes with the image is the same as the UUID which is on the ESPs we created during the RAID deployment. Since the ESP is mounted by UUID in the fstab, the nodes mount (one of the) ESP of the holder disks, i.e. one of the partitions we created. The patch would duplicated the line in the fstab, I guess. | 18:38 |
| * arne_wiebalck really really goes now :) | 18:40 | |
| *** priteau has joined #openstack-ironic | 18:42 | |
| *** priteau has quit IRC | 18:48 | |
| TheJulia | :) | 18:51 |
| rpioso | iurygregory, JayF: I would appreciate another core reviewer look at https://review.opendev.org/#/c/750605/. It is a cherry pick, which TheJulia has Code-Review+2. | 19:13 |
| patchbot | patch 750605 - ironic (stable/train) - DRAC: Fix RAID create_config clean step - 3 patch sets | 19:13 |
| *** priteau has joined #openstack-ironic | 19:14 | |
| *** k_mouza has quit IRC | 19:20 | |
| erbarr | TheJulia, hey sorry I didn't get a notification about your message but rpioso let me know, I think the log server must have gotten full yesterday but got cleared in the overnight deleting of old builds, then the mirroring was successful after that so the logs are working now for me, can you check to see if you can reach? | 19:24 |
| *** ociuhandu has joined #openstack-ironic | 19:24 | |
| *** priteau has quit IRC | 19:27 | |
| *** zzzeek has quit IRC | 19:49 | |
| *** ociuhandu has quit IRC | 19:50 | |
| *** zzzeek has joined #openstack-ironic | 19:51 | |
| TheJulia | erbarr: I'll recheck the job that I'm curious about and I'll go from there | 20:04 |
| TheJulia | erbarr: thanks! | 20:04 |
| erbarr | no problem, thanks for the notice | 20:05 |
| * TheJulia wonders if anyone has made an easy to setup 3rd party CI tool yet | 20:08 | |
| *** bfournie has quit IRC | 20:42 | |
| rpioso | JayF: Thank you! | 20:48 |
| JayF | np | 20:49 |
| *** ociuhandu has joined #openstack-ironic | 20:51 | |
| *** bfournie has joined #openstack-ironic | 21:00 | |
| clarkb | software factory is often suggested | 21:09 |
| clarkb | TheJulia: ^ that was for you | 21:09 |
| *** lmcgann has quit IRC | 21:10 | |
| *** ociuhandu has quit IRC | 21:13 | |
| *** ViniciusCoelho has quit IRC | 21:18 | |
| *** ociuhandu has joined #openstack-ironic | 21:18 | |
| *** ociuhandu has quit IRC | 21:25 | |
| *** kafilat has quit IRC | 21:54 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: DRAC: Fix RAID create_config clean step https://review.opendev.org/750605 | 22:10 |
| openstackgerrit | Verification of a change to openstack/ironic failed: DRAC: Fix a failure to create virtual disk bug https://review.opendev.org/755895 | 22:10 |
| *** tosky has quit IRC | 23:26 | |
| TheJulia | clarkb: thanks | 23:46 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!