| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-12 | |||
| 17:01:35 | artom | 'cuz I'm hitting the same thing, but in a different environment | |
| 18:58:12 | mnaser | i've got a (kinda) trivial thing to add, which is support for the IOMMU devices, it's essentially just an added block of xml to the domain (and probably modified with extra_specs): https://libvirt.org/formatdomain.html#iommu-devices | |
| 18:58:24 | mnaser | should i file a bug and get working on it or it's going to be a bit more than that? | |
| 19:42:22 | stephenfin | mnaser: https://review.opendev.org/c/openstack/nova/+/830646 | |
| 19:42:59 | mnaser | stephenfin: oh hey, that looks familiar :) | |
| 19:43:01 | stephenfin | mnaser: sean-k-mooney says its pretty much useless by itself and that nova needs to grow the ability to manage PCI addresses. I haven't had time to complete that work (it's big) | |
| 19:43:24 | mnaser | stephenfin: in my case, i've got a pci passthrough device that needs this to function properly | |
| 19:44:14 | mnaser | but yeah, this initial start is actually enough for my use case :> | |
| 19:46:01 | stephenfin | mnaser: If you remind me tomorrow, I can rebase that and probably fix the failing tests. It's too late today. If you want to take it for a spin yourself before then, feel free | |
| 19:46:41 | mnaser | stephenfin: i'll see if i can find sometime, i can take care of that too, and also implement another flag (aw_bits) since the libvirt we require now is new enough to support it (since 6.5.0 according to docs) | |
| 19:46:56 | mnaser | but if you take a shot at it, i won't be upset either :) | |
| #openstack-nova - 2022-04-13 | |||
| 03:32:59 | opendevreview | Wenping Song proposed openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| 05:55:00 | opendevreview | Wenping Song proposed openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| 05:56:24 | opendevreview | Wenping Song proposed openstack/nova-specs master: Usage of new trait of OWNER_NOVA https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| #openstack-nova - 2022-04-14 | |||
| 01:06:29 | opendevreview | Brin Zhang proposed openstack/nova-specs master: Re-propose remove tenant_id https://review.opendev.org/c/openstack/nova-specs/+/837789 | |
| 01:07:53 | opendevreview | Brin Zhang proposed openstack/nova-specs master: Re-propose remove tenant_id https://review.opendev.org/c/openstack/nova-specs/+/837789 | |
| 08:59:47 | gibi | sean-k-mooney, bauzas: the https://review.opendev.org/q/topic:sqlalchemy-20+project:openstack/placement series now have +2 from me all over. So it only needs a second core to land | |
| 09:00:06 | bauzas | gibi: I'll try to look at it later today | |
| 09:55:30 | sean-k-mooney[m] | i just finished going over them now | |
| 09:55:49 | gibi | stephenfin will be happy :) | |
| 09:55:53 | sean-k-mooney[m] | other then one nit in a commit message they all looked good to me so i sent them on there way | |
| 09:56:16 | stephenfin | 🥳 | |
| 10:18:23 | opendevreview | Merged openstack/placement master: db: Replace implicit conversion of SELECT into FROM https://review.opendev.org/c/openstack/placement/+/800910 | |
| 10:32:48 | opendevreview | Merged openstack/placement master: db: Replace 'as_scalar()' with 'scalar_subquery()' https://review.opendev.org/c/openstack/placement/+/801100 | |
| 10:51:15 | opendevreview | Stephen Finucane proposed openstack/nova master: libvirt: Ignore LibvirtConfigObject kwargs https://review.opendev.org/c/openstack/nova/+/830644 | |
| 10:51:16 | opendevreview | Stephen Finucane proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 10:51:16 | opendevreview | Stephen Finucane proposed openstack/nova master: libvirt: Remove unnecessary TODO https://review.opendev.org/c/openstack/nova/+/830645 | |
| 10:51:17 | opendevreview | Stephen Finucane proposed openstack/nova master: libvirt: Shuffle methods around https://review.opendev.org/c/openstack/nova/+/830648 | |
| 10:51:17 | opendevreview | Stephen Finucane proposed openstack/nova master: libvirt: Rename some config objects https://review.opendev.org/c/openstack/nova/+/830647 | |
| 10:51:18 | opendevreview | Stephen Finucane proposed openstack/nova master: libvirt: Remove unnecessary kwargs https://review.opendev.org/c/openstack/nova/+/837853 | |
| 10:52:04 | sean-k-mooney | stephenfin: is that the next series i shoudl add to my list :P | |
| 10:53:05 | stephenfin | sean-k-mooney: nope, that's https://review.opendev.org/q/topic:sqlalchemy-20+project:openstack/nova+is:open 0:) | |
| 10:54:11 | sean-k-mooney | :) added to the folder | |
| 12:10:32 | opendevreview | Merged openstack/nova master: Fix the PCI device capability dict creation https://review.opendev.org/c/openstack/nova/+/833899 | |
| 12:10:39 | opendevreview | Merged openstack/nova master: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/834382 | |
| 12:59:11 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Store pf_mac_address and vf_num in extra_info https://review.opendev.org/c/openstack/nova/+/834006 | |
| 13:09:33 | dmitriis | artom_: hey, apologies, just saw your message above about the lock_path option. I haven't managed to get to the bottom of it yet but I'm still getting it on my workstation: I get around 5 failures while running `tox -e py39` but not on the upstream CI. | |
| 13:10:23 | dmitriis | Will likely do an upgrade to 22.04 soon and see if it's still broken with updated packages | |
| 13:12:32 | sean-k-mooney | i didnt see those on a clean clone on 21.10 so ya maybe that will help | |
| 13:13:29 | sean-k-mooney | dmitriis: ill try and review https://review.opendev.org/c/openstack/nova/+/834006 later today but ill be off until tuesday so if i dont get to it today feel free to ping me when im back next week | |
| 13:14:11 | sean-k-mooney | gibi: by the way https://review.opendev.org/c/openstack/nova/+/829248 is not in merge confict and will need to be rebased | |
| 13:16:51 | sean-k-mooney | dmitriis gibi ye might want to stack both of ye're patches so that they dont conflict | |
| 13:17:51 | sean-k-mooney | techinially they are indepented (recordign the mac in the port forfile and recording it in the exta_info of the pci device) but also related | |
| 13:18:01 | dmitriis | sean-k-mooney: ack, ty | |
| 13:18:35 | sean-k-mooney | gerrit is not actully showing the patches as a confilct but im not sure if that will be the case when gibi rebases | |
| 13:19:00 | gibi | sean-k-mooney: thanks for the notice, I will rebase it | |
| 13:20:45 | sean-k-mooney | hehe | |
| 13:20:48 | sean-k-mooney | # TODO(stephenfin): The mock of 'migrate_disk_and_power_off' should | |
| 13:20:50 | sean-k-mooney | # probably be less...dumb | |
| 13:21:03 | sean-k-mooney | we are copy pasting that enough that we proably want to centralise that | |
| 13:21:26 | sean-k-mooney | stephenfin: did you have a fixture patch for that at some point | |
| 13:21:44 | stephenfin | not that I'm aware of, no. I tried a few times but it was hard | |
| 13:21:55 | sean-k-mooney | ack | |
| 13:22:18 | sean-k-mooney | right now we do with mock.patch('nova.virt.libvirt.driver.LibvirtDriver' | |
| 13:22:19 | sean-k-mooney | '.migrate_disk_and_power_off', return_value='{}'): | |
| 13:22:48 | sean-k-mooney | i was thinking of just putting that into one of the base class setup fucntion and keeping the comment | |
| 13:23:12 | sean-k-mooney | i actully dont know if we need to do somethign less dmub in most of the cases | |
| 13:23:22 | sean-k-mooney | so as a default thats proably fine | |
| 13:28:42 | artom_ | sean-k-mooney, so, for context, we're hitting it downstream in our OSP 17 unit tests on py39 | |
| 13:29:16 | artom_ | So I was (not so) secretly hoping that dmitriis solved it for us :) | |
| 13:30:43 | sean-k-mooney | i think it depend on your enviornment | |
| 13:31:03 | sean-k-mooney | ill spin up a c9s vm in a sec and see if i can repoduce | |
| 13:31:45 | sean-k-mooney | 17 would be stable wallaby i guess. i wonder if its stable branch specific or if it happens on master | |
| 13:43:37 | bauzas | stephenfin: https://review.opendev.org/c/openstack/placement/+/801104/9/placement/objects/allocation.py#166 hmpf, I'm a bit sad we need to use a private field for getting the mapping | |
| 13:43:52 | bauzas | sounds weird and anti-pattern | |
| 13:44:19 | bauzas | I tried to read the SQLA doc about the deprecation warning you mentioned, but this didn't help | |
| 13:45:31 | stephenfin | bauzas: agreed but I fixed it later https://review.opendev.org/c/openstack/placement/+/801107/9 | |
| 13:45:53 | bauzas | ah ok, I stopped on reading the series on that one | |
| 13:47:15 | sean-k-mooney | bauzas: stephen kept refactor to not use it seperate form the deprecation fix but i had the same feeling when reviewing it initally | |
| 13:47:30 | bauzas | sean-k-mooney: I just commented | |
| 13:47:45 | bauzas | oh by the way, I hate my bias | |
| 13:48:01 | bauzas | I said loudly "STOP DOING BLIND RECHECKS" and I fuckingly made one | |
| 13:48:18 | bauzas | bad manners are so easy to keep | |
| 13:48:43 | sean-k-mooney | hehe i noticed stephenfin actully copyt the fact that they rechecked beause the base patch failed to merge which made me happy :) | |
| 15:56:25 | opendevreview | Merged openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 15:56:27 | opendevreview | Merged openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 15:56:30 | opendevreview | Merged openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 15:56:32 | opendevreview | Merged openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 16:06:25 | opendevreview | Merged openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 18:41:32 | opendevreview | Elod Illes proposed openstack/nova stable/yoga: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838000 | |
| 18:41:36 | elodilles | melwitt: if this passes then i'll cherry pick them to older branches ^^^ | |
| 18:42:31 | melwitt | elodilles: thanks! I will watch it | |
| 19:16:38 | opendevreview | Elod Illes proposed openstack/nova stable/xena: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838007 | |
| 19:16:55 | elodilles | melwitt: though the gate is not finished still it seems the job settings (and e.g. requirements-check job) are OK, so I started cherry picking ^^^ | |
| 19:17:30 | melwitt | elodilles: ack, thanks! | |
| 19:24:40 | opendevreview | Elod Illes proposed openstack/nova stable/wallaby: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838030 | |
| 19:26:34 | opendevreview | Elod Illes proposed openstack/nova stable/victoria: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838032 | |
| 19:31:05 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838033 | |
| 19:40:18 | opendevreview | Merged openstack/placement master: tests: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/placement/+/828119 | |
| 19:41:24 | opendevreview | Elod Illes proposed openstack/nova stable/train: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838037 | |
| 19:47:38 | opendevreview | Elod Illes proposed openstack/nova stable/stein: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838038 | |
| 20:02:08 | melwitt | elodilles: hm, looks like this is saying on master too https://lists.openstack.org/pipermail/openstack-discuss/2022-April/028199.html so I guess it's not supposed to be stable-only? | |
| 20:05:18 | opendevreview | Elod Illes proposed openstack/nova stable/rocky: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838041 | |
| 20:06:50 | elodilles | melwitt: yes, though i wanted to wait for the resolution to drop it on master | |
| 20:07:08 | melwitt | ok | |
| 20:25:17 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Lookup nic feature by PCI address https://review.opendev.org/c/openstack/nova/+/838042 | |
| 20:26:03 | opendevreview | Merged openstack/placement master: db: Use Row, not LegacyRow https://review.opendev.org/c/openstack/placement/+/828305 | |
| 20:26:06 | opendevreview | Merged openstack/placement master: tox: Enable SQLAlchemy 2.0 warnings https://review.opendev.org/c/openstack/placement/+/801108 | |
| 23:08:20 | opendevreview | melanie witt proposed openstack/nova stable/train: Lookup nic feature by PCI address https://review.opendev.org/c/openstack/nova/+/838050 | |
| 23:47:15 | gmann | elodilles: melwitt +1, for master we can wait for resolution to merge but it is all good to drop from stable as they are broken | |