| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-04 | |||
| 11:50:20 | gibi | as there they have no libvirt interface at all | |
| 11:50:45 | sean-k-mooney[m] | in general if we could disable the caching in libvirt entirly i think we would | |
| 11:51:12 | sean-k-mooney[m] | i have serriously considered if we would be better not using libvirt for device tracking a few times | |
| 11:51:52 | sean-k-mooney[m] | https://review.opendev.org/c/openstack/nova/+/739131 | |
| 11:52:02 | sean-k-mooney[m] | that was the change i was thinkin of | |
| 11:52:05 | gibi | yeah, or creating a way to force libvirt to refresh the cache | |
| 11:52:36 | sean-k-mooney[m] | bind mount it to /dev/null on. disk ? | |
| 11:53:13 | gibi | is it an on disk cache? | |
| 11:53:19 | gibi | I assumed it is just in memory | |
| 11:53:28 | sean-k-mooney[m] | i think both | |
| 11:53:46 | sean-k-mooney[m] | its in memory in the vritnodedevd container | |
| 11:54:02 | sean-k-mooney[m] | but i think it also caches some info on disk | |
| 11:54:16 | sean-k-mooney[m] | i wonder can you just not use that container/deamon | |
| 11:54:19 | sean-k-mooney[m] | and run without it | |
| 11:54:26 | sean-k-mooney[m] | i should ask danpb | |
| 11:55:07 | sean-k-mooney[m] | i assume that wont actully work | |
| 11:55:21 | sean-k-mooney[m] | that deamon is more then just a cache | |
| 11:57:47 | gibi | I assume so | |
| 11:59:52 | sean-k-mooney[m] | what we really. want i think is to add a flag to https://libvirt.org/html/libvirt-libvirt-nodedev.html#virNodeListDevices to force it to probe | |
| 12:00:28 | gibi | yepp, that would help | |
| 12:00:36 | gibi | we would just alway set that flag | |
| 12:06:46 | opendevreview | Rajesh Tailor proposed openstack/nova master: Transport context to all threads https://review.opendev.org/c/openstack/nova/+/827467 | |
| 12:16:41 | gibi | sean-k-mooney[m]: when you have 2 minutes, could you look at this please? https://review.opendev.org/c/openstack/nova/+/845922 I got hit by it recently | |
| 12:17:10 | sean-k-mooney[m] | sure im just working on a doc for a meeting tomrrow | |
| 12:17:40 | sean-k-mooney[m] | hum im not familar with this bug but sure sound strait forward | |
| 12:18:08 | gibi | it happens on slow nodes | |
| 12:19:24 | sean-k-mooney[m] | 1.23 :) | |
| 12:19:43 | sean-k-mooney[m] | i would have also accepted a 42, 420 or 69 | |
| 12:20:17 | sean-k-mooney[m] | have you ever looked at our ping message in the rpc client. that one is my favorite | |
| 12:22:36 | gibi | lol, now I looked :D | |
| 12:30:14 | sean-k-mooney[m] | oh its the conductor ping not base api https://github.com/openstack/nova/blob/50fdbc752a9ca9c31488140ef2997ed59d861a41/nova/conductor/api.py#L67 | |
| 12:30:52 | sean-k-mooney[m] | but ya that makes me happy whenever i see it | |
| 12:33:38 | gibi | :) | |
| 13:00:20 | gibi | ...brace for impact | |
| 13:00:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Poison /sys access via various calls in test https://review.opendev.org/c/openstack/nova/+/844627 | |
| 13:00:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/850510 | |
| 13:00:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename [pci]passthrough_whitelist to device_spec https://review.opendev.org/c/openstack/nova/+/843834 | |
| 13:00:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec https://review.opendev.org/c/openstack/nova/+/843861 | |
| 13:00:32 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename whitelist in tests https://review.opendev.org/c/openstack/nova/+/843862 | |
| 13:00:32 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 13:00:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 13:00:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 13:00:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 13:00:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 13:00:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 13:00:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 13:00:38 | 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 | |
| 13:00:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move provider_tree RP creation to PciResourceProvider https://review.opendev.org/c/openstack/nova/+/850546 | |
| 13:00:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Retry /reshape at provider generation conflict https://review.opendev.org/c/openstack/nova/+/851358 | |
| 13:00:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Heal missing PCI allocation in the resource tracker https://review.opendev.org/c/openstack/nova/+/851359 | |
| 13:00:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement https://review.opendev.org/c/openstack/nova/+/850468 | |
| 13:01:17 | gibi | this should resolve all the mocking issues from ^^ | |
| 13:06:37 | Uggla | gibi, notify_about_instance_usage is a "legacy call" to notification and should not be used anymore ? | |
| 13:07:19 | gibi | Uggla: if you add a new notification for attaching a share then yes you should not use the legacy one | |
| 13:08:37 | Uggla | ok | |
| 13:11:20 | gibi | we even have a test case that would prevent you to introduce a new legacy notification :) | |
| 13:32:29 | opendevreview | Bence Romsics proposed openstack/nova master: Add limitation to docs about bug 1983570 https://review.opendev.org/c/openstack/nova/+/852168 | |
| 13:45:03 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds check for VM snapshot fail while quiesce https://review.opendev.org/c/openstack/nova/+/852171 | |
| 13:56:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add more test coverage for devname base dev spec https://review.opendev.org/c/openstack/nova/+/844625 | |
| 13:56:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extra tests for remote managed dev spec https://review.opendev.org/c/openstack/nova/+/844626 | |
| 13:56:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Unparent PciDeviceSpec from PciAddressSpec https://review.opendev.org/c/openstack/nova/+/844491 | |
| 13:56:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PciAddressSpec descendants to call super.__init__ https://review.opendev.org/c/openstack/nova/+/844565 | |
| 13:56:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove dead code from PhysicalPciAddress https://review.opendev.org/c/openstack/nova/+/844628 | |
| 13:56:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Clean up mapping input to address spec types https://review.opendev.org/c/openstack/nova/+/845765 | |
| 13:57:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove unused PF checking from get_function_by_ifname https://review.opendev.org/c/openstack/nova/+/845775 | |
| 13:57:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix type annotation of pci.Whitelist class https://review.opendev.org/c/openstack/nova/+/845780 | |
| 13:57:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move __str__ to the PciAddressSpec base class https://review.opendev.org/c/openstack/nova/+/845781 | |
| 14:02:40 | opendevreview | Merged openstack/nova master: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/848886 | |
| 14:02:48 | opendevreview | Merged openstack/nova master: Make test_wait_for_instance_event_* test time independent https://review.opendev.org/c/openstack/nova/+/845922 | |
| 15:31:54 | opendevreview | Dan Smith proposed openstack/nova master: Remove system scope from all APIs https://review.opendev.org/c/openstack/nova/+/848021 | |
| 15:31:54 | opendevreview | Dan Smith proposed openstack/nova master: Keep legacy admin behaviour in new RBAC https://review.opendev.org/c/openstack/nova/+/849209 | |
| 15:31:58 | dansmith | gmann: ^ | |
| 15:32:22 | gmann | #topic Open Reviews | |
| 15:32:29 | gmann | #link https://review.opendev.org/q/projects:openstack/governance+is:open | |
| 15:34:40 | gmann | #endmeeting | |
| 15:34:45 | gmann | ohh :) | |
| 15:35:38 | gmann | sorry, I used wrong channel. it was meant for TC channel | |
| 17:00:39 | opendevreview | Jan Hartkopf proposed openstack/nova master: add support for updating server's user_data https://review.opendev.org/c/openstack/nova/+/816157 | |
| 17:41:17 | melwitt | kashyap, stephenfin, sean-k-mooney[m]: I saw that someone else has proposed a patch for the live migration "no ram info was set" issue https://review.opendev.org/c/openstack/nova/+/852002 fyi if you might prefer that approach | |
| 18:36:07 | sean-k-mooney[m] | did i review that before. | |
| 18:36:16 | sean-k-mooney[m] | it sound vaguly familar | |
| 18:37:15 | sean-k-mooney[m] | https://review.opendev.org/c/openstack/nova/+/842687 is your version right | |
| 18:38:03 | sean-k-mooney[m] | i was previously +2 on your version and stephen currently is so i would prefer ot proceed with yours unless there is a reason not too | |
| 18:40:31 | sean-k-mooney[m] | they are both simialr and your patch has a release note so ill approve that now | |
| 18:44:03 | sean-k-mooney[m] | ok with that im calling it a day o/ catch ye all tomrrow | |
| 19:03:36 | melwitt | sean-k-mooney[m]: cool, thanks. just wanted to show yall in case there was a preference. seeya tomorrow o/ | |
| 19:56:47 | opendevreview | melanie witt proposed openstack/nova master: Workaround qemu query-migrate issue during live migration https://review.opendev.org/c/openstack/nova/+/842687 | |
| #openstack-nova - 2022-08-05 | |||
| 02:15:20 | opendevreview | anguoming proposed openstack/placement master: Remove unicode literal strings https://review.opendev.org/c/openstack/placement/+/845531 | |
| 03:38:42 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/851337 | |
| 06:05:19 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/852235 | |
| 07:12:45 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/852235 | |
| 07:12:46 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/852238 | |
| 08:27:49 | opendevreview | Alfredo Moralejo proposed openstack/nova stable/wallaby: Adapt to SQLAlchemy 1.4 https://review.opendev.org/c/openstack/nova/+/852099 | |
| 08:32:27 | kashyap | melwitt: Hi, looking | |
| 11:19:32 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove workaround for SQLAlchemy < 1.4 https://review.opendev.org/c/openstack/nova/+/852255 | |
| 13:05:01 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds img_config_drive in nova users docs https://review.opendev.org/c/openstack/nova/+/852265 | |
| 14:36:10 | opendevreview | Merged openstack/placement master: Remove unicode literal strings https://review.opendev.org/c/openstack/placement/+/845531 | |
| 14:59:37 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Fix a fixture for keypairs tests https://review.opendev.org/c/openstack/python-novaclient/+/852274 | |
| 15:33:30 | opendevreview | Takashi Natsume proposed openstack/nova master: doc: Update a PTL guide https://review.opendev.org/c/openstack/nova/+/852275 | |
| 16:29:26 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |