Wednesday, 2022-08-10

*** dasm|off is now known as Guest16700:20
*** ministry is now known as __ministry03:19
opendevreviewOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/nova/+/85133703:57
opendevreviewchenwei proposed openstack/nova master: Remove unicode literal from code  https://review.opendev.org/c/openstack/nova/+/85253805:47
gokhanishi folks, because of lack disk space, my instances are paused. after increaing disk space, how can I start these instances? now vm state is active but power state is paused. 06:33
gibimelwitt: we can alway ask the infra folks to remove a -2 07:23
sean-k-mooney[m]gibi can you rebase https://review.opendev.org/c/openstack/nova/+/852295/1 and i guess the rest of the pci series on the patch to make py310 voting and also fix the mocking issues in that patch10:09
gibiI will do that as soon as the py310 voting patch lands10:10
sean-k-mooney[m]gokhanisi:  hard reboot is the simplest way10:10
sean-k-mooney[m]gokhanisi:  you might be able to call the pause and unpause api to do it too but really if you run out of disk you cant really resume form the api without somethign like a hard reboot or stop10:12
*** bhagyashris_ is now known as bhagyashris11:32
kashyapCan anyone put this stable/wallaby backport through? - https://review.opendev.org/c/openstack/nova/+/849610 ([nova/libvirt] Support for checking and enabling SMM when needed11:52
kashyap)11:52
sean-k-mooney[m]nope its still blocked by the xena backport12:06
sean-k-mooney[m]https://review.opendev.org/c/openstack/nova/+/84967612:06
sean-k-mooney[m]elodilles:  ^ if you have time to review or gibi  perhaps12:07
gibidone12:10
elodillesas i see you already managed it :)12:20
opendevreviewRajesh Tailor proposed openstack/nova master: Fix rescue volume-based instance  https://review.opendev.org/c/openstack/nova/+/85273712:58
opendevreviewMerged openstack/nova master: Remove double mocking... again  https://review.opendev.org/c/openstack/nova/+/85246413:37
opendevreviewMerged openstack/nova stable/xena: [nova/libvirt] Support for checking and enabling SMM when needed  https://review.opendev.org/c/openstack/nova/+/84967613:40
sean-k-mooney[m]i might as well say this here too. just an fyi ill be on PTO returning tuseday once i sign off this evening14:23
sean-k-mooney[m]gibi: melwitt_  i reviewed as much of the pci and encypted disk features as i have brain power for this morning14:23
sean-k-mooney[m]both look mostly good but ill take a look at them again when im back14:24
gibisean-k-mooney[m]: thank you. I really appreciate the effort14:24
gibiI worked on the basics of the scheduling part of the feature today. the pre_filter part was pretty simple, but I still have to do the allocation_candidte filtering in hardware.py and the allocation correlation in the claim code14:28
sean-k-mooney[m]ack on pre_filter they normally are. the filtering in hardware.py hopefully wont be too hard the correcation might be a little more  complicated14:31
sean-k-mooney[m]did we merge your chaching change yet by the way14:32
gibiwhich change you are referring to?14:32
sean-k-mooney[m]the perfromace imporement for the hardware module by caching which nodes we had already checked14:32
gibihm14:33
gibiI have to look14:33
gibinope14:33
gibiit is still open https://review.opendev.org/c/openstack/nova/+/84589614:33
sean-k-mooney[m]ok it would likely be good to land that before you have to refactor that14:34
sean-k-mooney[m]stephenfin: ^14:34
sean-k-mooney[m]its pretty simple if you have the time14:34
sean-k-mooney[m]i also need to adress your feedback on https://review.opendev.org/c/openstack/nova/+/84616914:35
sean-k-mooney[m]i was going to work on the vdpa patch before i went on pto but this is smaller and more doable today so ill do that now14:35
gibiack, ping me and I can re-review14:35
gibiand good point about the cache, I already forgot it 14:36
gibihm, the py310 voting patch merged so I will also do the rebase of the series 14:41
gibithen we can land at least the bugfix in front of it14:42
opendevreviewsean mooney proposed openstack/nova master: update default numa allocation strategy  https://review.opendev.org/c/openstack/nova/+/84616914:45
sean-k-mooney[m]yep14:45
opendevreviewMerged openstack/nova master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/nova/+/85133714:49
melwittgibi: oh true, I forgot about that14:54
melwittsean-k-mooney[m]: thank you, appreciated14:54
sean-k-mooney[m]melwitt:  since your about care to look at https://review.opendev.org/q/topic:bug%252F197837214:57
melwittsean-k-mooney[m]: sure, will look after the bug call14:58
sean-k-mooney[m]oh ya i proably shoud get something to drink before that14:58
sean-k-mooney[m]you know water totally just water14:59
opendevreviewBalazs Gibizer proposed openstack/nova master: Request filter for PCI in placement  https://review.opendev.org/c/openstack/nova/+/85277115:00
melwittsean-k-mooney[m]: :D15:00
opendevreviewMerged openstack/nova stable/wallaby: [nova/libvirt] Support for checking and enabling SMM when needed  https://review.opendev.org/c/openstack/nova/+/84961015:23
opendevreviewBalazs Gibizer proposed openstack/nova master: Reproducer for bug 1983753  https://review.opendev.org/c/openstack/nova/+/85229516:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Update RequestSpec.pci_request for resize  https://review.opendev.org/c/openstack/nova/+/85229616:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Poison /sys access via various calls in test  https://review.opendev.org/c/openstack/nova/+/84462716:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Add compute restart capability for libvirt func tests  https://review.opendev.org/c/openstack/nova/+/85051016:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Rename [pci]passthrough_whitelist to device_spec  https://review.opendev.org/c/openstack/nova/+/84383416:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec  https://review.opendev.org/c/openstack/nova/+/84386116:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Rename whitelist in tests  https://review.opendev.org/c/openstack/nova/+/84386216:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting  https://review.opendev.org/c/openstack/nova/+/84618716:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits  https://review.opendev.org/c/openstack/nova/+/84621816:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Reject PCI dependent device config  https://review.opendev.org/c/openstack/nova/+/84643516:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config  https://review.opendev.org/c/openstack/nova/+/84643616:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag  https://review.opendev.org/c/openstack/nova/+/84621916:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Reject devname based device_spec config  https://review.opendev.org/c/openstack/nova/+/84646616:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration  https://review.opendev.org/c/openstack/nova/+/84647016:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Stop if tracking is disable after it was enabled before  https://review.opendev.org/c/openstack/nova/+/84700916:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Move provider_tree RP creation to PciResourceProvider  https://review.opendev.org/c/openstack/nova/+/85054616:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Retry /reshape at provider generation conflict  https://review.opendev.org/c/openstack/nova/+/85135816:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Heal missing simple PCI allocation in the resource tracker  https://review.opendev.org/c/openstack/nova/+/85135916:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Heal PCI allocation during resize  https://review.opendev.org/c/openstack/nova/+/85239616:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Handle PCI dev reconf with allocations  https://review.opendev.org/c/openstack/nova/+/85239716:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement  https://review.opendev.org/c/openstack/nova/+/85046816:22
opendevreviewBalazs Gibizer proposed openstack/nova master: Request filter for PCI in placement  https://review.opendev.org/c/openstack/nova/+/85277116:22
sean-k-mooney[m]just one or two patches :)16:22
gibiyeah just 22 :)16:23
gibiin the main series16:23
gibibut now it is rebased as the double mock fix landed16:23
sean-k-mooney[m]it does make it more digestable16:23
sean-k-mooney[m]cool16:23
sean-k-mooney[m]i think the first few are ready to merge16:23
gibiI checked locally I don't have double mock in the first two patches (the bugfix)16:23
sean-k-mooney[m]ack16:23
gibiI will check the rest locally too, but I have to log off in 10 minutes16:24
gibiOK it seems to be the series is clean from double mocks (at least running it locally)16:33
gibisean-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 seriea17:21
sean-k-mooney[m]yep still here17:31
sean-k-mooney[m]done17:32
sean-k-mooney[m]ill +w some of the later ones shortly17:32
sean-k-mooney[m]for a second there i tought something was broken with how few jobs those are running17:41
sean-k-mooney[m]then i realsed the patch im looking at is test only17:42
sean-k-mooney[m]so it only runs the unit and functional tests not all of tempest17:42
sean-k-mooney[m]well and pep8 and docs17:42
sean-k-mooney[m]gibi:  everything before https://review.opendev.org/c/openstack/nova/+/846187/18 should be now making its way through the gate17:45
sean-k-mooney[m]so thats the pci whitelist rename part17:45
sean-k-mooney[m]and the bug fix at the start17:46
sean-k-mooney[m]o/17:47
opendevreviewMerged openstack/nova master: Reproducer for bug 1983753  https://review.opendev.org/c/openstack/nova/+/85229518:06
gibisean-k-mooney[m]: thank you18:25
opendevreviewMerged openstack/nova master: Update RequestSpec.pci_request for resize  https://review.opendev.org/c/openstack/nova/+/85229619:47
opendevreviewMerged openstack/nova master: Poison /sys access via various calls in test  https://review.opendev.org/c/openstack/nova/+/84462719:47
opendevreviewMerged openstack/nova master: Add compute restart capability for libvirt func tests  https://review.opendev.org/c/openstack/nova/+/85051019:47
opendevreviewMerged openstack/nova master: Rename [pci]passthrough_whitelist to device_spec  https://review.opendev.org/c/openstack/nova/+/84383420:13
opendevreviewMerged openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec  https://review.opendev.org/c/openstack/nova/+/84386120:13
opendevreviewMerged openstack/nova master: Rename whitelist in tests  https://review.opendev.org/c/openstack/nova/+/84386220:13
opendevreviewMerged openstack/nova master: update default numa allocation strategy  https://review.opendev.org/c/openstack/nova/+/84616921:21
opendevreviewmelanie witt proposed openstack/nova master: Fix exception catch when volume mount fails  https://review.opendev.org/c/openstack/nova/+/85279523:30

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!