Monday, 2020-01-20

« Sunday, 2020-01-19 Index Tuesday, 2020-01-21 »
*** ociuhandu has joined #openstack-ironic00:10
*** ociuhandu has quit IRC00:12
*** ociuhandu has joined #openstack-ironic00:14
*** ociuhandu has quit IRC00:19
*** jdandrea has joined #openstack-ironic00:55
openstackgerritJulia Kreger proposed openstack/bifrost master: Vagrant: run ansible-playbook from venv  https://review.opendev.org/70288501:33
openstackgerritJulia Kreger proposed openstack/bifrost master: DNM? Disable IPv6 for vagrant test VM  https://review.opendev.org/70184701:33
openstackgerritJulia Kreger proposed openstack/bifrost master: Add a listen-address to the dnsmasq bind  https://review.opendev.org/70331401:33
*** Lucas_Gray has quit IRC01:40
openstackgerritJulia Kreger proposed openstack/bifrost master: Add a listen-address to the dnsmasq bind  https://review.opendev.org/70331401:43
*** TxGirlGeek has joined #openstack-ironic02:27
*** TxGirlGeek has quit IRC02:30
*** hoonetorg has quit IRC02:59
*** k_mouza has joined #openstack-ironic03:08
*** hoonetorg has joined #openstack-ironic03:12
*** k_mouza has quit IRC03:13
*** jdandrea has quit IRC03:22
*** k_mouza has joined #openstack-ironic03:47
*** k_mouza has quit IRC03:52
*** k_mouza has joined #openstack-ironic04:17
*** k_mouza has quit IRC04:21
*** mkrai_ has joined #openstack-ironic04:36
*** rcernin_ has joined #openstack-ironic04:56
*** ociuhandu has joined #openstack-ironic04:59
*** rcernin has quit IRC04:59
*** ociuhandu has quit IRC05:04
openstackgerritMerged openstack/ironic master: Fix entry paths for cleaning and deployment  https://review.opendev.org/70316005:19
*** mkrai__ has joined #openstack-ironic06:15
*** mkrai_ has quit IRC06:19
*** mkrai_ has joined #openstack-ironic06:43
*** mkrai__ has quit IRC06:46
*** mkrai_ has quit IRC06:51
*** rcernin_ has quit IRC07:14
*** etingof has joined #openstack-ironic07:16
arne_wiebalckGood morning, ironic!07:33
*** rpittau|afk is now known as rpittau07:51
rpittaugood morning ironic! o/07:51
*** mkrai_ has joined #openstack-ironic07:54
*** ociuhandu has joined #openstack-ironic08:02
*** ociuhandu has quit IRC08:02
*** ociuhandu has joined #openstack-ironic08:03
*** ociuhandu has quit IRC08:05
*** ociuhandu has joined #openstack-ironic08:05
*** jtomasek has joined #openstack-ironic08:07
*** afasano has joined #openstack-ironic08:07
*** mkrai_ has quit IRC08:10
*** ociuhandu has quit IRC08:11
openstackgerritRiccardo Pittau proposed openstack/ironic stable/train: Fix entry paths for cleaning and deployment  https://review.opendev.org/70333708:12
*** ociuhandu has joined #openstack-ironic08:12
*** tesseract has joined #openstack-ironic08:16
*** ociuhandu has quit IRC08:18
*** ociuhandu has joined #openstack-ironic08:19
*** ociuhandu has quit IRC08:22
*** ociuhandu has joined #openstack-ironic08:23
openstackgerritJerry Wang proposed openstack/ironic master: [PATCH] Add support for replacing WSME  https://review.opendev.org/67585308:25
*** ociuhandu has quit IRC08:27
*** ociuhandu has joined #openstack-ironic08:29
*** ociuhandu has quit IRC08:30
*** ociuhandu has joined #openstack-ironic08:30
mgoddardmorning all. I'm afraid I won't be able to make the meeting today08:41
szivianiGood morning, Ironic08:43
*** iurygregory has quit IRC08:49
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Define venv package for debian based distro  https://review.opendev.org/70334308:51
*** ociuhandu has quit IRC08:51
*** ociuhandu has joined #openstack-ironic08:52
*** ociuhandu has quit IRC08:53
*** ociuhandu has joined #openstack-ironic08:54
*** lucasagomes has joined #openstack-ironic08:54
*** khansa has joined #openstack-ironic08:55
rpittauTheJulia, dtantsur|afk : I think we'll need this ^^ to move forward with venv in bionic08:55
*** ociuhandu has quit IRC08:55
*** ociuhandu has joined #openstack-ironic08:58
*** ociuhandu has quit IRC09:03
*** ricolin has quit IRC09:04
*** iurygregory has joined #openstack-ironic09:09
iurygregorygood morning o/09:09
*** ociuhandu has joined #openstack-ironic09:10
openstackgerritMerged openstack/ironic-ui stable/rocky: Fix horizon dependency  https://review.opendev.org/70168709:13
*** ociuhandu has quit IRC09:14
*** ricolin has joined #openstack-ironic09:15
*** k_mouza has joined #openstack-ironic09:18
*** dougsz has joined #openstack-ironic09:20
*** jhesketh has quit IRC09:32
*** jhesketh has joined #openstack-ironic09:34
yolandahi... coming again with more problems on deployment. Now  'details': 'Writing image to device /dev/sda failed with exit code 1. stdout: write_image.sh: Erasing existing GPT and MBR data structures from /dev/sda\n. stderr: blockdev: cannot open /dev/sda: No medium found\n'}09:46
*** derekh has joined #openstack-ironic09:47
*** alexmcleod has joined #openstack-ironic09:49
yolandaderekh, is that some error you got?09:50
derekhyolanda: which error? I've just logged in so don't have the scrollback09:51
yolandaah ok09:51
yolanda 'details': 'Writing image to device /dev/sda failed with exit code 1. stdout: write_image.sh: Erasing existing GPT and MBR data structures from /dev/sda\n. stderr: blockdev: cannot open /dev/sda: No medium found\n'}09:51
derekhI don't think I got that error but if you look back through the logs can you see if you have a /dev/sda ?09:52
yolandamm it picks sdb :(09:56
yolandai see logs from ironic_lib_utils showing "Picked root device /dev/sdb" . But when i do ironic node show, in properties i see root_device=/dev/sda09:56
yolandacan i pass some hints in bm.json?09:57
*** mkrai_ has joined #openstack-ironic10:06
openstackgerritkhansa proposed openstack/sushy-cli master: Update toxi.ini file  https://review.opendev.org/70313910:08
yolandahi derekh so the disk exists. I already have a centos deployed on sda on that machine10:13
derekh"No medium found" <-- I'm wondering if that either a broken drive or something to do with the way the virtual media gets attached in hpe10:13
derekhyolanda: ok, so maybe (and I'm only guessing), the virt media is sda ? if your able to list the all the drives I wonder if /dev/sda is appearing as /dev/sdb , do you have a list in the logs10:15
yolandahow can i better enable logs on IPA from that env? i just can see conductor logs10:17
derekhhave you got this directory on your bootstrap node ? /var/lib/containers/storage/volumes/ironic/_data/log/ironic/deploy/10:20
derekhit should have the deploy logs10:21
yolandaok i redeployed because the node was set to available where IPA was still running and failing... let me check again10:23
openstackgerritMerged openstack/metalsmith stable/train: Stop requiring root size for whole disk images  https://review.opendev.org/69399910:25
*** k_mouza has quit IRC10:39
*** k_mouza has joined #openstack-ironic10:42
*** ociuhandu has joined #openstack-ironic10:51
*** ricolin has quit IRC10:55
yolandaderekh, that folder is empty, no logs there11:07
*** ociuhandu has quit IRC11:08
*** ociuhandu has joined #openstack-ironic11:09
yolandabut i could take an screenshot: https://imgur.com/a/W35tc9M11:12
yolandasee, sda seems to be taken as cd-rom yes11:12
*** dtantsur|afk is now known as dtantsur11:13
openstackgerritkhansa proposed openstack/sushy-cli master: Update toxi.ini file  https://review.opendev.org/70313911:13
dtantsurmorning ironic11:13
*** ociuhandu has quit IRC11:14
yolandaso i shall create an HPE profile that uses sdb?11:15
*** ociuhandu has joined #openstack-ironic11:17
derekhyolanda: looking at the screenshot, /dev/sda doesn't looks like a normal disk to me ?11:17
derekhyolanda: no harm in trying sdb to see what happens (or just change the default profile to sdb for a quick test)11:18
iurygregorymorning dtantsur11:18
yolandait is the virtualmedia dvdrom11:18
*** rpittau is now known as rpittau|bbl11:20
derekhyolanda: but thats /dev/sr011:20
yolandain the screenshot, it shows sda as Virtual DFD-ROM...11:21
yolandaDVD-ROM...11:21
yolandaah ok, sr0 sorry11:21
dtantsurFYI folks we still haven't got DIB 2.33.0 in upper-constraints, so the DIB jobs are still broken..11:22
derekhnp :-)11:22
yolandaderekh, and can it be because the disk is already partitioned?11:23
yolandaTYPE did not match. Wanted 'disk' but found : "KNAME="sda1""11:23
*** ociuhandu has quit IRC11:24
derekhI think thats ok, its doesn't pick the partitions, but does select sda ,11:24
yolandaok let me try looking at more logs11:25
yolandai don't have any text log, just what i can see from screen11:25
derekhok, no harm in trying sdb after that you can boot into the IPA image, ssh on and see if you can get more details11:26
derekhalso if you've disable cleaning, try turning it back on again to get rid of those partitions11:27
yolandaok i can enable11:27
*** k_mouza has quit IRC11:29
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Fix and return the CentOS 7 job  https://review.opendev.org/70337111:39
dtantsurfolks, looking for a 2nd +2 on https://review.opendev.org/#/c/703069/11:45
patchbotpatch 703069 - bifrost - Use upper-constraints for ironicclient, sushy and DIB - 5 patch sets11:45
openstackgerritkhansa proposed openstack/sushy-cli master: Update tox.ini file  https://review.opendev.org/70313911:47
yolandaderekh, weird, it worked now and no error. With the same commands11:49
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/stein: DNM test stable/stein CI  https://review.opendev.org/70337511:50
derekhstrange11:50
openstackgerritMerged openstack/sushy-cli master: Update tox.ini file  https://review.opendev.org/70313911:54
*** ociuhandu has joined #openstack-ironic12:02
*** Lucas_Gray has joined #openstack-ironic12:04
*** ociuhandu has quit IRC12:08
*** k_mouza has joined #openstack-ironic12:09
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent stable/train: Search for efi partition  https://review.opendev.org/70338012:17
*** hjensas has joined #openstack-ironic12:18
*** mnaser has quit IRC12:27
*** mnaser has joined #openstack-ironic12:27
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-python-agent stable/train: Avoid grub2-install when on UEFI boot mode  https://review.opendev.org/70338112:31
*** rh-jelabarre has joined #openstack-ironic12:58
dtantsurTheJulia: https://review.opendev.org/#/c/703258/ may have uncovered some issue with our dependencies13:00
patchbotpatch 703258 - bifrost (stable/stein) - Use upper-constraints for ironicclient, sushy and DIB - 1 patch set13:00
dtantsurinspector fails to start due to what seems to be an eventlet/greenlet problem13:00
*** Lucas_Gray has quit IRC13:02
*** Lucas_Gray has joined #openstack-ironic13:06
*** dougsz has quit IRC13:08
*** Lucas_Gray has quit IRC13:11
*** rpittau|bbl is now known as rpittau13:11
*** Lucas_Gray has joined #openstack-ironic13:15
*** hjensas has quit IRC13:21
*** hjensas has joined #openstack-ironic13:24
*** dougsz has joined #openstack-ironic13:28
*** gkadam has joined #openstack-ironic13:30
*** gkadam has quit IRC13:31
*** dougsz has quit IRC13:32
yolandahi, i'm hitting this error on ironic conductor: http://paste.openstack.org/show/788600/13:35
yolandaproblem seems to be that i try to set ipmi and redfish as enabled hardware types, but my boot interface seems to be thought just for redfish:13:36
yolandacat /etc/ironic/ironic.conf | grep ipmi13:36
yolandaenabled_hardware_types = ipmi,irmc,fake-hardware,redfish13:36
yolandaenabled_management_interfaces = ipmitool,irmc,fake,redfish13:36
yolandaenabled_power_interfaces = ipmitool,irmc,fake,redfish13:36
yolandaenabled_vendor_interfaces = ipmitool,no-vendor,fake13:36
yolanda default_boot_interface = redfish-virtual-media13:36
yolandahow can i deploy with redfish, but without having to disable other hardware types as ipmitool, to be able to use on other nodes?13:37
*** ociuhandu has joined #openstack-ironic13:39
*** dougsz has joined #openstack-ironic13:40
*** jdandrea has joined #openstack-ironic13:40
*** jdandrea has joined #openstack-ironic13:41
*** ociuhandu has quit IRC13:44
TheJuliadtantsur: can we classify it as \o/?13:47
TheJuliaAlso, good morning everyone13:47
*** quoth has joined #openstack-ironic13:48
iurygregorygood morning TheJulia o/13:55
rpittaugood morning TheJulia and congratulations! :)13:56
TheJuliathanks13:56
*** ricolin has joined #openstack-ironic13:57
iurygregorywell deserved =)13:57
dtantsurmorning TheJulia, probably :)13:58
*** rloo has joined #openstack-ironic14:00
* iurygregory is wondering what is the problem with tox-pep8 and tox-py36 since they are getting RETRY_LIMIT since the weekend =( 14:00
dtantsurmirror problems?14:01
dtantsurRETRY_LIMIT usually implies something in infra14:01
TheJuliapossibly mirror problems. or general connectivity to the test machine.14:01
iurygregoryI think I got this 3 times in the efi patch XD14:02
iurygregorybut looking at the zuul it's not that bad, I'm just unlucky14:02
dtantsuryolanda: what do you have in your enabled_boot_interfaces?14:06
* dtantsur assumes that check was in creation-time14:06
dtantsurin any case, using default_***_interface is a bit dangerous, I'd recommend against doing it for most cases14:07
*** ricolin has quit IRC14:07
dtantsurin favour of specifying boot_interface when creating a node14:07
dtantsurFolks, this should fix the centos7 job on stable branches: https://review.opendev.org/#/c/703371/14:09
patchbotpatch 703371 - ironic-python-agent-builder - Fix and return the CentOS 7 job - 1 patch set14:09
dtantsurnot mergeable until we get the DIB fix14:09
*** Lucas_Gray has quit IRC14:09
dtantsurTheJulia and others: WDYT about starting consuming DIB from git master in the CI?14:09
rpittaudtantsur: I'm more than ok with that14:10
dtantsurIt's not what we should be doing ideally (it's commonly consumed via pip), but I don't know how to handle the constant CI failures combined with days required to get an upper-constraints in14:10
*** ricolin has joined #openstack-ironic14:11
*** ociuhandu has joined #openstack-ironic14:11
TheJuliadtantsur: I'm +1 on that. We did it in bifrost.... with mixed results14:12
iurygregoryif it helps the CI +1 =)14:12
dtantsurokay, I'll cook a patch14:12
dtantsurincidentally, it was a proposal from one of the DIB cores14:13
rpittaudtantsur: just one downside is that if DIB changes too much and we need to follow the change, IPA-B won't be compatible anymore with latest stable version14:13
dtantsurrpittau: exactly.14:14
rpittaucan we have both? :D14:14
dtantsurhow do you imagine that? :)14:14
dtantsurI mean, we can have two jobs14:14
rpittauyeah, it's too much I guess in terms of resources14:14
*** mkrai_ has quit IRC14:15
dtantsuranother nice bonus: depends-on will start working with DIB14:15
rpittauthat is very nice indeed14:15
*** ociuhandu has quit IRC14:18
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Start installing DIB from Zuul-cloned sources in the CI  https://review.opendev.org/70340014:20
iurygregoryTheJulia, the uefi patches we want to backport only to stein and train?14:22
*** ricolin_ has joined #openstack-ironic14:24
*** ricolin_ has quit IRC14:26
*** ricolin has quit IRC14:26
*** ricolin has joined #openstack-ironic14:26
*** ab-a has joined #openstack-ironic14:27
dtantsuretingof, iurygregory, have you seen https://storyboard.openstack.org/#!/story/2007071 ?14:31
iurygregorydtantsur, looking now14:32
iurygregoryseems a problem in sushy, in my CI test I'm getting problem with the metrics only14:34
* etingof pulls redfish schema by the tail14:35
*** goldyfruit has joined #openstack-ironic14:38
dtantsuriurygregory: it may be yet another case of a creative schema interpretation on one of the ends14:38
iurygregoryyeah, I'm adding some debugs locally and testing with devstack14:39
dtantsurugh https://storyboard.openstack.org/#!/story/2007004. I don't even know how to approach this...14:39
*** tzumainn has joined #openstack-ironic14:40
dtantsurTheJulia: I feel terrible about it, but I think we may need to deprecate ironic-ui at some point...14:41
etingofactually, 'target' seems lowcased in all actions, including dmtf example -- http://demo.snmplabs.com/redfish/v1/Systems/437XR1138R2/14:43
TheJuliadtantsur: we already basically said "best effort" and "when it breaks, its over" That being said I did talk to someone recently about a horizon panel for ironic... I wonder if they are doing anything.14:44
TheJuliaiurygregory: I suspect stein and train is likely good, nobody has complained about older versions and that at least covers the newer deployments so they don't shoot themselves in the foot upon deploy attempting secure boot14:45
iurygregoryTheJulia, ack o/14:45
dtantsuroh, Python's stdlib urlsplit has changed in a backward incompatible way from 3.6 to 3.8, sweeeeeet14:54
dtantsur(breaking the iBMC driver)14:54
etingofdtantsur, re fishy idrac, I have an observation - https://storyboard.openstack.org/#!/story/200707114:55
*** kaifeng_ has joined #openstack-ironic14:56
*** stendulker has joined #openstack-ironic14:57
* TheJulia senses the wifey has the day off14:57
dtantsur"day off" sounds nice14:58
TheJuliadtantsur: Getting out the Deprecation stamp14:58
dtantsur+++14:58
rpiosoGood morning, ironic14:58
dtantsuretingof: yet another case of creative schema interpretation? :( what are we going to do about it?14:58
etingofeasy! poke at rpioso14:59
dtantsuretingof: I mean... we have to assume that such machines are out in the wild, and we have to support them14:59
rpiosodtantsur, etingof, iurygregory: The R630 is a 13G system (iDRAC 8), which I believe is different from etingof's system under test. Its iDRAC firmware is different.14:59
* rpioso comes out of hibernation14:59
rpiosoetingof: Please don't poke the bear.15:00
TheJulia#startmeeting ironic15:00
TheJuliao/15:00
openstackMeeting started Mon Jan 20 15:00:27 2020 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
iurygregoryo/15:00
rpittauo/15:00
rpiosoo/15:00
etingofo/15:00
rlooo/15:00
TheJuliaGood Morning Ironic!15:00
kaifeng_o/15:00
arne_wiebalcko/15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
TheJuliaOur agenda for the meeting looks fairly normal, so I guess we'll get started!15:01
TheJulia#topic Announcements/Reminders15:01
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:01
*** cdearborn has joined #openstack-ironic15:02
stendulkero/15:02
dtantsuro/15:02
cdearborno/15:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Fix incorrect ibmc_address parsing on Python 3.8  https://review.opendev.org/70341115:02
TheJuliaFor context, we're a few weeks away from the Ussuri-2 milestone.15:02
TheJuliaWe're planning a midcycle at CERN in about a month as well.15:03
TheJulia#link https://etherpad.openstack.org/p/ironic-ussuri-midcycle15:03
arne_wiebalckPlease register if you plan to attend in person.15:04
TheJuliaSome topics that would be good to discuss have already been added.15:04
arne_wiebalckMost have already I think.15:04
TheJulia++15:04
* TheJulia loves the sushy topic being added15:04
TheJuliaAnyway, does anyone else have anything to announce?15:04
arne_wiebalckI would not send anything on the ML for the mid cycle meetup.15:04
TheJuliaOh, I'm taking PTO tomorrow for personal reasons. If you guys really need me, just let me know.15:05
TheJuliaWell, if there is nothing else we can move on.15:06
TheJuliaIt appears that there were no action items last week.15:06
TheJuliaSo we can skip that15:07
TheJulia#topic Review subteam status reports15:07
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:07
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:07
TheJuliaLine 26215:08
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Fix and return the CentOS 7 job  https://review.opendev.org/70337115:09
TheJuliaOn scale/performance, who put that they were going to do scale testing in Feburary ?15:09
arne_wiebalckI did.15:10
dtantsurthe color looks like arne_wiebalck15:10
dtantsurright :)15:11
TheJuliathanks!15:11
arne_wiebalckit's just to say it is on our list for hte coming weeks15:11
TheJuliaLooks like the L3 spec could use some reviews15:11
TheJuliaarne_wiebalck: Excellent!15:11
yolandahi dtantsur , seems that the problem is that i had redfish-virtualmedia as default_boot_interface. If i move to ipxe as default, i can deploy15:12
* dtantsur needs to get back to the L3 spec15:12
TheJuliaIs uefi support for software raid still a desired thing?15:12
arne_wiebalckyes15:12
TheJuliaarne_wiebalck: using xXraphXx's patches?15:13
arne_wiebalckyes15:13
TheJuliak15:13
arne_wiebalckI just didn't have time yet15:13
dtantsuryolanda: generally, I recommend people NOT to use default_**_interface for vendor stuff specifically for this reasons: there's no sane default for other hardware types15:13
TheJuliaokay, I noticed, at least the ironic side was showing lots of failures, but it looks like it got uploaded when CI was not in great shape.15:13
dtantsurcompletely unrelated, but we missed an announcement that hit my mailbox 5 days ago: The next Project Teams Gathering (PTG) [1] will be in Vancouver, BC from June 8-11.15:14
TheJuliaBifrost dropping py2 and moving to py3 is moving forward. I actually had the agent start on my test VM and everything \o/15:15
TheJuliadtantsur: Thanks for pointing that out and reminding me. Somehow that feels a little worrisome to me as there is yet another format change.15:15
TheJuliaBut I've not raised that yet because life15:15
yolandadtantsur, ack. In that case the specific boot interface always need to be setup at node level, right?15:16
dtantsuryolanda: correct15:16
TheJuliais the zuulv3 migration basically done?15:16
TheJuliaiurygregory: I think you were working on that?15:16
dtantsurgrenade?15:17
TheJuliawell, until it is re-written, we can't do anything I don't think15:17
iurygregoryTheJulia, it's wip15:17
TheJuliak15:17
iurygregorytosky is working on the job for grenade I'm doing tests already15:17
TheJuliaiurygregory: could you add some more detail to the etherpad?15:17
TheJuliaahh, maybe just a date then15:17
iurygregorydoing now =)15:17
TheJuliathanks!15:18
TheJuliaHas anyone discussed the state callback work with nova recently?15:18
arne_wiebalcknot me15:19
TheJuliaWell the statuses look good to me15:19
openstackgerritTzu-Mainn Chen proposed openstack/python-ironicclient master: Add allocation owner  https://review.opendev.org/70272815:21
TheJuliaEveryone ready to move on to priorities for the coming week?15:22
iurygregory++15:22
dtantsur++15:22
rpittaulet's15:22
TheJulia#topic Deciding on priorities for the coming week15:23
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:23
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:23
TheJuliaStarting at line 16615:24
dtantsurI've added the ironicclient part of the ownership work15:24
TheJuliaI think the list looks fairly good, a number of bifrost patches landed last week, thanks for the reviews everyone15:25
TheJuliaIt is super close \o/15:25
rpittauTheJulia: this might also help https://review.opendev.org/70334315:25
patchbotpatch 703343 - bifrost - Define venv package for debian based distro - 1 patch set15:25
dtantsurspeaking of bifrost, I could use reviews of https://review.opendev.org/#/c/703069/ and its backports15:25
patchbotpatch 703069 - bifrost - Use upper-constraints for ironicclient, sushy and DIB - 5 patch sets15:25
TheJuliarpittau: If that does what I suspect, that might also save me a patch today :)15:26
rpittauhope so :)15:26
*** ociuhandu has joined #openstack-ironic15:27
TheJuliadtantsur: Added to the list for bifrost, hopefully we get the CI semi-working for at least bionic before we merge that patch though15:27
TheJuliaDoes anyone have anything else to add?15:28
dtantsurTheJulia: just note that stable bifrost is broken without it15:28
TheJuliadtantsur: yay!15:28
TheJuliagood point15:28
dtantsurand then stein exhibits some weird behavior WITH this patch. sooooo....15:29
TheJuliaugh15:29
TheJuliayay15:29
TheJuliaSpeaking of stein, has anyone looked at the stein sushy-tempest-ironic-partition-redfish-src job? https://review.opendev.org/#/c/701814/15:29
patchbotpatch 701814 - sushy (stable/stein) - SSC.disks_sizes_bytes handle CapacityBytes is None - 1 patch set15:29
rpiosoNot sure if this is on topic, but what's the state of the sushy gate?15:29
TheJuliaIt seems there is unhappiness15:29
rpiosoTheJulia: jinx15:29
TheJuliaMaster branch seems to be good, I rebased one of bdodd_'s patches late last week15:30
rpiosoTheJulia: That's what prompted my question.15:30
TheJuliaLikely someone just needs to look15:30
TheJuliaAnyway, everyone good with the list of priorites and moving on?15:30
TheJuliapriorities15:30
dtantsuretingof: another case of the same sushy sadness? https://storyboard.openstack.org/#!/story/200664115:31
dtantsurfixing this ^^^ should become a priority IMO15:31
rpiosoTheJulia: There's another cherry pick of that change to stable/train. It failed to be verified by the same tests.15:31
TheJuliaIthought that was already fixed...15:31
dtantsurhttps://review.opendev.org/670720 is a part of it15:32
patchbotpatch 670720 - sushy - Handle incomplete messages in MessageRegistry - 1 patch set15:32
TheJuliabut I saw another report of registry. Maybe we tweeked the registry code in review too much to tolerate the reality of BMCs?15:32
dtantsurbut here's probably more15:32
etingofoh, sushy15:32
*** ociuhandu has quit IRC15:32
TheJuliastendulker: Any reason for the WF-1 on the aformentioned patch?15:32
TheJuliaI'm wondering if ops feedback is from last week from johnthetubaguy?15:33
TheJuliamgoddard: ^?15:33
stendulkerTheJulia: It is not required. I had to check with f/w team and got it confirmed that its a firmware bug.15:34
etingofdiverting sushy from official schema can be complicated...15:34
stendulkerTheJulia: I will abandon it.15:34
TheJuliastendulker: but it seems we're running into missing registry items with other vendors :\15:34
stendulkeroh :(15:34
stendulkerShall we take this fix?15:35
dtantsureven with a firmware bug.. we cannot (unfortunately!) expect everyone to always chase the latest firmware..15:35
stendulkerdtantsur: I was testing pre-release bits of firmware...15:35
TheJuliaI think we should move forward on handling if it is not present... because I think missing registry entries is a thing in two other vendors as well, so I think it just makes sense15:35
yolandai was hitting this missingregistry errors, using latest firmware on my boxes last week15:36
dtantsurstendulker: yolanda hit it on something production, I think15:36
stendulkerWill remove workflow-115:36
TheJuliastendulker: thanks15:36
TheJuliaAdded to the sushy list15:37
TheJuliaMoving on!15:37
stendulkerTheJulia: np15:37
TheJulia#topic Baremetal SIG15:37
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:37
TheJulia#info If you have submitted case studies to the foundation, please feel free to add the contents to the whitepaper document.15:38
TheJulia#link https://docs.google.com/document/d/1BmB2JL_oG3lWXId_NXT9KWcBJjqgtnbmixIcNsfGooA/edit15:38
TheJuliaarne_wiebalck: Hopefully I'll be able to supply one for Red Hat... soon. :)15:39
arne_wiebalcknice! thanks15:39
TheJuliaI guess we're good to move to Open Discussion if there is nothing else?15:39
* TheJulia hears crickets15:41
TheJulia#topic Open Discussion15:41
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:41
kaifeng_i have filed a small rfe for discussion: https://storyboard.openstack.org/#!/story/200709915:41
TheJuliaThat seems reasonable15:42
TheJuliaI guess the code could also check if the port is already in-use...15:42
kaifeng_if the story provides enough information, i guess we can skip the spec?15:43
TheJuliaIt seems really small and like it shouldn't need a spec at all15:43
TheJuliamore like a logical improvement to me15:43
kaifeng_yes, we will save the allocated port as a driver internal info15:43
dtantsur"support an auto value for the ipmi_terminal_port" maybe just no value at all?15:45
*** k_mouza has quit IRC15:45
TheJuliadtantsur: I like that idea a lot15:45
dtantsurotherwise looks very reasonable to me15:45
TheJuliathat makes the feature way more usable15:45
kaifeng_it means ipmi_terminal_port can be a string 'auto'15:45
*** k_mouza has joined #openstack-ironic15:45
kaifeng_when user update the node with "auto" value, conductor will allocate a port from the configured range.15:46
TheJuliacould it be better to deprecate ipmi_terminal_port?15:46
dtantsuror at least make it an optional advanced-only setting?15:47
TheJuliaYeah... this feels like a detail that can be sorted in code review, to be honest15:48
TheJuliajust thinking from a usability standpoint... allocating as needed from a range is a way better way to handle these sorts of things15:49
kaifeng_well, i haven't thought about deprecation of ipmi_terminal_port, but it's possible15:49
*** k_mouza has quit IRC15:50
kaifeng_the main advantage of this is to avoid manual management of the console port, so we can avoid port conflicts and better takeover15:51
kaifeng_when it's done, i think we can start removing the ipmi_terminal_port15:52
TheJuliaI concur, I suspect the logic path is always going to need to check if a port is in use and possibly change ports up. The nova client code should be reading the field out of ironic when the port needs to be accessed15:52
dtantsurI'm mostly suggesting that we don't need the 'auto' value because a missing value will serve the same role15:52
rloo++ with dtantsur.15:54
kaifeng_yeah, that's a nice suggestion.15:54
TheJuliaMakes sense15:54
rloo(and will make it easier to deprecate/remove ipmi_terminal_port)15:54
TheJulia++15:55
TheJuliaAnyway, seems like we're done today...15:55
rlooi have been wondering about stashing things in driver_internal_info and whether we're stashing too many things there, but that might be another discussion15:55
kaifeng_i will update the story accordingly :)15:55
TheJuliarloo: I feel like that topic needs an example since it is driver state filed that a user can't edit.15:56
* rloo thinks we need to provide API for user to know which port is being used for console15:56
TheJuliarloo: we expose it as part of the console field data15:56
rloooh,in that case, it is fine.15:56
kaifeng_rloo: we provide console link if enabled15:56
TheJuliaokay15:57
TheJuliaWell, thanks everyone!15:57
TheJuliaHave a wonderful week!15:58
rpittauthanks! you too!15:58
TheJulia#endmeeting15:58
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."15:58
openstackMeeting ended Mon Jan 20 15:58:35 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:58
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-01-20-15.00.html15:58
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-01-20-15.00.txt15:58
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-01-20-15.00.log.html15:58
*** afasano has quit IRC15:58
openstackgerritMerged openstack/ironic master: Allow reading root_device from instance_info  https://review.opendev.org/70104815:59
rpiosodtantsur, etingof: I added a comment to https://storyboard.openstack.org/#!/story/200707115:59
*** afasano has joined #openstack-ironic15:59
*** k_mouza has joined #openstack-ironic15:59
rloodtantsur: wrt https://review.opendev.org/#/c/702927/, we need/want to move cleaning timeouts to cleanfail, even if node is in maintenance. So we're patching it anyway, but that code in base_manager doesn't seem quite right16:00
patchbotpatch 702927 - ironic - Nodes in maintenance didn't fail, when they should... - 1 patch set16:00
dtantsurrloo: you're probably right. we need to check if we need to pass maintenance=False in more places though, so I thought of returning to that patch with a fresh head16:01
rloodtantsur: if we always want to prevent handling of nodes in maintenance, then we might as well not put maintenance field in the filtering16:01
rloodtantsur: ah. ok, i did go through all calls to that method, but good for someone else to do so. hence the addition to the inspecting timeout call.16:01
rloodtantsur: i'll add my comments to that pr then. thx!16:02
*** ociuhandu has joined #openstack-ironic16:03
gudrutis21Hi Ironic 0/ Are the CI issues resolved ? ( trying to figure if Zuul tests fails because of my changes or not :) )16:04
TheJuliagudrutis21: things are generally working for us, what project are you having issues with?16:05
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Fix multiprocessing call in unit test  https://review.opendev.org/70343916:07
gudrutis21My changes to ironic inspector https://review.opendev.org/#/c/702215/ , have 2 failed tests.16:08
patchbotpatch 702215 - ironic-inspector - Added scope to introspection rules. - 3 patch sets16:08
*** kaifeng_ has quit IRC16:12
*** openstackgerrit has quit IRC16:13
*** gregwork has joined #openstack-ironic16:18
*** ociuhandu has quit IRC16:21
*** afasano has quit IRC16:21
*** ociuhandu has joined #openstack-ironic16:22
*** afasano has joined #openstack-ironic16:23
*** stendulker has quit IRC16:24
*** mkrai_ has joined #openstack-ironic16:27
*** iurygregory has quit IRC16:29
TheJuliagudrutis21: so grenade jobs can be a little touchy. If you didn't touch anything that involves upgrades, you may want to just try rechecking, otherwise it looks like the job has been working16:29
gudrutis21TheJulia, can I retriger tests without submitting new patch?16:30
TheJuliagudrutis21: leave a comment of just the word "recheck"16:31
gudrutis21TheJulia: comment as in code comment?16:33
gudrutis21I mean commenting on a specific line.16:33
gudrutis21Newer mind, looks like I figured it out16:42
rpittaugood night! o/16:44
*** rpittau is now known as rpittau|afk16:44
*** bdodd_ has quit IRC16:45
*** bfournie has quit IRC16:50
*** bfournie has joined #openstack-ironic16:52
*** bdodd has joined #openstack-ironic16:52
TheJuliarpittau|afk: goodnight16:53
TheJuliagudrutis21: awesome, sorry, I had to step away16:53
*** ricolin has quit IRC16:55
*** mkrai_ has quit IRC16:55
*** lucasagomes has quit IRC17:00
*** tesseract has quit IRC17:00
TheJuliadtantsur: I'm not sure about your change to bifrost package dependencies. Basically changed pure source install to just be install of the requirements.txt so I'm worried it will not install anything17:01
dtantsurTheJulia: there should be two steps there, no?17:02
TheJuliaanyway,, the job is broken due to the issue that rpittau|afk's patch fixes, so we should be able to recheck it once that patch merges and see how your fix behaves17:02
dtantsurin https://review.opendev.org/#/c/703069/5/playbooks/roles/bifrost-ironic-install/tasks/pip_install.yml line 61 installs the requirements, line 75 - the actual package17:02
patchbotpatch 703069 - bifrost - Use upper-constraints for ironicclient, sushy and DIB - 5 patch sets17:02
TheJuliadtantsur: three possible paths if memory serves, looks like you constrained the third's behavior17:02
TheJuliaoh wow17:03
TheJuliaI must be going blind17:03
TheJuliaokay, I see it17:03
TheJuliabut loading requirements.txt is supposed to be automatic on source installs....17:03
dtantsuryep, it is17:03
dtantsurbut we cannot use constraints on a source install17:03
dtantsurthus I split the two: first I install requirements with constraints17:04
dtantsurthen I install the package itself. requirements are already satisfied at this point17:04
TheJuliaokay, I see what you did there17:04
TheJuliathat makes a little more sense now that I've looked at it again17:04
TheJuliaThat should actually work really well, as long as pip doesn't decide to try and re-pull all the dependencies in17:04
*** openstackgerrit has joined #openstack-ironic17:13
openstackgerritMerged openstack/bifrost master: Define venv package for debian based distro  https://review.opendev.org/70334317:13
*** alexmcleod has quit IRC17:16
dtantsurTheJulia: without -U flag it should be satisfied with what is installed17:23
dtantsurI think our tox usage relies on this property as well17:23
*** ociuhandu_ has joined #openstack-ironic17:26
*** ociuhandu has quit IRC17:29
*** ociuhandu_ has quit IRC17:30
*** quoth has quit IRC17:31
dtantsurfolks, please review https://review.opendev.org/#/c/703400/ it's needed to unblock the IPA CI17:33
patchbotpatch 703400 - ironic-python-agent-builder - Start installing DIB from Zuul-cloned sources in t... - 1 patch set17:33
dtantsurthen https://review.opendev.org/#/c/703371/ is needed for stable branches17:33
patchbotpatch 703371 - ironic-python-agent-builder - Fix and return the CentOS 7 job - 2 patch sets17:33
*** dtantsur is now known as dtantsur|afk17:33
dtantsur|afkg'night17:33
*** k_mouza has quit IRC17:34
etingoffun fact: it takes 580 lines of json to validate network_data.json a bit -- https://review.opendev.org/#/c/703133/3/doc/api_schemas/network_data.json17:34
patchbotpatch 703133 - nova - Add JSON schema for network_data.json - 3 patch sets17:34
*** khansa has quit IRC17:35
openstackgerritJulia Kreger proposed openstack/ironic master: Deprecate ibmc  https://review.opendev.org/70346517:37
openstackgerritJulia Kreger proposed openstack/ironic master: [victoria] Remove ibmc  https://review.opendev.org/70346617:37
*** Goneri has quit IRC17:47
*** afasano has quit IRC17:51
*** bnemec has joined #openstack-ironic17:52
*** afasano has joined #openstack-ironic17:53
TheJuliajungleboyj: any news on xclarity ci? Last you mentioned it was being moved?17:53
openstackgerritJulia Kreger proposed openstack/ironic master: Deprecate irmc hardware type  https://review.opendev.org/70347217:55
jungleboyjTheJulia:  Yes, it has been moved and they are getting it up and running.  They had devstack working.17:56
TheJulia\o/17:56
TheJuliabfournie: Regarding 702293, I suspect there is another problem someplace :(17:56
TheJuliajungleboyj: \o/ for good news17:57
jungleboyjI have asked them to update me before the Chinese New Year.17:57
*** dougsz has quit IRC17:57
jungleboyjI am pushing them to get more outside help.  Not sure why that is something they are resistent to.17:57
TheJuliaCultures can be that way sadly17:58
jungleboyjIf you need to mark us unsupported, I understand.17:58
jungleboyjOtherwise, I am doing my best to keep them moving forward.17:59
jungleboyjWe also have been pushed to get the VMWare flavor of our product out first which hasn't helped.  The RH flavor is coming out in May.17:59
*** derekh has quit IRC18:01
bfournieTheJulia: with metalsmith-integration?18:06
*** bnemec has quit IRC18:08
TheJuliabfournie: yeah18:09
TheJuliajungleboyj: you guys are at least responding which means we have a point of contact18:09
*** andrein has quit IRC18:11
jungleboyjOk. Thank you. I will chat within them and get an update for this week18:11
*** andrein has joined #openstack-ironic18:11
TheJuliajungleboyj: I mean, you can also use the fact I just posted two deprecation warnings as... strong encouragement.18:12
*** k_mouza has joined #openstack-ironic18:12
TheJuliaAnd I'm happy to send a grumpy sounding email regarding third party CI18:12
TheJuliaBut... yeah.18:12
*** k_mouza has quit IRC18:16
jungleboyjOk. Sounds good. I can zenn18:28
jungleboyjSend thr grumpy email. I will cc you.18:28
TheJuliasent? or do you want me to send?18:30
*** hamzy__ has joined #openstack-ironic18:55
*** hamzy_ has quit IRC18:57
*** goldyfruit has quit IRC19:06
jungleboyjI can send.19:07
*** TxGirlGeek has joined #openstack-ironic19:10
jungleboyjMay be tomorrow.  I am on vacation today and trying to help my son ready for Finals.19:13
*** igordc has joined #openstack-ironic19:23
openstackgerritMerged openstack/ironic-python-agent-builder master: Start installing DIB from Zuul-cloned sources in the CI  https://review.opendev.org/70340019:24
*** stevebaker has joined #openstack-ironic19:26
stevebakergood morning19:27
TheJuliagood morning stevebaker19:34
TheJuliajungleboyj: no worries! enjoy pto19:34
*** goldyfruit has joined #openstack-ironic19:35
*** goldyfruit has quit IRC19:39
*** goldyfruit has joined #openstack-ironic19:40
TheJulialaying down because I'm starting to get a sinus infection and feel like I've been run over19:43
*** mbeierl has joined #openstack-ironic20:05
*** mbeierl1 has joined #openstack-ironic20:08
*** mbeierl has quit IRC20:09
*** mbeierl1 is now known as mbeierl20:09
*** Lucas_Gray has joined #openstack-ironic20:11
*** Lucas_Gray has quit IRC20:14
*** Lucas_Gray has joined #openstack-ironic20:16
TheJuliaNobodyCam: can you take a quick look at https://review.opendev.org/#/c/701590/ and https://review.opendev.org/#/c/701846 ?20:34
patchbotpatch 701590 - bifrost - Python3/Ansible conditional quarks - 9 patch sets20:34
patchbotpatch 701846 - bifrost - Hopefully final venv fixes - 6 patch sets20:34
*** goldyfruit has quit IRC20:40
*** goldyfruit has joined #openstack-ironic20:40
NobodyCamSure. Need a few minutes to finish up current task20:40
TheJuliano worries, thanks!20:44
*** afasano has quit IRC20:46
*** afasano has joined #openstack-ironic20:46
*** dsneddon has joined #openstack-ironic20:47
*** dsneddon has quit IRC20:54
openstackgerritMerged openstack/bifrost master: Use upper-constraints for ironicclient, sushy and DIB  https://review.opendev.org/70306920:55
*** Wryhder has joined #openstack-ironic21:11
*** Lucas_Gray has quit IRC21:11
*** Wryhder is now known as Lucas_Gray21:12
openstackgerritJulia Kreger proposed openstack/bifrost master: Python3/Ansible conditional quarks  https://review.opendev.org/70159021:18
openstackgerritJulia Kreger proposed openstack/bifrost master: Hopefully final venv fixes  https://review.opendev.org/70184621:18
openstackgerritJulia Kreger proposed openstack/bifrost master: Remove CI credential setup  https://review.opendev.org/70210721:18
openstackgerritJulia Kreger proposed openstack/bifrost master: Sync pip_install files  https://review.opendev.org/70350221:18
openstackgerritJulia Kreger proposed openstack/bifrost master: Add a listen-address to the dnsmasq bind  https://review.opendev.org/70331421:18
TheJuliaof course some rebasing required21:18
openstackgerritJulia Kreger proposed openstack/bifrost master: Enable Fasttrack  https://review.opendev.org/70350621:28
*** Wryhder has joined #openstack-ironic21:38
*** Lucas_Gray has quit IRC21:39
*** Wryhder is now known as Lucas_Gray21:39
openstackgerritJulia Kreger proposed openstack/bifrost master: Add a listen-address to the dnsmasq bind  https://review.opendev.org/70331421:45
openstackgerritJulia Kreger proposed openstack/bifrost master: Enable Fasttrack  https://review.opendev.org/70350621:46
*** rcernin has joined #openstack-ironic21:49
*** dsneddon has joined #openstack-ironic21:50
*** patchbot has quit IRC22:07
*** patchbot has joined #openstack-ironic22:13
*** Lucas_Gray has quit IRC22:20
*** Lucas_Gray has joined #openstack-ironic22:22
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Agent token support  https://review.opendev.org/69261422:35
openstackgerritJulia Kreger proposed openstack/ironic master: [victoria] Remove ibmc  https://review.opendev.org/70346622:40
openstackgerritJulia Kreger proposed openstack/ironic master: Collect additional post-run information  https://review.opendev.org/67858022:44
openstackgerritJulia Kreger proposed openstack/ironic master: Pre-shared agent token  https://review.opendev.org/69260922:46
openstackgerritJulia Kreger proposed openstack/ironic master: Enable agent_token for virtual media boot  https://review.opendev.org/69758422:46
openstackgerritJulia Kreger proposed openstack/ironic master: Send our token back to the agent  https://review.opendev.org/69758522:46
openstackgerritJulia Kreger proposed openstack/ironic master: Signal agent token is required  https://review.opendev.org/69930422:47
openstackgerritJulia Kreger proposed openstack/bifrost master: Add a listen-address to the dnsmasq bind  https://review.opendev.org/70331422:51
openstackgerritJulia Kreger proposed openstack/bifrost master: Enable Fasttrack  https://review.opendev.org/70350622:51
*** jtomasek has quit IRC22:53
* TheJulia waits for ci results of joy or sadness22:56
*** TxGirlGeek has quit IRC22:59
openstackgerritMerged openstack/ironic master: Fix incorrect ibmc_address parsing on Python 3.8  https://review.opendev.org/70341123:06
*** TxGirlGeek has joined #openstack-ironic23:27
*** dsneddon has quit IRC23:30
*** TxGirlGeek has quit IRC23:32
openstackgerritJulia Kreger proposed openstack/bifrost master: Try shorthand format for console  https://review.opendev.org/70352223:44
*** rloo has quit IRC23:47
*** dsneddon has joined #openstack-ironic23:58
« Sunday, 2020-01-19 Index Tuesday, 2020-01-21 »

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