Tuesday, 2020-10-06

*** trandles has quit IRC00:06
*** Goneri has quit IRC00:19
*** jj123 has left #openstack-ironic00:33
openstackgerritSteve Baker proposed openstack/ironic master: Fix impitool timing argument calculation  https://review.opendev.org/75618300:40
*** zzzeek has quit IRC00:43
*** zzzeek has joined #openstack-ironic00:45
*** ricolin_ has joined #openstack-ironic00:55
*** ricolin_ has quit IRC01:01
*** ricolin_ has joined #openstack-ironic01:01
jandersgood morning Ironic o/01:08
*** HardCase has quit IRC01:13
*** HardCase has joined #openstack-ironic01:14
*** gyee has quit IRC01:25
openstackgerritSteve Baker proposed openstack/ironic master: Fix ipmitool timing argument calculation  https://review.opendev.org/75618301:35
stevebakerbfournie: fixed, thanks ^^01:36
*** zzzeek has quit IRC01:49
*** zzzeek has joined #openstack-ironic01:51
*** mkrai has joined #openstack-ironic02:51
*** uzumaki has joined #openstack-ironic03:16
*** zzzeek has quit IRC03:17
*** zzzeek has joined #openstack-ironic03:19
*** Qianbiao has joined #openstack-ironic03:49
*** zzzeek has quit IRC04:01
*** zzzeek has joined #openstack-ironic04:05
*** yolanda has quit IRC04:07
*** Qianbiao has quit IRC04:35
*** Qianbiao has joined #openstack-ironic04:36
*** rcernin_ has joined #openstack-ironic04:48
*** rcernin has quit IRC04:50
*** abdysn has joined #openstack-ironic05:26
*** mkrai has quit IRC05:43
*** Qianbiao has quit IRC05:45
*** alexmcleod has quit IRC05:48
*** mkrai has joined #openstack-ironic05:54
*** gouthamr has quit IRC06:05
*** bbezak has quit IRC06:05
*** rajinir has quit IRC06:05
*** portdirect has quit IRC06:05
*** weshay|ruck has quit IRC06:05
*** mgoddard has quit IRC06:05
*** openstackgerrit has quit IRC06:05
*** dansmith has quit IRC06:05
*** dking has quit IRC06:05
*** bbezak has joined #openstack-ironic06:05
*** dansmith has joined #openstack-ironic06:05
*** gouthamr has joined #openstack-ironic06:05
*** mgoddard has joined #openstack-ironic06:05
*** portdirect has joined #openstack-ironic06:05
*** uzumaki has quit IRC06:06
*** weshay has joined #openstack-ironic06:06
*** v1k0d3n has quit IRC06:07
*** v1k0d3n has joined #openstack-ironic06:08
*** rajinir has joined #openstack-ironic06:10
*** dking has joined #openstack-ironic06:10
*** jtomasek has joined #openstack-ironic06:14
iurygregorygood morning Ironic!06:37
jandershey iurygregory06:39
jandershow are you?06:39
arne_wiebalckGood morning, ironic!06:40
jandersgood morning arne_wiebalck06:43
janderso/06:43
arne_wiebalckhey janders o/06:43
arne_wiebalckhey iurygregory o/06:43
*** tzumainn has quit IRC06:56
*** Qianbiao has joined #openstack-ironic07:08
*** mkrai has quit IRC07:16
*** ociuhandu has joined #openstack-ironic07:36
*** tosky has joined #openstack-ironic07:44
*** rpittau|afk is now known as rpittau07:45
rpittaugood morning ironic! o/07:45
*** mnasiadka has joined #openstack-ironic07:47
*** bfournie has quit IRC07:51
*** bfournie has joined #openstack-ironic07:52
*** alexmcleod has joined #openstack-ironic08:04
iurygregorymorning rpittau o/08:04
rpittauhey iurygregory :)08:05
iurygregoryQianbiao, fyi stable train seems happy on ipa https://review.opendev.org/#/c/756082/08:05
patchbotpatch 756082 - ironic-python-agent (stable/train) - Increase memory for tinyipa jobs - 1 patch set08:05
rpittaudidn't think that change would have go so far back in time :D08:05
iurygregoryyeah08:05
*** lucasagomes has joined #openstack-ironic08:06
QianbiaoGood morning iurygregory rpittau08:07
rpittauhey Qianbiao :)08:07
Qianbiaothanks for update iurygregory08:07
Qianbiaohey o/08:07
Qianbiaomay online core give a +2 on this: https://review.opendev.org/#/c/756082/08:11
patchbotpatch 756082 - ironic-python-agent (stable/train) - Increase memory for tinyipa jobs - 1 patch set08:11
rpittauQianbiao: I was doing that now, should be sortede08:11
rpittausorted*08:11
Qianbiaook thanks.08:11
rpittauand look what's green https://review.opendev.org/#/c/739448/ :)08:12
patchbotpatch 739448 - ironic - migrate testing to ubuntu focal - 20 patch sets08:12
rpittaunow to see if we want to disable dstat08:12
iurygregoryrpittau, the dstat bug was affecting redfish jobs right?08:19
rpittaummmm08:19
rpittauI see 3 jobs affected08:19
rpittauironic-tempest-partition-uefi-redfish-vmedia08:19
iurygregoryI remember vmedia08:19
iurygregoryyeah08:19
rpittauironic-tempest-ipa-partition-pxe_ipmitool08:19
rpittauironic-tempest-bios-ipmi-direct-tinyipa08:20
iurygregoryok wasn't driver specific08:20
rpittaudoesnt seem so08:20
iurygregoryyeah08:20
iurygregoryI've sent a message to some people to see if they have a contact of a person that works on dstat on ubuntu08:21
rpittauin theory the patch is already up, the package needs to be compiled and made available08:21
iurygregoryyeah I just saw in the bz08:21
iurygregorylaunchpad*08:21
iurygregorygerrit bot is on PTO...08:23
*** rcernin_ has quit IRC08:35
*** openstackgerrit has joined #openstack-ironic08:40
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Explicitly unset OS_CLOUD in the generated openrc  https://review.opendev.org/75611708:40
*** dtantsur|afk is now known as dtantsur08:41
dtantsurmorning ironic08:41
* iurygregory think dtantsur has powers ... openstackgerrit joined the channel right before he sent a patch08:41
dtantsur\o/08:41
iurygregorygood morning dtantsur =)08:41
dtantsurhopefully-better-than-mine morning iurygregory :D08:42
iurygregoryyou have powers =)08:42
dtantsurnow I only need some sleep..08:43
iurygregorygo sleep =)08:43
iurygregoryI need to create some courage and go to the post office today...08:44
dtantsuriurygregory: I wish you courage!08:45
*** uzumaki has joined #openstack-ironic08:45
iurygregorydtantsur, tks!08:45
*** derekh has joined #openstack-ironic08:45
*** derekh has quit IRC08:45
*** derekh has joined #openstack-ironic08:46
*** Abdallahyas has joined #openstack-ironic08:46
dtantsuriurygregory: do you need to recover something from the sticky hands of customs? :)08:50
*** abdysn has quit IRC08:50
iurygregorydtantsur, I need to send things to Prague so they can update my long-term residence card ....08:51
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Test moving to focal  https://review.opendev.org/75623008:52
iurygregorybefore they would grab the docs in the company, now I need to send to them XD08:52
dtantsurouch08:52
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: [DNM] Test moving to focal  https://review.opendev.org/75623008:52
dtantsurI remember going in person somewhere, but that was clearly pre-corona08:52
iurygregoryI'm sure it will be a funny experience since this will be the first time I'm sending something via Ceska Posta =D08:53
rpittauwondering if it would be faster to just go in person? :)08:53
dtantsuriurygregory: 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 sending08:54
jandershey rpittau and dtantsur o/08:54
jandersrpittau how was your holiday?08:54
*** priteau has joined #openstack-ironic08:54
iurygregoryrpittau, I'm not going to take a bus/train to go to Prague with 3K new cases almost every day in CZ =)08:54
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11 to build tinyipa  https://review.opendev.org/71164708:55
iurygregorydtantsur, yeah08:55
iurygregoryI got some tips from the amazing shadower already \o/08:55
dtantsurviva la shadower \o/08:55
rpittauhey janders :)08:55
dtantsurg'afternoon janders08:56
rpittaujanders: it was great, even if we're very stressed because of the pandemic, we got too close to the virus08:56
jandersrpittau oh dear :( sorry to hear the virus part. Great to hear holiday was great though! :)08:57
openstackgerritWenping Song proposed openstack/ironic-specs master: Remove usage of six  https://review.opendev.org/75623108:59
*** SuiongNg|ITRI_TW has joined #openstack-ironic09:00
SuiongNg|ITRI_TWHi, dtantsur09:01
dtantsuro/09:01
rpittaujanders: yep, we got a little scared twice, luckily all good, in the end we've decided to stay in Sardegna for the moment09:01
jandersrpittau good call09:02
SuiongNg|ITRI_TWdtantsur: Sorry for the late reply. As for the problem last week. I did have https://review.opende09:02
SuiongNg|ITRI_TWv.org/744655/ in my environment and it did fix my problem. But the code changed after 744655 (in ma09:02
SuiongNg|ITRI_TWster branch and also made its way into ubuntu's victoria pre-release). I think later changes partia09:02
SuiongNg|ITRI_TWlly re-introduce the problem. Since the later version is in conflict with 744655, I had to remove 709:02
SuiongNg|ITRI_TW44655. The later code does account for the original version of the bug though, but not https://revi09:02
SuiongNg|ITRI_TWew.opendev.org/75426009:02
jandersrpittau being on an island is more often a good thing than not in the circumstances were in09:02
dtantsurSuiongNg|ITRI_TW: please try https://review.opendev.org/#/c/756115/ on top of recent changes09:03
patchbotpatch 756115 - ironic - Wiping agent tokens on reboot via API - take 2 - 1 patch set09:03
rpittaujanders: definitely :)09:03
SuiongNg|ITRI_TWdtantsur: thanks. I'm currently out-of-office. I'll try that asap.09:05
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal  https://review.opendev.org/75364209:14
*** k_mouza has joined #openstack-ironic09:15
*** k_mouza has quit IRC09:19
*** k_mouza has joined #openstack-ironic09:19
rpittauTheJulia: hi! I'm not sure why this https://review.opendev.org/747633 was blocked :/09:25
patchbotpatch 747633 - ironic (stable/ussuri) - Increase memory of tinyipa vms - 1 patch set09:25
*** Qianbiao_ has joined #openstack-ironic09:26
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Increase memory for tinyipa jobs  https://review.opendev.org/75608209:27
*** hjensas__ is now known as hjensas09:42
*** priteau has quit IRC09:54
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Migrate IPE job to focal  https://review.opendev.org/75364210:10
iurygregorydstat was affecting IPE because I'm using devstack as parent job /me facepalm10:11
*** yolanda has joined #openstack-ironic10:13
rpittaudstat issue is also preventing to switch other jobs to focal, we need to merge https://review.opendev.org/739448, it'll break ironic-python-agent10:17
patchbotpatch 739448 - ironic - migrate testing to ubuntu focal - 20 patch sets10:17
iurygregorycan we give a try disabling to see how it goes?10:18
rpittauyeah, that's what I'm doing10:19
iurygregoryrpittau++10:20
rpittauthere's also one more "little" problem10:20
iurygregory"litlle"10:20
iurygregorytell me more =)10:20
rpittaufocal doesn't have support for python2, so in dib all the yum related tools can't run10:21
rpittaulike yumdownloader10:21
rpittauTL;DR we can't build centos based ipa ramdisk on focal using dib10:22
*** SuiongNg|ITRI_TW has quit IRC10:22
rpittauan example in the src jobs here: https://review.opendev.org/#/c/756230/10:23
patchbotpatch 756230 - ironic-python-agent - [DNM] Test moving to focal - 2 patch sets10:23
dtantsurrpittau: we're using centos 8, why yum? is it a bug in DIB?10:24
iurygregoryoh I remember you said something about it10:24
rpittaudtantsur: dib uses yumdownloader to initiate the chroot environment downlading the base packages like basesystem, centos-release, etc10:26
rpittauthere is currently no alternative to that AFAIK10:26
dtantsurrpittau: well, recent fedora do it somehow :)10:26
dtantsurI guess we should move it to #openstack-dib and continue there10:26
rpittaudtantsur: recent fedora ?10:27
dtantsurrpittau: I mean, any OS is bootstrapped via the same process, it has to be possible10:27
dtantsurI guess DIB is not aware of that though10:27
derekhdtantsur: 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
dtantsurderekh: not exactly. it's possible only in theory (and not trivial even there), I don't think it works in practice.10:30
dtantsurwe're working on other approaches to identify nodes10:31
dtantsurTheJulia made a patch to match IPMI addresses against Redfish addresses (because they match very often)10:31
dtantsurjanders is working on MAC address detection for Redfish nodes10:31
rpittaudtantsur: the DIB jobs still run on bionic, that's why all's good on their side10:32
derekhdtantsur: ack, thanks10:33
*** bbezak has quit IRC10:33
*** PrinzElvis has quit IRC10:33
*** tkajinam has quit IRC10:33
dtantsurrpittau: yeah, it should be raised to them (and to whoever is driving the Focal goal)10:33
rpittauyep10:33
*** bbezak has joined #openstack-ironic10:34
*** PrinzElvis has joined #openstack-ironic10:34
*** tkajinam has joined #openstack-ironic10:34
rpittauI had a quick discussion a month ago about that, I will raise it again10:35
openstackgerritYogesh Ramachandra proposed openstack/ironic master: Fix idrac-wsman bios factory-reset setting  https://review.opendev.org/74869610:40
*** mkrai has joined #openstack-ironic10:44
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Declare support for Fedora 32 and openSUSE Leap 15.2  https://review.opendev.org/75625410:47
dtantsurrpittau: re bifrost CI ^^^10:48
dtantsurwill have to be applied to Ussuri+ since the CI changes have been done there10:49
openstackgerritDmitry Tantsur proposed openstack/bifrost master: [PoC] A naive attempt at creating an upgrade job  https://review.opendev.org/75037610:58
openstackgerritAija Jaunteva proposed openstack/ironic master: Update `cleaning_error_handler`  https://review.opendev.org/75625811:00
openstackgerritMark Goddard proposed openstack/bifrost master: Install DIB dependencies in a separate role  https://review.opendev.org/75600511:06
janderssee you tomorrow Ironic o/11:10
*** ricolin_ has quit IRC11:10
*** ricolin has joined #openstack-ironic11:11
*** Abdallahyas has quit IRC11:21
*** abdysn has joined #openstack-ironic11:22
*** Qianbiao_ has quit IRC11:32
*** thiagop has joined #openstack-ironic11:34
*** mkrai has quit IRC11:41
*** mkrai has joined #openstack-ironic11:41
*** Goneri has joined #openstack-ironic11:48
*** mkrai has quit IRC12:00
*** zzzeek has quit IRC12:17
*** zzzeek has joined #openstack-ironic12:18
*** jtomasek_ has joined #openstack-ironic12:23
*** jtomasek has quit IRC12:26
QianbiaoHello, is "Customization Script" feature in Openstack Web console maps to ironic configdrive?12:30
arne_wiebalckThis is the user-data used by cloud-init. It is stored on the config drive.12:43
*** jamesdenton has quit IRC12:46
Qianbiaoarne_wiebalck ty12:47
*** jamesdenton has joined #openstack-ironic12:47
QianbiaoIn ramdisk deploy interface document: https://docs.openstack.org/ironic/train/admin/interfaces/deploy.html12:48
Qianbiaoit says: Configuration drives are not supported.12:48
QianbiaoIt means we can not add any customer init to it then?12:48
Qianbiaothen how user can login to the server. (customer image with per-set ssh-key)?12:49
*** rh-jelabarre has joined #openstack-ironic13:00
openstackgerritMerged openstack/bifrost master: Declare support for Fedora 32 and openSUSE Leap 15.2  https://review.opendev.org/75625413:02
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/victoria: Declare support for Fedora 32 and openSUSE Leap 15.2  https://review.opendev.org/75628713:02
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: DNM test CI  https://review.opendev.org/75628813:03
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/victoria: Switch the CI to stable/victoria  https://review.opendev.org/75628913:05
openstackgerritAija Jaunteva proposed openstack/ironic master: Fix redfish BIOS apply config error handling  https://review.opendev.org/75577613:13
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Use bionic nodeset for DIB centos src jobs  https://review.opendev.org/75629113:18
Qianbiaoarne_wiebalck SIG will starts in an hour right?13:18
Qianbiao40 minutes later13:19
dtantsurseems so13:20
TheJuliagood morning13:22
rpittaugood mornning TheJulia :)13:23
QianbiaoMorning TheJulia13:23
QianbiaoFor ramdisk deploy, we can not use configdrive, but we can still use metadata service to pass user-data, right?13:23
TheJuliayes13:24
TheJuliawelcome back rpittau13:24
rpittauthanks :)13:25
*** rloo has joined #openstack-ironic13:25
TheJuliaso did I miss anything yesterday?13:30
arne_wiebalckQianbiao: yes, in about 30 mins13:31
Qianbiaook13:32
arne_wiebalckdtantsur: TheJulia: it seems disabling the udev queues does not help with the RAID issue, but re-adding does13:32
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Re-add missing devices  https://review.opendev.org/75630013:33
TheJuliaarne_wiebalck: re-adding as in ripping out the raid and reconstructing?13:33
arne_wiebalckTheJulia: no, checking what was ejected and stuffing it back in13:34
TheJuliaoh yeah, that would work13:34
arne_wiebalckI 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 tests13:36
arne_wiebalckudev still remains a suspect13:36
arne_wiebalckit may detect the new partitions just created and grabs them while we put the superblock on it13:37
TheJuliai 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 anymore13:37
arne_wiebalckthe udev rules for md also skip any action if run while anaconda runs, with a comment saying that anaconda brings up devices ... seems like is13:37
arne_wiebalck*us13:37
TheJuliahmm13:38
* arne_wiebalck looks for the link ...13:38
TheJuliaYeah, i guess we really don't need ot trying to be helpful13:38
openstackgerritMerged openstack/bifrost stable/ussuri: Change LANG in install-deps.sh to en_US.UTF-8  https://review.opendev.org/75621413:39
openstackgerritMerged openstack/bifrost stable/victoria: Declare support for Fedora 32 and openSUSE Leap 15.2  https://review.opendev.org/75628713:39
arne_wiebalckTheJulia: https://github.com/neilbrown/mdadm/blob/master/udev-md-raid-assembly.rules#L513:40
arne_wiebalckTheJulia: so, there seems to be some unwanted interference and maybe that bites us as well13:40
arne_wiebalckTheJulia: anyway, I am still deleting and creating nodes in a loop and the above patch seems to at least patch it13:41
* arne_wiebalck would not call this a fix13:41
*** thiagop has quit IRC13:42
*** tzumainn has joined #openstack-ironic13:44
* arne_wiebalck called it a fix in the reno :-S13:44
*** thiagop has joined #openstack-ironic13:45
TheJuliaheh13:46
TheJuliashort of knowing we have disabled udev in this case, it is a fix13:46
arne_wiebalckeither udev is not to blame for this problem, or I disabled it the wrong way13:48
arne_wiebalckudev is not very verbose either, at least not by defautl13:48
arne_wiebalckdefault13:48
TheJuliadtantsur: 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
dtantsurTheJulia: morning! no objections from me.13:51
dtantsurI hope you feel better today13:51
TheJuliaokay, 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 work13:51
TheJuliaa lot better, slept for like 5 additional hours yesterday13:52
dtantsurnice13:52
dtantsurtoday it's my turn though13:52
dtantsurI guess I'll skip the SIG meeting, unless the agenda includes a zombie show13:53
dtantsurarne_wiebalck: ^^13:53
TheJuliaugh, feel better!13:54
dtantsurthx13:54
iurygregorygood morning TheJulia =)13:56
arne_wiebalckdtantsur: get better ... no zombies I hope!14:00
*** abdysn has quit IRC14:00
openstackgerritDmitry Tantsur proposed openstack/ironic master: Trivial: use the correct error handler for do_next_clean_step  https://review.opendev.org/75632614:38
iurygregoryIPE seems happy with ubuntu focal after disabling dstat \o/14:46
rpittauI guess we'll have to disable it globally14:52
rpittauIF we want to use focal14:52
openstackgerritVerification of a change to openstack/ironic-tempest-plugin failed: Wait for power off before allocation tests  https://review.opendev.org/75566114:53
*** trandles has joined #openstack-ironic14:54
*** thiagop has quit IRC14:55
TheJuliadisabling dstat?!?15:01
rpittauyeah15:01
rpittauit crashes on focal15:01
TheJulia\o/15:01
iurygregoryyeah \o/15:01
iurygregoryit's a party15:02
rpittau\o/15:02
rpittauan alternative that I'm considering for centos images is to convert to centos-8 nodeset15:02
rpittauconsidering that yum-tools is not in focal15:02
rpittauI'm not sure of the amount of work that requires though15:03
dtantsurI suspect fixing DIB is easier..15:03
rpittauI'm not sure15:03
dtantsuryou're suggesting to make devstack CI work reliably on centos 815:05
dtantsurthat's no small adventure :)15:05
rpittauyeah :D15:05
iurygregoryit's a huge adventure15:05
iurygregorythe roller coaster has a lot of loops and missing rails15:06
Qianbiaodtantsur It says on dev document that it supports centos8?15:08
Qianbiao* devstack document15:08
rpittauQianbiao: yeah, the basic devstack, we're talking about devstack + ironic :)15:09
Qianbiao:) ok15:09
openstackgerritDmitry Tantsur proposed openstack/ironic master: Refactoring: split away continue_node_deploy/clean  https://review.opendev.org/75634315:27
*** trandles has quit IRC15:30
*** uzumaki has quit IRC15:43
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Use focal for tinyipa src jobs used by ipa-builder  https://review.opendev.org/75635315:43
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Do not use RPC continue_node_deploy/clean in agent_base  https://review.opendev.org/75635415:45
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [DNM] test converting src jobs to focal  https://review.opendev.org/75635515:46
rpittaugood night! o/15:48
*** rpittau is now known as rpittau|afk15:48
openstackgerritMerged openstack/ironic-python-agent stable/train: Increase memory for tinyipa jobs  https://review.opendev.org/75608215:48
*** k_mouza has quit IRC15:52
*** thiagop has joined #openstack-ironic15:52
*** k_mouza has joined #openstack-ironic16:02
*** lucasagomes has quit IRC16:03
*** Qianbiao has quit IRC16:05
*** mkrai has joined #openstack-ironic16:11
openstackgerritMerged openstack/bifrost stable/victoria: Another case for skip_package_install  https://review.opendev.org/75611616:12
openstackgerritMerged openstack/bifrost master: Explicitly unset OS_CLOUD in the generated openrc  https://review.opendev.org/75611716:12
*** rloo has quit IRC16:15
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/victoria: Explicitly unset OS_CLOUD in the generated openrc  https://review.opendev.org/75636416:17
*** mkrai has quit IRC16:18
*** dtantsur is now known as dtantsur|afk16:18
dtantsur|afko/16:18
*** rloo has joined #openstack-ironic16:18
*** trandles has joined #openstack-ironic16:25
*** rloo has quit IRC16:29
*** rloo has joined #openstack-ironic16:30
iurygregoryTheJulia, 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.o16:34
TheJuliaiurygregory: yeah, because it got refactored so the code path could be reused by both drivers16:36
*** rh-jlabarre has joined #openstack-ironic16:44
*** rh-jelabarre has quit IRC16:45
arne_wiebalckbye everyone o/16:45
*** rloo has quit IRC16:48
*** ociuhandu has quit IRC16:48
*** rloo has joined #openstack-ironic16:49
iurygregoryTheJulia, yeah I was just wondering that would make more sense to have a class testing with ilo parameters..16:49
TheJuliaeither or both I guess16:50
*** jtomasek_ has quit IRC16:54
*** derekh has quit IRC17:04
openstackgerritMerged openstack/ironic-python-agent master: Add basic retries for inspection  https://review.opendev.org/74559817:07
*** k_mouza has quit IRC17:25
*** k_mouza has joined #openstack-ironic17:32
*** k_mouza has quit IRC17:35
*** dking has quit IRC17:56
openstackgerritMerged openstack/bifrost stable/victoria: Switch the CI to stable/victoria  https://review.opendev.org/75628917:58
*** larsks has quit IRC18:00
*** gyee has joined #openstack-ironic18:16
*** k_mouza has joined #openstack-ironic18:33
*** k_mouza has quit IRC18:40
*** k_mouza has joined #openstack-ironic18:47
*** k_mouza has quit IRC18:51
*** k_mouza has joined #openstack-ironic18:56
*** k_mouza has quit IRC19:00
*** k_mouza has joined #openstack-ironic19:05
*** k_mouza has quit IRC19:09
*** yolanda has quit IRC19:20
*** yolanda has joined #openstack-ironic19:20
*** k_mouza has joined #openstack-ironic19:23
*** k_mouza has quit IRC19:27
*** lmcgann has joined #openstack-ironic19:45
*** k_mouza has joined #openstack-ironic19:51
*** tosky has quit IRC19:52
*** k_mouza has quit IRC19:55
*** ociuhandu has joined #openstack-ironic19:56
lmcgannWorking 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
lmcgannAlso, 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
lmcgannEven 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 IRC20:08
*** k_mouza has joined #openstack-ironic20:09
*** k_mouza has quit IRC20:13
TheJuliayes, enrollment is before validation and in enrollment state we may know literally nothing about the machine20:13
TheJuliavalidation can be called at any time and it is an interface sanity check/status check for the most part20:13
TheJulialikely you just want to do on deployment/cleaning, since deleting is actually cleaning20:14
TheJuliaand cleaning runs before nodes are available to provision20:14
lmcgannso then, perhaps node_enrollment should become node_manageable20:14
TheJuliaso, manageable is "I think i can manage it, would you like to manage it?"20:14
lmcgannor immediately after validation (which is manageable, it just sounds misleading)20:15
TheJuliaprovide verb moves the node into cleaning and eventually available state20:15
TheJuliaprovide verb basically means "yes, you manage it"20:15
*** zzzeek has quit IRC20:15
*** zzzeek has joined #openstack-ironic20:16
lmcgannIm 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
lmcgannand 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 verifier20:20
*** thiagop has quit IRC20:43
*** k_mouza has joined #openstack-ironic20:52
*** larsks has joined #openstack-ironic20:53
*** k_mouza has quit IRC20:56
*** k_mouza has joined #openstack-ironic21:05
rpiosoTheJulia: Should https://docs.openstack.org/ironic/victoria/ be working? I get a 404.21:05
*** k_mouza has quit IRC21:09
TheJuliaugh uefi on supermicro hardware + supermicro hardware netbooting == ugh21:12
TheJuliarpioso: not quite yet if memory seves21:12
TheJuliaserves21:12
TheJuliait might take a patch merging though at this point21:12
rpiosoTheJulia: Thank you!21:12
TheJulialmcgann: I suspect you may be overthinking it slightly (a normal reaction), would it be possible to talk through it tomorrow?21:13
lmcgannTheJulia sure21:19
TheJuliabfournie: https://www.supermicro.com/support/faqs/faq.cfm?faq=2220821:21
*** HardCase has quit IRC21:36
*** lmcgann has quit IRC22:02
*** k_mouza has joined #openstack-ironic22:10
*** k_mouza has quit IRC22:15
TheJuliabfournie: 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-ironic22:20
*** k_mouza has quit IRC22:24
openstackgerritMerged openstack/ironic-tempest-plugin master: Wait for power off before allocation tests  https://review.opendev.org/75566122:29
*** rloo has quit IRC22:30
openstackgerritRichard G. Pioso proposed openstack/ironic stable/train: DRAC: Fix a failure to create virtual disk bug  https://review.opendev.org/75589522:31
*** k_mouza has joined #openstack-ironic22:40
bfournieTheJulia: what should IPA use to detect actual boot mode?22:43
TheJuliagood question22:44
TheJuliahmmm22:44
*** k_mouza has quit IRC22:45
TheJuliabfournie: https://github.com/openstack/ironic-python-agent/blob/99dee5067ea4f06d3083170d801e600f46842170/ironic_python_agent/hardware.py#L118922:46
TheJuliabfournie: basically it is not seeing efi firmware present at all which makes sense if the network card is bios booting22:53
TheJuliawhich is what would happen if it was relying upon the bios image in the chip instead of the uefi image22:53
bfournieTheJulia: 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:114922:54
TheJuliabingo22:54
TheJuliabfournie: I sent you photos from my machine at home22:54
TheJuliahopefully they help22:54
bfournieTheJulia: that's it, yes awesome photos!  we'll get on a supermicro and change those settings tomorrow22:55
TheJulia\o/22:55
bfourniethanks a lot TheJulia !!22:56
TheJuliano problem :)22:56
*** rcernin_ has joined #openstack-ironic22:56
TheJulianow, to unwind my brain a little before I cook dinner22:56
*** rcernin_ has quit IRC22:58
*** rcernin has joined #openstack-ironic22:58
*** gudrutis has quit IRC23:11
*** gudrutis has joined #openstack-ironic23:12
jandersGood morning Ironic o/23:32
*** k_mouza has joined #openstack-ironic23:42
jandersTheJulia 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
TheJuliaAdd to the ether pad?23:45
jandersdo you have a link handy? (googling results in a *ton* of old links)23:46
jandersdisregard, I was able to modify old URL (this didn't work last time I tried)23:47
*** k_mouza has quit IRC23:47

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!