*** trandles has quit IRC | 00:06 | |
*** Goneri has quit IRC | 00:19 | |
*** jj123 has left #openstack-ironic | 00:33 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Fix impitool timing argument calculation https://review.opendev.org/756183 | 00:40 |
---|---|---|
*** zzzeek has quit IRC | 00:43 | |
*** zzzeek has joined #openstack-ironic | 00:45 | |
*** ricolin_ has joined #openstack-ironic | 00:55 | |
*** ricolin_ has quit IRC | 01:01 | |
*** ricolin_ has joined #openstack-ironic | 01:01 | |
janders | good morning Ironic o/ | 01:08 |
*** HardCase has quit IRC | 01:13 | |
*** HardCase has joined #openstack-ironic | 01:14 | |
*** gyee has quit IRC | 01:25 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Fix ipmitool timing argument calculation https://review.opendev.org/756183 | 01:35 |
stevebaker | bfournie: fixed, thanks ^^ | 01:36 |
*** zzzeek has quit IRC | 01:49 | |
*** zzzeek has joined #openstack-ironic | 01:51 | |
*** mkrai has joined #openstack-ironic | 02:51 | |
*** uzumaki has joined #openstack-ironic | 03:16 | |
*** zzzeek has quit IRC | 03:17 | |
*** zzzeek has joined #openstack-ironic | 03:19 | |
*** Qianbiao has joined #openstack-ironic | 03:49 | |
*** zzzeek has quit IRC | 04:01 | |
*** zzzeek has joined #openstack-ironic | 04:05 | |
*** yolanda has quit IRC | 04:07 | |
*** Qianbiao has quit IRC | 04:35 | |
*** Qianbiao has joined #openstack-ironic | 04:36 | |
*** rcernin_ has joined #openstack-ironic | 04:48 | |
*** rcernin has quit IRC | 04:50 | |
*** abdysn has joined #openstack-ironic | 05:26 | |
*** mkrai has quit IRC | 05:43 | |
*** Qianbiao has quit IRC | 05:45 | |
*** alexmcleod has quit IRC | 05:48 | |
*** mkrai has joined #openstack-ironic | 05:54 | |
*** gouthamr has quit IRC | 06:05 | |
*** bbezak has quit IRC | 06:05 | |
*** rajinir has quit IRC | 06:05 | |
*** portdirect has quit IRC | 06:05 | |
*** weshay|ruck has quit IRC | 06:05 | |
*** mgoddard has quit IRC | 06:05 | |
*** openstackgerrit has quit IRC | 06:05 | |
*** dansmith has quit IRC | 06:05 | |
*** dking has quit IRC | 06:05 | |
*** bbezak has joined #openstack-ironic | 06:05 | |
*** dansmith has joined #openstack-ironic | 06:05 | |
*** gouthamr has joined #openstack-ironic | 06:05 | |
*** mgoddard has joined #openstack-ironic | 06:05 | |
*** portdirect has joined #openstack-ironic | 06:05 | |
*** uzumaki has quit IRC | 06:06 | |
*** weshay has joined #openstack-ironic | 06:06 | |
*** v1k0d3n has quit IRC | 06:07 | |
*** v1k0d3n has joined #openstack-ironic | 06:08 | |
*** rajinir has joined #openstack-ironic | 06:10 | |
*** dking has joined #openstack-ironic | 06:10 | |
*** jtomasek has joined #openstack-ironic | 06:14 | |
iurygregory | good morning Ironic! | 06:37 |
janders | hey iurygregory | 06:39 |
janders | how are you? | 06:39 |
arne_wiebalck | Good morning, ironic! | 06:40 |
janders | good morning arne_wiebalck | 06:43 |
janders | o/ | 06:43 |
arne_wiebalck | hey janders o/ | 06:43 |
arne_wiebalck | hey iurygregory o/ | 06:43 |
*** tzumainn has quit IRC | 06:56 | |
*** Qianbiao has joined #openstack-ironic | 07:08 | |
*** mkrai has quit IRC | 07:16 | |
*** ociuhandu has joined #openstack-ironic | 07:36 | |
*** tosky has joined #openstack-ironic | 07:44 | |
*** rpittau|afk is now known as rpittau | 07:45 | |
rpittau | good morning ironic! o/ | 07:45 |
*** mnasiadka has joined #openstack-ironic | 07:47 | |
*** bfournie has quit IRC | 07:51 | |
*** bfournie has joined #openstack-ironic | 07:52 | |
*** alexmcleod has joined #openstack-ironic | 08:04 | |
iurygregory | morning rpittau o/ | 08:04 |
rpittau | hey iurygregory :) | 08:05 |
iurygregory | Qianbiao, fyi stable train seems happy on ipa https://review.opendev.org/#/c/756082/ | 08:05 |
patchbot | patch 756082 - ironic-python-agent (stable/train) - Increase memory for tinyipa jobs - 1 patch set | 08:05 |
rpittau | didn't think that change would have go so far back in time :D | 08:05 |
iurygregory | yeah | 08:05 |
*** lucasagomes has joined #openstack-ironic | 08:06 | |
Qianbiao | Good morning iurygregory rpittau | 08:07 |
rpittau | hey Qianbiao :) | 08:07 |
Qianbiao | thanks for update iurygregory | 08:07 |
Qianbiao | hey o/ | 08:07 |
Qianbiao | may online core give a +2 on this: https://review.opendev.org/#/c/756082/ | 08:11 |
patchbot | patch 756082 - ironic-python-agent (stable/train) - Increase memory for tinyipa jobs - 1 patch set | 08:11 |
rpittau | Qianbiao: I was doing that now, should be sortede | 08:11 |
rpittau | sorted* | 08:11 |
Qianbiao | ok thanks. | 08:11 |
rpittau | and look what's green https://review.opendev.org/#/c/739448/ :) | 08:12 |
patchbot | patch 739448 - ironic - migrate testing to ubuntu focal - 20 patch sets | 08:12 |
rpittau | now to see if we want to disable dstat | 08:12 |
iurygregory | rpittau, the dstat bug was affecting redfish jobs right? | 08:19 |
rpittau | mmmm | 08:19 |
rpittau | I see 3 jobs affected | 08:19 |
rpittau | ironic-tempest-partition-uefi-redfish-vmedia | 08:19 |
iurygregory | I remember vmedia | 08:19 |
iurygregory | yeah | 08:19 |
rpittau | ironic-tempest-ipa-partition-pxe_ipmitool | 08:19 |
rpittau | ironic-tempest-bios-ipmi-direct-tinyipa | 08:20 |
iurygregory | ok wasn't driver specific | 08:20 |
rpittau | doesnt seem so | 08:20 |
iurygregory | yeah | 08:20 |
iurygregory | I've sent a message to some people to see if they have a contact of a person that works on dstat on ubuntu | 08:21 |
rpittau | in theory the patch is already up, the package needs to be compiled and made available | 08:21 |
iurygregory | yeah I just saw in the bz | 08:21 |
iurygregory | launchpad* | 08:21 |
iurygregory | gerrit bot is on PTO... | 08:23 |
*** rcernin_ has quit IRC | 08:35 | |
*** openstackgerrit has joined #openstack-ironic | 08:40 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Explicitly unset OS_CLOUD in the generated openrc https://review.opendev.org/756117 | 08:40 |
*** dtantsur|afk is now known as dtantsur | 08:41 | |
dtantsur | morning ironic | 08:41 |
* iurygregory think dtantsur has powers ... openstackgerrit joined the channel right before he sent a patch | 08:41 | |
dtantsur | \o/ | 08:41 |
iurygregory | good morning dtantsur =) | 08:41 |
dtantsur | hopefully-better-than-mine morning iurygregory :D | 08:42 |
iurygregory | you have powers =) | 08:42 |
dtantsur | now I only need some sleep.. | 08:43 |
iurygregory | go sleep =) | 08:43 |
iurygregory | I need to create some courage and go to the post office today... | 08:44 |
dtantsur | iurygregory: I wish you courage! | 08:45 |
*** uzumaki has joined #openstack-ironic | 08:45 | |
iurygregory | dtantsur, tks! | 08:45 |
*** derekh has joined #openstack-ironic | 08:45 | |
*** derekh has quit IRC | 08:45 | |
*** derekh has joined #openstack-ironic | 08:46 | |
*** Abdallahyas has joined #openstack-ironic | 08:46 | |
dtantsur | iurygregory: do you need to recover something from the sticky hands of customs? :) | 08:50 |
*** abdysn has quit IRC | 08:50 | |
iurygregory | dtantsur, I need to send things to Prague so they can update my long-term residence card .... | 08:51 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Test moving to focal https://review.opendev.org/756230 | 08:52 |
iurygregory | before they would grab the docs in the company, now I need to send to them XD | 08:52 |
dtantsur | ouch | 08:52 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: [DNM] Test moving to focal https://review.opendev.org/756230 | 08:52 |
dtantsur | I remember going in person somewhere, but that was clearly pre-corona | 08:52 |
iurygregory | I'm sure it will be a funny experience since this will be the first time I'm sending something via Ceska Posta =D | 08:53 |
rpittau | wondering if it would be faster to just go in person? :) | 08:53 |
dtantsur | iurygregory: I think they'll make you feel in some form, and you'll need to know the right word for the type of the postage you're sending | 08:54 |
janders | hey rpittau and dtantsur o/ | 08:54 |
janders | rpittau how was your holiday? | 08:54 |
*** priteau has joined #openstack-ironic | 08:54 | |
iurygregory | rpittau, I'm not going to take a bus/train to go to Prague with 3K new cases almost every day in CZ =) | 08:54 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11 to build tinyipa https://review.opendev.org/711647 | 08:55 |
iurygregory | dtantsur, yeah | 08:55 |
iurygregory | I got some tips from the amazing shadower already \o/ | 08:55 |
dtantsur | viva la shadower \o/ | 08:55 |
rpittau | hey janders :) | 08:55 |
dtantsur | g'afternoon janders | 08:56 |
rpittau | janders: it was great, even if we're very stressed because of the pandemic, we got too close to the virus | 08:56 |
janders | rpittau oh dear :( sorry to hear the virus part. Great to hear holiday was great though! :) | 08:57 |
openstackgerrit | Wenping Song proposed openstack/ironic-specs master: Remove usage of six https://review.opendev.org/756231 | 08:59 |
*** SuiongNg|ITRI_TW has joined #openstack-ironic | 09:00 | |
SuiongNg|ITRI_TW | Hi, dtantsur | 09:01 |
dtantsur | o/ | 09:01 |
rpittau | janders: yep, we got a little scared twice, luckily all good, in the end we've decided to stay in Sardegna for the moment | 09:01 |
janders | rpittau good call | 09:02 |
SuiongNg|ITRI_TW | dtantsur: Sorry for the late reply. As for the problem last week. I did have https://review.opende | 09:02 |
SuiongNg|ITRI_TW | v.org/744655/ in my environment and it did fix my problem. But the code changed after 744655 (in ma | 09:02 |
SuiongNg|ITRI_TW | ster branch and also made its way into ubuntu's victoria pre-release). I think later changes partia | 09:02 |
SuiongNg|ITRI_TW | lly re-introduce the problem. Since the later version is in conflict with 744655, I had to remove 7 | 09:02 |
SuiongNg|ITRI_TW | 44655. The later code does account for the original version of the bug though, but not https://revi | 09:02 |
SuiongNg|ITRI_TW | ew.opendev.org/754260 | 09:02 |
janders | rpittau being on an island is more often a good thing than not in the circumstances were in | 09:02 |
dtantsur | SuiongNg|ITRI_TW: please try https://review.opendev.org/#/c/756115/ on top of recent changes | 09:03 |
patchbot | patch 756115 - ironic - Wiping agent tokens on reboot via API - take 2 - 1 patch set | 09:03 |
rpittau | janders: definitely :) | 09:03 |
SuiongNg|ITRI_TW | dtantsur: thanks. I'm currently out-of-office. I'll try that asap. | 09:05 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal https://review.opendev.org/753642 | 09:14 |
*** k_mouza has joined #openstack-ironic | 09:15 | |
*** k_mouza has quit IRC | 09:19 | |
*** k_mouza has joined #openstack-ironic | 09:19 | |
rpittau | TheJulia: hi! I'm not sure why this https://review.opendev.org/747633 was blocked :/ | 09:25 |
patchbot | patch 747633 - ironic (stable/ussuri) - Increase memory of tinyipa vms - 1 patch set | 09:25 |
*** Qianbiao_ has joined #openstack-ironic | 09:26 | |
openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Increase memory for tinyipa jobs https://review.opendev.org/756082 | 09:27 |
*** hjensas__ is now known as hjensas | 09:42 | |
*** priteau has quit IRC | 09:54 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal https://review.opendev.org/753642 | 10:10 |
iurygregory | dstat was affecting IPE because I'm using devstack as parent job /me facepalm | 10:11 |
*** yolanda has joined #openstack-ironic | 10:13 | |
rpittau | dstat issue is also preventing to switch other jobs to focal, we need to merge https://review.opendev.org/739448, it'll break ironic-python-agent | 10:17 |
patchbot | patch 739448 - ironic - migrate testing to ubuntu focal - 20 patch sets | 10:17 |
iurygregory | can we give a try disabling to see how it goes? | 10:18 |
rpittau | yeah, that's what I'm doing | 10:19 |
iurygregory | rpittau++ | 10:20 |
rpittau | there's also one more "little" problem | 10:20 |
iurygregory | "litlle" | 10:20 |
iurygregory | tell me more =) | 10:20 |
rpittau | focal doesn't have support for python2, so in dib all the yum related tools can't run | 10:21 |
rpittau | like yumdownloader | 10:21 |
rpittau | TL;DR we can't build centos based ipa ramdisk on focal using dib | 10:22 |
*** SuiongNg|ITRI_TW has quit IRC | 10:22 | |
rpittau | an example in the src jobs here: https://review.opendev.org/#/c/756230/ | 10:23 |
patchbot | patch 756230 - ironic-python-agent - [DNM] Test moving to focal - 2 patch sets | 10:23 |
dtantsur | rpittau: we're using centos 8, why yum? is it a bug in DIB? | 10:24 |
iurygregory | oh I remember you said something about it | 10:24 |
rpittau | dtantsur: dib uses yumdownloader to initiate the chroot environment downlading the base packages like basesystem, centos-release, etc | 10:26 |
rpittau | there is currently no alternative to that AFAIK | 10:26 |
dtantsur | rpittau: well, recent fedora do it somehow :) | 10:26 |
dtantsur | I guess we should move it to #openstack-dib and continue there | 10:26 |
rpittau | dtantsur: recent fedora ? | 10:27 |
dtantsur | rpittau: I mean, any OS is bootstrapped via the same process, it has to be possible | 10:27 |
dtantsur | I guess DIB is not aware of that though | 10:27 |
derekh | dtantsur: you mentioned last week something about being able to identify a BMC ip address when using redfish, similar to how we do it for ipmi, do I have that right and if so can you point me at it? | 10:29 |
dtantsur | derekh: not exactly. it's possible only in theory (and not trivial even there), I don't think it works in practice. | 10:30 |
dtantsur | we're working on other approaches to identify nodes | 10:31 |
dtantsur | TheJulia made a patch to match IPMI addresses against Redfish addresses (because they match very often) | 10:31 |
dtantsur | janders is working on MAC address detection for Redfish nodes | 10:31 |
rpittau | dtantsur: the DIB jobs still run on bionic, that's why all's good on their side | 10:32 |
derekh | dtantsur: ack, thanks | 10:33 |
*** bbezak has quit IRC | 10:33 | |
*** PrinzElvis has quit IRC | 10:33 | |
*** tkajinam has quit IRC | 10:33 | |
dtantsur | rpittau: yeah, it should be raised to them (and to whoever is driving the Focal goal) | 10:33 |
rpittau | yep | 10:33 |
*** bbezak has joined #openstack-ironic | 10:34 | |
*** PrinzElvis has joined #openstack-ironic | 10:34 | |
*** tkajinam has joined #openstack-ironic | 10:34 | |
rpittau | I had a quick discussion a month ago about that, I will raise it again | 10:35 |
openstackgerrit | Yogesh Ramachandra proposed openstack/ironic master: Fix idrac-wsman bios factory-reset setting https://review.opendev.org/748696 | 10:40 |
*** mkrai has joined #openstack-ironic | 10:44 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Declare support for Fedora 32 and openSUSE Leap 15.2 https://review.opendev.org/756254 | 10:47 |
dtantsur | rpittau: re bifrost CI ^^^ | 10:48 |
dtantsur | will have to be applied to Ussuri+ since the CI changes have been done there | 10:49 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [PoC] A naive attempt at creating an upgrade job https://review.opendev.org/750376 | 10:58 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Update `cleaning_error_handler` https://review.opendev.org/756258 | 11:00 |
openstackgerrit | Mark Goddard proposed openstack/bifrost master: Install DIB dependencies in a separate role https://review.opendev.org/756005 | 11:06 |
janders | see you tomorrow Ironic o/ | 11:10 |
*** ricolin_ has quit IRC | 11:10 | |
*** ricolin has joined #openstack-ironic | 11:11 | |
*** Abdallahyas has quit IRC | 11:21 | |
*** abdysn has joined #openstack-ironic | 11:22 | |
*** Qianbiao_ has quit IRC | 11:32 | |
*** thiagop has joined #openstack-ironic | 11:34 | |
*** mkrai has quit IRC | 11:41 | |
*** mkrai has joined #openstack-ironic | 11:41 | |
*** Goneri has joined #openstack-ironic | 11:48 | |
*** mkrai has quit IRC | 12:00 | |
*** zzzeek has quit IRC | 12:17 | |
*** zzzeek has joined #openstack-ironic | 12:18 | |
*** jtomasek_ has joined #openstack-ironic | 12:23 | |
*** jtomasek has quit IRC | 12:26 | |
Qianbiao | Hello, is "Customization Script" feature in Openstack Web console maps to ironic configdrive? | 12:30 |
arne_wiebalck | This is the user-data used by cloud-init. It is stored on the config drive. | 12:43 |
*** jamesdenton has quit IRC | 12:46 | |
Qianbiao | arne_wiebalck ty | 12:47 |
*** jamesdenton has joined #openstack-ironic | 12:47 | |
Qianbiao | In ramdisk deploy interface document: https://docs.openstack.org/ironic/train/admin/interfaces/deploy.html | 12:48 |
Qianbiao | it says: Configuration drives are not supported. | 12:48 |
Qianbiao | It means we can not add any customer init to it then? | 12:48 |
Qianbiao | then how user can login to the server. (customer image with per-set ssh-key)? | 12:49 |
*** rh-jelabarre has joined #openstack-ironic | 13:00 | |
openstackgerrit | Merged openstack/bifrost master: Declare support for Fedora 32 and openSUSE Leap 15.2 https://review.opendev.org/756254 | 13:02 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/victoria: Declare support for Fedora 32 and openSUSE Leap 15.2 https://review.opendev.org/756287 | 13:02 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/ussuri: DNM test CI https://review.opendev.org/756288 | 13:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/victoria: Switch the CI to stable/victoria https://review.opendev.org/756289 | 13:05 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Fix redfish BIOS apply config error handling https://review.opendev.org/755776 | 13:13 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Use bionic nodeset for DIB centos src jobs https://review.opendev.org/756291 | 13:18 |
Qianbiao | arne_wiebalck SIG will starts in an hour right? | 13:18 |
Qianbiao | 40 minutes later | 13:19 |
dtantsur | seems so | 13:20 |
TheJulia | good morning | 13:22 |
rpittau | good mornning TheJulia :) | 13:23 |
Qianbiao | Morning TheJulia | 13:23 |
Qianbiao | For ramdisk deploy, we can not use configdrive, but we can still use metadata service to pass user-data, right? | 13:23 |
TheJulia | yes | 13:24 |
TheJulia | welcome back rpittau | 13:24 |
rpittau | thanks :) | 13:25 |
*** rloo has joined #openstack-ironic | 13:25 | |
TheJulia | so did I miss anything yesterday? | 13:30 |
arne_wiebalck | Qianbiao: yes, in about 30 mins | 13:31 |
Qianbiao | ok | 13:32 |
arne_wiebalck | dtantsur: TheJulia: it seems disabling the udev queues does not help with the RAID issue, but re-adding does | 13:32 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Re-add missing devices https://review.opendev.org/756300 | 13:33 |
TheJulia | arne_wiebalck: re-adding as in ripping out the raid and reconstructing? | 13:33 |
arne_wiebalck | TheJulia: no, checking what was ejected and stuffing it back in | 13:34 |
TheJulia | oh yeah, that would work | 13:34 |
arne_wiebalck | I still don't get why the disks get ejected immediately ... udev was a suspect, but disabling the event processing did make a difference in my tests | 13:36 |
arne_wiebalck | udev still remains a suspect | 13:36 |
arne_wiebalck | it may detect the new partitions just created and grabs them while we put the superblock on it | 13:37 |
TheJulia | i kind of seem remember running into a similar thing when one of my members was ejected right after creation in days along ago with versions of the kernel that we don't speak of anymore | 13:37 |
arne_wiebalck | the udev rules for md also skip any action if run while anaconda runs, with a comment saying that anaconda brings up devices ... seems like is | 13:37 |
arne_wiebalck | *us | 13:37 |
TheJulia | hmm | 13:38 |
* arne_wiebalck looks for the link ... | 13:38 | |
TheJulia | Yeah, i guess we really don't need ot trying to be helpful | 13:38 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Change LANG in install-deps.sh to en_US.UTF-8 https://review.opendev.org/756214 | 13:39 |
openstackgerrit | Merged openstack/bifrost stable/victoria: Declare support for Fedora 32 and openSUSE Leap 15.2 https://review.opendev.org/756287 | 13:39 |
arne_wiebalck | TheJulia: https://github.com/neilbrown/mdadm/blob/master/udev-md-raid-assembly.rules#L5 | 13:40 |
arne_wiebalck | TheJulia: so, there seems to be some unwanted interference and maybe that bites us as well | 13:40 |
arne_wiebalck | TheJulia: anyway, I am still deleting and creating nodes in a loop and the above patch seems to at least patch it | 13:41 |
* arne_wiebalck would not call this a fix | 13:41 | |
*** thiagop has quit IRC | 13:42 | |
*** tzumainn has joined #openstack-ironic | 13:44 | |
* arne_wiebalck called it a fix in the reno :-S | 13:44 | |
*** thiagop has joined #openstack-ironic | 13:45 | |
TheJulia | heh | 13:46 |
TheJulia | short of knowing we have disabled udev in this case, it is a fix | 13:46 |
arne_wiebalck | either udev is not to blame for this problem, or I disabled it the wrong way | 13:48 |
arne_wiebalck | udev is not very verbose either, at least not by defautl | 13:48 |
arne_wiebalck | default | 13:48 |
TheJulia | dtantsur: would you be okay if I go back to my initial approach for a uefi job for metalsmith and have it as a separate job instead of trying to just switch the job over from 7 all at once. | 13:49 |
dtantsur | TheJulia: morning! no objections from me. | 13:51 |
dtantsur | I hope you feel better today | 13:51 |
TheJulia | okay, that will make it easier to get something heading down the path of working since it looks like we realistically need IPA changes for partition+uefi to work | 13:51 |
TheJulia | a lot better, slept for like 5 additional hours yesterday | 13:52 |
dtantsur | nice | 13:52 |
dtantsur | today it's my turn though | 13:52 |
dtantsur | I guess I'll skip the SIG meeting, unless the agenda includes a zombie show | 13:53 |
dtantsur | arne_wiebalck: ^^ | 13:53 |
TheJulia | ugh, feel better! | 13:54 |
dtantsur | thx | 13:54 |
iurygregory | good morning TheJulia =) | 13:56 |
arne_wiebalck | dtantsur: get better ... no zombies I hope! | 14:00 |
*** abdysn has quit IRC | 14:00 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Trivial: use the correct error handler for do_next_clean_step https://review.opendev.org/756326 | 14:38 |
iurygregory | IPE seems happy with ubuntu focal after disabling dstat \o/ | 14:46 |
rpittau | I guess we'll have to disable it globally | 14:52 |
rpittau | IF we want to use focal | 14:52 |
openstackgerrit | Verification of a change to openstack/ironic-tempest-plugin failed: Wait for power off before allocation tests https://review.opendev.org/755661 | 14:53 |
*** trandles has joined #openstack-ironic | 14:54 | |
*** thiagop has quit IRC | 14:55 | |
TheJulia | disabling dstat?!? | 15:01 |
rpittau | yeah | 15:01 |
rpittau | it crashes on focal | 15:01 |
TheJulia | \o/ | 15:01 |
iurygregory | yeah \o/ | 15:01 |
iurygregory | it's a party | 15:02 |
rpittau | \o/ | 15:02 |
rpittau | an alternative that I'm considering for centos images is to convert to centos-8 nodeset | 15:02 |
rpittau | considering that yum-tools is not in focal | 15:02 |
rpittau | I'm not sure of the amount of work that requires though | 15:03 |
dtantsur | I suspect fixing DIB is easier.. | 15:03 |
rpittau | I'm not sure | 15:03 |
dtantsur | you're suggesting to make devstack CI work reliably on centos 8 | 15:05 |
dtantsur | that's no small adventure :) | 15:05 |
rpittau | yeah :D | 15:05 |
iurygregory | it's a huge adventure | 15:05 |
iurygregory | the roller coaster has a lot of loops and missing rails | 15:06 |
Qianbiao | dtantsur It says on dev document that it supports centos8? | 15:08 |
Qianbiao | * devstack document | 15:08 |
rpittau | Qianbiao: yeah, the basic devstack, we're talking about devstack + ironic :) | 15:09 |
Qianbiao | :) ok | 15:09 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Refactoring: split away continue_node_deploy/clean https://review.opendev.org/756343 | 15:27 |
*** trandles has quit IRC | 15:30 | |
*** uzumaki has quit IRC | 15:43 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Use focal for tinyipa src jobs used by ipa-builder https://review.opendev.org/756353 | 15:43 |
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:45 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: [DNM] test converting src jobs to focal https://review.opendev.org/756355 | 15:46 |
rpittau | good night! o/ | 15:48 |
*** rpittau is now known as rpittau|afk | 15:48 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/train: Increase memory for tinyipa jobs https://review.opendev.org/756082 | 15:48 |
*** k_mouza has quit IRC | 15:52 | |
*** thiagop has joined #openstack-ironic | 15:52 | |
*** k_mouza has joined #openstack-ironic | 16:02 | |
*** lucasagomes has quit IRC | 16:03 | |
*** Qianbiao has quit IRC | 16:05 | |
*** mkrai has joined #openstack-ironic | 16:11 | |
openstackgerrit | Merged openstack/bifrost stable/victoria: Another case for skip_package_install https://review.opendev.org/756116 | 16:12 |
openstackgerrit | Merged openstack/bifrost master: Explicitly unset OS_CLOUD in the generated openrc https://review.opendev.org/756117 | 16:12 |
*** rloo has quit IRC | 16:15 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost stable/victoria: Explicitly unset OS_CLOUD in the generated openrc https://review.opendev.org/756364 | 16:17 |
*** mkrai has quit IRC | 16:18 | |
*** dtantsur is now known as dtantsur|afk | 16:18 | |
dtantsur|afk | o/ | 16:18 |
*** rloo has joined #openstack-ironic | 16:18 | |
*** trandles has joined #openstack-ironic | 16:25 | |
*** rloo has quit IRC | 16:29 | |
*** rloo has joined #openstack-ironic | 16:30 | |
iurygregory | TheJulia, when looking at the problem to inspect idrac because of the key error I've found something interesting (maybe is just me) , the tests for image_utils are focused on Redfish https://github.com/openstack/ironic/blob/master/ironic/tests/unit/drivers/modules/test_image_utils.py#L420-L422 but this seems to be ilo specific now O.o | 16:34 |
TheJulia | iurygregory: yeah, because it got refactored so the code path could be reused by both drivers | 16:36 |
*** rh-jlabarre has joined #openstack-ironic | 16:44 | |
*** rh-jelabarre has quit IRC | 16:45 | |
arne_wiebalck | bye everyone o/ | 16:45 |
*** rloo has quit IRC | 16:48 | |
*** ociuhandu has quit IRC | 16:48 | |
*** rloo has joined #openstack-ironic | 16:49 | |
iurygregory | TheJulia, yeah I was just wondering that would make more sense to have a class testing with ilo parameters.. | 16:49 |
TheJulia | either or both I guess | 16:50 |
*** jtomasek_ has quit IRC | 16:54 | |
*** derekh has quit IRC | 17:04 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Add basic retries for inspection https://review.opendev.org/745598 | 17:07 |
*** k_mouza has quit IRC | 17:25 | |
*** k_mouza has joined #openstack-ironic | 17:32 | |
*** k_mouza has quit IRC | 17:35 | |
*** dking has quit IRC | 17:56 | |
openstackgerrit | Merged openstack/bifrost stable/victoria: Switch the CI to stable/victoria https://review.opendev.org/756289 | 17:58 |
*** larsks has quit IRC | 18:00 | |
*** gyee has joined #openstack-ironic | 18:16 | |
*** k_mouza has joined #openstack-ironic | 18:33 | |
*** k_mouza has quit IRC | 18:40 | |
*** k_mouza has joined #openstack-ironic | 18:47 | |
*** k_mouza has quit IRC | 18:51 | |
*** k_mouza has joined #openstack-ironic | 18:56 | |
*** k_mouza has quit IRC | 19:00 | |
*** k_mouza has joined #openstack-ironic | 19:05 | |
*** k_mouza has quit IRC | 19:09 | |
*** yolanda has quit IRC | 19:20 | |
*** yolanda has joined #openstack-ironic | 19:20 | |
*** k_mouza has joined #openstack-ironic | 19:23 | |
*** k_mouza has quit IRC | 19:27 | |
*** lmcgann has joined #openstack-ironic | 19:45 | |
*** k_mouza has joined #openstack-ironic | 19:51 | |
*** tosky has quit IRC | 19:52 | |
*** k_mouza has quit IRC | 19:55 | |
*** ociuhandu has joined #openstack-ironic | 19:56 | |
lmcgann | Working on the security_interface spec I had a question about the state machine in ironic. I'd imagine we want to make all of the methods in the interface invoked immediately upon when the node hits the state of the method's namesake. So for a method which enrolls the node, isnt node enrollment something that happens before node validation, the state wherein we see that drivers are actually good to be used? | 20:00 |
lmcgann | Also, at one point someone proposed the idea that we have one method wherein we just pass in the state of the node at that moment, would having four or so methods to cover a bare minimum number of states (enroll?, deployment, cleaning, deleting) as well as this state hookable method be a reasonable idea? | 20:03 |
lmcgann | Even though in theory the minimum states arent actually needed at that point, it might provide a guide as to what you should strive to cover with your interface. | 20:04 |
*** ociuhandu has quit IRC | 20:08 | |
*** k_mouza has joined #openstack-ironic | 20:09 | |
*** k_mouza has quit IRC | 20:13 | |
TheJulia | yes, enrollment is before validation and in enrollment state we may know literally nothing about the machine | 20:13 |
TheJulia | validation can be called at any time and it is an interface sanity check/status check for the most part | 20:13 |
TheJulia | likely you just want to do on deployment/cleaning, since deleting is actually cleaning | 20:14 |
TheJulia | and cleaning runs before nodes are available to provision | 20:14 |
lmcgann | so then, perhaps node_enrollment should become node_manageable | 20:14 |
TheJulia | so, manageable is "I think i can manage it, would you like to manage it?" | 20:14 |
lmcgann | or immediately after validation (which is manageable, it just sounds misleading) | 20:15 |
TheJulia | provide verb moves the node into cleaning and eventually available state | 20:15 |
TheJulia | provide verb basically means "yes, you manage it" | 20:15 |
*** zzzeek has quit IRC | 20:15 | |
*** zzzeek has joined #openstack-ironic | 20:16 | |
lmcgann | Im also trying to think what each of these methods would do in keylime. I don't think any of these methods should actually do any attestation. We need one method to register the node with the verifier, and one to deregister it. I'm not sure which steps these should be at. | 20:19 |
lmcgann | and if we want to trigger an attestation, I'm not sure it always matters what state we are in provided we are registered with the verifier | 20:20 |
*** thiagop has quit IRC | 20:43 | |
*** k_mouza has joined #openstack-ironic | 20:52 | |
*** larsks has joined #openstack-ironic | 20:53 | |
*** k_mouza has quit IRC | 20:56 | |
*** k_mouza has joined #openstack-ironic | 21:05 | |
rpioso | TheJulia: Should https://docs.openstack.org/ironic/victoria/ be working? I get a 404. | 21:05 |
*** k_mouza has quit IRC | 21:09 | |
TheJulia | ugh uefi on supermicro hardware + supermicro hardware netbooting == ugh | 21:12 |
TheJulia | rpioso: not quite yet if memory seves | 21:12 |
TheJulia | serves | 21:12 |
TheJulia | it might take a patch merging though at this point | 21:12 |
rpioso | TheJulia: Thank you! | 21:12 |
TheJulia | lmcgann: I suspect you may be overthinking it slightly (a normal reaction), would it be possible to talk through it tomorrow? | 21:13 |
lmcgann | TheJulia sure | 21:19 |
TheJulia | bfournie: https://www.supermicro.com/support/faqs/faq.cfm?faq=22208 | 21:21 |
*** HardCase has quit IRC | 21:36 | |
*** lmcgann has quit IRC | 22:02 | |
*** k_mouza has joined #openstack-ironic | 22:10 | |
*** k_mouza has quit IRC | 22:15 | |
TheJulia | bfournie: I'm thinking we should change IPA's bootloader install logic to actually skip it if it looks like a whole disk && uefi bootmode is requested, and actual boot mode is bios, but since apparently it can be configured... | 22:17 |
*** k_mouza has joined #openstack-ironic | 22:20 | |
*** k_mouza has quit IRC | 22:24 | |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Wait for power off before allocation tests https://review.opendev.org/755661 | 22:29 |
*** rloo has quit IRC | 22:30 | |
openstackgerrit | Richard G. Pioso proposed openstack/ironic stable/train: DRAC: Fix a failure to create virtual disk bug https://review.opendev.org/755895 | 22:31 |
*** k_mouza has joined #openstack-ironic | 22:40 | |
bfournie | TheJulia: what should IPA use to detect actual boot mode? | 22:43 |
TheJulia | good question | 22:44 |
TheJulia | hmmm | 22:44 |
*** k_mouza has quit IRC | 22:45 | |
TheJulia | bfournie: https://github.com/openstack/ironic-python-agent/blob/99dee5067ea4f06d3083170d801e600f46842170/ironic_python_agent/hardware.py#L1189 | 22:46 |
TheJulia | bfournie: basically it is not seeing efi firmware present at all which makes sense if the network card is bios booting | 22:53 |
TheJulia | which is what would happen if it was relying upon the bios image in the chip instead of the uefi image | 22:53 |
bfournie | TheJulia: Ah, I see, yes - Oct 05 16:25:14 master-0 ironic-python-agent[2176]: 2020-10-05 16:25:14.946 2176 DEBUG root [-] The current boot mode is bios get_boot_info /usr/lib/python3.6/site-packages/ironic_python_agent/hardware.py:1149 | 22:54 |
TheJulia | bingo | 22:54 |
TheJulia | bfournie: I sent you photos from my machine at home | 22:54 |
TheJulia | hopefully they help | 22:54 |
bfournie | TheJulia: that's it, yes awesome photos! we'll get on a supermicro and change those settings tomorrow | 22:55 |
TheJulia | \o/ | 22:55 |
bfournie | thanks a lot TheJulia !! | 22:56 |
TheJulia | no problem :) | 22:56 |
*** rcernin_ has joined #openstack-ironic | 22:56 | |
TheJulia | now, to unwind my brain a little before I cook dinner | 22:56 |
*** rcernin_ has quit IRC | 22:58 | |
*** rcernin has joined #openstack-ironic | 22:58 | |
*** gudrutis has quit IRC | 23:11 | |
*** gudrutis has joined #openstack-ironic | 23:12 | |
janders | Good morning Ironic o/ | 23:32 |
*** k_mouza has joined #openstack-ironic | 23:42 | |
janders | TheJulia how do I propose topics for discussion for the PTG? (I wanted to propose adding a discussion about NVMe discard based storage cleaning) | 23:43 |
TheJulia | Add to the ether pad? | 23:45 |
janders | do you have a link handy? (googling results in a *ton* of old links) | 23:46 |
janders | disregard, I was able to modify old URL (this didn't work last time I tried) | 23:47 |
*** k_mouza has quit IRC | 23:47 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!