Wednesday, 2023-12-13

opendevreviewmelanie witt proposed openstack/nova master: Support rebuild and unshelve with ephemeral encryption  https://review.opendev.org/c/openstack/nova/+/87093900:36
opendevreviewOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/nova/+/90342702:29
*** jph5 is now known as jph108:19
*** jph1 is now known as jph08:20
opendevreviewVasyl Saienko proposed openstack/nova master: Fix returning empty availability zone  https://review.opendev.org/c/openstack/nova/+/90287509:04
*** elodilles_pto is now known as elodilles09:06
alisafarisean-k-mooney Hi, could you check out these backports? https://review.opendev.org/q/topic:%22fix-traits-to-cpu-flags-mapping%2209:11
elodillesbauzas: sorry, i forgot to let you know that i'll be on PTO yesterday :/10:04
bauzaselodilles: no worries at all :)10:04
elodilleso:)10:04
elodillesnevertheless, i have a question: have you heard about this qemu feature: cpu-pm? -> https://patchwork.kernel.org/project/qemu-devel/patch/20180622003140.164613-2-mst@redhat.com/10:05
elodillesor maybe the question is also to sean-k-mooney ^^^10:06
elodillesdo you remember maybe if someone was preparing to include this in OpenStack? a patch, a blueprint, or something like that?10:07
opendevreviewNobuhiro MIKI proposed openstack/nova master: scheduler: Request COMPUTE_ADDRESS_SPACE_* traits  https://review.opendev.org/c/openstack/nova/+/89576110:14
opendevreviewNobuhiro MIKI proposed openstack/nova master: libvirt: Add config option to support maxphysaddr  https://review.opendev.org/c/openstack/nova/+/89575910:21
bauzaselodilles: sorry, just saw your ping10:29
bauzaselodilles: nope, wasn't aware of it10:29
elodillesbauzas: thanks. well, i'm not even sure it's still there in qemu, and i haven't seen sign of it in libvirt either, though i thought it worth a question o:)10:38
opendevreviewStephen Finucane proposed openstack/nova master: docs: Address nits in new security group doc  https://review.opendev.org/c/openstack/nova/+/90359012:50
gibisean-k-mooney: an easy power management bugfix backport that has elodilles' +2 https://review.opendev.org/c/openstack/nova/+/90315113:38
sean-k-mooney"raise exception.InvalidConfiguration(msg) from e" ah your using that to convert the excpetion cool13:47
sean-k-mooney+2w13:49
artomOoo, `raise from` is a thing?14:05
sean-k-mooneyits a python 3 only thing14:05
artomSo I could import zombie; raise zombie from the_dead? :D14:05
sean-k-mooneybut yes14:05
sean-k-mooney...14:05
sean-k-mooneyyour a terrible terrible person. a good person but a teribel one :P14:05
artom:D14:06
gibi:D14:09
gibiartom: you need to declare the_dead before 14:10
gibi:)14:10
sean-k-mooneyartom: +2 on https://review.opendev.org/c/openstack/nova/+/899776 but add a release note please if you respin14:15
sean-k-mooneygibi: if your review then ^ is small14:18
artomYeah, agree on the release note14:18
sean-k-mooneyits not vital but it woudl be good to advertise this. if you want to respin anyway jsut ping me and ill take a look again14:19
opendevreviewArtom Lifshitz proposed openstack/nova master: libvirt: Stop unconditionally enabling evmcs  https://review.opendev.org/c/openstack/nova/+/89977614:23
artomDone ^^ :)14:23
sean-k-mooneyack +2'd14:30
kashyapelodilles: Hi, I was just pointed to your question about https://patchwork.kernel.org/project/qemu-devel/patch/20180622003140.164613-2-mst@redhat.com/14:47
kashyapelodilles: It's a QEMU command-line option, as you see.  We don't have a BP (not that I know of) for it14:47
kashyapHave you tested it w/ "<qemu:commandline>14:48
kashyap... in the guest XML?  Do you see noticeable benefits?14:48
sean-k-mooneywe do not allow <qemu:commandline> to be used in nova14:49
sean-k-mooneyso even if that worked it would need to be expose via an actual libvirt element 14:49
sean-k-mooney its fine for testing hwoever14:49
gibiartom, sean-k-mooney +2+W14:50
sean-k-mooneyelodilles: so i had heard about the fucntionaltiy but not looked into it much14:53
sean-k-mooneyelodilles: i could see use in the future enableing this by default for guest with pinned but not mixed cpus14:54
sean-k-mooneysince this does not seam to be per core we cant supprot mixed cpus14:54
*** d34dh0r5- is now known as d34dh0r5314:59
elodilleskashyap: i'm not aware of any testing, but yeah, that's probably would be a good start to see if it has any benefit.15:00
kashyapsean-k-mooney: I of course know we don't allow.  I explicitly said "testing" there15:01
kashyapThe qemu:commandline thing.  It is alwys mentioned in context of developer testing in this channel for a million years now.15:01
kashyapelodilles: Yeah; some data on that would be good.  What made you think of that feature?15:01
elodillessean-k-mooney: ACK, yepp, only for guests with pinned CPUs, yepp. that was my understanding too.15:01
sean-k-mooneyelodilles: if you do try this in devstack just not i disabled the apic by default in devstack becasue of a bug in cirros 5.x15:02
elodilleskashyap: well, i got a question from downstream team :)15:02
kashyapAh, makes sense :)15:02
sean-k-mooneythat bug nolonger exists in 6.x but i have not disabeld that workaround15:02
sean-k-mooneyso you might need to makesure it off in your env if you try it15:03
sean-k-mooneyoff meaing the workaroudn is disabeld and the apic is present in the guest15:03
sean-k-mooneyhttps://docs.openstack.org/nova/latest/configuration/config.html#workarounds.libvirt_disable_apic15:03
sean-k-mooneyoh actully ignore that15:04
sean-k-mooneyits acpi that is used for power not apic15:05
elodilles:)15:07
kashyapYeah, confusing acronyms15:08
opendevreviewMerged openstack/nova stable/2023.2: [pwmgmt]ignore missin governor when cpu_state used  https://review.opendev.org/c/openstack/nova/+/90315116:22
opendevreviewBalazs Gibizer proposed openstack/nova stable/2023.1: [pwmgmt]ignore missin governor when cpu_state used  https://review.opendev.org/c/openstack/nova/+/90355316:52
gibielodilles, sean-k-mooney: thanks for the review on the backport of the power management fix here is the Antelope backport ^^16:53
gibistill clean cherry-pick16:53
elodillesgibi: will look into it17:00
gibithanks!17:34
sean-k-mooneylooks liek bobcat one already merged. that was fastere then i expected which is nice18:01
sean-k-mooneyi agree its a clean backport and the commit ectra is all correct so +2w18:02
sean-k-mooneyi rememeber correctly this shoudl be the final backport correct18:02
sean-k-mooneythe feature was added in antelope18:02
sean-k-mooneyso once that merges the issue will be closed across all affected branches18:03
opendevreviewArtom Lifshitz proposed openstack/nova master: POC: glanceclient streaming uploads  https://review.opendev.org/c/openstack/nova/+/90361118:32
sean-k-mooneythat sound like a new feature18:32
sean-k-mooneyhum just enabing an exisitng glance one. i wonder if that needs to be a config option however18:33
sean-k-mooneyi think streaming upload/download may have some depenices on how glance or the webserver/loadbalance its deployed behind are configured18:34
sean-k-mooneyit was added by https://opendev.org/openstack/glance/commit/7f761e16ce69ec549be1a66bd2797e53d0a1d988?style=unified&whitespace=show-all&show-outdated=18:34
sean-k-mooneyapplication/octet-stream that shoudl work18:36
sean-k-mooneyits the same content type as a normal upload18:37
opendevreviewStephen Finucane proposed openstack/nova master: docs: Address nits in new security group doc  https://review.opendev.org/c/openstack/nova/+/90359019:53
opendevreviewArtom Lifshitz proposed openstack/nova master: POC: glanceclient streaming uploads  https://review.opendev.org/c/openstack/nova/+/90361119:59
opendevreviewMerged openstack/nova master: Use SDK for remaining ironic driver calls  https://review.opendev.org/c/openstack/nova/+/86779520:00
artomsean-k-mooney, so chunked is apparently not the same as streaming20:03
artomThough if I'm honest I'm not sure I understand the difference on the wire. The mechanics appear to be - pass a generator to Python requests = chunked, pass a file handler = streaming20:04
artomMaybe I should check what Nova currently passes... I kinda assumed it just yeeted the entire image in the POST body20:04
artomHrmp, no mention of 'chunked' anywhere in our git log20:05
opendevreviewJay Faulkner proposed openstack/nova master: [ironic] Partition & use cache for list_instance*  https://review.opendev.org/c/openstack/nova/+/90083120:06
opendevreviewMerged openstack/nova master: docs: Revamp the security groups guide  https://review.opendev.org/c/openstack/nova/+/90350720:32
opendevreviewMerged openstack/nova master: docs: Address nits in new security group doc  https://review.opendev.org/c/openstack/nova/+/90359020:37
JayFgibi: When you get a chance, https://review.opendev.org/c/openstack/nova/+/867797/ and the children below it are almost ready to land in that big Ironic<>Nova openstacksdk migration chain; I've been trying to usher these through the gate so a review and approval there (and on the other patches in the stack, as appropriate) would be appreciated. Thank you!20:46
opendevreviewMerged openstack/nova master: [codespell] fix typos in tests  https://review.opendev.org/c/openstack/nova/+/89721323:46

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