Tuesday, 2020-04-14

*** openstack has joined #openstack-ironic09:54
*** ChanServ sets mode: +o openstack09:54
dtantsurrpittau: hey, do you remember how the story ended with this "no IP addresses" problem: https://zuul.opendev.org/t/openstack/build/d8045db4f01e4231abb4d5850d060395/log/job-output.txt#4902 ?09:56
rpittauoh gosh no.....09:57
dtantsurikr?09:58
rpittauI remember the issue, I don't remember what was done09:58
dtantsurI assume it has been fixed for ironic? I wonder why inspector is still sufferring09:59
dtantsurgmann: hey, was it you who fixed ^^^?09:59
dtantsurI assume it was https://review.opendev.org/#/c/713014/10:00
patchbotpatch 713014 - grenade (stable/stein) - Use stable constraint for Tempest pinned stable br... (MERGED) - 2 patch sets10:00
dtantsuralthough.. the failure used to be on the rocky side, and this time it's on the stein side10:02
rpittaumaybe the same change is needed on stable/train ?10:06
dtantsurrpittau: it's rocky->stein, train is not involved10:07
* dtantsur hates grenade (and tempest)10:07
rpittauoh ok, I thought it was stein->train that was failing now10:07
*** ricolin has quit IRC10:09
dtantsurnope, it's still same grenade, but now when running tests on the new side10:09
*** k_mouza has quit IRC10:17
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/stein: DNM test stable/stein  https://review.opendev.org/71989510:19
iurygregoryit was something related to another venv being used for the test and the baremetal service wasn't available there10:22
dtantsuriurygregory: I think https://review.opendev.org/#/c/719873/ it a better solution since it doesn't rely on configuration10:22
patchbotpatch 719873 - tempest - Explicitly handle baremetal ports when checking if... - 1 patch set10:22
iurygregorydtantsur, ++10:23
rpittauthat should handle it10:23
*** rpittau is now known as rpittau|bbl10:23
iurygregory:q10:24
iurygregorydamm not vim =P10:25
dtantsurone cannot just exit vim!10:26
iurygregoryhehehe10:27
iurygregoryhas anyone tried to bring up a devstack up today? I'm getting /opt/stack/devstack/lib/infra: line 32: virtualenv: command not found O.o10:28
*** k_mouza has joined #openstack-ironic10:29
priteauiurygregory: I discovered this last week too, using CentOS 7. Fixed by `sudo pip3 install virtualenv`10:30
dtantsurI need to rebuild mine.. but I've been avoiding it for a while10:30
dtantsuror maybe install via yum?10:30
priteauThis might fix it: https://review.opendev.org/#/c/712609/5/stackrc10:31
patchbotpatch 712609 - devstack - Use venv for VIRTUALENV_CMD - 5 patch sets10:31
iurygregoryaha!10:31
iurygregorytks priteau o/10:31
* iurygregory testing the patch locally10:40
*** priteau has quit IRC11:14
dtantsuriurygregory: lemme know if it works for you11:15
iurygregorydtantsur, it failed in something different this time =) ( I forgot to add IRONIC_ENABLED_HARDWARE_TYPES="ipmi" in the local.conf =)11:16
*** mkrai has quit IRC11:25
*** mkrai_ has joined #openstack-ironic11:25
*** cdearborn has joined #openstack-ironic11:30
*** ociuhandu has quit IRC11:41
*** hjensas|afk is now known as hjensas11:44
*** rh-jelabarre has joined #openstack-ironic11:51
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image  https://review.opendev.org/71621611:53
*** jamesdenton_wfh has joined #openstack-ironic11:57
*** jamesdenton_wfh is now known as jamesdenton11:57
*** mkrai has joined #openstack-ironic12:05
*** mkrai_ has quit IRC12:06
*** tkajinam has quit IRC12:12
*** ociuhandu has joined #openstack-ironic12:19
arne_wiebalckIt is not Monday (though almost), but I have some active nodes without an instance ... is there an API way (without removing them from Ironic) to get them back into a sane provision state (managed)?12:27
arne_wiebalckOr is it time for the db scalpel?12:28
arne_wiebalck*manageable12:28
*** rpittau|bbl is now known as rpittau12:31
*** ociuhandu has quit IRC12:34
*** ociuhandu has joined #openstack-ironic12:35
iurygregoryrpittau, I got wholedisk+uefi working locally \o/12:36
iurygregorybut not with dib =(12:36
rpittauiurygregory: great :)12:36
iurygregorywill push an update to the patch12:36
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job  https://review.opendev.org/71445312:44
* iurygregory fingers crossed12:47
dtantsurarne_wiebalck: $ openstack baremetal node undeploy && openstack baremetal node manage?12:50
dtantsuror did I misunderstand you?12:50
arne_wiebalckdtantsur: I think you understood well, will give this a try, didn't see this option while checking the state diagram12:51
arne_wiebalckdtantsur: node moved to cleaning :)12:52
dtantsurarne_wiebalck: yeah, when it's done it will be available, then you can 'manage'12:53
dtantsuror do you have any troubles with cleaning?12:53
arne_wiebalckdtantsur: available is even better :)12:53
arne_wiebalckdtantsur: usually no12:53
dtantsurgood :) 'node undeploy' is roughly what nova does12:53
dtantsuron 'server delete'12:54
arne_wiebalckI saw delete, but didn't want to remove/add the node12:54
arne_wiebalckwhy is undeploy not on the diagram?12:55
arne_wiebalckdtantsur: thanks, btw :)12:56
*** rloo has joined #openstack-ironic12:58
arne_wiebalckdtantsur: worked perfectly fine, thanks again!13:00
dtantsurnp :)13:00
dtantsurarne_wiebalck: re diagram: the actual API action has a very unfortunate name "deleted", which we changed in our OSC plugin13:00
arne_wiebalckdtantsur: ah, that explains it, thx13:01
dtantsurI'm seriously pondering a new microversion adding an alias for it13:01
*** Goneri has joined #openstack-ironic13:10
arne_wiebalcklooks sensible to me (obviously :))13:10
* dtantsur files an RFE13:14
dtantsurarne_wiebalck: https://storyboard.openstack.org/#!/story/200755113:19
dtantsura really low-hanging fruit for anyone who need a task ^^^13:19
arne_wiebalckdtantsur: great, thanks13:20
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] Dump info on pxeboot for uefi jobs  https://review.opendev.org/71995613:22
*** mjturek1 has joined #openstack-ironic13:23
TheJuliagood morning13:23
dtantsurmorning TheJulia13:23
*** diga_ has joined #openstack-ironic13:23
*** diga_ has quit IRC13:24
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf  https://review.opendev.org/71688913:26
Qianbiaomorning ironic.13:28
Qianbiaoeventlet does have a bug, i can reproduce on almost all version.13:29
Qianbiaoset watch_log_file to true in ironic.conf13:29
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Fix grub2 pxe job with native bionic ovmf  https://review.opendev.org/71688913:29
Qianbiaothen restart ironic-api, log will tell it.13:29
gmanndtantsur: is that baremetal service unavailable issue ?13:31
dtantsurgmann: yeah, now popping up in the inspector grenade job13:31
dtantsurI've posted a patch that reworks the check in tempest13:31
gmanndtantsur: ok, let me check after QA office hour.13:31
gmannsure13:31
*** tkajinam has joined #openstack-ironic13:33
arne_wiebalckQianbiao: if you haven't done so, could you please file an issue on storyboard, with a description, so that we keep track of this bug?13:40
arne_wiebalckQianbiao: and, of course, if you have a fix ... ;)13:40
*** jamesdenton has quit IRC13:40
Qianbiao<arne_wiebalck> will try new an issue now.13:41
Qianbiaodo not know whether i can get my account back. it;s a long time since last login.13:41
arne_wiebalckQianbiao: thanks!13:43
*** andrein has quit IRC13:50
*** vdrok has quit IRC13:50
*** rpittau has quit IRC13:50
*** ab-a has quit IRC13:51
*** mkrai has quit IRC13:51
*** andrein has joined #openstack-ironic13:51
Qianbiao<arne_wiebalck> https://storyboard.openstack.org/#!/story/200755313:51
*** vdrok has joined #openstack-ironic13:51
*** rpittau has joined #openstack-ironic13:51
*** ab-a has joined #openstack-ironic13:51
arne_wiebalckQianbiao: great, thanks!13:52
QianbiaoHello, should i submit a spec for this story? https://storyboard.openstack.org/#!/story/200755413:55
Qianbiaoor just submit code for review13:55
*** derekh has quit IRC13:56
*** derekh has joined #openstack-ironic13:56
iurygregoryI don't think a spec would be necessary tbh13:59
iurygregoryQianbiao, ^13:59
*** derekh has quit IRC13:59
Qianbiao<iurygregory> ok thanks, btw, what is tbh?13:59
Qianbiaoto be honest?13:59
iurygregoryyup14:00
iurygregory=)14:00
*** derekh has joined #openstack-ironic14:00
Qianbiaook14:00
dtantsurthese tla's! :D14:00
Qianbiaolol.. what is tla's ...14:00
iurygregoryno idea lol14:00
iurygregoryif it was tls I would know XD14:01
dtantsurThree Letter Acronym14:01
dtantsurTLA which ironically is itself a TLA14:01
QianbiaoSearch Results14:01
QianbiaoWeb results14:01
Qianbiao  Three-letter acronym - Wikipedia14:01
iurygregorywoa14:01
Qianbiaorecurse is recurse :)14:02
dtantsurTo understand recursion you need to understand recursion.14:03
*** ociuhandu has quit IRC14:03
mbeierlIsn't TLA technically a Three Letter Abbreviation, as doesn't an acronym need to be pronounceable?14:03
dtantsurprobably, I'm not strong in these terms14:04
rpittauthere might be a little issue with that RFE, the huawei ci is not active anymore14:04
rpiosoGood morning, ironic14:04
mbeierl:)14:04
*** ociuhandu has joined #openstack-ironic14:04
Qianbiao<rpittau> we will bring CI up again.14:04
mbeierlo/ rpioso14:04
rpittauQianbiao: ok, good :)14:04
rpiosombeierl: lol14:05
Qianbiaobecause openstack move to opendev, our old CI is broken now.14:05
rpiosombeierl: \o14:05
dtantsurQianbiao: the RFE is mostly good, maybe update it with a bit of technical details?14:05
dtantsurjust an overview how it is implemented?14:05
Qianbiaoin story? dtantsur14:06
openstackgerritMerged openstack/ironic master: Refactor AgentBase.heartbeat and process_next_step  https://review.opendev.org/71814214:06
dtantsurQianbiao: yep, just update the description14:06
TheJuliaWell, TLAs could be more than just... three characters14:06
TheJuliaThinking Memes from the 2000s14:06
dtantsur(it was re https://storyboard.openstack.org/#!/story/2007554 for the context)14:06
Qianbiao<dtantsur> ok will do.14:06
rpittauQianbiao: I don't think the bug for watch_log_file is actually in ironic, that might be oslo_log14:09
Qianbiaothe story board's Description input box is "old style" .14:10
Qianbiao<rpittau> yes, not directly caused by ironic.14:10
Qianbiaobut it does appear when startup ironic..14:10
iurygregorygood morning TheJulia o/14:10
Qianbiaoand it effects almost all version i know.14:11
rpittauQianbiao: yeah, I get that, but we can't fix that in ironic14:11
Qianbiaorpittau for oslo_log lib, it does not have bug too. because it does not need eventlet lib14:11
Qianbiaoall issue is caused by lib un-compatible14:12
akahatdtantsur, hello14:12
openstackgerritIlya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object  https://review.opendev.org/68791014:12
akahatdtantsur, I just want to know that are there any modules written for openstack baremetal port and other baremetal commands?14:12
dtantsurakahat: I'm pretty sure you can create ports via the os_ironic or os_ironic_node module14:13
erbarrgood morning ironic14:13
dtantsurI don't think there are separate modules though14:13
dtantsurakahat: you may talk to sshaidm (not online now) re his plans, he's been reworking some modules14:13
dtantsurmorning erbarr14:13
erbarrdtantsur, about this https://review.opendev.org/#/c/719687/14:15
patchbotpatch 719687 - ironic - Continuous Mode Bug Workaround for redfish - 2 patch sets14:15
akahatdtantsur, great. I'll talk to sshaidm.14:15
erbarrthat won't affect the flow of the 3rd Party CI jobs?14:15
akahatdtantsur, thank you!! :)14:15
*** tkajinam has quit IRC14:17
*** uzumaki has joined #openstack-ironic14:17
*** tkajinam has joined #openstack-ironic14:18
dtantsurerbarr: the jobs - yes. but not the production code (i.e. your users)14:18
*** mjturek1 is now known as mjturek14:19
*** JasonF has joined #openstack-ironic14:22
Qianbiaodtantsur good now? https://storyboard.openstack.org/#!/story/200755414:25
erbarrokay, I believe that's the intent but I'm checking14:26
dtantsurQianbiao: much better. By the way, are you using standard redfish features or an OEM extension?14:31
Qianbiao<dtantsur> old question again :). i think it's pretty like standard redfish for oob14:32
dtantsurQianbiao: I'm asking it because i wonder if it will be ibmc-specific or a generic redfish implementation14:33
Qianbiaowe will using python-ibmcclient writen by ourself but not sushy14:33
Qianbiaosushy could not match our requirement.14:33
Qianbiaodtantsur i think it's ibmc specific, but the API is a bit different from standard redfish.14:34
Qianbiaoand for inband part, it's totally different14:34
dtantsurQianbiao: let's clarify this as well in the story. And please explain why you need two approaches (in-band and OOB).14:35
Qianbiaook.14:35
Qianbiao<dtantsur> updated.14:43
Qianbiaobtw, when is the plan release time for unsuri14:45
uzumakihey everyone! o/ how y'all doing14:45
*** tzumainn has joined #openstack-ironic14:46
QianbiaoI am sleepy. :)14:46
dtantsurQianbiao: https://releases.openstack.org/ussuri/schedule.html14:47
dtantsurhey hey uzumaki, just back from an extended weekend14:47
dtantsurwant more weekends :)14:47
erbarrhi uzumaki14:47
* iurygregory agrees with dtantsur 14:47
uzumakidtantsur, ah! those, I really miss those :(14:47
uzumakierbarr, hey o/14:47
iurygregoryofc extended weekend without using PTO *-*14:47
* uzumaki agrees with iurygregory dtantsur and everybody else who believes in long weekends14:48
dtantsurQianbiao: last question. for in-band RAID, will you extend ironic-python-agent?14:48
Qianbiaodtantsur no.14:48
Qianbiaowe have REST API for that too.14:48
Qianbiaodoes not need a customer IPA14:49
Qianbiaodtantsur next month!!!  will it release as plan.14:49
dtantsurQianbiao: very likely14:49
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830914:49
* uzumaki think there needs to be a party after the release14:49
Qianbiao<dtantsur> i need to finish the story before next month if so.14:50
dtantsurQianbiao: you should really hurry up then. please don't wait for the last week, the code may not get in.14:50
dtantsurand notify TheJulia since we're essentially in soft feature freeze now14:51
Qianbiaook14:52
*** ricolin has joined #openstack-ironic15:03
*** ricolin_ has joined #openstack-ironic15:03
*** ricolin_ has quit IRC15:03
TheJuliasorry, on the OSF board call15:09
TheJuliawhats up?15:09
dtantsurTheJulia: a late driver feature for ussuri15:10
uzumakiheyyyy TheJulia o/ long time no see15:12
TheJuliao/15:12
uzumakihow you been? and your omlettes?15:12
TheJuliaI've been alright. Haven't made any omlettes recently15:12
uzumakiah, too bad. the omlette part, not that you've been alright, glad to know you have been!15:13
TheJuliaQianbiao: So, the conundrum really is that unless you have patches up say... right now for the feature, it will be near impossible for us to get it into the ussuri release because we will very likely need to release next week.15:15
rpittauthe rc1 deadline is april 2315:15
dtantsuryeah, rc1 is essentially a hard feature freeze15:16
* uzumaki finds this very interesting and educational15:16
rpittauthat would require some lickety-split review15:16
dtantsurTheJulia: JFYI there are some discussions around renaming modules for the new openstack collection15:16
*** k_mouza has quit IRC15:17
dtantsurtl;dr os_ironic probably becomes baremetal_node, os_ironic_node - baremetal_node_action15:17
TheJuliadtantsur: sounds good15:17
dtantsurk, great15:17
dtantsursomebody will have to update bifrost eventually...15:17
TheJuliaif the modules are not completely changed behavior wise, then it should be fairly trivial15:18
dtantsurTheJulia: not yet ;)15:18
dtantsurI mean, not for the near future and the upcoming ansible 2.1015:18
uzumakiwhat is this 'openstack collection'? dtantsur TheJulia15:18
dtantsuruzumaki: ansible is splitting out parts of its built-in modules15:18
dtantsurin so called collections15:18
uzumakiah! I see15:19
*** ociuhandu has quit IRC15:19
dtantsuruzumaki: https://opendev.org/openstack/ansible-collections-openstack is ours15:19
dtantsuransible 2.10 won't have built-in openstack modules, the collection will be used instead15:19
uzumakiHrmmm... interesting, why is the renaming necessary?15:19
dtantsurI *somehow* ended up a core member of this initiative :D15:19
* uzumaki suspects that highly :D xD15:19
dtantsurfor ironic ones? two reasons:15:20
dtantsur1) all other modules use object names (server, dns_zone), not project code names (ironic)15:20
uzumakiI think since it's part of openstack collection, calling it ironic would be weird, since it's simply bare metal15:20
dtantsur2) ironic vs ironic_node is confusing since the former is actually creating/deleting nodes15:20
uzumakioh, you've got more technical reasons, that figures, you being a core member and all dtantsur ;)15:21
*** ociuhandu has joined #openstack-ironic15:21
TheJuliaQianbiao: so no real objection if the patches are up, but us backporting the patches to bring the functionality into the openstack stable branch is kind of a non-starter. What could be done is we release a new version upstream once the feature lands15:21
uzumakidtantsur, this is very interesting.. thanks!15:22
QianbiaoQianbiao really thanks. i will finish it asap.15:23
QianbiaoTheJulia15:23
*** JasonF is now known as JayF15:25
*** ociuhandu has quit IRC15:25
*** k_mouza has joined #openstack-ironic15:26
uzumakiBy the way, would appreciate a quick look on this https://review.opendev.org/#/c/713450/, let me know what's the way forward15:26
patchbotpatch 713450 - ironic - Implement the bios-interface for idrac-wsman driver - 23 patch sets15:26
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251615:26
*** mkrai has joined #openstack-ironic15:31
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network hook  https://review.opendev.org/71859415:32
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Add BasePhysnetHook  https://review.opendev.org/72001015:32
*** mkrai_ has joined #openstack-ironic15:33
*** Qianbiao is now known as Qianbiao|afk15:33
*** Qianbiao|afk is now known as Qianbiao|leave15:33
*** Qianbiao|leave has quit IRC15:33
openstackgerritRiccardo Pittau proposed openstack/ironic master: BUILD_TIMEOUT is not needed anymore  https://review.opendev.org/72001215:34
*** mkrai has quit IRC15:35
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network hook  https://review.opendev.org/71859415:37
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network hook  https://review.opendev.org/71859415:38
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: BUILD_TIMEOUT is not needed anymore  https://review.opendev.org/72001615:39
iurygregoryironic-tempest-ipa-wholedisk-uefi-ipmi-iscsi-tinyipa success  \o/15:39
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: BUILD_TIMEOUT is not needed anymore  https://review.opendev.org/72001815:40
*** uzumaki has quit IRC15:40
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251615:47
iurygregorywtf success, but the logs saying it didn't work D:15:48
iurygregorysad 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.txt15:49
JayFDo we know of anyone running a newer IPA against an older Ironic? Are there any non-obvious concerns? (I've read all the commit titles and release notes)15:50
*** uzumaki has joined #openstack-ironic15:50
JayFI'm looking into running IPA as new as Stein with Ironic stable/ocata15:50
*** bdodd has quit IRC15:51
*** bdodd has joined #openstack-ironic15:55
*** ociuhandu has joined #openstack-ironic15:56
TheJuliaKnow of not really, that _should_ work I think15:57
TheJuliawe REALLY try not to break IPA15:58
TheJuliaas your likely very well aware :)15:58
*** lucasagomes has quit IRC16:00
*** gyee has joined #openstack-ironic16:01
rpittaubye, good night! o/16:02
*** rpittau is now known as rpittau|afk16:02
iurygregory'mount /dev/sda15 /tmp/tmpgzm12381/boot/efi' failed. Not Retrying. why ipa why D:16:03
JayFTheJulia: yeah, there's something in Queens where the agent became aware of Ironic microversions, and started sending agent_version back to Ironic, but that's the only thing I saw that could be an issue. Nothing that I could find that wouldn't be trivial to patch out or fix.16:03
JayFTheJulia: ty for confirming my research :) I always like asking someone smarter for verification :P16:03
iurygregoryI will call it a day and try more tomorrow, bye everyone o/16:04
TheJuliaiurygregory: time to ls the folders?16:04
iurygregoryfrom CI?16:04
iurygregoryO.o16:04
iurygregoryI will try to test locally with more debug16:04
iurygregorythe workflow seems correct but fails to mount the partition16:05
ajyaJayF, recently I run older Ironic with the latest IPA - there were some errors around the API tokens.16:05
JayFYeah, I think that's only in master.16:05
iurygregorytoken was added in master (aka ussuri)16:06
*** mkrai_ has quit IRC16:09
ajyaright, should be fine then16:09
TheJuliaiurygregory: yeah, I did a patch to do an ls of the root folder, it could be you have a boot folder but no contents in that folder for efi16:14
*** tkajinam has quit IRC16:20
openstackgerritMerged openstack/ironic master: Improve `redfish` set-boot-device behaviour  https://review.opendev.org/71111816:21
arne_wiebalckbye everyone o/16:24
*** yaawang_ has joined #openstack-ironic16:32
*** yaawang has quit IRC16:33
*** ricolin has quit IRC16:35
openstackgerritAija Jaunteva proposed openstack/ironic master: Update iDRAC doc about soft power off timeout  https://review.opendev.org/71876116:45
*** derekh has quit IRC16:54
iurygregoryTheJulia, the lsblk should show the partition I've found uefi correct ? 'Found efi partition 15 on device /dev/sda'  the lsblk from deploy doesn't show sda15 http://paste.openstack.org/show/792113/ O.o16:57
*** dtantsur is now known as dtantsur|afk16:57
dtantsur|afko/16:57
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client master: Document the new standalone CLI  https://review.opendev.org/72003417:05
*** k_mouza has quit IRC17:21
*** Goneri has quit IRC17:23
*** jamesdenton_wfh has joined #openstack-ironic17:35
*** ociuhandu has quit IRC17:40
*** ociuhandu has joined #openstack-ironic17:41
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface  https://review.opendev.org/68818417:43
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface  https://review.opendev.org/68826617:43
openstackgerritIlya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data`  https://review.opendev.org/68827217:44
openstackgerritIlya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface  https://review.opendev.org/68889617:44
openstackgerritIlya Etingof proposed openstack/ironic master: Add L3 boot section to the docs  https://review.opendev.org/68984417:44
*** ociuhandu has quit IRC17:46
*** Goneri has joined #openstack-ironic17:52
*** alexmcleod has quit IRC17:55
*** uzumaki has quit IRC18:02
*** ociuhandu has joined #openstack-ironic18:14
*** k_mouza has joined #openstack-ironic18:21
*** k_mouza has quit IRC18:26
*** ociuhandu has quit IRC18:50
*** ociuhandu has joined #openstack-ironic18:51
*** ociuhandu has quit IRC18:55
TheJuliahmmm... tempest, why do you always like timing out after 10 minutes18:59
*** Lucas_Gray has joined #openstack-ironic19:11
*** ociuhandu has joined #openstack-ironic19:16
etingofam I the only one who runs devstack in centos7? I keep hitting silly bugs19:21
*** Lucas_Gray has quit IRC19:21
*** Lucas_Gray has joined #openstack-ironic19:22
iurygregoryetingof, if would switch to centos8 because of python3 /me never tried devstack in centos719:28
etingofit used to work in the past19:29
etingofso what is the best OS for devstack?19:29
openstackgerritJulia Kreger proposed openstack/ironic master: Extend timeout on CI job with automated cleaning  https://review.opendev.org/72006519:29
TheJuliaI really dislike tempest timeouts19:30
TheJuliaI _think_ ^^ is why our gate keep sfailing19:30
iurygregorynow with python3 only centos8 or ubuntu bionic19:30
TheJuliaYeah, I think both worked out of the box last time I ran devstack on each19:31
iurygregoryI thought we had a bigger timeout for tempest when running automated cleaning o.O19:31
TheJuliaif I can at least get a +1 or two on https://review.opendev.org/720065, I'm going to  approve it to push it through the gate since if it passes the odds will seem better19:31
patchbotpatch 720065 - ironic - Extend timeout on CI job with automated cleaning - 1 patch set19:31
TheJuliaiurygregory: we only do metadata erasure19:31
TheJuliathe fun thing is... comparing log times, it still seems to only wait about 10 minutes19:32
iurygregoryTheJulia, you have a +1 =)19:32
TheJuliaOn one test, it asked nova to delete the instance at 17:52:37 and gave up on the job at 18:02:21 as the last check that proceeded... which means for some reason it had a 600 second timeout19:33
TheJuliaexcept it was configured for 720 :(19:33
TheJuliaiurygregory: thanks!19:33
jamesdenton_wfhHi TheJulia - you have a sec to refresh your memory on https://review.opendev.org/#/c/557051/?19:35
patchbotpatch 557051 - networking-generic-switch - VLAN not tagged on every switchs (ABANDONED) - 6 patch sets19:35
*** jamesdenton_wfh is now known as jamesdenton19:35
iurygregoryyw =)19:36
TheJuliajamesdenton: looking19:37
iurygregorywe can also try to increase OS_TEST_TIMEOUT by setting tempest_test_timeout I think19:38
iurygregoryI will take a look at this tomorrow19:38
TheJuliaiurygregory: so I looked at what the vars that are getting loaded, it all seems 720 seconds19:39
TheJuliawhich is... maddening19:39
TheJuliajamesdenton: sure! whats up19:39
jamesdentongreat! so, i'd like to resurrect that patch, but more to address the basic active/backup use-case rather than lacp.19:40
jamesdentonsince there is no real switch configuration needed for an active/passive bond outside of ensure both members have the same config19:40
*** Lucas_Gray has quit IRC19:41
*** Lucas_Gray has joined #openstack-ironic19:45
iurygregoryO.o omg thats strange19:45
TheJuliajamesdenton: I can unabandon it if you would like :)19:46
TheJuliaMy only real objection to it was lack of unit testing on the patch19:46
jamesdentonyes, that would be great! i am happy to work up tests, i just need to better understand how. i'm not super familiar with the codebase yet19:47
jamesdentonso i'm trying to get spun up on that19:47
TheJuliajamesdenton: well, if you have questions we're here to help! :)19:54
TheJuliagoing to unabandon for you now19:54
TheJuliadone19:55
jamesdentonthanks again :)19:55
*** Goneri has quit IRC19:57
*** Goneri has joined #openstack-ironic19:58
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Add BasePhysnetHook  https://review.opendev.org/72001020:00
TheJulia:)20:14
openstackgerritJames Denton proposed openstack/networking-generic-switch master: VLAN not tagged on every switchs  https://review.opendev.org/55705120:17
*** fdegir4 has quit IRC20:19
*** maelk has quit IRC20:19
*** fdegir4 has joined #openstack-ironic20:19
*** fdegir4 has quit IRC20:20
TheJulia\o/20:21
TheJuliajamesdenton: oh, I guess you don't have workflow-1 as your not the original submitter?20:21
openstackgerritChristopher Dearborn proposed openstack/ironic master: DRAC: Added redfish management interface issue  https://review.opendev.org/72007620:29
jamesdentonWell, i added it just out of habit, i guess. i only rebased it20:32
jamesdentonthe plan is to write the tests and then kick the tires (again) on some gear20:33
TheJuliajamesdenton: okay! Let us know what sort of gear you kicked the tires with because that will help with context20:40
*** Lucas_Gray has quit IRC20:44
*** ociuhandu has quit IRC20:45
*** ociuhandu has joined #openstack-ironic20:46
*** ociuhandu has quit IRC20:52
*** Lucas_Gray has joined #openstack-ironic20:58
*** ociuhandu has joined #openstack-ironic21:02
*** ociuhandu has quit IRC21:06
*** Lucas_Gray has quit IRC21:06
*** Lucas_Gray has joined #openstack-ironic21:09
*** Lucas_Gray has quit IRC21:15
*** ociuhandu has joined #openstack-ironic21:20
*** Lucas_Gray has joined #openstack-ironic21:20
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Port physical network hook  https://review.opendev.org/71859421:22
*** Lucas_Gray has quit IRC21:32
*** Lucas_Gray has joined #openstack-ironic21:42
*** Lucas_Gray has quit IRC21:52
*** ociuhandu has quit IRC21:55
*** ociuhandu has joined #openstack-ironic21:56
*** ociuhandu has quit IRC22:01
*** ociuhandu has joined #openstack-ironic22:46
*** tkajinam has joined #openstack-ironic22:46
*** ociuhandu has quit IRC23:01
*** rh-jelabarre has quit IRC23:06
*** rloo has quit IRC23:22
*** jamesdenton has quit IRC23:33
*** ociuhandu has joined #openstack-ironic23:35
TheJuliacdearborn: Added issue?23:40
openstackgerritJulia Kreger proposed openstack/ironic-specs master: WIP - Victoria Cycle Priorit(y|ies)  https://review.opendev.org/72010023:49
TheJuliawow, I like that revision set number23:50
*** ociuhandu has quit IRC23:51

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