Wednesday, 2020-04-01

« Tuesday, 2020-03-31 Index Thursday, 2020-04-02 »
*** ociuhandu has joined #openstack-ironic01:04
*** ociuhandu has quit IRC01:08
*** dsneddon has quit IRC01:21
*** yaawang has quit IRC02:08
*** yaawang has joined #openstack-ironic02:08
*** ijw_ has joined #openstack-ironic02:37
*** ijw has quit IRC02:41
*** mkrai has joined #openstack-ironic02:53
*** tzumainn has quit IRC03:22
*** zaneb has quit IRC03:49
*** zaneb has joined #openstack-ironic03:54
*** cp- has quit IRC04:53
openstackgerritAnkit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver  https://review.opendev.org/67799605:13
*** ociuhandu has joined #openstack-ironic05:52
*** ociuhandu has quit IRC05:57
*** cp- has joined #openstack-ironic05:57
*** mkrai has quit IRC06:27
arne_wiebalckGood morning, ironic!06:36
*** mkrai has joined #openstack-ironic06:38
*** irclogbot_1 has quit IRC06:49
*** spotz has quit IRC06:51
*** stevebaker has quit IRC06:51
*** jhesketh has quit IRC06:52
*** irclogbot_3 has joined #openstack-ironic06:52
*** stevebaker has joined #openstack-ironic06:53
*** irclogbot_3 has quit IRC06:53
*** jhesketh has joined #openstack-ironic06:53
*** jerry1 has quit IRC06:53
*** tonyb has quit IRC06:54
*** irclogbot_3 has joined #openstack-ironic06:54
*** jerry1 has joined #openstack-ironic06:54
*** ociuhandu has joined #openstack-ironic06:55
*** yaawang has quit IRC06:56
*** dking_desktop has quit IRC06:58
*** yaawang has joined #openstack-ironic06:59
*** tonyb has joined #openstack-ironic06:59
*** ociuhandu has quit IRC07:00
*** etingof has joined #openstack-ironic07:00
*** priteau has joined #openstack-ironic07:00
*** ociuhandu has joined #openstack-ironic07:03
*** dougsz has joined #openstack-ironic07:04
*** ociuhandu has quit IRC07:08
*** rcernin has quit IRC07:15
iurygregorygood morning arne_wiebalck and Ironic o/07:27
*** ociuhandu has joined #openstack-ironic07:28
arne_wiebalckhey iurygregory o/07:29
*** rpittau|afk is now known as rpittau07:37
rpittaugood morning ironic! o/07:37
gudrutis2morning ironic o/07:53
etingofgood morning everyone o/07:54
*** dougsz has quit IRC07:55
gudrutis2rpittau: should I do something with my MR ( intrespection scope) or wait for your changes to be merged first?07:56
rpittaugudrutis2: sit cozy and wait :)07:57
rpittauhey etingof :)07:57
gudrutis2rpittau: ok, it is not like I will be going anywhere soon :)07:57
iurygregorymorning rpittau gudrutis2 etingof o/08:03
rpittauhey iurygregory :)08:03
*** dtantsur|afk is now known as dtantsur08:04
dtantsurmorning ironic08:04
iurygregorymorning dtantsur08:05
rpittauhey dtantsur :)08:05
*** ociuhandu has quit IRC08:08
*** ociuhandu has joined #openstack-ironic08:09
*** ociuhandu has quit IRC08:14
dtantsurarne_wiebalck: morning, have a second for https://review.opendev.org/#/c/715882/ ? we'll need it for a release that is due this week.08:14
patchbotpatch 715882 - ironic-lib - Bump hacking to 3.0.0 - 3 patch sets08:14
arne_wiebalckdtantsur: checking ...08:14
*** ociuhandu has joined #openstack-ironic08:15
*** lucasagomes has joined #openstack-ironic08:18
*** tkajinam has quit IRC08:18
arne_wiebalckdone08:20
dtantsurthx!08:22
openstackgerritRiccardo Pittau proposed openstack/ironic master: [trivial] General cleaning of jobs  https://review.opendev.org/71041508:36
*** alexmcleod has joined #openstack-ironic08:49
openstackgerritMerged openstack/metalsmith master: Bump hacking to 3.0.0  https://review.opendev.org/71597108:50
*** priteau has quit IRC09:03
iurygregoryand again the logs from HP CI are unavailable09:10
openstackgerritMerged openstack/networking-generic-switch master: Bump hacking to 3.0.0  https://review.opendev.org/71597009:14
iurygregoryin the wholedisk uefi the ironic python agent failed to start, ideas?09:21
openstackgerritMerged openstack/ironic-lib master: Bump hacking to 3.0.0  https://review.opendev.org/71588209:32
rpittauiurygregory: [  514.640956] modprobe[308]: modprobe: FATAL: Module vfat not found in directory /lib/modules/4.18.0-147.5.1.el8_1.x86_6409:36
rpittauthat is quite interesting09:36
iurygregoryrpittau, yup  I saw that but not idea on how to fix D:09:36
rpittauthat's on the ramdisk side, so probably we're shipping broken centos8 ipa ramdisks09:37
rpittauI saw other issues in other jobs09:37
rpittauwith dib09:37
iurygregoryrpittau, so centos8 is broken again? D:09:37
rpittauor it's a terrible April fool09:38
iurygregoryhehehe09:38
iurygregoryi don't know what I would do to the person who did this joke XD09:39
rpittauI wonder if we're installing/loading the wrong kernel09:40
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [DNM] test ci  https://review.opendev.org/71490009:47
*** k_mouza has joined #openstack-ironic09:53
rpittauthe module seems to be there and the version seems consistent between ramdisk and kernel image09:53
iurygregorymaybe I'm missing a configuration09:54
*** seba has joined #openstack-ironic09:54
rpittauiurygregory: I saw you did a recheck, let's see how it goes09:54
iurygregoryrpittau, I checked the result already09:55
iurygregoryhttps://a58e6902d1fdba2dd4c0-cd725815d30cf2ab8520aad695ae6c76.ssl.cf5.rackcdn.com/714453/10/check/ironic-tempest-ipa-wholedisk-uefi-ipmi-direct-dib/1b245fd/controller/logs/ironic-bm-logs/node-0_console_2020-04-01-08%3A56%3A31_log.txt09:55
iurygregoryzuul still running but the job is finished09:55
rpittauI don't think it's a config issue, that usually happens when a module is not installed or the wrong kernel is loaded, but I checked the ramdisk and it looks ok09:56
rpittaulib/modules/4.18.0-147.5.1.el8_1.x86_64/kernel/fs/fat/vfat.ko.xz09:57
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image  https://review.opendev.org/71621610:04
*** gudrutis2 has quit IRC10:08
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add RPC objects for deployment API  https://review.opendev.org/70377810:09
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Conductor changes for deployment API  https://review.opendev.org/70736210:09
iurygregoryrpittau, LOL i only gave 90 to IRONIC_TEMPEST_BUILD_TIMEOUT10:14
rpittauthat's a bit low :)10:15
iurygregoryrpittau, yup! I only saw because you put the comment about dib being default XD10:15
rpittauand I'm selectively blind, so I ignored it :D10:16
rpittauor maybe because I'm starving10:16
rpittaucyl10:16
*** rpittau is now known as rpittau|bbl10:16
iurygregoryi'm kinda blind also XD10:16
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job  https://review.opendev.org/71445310:17
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Only check for partitions on devices that are part of software RAID  https://review.opendev.org/71654210:17
dtantsurarne_wiebalck: wdyt about this approach ^^^?10:17
dtantsur(I can merge it with the parent patch if it looks good to you)10:18
openstackgerritVerification of a change to openstack/ironic-tempest-plugin failed: Bump hacking to 3.0.0  https://review.opendev.org/71593810:18
*** ociuhandu has quit IRC10:21
*** ociuhandu has joined #openstack-ironic10:22
*** ociuhandu has quit IRC10:28
*** mkrai has quit IRC10:29
*** mkrai has joined #openstack-ironic10:31
*** rcernin has joined #openstack-ironic10:34
*** ociuhandu has joined #openstack-ironic10:44
*** ociuhandu has quit IRC10:45
*** ociuhandu has joined #openstack-ironic10:46
*** derekh has joined #openstack-ironic10:49
*** khansa has joined #openstack-ironic10:50
*** ociuhandu has quit IRC10:59
*** ociuhandu has joined #openstack-ironic11:00
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Only check for partitions on devices that are part of software RAID  https://review.opendev.org/71654211:05
*** ociuhandu has quit IRC11:05
*** jcoufal has joined #openstack-ironic11:10
*** accakks has joined #openstack-ironic11:27
*** dhellmann_ has joined #openstack-ironic11:31
*** dhellmann has quit IRC11:32
*** dhellmann_ is now known as dhellmann11:32
*** cdearborn has joined #openstack-ironic11:33
*** ociuhandu has joined #openstack-ironic11:37
*** ociuhandu has quit IRC11:47
*** ociuhandu has joined #openstack-ironic11:48
*** ricolin has quit IRC11:54
*** ociuhandu has quit IRC11:56
*** ociuhandu has joined #openstack-ironic12:01
*** rpittau|bbl is now known as rpittau12:07
*** spotz has joined #openstack-ironic12:07
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add `--show-traffic` feature to aid Redfish diagnostics  https://review.opendev.org/71341812:09
*** dsneddon has joined #openstack-ironic12:10
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/train: Rescan after restarting the md device  https://review.opendev.org/70882712:15
*** dmellado has quit IRC12:15
*** Goneri has joined #openstack-ironic12:16
*** rh-jelabarre has joined #openstack-ironic12:18
openstackgerritRiccardo Pittau proposed openstack/ironic-lib master: [trivial] add description of ignored flake8 checks  https://review.opendev.org/71656712:18
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876812:19
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830912:19
openstackgerritIlya Etingof proposed openstack/ironic master: Validate node `network_data` field contents  https://review.opendev.org/71461412:19
*** mkrai has quit IRC12:21
*** dmellado has joined #openstack-ironic12:23
*** derekh has quit IRC12:25
*** rh-jlabarre has joined #openstack-ironic12:36
*** rh-jelabarre has quit IRC12:37
*** ociuhandu has quit IRC12:37
*** derekh has joined #openstack-ironic12:45
arne_wiebalckdtantsur: looks like what we want :)12:45
arne_wiebalckdtantsur: another thought, I guess we need to restrict cleaning as well to the devices in the RAID ... or do we already do this?12:45
arne_wiebalckdtantsur: beforehand we assumed all devices belong to us, so I guess my original code also cleaned all devices12:46
*** rloo has joined #openstack-ironic12:51
dtantsurarne_wiebalck: cleaning always works for all devices, I'm not sure why we would change that..12:52
dtantsurarne_wiebalck: also, could you then re-review https://review.opendev.org/#/c/713209/ with this follow-up in mind?12:53
patchbotpatch 713209 - ironic-python-agent - Allow specifying target devices for software RAID - 3 patch sets12:53
arne_wiebalckdtantsur: hmm, ok ... and delete_configuration?12:53
* arne_wiebalck has not checked the code12:54
dtantsursorry, I didn't quite get the question12:54
arne_wiebalckdtantsur: manual cleaning passing delete_configuration as a step12:54
dtantsurarne_wiebalck: what's wrong with it? I think it works as before (same as for hw RAID)12:55
arne_wiebalckdtantsur: delete_configuration touches all devices12:58
dtantsurwell, right12:58
dtantsurif we want it to become selective as well, that's an addition well beyond the current scope12:59
dtantsur(since we'll have to update hardware RAID too)12:59
dtantsurbut I'm not sure we do, honestly12:59
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface  https://review.opendev.org/68818413:00
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface  https://review.opendev.org/68826613:01
openstackgerritIlya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data`  https://review.opendev.org/68827213:01
*** ricolin_ has joined #openstack-ironic13:01
arne_wiebalckdtantsur: ok (sorry, tech support interruption :-)13:03
arne_wiebalckdtantsur: my thought was we should probably only remove what we created13:04
dtantsur(my meeting now, sorry)13:04
dtantsurarne_wiebalck: not necessary, we may need to wipe what exists13:04
*** khansa has quit IRC13:07
*** jdandrea has joined #openstack-ironic13:14
arne_wiebalckdtantsur: ok13:14
arne_wiebalckdtantsur: last question, will I need to rebase the UEFI patch when I W+1 your patches? :-D13:15
*** ociuhandu has joined #openstack-ironic13:15
*** ociuhandu has quit IRC13:20
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] test ovmf manual upgrade  https://review.opendev.org/69230913:21
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] test ovmf default package with dib  https://review.opendev.org/71203313:21
dtantsurarne_wiebalck: it's not unlikely, unfortunately. I can pull your patch and see if it rebases cleanly13:29
dtantsurarne_wiebalck: oh, https://review.opendev.org/#/c/696156/ already shows "cannot merge"13:29
patchbotpatch 696156 - ironic-python-agent - Software RAID: Add UEFI support - 9 patch sets13:29
dtantsursoo.. nothing will change with my patch? :)13:29
arne_wiebalckhuh, doesn't show it for me ...13:30
dtantsurlemme see locally13:31
dtantsurarne_wiebalck: I confirmed, it has conflicts with master already13:31
arne_wiebalckdtantsur: why do I not see this one the change ...13:32
arne_wiebalckon13:32
dtantsurweird indeed13:32
arne_wiebalckdtantsur: the change displays this for you?13:33
dtantsuryeah13:33
dtantsurin the right side, after "Strategy"13:34
*** ociuhandu has joined #openstack-ironic13:35
arne_wiebalcksays "Merge if Necessary" for me13:35
* arne_wiebalck checks with different device13:36
dtantsurand then I have a red "cannot merge" further to the right13:37
arne_wiebalckyes, have this on the 2nd device as well ... confusing13:38
dtantsurmagic13:38
TheJuliao/13:39
arne_wiebalckchrome on ubuntu: looks good13:40
arne_wiebalckfirefox on ubuntu: cannot merge13:40
arne_wiebalckchrome on mac: cannot merge13:40
arne_wiebalck--> cannot merge wins 2:113:40
* TheJulia blinks13:45
*** mkrai has joined #openstack-ironic13:46
TheJuliarpioso: cdearborn: any positive feedback regarding https://review.opendev.org/#/c/710751/ ?13:47
patchbotpatch 710751 - sushy - Add `set_system_boot_options` method - 4 patch sets13:47
*** tzumainn has joined #openstack-ironic13:49
TheJuliahjensas: ohhhh ahhhh http://paste.openstack.org/show/791438/13:51
iurygregoryarne_wiebalck, cache problems =)13:52
TheJuliait is always a cache problem... when it is no a dns problem... when it is not a dns cache problem... when it is not a bgp problem.13:52
TheJulias/no/not/13:52
TheJuliawhen it is not an ospf problem13:52
TheJuliawhen it is not an addressing problem13:53
TheJuliawhen it is not a link layer (hey, you, stop cutting those cables!)13:53
iurygregorydns dhcp13:53
iurygregorynetworking problems \o/13:54
iurygregoryit's green! https://review.opendev.org/#/c/492114/13:54
patchbotpatch 492114 - ironic - Change [deploy]/default_boot_option to local - 31 patch sets13:54
iurygregoryafter 2 rechecks only13:55
TheJuliawoot!13:56
iurygregory\o/13:56
TheJulia🚢 it!13:57
iurygregoryWOOT13:57
iurygregoryemoji13:57
iurygregoryTheJulia, ++13:57
rpittauI will +2 if my connection allows me14:01
cdearbornTheJulia, at this point I have all of the logs from the testing.  Will have some feedback today sometime14:01
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Only check for partitions on devices that are part of software RAID  https://review.opendev.org/71654214:02
TheJuliacdearborn: thanks! My urgency is because we need to go ahead and make our sushy release for this cycle asap14:02
cdearbornTheJulia: aok - will get it done14:03
TheJulia<314:03
iurygregoryrpittau, your centos8 test went ok?14:06
dtantsurTheJulia: something fishy is going on in the inspector job: https://zuul.opendev.org/t/openstack/build/fba93ff1e3f4423fa94ad3a18375d0a0/log/controller/logs/ironic-bm-logs/node-0_no_ansi_2020-04-01-12:14:11_log.txt#147914:07
dtantsurthis is from https://review.opendev.org/#/c/492114/14:07
patchbotpatch 492114 - ironic - Change [deploy]/default_boot_option to local - 31 patch sets14:07
rpittauiurygregory: it looks all good on the build process, that error is really weird14:09
TheJuliadtantsur: what image is being deployed?14:10
iurygregoryrpittau, yeah D:14:10
iurygregorydamm it D:14:10
TheJuliaWe really need to just peek at the filesystem and debug log it if any of those mounts fail14:10
dtantsura good question, I assumed whole disk..14:11
TheJuliabecause that that cod ewill fail if the target folders are missing at all14:12
TheJuliahmmm14:12
* TheJulia writes a quick debugging patch14:12
rpittauTheJulia, dtantsur, did you see the error in the wholedisk+uefi job we were discussing before ?14:12
dtantsurmmm, it seems that the job is a partition one14:12
dtantsurrpittau: no, missed it14:12
iurygregoryhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4a7/714453/11/check/ironic-tempest-ipa-wholedisk-uefi-ipmi-direct-dib/4a774db/controller/logs/ironic-bm-logs/node-0_console_2020-04-01-11%3A09%3A42_log.txt14:13
rpittaumodprobe[308]: modprobe: FATAL: Module vfat not found in directory /lib/modules/4.18.0-147.5.1.el8_1.x86_6414:13
TheJuliaIf partition, i wonder if the scenario needs to netboot14:13
dtantsurTheJulia: we may need to update the jobs on other projects (inspector, ironic-lib, IPA) before we can switch the default14:13
TheJuliarpittau: that is... epic. :(14:13
dtantsurrpittau: ah, yeah, remembered. not that it makes any sense to me..14:13
iurygregorymodprobe: FATAL: Module vfat not found in directory /lib/modules/4.18.0-147.5.1.el8_1.x86_64 Failed to start Ironic Python Agent.14:13
iurygregory=(14:13
rpittauit really doesn't make sense, I checked the ramdisk and the module is there14:14
iurygregoryBad 1st April joke from CI =(14:14
dtantsurexactly14:16
rpittaucan we move forward to tomorrow ?14:16
dtantsuryou mean, go to bed right now? I'm in14:17
rpittausomething like that, yeah14:17
iurygregory++14:17
iurygregoryin 43 min it's April 2 in Japan14:18
iurygregoryso CI should go back to normal lol14:18
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Debug logging for failed disk folder mount with _install_grub2  https://review.opendev.org/71660314:21
TheJuliaI _think_ that will work14:21
* dtantsur -> quick snack14:21
TheJuliaat least, give us an idea of what may be part of the issue14:22
TheJuliaiurygregory: only if we run CI in japan14:23
iurygregoryTheJulia, =( I know, I was trying to stay positive XD14:24
TheJuliaso it is interesting that /dev is the first thing missing. It makes me wonder if /dev is a myth14:24
TheJuliaor the filesystem is empty14:24
iurygregoryin 1st april is a myth14:24
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-lib master: Add netboot to all jobs  https://review.opendev.org/71660514:28
iurygregorydtantsur, to avoid problems for ironic-lib ^14:28
TheJuliadtantsur: so it is cirros 0.4.0, which means it has no /dev14:28
TheJuliawhich means, that job needs to netboot as long as it runs cirros14:28
dtantsurwonderful..14:29
dtantsurcirros 0.5.1 may fix it, but it comes with its own problems14:29
dtantsurTheJulia: well, I remembered, cirros does have an empty filesystem. I think it copies the contents of the ramdisk on the first boot.14:30
TheJuliadtantsur: yup14:32
openstackgerritDmitry Tantsur proposed openstack/metalsmith master: DNM watch file system usage  https://review.opendev.org/71660814:36
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211414:37
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM test with cirros 0.5.1  https://review.opendev.org/71272814:38
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Set default boot option to netboot for jobs using cirros  https://review.opendev.org/71661314:41
iurygregoryTheJulia, ^ you missed the grenade job for inspector14:42
iurygregoryand you removed an s from vars14:42
TheJuliadoh14:43
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Set default boot option to netboot for jobs using cirros  https://review.opendev.org/71661314:45
TheJuliathat should be better14:45
TheJuliagood catch w/r/t grenade14:45
iurygregorynp I was doing locally the change14:45
dtantsuroh no, now we need a way to return values from in-band deploy steps....14:47
rpiosoGood morning, ironic14:47
iurygregoryrpioso, morning14:47
rpiosoiurygregory: :-)14:48
rpittauhey rpioso :)14:48
rpiosorpittau: Yo! o/14:48
openstackgerritJulia Kreger proposed openstack/ironic master: Make deploy step failure logging indicate the error  https://review.opendev.org/71451114:50
TheJuliadtantsur: don't we hae a return value interface in ipa for command status on async tasks?14:50
dtantsurTheJulia: for tasks - yes, but not for deploy steps14:54
dtantsurand this result is needed in *another* deploy step14:54
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image  https://review.opendev.org/71621614:54
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830914:58
etingofironic is playing April fool on me - despite massive changes in the code, unit tests pass15:00
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface  https://review.opendev.org/68889615:01
yolandahi etingof , you  know a lot about vbmc, right?15:04
yolandai get this error all the time:  vbmcd --foreground15:04
yolanda2020-04-01 13:53:28,973.973 5538 ERROR VirtualBMC [-] server PID #0 still running15:04
etingofyolanda, o/ I am aware of some of its glitches, yes15:05
yolandai am unable to start vbmcd and no idea why... no vbmc there running before. I even restarted the server before running the command15:05
etingofhold on15:05
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Change ipa jobs to netboot  https://review.opendev.org/71662415:06
iurygregoryTheJulia, do you mind if I update your inspector patch since it's missing the config for other job?15:07
etingofyolanda, cat ~/.vbmc/master.pid15:07
TheJuliaiurygregory: you may update any patch I post :)15:07
TheJuliaiurygregory: just let me know :)15:07
yolanda015:07
yolandai remove that?15:07
iurygregoryTheJulia, will do now =)15:08
etingofyolanda, yes15:08
yolandai was looking under /var/run15:08
* etingof can't help but wonder how that 0 appeared there...15:08
etingofof course process 0 is always found...15:08
yolandaok now it runs15:08
etingof\o/15:09
yolandano idea on how that 0 got written15:09
yolandathx!15:09
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Set default boot option to netboot for jobs using cirros  https://review.opendev.org/71661315:10
bfourniegood morning TheJulia, is it possible to backport the ipmi logging patch to Train? https://review.opendev.org/#/c/712695/15:11
patchbotpatch 712695 - ironic - Move ipmi logging to a separate option (MERGED) - 2 patch sets15:11
iurygregorylooking at the patch sounds like a no I would say bfournie (release not with features and upgrade) and also have a new config option..15:18
*** mkrai has quit IRC15:19
openstackgerritIlya Etingof proposed openstack/ironic master: Add L3 boot section to the docs  https://review.opendev.org/68984415:21
dtantsuriurygregory: a new config option alone is not an issue with backporting15:27
iurygregorydtantsur, oh =D15:28
dtantsurand I think I'm fine with making this an exception15:28
iurygregoryso I would say it's a good idea to backport15:28
*** mkrai has joined #openstack-ironic15:29
TheJuliabfournie: would you be okay if we changed the default value to be true in the backport?15:33
TheJuliathatwould mean the option would need to be explicitly oppted into behavior wise15:34
TheJuliawhich would be in conformance with backport rules and regulations15:34
*** gyee has joined #openstack-ironic15:36
*** lucasagomes has quit IRC15:51
iurygregorybye everyone o/15:52
iurygregorysee you tomorrow15:52
bfournieTheJulia, dtantsur: thanks, yes making the default value true is certainly fine16:01
TheJuliabfournie: I'll go ahead and take care of that in a little bit16:02
dtantsur*shrugs* I'd even leave false, it's for debugging after all16:02
dtantsurand true doesn't quite help with debugging most of the time16:02
TheJuliaTrue16:02
TheJuliaThis is very true16:02
* TheJulia checks to see if the logging bot has saved this discussion16:03
* TheJulia dances around in total agreement while waiting for the bot to log it so this can go into the commit message16:04
* TheJulia may have lost her mind16:04
dtantsur:D16:05
TheJuliadtantsur: rpittau: and anyone else: If cdearborn comes back with good news, are we good to go ahead and approve/merge https://review.opendev.org/#/c/710751/ today?16:06
patchbotpatch 710751 - sushy - Add `set_system_boot_options` method - 4 patch sets16:06
dtantsurTheJulia: I'm typing human-readable words into a computer, sorry16:07
TheJulia1001000111010100010116:07
TheJulia<316:07
dtantsur:)16:07
rpittauI'm not sure of anything today, but ok :D16:07
* TheJulia offers rpittau a virtual hug if it helps16:07
rpittauTheJulia: thanks :)16:08
JayFI had a dream, there were all these 1's and 0's... and I think I saw a 216:08
dtantsurjust a glitch in the matrix, move on16:08
* TheJulia gives rpittau a hug16:08
rpittauJayF: if you squeeze your eyes hard enough you can see a 3 too16:08
TheJuliaJayF: a 2!?!?!? What dastardly evil bit flip occured for that?!?16:08
rpittauTheJulia: thanks! :D16:08
JayFIt's a futurama reference. The response there is "Don't worry Bender, there's no such thing as 2"16:09
rpittauright16:09
TheJuliaJayF: ahh futurama! Perhaps that should be watched16:09
* rpittau reminds himself to not manually cancel virtual volumes16:09
JayFIf you're not watching WestWorld, you should fix that first :D16:09
rpittauTheJulia: I'll check that patch later for updates from cdearborn16:15
rpittaubye humans and machines, good night! o/16:15
*** rpittau is now known as rpittau|afk16:15
TheJuliarpittau|afk: thanks, goodnight!16:15
openstackgerritJulia Kreger proposed openstack/ironic stable/train: Move ipmi logging to a separate option  https://review.opendev.org/71665116:20
TheJuliabfournie: how far back do we want to take ^^^?16:20
bfournieTheJulia: train is fine16:21
bfournieTheJulia: thanks!16:21
TheJuliabfournie: ack, enjoy!16:23
bfournieTheJulia: I will enjoy not seeing those "error" messages in customer reports :-)16:24
TheJuliaheh16:25
*** mkrai has quit IRC16:35
TheJuliarpioso: cdearborn: that recent redfish issue in bmc firwmare on the last generation... do you remember what version that was introduced in?16:37
dtantsurJayF: I have reservations about WestWorld 3 after watching its trailer (and the 2nd part)16:46
JayFdtantsur: three episodes in it's been pretty great16:46
dtantsurnice!16:46
dtantsurI don't think it's on Amazon Prime already though..16:46
JayFno, it's all on HBO16:47
dtantsurwell, the first 2 parts did land on amazon eventually. so I have some hopes.16:47
dtantsurI don't think I can connect HBO here.16:47
JayF*cough* yarr, matey *cough* :D16:48
dtantsur:D16:48
dtantsura problematic option here, but nothing impossible16:48
* dtantsur has sent a THING to openstack-discuss17:04
dtantsurand on this positive note I'm calling it a day. see you17:07
*** dtantsur is now known as dtantsur|afk17:07
hjensasTheJulia: big pastebin, but dual-stack options seems to work.17:23
*** jcoufal has quit IRC17:28
TheJuliaJayF: When is the first talking that day anyway?17:29
TheJuliaI <3 the THING17:29
JayFTheJulia: ...what? I don't get what you mean17:29
TheJuliaJayF: talk like a pirate day, I think there are two?17:29
JayF9/19 is the only one I know of17:29
JayFit coincides with my old best friend in Raleigh's bday, so I always remember it17:30
cdearbornTheJulia, the issue where"Setting a server to PXE boot Continuously via Redfish fails" was introduced in the 13g firmware 2.70.70.70.  The issue did not exist in 2.63.60.6117:30
TheJuliacdearborn: eek, okay. I've let our scale lab folks know because they were going to do a big test in the next few weeks17:32
cdearbornTheJulia: FYI, the issue exists in 14g firmware too, versions 4.00.00.00 & 4.10.10.10.  It did not exist in version 3.32.32.32.17:35
TheJuliacdearborn: wow, okay17:36
cdearbornTheJulia: for what it's worth it will be fixed in the next release of both 13g & 14g firmware17:36
openstackgerritMerged openstack/ironic-inspector master: Install pymemcache explicitly  https://review.opendev.org/71627617:36
openstackgerritMerged openstack/ironic-python-agent master: Allow specifying target devices for software RAID  https://review.opendev.org/71320917:36
TheJuliacdearborn: oh cool, thanks17:38
*** ociuhandu has quit IRC17:45
*** ociuhandu has joined #openstack-ironic17:49
arne_wiebalckbye everyone o/17:50
cdearbornarne_wiebalck: good night!17:51
*** ociuhandu has quit IRC17:56
*** gmann is now known as gmann_lunch17:56
cdearbornTheJulia: I took a look at the test results, and it appears that https://review.opendev.org/#/c/710751/ helps, but it does not completely resolve the problem.  I believe what is happening is that it works up until ironic does a persistent set_boot_device at the very end17:57
patchbotpatch 710751 - sushy - Add `set_system_boot_options` method - 4 patch sets17:57
cdearbornTheJulia: In order to get a successful deployment, we had to set force_persistent_boot_device=Never17:57
cdearbornTheJulia: But I believe that could result in a node that may not boot from the right device following ironic deployment17:58
TheJuliacdearborn: That is already the risk ultimately17:59
TheJuliaso helping some is better than nothing17:59
cdearbornTheJulia: that is if the persistent boot device were not set correctly prior to deployment17:59
cdearbornTheJulia: I agree.18:00
TheJuliacdearborn: indeed, that is already experienced in some cases on some hardware so I've been telling people to ensure that they have their permenant settings set for desired end state, not deployment state18:00
cdearbornTheJulia: yes - that is exactly what we do in JetStream18:00
TheJuliacdearborn: so the question boils down, do we merge the patch?18:01
cdearbornTheJulia: so from my perspective, I would like to see the change go in, because it does help some, and some help is always greatly appreciated.  Plus rpioso pointed out that it will allow setting boot mode and boot device separately18:02
TheJuliaIndeed, which was one of the reasons I want it :)18:03
TheJuliawe can do some last minute changes in ironic and hopefully make bmcs happier that way18:03
TheJuliaI've put my +2 on it, rpittau|afk fyi18:03
TheJuliahttps://review.opendev.org/#/c/71075118:03
patchbotpatch 710751 - sushy - Add `set_system_boot_options` method - 4 patch sets18:03
*** k_mouza has quit IRC18:03
iurygregoryTheJulia, it's ok to merge so? =)18:04
cdearbornTheJulia: :-) yes, we believe will will have an issue in deploy steps if both boot mode and boot device need to be set.  testing that is up next18:04
iurygregoryI can +A since we will need a release asap18:05
cdearbornThanks all!!!18:05
TheJuliaiurygregory: sure!18:05
iurygregory*boom*18:05
TheJuliaonce it is merged I can go ahead and put the release change out there18:05
iurygregorynow i need to start the dinner your my gf will complain XD18:05
iurygregoryor*18:05
cdearbornTheJulia: when should we be targeting to cut a release of python-dracclient?  would like to not be holding things up this time around :)18:06
*** k_mouza has joined #openstack-ironic18:06
TheJuliacdearborn: technically tomorrow18:06
TheJuliabut your also a driver library and not bound by constraints18:07
TheJuliaafaik18:07
TheJuliaand we can independently change driver-requirements.txt18:07
cdearbornTheJulia: aok - will try to get the dangling chads wrapped up...18:07
TheJuliaheh18:08
TheJuliaokay18:08
rpiosocdearborn, TheJulia: Thank you for your input on that sushy change. I updated my review with a +1.18:18
*** k_mouza has quit IRC18:19
*** kaifeng has quit IRC18:21
openstackgerritVishal Manchanda proposed openstack/ironic-ui master: Fix pyScss version in lower-constraints.txt  https://review.opendev.org/71671218:21
*** ociuhandu has joined #openstack-ironic18:25
*** gmann_lunch is now known as gmann18:28
*** ociuhandu has quit IRC18:28
*** ociuhandu has joined #openstack-ironic18:29
*** k_mouza has joined #openstack-ironic18:33
*** k_mouza has quit IRC18:34
*** k_mouza has joined #openstack-ironic18:35
openstackgerritMerged openstack/ironic-lib master: [trivial] add description of ignored flake8 checks  https://review.opendev.org/71656718:37
*** k_mouza has quit IRC18:39
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Use auth values from neutron conf when managing Neutron ports  https://review.opendev.org/70867818:54
openstackgerritIlya Etingof proposed openstack/ironic master: Validate node `network_data` field contents  https://review.opendev.org/71461418:56
openstackgerritMerged openstack/sushy master: Add `set_system_boot_options` method  https://review.opendev.org/71075118:57
openstackgerritMerged openstack/ironic-ui master: Fix pyScss version in lower-constraints.txt  https://review.opendev.org/71671219:10
*** alexmcleod has quit IRC19:18
*** Lucas_Gray has joined #openstack-ironic19:20
*** Lucas_Gray has quit IRC19:21
*** Lucas_Gray has joined #openstack-ironic19:22
*** Lucas_Gray has quit IRC19:26
*** Lucas_Gray has joined #openstack-ironic19:32
*** Lucas_Gray has quit IRC19:32
*** early` has quit IRC19:36
*** ociuhandu has quit IRC19:39
*** derekh has quit IRC19:40
*** ociuhandu has joined #openstack-ironic19:40
*** early has joined #openstack-ironic19:44
*** ociuhandu has quit IRC19:45
*** k_mouza has joined #openstack-ironic19:48
*** k_mouza has quit IRC19:50
*** ociuhandu has joined #openstack-ironic19:57
*** stevebaker has quit IRC20:13
*** stevebaker has joined #openstack-ironic20:13
*** igordc has joined #openstack-ironic20:27
*** Lucas_Gray has joined #openstack-ironic21:01
*** ociuhandu has quit IRC21:16
*** ociuhandu has joined #openstack-ironic21:17
*** ociuhandu has quit IRC21:22
TheJuliadtantsur|afk: I've left a comment for you on https://review.opendev.org/#/c/71675321:49
patchbotpatch 716753 - releases - Release final metalsmith for ussuri - 2 patch sets21:49
*** ijw has joined #openstack-ironic21:57
*** ijw_ has quit IRC22:01
*** Lucas_Gray has quit IRC22:03
*** ijw_ has joined #openstack-ironic22:03
*** Lucas_Gray has joined #openstack-ironic22:06
*** ijw has quit IRC22:06
*** melwitt has quit IRC22:08
*** Lucas_Gray has quit IRC22:10
*** Lucas_Gray has joined #openstack-ironic22:12
*** Lucas_Gray has quit IRC22:12
*** Lucas_Gray has joined #openstack-ironic22:13
*** k_mouza has joined #openstack-ironic22:15
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface  https://review.opendev.org/68818422:19
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface  https://review.opendev.org/68826622:19
openstackgerritIlya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data`  https://review.opendev.org/68827222:20
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface  https://review.opendev.org/68889622:20
openstackgerritIlya Etingof proposed openstack/ironic master: Add L3 boot section to the docs  https://review.opendev.org/68984422:20
*** rh-jlabarre has quit IRC22:26
*** jdandrea has quit IRC22:38
*** melwitt has joined #openstack-ironic22:51
*** tkajinam has joined #openstack-ironic22:55
*** Lucas_Gray has quit IRC23:13
*** tzumainn has quit IRC23:18
openstackgerritMerged openstack/ironic-inspector master: Bump hacking to 3.0.0  https://review.opendev.org/71588423:27
openstackgerritMerged openstack/ironic-tempest-plugin master: Bump hacking to 3.0.0  https://review.opendev.org/71593823:27
*** k_mouza has quit IRC23:30
« Tuesday, 2020-03-31 Index Thursday, 2020-04-02 »

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