Wednesday, 2020-04-15

*** threestrands has joined #openstack-ironic00:00
openstackgerritVerification of a change to openstack/ironic failed: Extend timeout on CI job with automated cleaning  https://review.opendev.org/72006500:02
*** cdearborn has quit IRC00:23
*** ricolin has joined #openstack-ironic01:13
*** ricolin_ has joined #openstack-ironic01:19
*** ricolin has quit IRC01:21
*** mraineri has joined #openstack-ironic01:26
*** Goneri has quit IRC01:37
openstackgerritzhufl proposed openstack/ironic master: Fix missing print format in log messages  https://review.opendev.org/72011401:41
*** Qianbiao has joined #openstack-ironic01:50
Qianbiaomorning ironic.01:54
*** ociuhandu has joined #openstack-ironic01:56
*** mkrai has joined #openstack-ironic02:07
*** ociuhandu has quit IRC02:13
*** k_mouza has joined #openstack-ironic02:23
*** k_mouza has quit IRC02:27
*** tzumainn has quit IRC02:51
*** mkrai has quit IRC03:23
*** mkrai has joined #openstack-ironic03:24
*** hamzy_ has joined #openstack-ironic03:29
*** hamzy has quit IRC03:31
*** mkrai has quit IRC04:05
*** mkrai_ has joined #openstack-ironic04:06
*** gyee has quit IRC04:13
*** mkrai has joined #openstack-ironic04:14
*** ociuhandu has joined #openstack-ironic04:15
*** mkrai_ has quit IRC04:17
*** ociuhandu has quit IRC04:33
*** mkrai has quit IRC05:01
*** Qianbiao has quit IRC05:05
*** mkrai has joined #openstack-ironic05:06
*** dhellmann has quit IRC05:16
openstackgerritSteve Baker proposed openstack/ironic master: Centralise imports of wsme types  https://review.opendev.org/70389805:28
openstackgerritSteve Baker proposed openstack/ironic master: Don't use wsme test webapp for patch tests  https://review.opendev.org/70448505:28
openstackgerritSteve Baker proposed openstack/ironic master: Do all serialization in the expose decorator  https://review.opendev.org/70448605:28
openstackgerritSteve Baker proposed openstack/ironic master: Add validate decorator to expose  https://review.opendev.org/70448705:28
openstackgerritSteve Baker proposed openstack/ironic master: WIP Add function definition handling  https://review.opendev.org/70448805:28
openstackgerritSteve Baker proposed openstack/ironic master: WIP Add json parsing to args  https://review.opendev.org/70448905:28
openstackgerritSteve Baker proposed openstack/ironic master: WIP Add wsme core types, remove WSME  https://review.opendev.org/70449005:28
*** dhellmann has joined #openstack-ironic05:31
*** dhellmann has quit IRC05:38
*** ociuhandu has joined #openstack-ironic05:39
*** dhellmann has joined #openstack-ironic05:45
*** ociuhandu has quit IRC05:51
*** threestrands has quit IRC05:54
*** dhellmann has quit IRC05:58
*** Qianbiao has joined #openstack-ironic05:59
*** dhellmann has joined #openstack-ironic06:13
*** ociuhandu has joined #openstack-ironic06:26
*** dhellmann has quit IRC06:34
*** Qianbiao is now known as Qianbiao|afk06:36
*** bdodd has quit IRC06:36
*** bdodd has joined #openstack-ironic06:39
*** dhellmann has joined #openstack-ironic06:40
iurygregorygood morning Ironic!06:41
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Add BasePhysnetHook  https://review.opendev.org/72001006:45
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network hook  https://review.opendev.org/71859406:45
arne_wiebalckGood morning iurygregory and ironic!06:53
iurygregoryarne_wiebalck, o/07:05
*** Qianbiao|afk is now known as Qianbiao07:09
Qianbiaohello ironic.07:09
Qianbiaohello, when deploy os to baremetal, an error occurs: openstack:85c78981-f85a-49e1-9c67-0b3cb93dfc4c-lun-1 bs=1M oflag=direct (no filter matched)07:10
Qianbiaoany hints?07:10
Qianbiaolog details: http://paste.openstack.org/show/792137/07:11
*** rpittau|afk is now known as rpittau07:28
rpittaugood morning ironic! o/07:28
*** iurygregory has quit IRC07:31
*** iurygregory has joined #openstack-ironic07:32
iurygregorymorning rpittau o/07:33
rpittauhey iurygregory :)07:33
*** ociuhandu has quit IRC07:40
*** ociuhandu has joined #openstack-ironic07:41
iurygregoryQianbiao, to me it seems it couldn't configure the node, do you have some conductors and ipa logs ?07:41
iurygregorymaybe you are using a wrong partition07:42
Qianbiao<iurygregory> log details: http://paste.openstack.org/show/792137/07:43
Qianbiaoyour name contains a lot of wierd chars when copy07:43
iurygregoryQianbiao, I was looking at this link, but this is not enough I would say07:43
Qianbiaoi see, there are only those useful log in conductor07:44
iurygregoryconductor, ipa07:44
iurygregorymore details are always welcome =)07:44
Qianbiaoit seems this path does not exists: /var/lib/ironic/http/images/85c78981-f85a-49e1-9c67-0b3cb93dfc4c/disk07:44
Qianbiaoand instead it has : 7a91f5a4-12f8-4886-8da0-5e08c98354ce  ubuntu-xenial-16.04.meta  ubuntu-xenial-16.04.qcow207:45
Qianbiaonot sure why it got a wrong id07:45
iurygregoryhummm07:45
iurygregoryyeah07:45
iurygregoryyou will need to try again .-.07:46
*** ociuhandu has quit IRC07:46
Qianbiaosure will figure it out07:46
iurygregorygood luck!07:47
iurygregoryarne_wiebalck, rpittau if a partition is found in /dev/sda the deploy logs should show this partition in the lsblk output right?07:52
rpittauiurygregory: yes, remind me the exact command we're using07:55
iurygregorylet me find here07:55
iurygregoryIm puzzled since uefi partition is found in sda15 but lsblk doesn't show O.o07:56
openstackgerritVerification of a change to openstack/ironic failed: Extend timeout on CI job with automated cleaning  https://review.opendev.org/72006507:56
iurygregoryrpittau, https://github.com/openstack/ironic-python-agent/blob/3da356de257e10ddbad1f84054d1d2357d152941/ironic_python_agent/utils.py#L6707:57
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: BUILD_TIMEOUT is not needed  https://review.opendev.org/72001607:57
iurygregoryIn the logs for wholedisk http://paste.openstack.org/show/792141/07:58
arne_wiebalckiurygregory: this is not a partition which comes with the image, is it? Since the lsblk output is from before the image is dumped.07:58
arne_wiebalckI would think :)07:58
iurygregoryit's from before ? O.o even if we only get the deploy logs after ipa runs?07:59
rpittauyeah, that depends on when the command runs07:59
iurygregoryhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_240/714453/12/check/ironic-tempest-ipa-wholedisk-uefi-ipmi-iscsi-tinyipa/2402d0c/controller/logs/ironic-bm-logs/node-0_console_2020-04-14-14%3A45%3A49_log.txt07:59
*** lucasagomes has joined #openstack-ironic08:14
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] Dump info on pxeboot for uefi jobs  https://review.opendev.org/71995608:15
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client master: Document the new standalone CLI  https://review.opendev.org/72003408:17
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf  https://review.opendev.org/71688908:18
*** tkajinam has quit IRC08:21
*** ociuhandu has joined #openstack-ironic08:22
openstackgerritMahendra Kamble proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port  https://review.opendev.org/69332808:23
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251608:24
mgoddardmorning08:25
mgoddardhas anyone looked at https://tinkerbell.org/ ?08:25
iurygregoryfirst time i hear about it08:26
mgoddardmight be worth a look08:26
*** ociuhandu has quit IRC08:29
iurygregoryOct 14, 2019 O.o08:29
*** ociuhandu has joined #openstack-ironic08:29
iurygregoryand the other projects are very new O.o08:30
*** ricolin_ is now known as ricolin08:30
openstackgerritDmitry Tantsur proposed openstack/ironic master: Use trailing slash in the agent command URL  https://review.opendev.org/72015008:30
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Convert job to dib  https://review.opendev.org/71869808:35
*** alexmcleod has joined #openstack-ironic08:39
*** Qianbiao has quit IRC08:44
*** dtantsur|afk is now known as dtantsur08:49
dtantsurmorning ironic, says man debugging agent tokens08:49
*** Qianbiao has joined #openstack-ironic08:50
rpittaugood morning dtantsur08:50
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: Document deploy_boot_mode and boot_option for standalone deployments  https://review.opendev.org/72015608:55
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: DNM try ussuri IPA  https://review.opendev.org/72015708:57
*** k_mouza has joined #openstack-ironic09:01
*** dhellmann has quit IRC09:05
iurygregorymorning dtantsur09:07
*** dhellmann has joined #openstack-ironic09:11
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830909:22
Qianbiaohello, ironic.09:30
Qianbiaoneed help. iscsi seems could not load right image, logs: http://static.turnbig.net/ironic/09:31
Qianbiaothis is logs from IPA: http://static.turnbig.net/ironic/deploy/4269dbec-b05d-4471-83d5-3cb3cae8528c_2020-04-15-09-19-09.tar.gz09:31
dtantsurTheJulia: could you remove -2 from https://review.opendev.org/#/c/708827/ please? we need to land something to rebuild the train image09:34
patchbotpatch 708827 - ironic-python-agent (stable/train) - Rescan after restarting the md device - 2 patch sets09:34
dtantsurTheJulia: also I don't think https://storyboard.openstack.org/#!/story/2007455 is fixed. It won't even reach the part that your patch is touching.09:39
dtantsurthe fact that UEFI whole disk images don't work if the efibootmgr approach fails is.. frustrating to people09:42
dtantsuriurygregory: did you have any success with ^^^?09:43
iurygregorydtantsur, I got a strange working job... it succeeds with tempest but the node logs tell me something completely different https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_240/714453/12/check/ironic-tempest-ipa-wholedisk-uefi-ipmi-iscsi-tinyipa/2402d0c/controller/logs/ironic-bm-logs/node-0_console_2020-04-14-14%3A45%3A49_log.txt09:44
dtantsurWUT09:45
iurygregoryyeah that was my reaction also =)09:45
iurygregorywe found a efi partition in sda15 but the mount fails09:46
dtantsuriurygregory: the person I'm talking about reports a similar thing: bootloader fails, but the OS boots09:56
iurygregorywut?! =O09:57
iurygregoryblack magic09:57
*** jamesdenton_wfh has joined #openstack-ironic10:10
openstackgerritAija Jaunteva proposed openstack/ironic master: Update iDRAC doc about vendor passthru timeout  https://review.opendev.org/72017310:11
openstackgerritMerged openstack/ironic-python-agent master: Move minimum ironic version to latest ocata  https://review.opendev.org/71845010:22
dtantsuriurygregory: wanna more magic? in ubuntu images the esp partition doesn't have the esp flag, like here https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/tests/unit/test_utils.py#L5510:22
dtantsurso it's only called ESP and has only 'boot' flag10:22
dtantsurlovely....10:22
dtantsurI wonder if we can rely on name+boot as well10:22
*** rpittau is now known as rpittau|bbl10:23
dtantsuroh, esp is an alias for boot on GPT. needs fixing.10:29
iurygregoryOMG10:35
iurygregorydtantsur, I can look at this latter ( I have a session to present in 25min .-.) 5hrs of event .-.10:36
dtantsuriurygregory: I'm working on a patch, don't worry10:36
iurygregorydtantsur, tks!10:36
* iurygregory will be afk the whole afternoon basically10:36
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: A boot partition on a GPT disk should be considered an EFI partition  https://review.opendev.org/72018410:46
dtantsuriurygregory: if you could take a quick look ^^^ would be awesome10:46
iurygregorydtantsur, looking now10:47
iurygregorydtantsur, makes sense to me!10:48
*** derekh has joined #openstack-ironic10:49
*** jtomasek has quit IRC10:56
*** jtomasek has joined #openstack-ironic11:01
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Fix the token logic to be compatible with older ironic  https://review.opendev.org/72018511:01
dtantsurTheJulia: ^^^11:02
Qianbiaohello, may someone help look at a deployment problem, takes me a long time and could not figure out why. logs: http://paste.openstack.org/show/792146/ http://paste.openstack.org/show/792147/11:02
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: DNM try ussuri IPA  https://review.opendev.org/72015711:04
*** etingof is now known as etingof|brb11:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830911:08
QianbiaoAm i using a wrong IPA? the version i used is tinyipa-master.11:08
Qianbiaois tinyipa-master ready for ussuri11:08
*** Qianbiao is now known as Qianbiao|afk11:15
*** cdearborn has joined #openstack-ironic11:15
*** iurygregory is now known as iurygregory_afk11:31
*** Qianbiao|afk has quit IRC11:37
*** Qianbiao|afk has joined #openstack-ironic11:38
dtantsurtinyipa-master is ussuri, yes11:41
*** ociuhandu has quit IRC11:42
arne_wiebalckdtantsur: iurygregory_afk: I haven't looked at the context tbh, but will 720184 cause issues for BIOS images which use GPT?11:48
iurygregory_afkarne_wiebalck, when I looked at the changes it seems that we are only touching the uefi check11:51
arne_wiebalckiurygregory_afk: as in, we know this is UEFI and we only need to find the partition?11:52
*** dsneddon has quit IRC11:53
iurygregory_afkarne_wiebalck, as in we are only touching get_efi_part_on_device =)11:53
iurygregory_afki don't think the path for bios is using this function =)11:53
arne_wiebalckiurygregory_afk: ok, thanks11:54
arne_wiebalckiurygregory_afk: I triggered on this as we have BIOS images with GPT11:54
iurygregory_afkarne_wiebalck, gotcha =) no worries11:55
*** rh-jelabarre has joined #openstack-ironic12:07
*** etingof|brb is now known as etingof12:10
dtantsurarne_wiebalck: BIOS images that use GPT use a different partition type for their partitions12:19
dtantsurnamely, legacy_boot12:19
dtantsurs/their/legacy boot/12:20
arne_wiebalckright, but I thought the patch will use the existence of the GPT to make assumptions about the type12:20
dtantsurarne_wiebalck: it does, check the updated utils12:20
*** Goneri has joined #openstack-ironic12:21
arne_wiebalckdtantsur: that was my worry :)12:22
arne_wiebalckdtantsur: that the sheer existence of a GPT would imply an EFI partition and hence UEFI, but the logic seems to be we are in UEFI mode, cannot find an EFI partition and now assume the one with the GPT is the one we're looking for12:24
dtantsurarne_wiebalck: a partition with label "boot" in a GPT table is an EFI partition, at least according to parted docs.12:24
dtantsurbut yes, we seem to check the boot mode as well12:25
*** ociuhandu has joined #openstack-ironic12:31
*** ociuhandu has quit IRC12:32
*** ociuhandu has joined #openstack-ironic12:32
arne_wiebalckdtantsur: our images have a GPT partition table with type "BIOS boot"12:34
dtantsurarne_wiebalck: right, it will probably be legacy_boot in parted12:35
dtantsurmmmm, sorry, bios_grub12:35
dtantsurarne_wiebalck: see an example I got from a real deployment: https://storyboard.openstack.org/#!/story/2007455#comment-15563912:36
arne_wiebalckdtantsur: "BIOS boot" is what parted displays12:36
dtantsurdo you call it with the same arguments? could you paste the output?12:36
arne_wiebalckoh, these are flags, I was talking about type12:37
*** rloo has joined #openstack-ironic12:37
arne_wiebalcksure, once sec ...12:37
dtantsurwe seem to be looking at flags12:37
dtantsurbut it's a good idea to check if "BIOS boot" in the name will trigger an incorrect path12:38
arne_wiebalcksorry, I created some confusion12:38
dtantsurmeh, our regex logic is bad indeed12:39
dtantsurarne_wiebalck: could you still paste your output?12:39
arne_wiebalckhttp://paste.openstack.org/show/792157/12:39
arne_wiebalckI mixed fdisk and parted12:39
arne_wiebalckall there ^^12:39
arne_wiebalckthe flag is indeed bios_grub as you said12:40
arne_wiebalckthis is on a VM, btw12:40
arne_wiebalcknot a physical node12:40
dtantsurarne_wiebalck: thanks! yeah, I've realized that having "BIOS boot" in a name will trigger the incorrect logic. Thinking how to fix it...12:40
*** yolanda has quit IRC12:43
dtantsurarne_wiebalck: could you also try parted with -m flag? I'm trying to avoid parsing the human-readable representation.12:48
arne_wiebalcksure, one sec ...12:48
*** rpittau|bbl is now known as rpittau12:49
arne_wiebalckdtantsur: http://paste.openstack.org/show/792158/12:51
dtantsurthanks!12:51
TheJuliagood morning12:52
dtantsurmorning TheJulia12:52
iurygregory_afkgood morning TheJulia12:54
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] A boot partition on a GPT disk should be considered an EFI partition  https://review.opendev.org/72018412:55
dtantsurarne_wiebalck, iurygregory_afk, this is what I ended up with ^^^ (needs unit test updates)12:55
iurygregory_afkdtantsur, in 5 min I will look after the session is over here o/12:56
* dtantsur wonders if IPA has any jobs with UEFI12:56
iurygregory_afkdtantsur, I would say no .-.12:57
iurygregory_afkhttps://github.com/openstack/ironic-python-agent/blob/master/zuul.d/project.yaml12:57
dtantsurthat's bad..12:57
iurygregory_afkyeah12:57
dtantsurso I'm essentially shooting in darkness :)12:57
iurygregory_afkboom!12:57
openstackgerritMerged openstack/ironic master: Extend timeout on CI job with automated cleaning  https://review.opendev.org/72006512:58
TheJuliadtantsur:  /me blinks13:03
TheJuliaerr13:03
* TheJulia blinks13:03
dtantsur:)13:03
dtantsurI'd recommend against opening eyes today, but since you already did...13:03
TheJuliadownside of changing the base ipa image, things are more unstable again13:04
dtantsurnot entirely unexpected, unfortunately13:04
TheJuliayeah13:04
TheJuliadid you ever figure out why all timeoutes in tempest really like 12 minutes?13:05
TheJuliaerr13:05
TheJuliareally like 10 minutes when set to 1213:05
dtantsurfirst time I hear about it13:05
TheJuliayou've banged your head against it before13:05
TheJuliaI remember the grumpiness of that day13:05
TheJulia:)13:05
dtantsurprobably banged it too hard :D13:05
rpittauTheJulia: timeouts are confusing me a lot too, the variables for the tempest jobs are 3 apparently13:08
TheJuliayes!13:08
TheJuliathey are :(13:08
* TheJulia gets out lots of booze13:08
TheJuliasurely someone has created a beverage called "the tempest"...13:08
* rpittau gets a big mug13:09
dtantsurTheJulia: vodka with a lot of vodka and a small shot of vodka?13:09
TheJuliaSo "the tempest" is a riff on a "Dark and Stormy"13:10
rpittauI'm trying to figure out the issues with the dib images on some jobs that are timing out, can't see evidence of failures in the logs13:10
TheJuliahttps://www.foodandwine.com/recipes/tempest13:10
rpittaulike this one https://zuul.opendev.org/t/openstack/build/a95f6baa700d4917a6509e4fe669e4c213:11
TheJuliayeah, so there is an outer timeout on tests, then inner step level timeouts13:11
rpittaummm13:11
openstackgerritMerged openstack/ironic master: Software RAID: Pass the boot mode to the IPA  https://review.opendev.org/69616913:12
TheJuliaohhh that job is running erase_devices13:15
arne_wiebalckwoot13:15
TheJuliaso that is going to need a longer timeout13:16
TheJuliaso think 400-500 seconds for the ramdisk to be up and IPA running13:16
*** mbeierl has quit IRC13:16
arne_wiebalckthanks TheJulia and stendulker to throw 696169 under the zuul recheck bus over and over again!13:17
TheJuliaand then think another 300+ seconds to just run erase devices in the best cases with any io pressure13:17
TheJuliaarne_wiebalck: it was all timeout stuffs with other stuff, so happy to drive a bus over the patches a few... too many times13:17
*** mbeierl has joined #openstack-ironic13:18
rpittausounds like frogger but with the patch instead of the frog13:18
* TheJulia gets out the super glue to glue IRC clients to the room13:18
TheJulia:)13:18
TheJuliarpittau: very much so13:18
rpittauTheJulia: I have the timout already at 2400, will try with 3600 then13:19
TheJuliaso I think we should add more logging to our tempest plugin13:20
TheJuliaspecifically around bm wait states13:20
TheJuliaso when the match fails or we begin to exit out of them, we actually log that13:20
rpittauooohh non-standalone inspector is ok with "just" 240013:22
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Convert job to dib  https://review.opendev.org/71869813:22
TheJuliadtantsur: commented on your comment on https://review.opendev.org/#/c/720100/1 :)13:23
patchbotpatch 720100 - ironic-specs - WIP - Victoria Cycle Priorit(y|ies) - 1 patch set13:23
TheJuliafood for thought13:23
* iurygregory_afk grabs more coffee to prepare my mind for more 2hrs of programing contest13:24
iurygregory_afkdtantsur, are you sharing vodka?13:25
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251613:25
* iurygregory_afk only saw the word =P13:25
iurygregory_afkTheJulia, I added a comment for the priorities doc13:28
dtantsurTheJulia: running erase_devices is a no-go on virt13:30
dtantsurif we do it, we need to stop. it can take arbitrary amount of time, from minutes to hours13:30
dtantsuriurygregory_afk: vodka and programming contests? sounds appropriate :D13:30
iurygregory_afkdtantsur, it does!13:31
*** dhellmann has quit IRC13:32
iurygregory_afkI remember when I was learning python, the professor gave a list of questions for the class to solve, and there was 1 question that was pretty hard, one guy found the solution while we were at pub drinking13:33
dtantsurI'm pretty sure that's how python was invented, so fair enough13:34
*** dhellmann has joined #openstack-ironic13:35
dtantsurTheJulia: FYI https://storyboard.openstack.org/#!/story/200755713:35
*** rcernin has quit IRC13:37
dtantsurstable cores, please see https://review.opendev.org/#/c/718728/13:46
patchbotpatch 718728 - ironic-inspector (stable/stein) - [stable-only] Make ironicclient initialization fut... - 2 patch sets13:46
*** tzumainn has joined #openstack-ironic13:48
etingofwhat's the BM whitepaper meeting url please?14:00
dtantsuretingof:  https://cern.zoom.us/j/9569667382314:01
etingofo/14:01
TheJuliashould be able to join soon-ish, wrapping up a call14:01
Qianbiao|afkhello,  can we use qcow2 img in this link directly? https://cloud-images.ubuntu.com/releases/xenial/release/14:02
dtantsurQianbiao|afk: worth giving a try. sometimes it works (e.g. with centos images).14:04
*** Qianbiao|afk is now known as Qianbiao14:04
QianbiaoHmm, i got an error: Failed to check the number of primary partitions present on /dev/disk/by-path/ip-192.168.0.180:3260-iscsi-iqn.2008-10.org.openstack:4269dbec-b05d-4471-83d5-3cb3cae8528c-lun-1 for node 4269dbec-b05d-4471-83d5-3cb3cae8528c.14:04
Qianbiaomay it because the qcow2 image is broken?14:05
etingofzoom is hopelsss14:05
dtantsuryeah, not the easier one to set up14:06
dtantsur* easiest14:06
*** jcoufal has joined #openstack-ironic14:11
*** mkrai has quit IRC14:26
*** ociuhandu has quit IRC14:36
*** mkrai has joined #openstack-ironic14:37
rpiosoGood morning, ironic14:38
*** mkrai_ has joined #openstack-ironic14:42
*** mkrai_ has quit IRC14:42
*** mkrai_ has joined #openstack-ironic14:42
*** mkrai has quit IRC14:42
*** tkajinam has joined #openstack-ironic14:43
*** ociuhandu has joined #openstack-ironic14:43
jungleboyjarne_wiebalck:  Sorry for being so quiet.  I do still have it in my todos to read through the whole paper and make comments.  :-)14:52
arne_wiebalckjungleboyj: thanks for joining, your feedback will be very welcome!14:52
jungleboyj:-)14:53
*** mkrai_ has quit IRC14:54
*** mkrai_ has joined #openstack-ironic14:54
*** dtantsur is now known as dtantsur|brb14:55
*** mkrai_ has quit IRC14:55
TheJulia++14:58
*** priteau has joined #openstack-ironic15:00
* TheJulia wonders why her current cup of coffee tastes burnt15:02
*** gyee has joined #openstack-ironic15:04
rpittauwell apparently it's really just a matter of how long we want to wait https://review.opendev.org/#/c/718698/15:07
patchbotpatch 718698 - python-ironic-inspector-client - Convert job to dib - 3 patch sets15:07
*** jiriprox has joined #openstack-ironic15:07
*** ianychoi has quit IRC15:13
iurygregory_afkhttp://www.devstack.co.kr/ WUT O.o15:14
TheJuliasafe?15:14
iurygregory_afkyup15:14
iurygregory_afkMatthew Treinish shared in Promoting Ironic thread15:15
* iurygregory_afk never thought that devstack could be a name of a company XD15:16
rpittauI wonder if that is "legal"15:16
iurygregory_afkrpittau, I had this feeling also =)15:16
iurygregory_afkI also hope they don't use devstack in their product lol15:17
rpittaualso I could never use the excuse that "works in devstack" :P15:17
iurygregory_afkI've used this twice in my life so far15:17
jiriproxHi, I would like to ask you for help.15:18
jiriproxI’m trying to integrate new baremetal machine with SmartNIC into our Openstack environment.15:19
jiriproxIs there anything what needs to be added into ramdisk with IPA which is used to boot host (with SmartNIC)?15:19
iurygregory_afkafik ironic has the support for it and neutron too, and you probably need is_smartnic set15:20
iurygregory_afkI don't think you need to change the ramdisk15:21
rpittaumaybe specific kernel drivers?15:21
openstackgerritVerification of a change to openstack/ironic failed: Use trailing slash in the agent command URL  https://review.opendev.org/72015015:27
jiriproxSetting of is_smartnic in port connected to baremetal node seems to be not enough. I'm looking for someone who has experience with smartnic case.15:28
jiriproxKernel driver is available as well.15:29
*** Goneri has quit IRC15:30
jiriproxIt is using virtio driver.15:30
*** Goneri has joined #openstack-ironic15:33
*** Qianbiao has quit IRC15:40
*** iurygregory_afk is now known as iurygregory15:41
TheJuliajiriprox: Nothing in terms of IPA, but the smartnic integration flag is largely controlling how/when the machine is powered up because network programming via neutron needs the card powered up15:47
*** ianychoi has joined #openstack-ironic15:47
*** jcoufal has quit IRC15:48
*** jcoufal has joined #openstack-ironic15:52
*** tkajinam has quit IRC15:53
jiriproxTheJulia: Yes, that I've seen in https://review.opendev.org/#/c/583573/. Nevertheless I can't find anything what should be done on SmartNIC itself. If I understood it well then there is no procedure available for automatic OS deploy and configuration of SmartNIC.15:57
patchbotpatch 583573 - ironic - Add support for Smart NICs (MERGED) - 34 patch sets15:57
*** bdodd has quit IRC15:58
*** bdodd has joined #openstack-ironic16:01
*** lucasagomes has quit IRC16:09
*** dtantsur|brb is now known as dtantsur16:11
*** priteau has quit IRC16:12
*** jcoufal has quit IRC16:12
dtantsuroh donnyd has dropped a bomb :) (re the ironic ML thread)16:13
donnydplease don't hurt me16:13
arne_wiebalckdonnyd: ++16:13
*** ociuhandu has quit IRC16:20
*** jcoufal has joined #openstack-ironic16:20
donnydI feel like people may be about to send me 30 page responses... the lack of short ones at this point are semi concerning16:23
dtantsur:D16:25
donnydthis is also probably going to get me in trouble... but why is metal3 (kube wrapper for ironic) not using the opendev tooling? It would make life much mo easier wouldn't it ?16:27
dtantsurdonnyd: I actually suggested it not so long ago. The response it that metal3 shares more development with k8s than with openstack.16:28
dtantsurironic is largely consumed as is16:28
donnyd??? LOL - its largely ironic with some kube tooling..16:29
donnydits clearly outside my scope, purpose and level of clout... just doesn't make a lot of sense is all16:31
*** yaawang has joined #openstack-ironic16:32
donnydthanks for answer though dtantsur16:32
dtantsurdonnyd: well, images with ironic is only a small part of the metal3 project. it's bigger part is the newly developed baremetal operator and some installer bits16:33
arne_wiebalckbye everyone o/16:33
dtantsursee you arne_wiebalck16:33
rpittaubye, good night! o/16:33
donnydlater arne_wiebalck16:33
*** rpittau is now known as rpittau|afk16:33
*** yaawang_ has quit IRC16:34
donnyddtantsur: but functionally metal3 is a baremetal deployer for kube no ?16:34
*** ociuhandu has joined #openstack-ironic16:34
donnydlike I get the what is in development.. but what actual functions outside of native ironic stuff does it actually do ? ... I ask because I am curious16:35
dtantsurdonnyd: it provides custom resources for bare metal provisioning and for adding bare metal nodes to the cluster16:36
dtantsurit does not expose ironic itself to the users though16:37
donnydso it's CRD wrappers for ironic ?16:37
donnydlike translate this kube crd api call to an ironic one ?16:38
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: A boot partition on a GPT disk should be considered an EFI partition  https://review.opendev.org/72018416:39
dtantsurdonnyd: essentially yes16:39
donnydso now it makes even more sense to have it in the same CI ecosystem as ironic IMO16:40
donnydbut that is my +1 thought.. which means literally nothing16:40
donnydbecause wouldn't someone want to know for sure in some form of vetted test infra that the kube crd api call does in fact do the thing you wanted it to do...16:42
donnydlike we already have the test methods, process, and validation for metal in opendev... but then metal3 is tested somewhere else completely... comical IMO16:43
*** jcoufal has quit IRC16:45
dtantsurI think they care less about how precise we deploy bare metal, but more for end-to-end testing in the k8s context. I.e. that we can start with powered off machines and end up with a working cluster.16:47
donnydI don't want you to feel like I am harassing you here dtantsur  - I am just curious is all16:48
dtantsurno worries, I'm totally fine (but will have to leave in ~ 10 mins)16:49
donnydso the working cluster is provisioned by ironic?16:49
donnydat the bottom end16:49
dtantsurdonnyd: also worth noting that the people here are not really the main driving force behind the whole metal3. People writing the bare metal operators don't all have any openstack/opendev exposure.16:50
dtantsuryes, this is correct16:50
donnydyea that makes sense16:51
donnydjust a though is all.. like kinda falling in like with the desire to move ironic to a top level project would make a ton of sense if metal3 was coming along with it16:52
donnydinline*16:52
dtantsurit does make sense to me, but I'm not in charge there :)16:52
donnydthanks for helping me to understand dtantsur16:53
donnyd:)16:53
dtantsuryou're welcome16:53
dtantsurnow, folks, I'm slowly calling it a day16:53
*** dtantsur is now known as dtantsur|afk16:53
dtantsur|afko/16:53
*** k_mouza has quit IRC16:55
*** jcoufal has joined #openstack-ironic16:58
*** ociuhandu has quit IRC17:01
*** ociuhandu has joined #openstack-ironic17:02
*** priteau has joined #openstack-ironic17:05
*** ociuhandu has quit IRC17:08
*** ociuhandu has joined #openstack-ironic17:09
*** ricolin has quit IRC17:10
*** ociuhandu has quit IRC17:14
*** derekh has quit IRC17:17
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network hook  https://review.opendev.org/71859417:20
*** priteau has quit IRC17:26
*** yolanda has joined #openstack-ironic17:27
TheJuliacalling it a day sounds really nice17:35
TheJuliajiriprox: so the way it is supposed to work is out of band network configuration. Ironic doesn't deploy that agent on to the smartnic because it is something a running OS shouldn't be able to see and in working with the neutron folks it was determined because of the tie-ins and the needs/ability requirement to tie things in, that the networking had to inter-op with neutron for that networking config17:37
TheJuliadonnyd: A lot of that separation is also product driven in terms of decrees of how things shall be done17:38
TheJuliadonnyd: and they also want to spin up lots of memory intensive resources, so I'm kind of all for it staying out of OpenDev since we maintain a stable API, they should bea ble to leverage that.17:39
TheJuliaBut I'm also kind of crazy17:39
donnydTheJulia: So metal3 needs different flavors of test node for opendev to be more useful. That is what I heard you say17:42
TheJuliaAIUI, yeah17:44
TheJuliaGranted, I'm on the periphery of that discussion so my understanding may very well be wrong17:45
*** ociuhandu has joined #openstack-ironic17:52
*** dsneddon has joined #openstack-ironic18:11
*** ociuhandu has quit IRC18:15
*** dsneddon has quit IRC18:16
*** ociuhandu has joined #openstack-ironic18:22
*** Lucas_Gray has joined #openstack-ironic18:23
*** jamesdenton_wfh has quit IRC18:27
openstackgerritIlya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object  https://review.opendev.org/68791018:30
openstackgerritChristopher Dearborn proposed openstack/ironic master: DRAC: Added redfish management interface issue  https://review.opendev.org/72007618:31
*** alexmcleod has quit IRC18:31
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface  https://review.opendev.org/68818418:32
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface  https://review.opendev.org/68826618:32
openstackgerritIlya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data`  https://review.opendev.org/68827218:32
*** ociuhandu has quit IRC18:38
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211418:46
TheJuliaWant to trade: Cute cat photos for reviews on: https://review.opendev.org/#/c/714212/18:47
patchbotpatch 714212 - ironic - "dual stack" support for PXE/iPXE - 11 patch sets18:47
*** jcoufal has quit IRC18:49
openstackgerritIlya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data`  https://review.opendev.org/68827218:50
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface  https://review.opendev.org/68889618:52
openstackgerritIlya Etingof proposed openstack/ironic master: Add L3 boot section to the docs  https://review.opendev.org/68984418:52
* iurygregory agrees with TheJulia 18:56
iurygregorybut I want cute panda photos XD18:56
etingofthat patch is worth the whole litter of cute kittens18:58
TheJuliahttps://www.google.com/url?sa=i&url=https%3A%2F%2Fwww.pinterest.com%2Fpin%2F26880929001560383%2F&psig=AOvVaw3BTeabTxlh0eH4CtAk6V35&ust=1587064012947000&source=images&cd=vfe&ved=0CAIQjRxqFwoTCJC49aaQ6-gCFQAAAAAdAAAAABAD19:07
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251619:13
*** ociuhandu has joined #openstack-ironic19:23
jiriproxTheJulia: Does the agent mean neutron-openvswitch-agent and ovs which should be installed on SmartNIC in advance? Is there any configuration guide available for SmartNC case?19:25
TheJuliajiriprox: as I understand it, yes. My understanding is it would be the same case of use as if your using a smartnic offloading with  the host as a compute node.19:26
* TheJulia wonders where that doc ended up19:27
TheJuliahttps://specs.openstack.org/openstack/ironic-specs/specs/approved/support-smart-nic.html was the spec19:28
TheJuliafor our side19:28
*** ociuhandu has quit IRC19:29
iurygregorywould be worth asking in #openstack-neutron to confirm with them jiriprox19:31
TheJuliaYeah, I think it woudl be, I'm struggling to find docs and I thought there was stuff written up by Hamdyk19:35
jiriproxTheJulia: I've seen that guide.What is missing there and what I'm looking for is guide how to prepare SmartNIC side to be able to communicate with Ironic.19:36
TheJuliafor a baremetal tenant machine?19:37
jiriproxYes19:38
TheJuliahttps://specs.openstack.org/openstack/neutron-specs/specs/stein/neutron-ovs-agent-support-baremetal-with-smart-nic.html is the neutron spec19:38
* TheJulia sighs19:40
TheJuliahttps://review.opendev.org/#/c/586252/ was the patch for neutron19:40
patchbotpatch 586252 - neutron - Add Smart NIC representor port to integration bridge (MERGED) - 46 patch sets19:40
TheJuliaso it does look like hamdyk is still active19:41
TheJuliaI feel like we should seek his input because as I understand it, the agents should have been pre-setup pointed at Neutron from with-in the smartnics and Ironic itself just uses the neutron REST api19:42
*** yolanda has quit IRC19:50
jiriproxTheJulia: This is my understanding as well.19:56
*** yolanda has joined #openstack-ironic20:03
*** ociuhandu has joined #openstack-ironic20:04
TheJuliaI need to lay down for a little bit, I'm getting a migraine forming20:07
TheJuliajiriprox: do you want to reach out to hamdyk? Or do you want me to? I am guessing it will have to be an email20:12
*** ociuhandu has quit IRC20:14
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network CIDR map hook  https://review.opendev.org/71859420:25
openstackgerritMerged openstack/ironic master: Use trailing slash in the agent command URL  https://review.opendev.org/72015020:27
*** jiriprox has quit IRC20:28
*** k_mouza has joined #openstack-ironic20:56
*** k_mouza has quit IRC21:00
*** ociuhandu has joined #openstack-ironic21:02
*** ociuhandu has quit IRC21:09
*** Lucas_Gray has quit IRC21:18
*** Lucas_Gray has joined #openstack-ironic21:24
*** Lucas_Gray has quit IRC21:29
*** jtomasek has quit IRC21:42
*** ociuhandu has joined #openstack-ironic21:57
*** Lucas_Gray has joined #openstack-ironic22:00
*** ociuhandu has quit IRC22:07
*** ociuhandu has joined #openstack-ironic22:08
*** ociuhandu has quit IRC22:13
* TheJulia sends email to mellenox contacts regarding smartnic docs.22:15
TheJuliaand mailbox not found for hamdyk22:16
TheJulia:(22:16
TheJuliaWell, I know what happened now22:16
*** jiriprox has joined #openstack-ironic22:23
* etingof is not happy that centos8 ramdisk fails to mount root22:42
*** rcernin has joined #openstack-ironic22:45
*** rloo has quit IRC22:45
*** Goneri has quit IRC22:47
*** tkajinam has joined #openstack-ironic22:59
* TheJulia blinks23:02
TheJuliathat seems like a major problem23:02
TheJuliadonnyd: did you really expect people to throw blunt objects? This is ironic! We throw soft cuddly things... like... stuffed animals!23:06
*** rh-jelabarre has quit IRC23:12
*** Lucas_Gray has quit IRC23:23
donnydI figured someone would at the least get a kick out of it23:32
*** jiriprox has quit IRC23:33
*** ociuhandu has joined #openstack-ironic23:37
*** ociuhandu has quit IRC23:49

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!