| opendevreview | sean mooney proposed openstack/nova master: doc: Add guide for NVIDIA vGPU with SR-IOV variant drivers https://review.opendev.org/c/openstack/nova/+/994160 | 01:38 |
|---|---|---|
| opendevreview | sean mooney proposed openstack/nova master: libvirt: Support Cyborg PCI ARQ managed mode https://review.opendev.org/c/openstack/nova/+/994579 | 01:38 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 02:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | 03:07 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | 03:07 |
| opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | 03:11 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/nova/+/986946 | 03:16 |
| opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | 03:17 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | 03:17 |
| *** ykarel_ is now known as ykarel | 04:24 | |
| ykarel | clarkb, thx for reminding that, i had completely forgot about that :) | 04:25 |
| ykarel | Will check on this in next few days and see how it can be unblocked | 04:25 |
| ykarel | will start with the impacted job and see | 04:26 |
| opendevreview | Seyeong Kim proposed openstack/nova master: Avoid serializing orphaned instance in log https://review.opendev.org/c/openstack/nova/+/992574 | 06:44 |
| *** raukadah is now known as chandankumar | 11:07 | |
| opendevreview | Stephen Finucane proposed openstack/os-resource-classes master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-resource-classes/+/994678 | 11:22 |
| opendevreview | Stephen Finucane proposed openstack/osc-placement master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/osc-placement/+/994679 | 11:22 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: pre-commit: Bump versions https://review.opendev.org/c/openstack/os-vif/+/994676 | 11:22 |
| opendevreview | Stephen Finucane proposed openstack/os-traits master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-traits/+/994680 | 11:22 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-vif/+/994677 | 11:22 |
| opendevreview | Merged openstack/os-vif master: pre-commit: Bump versions https://review.opendev.org/c/openstack/os-vif/+/994676 | 11:49 |
| opendevreview | Stephen Finucane proposed openstack/osc-placement master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/osc-placement/+/994679 | 12:49 |
| opendevreview | Elod Illes proposed openstack/python-novaclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-novaclient/+/994731 | 13:22 |
| opendevreview | Merged openstack/osc-placement master: Update support python versions https://review.opendev.org/c/openstack/osc-placement/+/991603 | 13:27 |
| opendevreview | Kamil Sambor proposed openstack/nova master: Add concurrency_backend config option for Nova services https://review.opendev.org/c/openstack/nova/+/993760 | 14:05 |
| Uggla | Note: I could not do the upstream triage this week. So cancelling it. | 14:21 |
| gibi | Uggla: ack | 14:21 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | 15:12 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect AMD SEV-SNP support https://review.opendev.org/c/openstack/nova/+/994764 | 15:12 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | 16:03 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | 16:04 |
| opendevreview | Merged openstack/os-vif master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-vif/+/994677 | 16:37 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | 16:38 |
| melwitt | sean-k-mooney: got the WIP ceph ipv6 passing and put it on top of the RBD live migration XML patch. the majority of the time was fighting with ipv6 brackets 🙄 the whole collection, if you're curious https://review.opendev.org/q/topic:%22ipv6-multinode%22 | 16:39 |
| sean-k-mooney | ah so you needed the know host request to allow the live migraiton to complete | 16:40 |
| sean-k-mooney | i wonder if it woul be worth having devstack or the devstack base job generate a tiny ~/.ssh/config file that auto accpate new hosts | 16:41 |
| sean-k-mooney | https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/994583 is more correct in general but we dont really need to validate it in general for ci | 16:41 |
| sean-k-mooney | basiclly we could also do | 16:43 |
| sean-k-mooney | Host * | 16:43 |
| sean-k-mooney | StrictHostKeyChecking accept-new | 16:43 |
| sean-k-mooney | or set it to no | 16:43 |
| melwitt | sean-k-mooney: I think known hosts was needed for cold migration rather than live. for a time I had live migration working but cold was broken due to lack of known hosts | 16:44 |
| sean-k-mooney | its used for both | 16:44 |
| sean-k-mooney | we dont use tls by defult in ci so we default to live mgiration over ssh | 16:44 |
| melwitt | ok. I am not sure what combo of things made it so that live would work but cold not but I had that state for a bit until I fixed it | 16:44 |
| sean-k-mooney | but your current patch is valid and more correct to refesh the know host after we create teh bridge | 16:45 |
| melwitt | yeah it seemed to make sense | 16:46 |
| sean-k-mooney | melwitt: so i guess the next step would be to rebase https://review.opendev.org/c/zuul/zuul-jobs/+/994521 on top of https://review.opendev.org/c/zuul/zuul-jobs/+/959393 | 16:46 |
| melwitt | yeah, good point, I think ykarel said he might be able to pick up that patch again | 16:48 |
| opendevreview | Merged openstack/osc-placement master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/osc-placement/+/994679 | 16:48 |
| opendevreview | Merged openstack/os-traits master: zuul: Use openstack-python3-next-jobs template https://review.opendev.org/c/openstack/os-traits/+/994680 | 16:49 |
| melwitt | I cc myself | 16:49 |
| melwitt | sean-k-mooney: note that the zuul-jobs refresh is not actually used in the openstack case bc the openstack-zuul-jobs playbook is used instead. but I put in both places for consistency | 16:50 |
| melwitt | apparently if openstack-zuul-jobs did not override with its own playbook the base zuul-jobs playbook would be used | 16:51 |
| sean-k-mooney | ah right i guess we were workign around a specific issue in our own ci env at somepoint | 16:52 |
| melwitt | and I did note yalls comments that it's likely unnecessary or undesirable to make the ipv6 setup optional in zuul-jobs but I was wary of it breaking anything if there's something wrong with it that I'm not aware of | 16:52 |
| sean-k-mooney | i woudl allow it to be optional i just proably would have turned it on by defualt | 16:53 |
| melwitt | on the other hand making it always on would shake out issues better and also prevent new potential issues better. so either way is cool I think, just mentioning why I didn't change that right away | 16:53 |
| sean-k-mooney | but im to overried that in our jobs in devstack | 16:53 |
| melwitt | gotcha | 16:54 |
| sean-k-mooney | we can either enable it here https://github.com/openstack/devstack/blob/master/.zuul.yaml#L378 or here https://github.com/openstack/devstack/blob/master/.zuul.yaml#L741 | 16:55 |
| sean-k-mooney | i think you alray have that in the nova? job right it just addign the flag to the vars | 16:56 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | 16:58 |
| melwitt | sean-k-mooney: yeah just added to vars https://review.opendev.org/c/openstack/nova/+/984623/26/.zuul.yaml | 17:06 |
| opendevreview | Ghanshyam Maan proposed openstack/placement master: Update policy scope enforcement testing https://review.opendev.org/c/openstack/placement/+/987568 | 17:08 |
| sean-k-mooney | melwitt: so im ok with the fix patch if your ok with https://review.opendev.org/c/openstack/nova/+/993597/4/nova/tests/functional/regressions/test_bug_1741364.py then im ok to upgrade to +2 | 17:19 |
| opendevreview | Ghanshyam Maan proposed openstack/placement master: Remove disabling the scope enforecment https://review.opendev.org/c/openstack/placement/+/994790 | 17:28 |
| gmaan | stephenfin: updated the enforce_scope removal changes for placement, separating the upgrade impact release notes with mentioning of oslo.policy new version with updating the same in requirements.txt. let me know if that works for you, holding the nova update till then https://review.opendev.org/c/openstack/placement/+/994790 | 17:33 |
| melwitt | sean-k-mooney: I think it's a good point about the base class, I had missed the weirdness with imagebackend | 17:41 |
| melwitt | I added a comment | 17:41 |
| sean-k-mooney | i dont really have an issue using that as a base normally but when it reuqired use to mock or in this case unmock a mock i hesitate | 17:42 |
| sean-k-mooney | melwitt: i remember gibi spending dime removing a bunch of double mocking in the past | 17:42 |
| melwitt | sean-k-mooney: yeah I think that's totally valid | 17:42 |
| sean-k-mooney | https://github.com/openstack/nova/commit/bf654e3a4a8f690ad0bec0955690bf4fadf98dba | 17:43 |
| sean-k-mooney | "Remove double mocking... again" | 17:43 |
| sean-k-mooney | :) | 17:43 |
| melwitt | heh | 17:44 |
| melwitt | it would be nice if we could detect those and prevent | 17:44 |
| sean-k-mooney | hum its python so anything is possible but it normally does not cause use problems | 17:45 |
| *** haleyb is now known as haleyb|out | 18:11 | |
| opendevreview | Ashish Gupta proposed openstack/nova master: tests: use file-backed Placement SQLite in functional threading https://review.opendev.org/c/openstack/nova/+/992581 | 23:54 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!