Friday, 2021-09-24

opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Yoga: Change default boot mode to uefi  https://review.opendev.org/c/openstack/ironic/+/81024900:10
opendevreviewMerged openstack/ironic master: Clean up caches periodically  https://review.opendev.org/c/openstack/ironic/+/81042800:21
iurygregorygood morning Ironic, happy friday!05:49
jandersgood morning iurygregory o/06:23
jandersHappy Friday! :)06:23
iurygregoryhey janders o/06:23
opendevreviewIury Gregory Melo Ferreira proposed openstack/bifrost stable/xena: Change git_branch in stable/xena  https://review.opendev.org/c/openstack/bifrost/+/81081706:40
*** rpittau|afk is now known as rpittau06:41
rpittaugood morning ironic! o/06:41
iurygregorygood morning rpittau o/06:41
rpittauhey iurygregory :)06:42
jandershey rpittau o/06:52
jandersHappy Friday06:52
arne_wiebalckGood morning iurygregory rpittau janders and Ironic!06:52
rpittauhey janders :)06:53
iurygregoryhey arne_wiebalck o/06:53
rpittauhey arne_wiebalck :)06:53
dtantsurmorning ironic07:03
dtantsurcould I get a 2nd +2 on https://review.opendev.org/810703 please? I'm trying to fix all corner cases of sushy-tools loading again..07:06
iurygregorymorning dtantsur 07:08
iurygregorylooking now07:08
opendevreviewIury Gregory Melo Ferreira proposed openstack/bifrost stable/xena: Change git_branch in stable/xena  https://review.opendev.org/c/openstack/bifrost/+/81081707:12
opendevreviewDmitry Tantsur proposed openstack/ironic master: PXE: avoid trailing dots in exception messages  https://review.opendev.org/c/openstack/ironic/+/81082807:32
jandershey dtantsur o/07:41
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic stable/xena: Set IPA download branch in Xena  https://review.opendev.org/c/openstack/ironic/+/81083507:59
opendevreviewMerged openstack/sushy-tools master: Fix passing configuration via WSGI  https://review.opendev.org/c/openstack/sushy-tools/+/81070308:06
dtantsuriurygregory: have we created an IPA-builder stable/xena? I don't see it in reno.08:06
iurygregorydtantsur, we created stable/xena for sure08:07
iurygregoryhttps://github.com/openstack/ironic-python-agent-builder/commit/bffece3c14b71066fda074932c087487a896108e08:07
dtantsurthen I'm confused by https://docs.openstack.org/releasenotes/ironic-python-agent-builder/08:07
iurygregoryO.o08:08
iurygregorynow I'm also confused :D08:08
iurygregorydtantsur, I think I've found the problem08:13
iurygregoryok maybe not...08:15
iurygregoryironic has the same config...08:15
rpittauseems like a rendering problem of the page ?08:15
dtantsuror maybe failing post jobs08:16
rpittaumaybe, looking at the console the source looks older08:20
iurygregoryI think it's failing post jobs, locally running tox -e releasenotes I can see xena.html08:20
iurygregoryhumm just wondering did we release tenks? :D 08:34
* dtantsur has no idea about tenks08:35
iurygregorysince is independent we don't need to worry I think08:35
dtantsurthen yes08:36
iurygregorydtantsur, fyi https://github.com/openstack/tenks :D08:36
iurygregoryI think is something mgoddard uses (not 100% sure)08:36
dtantsurwell, yeah, I'm aware that it exists, but that's it08:36
iurygregorywe consider ipa-builder a tool and I'm updating our docs about release so I'm thinking in adding ipa-builder in a separate paragraph under Normal release https://paste.opendev.org/show/809553/ 08:40
iurygregorymaybe we should also add a section Manual Release and include ironic-staging-drivers with the link https://ironic-staging-drivers.readthedocs.io/en/latest/releasing.html ?08:46
dtantsur++08:47
opendevreviewRiccardo Pittau proposed openstack/bifrost master: Support debian 11 "bullseye"  https://review.opendev.org/c/openstack/bifrost/+/80632808:52
iurygregorydtantsur, funny thing after train we never pinned the TEMPEST_BAREMETAL_MAX_MICROVERSION in devstack/lib/ironic 09:28
iurygregorybut in some stable branches we have a pin for the ironic-tempest-plugin version09:29
iurygregoryI'm wondering if tempest is smart to figure out the right micro version or not 09:30
opendevreviewMerged openstack/ironic stable/wallaby: Fix idrac-wsman having Completed with Errors jobs  https://review.opendev.org/c/openstack/ironic/+/81048709:32
opendevreviewMerged openstack/ironic stable/xena: Update iDRAC doc for idrac-redfish RAID  https://review.opendev.org/c/openstack/ironic/+/81048509:32
opendevreviewMerged openstack/ironic stable/wallaby: Update iDRAC doc for idrac-redfish RAID  https://review.opendev.org/c/openstack/ironic/+/81048609:32
dtantsuriurygregory: if plugin is pinned, it won't have newer tests10:30
iurygregoryyeah I was thinking about this =) (we normally pin when we see the failures...)10:31
mnasiadkadtantsur: so, that instance_info property works - but what in case when my rootfs is on an LV? IPA complains TYPE does not match. My image has ESP partition (/boot/efi) and LVM PV (and then LV with / filesystem).12:55
mnasiadkaWhen I tried to pass LVM PVs UUID - then IPA was trying to mount it...12:58
mnasiadkaor maybe it's a different issue now13:06
dtantsurI don't remember how it works well. arne_wiebalck?13:13
mnasiadkahmm, for some reason it seems I have md0 and md127 at the same time (in IPA) - could that mean I need to disable some fake-raid in server's BIOS?13:18
arne_wiebalckmnasiadka: the code will try to mount the root fs and will look for it by UUID ... do you see a "Finding the partition ... failed" error?13:20
arne_wiebalckmnasiadka: also, I see your image has an ESP, so this is UEFI boot mode?13:21
mnasiadkaarne_wiebalck: I don't think I'm setting any boot mode, should I?13:23
arne_wiebalckmnasiadka: no, but if the node boots in BIOS mode the code path is different from when it boots in UEFI mode13:24
mnasiadkadriver_internal_info states deploy_boot_mode: uefi13:24
arne_wiebalckok13:24
mnasiadkait's HP Gen10+ using ilo5 driver13:24
TheJuliagodo morning13:27
arne_wiebalckTheJulia: o/13:36
arne_wiebalckmnasiadka: https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/extensions/image.py#L90 may be the issue as it only allows TYPEs part and md when finding the root device.13:37
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic stable/xena: Set IPA download branch and MAX_MICROVERSION  https://review.opendev.org/c/openstack/ironic/+/81083513:37
iurygregorydtantsur, I've update stable/xena with the right ipa version and the max api version to be used in tempest, one thing I didn't understand from our docs is "update links in the documentation (ironic/doc/source/) to point to the branched versions of any openstack projects’ (that branch) documents. As of Pike release, the only outlier is diskimage-builder."13:38
iurygregorygood morning TheJulia =)13:38
arne_wiebalckmnasiadka: having more debugging and/or extending that list by 'lvm' (if that is the TYPE may be worth a try)13:39
arne_wiebalck*(if that is the TYPE) may be worth a try13:39
mnasiadkaarne_wiebalck: let me try - will surely come back ;)13:39
arne_wiebalckmnasiadka: yeah, let us know!13:40
arne_wiebalckmnasiadka: there should be a "No partition with UUID ... found" message which I do not see in your logs ... more debug logs are required I think.13:46
mnasiadkawill do, but now I need to fetch my daughter from school ;)13:47
arne_wiebalcksure, first things first :)13:47
TheJuliabraaaainnns13:53
dtantsurmorning TheJulia 14:06
dtantsuriurygregory: I think we're slowly migrating to use special cross-document links instead of updating them manually14:06
dtantsuror rather: I don't think we've ever updated them manually14:06
iurygregorygot it, should we just remove this? =)14:07
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Update release doc  https://review.opendev.org/c/openstack/ironic/+/81091814:09
dtantsurprobably14:11
dtantsurmaybe we it could be an outreachy selection task to migrate the remaining links to openstack cross-project links14:12
iurygregory++14:13
TheJulia++14:27
TheJuliaiurygregory: https://github.com/OpenStackweb/ironic-website/pull/4316:01
* iurygregory looks16:02
TheJuliasdanni: did you come up with any more questions w/r/t RBAC?16:03
iurygregoryTheJulia, I've added just a few nits I noticed16:11
TheJuliaok16:15
arne_wiebalckbye everyone, have a good weekend o/16:16
iurygregorybye arne_wiebalck o/16:17
* TheJulia hopes it works now and the date is now fixed16:22
sdanniTheJulia: I think I've figured out how to use a system scoped admin. I donI'm now testing if other services like Neutron, Cinder are compatible with system scoped admin16:26
sdanni*i don't have any questions so far. I'm now testing if other services like Neutron, Cinder are compatible with system scoped admin16:27
TheJuliasdanni: cinder is still implementing, AIUI16:33
TheJuliaiurygregory: why removing the sem-ver note in https://review.opendev.org/c/openstack/ironic/+/810918/1/doc/source/contributor/releasing.rst ?16:36
iurygregoryTheJulia, https://meetings.opendev.org/irclogs/%23openstack-ironic/%23openstack-ironic.2021-09-23.log.html#t2021-09-23T09:40:1316:37
iurygregoryit was based on what dtantsur told me yesterday =)16:38
dtantsurthe automatic patches already do it16:38
TheJuliaoh, okay16:38
TheJuliait *can* be really required in some cases, but I guess those cases are now more rare16:39
iurygregoryso maybe we keep and we add a note?16:39
TheJuliamaybe *shrugs*16:40
TheJuliaoh gerrit16:43
TheJuliawhy16:43
opendevreviewJulia Kreger proposed openstack/ironic master: Remove legacy rpm install list and use bindep  https://review.opendev.org/c/openstack/ironic/+/81031316:44
opendevreviewJulia Kreger proposed openstack/ironic master: DNM/WIP - Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773916:45
NobodyCamGood Morning and OFC...16:45
NobodyCamTGIF!16:45
iurygregorymorning NobodyCam o/16:45
NobodyCamhey hey iurygregory 16:45
NobodyCamready for the weekend16:46
TheJuliaHappy Friday NobodyCam!16:46
NobodyCamhappy Friday TheJulia 16:46
iurygregoryyes \o/ (already enjoying a nice beer :D )16:46
NobodyCamsweet !16:47
dtantsurnice :)16:50
dtantsurokay, I think I'll go now, have a great weekend16:50
NobodyCamhave a great weekend dtantsur 16:51
iurygregorybye dtantsur o/ have a great weekend16:51
NobodyCamdid not DIB have a ntp element... where did that go17:37
TheJuliaNobodyCam: for the deployed workload... I don't ever really remember one tbh19:02
TheJuliaNobodyCam: fwiw, agent since starting at some point can be told to sync the clock19:03
NobodyCamYea I couldn't find a ntp or chrony ele19:03
NobodyCamelement... so I wrote one19:03
* TheJulia feels like she has reached hurtybrain state with bios->uefi19:06
NobodyCamRum helps19:07
TheJuliaeh, I was thinking more along the lines of an afternoon nap, tbh19:37
TheJuliait is like... raining here.19:44
TheJuliaalmost .4"20:34
TheJuliaof rain so far20:35
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Yoga: Change default boot mode to uefi  https://review.opendev.org/c/openstack/ironic/+/81024920:47
TheJuliaNobodyCam: .58" on the rain guage console, wunderground says we're like 2x that20:51
NobodyCamyea, Been watching the drops hit the window... :'( 20:52
TheJuliaon a plus side, it is like 25% of our rain for the year20:55
NobodyCamhahahaha so true!20:55

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