| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-25 | |||
| 12:45:56 | gibi | good question | |
| 12:45:58 | gibi | looking | |
| 12:46:44 | gibi | we merged https://review.opendev.org/c/openstack/placement/+/851966 that changed the testing strategy | |
| 12:47:31 | gibi | so the new os-traits should not break the placement tests | |
| 12:47:40 | sean-k-mooney | cool | |
| 12:48:06 | gibi | release patch is up https://review.opendev.org/c/openstack/releases/+/854617 | |
| 12:50:53 | sean-k-mooney | hum it proably coudl have been 2.8.1 but im ok with this i guess | |
| 12:52:31 | gibi | it is not a bugfix | |
| 12:52:55 | sean-k-mooney | ya but adding a constnat is not really a feature either | |
| 12:53:01 | sean-k-mooney | im +1 on it in any case | |
| 12:53:10 | sean-k-mooney | so the bot shoudl addd the ptl approved flag soon | |
| 12:53:15 | gibi | thanks | |
| 12:53:32 | sean-k-mooney | feature bumps are what we did in the past anyway | |
| 12:53:41 | sean-k-mooney | so your being consitnet with that | |
| 13:13:27 | gibi | ... it tells a lot that our func test helper mixin does help testing server multicreate :) | |
| 13:13:43 | gibi | but it wont stop me | |
| 13:14:09 | gibi | that is the last item on the PCI bucket list (as revert resize and rescheduler seems to work) | |
| 13:38:41 | sean-k-mooney | ack. i think punting multi create is vaild given the know issue there if it comes to that | |
| 13:39:08 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix rescue volume-based instance https://review.opendev.org/c/openstack/nova/+/852737 | |
| 13:47:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Test reschedule with PCI in placement https://review.opendev.org/c/openstack/nova/+/854626 | |
| 13:47:53 | gibi | sean-k-mooney: ack, I give it a try but I will limit the effort | |
| 13:50:48 | sean-k-mooney | given its an exsiting limiation for ports with resouce request or any other usage of nested resouce providers i think its accpatble to adress this genericaly in a future release. | |
| 13:51:06 | sean-k-mooney | and posibly even backport it as its a know bug/limiation depending on what the fix looks like | |
| 13:53:05 | opendevreview | Arnaud Morin proposed openstack/nova master: Allow plugging port when device_id is instance_id https://review.opendev.org/c/openstack/nova/+/854627 | |
| 13:53:33 | sean-k-mooney | so hard no on ^ | |
| 13:53:35 | sean-k-mooney | #right | |
| 13:54:18 | amorin | hello | |
| 13:54:30 | sean-k-mooney | o/ | |
| 13:54:40 | sean-k-mooney | amorin: we do not want to supprot that workflow | |
| 13:54:51 | sean-k-mooney | its not a bug or oversite | |
| 13:54:54 | amorin | I am talking about this on neutron side | |
| 13:54:55 | sean-k-mooney | its intentoally not supproted | |
| 13:55:42 | amorin | why so? | |
| 13:56:03 | sean-k-mooney | for one it wont work for sriov type ports | |
| 13:56:17 | sean-k-mooney | it also wont populate teh virtual interfaces table correctly | |
| 13:56:20 | sean-k-mooney | in the nova cell db | |
| 13:56:37 | sean-k-mooney | and it has the potential to bypass some of our current polciy seeitngs | |
| 13:57:11 | sean-k-mooney | amorin: so this is not supproted the only way to attach or detach an inteface form a nova instance is via the nova api | |
| 13:57:14 | amorin | actually, I mostly agree with you, but we will need to talk with neutron then | |
| 13:57:33 | sean-k-mooney | yes we need to make the device_id field admin only | |
| 13:57:40 | sean-k-mooney | it will be a topic at the next ptg | |
| 13:57:42 | amorin | agree | |
| 13:57:44 | amorin | ok nice | |
| 13:57:53 | amorin | is it officially in the agenda? | |
| 13:58:10 | sean-k-mooney | am i dont think we have an adgenda for the corss project sessions yet | |
| 13:58:20 | sean-k-mooney | if not feel free to add it | |
| 13:58:37 | sean-k-mooney | gibi: do we have an etherpad yet | |
| 13:58:40 | amorin | perfect, is the etherpad already created? | |
| 13:58:48 | gibi | sean-k-mooney: good question | |
| 13:59:16 | sean-k-mooney | we had https://etherpad.opendev.org/p/nova-ptg-columbus-ops-presence | |
| 13:59:24 | sean-k-mooney | but i dont think we create the nova one yet | |
| 13:59:29 | gibi | I don't find it | |
| 13:59:38 | gibi | so I guess you are right we haven't created one yert | |
| 13:59:55 | sean-k-mooney | amorin: we did not know the name of the release when sylvain went on PTO | |
| 14:00:00 | sean-k-mooney | so i dont think we have one yet | |
| 14:00:08 | amorin | ack, no hurries here | |
| 14:00:19 | amorin | we can keep the review open until we fill the PTG etherpad then? | |
| 14:04:51 | sean-k-mooney | sure | |
| 14:05:04 | sean-k-mooney | i also added some more context to my -2 | |
| 14:05:16 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/854627/1#message-f3f27942e7252b5c074a3e95b2d09521d5187d3e | |
| 14:05:25 | sean-k-mooney | so that shoudl help set the stage for this with neutron | |
| 14:06:43 | amorin | ack, will go tomorrow on neutron-driver with your comment as argument | |
| 14:10:18 | opendevreview | Takashi Natsume proposed openstack/nova master: Add missing descriptions in HACKING.rst https://review.opendev.org/c/openstack/nova/+/853054 | |
| 16:08:02 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix rescue volume-based instance https://review.opendev.org/c/openstack/nova/+/852737 | |
| 16:22:40 | opendevreview | Takashi Natsume proposed openstack/nova master: Add a hacking rule for the setDaemon method https://review.opendev.org/c/openstack/nova/+/854653 | |
| 16:57:24 | gibi | sean-k-mooney: re multi create: I think it works with PCI in placement. I had to fix how we generate the Selection objects as they were not aware of the fact the the scheduler already allocated some resources in placement. | |
| 16:58:12 | gibi | the re-used the same allocation request | |
| 16:58:22 | gibi | *the selection generation | |
| 16:59:35 | gibi | anyhow I think it works now | |
| 16:59:45 | gibi | so I'm happy :) | |
| 17:21:32 | sean-k-mooney | ok thats good news | |
| 17:23:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Test reschedule with PCI in placement https://review.opendev.org/c/openstack/nova/+/854626 | |
| 17:23:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support multi create with PCI in placement https://review.opendev.org/c/openstack/nova/+/854663 | |
| 17:43:59 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 17:44:00 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 17:44:00 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 17:44:01 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 17:44:01 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 17:44:02 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 17:44:02 | opendevreview | ribaudr proposed openstack/nova master: Bump compute version and check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 17:44:03 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 17:44:04 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 17:44:04 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 17:44:06 | opendevreview | ribaudr proposed openstack/nova master: Add instance.power_off_error notification https://review.opendev.org/c/openstack/nova/+/852278 | |
| 17:44:06 | opendevreview | ribaudr proposed openstack/nova master: Add instance.power_on_error notification https://review.opendev.org/c/openstack/nova/+/852084 | |
| 17:44:08 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 17:44:08 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | |
| 17:44:10 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.93 https://review.opendev.org/c/openstack/nova/+/852088 | |
| 17:44:10 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 18:06:30 | opendevreview | Merged openstack/nova master: Trigger reschedule if PCI consumption fail on compute https://review.opendev.org/c/openstack/nova/+/853611 | |
| 22:33:24 | opendevreview | Merged openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 22:54:04 | opendevreview | Merged openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| #openstack-nova - 2022-08-26 | |||
| 04:42:59 | opendevreview | Merged openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 04:46:37 | opendevreview | Merged openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 04:51:14 | opendevreview | Merged openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 04:58:45 | opendevreview | Merged openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 04:58:52 | opendevreview | Merged openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 04:59:43 | opendevreview | Merged openstack/nova master: Stop if tracking is disable after it was enabled before https://review.opendev.org/c/openstack/nova/+/847009 | |
| 05:01:39 | opendevreview | Merged openstack/nova master: Move provider_tree RP creation to PciResourceProvider https://review.opendev.org/c/openstack/nova/+/850546 | |
| 05:30:10 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/854499 | |