Friday, 2020-06-05

*** rcernin has joined #openstack-ironic00:02
*** rcernin has quit IRC00:06
*** diurnalist has quit IRC01:10
*** xiaolin has joined #openstack-ironic01:29
*** xiaolin has quit IRC01:45
*** diurnalist has joined #openstack-ironic01:50
*** xiaolin has joined #openstack-ironic01:51
*** Qianbiao has joined #openstack-ironic01:56
*** tzumainn has quit IRC02:27
*** rcernin has joined #openstack-ironic02:34
*** michchap has quit IRC02:35
*** rcernin has quit IRC02:41
*** ricolin has joined #openstack-ironic02:54
*** k_mouza has joined #openstack-ironic03:06
*** k_mouza has quit IRC03:10
*** tkajinam is now known as tkajinam|lunch03:19
*** tkajinam|lunch is now known as tkajinam03:19
*** priteau has joined #openstack-ironic03:39
*** priteau has quit IRC03:46
*** uzumaki has quit IRC05:26
*** diurnalist has quit IRC05:43
*** yaawang has quit IRC05:47
*** yaawang has joined #openstack-ironic05:47
arne_wiebalckGood morning, ironic!05:48
*** yaawang has quit IRC06:01
*** yaawang has joined #openstack-ironic06:02
*** belmoreira has joined #openstack-ironic06:33
*** dougsz has joined #openstack-ironic06:57
*** rpittau|afk is now known as rpittau07:04
rpittaugood morning ironic! o/07:04
*** priteau has joined #openstack-ironic07:11
*** yaawang has quit IRC07:18
*** yaawang has joined #openstack-ironic07:19
iurygregorygood morning arne_wiebalck rpittau and Ironic o/07:22
rpittauhey iurygregory :)07:22
arne_wiebalckhey iurygregory rpittau o/07:22
rpittauhey arne_wiebalck :)07:22
*** yolanda has joined #openstack-ironic07:25
*** priteau has quit IRC07:35
Qianbiaohello, the baremetal deploy blocked after loading deploy_ramdisk .07:46
Qianbiaosometimes. not always, is there a known issue connect to this?07:47
Qianbiaothe console shows "Loading xxxxx/deploy_ramdisk... ok", and then stucked there.07:49
*** aedc_ has joined #openstack-ironic07:57
*** zigo_ is now known as zigo07:58
*** priteau has joined #openstack-ironic08:02
*** jawad_axd has joined #openstack-ironic08:03
iurygregoryQianbiao, I would say to try to debug when doing the deploy, the troubleshooting may give some ideas https://docs.openstack.org/ironic/latest/admin/troubleshooting.html08:06
*** yaawang has quit IRC08:08
Qianbiaoiurygregory i have tried to get deploy.log under /var/log/ironic/deploy08:08
Qianbiaobut it seems nothing related to this failed deploy08:08
iurygregoryyour setup is configured to get the deploy logs?08:08
*** yaawang has joined #openstack-ironic08:09
iurygregorymaybe you changed the dir where it should be stored08:09
Qianbiaoyes, collect log always08:09
Qianbiaothe dir is right. I can find other deploy log there.08:09
iurygregorygotcha, strange .-.08:09
Qianbiaoit's not always reproduce08:09
QianbiaoI do not how to reproduce it.08:10
*** lucasagomes has joined #openstack-ironic08:10
*** ravsingh has joined #openstack-ironic08:19
rpittauQianbiao: is that on real hardware? cause it smells like network issues to me08:20
Qianbiaoyes it's ibmc baremetal server.08:20
Qianbiaoafter loading deploy_ramdisk image, it should start the tiny os, right?08:21
Qianbiaonot sure why it does not.08:21
*** priteau has quit IRC08:22
rpittauQianbiao: yes, after loading the deploy_ramdisk it should load the kernel from tinycore08:23
*** priteau has joined #openstack-ironic08:24
Qianbiaorpittau the network for what may cause the issue, i will check it.08:24
openstackgerritRiccardo Pittau proposed openstack/ironic master: Enforce autospec in test_deploy_utils module  https://review.opendev.org/73376708:24
Qianbiaoit's a standalone env like bifrost.08:24
rpittauQianbiao: flat network? it could also be an issue with the memory on the server or a corrupted ramdisk, have you tried loading a different ramdisk in memory?08:26
Qianbiaorpittau i am using tinyipa, and it works at most times.08:27
Qianbiaonetwork setting is noop08:28
Qianbiaoip offered from dhcp.08:28
Qianbiaodnsmasq08:28
Qianbiaotinyos is syslinux-6.03.tgz08:29
rpittauQianbiao: are you using an official tinyipa ?08:29
Qianbiaoyes08:29
rpittauQianbiao: from master or from a stable branch?08:30
*** tkajinam has quit IRC08:31
Qianbiaoit's ussuri for now.08:31
Qianbiaostable ussuri.08:31
openstackgerritArne Wiebalck proposed openstack/ironic-inspector master: [doc] Add misconfigured BMC channel to trouble shooting guide  https://review.opendev.org/73377008:32
*** dtantsur|afk is now known as dtantsur08:34
rpittauok, a bit hard to exclude anything, but first I would still check potential network issues or problems with the memory of the server08:35
rpittauif the ramdisk works most of the times it might be that sometimes it ended up in a corrupted state after the download08:35
dtantsurmorning ironic08:35
rpittauhey dtantsur :)08:35
dtantsuriurygregory: hey, if you have a second: https://review.opendev.org/#/c/733636/08:37
patchbotpatch 733636 - bifrost - Set disable_dnsmasq_dns to True - 1 patch set08:37
dtantsurrpittau: o/ after thinking more about bifrost-cli, I'd like to reduce its scope to NOT include enroll and deploy08:37
dtantsurand build-images likely as well. actually, I'm considering deprecating the built-in DIB support in favour of people just calling IPA-builder or DIB08:37
rpittaudtantsur: that sounds good, so basically just an install tool ?08:38
dtantsuryep08:38
Qianbiaorpittau yes, it's possible08:38
dtantsur"How do I get from clean OS to ironic running?"08:38
Qianbiaoi will ask hardware engineer to have a look08:38
dtantsurwithin the V timeframe I'd like to also have 'bifrost-cli update' which is something we don't quite support yet08:39
Qianbiaodtantsur for personal, i think if we could get a simple config.yaml file and document it.08:39
Qianbiaothere are no need to create a cli08:39
Qianbiaoansible-playbook is good enough08:39
rpittaudtantsur: I think it makes sense, and the upgrade is something that we really need for testing08:39
dtantsuransible-playbooks has a downside of not making the options obvious08:40
Qianbiaoansible-playbook xxxx -tag deploy|install|xxxx08:40
dtantsuralso there is an aspect of not having 'ansible-playbook' :)08:40
Qianbiao:) ok08:41
rpittauprobably a new set of playbooks for the upgrade08:41
* rpittau second breakfast time08:42
iurygregorydtantsur, sure08:42
dtantsurrpittau: I hope upgrade can be done by a new flag to the existing install playbooks08:43
dtantsurlike, allowing git to update checkouts, making sure we run online migrations first, etc08:44
Qianbiaodtantsur it's not that easy.. if user update ironic.conf manually? or ironic update some config items.08:44
dtantsuriurygregory: thanks! could you also check other outstanding bifrost patches when you have more time? some of them are required for the next ussuri release (also cc mgoddard)08:45
dtantsurQianbiao: I don't think we support manual updates08:45
iurygregorydtantsur, will do o/08:45
Qianbiaodtantsur i see.08:47
Qianbiaofor me, the diffcult part for upgrade is the conf file. sometimes it's not compatible between versions.08:47
Qianbiaoupgrade itself is pretty simple, just install new code, and do db sync.08:48
iurygregorydtantsur, also I was able to get ansible 2.9 working o/ will update the patch with a release note and the change that rpittau pointed in the review08:48
dtantsur\o/08:49
* iurygregory needs to say tks to my friend who helped me hehehe08:49
*** uzumaki has joined #openstack-ironic08:53
iurygregorydtantsur, you may have a conflict on https://review.opendev.org/#/c/733075/ after CI merges https://review.opendev.org/#/c/733150/09:08
patchbotpatch 733075 - bifrost - Switch to using virtual envs by default - 2 patch sets09:08
patchbotpatch 733150 - bifrost - Drop hard requirement on debootstrap and EPEL - 3 patch sets09:08
openstackgerritAndreas Jaeger proposed openstack/ironic-python-agent master: DNM Test ironic-python-agent-check-image-dib-centos8  https://review.opendev.org/73378109:10
dtantsuriurygregory: not strictly necessary, but I can try rebasing in advance09:10
openstackgerritMerged openstack/bifrost master: Set disable_dnsmasq_dns to True  https://review.opendev.org/73363609:10
dtantsuriurygregory: nope, it rebases cleanly09:11
iurygregorydtantsur, oh awesome!09:12
iurygregoryI will finish the review09:12
*** uzumaki has quit IRC09:12
openstackgerritDmitry Tantsur proposed openstack/bifrost master: bifrost-create-vm-nodes: generate a normal nodes.json  https://review.opendev.org/73378309:14
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: Set disable_dnsmasq_dns to True  https://review.opendev.org/73378409:14
*** yaawang has quit IRC09:16
openstackgerritMerged openstack/bifrost master: Enable manual-management and the noop management interface  https://review.opendev.org/73315609:16
*** yaawang has joined #openstack-ironic09:17
*** rcernin has joined #openstack-ironic09:18
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Quick start Bifrost CLI  https://review.opendev.org/73355909:18
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Quick start Bifrost CLI  https://review.opendev.org/73355909:20
*** rcernin has quit IRC09:22
*** AJaeger has joined #openstack-ironic09:24
openstackgerritMerged openstack/ironic master: Enable Basic HTTP authentication middleware  https://review.opendev.org/72746709:25
openstackgerritMerged openstack/ironic master: Add troubleshooting docs on -ing state failures.  https://review.opendev.org/73329409:25
AJaegerI see the job  ironic-python-agent-builder failing everywhere since yesterday, see https://zuul.opendev.org/t/openstack/builds?job_name=ironic-python-agent-check-image-dib-centos809:25
AJaegeranybody an idea or fix for this?09:26
rpittauAJaeger: yeah, I saw that too, we're mising a dep09:26
AJaegerlibffi-devel?09:26
rpittauAJaeger: it looks like it, I'm testing locally directly on ipa-builder09:26
AJaegerrpittau: thanks, I'll wait ;)09:27
rpittauok :)09:27
openstackgerritMerged openstack/bifrost master: Install uWSGI from PyPI to repair CentOS 8 support  https://review.opendev.org/73307609:27
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: Install uWSGI from PyPI to repair CentOS 8 support  https://review.opendev.org/73378909:27
dtantsurrpittau: we can post a fix and watch it in the CI as well09:32
openstackgerritMerged openstack/bifrost master: Support Ubuntu Focal (20.04)  https://review.opendev.org/73306009:32
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: Support Ubuntu Focal (20.04)  https://review.opendev.org/73379309:34
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Fix centos8 job  https://review.opendev.org/73379409:37
rpittaukind of weird seeing that pacakge suddenly missing :/09:41
openstackgerritMerged openstack/bifrost master: Drop hard requirement on debootstrap and EPEL  https://review.opendev.org/73315009:42
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: Drop hard requirement on debootstrap and EPEL  https://review.opendev.org/73379709:43
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Switch to using virtual envs by default  https://review.opendev.org/73307509:46
arne_wiebalckOps question: when the motherboard of an active node has been replaced, the address column in the ports table will be wrong (since the MAC has changed), so will all subsequent Ironic actions on this node fail since Ironic will not be able to find the node?09:46
dtantsurrpittau, iurygregory, could you re-review https://review.opendev.org/#/c/733075 ? no code changes, gerrit reported a bogus merge conflict :(09:47
patchbotpatch 733075 - bifrost - Switch to using virtual envs by default - 3 patch sets09:47
dtantsurarne_wiebalck: lookup and neutron DHCP won't work any more09:48
arne_wiebalckdtantsur: do we have a way to cure this?09:48
iurygregorydtantsur, bad gerrit =(09:48
dtantsurarne_wiebalck: you can update a MAC on an active node (I don't remember if you need maintenance mode for that)09:48
dtantsuriurygregory: right? it rebased locally without conflict..09:48
iurygregoryblack magic of our CI \o/09:49
rpittaupretty sure it's aliens09:50
arne_wiebalckdtantsur: oh, rly? which command would that be? can't see anything rn09:51
dtantsurarne_wiebalck: $ openstack baremetal port set --address=<new> <port UUID>09:51
arne_wiebalckdtantsur: yep, that's what I was looking for, thanks!09:52
openstackgerritDmitry Tantsur proposed openstack/bifrost master: bifrost-create-vm-nodes: generate a normal nodes.json  https://review.opendev.org/73378309:53
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Quick start Bifrost CLI  https://review.opendev.org/73355909:54
*** yaawang has quit IRC09:56
*** yaawang has joined #openstack-ironic09:57
rpittauthe theory of "everything else fails in a fix patch" seems getting more and more confirmations09:58
dtantsur+++09:58
openstackgerritIury Gregory Melo Ferreira proposed openstack/bifrost master: Moving to ansible 2.9  https://review.opendev.org/73347910:01
iurygregoryand we don't have the "one patch to fix them all"10:01
*** k_mouza has joined #openstack-ironic10:03
*** rpittau is now known as rpittau|bbl10:08
rpittau|bblsadness amplifies -> https://storyboard.openstack.org/#!/story/200775910:13
* rpittau|bbl milanese and fries needed, ttyl10:13
iurygregorydamm10:16
*** k_mouza has quit IRC10:24
*** k_mouza has joined #openstack-ironic10:25
dtantsursigh10:28
*** k_mouza has quit IRC10:29
openstackgerritIury Gregory Melo Ferreira proposed openstack/bifrost master: Moving to ansible 2.9  https://review.opendev.org/73347910:34
*** diurnalist has joined #openstack-ironic10:34
*** jawad_axd has quit IRC10:35
AJaegerrpittau|bbl: thansk, your change fixed the job! Enjoy lunch!10:38
*** diurnalist has quit IRC10:39
* dtantsur wonders where libffi-dev disappeared overnight10:41
AJaegerdtantsur: new package update that does not require it anymore?10:42
*** k_mouza has joined #openstack-ironic10:43
dtantsuryeah, likely10:44
*** derekh has joined #openstack-ironic10:56
*** jawad_axd has joined #openstack-ironic10:57
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Use the new extension call for getting partition UUIDs  https://review.opendev.org/73174510:58
openstackgerritMerged openstack/bifrost master: Switch to using virtual envs by default  https://review.opendev.org/73307510:58
openstackgerritDmitry Tantsur proposed openstack/ironic stable/ussuri: Fix fast track when exiting cleaning  https://review.opendev.org/73380310:59
*** jawad_axd has quit IRC11:02
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: Switch to using virtual envs by default  https://review.opendev.org/73380811:03
*** k_mouza has quit IRC11:04
*** k_mouza has joined #openstack-ironic11:05
openstackgerritMerged openstack/bifrost master: Install metalsmith by default  https://review.opendev.org/73309511:13
openstackgerritMerged openstack/bifrost master: Documentation: minor fixes in the user guide  https://review.opendev.org/73348611:13
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/ussuri: Documentation: minor fixes in the user guide  https://review.opendev.org/73381111:15
dtantsurmgoddard: we can release bifrost ussuri when we merge https://review.opendev.org/#/q/status:open+project:openstack/bifrost+branch:stable/ussuri (help appreciated)11:17
mgoddarddtantsur: not stable core11:17
dtantsurmgoddard: I think bifrost doesn't follow the stable policy and thus allows any core to merge changes11:17
* dtantsur cannot check it since he's a stable core11:17
mgoddarddtantsur: would like to release stables ASAP due to the DIB breakage11:18
mgoddarddtantsur: seems you are right11:18
dtantsurmgoddard: yep, but the breakage didn't (IIRC) affect Ussuri11:18
dtantsurso let's release it separately11:18
iurygregorydtantsur, we do consider https://review.opendev.org/733793 important for stable/ussuri?11:18
patchbotpatch 733793 - bifrost (stable/ussuri) - Support Ubuntu Focal (20.04) - 1 patch set11:18
dtantsuriurygregory: yep, why not?11:18
mgoddardright, that's why I removed it from the patch11:18
iurygregorydtantsur, well in theory focal will be something for victoria11:19
iurygregorybut since it works why not =)11:19
mgoddardas usual, the first release of bifrost ussuri doesn't pin to stable/ussuri. Would be nice to release to get that fix11:19
dtantsurmgoddard: oh! good call. I can do the change.11:19
mgoddarddtantsur: I think it was fixed, just not released11:19
* iurygregory facepalm after reading what mgoddard said =X11:19
mgoddardiurygregory: happens every time11:20
dtantsurah, right11:20
mgoddardI suggested pinning on master then reverting but didn't get traction11:20
* dtantsur is considering implications11:21
mgoddardit means bifrost has to release after other ironic projects11:21
dtantsuryeah, this is probably the only implication11:21
dtantsurdunno, let's start an ML thread?11:21
dtantsuriurygregory: "since it works" is my take on it :) we had to kick out Xenial in Ussuri, so I thought adding support for the latest LTS Ubuntu won't hurt.11:22
iurygregorydtantsur, yeah =)11:22
dtantsurif we consider bifrost as a way for people to get into ironic, supporting latest versions makes a lot of sense11:23
* iurygregory goes eat, meating in 37 min11:23
*** belmoreira has quit IRC11:36
*** k_mouza has quit IRC11:43
*** k_mouza has joined #openstack-ironic11:46
openstackgerritMerged openstack/bifrost stable/ussuri: Set disable_dnsmasq_dns to True  https://review.opendev.org/73378411:56
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Restore installing DIB-specific dependencies  https://review.opendev.org/73381511:56
dtantsurmgoddard: good catch on ^^ (will backport to ussuri as well)11:56
*** rpittau|bbl is now known as rpittau11:59
*** dougsz has quit IRC12:04
openstackgerritMerged openstack/ironic master: If the "[conductor]XXX_timeout" is less than 0,disable periodic task  https://review.opendev.org/72379512:05
openstackgerritMerged openstack/bifrost stable/ussuri: Install uWSGI from PyPI to repair CentOS 8 support  https://review.opendev.org/73378912:06
openstackgerritMerged openstack/bifrost stable/ussuri: Documentation: minor fixes in the user guide  https://review.opendev.org/73381112:06
openstackgerritMerged openstack/bifrost stable/ussuri: Drop hard requirement on debootstrap and EPEL  https://review.opendev.org/73379712:08
*** k_mouza has quit IRC12:13
*** k_mouza has joined #openstack-ironic12:14
*** rh-jelabarre has joined #openstack-ironic12:14
*** derekh has quit IRC12:20
*** rh-jelabarre has quit IRC12:22
openstackgerritMerged openstack/ironic master: Follow-up patch for ilo out-of-band `erase_devices`  https://review.opendev.org/67992712:22
dtantsurOUCH12:29
dtantsurironic master also fails with missing ffi-dev?12:29
rpittaulol12:29
dtantsurah, no, my patch uses IPA-builder12:30
* rpittau puts the rum bottle back in the shelf12:31
dtantsurrpittau: let's also put ffi-dev to https://opendev.org/openstack/ironic-python-agent/src/branch/master/bindep.txt12:31
rpittauyup12:31
dtantsurrpittau: ugh, https://review.opendev.org/#/c/733794/ is incomplete12:32
patchbotpatch 733794 - ironic-python-agent-builder - Fix centos8 job - 1 patch set12:32
dtantsuriurygregory: can you remove W+1 from ^^12:33
dtantsurrpittau: sorry for missing it, we need to update pkg-map to account for distro differences. libffi-devel is an RH name.12:33
iurygregorydtantsur, done12:33
dtantsurthx!12:33
rpittauright...12:33
dtantsuras an aside, we should move most of package-installs to bindep in IPA itself12:34
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Add ffi development lib to bindep  https://review.opendev.org/73382112:35
iurygregorydtantsur, you pinged right after i finished my meeting hehe12:36
*** derekh has joined #openstack-ironic12:36
dtantsurgood :)12:36
openstackgerritDmitry Tantsur proposed openstack/ironic master: Use the new extension call for getting partition UUIDs  https://review.opendev.org/73174512:39
openstackgerritMerged openstack/bifrost stable/ussuri: Support Ubuntu Focal (20.04)  https://review.opendev.org/73379312:40
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Install missing ffi development library  https://review.opendev.org/73379412:41
openstackgerritDmitry Tantsur proposed openstack/ironic master: Use the new extension call for getting partition UUIDs  https://review.opendev.org/73174512:41
rpittau5th recheck for the stable/ussuri memory fix12:43
dtantsur\o/12:43
*** rh-jelabarre has joined #openstack-ironic12:43
* rpittau takes the rum bottle back on the desk12:43
iurygregorywow12:43
*** michchap has joined #openstack-ironic12:48
openstackgerritMerged openstack/bifrost master: Moving to ansible 2.9  https://review.opendev.org/73347912:51
openstackgerritRiccardo Pittau proposed openstack/ironic master: Enforce autospec in test_ipxe module  https://review.opendev.org/73382412:51
*** rloo has joined #openstack-ironic12:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy  https://review.opendev.org/69830912:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps  https://review.opendev.org/73382512:52
TheJuliagood morning13:05
rpittaugood morning TheJulia :)13:05
dtantsurmorning TheJulia13:05
iurygregorygood morning TheJulia =)13:07
* TheJulia sips coffee and tries to wake up13:09
openstackgerritRiccardo Pittau proposed openstack/ironic master: Enforce autospec in test_ipxe and test_pxe  https://review.opendev.org/73382413:16
* TheJulia looks a patch and wonders if it is too early to enjoy some tuaca13:19
*** rcernin has joined #openstack-ironic13:19
dtantsurdepends on which patch (and what tuaca is)13:20
* iurygregory googles tuaca13:20
TheJuliasushy-tools13:20
TheJuliaamd google :)13:20
iurygregoryomg liqueur wow13:20
iurygregory70%?13:20
TheJuliaI don't think we can get it that strong here13:21
dtantsuriurygregory: wikipedia says  70 proof (35% ABV).13:21
dtantsurdunno what proof is, but 35% sounds reasonable13:21
rpittauis tuaca good with sushy ? :)13:21
iurygregoryyeah13:21
TheJuliarpittau: good question13:21
iurygregoryrpittau, probably with sushy-tools13:21
dtantsur:D13:21
iurygregoryor sushi =)13:21
dtantsurwell, a sweet liqueur with fish..13:22
dtantsurprobably when TheJulia will start reviewing my deploy steps patches? :D13:23
TheJuliathose are a fine whiskey sort of patches13:23
iurygregoryhehehe13:23
*** rcernin has quit IRC13:24
dtantsurmgoddard: not sure about you, but every time I read has_decomposed_deploy_steps, I interpret "decomposed" in its another meaning, and it's pretty ironic13:25
iurygregorydecomposed sounds like you have a huge step and you split in many that in the end will do the same thing of the big step ...13:26
dtantsurthis is the reality. my brain substitutes something like "rotten" though13:27
iurygregorywpw13:27
iurygregorywow*13:27
TheJuliaThere is one thing that worries me about deploy steps which is the comment that dtantsur left on the oom prevention fix I've been working on that apparently we won't be able to stall out a step until the next heartbeat?13:28
dtantsurmmm, not sure I understand13:29
dtantsurI think it's a good thing about deploy steps that you can stop one and get back to it in 2 mins?13:29
dtantsur(or how often do we heartbeat?)13:29
TheJulia120-300 seconds by default13:30
TheJuliaI guess for deploy steps we would need to instead set appropriate flags and then return nothing?13:31
*** bnemec is now known as beekneemech13:32
dtantsurTheJulia: DEPLOYWAIT+flags, I think13:32
dtantsurand I may be wrong since I still fail to keep the whole picture in my head13:32
* dtantsur had to draw the interaction inside agent_base.py and agent.py and his whiteboard13:33
iurygregorywow13:42
iurygregorydtantsur, you took a picture of it?13:43
dtantsuryeah13:50
TheJuliadtantsur: that is my problem right now with stress, I'm second guessing myself too :(13:52
TheJuliaon a plus side, it is a perfect time to build a compatability chart13:52
openstackgerritRiccardo Pittau proposed openstack/bifrost master: [WIP] Convert with_items to loop  https://review.opendev.org/73384713:53
TheJuliarpioso: I'm nowhere near done on that chart, and we may need several so it might be a few days before I can cycle to the questions13:53
iurygregoryironic-python-agent-check-image-dib-fedora it's broken for a while right?13:54
iurygregoryat least looking at https://zuul.openstack.org/builds?job_name=ironic-python-agent-check-image-dib-fedora13:54
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose the core deploy step of the direct deploy  https://review.opendev.org/69830914:14
openstackgerritDmitry Tantsur proposed openstack/ironic master: DNM testing deploy steps  https://review.opendev.org/73382514:14
openstackgerritMerged openstack/sushy-tools master: Read current domain state from XML on filesystem  https://review.opendev.org/73315214:17
openstackgerritDmitry Tantsur proposed openstack/sushy-tools master: Simplify the CI jobs and add a vmedia+UEFI job  https://review.opendev.org/73349514:18
*** ricolin_ has joined #openstack-ironic14:26
*** ricolin has quit IRC14:29
openstackgerritMerged openstack/bifrost stable/ussuri: Switch to using virtual envs by default  https://review.opendev.org/73380814:32
*** Qianbiao has quit IRC14:36
*** Qianbiao has joined #openstack-ironic14:37
openstackgerritDmitry Tantsur proposed openstack/bifrost master: Set default deploy/rescue kernel/ramdisk in ironic.conf  https://review.opendev.org/73386014:41
*** diurnalist has joined #openstack-ironic14:52
*** rcernin has joined #openstack-ironic15:20
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Add api-ref for indicators API  https://review.opendev.org/73214915:24
*** rcernin has quit IRC15:24
*** gyee has joined #openstack-ironic15:44
*** Qianbiao has quit IRC15:50
dtantsurcurl | sudo, lol JayF16:02
iurygregoryLOL16:03
*** tkajinam has joined #openstack-ironic16:08
*** dtantsur is now known as dtantsur|afk16:10
dtantsur|afkhave a great weekend!16:10
*** rpittau is now known as rpittau|afk16:10
rpittau|afkbye all! o/16:10
*** lucasagomes has quit IRC16:15
TheJuliaso random thought, it felt like we tried to compress things a little too much, thoughts?16:22
JayFI don't think I have a wide enough lens on the state of the project to judge that. I do know if the PTG had been much more time consuming, I'd have had to "pick and choose" what sessions I participated in.16:23
*** alexmcleod has quit IRC16:25
*** shadower has quit IRC16:35
TheJuliathis is true16:37
* TheJulia suspects today is going to require a nap16:37
*** shadower has joined #openstack-ironic16:41
*** tkajinam has quit IRC16:49
openstackgerritRiccardo Pittau proposed openstack/ironic master: Enforce autospec in test_snmp  https://review.opendev.org/73389116:51
TheJulia\o/ no more calls this week17:07
*** derekh has quit IRC17:09
*** ricolin_ has quit IRC17:11
openstackgerritMerged openstack/ironic-python-agent-builder master: Install missing ffi development library  https://review.opendev.org/73379417:12
*** AJaeger has left #openstack-ironic17:13
arne_wiebalckTheJulia: If that means you're bored, I have put up multiple doc patches for Ironic and the Inspector re troubleshooting :-P17:15
openstackgerritMerged openstack/bifrost stable/stein: CI: reduce memory and disk for testing nodes  https://review.opendev.org/73356017:17
*** rcernin has joined #openstack-ironic17:21
*** priteau has quit IRC17:21
TheJuliaarne_wiebalck: Heh, I'm going to first go get some tasty lunch and then likely work on the capabilities doc17:25
*** rcernin has quit IRC17:26
arne_wiebalckTheJulia: ;)17:28
arne_wiebalckTheJulia: Enjoy the call free rest of Friday and have a good week-end!17:28
TheJuliayou too arne_wiebalck17:28
arne_wiebalckTheJulia: Thanks again for managing the PTG!17:28
arne_wiebalckBye all o/17:29
TheJuliamaelk: your boot image requirements, have you had a chance to look at the ramdisk deploy interface?17:29
TheJuliahave a wonderful weekend!17:29
TheJuliamaelk: also, w/r/t mtls, it might be good to also discuss during one of our weekly meetings17:35
*** ociuhandu has joined #openstack-ironic18:25
*** ociuhandu has quit IRC18:26
*** ociuhandu_ has joined #openstack-ironic18:26
*** ociuhandu_ has quit IRC18:27
*** ociuhandu has joined #openstack-ironic18:27
*** ociuhandu has quit IRC18:32
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-lib master: DNM - Testing CI  https://review.opendev.org/73390918:33
*** k_mouza has quit IRC18:35
openstackgerritVerification of a change to openstack/ironic failed: CI: raise DIB jobs memory to 3 GiB  https://review.opendev.org/73351918:35
*** ociuhandu has joined #openstack-ironic18:35
TheJuliaci hates us18:36
*** ociuhandu has quit IRC18:40
iurygregoryyes =(18:46
iurygregoryand ironic-lib seems broken18:46
iurygregorythe unit tests at least18:46
larsksIf I'm using ironic without keystone (and w/o auth), how to I tell the cli (openstack baremetal ...) what endpoint to use?18:52
*** dustinc has joined #openstack-ironic19:01
*** aedc has joined #openstack-ironic19:07
*** aedc_ has quit IRC19:08
*** ravsingh has quit IRC19:11
*** dougsz has joined #openstack-ironic19:16
TheJulialarsks: one moment19:18
TheJuliahttps://github.com/openstack/bifrost/blob/master/env-vars19:19
larsksTheJulia: perfect, thanks!19:21
*** aedc has quit IRC19:23
*** jtomasek has quit IRC19:27
*** dougsz has quit IRC19:58
*** xiaolin has quit IRC20:06
iurygregoryseems zeroconf 0.27.1 broke us hehe20:30
TheJulia:(20:30
TheJuliahow bad/20:30
TheJulia?20:30
iurygregory3 tests failling in ironic-lib20:34
iurygregoryonly the mdns20:34
iurygregoryI think it was zeroconf trying to check locally20:34
iurygregoryoh ok ci install 0.27.020:34
* iurygregory looks at the changes in code for zeroconf20:35
iurygregoryBackwards incompatible:20:43
iurygregory    Removed previously deprecated ServiceInfo address constructor parameter and property20:43
iurygregoryAha!20:44
iurygregoryeither we should update our code to fix sicne we don't have address anymore or add a limit <0.27.0 in our requirements for now20:46
iurygregory0.26.3 works fine but 0.27.0 or 0.27.1 boom20:47
TheJuliaif you don't mind, go ahead and post a requirements limit for it20:47
iurygregorysure, do we need a release note?20:48
TheJuliais it in openstack/requirements/upper-constraints.txt?20:48
iurygregorywell I think we can fix by adding <0.27.0 in the requirements.txt for ironic-lib20:50
TheJuliaThe requirements tests may file a complaint with that, but always worth a try :)20:50
iurygregoryhttps://github.com/openstack/requirements/blob/master/upper-constraints.txt#L10420:50
iurygregoryoh truth20:51
TheJulia:)20:51
iurygregorylet me test the requirements locally20:51
TheJuliaoik20:51
TheJuliaok20:51
* TheJulia goes back to the compatabilit ymatrix20:51
iurygregoryERROR: Requirement for package zeroconf excludes a version not excluded in the global list20:53
iurygregoryyay =(20:53
TheJuliawe do it so projects can co-exist without breaking20:53
iurygregoryyeah20:54
iurygregoryIm going to check with requirements to see what we can do20:54
TheJuliapost a chance to the upper constraints20:54
TheJuliaerr20:54
TheJuliarequirements20:54
TheJuliaupper-constraints in the requirements repo is the latest gate tested version20:55
TheJuliasince we're likely the only ones who run it, it can be problematic20:55
iurygregoryack20:57
* TheJulia plays rammstein and wonders how many columns this table is going to end up with20:58
iurygregory++21:01
iurygregoryhttps://review.opendev.org/733920 done21:11
patchbotpatch 733920 - requirements - Pin zeroconf to 0.26.3 - 1 patch set21:11
TheJuliagetting there, looks like I've got everything that is not workflow related or highly vendor specific21:15
TheJuliaor network/deploy interface related21:15
TheJuliaI need to go lay down for a little bit21:15
*** rcernin has joined #openstack-ironic21:23
*** rcernin has quit IRC21:27
*** goldyfruit has joined #openstack-ironic21:37
openstackgerritMerged openstack/ironic master: Add `get_node_network_data` method to NetworkInterface  https://review.opendev.org/68818421:39
*** dsneddon has quit IRC21:40
*** bfournie has quit IRC21:40
*** dsneddon has joined #openstack-ironic21:40
*** goldyfruit has quit IRC21:41
iurygregorywow CI merged something =O21:46
*** bfournie has joined #openstack-ironic21:53
*** rloo has quit IRC22:00
*** dustinc has quit IRC22:01
*** rh-jelabarre has quit IRC22:22
*** ociuhandu has joined #openstack-ironic22:31
*** ociuhandu has quit IRC22:36
TheJuliagasp! Maybe it doesn't hate us22:38
TheJulia100%22:38
*** jawad_axd has joined #openstack-ironic22:42
*** rloo has joined #openstack-ironic22:45
*** jawad_axd has quit IRC22:47
*** rcernin has joined #openstack-ironic23:02
*** ab-a has quit IRC23:20
*** ab-a has joined #openstack-ironic23:21
*** rcernin has quit IRC23:27

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