| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-12 | |||
| 17:41:05 | sean-k-mooney | but if you already have that no need to duplciate it as you said | |
| 17:41:06 | JayF | which is actually information that people might want | |
| 17:41:14 | JayF | IP is in neutron, we don't have it in Ironic | |
| 17:41:36 | JayF | Ironic holds more of the information about the physical port: mac address, binding profile/metadata about how it's physically connected to switches, etc | |
| 17:45:06 | sean-k-mooney | the ip stuff was a relitivly late addtion | |
| 17:45:44 | sean-k-mooney | https://github.com/openstack/nova/commit/838370a49014351051bbef2d1c2ada1f47ac2bfb | |
| 17:46:03 | sean-k-mooney | it was added in wallaby | |
| 17:46:42 | sean-k-mooney | anyway that data would be avialable for you to consume if it was useful | |
| 17:47:09 | sean-k-mooney | JayF: this is the spec we used to add that https://specs.openstack.org/openstack/nova-specs/specs/wallaby/implemented/libvirt-driver-ip-metadata.html | |
| 18:39:29 | opendevreview | Stephen Finucane proposed openstack/nova master: Bump hacking version https://review.opendev.org/c/openstack/nova/+/903529 | |
| 18:39:30 | opendevreview | Stephen Finucane proposed openstack/nova master: Resolve mypy error https://review.opendev.org/c/openstack/nova/+/903530 | |
| 18:46:31 | stephenfin | That mypy fix is required to unblock an u-c bump, while the other fix and the u-c bump are required to fix installing nova (for unit tests) on Python 3.12 (Fedora 39) ^ | |
| 22:01:15 | JayF | sean-k-mooney: johnthetubaguy: Is there anything I can do to help get https://review.opendev.org/c/openstack/nova/+/900831 landed? Was hoping to get this in front of the sharding re-proposal patch, which I was hoping to do this week (about to go rev the spec) | |
| 22:41:38 | opendevreview | Merged openstack/nova master: Fix regression breaking Ironic boot-from-volume https://review.opendev.org/c/openstack/nova/+/903324 | |
| 22:41:55 | JayF | Huzzah! | |
| 23:25:07 | opendevreview | Jay Faulkner proposed openstack/nova-specs master: Re-submit Ironic-shards for Caracal https://review.opendev.org/c/openstack/nova-specs/+/902698 | |
| 23:51:59 | opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Partition & use cache for list_instance* https://review.opendev.org/c/openstack/nova/+/900831 | |
| #openstack-nova - 2023-12-13 | |||
| 00:36:52 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild and unshelve with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870939 | |
| 02:29:11 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/903427 | |
| 09:04:53 | opendevreview | Vasyl Saienko proposed openstack/nova master: Fix returning empty availability zone https://review.opendev.org/c/openstack/nova/+/902875 | |
| 09:11:18 | alisafari | sean-k-mooney Hi, could you check out these backports? https://review.opendev.org/q/topic:%22fix-traits-to-cpu-flags-mapping%22 | |
| 10:04:37 | elodilles | bauzas: sorry, i forgot to let you know that i'll be on PTO yesterday :/ | |
| 10:04:51 | bauzas | elodilles: no worries at all :) | |
| 10:04:57 | elodilles | o:) | |
| 10:05:35 | elodilles | nevertheless, 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:06:02 | elodilles | or maybe the question is also to sean-k-mooney ^^^ | |
| 10:07:36 | elodilles | do you remember maybe if someone was preparing to include this in OpenStack? a patch, a blueprint, or something like that? | |
| 10:14:40 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: scheduler: Request COMPUTE_ADDRESS_SPACE_* traits https://review.opendev.org/c/openstack/nova/+/895761 | |
| 10:21:53 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Add config option to support maxphysaddr https://review.opendev.org/c/openstack/nova/+/895759 | |
| 10:29:18 | bauzas | elodilles: sorry, just saw your ping | |
| 10:29:23 | bauzas | elodilles: nope, wasn't aware of it | |
| 10:38:10 | elodilles | bauzas: 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:) | |
| 12:50:09 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Address nits in new security group doc https://review.opendev.org/c/openstack/nova/+/903590 | |
| 13:38:56 | gibi | sean-k-mooney: an easy power management bugfix backport that has elodilles' +2 https://review.opendev.org/c/openstack/nova/+/903151 | |
| 13:47:56 | sean-k-mooney | "raise exception.InvalidConfiguration(msg) from e" ah your using that to convert the excpetion cool | |
| 13:49:48 | sean-k-mooney | +2w | |
| 14:05:10 | artom | Ooo, `raise from` is a thing? | |
| 14:05:22 | sean-k-mooney | its a python 3 only thing | |
| 14:05:23 | artom | So I could import zombie; raise zombie from the_dead? :D | |
| 14:05:24 | sean-k-mooney | but yes | |
| 14:05:36 | sean-k-mooney | ... | |
| 14:05:56 | sean-k-mooney | your a terrible terrible person. a good person but a teribel one :P | |
| 14:06:11 | artom | :D | |
| 14:09:21 | gibi | :D | |
| 14:10:01 | gibi | artom: you need to declare the_dead before | |
| 14:10:17 | gibi | :) | |
| 14:15:57 | sean-k-mooney | artom: +2 on https://review.opendev.org/c/openstack/nova/+/899776 but add a release note please if you respin | |
| 14:18:06 | sean-k-mooney | gibi: if your review then ^ is small | |
| 14:18:21 | artom | Yeah, agree on the release note | |
| 14:19:30 | sean-k-mooney | its not vital but it woudl be good to advertise this. if you want to respin anyway jsut ping me and ill take a look again | |
| 14:23:20 | opendevreview | Artom Lifshitz proposed openstack/nova master: libvirt: Stop unconditionally enabling evmcs https://review.opendev.org/c/openstack/nova/+/899776 | |
| 14:23:25 | artom | Done ^^ :) | |
| 14:30:54 | sean-k-mooney | ack +2'd | |
| 14:47:00 | kashyap | elodilles: 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:11 | kashyap | elodilles: It's a QEMU command-line option, as you see. We don't have a BP (not that I know of) for it | |
| 14:48:02 | kashyap | Have you tested it w/ "<qemu:commandline> | |
| 14:48:43 | kashyap | ... in the guest XML? Do you see noticeable benefits? | |
| 14:49:09 | sean-k-mooney | we do not allow <qemu:commandline> to be used in nova | |
| 14:49:29 | sean-k-mooney | so even if that worked it would need to be expose via an actual libvirt element | |
| 14:49:46 | sean-k-mooney | its fine for testing hwoever | |
| 14:50:11 | gibi | artom, sean-k-mooney +2+W | |
| 14:53:51 | sean-k-mooney | elodilles: so i had heard about the fucntionaltiy but not looked into it much | |
| 14:54:12 | sean-k-mooney | elodilles: i could see use in the future enableing this by default for guest with pinned but not mixed cpus | |
| 14:54:30 | sean-k-mooney | since this does not seam to be per core we cant supprot mixed cpus | |
| 15:00:40 | elodilles | kashyap: 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:01:10 | kashyap | sean-k-mooney: I of course know we don't allow. I explicitly said "testing" there | |
| 15:01:38 | kashyap | The qemu:commandline thing. It is alwys mentioned in context of developer testing in this channel for a million years now. | |
| 15:01:54 | kashyap | elodilles: Yeah; some data on that would be good. What made you think of that feature? | |
| 15:01:55 | elodilles | sean-k-mooney: ACK, yepp, only for guests with pinned CPUs, yepp. that was my understanding too. | |
| 15:02:34 | sean-k-mooney | elodilles: if you do try this in devstack just not i disabled the apic by default in devstack becasue of a bug in cirros 5.x | |
| 15:02:35 | elodilles | kashyap: well, i got a question from downstream team :) | |
| 15:02:45 | kashyap | Ah, makes sense :) | |
| 15:02:50 | sean-k-mooney | that bug nolonger exists in 6.x but i have not disabeld that workaround | |
| 15:03:01 | sean-k-mooney | so you might need to makesure it off in your env if you try it | |
| 15:03:19 | sean-k-mooney | off meaing the workaroudn is disabeld and the apic is present in the guest | |
| 15:03:50 | sean-k-mooney | https://docs.openstack.org/nova/latest/configuration/config.html#workarounds.libvirt_disable_apic | |
| 15:04:46 | sean-k-mooney | oh actully ignore that | |
| 15:05:01 | sean-k-mooney | its acpi that is used for power not apic | |
| 15:07:38 | elodilles | :) | |
| 15:08:41 | kashyap | Yeah, confusing acronyms | |
| 16:22:23 | opendevreview | Merged openstack/nova stable/2023.2: [pwmgmt]ignore missin governor when cpu_state used https://review.opendev.org/c/openstack/nova/+/903151 | |
| 16:52:03 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: [pwmgmt]ignore missin governor when cpu_state used https://review.opendev.org/c/openstack/nova/+/903553 | |
| 16:53:06 | gibi | elodilles, sean-k-mooney: thanks for the review on the backport of the power management fix here is the Antelope backport ^^ | |
| 16:53:11 | gibi | still clean cherry-pick | |
| 17:00:40 | elodilles | gibi: will look into it | |
| 17:34:11 | gibi | thanks! | |
| 18:01:39 | sean-k-mooney | looks liek bobcat one already merged. that was fastere then i expected which is nice | |
| 18:02:43 | sean-k-mooney | i agree its a clean backport and the commit ectra is all correct so +2w | |
| 18:02:43 | sean-k-mooney | i rememeber correctly this shoudl be the final backport correct | |
| 18:02:43 | sean-k-mooney | the feature was added in antelope | |
| 18:03:10 | sean-k-mooney | so once that merges the issue will be closed across all affected branches | |
| 18:32:20 | opendevreview | Artom Lifshitz proposed openstack/nova master: POC: glanceclient streaming uploads https://review.opendev.org/c/openstack/nova/+/903611 | |
| 18:32:54 | sean-k-mooney | that sound like a new feature | |
| 18:33:31 | sean-k-mooney | hum just enabing an exisitng glance one. i wonder if that needs to be a config option however | |
| 18:34:05 | sean-k-mooney | i think streaming upload/download may have some depenices on how glance or the webserver/loadbalance its deployed behind are configured | |
| 18:34:48 | sean-k-mooney | it was added by https://opendev.org/openstack/glance/commit/7f761e16ce69ec549be1a66bd2797e53d0a1d988?style=unified&whitespace=show-all&show-outdated= | |
| 18:36:46 | sean-k-mooney | application/octet-stream that shoudl work | |
| 18:37:05 | sean-k-mooney | its the same content type as a normal upload | |
| 19:53:56 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Address nits in new security group doc https://review.opendev.org/c/openstack/nova/+/903590 | |
| 19:59:15 | opendevreview | Artom Lifshitz proposed openstack/nova master: POC: glanceclient streaming uploads https://review.opendev.org/c/openstack/nova/+/903611 | |