| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-10 | |||
| 16:22:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 16:22:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 16:22:32 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 16:22:32 | opendevreview | Balazs Gibizer proposed openstack/nova master: Stop if tracking is disable after it was enabled before https://review.opendev.org/c/openstack/nova/+/847009 | |
| 16:22:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move provider_tree RP creation to PciResourceProvider https://review.opendev.org/c/openstack/nova/+/850546 | |
| 16:22:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Retry /reshape at provider generation conflict https://review.opendev.org/c/openstack/nova/+/851358 | |
| 16:22:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Heal missing simple PCI allocation in the resource tracker https://review.opendev.org/c/openstack/nova/+/851359 | |
| 16:22:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Heal PCI allocation during resize https://review.opendev.org/c/openstack/nova/+/852396 | |
| 16:22:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle PCI dev reconf with allocations https://review.opendev.org/c/openstack/nova/+/852397 | |
| 16:22:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement https://review.opendev.org/c/openstack/nova/+/850468 | |
| 16:22:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Request filter for PCI in placement https://review.opendev.org/c/openstack/nova/+/852771 | |
| 16:22:55 | sean-k-mooney[m] | just one or two patches :) | |
| 16:23:05 | gibi | yeah just 22 :) | |
| 16:23:10 | gibi | in the main series | |
| 16:23:21 | gibi | but now it is rebased as the double mock fix landed | |
| 16:23:23 | sean-k-mooney[m] | it does make it more digestable | |
| 16:23:31 | sean-k-mooney[m] | cool | |
| 16:23:40 | sean-k-mooney[m] | i think the first few are ready to merge | |
| 16:23:40 | gibi | I checked locally I don't have double mock in the first two patches (the bugfix) | |
| 16:23:57 | sean-k-mooney[m] | ack | |
| 16:24:05 | gibi | I will check the rest locally too, but I have to log off in 10 minutes | |
| 16:33:31 | gibi | OK it seems to be the series is clean from double mocks (at least running it locally) | |
| 17:21:49 | gibi | sean-k-mooney[m]: https://review.opendev.org/c/openstack/nova/+/852295/2 is green now. so if you still around that could unblock some of the patches in the pci seriea | |
| 17:31:22 | sean-k-mooney[m] | yep still here | |
| 17:32:06 | sean-k-mooney[m] | done | |
| 17:32:25 | sean-k-mooney[m] | ill +w some of the later ones shortly | |
| 17:41:58 | sean-k-mooney[m] | for a second there i tought something was broken with how few jobs those are running | |
| 17:42:12 | sean-k-mooney[m] | then i realsed the patch im looking at is test only | |
| 17:42:29 | sean-k-mooney[m] | so it only runs the unit and functional tests not all of tempest | |
| 17:42:40 | sean-k-mooney[m] | well and pep8 and docs | |
| 17:45:27 | sean-k-mooney[m] | gibi: everything before https://review.opendev.org/c/openstack/nova/+/846187/18 should be now making its way through the gate | |
| 17:45:55 | sean-k-mooney[m] | so thats the pci whitelist rename part | |
| 17:46:01 | sean-k-mooney[m] | and the bug fix at the start | |
| 17:47:18 | sean-k-mooney[m] | o/ | |
| 18:06:59 | opendevreview | Merged openstack/nova master: Reproducer for bug 1983753 https://review.opendev.org/c/openstack/nova/+/852295 | |
| 18:25:27 | gibi | sean-k-mooney[m]: thank you | |
| 19:47:07 | opendevreview | Merged openstack/nova master: Update RequestSpec.pci_request for resize https://review.opendev.org/c/openstack/nova/+/852296 | |
| 19:47:14 | opendevreview | Merged openstack/nova master: Poison /sys access via various calls in test https://review.opendev.org/c/openstack/nova/+/844627 | |
| 19:47:21 | opendevreview | Merged openstack/nova master: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/850510 | |
| 20:13:43 | opendevreview | Merged openstack/nova master: Rename [pci]passthrough_whitelist to device_spec https://review.opendev.org/c/openstack/nova/+/843834 | |
| 20:13:49 | opendevreview | Merged openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec https://review.opendev.org/c/openstack/nova/+/843861 | |
| 20:13:56 | opendevreview | Merged openstack/nova master: Rename whitelist in tests https://review.opendev.org/c/openstack/nova/+/843862 | |
| 21:21:25 | opendevreview | Merged openstack/nova master: update default numa allocation strategy https://review.opendev.org/c/openstack/nova/+/846169 | |
| 23:30:07 | opendevreview | melanie witt proposed openstack/nova master: Fix exception catch when volume mount fails https://review.opendev.org/c/openstack/nova/+/852795 | |
| #openstack-nova - 2022-08-11 | |||
| 01:53:00 | opendevreview | Merged openstack/nova master: Optimize numa_fit_instance_to_host https://review.opendev.org/c/openstack/nova/+/845896 | |
| 01:53:07 | opendevreview | Merged openstack/nova master: Remove workaround for SQLAlchemy < 1.4 https://review.opendev.org/c/openstack/nova/+/852255 | |
| 03:36:13 | opendevreview | Merged openstack/nova master: manager: Reduce unnecessary calls https://review.opendev.org/c/openstack/nova/+/795027 | |
| 03:36:21 | opendevreview | Merged openstack/nova master: objects: Don't use generic 'Field' container https://review.opendev.org/c/openstack/nova/+/738239 | |
| 04:24:53 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix rescue volume-based instance https://review.opendev.org/c/openstack/nova/+/852737 | |
| 08:34:40 | opendevreview | Bence Romsics proposed openstack/nova master: Add limitation to docs about bug 1983570 https://review.opendev.org/c/openstack/nova/+/852168 | |
| 11:20:33 | opendevreview | Merged openstack/nova stable/wallaby: Adapt to SQLAlchemy 1.4 https://review.opendev.org/c/openstack/nova/+/852099 | |
| 17:05:44 | dansmith | melwitt: do we really have no test class for ConductorTaskManager ? | |
| 17:05:58 | dansmith | I see various mocks of it, but... | |
| 17:07:16 | dansmith | sorry, conductor.ComputeTaskManager I mean | |
| 17:16:50 | melwitt | uh dunno. I will look | |
| 17:19:06 | melwitt | this says it's for ComputeTaskManager https://github.com/openstack/nova/blob/master/nova/tests/unit/conductor/test_conductor.py#L2238 | |
| 17:20:06 | melwitt | dansmith: ^ | |
| 17:23:07 | dansmith | ah, yeah, okay, I found that in the greps but it seemed like that was more for tasks/* but yeah I guess maybe that's it | |
| 17:35:21 | opendevreview | Dan Smith proposed openstack/nova master: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/852900 | |
| 17:35:22 | opendevreview | Dan Smith proposed openstack/nova master: Avoid startup failure for keystone failures https://review.opendev.org/c/openstack/nova/+/852901 | |
| 17:35:33 | dansmith | melwitt: gibi_pto looks to be gone already, but I think this is a reasonable compromise ^ | |
| 17:40:45 | melwitt | ack, will look | |
| 17:46:42 | dansmith | I haven't run functional tests on that, but it works for me in my devstack if I stop keystone and restart conductor | |
| 18:12:20 | melwitt | ack | |
| 18:40:17 | opendevreview | Dan Smith proposed openstack/nova master: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/852900 | |
| 18:40:18 | opendevreview | Dan Smith proposed openstack/nova master: Avoid n-cond startup abort for keystone failures https://review.opendev.org/c/openstack/nova/+/852901 | |
| 18:49:29 | opendevreview | Dan Smith proposed openstack/nova master: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/852900 | |
| 18:49:30 | opendevreview | Dan Smith proposed openstack/nova master: Avoid n-cond startup abort for keystone failures https://review.opendev.org/c/openstack/nova/+/852901 | |
| 18:49:35 | dansmith | melwitt: that second set is for you ^ :) | |
| 18:50:23 | dansmith | good call on the other exceptions, and the unified handling helps make sure we log the right exception (not responding) for those too | |
| 18:55:57 | dansmith | log the right *message* I mean | |
| 18:56:59 | melwitt | :) | |
| #openstack-nova - 2022-08-12 | |||
| 11:22:47 | opendevreview | Takashi Natsume proposed openstack/nova master: Fix a deprecation warning about distutils https://review.opendev.org/c/openstack/nova/+/852947 | |
| 11:55:49 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove unnecessary testing code https://review.opendev.org/c/openstack/python-novaclient/+/852949 | |
| 12:07:39 | opendevreview | Takashi Natsume proposed openstack/nova master: Fix a deprecation warning about distutils https://review.opendev.org/c/openstack/nova/+/852947 | |
| 13:19:50 | opendevreview | Jan Hartkopf proposed openstack/nova master: add support for updating server's user_data https://review.opendev.org/c/openstack/nova/+/816157 | |
| 20:08:33 | ricolin | sean-k-mooney[m]: I though it's okay to set property in flavor and override property in image? So if they're both set, we can consider flavor property is there to override image property. Or they not suppose to set at the same time? | |
| 20:11:09 | opendevreview | Rico Lin proposed openstack/nova master: Add locked_memory extra spec and image property https://review.opendev.org/c/openstack/nova/+/778347 | |
| 20:11:09 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 20:11:10 | opendevreview | Rico Lin proposed openstack/nova master: Add traits for viommu model https://review.opendev.org/c/openstack/nova/+/844507 | |
| #openstack-nova - 2022-08-13 | |||
| 14:39:21 | opendevreview | Takashi Natsume proposed openstack/nova master: Add missing descriptions in HACKING.rst https://review.opendev.org/c/openstack/nova/+/853054 | |
| #openstack-nova - 2022-08-14 | |||
| 20:33:16 | opendevreview | Rico Lin proposed openstack/nova master: Add locked_memory extra spec and image property https://review.opendev.org/c/openstack/nova/+/778347 | |
| 20:33:17 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 20:33:17 | opendevreview | Rico Lin proposed openstack/nova master: Add traits for viommu model https://review.opendev.org/c/openstack/nova/+/844507 | |
| #openstack-nova - 2022-08-15 | |||
| 04:11:47 | opendevreview | Ke Niu proposed openstack/nova master: remove unicode prefix from code https://review.opendev.org/c/openstack/nova/+/853112 | |
| 04:12:59 | opendevreview | Ke Niu proposed openstack/nova master: remove unicode prefix from code https://review.opendev.org/c/openstack/nova/+/853112 | |
| 04:14:39 | opendevreview | Ke Niu proposed openstack/nova master: Cleanup py27 support https://review.opendev.org/c/openstack/nova/+/853113 | |
| 13:57:41 | zigo | I'm getting failed unit tests in Nova (which is RC bug in Debian since that's failed to build...): | |
| 13:57:41 | zigo | https://bugs.debian.org/1017217 | |
| 13:57:44 | zigo | Can anyone help? | |
| 14:00:59 | zigo | ricolin: I'm not sure, when reading the specs, but what is vIOMMU for? (note: I know what a PMMU is...) | |
| 14:35:54 | artom | zigo, only thing I can think of is the recently merged double-mocking patch | |
| 14:36:29 | zigo | artom: That's Stable / Yoga that I'm talking about, does this match? | |
| 14:36:39 | zigo | Was it backported? | |
| 14:37:06 | zigo | The thing is, it built fine 10 days ago in Debian Unstable. I'm currently trying to use snapshot.debian.org to see what package changed since ... | |
| 14:38:28 | artom | zigo, hrmm, no, it's https://review.opendev.org/c/openstack/nova/+/852464 and https://review.opendev.org/c/openstack/nova/+/851445 that I'm thinking of | |
| 14:39:28 | zigo | None of these seem to have been backported to Yoga, so it's unlikely the issue here. | |
| 14:39:59 | zigo | In a minute, I'll see what package has changed... | |
| 14:44:07 | artom | zigo, super bizarre... something is causing the unit test to apprently redirect | |
| 14:45:22 | zigo | Yeah, looks scary too ... | |