Wednesday, 2021-05-26

jandersgood morning Ironic o/00:30
TheJuliao/01:09
openstackgerritSteve Baker proposed openstack/ironic-python-agent master: Make _get_efi_bootloaders return relative paths  https://review.opendev.org/c/openstack/ironic-python-agent/+/79308201:14
openstackgerritSteve Baker proposed openstack/ironic-python-agent master: [WIP] Don't grub2-install overwrite EFI bootloaders  https://review.opendev.org/c/openstack/ironic-python-agent/+/79308301:14
*** paras333 has joined #openstack-ironic01:27
*** paras333 has quit IRC01:31
*** gyee has quit IRC02:05
*** stevebaker has quit IRC02:45
*** rloo has quit IRC02:57
*** stevebaker has joined #openstack-ironic03:07
*** jawad_axd has joined #openstack-ironic03:22
*** jawad_axd has quit IRC03:27
*** paras333 has joined #openstack-ironic03:38
*** paras333 has quit IRC03:42
*** ociuhandu has joined #openstack-ironic04:31
*** ociuhandu has quit IRC04:36
*** mkrai has joined #openstack-ironic04:37
*** ccstone has quit IRC06:09
*** eandersson has quit IRC06:09
*** ccstone has joined #openstack-ironic06:10
*** eandersson has joined #openstack-ironic06:10
*** tzumainn has quit IRC06:12
iurygregorygood morning Ironic06:21
stevebakeriurygregory: good evening!06:22
iurygregoryhey stevebaker o/06:22
iurygregorystevebaker, if you have some time tomorrow can you take a look at https://review.opendev.org/c/openstack/releases/+/79074806:23
stevebakeriurygregory: sure thing06:23
iurygregorytks!06:23
jandershey iurygregory06:24
stevebakeriurygregory: how will that impact downstream product which consumes stable/train backports? I still backport to train on the regular06:25
iurygregoryhey janders o/06:25
iurygregorystevebaker, if I recall correctly EM we can still merge things06:26
iurygregoryExtended Maintenance06:26
iurygregory06:26
iurygregoryWhile there are community members maintaining it.06:26
iurygregory06:26
iurygregoryAll bugfixes (that meet the criteria described below) are appropriate. No Releases produced, reduced CI commitment.06:26
stevebakerok, no releases, reduced CI. That sounds fine to me06:27
iurygregoryyeah, not sure if it would impact the releases part06:27
iurygregorydon't you need rdo packages matching upstream releases etc?06:28
stevebakerit wouldn't I think. Downstream take changes from git when they sync06:28
iurygregoryhummm gotcha06:28
stevebakerstable/train just needs to exist as a branch, and reviews to merge06:28
iurygregorystevebaker, cool!06:29
iurygregorytime for more coffee :D06:36
arne_wiebalckGood morning, stevebaker, iurygregory, and Ironic!06:41
*** ankkumar_ has joined #openstack-ironic06:45
*** ankkumar_ has quit IRC06:46
cennGood morning arne_wielbalck, stevenbaker, iurygregory, Ironic :)06:53
*** jawad_axd has joined #openstack-ironic06:56
iurygregoryhey arne_wiebalck o/06:58
iurygregoryhey cenn o/06:58
*** pmannidi has quit IRC07:07
*** pmannidi has joined #openstack-ironic07:20
*** pmannidi has quit IRC07:20
*** tosky has joined #openstack-ironic07:35
*** k_mouza has joined #openstack-ironic07:38
*** rpittau|afk is now known as rpittau07:41
rpittaugood morning ironic! o/07:41
*** k_mouza has quit IRC07:43
cennmorning rpittau07:47
rpittauhey cenn :)07:50
*** sshnaidm is now known as sshnaidm|afk07:58
*** lucasagomes has joined #openstack-ironic08:03
openstackgerritRiccardo Pittau proposed openstack/ironic stable/ussuri: Make test-setup.sh compatible with mysql8  https://review.opendev.org/c/openstack/ironic/+/79304308:03
iurygregoryhey rpittau o/08:04
rpittauhey iurygregory :)08:04
*** mkrai has quit IRC08:12
*** ociuhandu has joined #openstack-ironic08:13
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-lib master: Add initial privsep structure  https://review.opendev.org/c/openstack/ironic-lib/+/74553608:27
*** anuradha1904 has joined #openstack-ironic08:32
*** dtantsur|afk is now known as dtantsur08:56
dtantsurmorning ironic08:56
*** lourot has quit IRC09:01
openstackgerritVerification of a change to openstack/bifrost failed: Install DIB dependencies in bifrost-ironic-install role  https://review.opendev.org/c/openstack/bifrost/+/75600509:01
*** lourot has joined #openstack-ironic09:02
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic stable/victoria: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/78650709:02
cenngood motning dtantsur09:03
cenn*morning09:03
iurygregorymorning dtantsur09:09
*** tzumainn has joined #openstack-ironic09:29
*** ociuhandu has quit IRC09:45
*** logan- has quit IRC09:46
*** logan- has joined #openstack-ironic09:56
*** ociuhandu has joined #openstack-ironic09:59
*** k_mouza has joined #openstack-ironic09:59
*** ociuhandu has quit IRC10:04
*** ociuhandu has joined #openstack-ironic10:16
*** ociuhandu has quit IRC10:20
*** ociuhandu has joined #openstack-ironic10:24
*** sshnaidm|afk is now known as sshnaidm10:25
*** ociuhandu has quit IRC10:27
*** ociuhandu has joined #openstack-ironic10:27
*** mgoddard has quit IRC10:27
openstackgerritIury Gregory Melo Ferreira proposed openstack/sushy master: EventService support  https://review.opendev.org/c/openstack/sushy/+/79205310:37
*** ociuhandu has quit IRC10:37
*** mkrai has joined #openstack-ironic10:38
*** ociuhandu has joined #openstack-ironic10:50
*** mkrai_ has joined #openstack-ironic10:52
*** mkrai has quit IRC10:53
*** k_mouza has quit IRC10:57
*** k_mouza has joined #openstack-ironic11:13
*** ociuhandu has quit IRC11:18
*** k_mouza has quit IRC11:19
*** k_mouza has joined #openstack-ironic11:19
dtantsurarne_wiebalck: (inspired by a question of k8s slack) would it make sense to somehow (SMART?) verify disks before building RAID on them (or even deploying)?11:29
*** Perfidious has quit IRC11:32
*** paras333 has joined #openstack-ironic11:33
*** ociuhandu has joined #openstack-ironic11:36
*** ociuhandu has quit IRC11:38
*** ociuhandu has joined #openstack-ironic11:38
*** ociuhandu has quit IRC11:38
*** ociuhandu has joined #openstack-ironic11:44
*** ociuhandu has quit IRC11:44
*** Qianbiao has joined #openstack-ironic11:48
arne_wiebalckdtantsur: How about having a such a "health check" during cleaning instead? I would think that nodes should be prevented to move to available when they are broken.11:56
*** ociuhandu has joined #openstack-ironic11:59
dtantsurI guess both make sense if it can be done quickly11:59
arne_wiebalckdtantsur: Yes, it needs to be quick in any case.12:02
arne_wiebalckdtantsur: Not sure there is much to gain to check the disks before RAID or deploy, though: this would fail cleaning (or deploy), which would also flagged by the RAID commands (and I usually trust md as the layer to tell me when a disk is broken).12:04
arne_wiebalckdtantsur: For instance, I don't think we do regular smart test, but wait for an IO to fail, have md handle it (or not), and then take action.12:05
*** ociuhandu has quit IRC12:05
arne_wiebalcks/test/tests/12:05
*** ociuhandu has joined #openstack-ironic12:07
*** mkrai_ has quit IRC12:12
dtantsurI see. So it's more for non-RAID case?12:12
*** ociuhandu has quit IRC12:13
*** ociuhandu has joined #openstack-ironic12:15
*** ociuhandu has quit IRC12:22
*** ociuhandu has joined #openstack-ironic12:24
arne_wiebalckIt would not really hurt in either case, I guess. I would still advocate to make this part of (auto-)cleaning, not move it forward if something odd is detected, and trust that a node does not break  between the time it was cleaned (and checked) and deployment. Having this in cleaning would also check nodes upon instance deletion, so they are taken out of the pool and not advertised to placement.12:28
*** ociuhandu has quit IRC12:28
*** mkrai_ has joined #openstack-ironic12:31
dtantsuryep12:32
dtantsurdo you have an idea which commands we could run?12:32
arne_wiebalckdtantsur: let me check with my hardware colleagues ...12:33
dtantsurI guess the problem with cleaning is that we don't know which device will be root12:35
dtantsurI'm not sure we want to abort in case of any disks misbehaving12:36
arne_wiebalckone could also argue that for a node with N disks and a check enabled cleaning should fail if any of the disks fail (as the user expects N working disks)12:40
dtantsuryep, but I'd be worried about enabling it by default12:40
dtantsurwhile for the root disk I'd like to have some sanity check12:40
dtantsurto make sure we're not deploying on something broken12:40
arne_wiebalckif it is broken broken, deployment may fail anyway12:42
*** ociuhandu has joined #openstack-ironic12:42
arne_wiebalckthe root disk may be picked after cleaning? e.g. since hints are set only after?12:43
dtantsuryep, and usually is12:43
arne_wiebalckok12:44
*** ociuhandu has quit IRC12:44
*** ociuhandu has joined #openstack-ironic12:45
arne_wiebalckIs the "root disk is fine, other disks may be broken" really a realistic use case? Wouldn't you expect the hardware you get is ok?12:46
arne_wiebalckOur users would give the node back, I think :)12:46
* arne_wiebalck sent the check command question off to the hardware colleagues12:47
dtantsuryeah, maybe it's not a viable use case, so we're fine as long as we have an "off" switch12:50
*** coreycb has quit IRC12:50
openstackgerritVerification of a change to openstack/ironic failed: Delay rendering configdrive  https://review.opendev.org/c/openstack/ironic/+/79047213:02
iurygregoryI'm slow =(13:03
iurygregoryops (gerrit is slow =P )13:04
TheJuliagood morning13:04
*** ociuhandu has quit IRC13:04
iurygregorygood morning TheJulia13:04
arne_wiebalckdtantsur: I got some feedback from the hardware experts13:04
dtantsur\o/13:05
dtantsurmorning TheJulia13:05
arne_wiebalckdtantsur: heh, wait :-D13:05
*** coreycb has joined #openstack-ironic13:05
arne_wiebalckdtantsur: it is not obvious to say when a disk is really healthy, esp. not within a short time13:05
arne_wiebalckdtantsur: smartctl -H /dev/xyz is an option13:06
arne_wiebalckdtantsur: ideally, though, we make this configurable so that deployments can adapt this to the conditions with their vendors (which is a good suggestion I think)13:06
dtantsurarne_wiebalck: clean steps can be overridden13:07
arne_wiebalckdtantsur: like, there are individual conditions under which a disk can be replaced13:07
arne_wiebalckdtantsur: true, but making things configurable increases accessibility for operators I would think13:08
dtantsurhardware managers are a sort of configuration13:08
dtantsuryou anyway need to build a custom ramdisk for custom tools13:08
*** ociuhandu has joined #openstack-ironic13:09
arne_wiebalckdtantsur: I was more thinking about the -A option to smartctl rather than custom tools13:09
arne_wiebalckdtantsur: sure, if you want to run your downstream tool, I agree with you13:10
arne_wiebalckdtantsur: if an operator wants different thresholds for certain smart parameters he has in the contract, that is maybe different13:10
dtantsurmm, makes sense13:10
*** coreycb has quit IRC13:10
dtantsurarne_wiebalck: could you draft an RFE for that (not necessary implement)?13:10
arne_wiebalckdtantsur: have such a check as a step or before deployment?13:11
arne_wiebalckdtantsur: in any case: yes :)13:12
dtantsurboth?13:12
dtantsurclean step - for all disks, before deployment - for root13:12
TheJuliaokay, coffee()13:12
dtantsurjust to be really-really sure it's not going to die in 6 hours13:12
arne_wiebalckdtantsur: sounds good, yes13:12
arne_wiebalckdtantsur: I will draft sth13:13
TheJuliaHey, if were going to start evaluating smart data, can we please also check the thermal tempature and refuse to proceed if it is above something sane?13:13
arne_wiebalckheh13:13
iurygregorywow13:13
dtantsurTheJulia: will it be high on a freshly booted ramdisk?13:13
TheJuliadtantsur: shouldn't be at all, but say if it is above 120-140F, the datacenter has a major issue and we shouldn't try to put workloads into that sort of environment13:14
dtantsurfair13:14
arne_wiebalckTheJulia: I don't think the disk check should raise this.13:14
TheJuliaI've literally walked into datacenters which were on the verge of thermal tripping at the transformers13:14
arne_wiebalckor: if you rely on the disk check for this, there is maybe another issue :)13:14
TheJuliabut if a disk sees insane tempatures, there is a major cooling issue13:14
arne_wiebalckTheJulia: not saying, temperature is no issue, I have been hit by this several times13:15
arne_wiebalckTheJulia: I am saying this should not be wired into a disk health check13:15
arne_wiebalckTheJulia: there should be monitoring, or other health checks13:16
TheJuliaI kind of disagree, but not at -1 level. Just I've charted failure rates against my data center tempatures, but that was also way in the past13:16
TheJuliaThis was also many years ago13:16
*** k_mouza_ has joined #openstack-ironic13:16
arne_wiebalckTheJulia: the MB knows the temperature, it can be read from the BMC, ... we could have other checks13:16
TheJuliatrue13:17
TheJuliabut some mfgrs put drives in really odd places too13:17
TheJulialike on exhaust air13:17
arne_wiebalcktrue ... but then we would also need to decide what a sane temperature is for the disk at hand13:18
TheJuliaindeed13:18
dtantsursounds like another optional clean step?13:18
arne_wiebalckdtantsur: ++13:18
*** k_mouza has quit IRC13:19
arne_wiebalckthinks the cleaning framework is evolving into an "eierlegende Wollmilchsau" ... (this is for dtantsur, everyone else has to google I guess :))13:21
iurygregory=(13:21
iurygregorynot fair13:21
*** ociuhandu has quit IRC13:21
* dtantsur rofl13:21
iurygregoryarne_wiebalck, LMAO13:22
arne_wiebalck:-D13:22
dtantsurarne_wiebalck: well, it has long ago. definitely after we switched RAID to it.13:22
iurygregoryomg13:22
iurygregorymy reaction is WTH13:22
*** ociuhandu has joined #openstack-ironic13:22
arne_wiebalckdtantsur: just today we came up with a new use case for it13:22
dtantsur\o/13:22
arne_wiebalckdtantsur: when nodes are retired, they may stay around for a while13:22
arne_wiebalckdtantsur: with power, network ... etc13:22
arne_wiebalckdtantsur: we try do some voluntary computing on them while they wait for removal13:23
TheJuliaarne_wiebalck wins the funniest reference award for the day13:23
arne_wiebalckdtantsur: since the "retired" flag is set, we cannot instantiate them (easily)13:23
dtantsurarne_wiebalck: it sounds like you need a separate resource class rather than retired13:23
TheJuliadtantsur: ++13:23
arne_wiebalckdtantsur: so we make this a clean step to run boinc13:24
* dtantsur looks at arne_wiebalck attaching a saddle to the Wollmilchsau13:24
arne_wiebalckdtantsur: yep, I guess this would also work13:24
arne_wiebalckdtantsur: LOL13:24
*** ociuhandu has quit IRC13:26
*** rloo has joined #openstack-ironic13:26
openstackgerritVerification of a change to openstack/ironic failed: Delay rendering configdrive  https://review.opendev.org/c/openstack/ironic/+/79047213:26
TheJuliadtantsur: a saddle for pixie boots to ride on!13:28
dtantsur\o/13:30
* arne_wiebalck is looking forward to the new Ironic t-shirts13:33
iurygregorywoot13:34
iurygregorynew t-shirts \o/13:34
dtantsur:D13:35
TheJuliafirst, we need an artist!13:35
openstackgerritMateusz Kowalski proposed openstack/sushy-tools master: Use /dev/sdx instead of /dev/sdc when mounting boot images  https://review.opendev.org/c/openstack/sushy-tools/+/79315813:45
*** ociuhandu has joined #openstack-ironic13:49
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/wallaby: Raise qemu-img memory limit to 2 GiB  https://review.opendev.org/c/openstack/ironic-lib/+/79305113:51
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/victoria: Raise qemu-img memory limit to 2 GiB  https://review.opendev.org/c/openstack/ironic-lib/+/79305213:52
*** ociuhandu has quit IRC13:59
*** ociuhandu has joined #openstack-ironic13:59
TheJulia*sigh*14:01
* dtantsur agrees with sigh even if he's not sure what the sigh is about14:01
*** SpamapS has quit IRC14:01
cenn:p14:04
TheJuliadtantsur: soo many things14:07
*** tkajinam has quit IRC14:12
*** tkajinam has joined #openstack-ironic14:12
dtantsur:(14:19
*** ociuhandu has quit IRC14:24
*** ociuhandu has joined #openstack-ironic14:25
*** ociuhandu has quit IRC14:28
*** ociuhandu has joined #openstack-ironic14:29
*** sdanni has joined #openstack-ironic14:30
*** JayF has quit IRC14:30
*** lmcgann has joined #openstack-ironic14:31
openstackgerritJulia Kreger proposed openstack/bifrost master: Add a benchmark job for ironic  https://review.opendev.org/c/openstack/bifrost/+/79270514:35
TheJuliaany chance I could get another review on https://review.opendev.org/c/openstack/ironic/+/78833514:39
TheJuliao/ jlvillal14:42
jlvillalo/ TheJulia :)14:43
jlvillalI hope all is going well with you (besides the IRC service drama) :)14:43
TheJuliaomg drama14:46
* TheJulia seeks anti-drama today14:46
*** mkrai_ has quit IRC14:47
openstackgerritMerged openstack/ironic master: Add basic tools for benchmarking  https://review.opendev.org/c/openstack/ironic/+/78833515:10
iurygregoryif any core has some time to look at a sushy change I would appreciate =) https://review.opendev.org/c/openstack/sushy/+/79205315:15
dtantsurhey-hey jlvillal, how are you?15:15
jlvillalПривет dtantsur!  Things are going well :)  Doing part-time work and in spare-time contributing to some opensource projects (python-gitlab for the most part at the moment).15:17
jlvillalHow about you?15:17
jlvillalOh and also working on Beaker stuff, which you may have heard of ;)15:17
dtantsurjlvillal: not bad, not bad. waiting for restrictions to be over, working on http://metal3.io/15:17
dtantsurreplacing it with ironic, I hope? :-P15:18
jlvillalhehe15:18
iurygregoryhahaha beaker won't be able to do that15:18
jlvillalNot yet. Beaker for us is quite useful since we are trying to test hardware/software.15:18
*** strigazi has quit IRC15:19
*** strigazi has joined #openstack-ironic15:19
*** strigazi has quit IRC15:19
*** strigazi has joined #openstack-ironic15:20
*** ociuhandu has quit IRC15:30
*** dsneddon has quit IRC15:39
*** ociuhandu has joined #openstack-ironic15:50
*** dhellmann has quit IRC15:52
*** Qianbiao has quit IRC15:52
*** ociuhandu has quit IRC15:55
*** EmilienM has left #openstack-ironic16:01
rpittaubye! o/16:03
*** rpittau is now known as rpittau|afk16:03
*** lucasagomes has quit IRC16:03
*** anuradha1904 has quit IRC16:11
NobodyCamGood Morning Ironic folks. Happy Hump Day.16:23
NobodyCamjust a side question anyone had any experience attempting to leverage Ironic across an overlay network?16:24
dtantsurEDETAILSNEEDED16:28
iurygregorymorning NobodyCam16:29
NobodyCamhehehehe ... Good morning dtantsur and iurygregory16:29
NobodyCambasically running remote conductors over a overlay network16:32
TheJuliaso conductors as vms inside an existing openstack infra?16:49
*** lourot has quit IRC16:55
*** lourot has joined #openstack-ironic16:55
openstackgerritMerged openstack/ironic-lib stable/wallaby: Raise qemu-img memory limit to 2 GiB  https://review.opendev.org/c/openstack/ironic-lib/+/79305116:56
NobodyCamWell that maybe a test env I am investigating ..16:59
*** lourot has quit IRC17:04
*** lourot has joined #openstack-ironic17:05
*** gyee has joined #openstack-ironic17:08
*** lourot has quit IRC17:09
*** lourot has joined #openstack-ironic17:09
openstackgerritMerged openstack/ironic-lib stable/victoria: Raise qemu-img memory limit to 2 GiB  https://review.opendev.org/c/openstack/ironic-lib/+/79305217:10
openstackgerritMerged openstack/ironic-python-agent master: Rewrite write_image.sh in Python  https://review.opendev.org/c/openstack/ironic-python-agent/+/79193517:17
*** eandersson has quit IRC17:23
*** fdegir has quit IRC17:23
*** rnoriega_ has quit IRC17:23
*** k_mouza_ has quit IRC17:24
*** eandersson0 has joined #openstack-ironic17:25
*** k_mouza has joined #openstack-ironic17:25
*** fdegir8 has joined #openstack-ironic17:25
*** ianw_ has joined #openstack-ironic17:25
TheJuliadtantsur: seems like we may need to just drop qemu-img memory bounding17:25
TheJulia1.4GB to convert cirros....17:25
TheJuliaNobodyCam: I guess it should work, but tftp mtus would need to be adjusted.17:25
*** eandersson0 is now known as eandersson17:25
*** ianw has quit IRC17:25
*** ianw_ is now known as ianw17:25
iurygregory1.4 to convert a small cirros? O.o17:26
TheJuliayup, that is what someone reported independently just trying to see how it ended up chewing up ram17:27
TheJuliato which my comment is... "uhhh. that is bad."17:27
iurygregoryyeah =( very bad17:29
*** k_mouza has quit IRC17:30
*** k_mouza has joined #openstack-ironic17:31
*** rnoriega_ has joined #openstack-ironic17:32
openstackgerritJulia Kreger proposed openstack/ironic master: Update project conundrum related docs  https://review.opendev.org/c/openstack/ironic/+/79321117:33
* TheJulia whistles17:33
*** k_mouza has quit IRC17:36
openstackgerritJulia Kreger proposed openstack/ironic master: Update project conundrum related docs  https://review.opendev.org/c/openstack/ironic/+/79321117:37
iurygregory++ ^17:43
*** irclogbot_1 has quit IRC17:43
*** irclogbot_1 has joined #openstack-ironic17:48
*** rh-jelabarre has quit IRC17:49
*** gyee has quit IRC17:52
*** dsneddon has joined #openstack-ironic18:00
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/wallaby: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318618:03
iurygregoryEveryone, please take a look at the openstack-discuss list about our upstream meeting on May 31st18:03
*** larsks has left #openstack-ironic18:04
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/victoria: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318718:04
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/ussuri: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318818:04
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/train: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318918:05
*** openstack has quit IRC18:09
*** openstack has joined #openstack-ironic18:10
*** ChanServ sets mode: +o openstack18:10
iurygregoryyw18:11
*** k_mouza has joined #openstack-ironic18:12
*** buhman has left #openstack-ironic18:14
*** k_mouza has quit IRC18:16
openstackgerritMerged openstack/ironic master: Update project conundrum related docs  https://review.opendev.org/c/openstack/ironic/+/79321118:27
*** bfournie has quit IRC18:30
*** bfournie has joined #openstack-ironic18:31
*** zer0c00l|afk has quit IRC18:34
*** k_mouza has joined #openstack-ironic18:43
*** k_mouza has quit IRC18:48
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Limit qemu-img execution arenas  https://review.opendev.org/c/openstack/ironic-python-agent/+/79322518:59
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Limit qemu-img execution arenas  https://review.opendev.org/c/openstack/ironic-python-agent/+/79322519:01
*** iurygregory has quit IRC19:10
*** iurygregory has joined #openstack-ironic19:10
*** iurygregory has quit IRC19:11
*** iurygregory has joined #openstack-ironic19:12
*** fdegir8 is now known as fdegir19:29
*** openstackstatus has quit IRC19:50
*** openstackstatus has joined #openstack-ironic19:51
*** ChanServ sets mode: +v openstackstatus19:51
* TheJulia has this feeling that she just needs to force peopel to get in a room and talk19:53
TheJuliapeople19:53
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Limit qemu-img execution arenas  https://review.opendev.org/c/openstack/ironic-python-agent/+/79322520:04
TheJuliawould appreciate some reviews on ^20:06
TheJuliaapparently it is the source of our random image deployment failures20:06
*** bburns_ has quit IRC20:11
iurygregoryinteresting =) wondering if we want a config option for that =)20:15
*** bburns has joined #openstack-ironic20:15
TheJuliaiurygregory: unlikely, it is at 3 mallocs by then20:17
TheJuliaand it seems like it may just work anyway, because it is if one thread locks another as it was explained to me20:18
iurygregoryyeah20:18
iurygregorylgtm the changes20:18
iurygregoryI will wait for CI =)20:19
TheJuliak20:19
TheJuliaI need to do the same for ironic-lib20:19
TheJuliaand get it all backported all the way to trraaaaaainnnnn20:19
iurygregory*funny*20:19
TheJuliachoooo chooooo20:19
iurygregoryouch20:19
iurygregoryspeaking about train we decided to move to EM20:19
TheJuliasounds good, we just won't cut releases anymore20:20
TheJuliawhich makes sense20:20
TheJuliain that case I'll +1 the em change20:20
iurygregoryyup, I was afraid we would need releases to sync things downstream etc20:20
TheJulianope, its build rev just climbs then20:21
TheJuliaso like... grub 2.02-9920:21
*** bburns has quit IRC20:25
*** bburns has joined #openstack-ironic20:25
*** bburns has joined #openstack-ironic20:26
*** penick has joined #openstack-ironic20:28
*** penick has quit IRC20:28
*** bburns has quit IRC20:28
*** bburns has joined #openstack-ironic20:28
*** bburns has quit IRC20:29
*** bburns has joined #openstack-ironic20:29
stevebakermorning20:32
openstackgerritMerged openstack/ironic master: Delay rendering configdrive  https://review.opendev.org/c/openstack/ironic/+/79047220:32
iurygregorymorning stevebaker20:34
*** guilhermesp has quit IRC20:36
*** guilhermesp has joined #openstack-ironic20:37
*** k_mouza has joined #openstack-ironic20:44
*** thiagop has joined #openstack-ironic20:46
*** thiagop is now known as outbrito20:46
*** paras333 has quit IRC20:46
openstackgerritJulia Kreger proposed openstack/ironic-lib master: Limit the number of malloc arenas for qemu-img convert  https://review.opendev.org/c/openstack/ironic-lib/+/79323820:47
TheJuliaiurygregory: ^ ironic-lib version20:47
iurygregoryTheJulia, ack20:48
*** k_mouza has quit IRC20:49
*** openstack has joined #openstack-ironic21:25
*** ChanServ sets mode: +o openstack21:25
*** sdanni has quit IRC21:34
*** hjensas has quit IRC21:40
*** hjensas has joined #openstack-ironic21:43
*** hjensas has quit IRC21:45
*** hjensas has joined #openstack-ironic21:45
*** melwitt has quit IRC21:51
*** melwitt has joined #openstack-ironic21:51
*** stevebaker has quit IRC22:01
*** k_mouza has joined #openstack-ironic22:02
*** k_mouza has quit IRC22:03
openstackgerritMerged openstack/ironic-python-agent stable/wallaby: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318622:07
openstackgerritMerged openstack/ironic-python-agent stable/ussuri: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318822:07
*** melwitt has quit IRC22:14
*** melwitt has joined #openstack-ironic22:14
*** lmcgann has quit IRC22:21
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Fix NVMe Partition image on UEFI  https://review.opendev.org/c/openstack/ironic-python-agent/+/79318722:35
*** tosky has quit IRC22:58
*** k_mouza has joined #openstack-ironic22:59
*** k_mouza has quit IRC23:04
jandersgood morning Ironic o/23:54
TheJuliagood morning23:57

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