| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-18 | |||
| 08:14:00 | bauzas | either way, I also need to rebase my work on Uggla's unshelve-to-host API part | |
| 08:14:05 | sean-k-mooney | i jsut would not personally use it | |
| 08:14:19 | bauzas | sean-k-mooney: honestly, I need to consider the RFC | |
| 08:14:38 | sean-k-mooney | ok no rush | |
| 08:15:06 | sean-k-mooney | so are we agreeing on merging the unshelve-to-host change first | |
| 08:15:10 | bauzas | in theory, keynames are quite well defined as being the user@machine | |
| 08:15:12 | sean-k-mooney | to not have a microverion conflict | |
| 08:15:29 | bauzas | sean-k-mooney: yeah, I was about to propose an etherpad tomorrow at the nova meeting | |
| 08:15:37 | sean-k-mooney | ok cool | |
| 08:15:43 | bauzas | to identify quite a sequence of API changes | |
| 08:19:01 | Uggla | bauzas, fyi I'm trying to fix what I have broken in my local virtiofs patches, then I will review the unshelve comments from gibi and yours. | |
| 08:19:27 | gibi | Uggla: ack | |
| 08:22:54 | bauzas | Uggla: np, I'd like to test some crazy thing, being to use the 2.92 microversion *without* rebasing on your series | |
| 08:37:52 | sean-k-mooney | gibi: quick question you have a rather nice doc on ports with resocue request in the admin section | |
| 08:37:53 | sean-k-mooney | https://github.com/openstack/nova/blob/master/doc/source/admin/ports-with-resource-requests.rst | |
| 08:38:42 | sean-k-mooney | i was goign to add a similar doc to the reference section orgianlly for vdpa but now im reconsidering | |
| 08:38:52 | sean-k-mooney | do you think the admin section would be better | |
| 08:39:48 | sean-k-mooney | i think it proably is since admin sepam sotb be where we have vgpu and sev ectra | |
| 08:40:22 | sean-k-mooney | i should proably cross link it form https://github.com/openstack/nova/blob/master/doc/source/admin/networking.rst too | |
| 08:40:41 | gibi | it feels more like an adminy thing | |
| 08:41:12 | gibi | +1 on crosslinking | |
| 08:41:53 | sean-k-mooney | ok ill go work on that so. i think the first vdpa patch is basically ready to merge but stephen raise a good point about docs so im goign to work on that now and then push a new version | |
| 08:50:31 | gibi | ack | |
| 09:37:56 | bauzas | ok, yoga open specs abandoned | |
| 09:38:42 | bauzas | as I said in my email, I won't -2 yet https://review.opendev.org/c/openstack/nova-specs/+/849488 until next meeting | |
| 09:39:01 | bauzas | and I appreciate reviews of https://review.opendev.org/c/openstack/nova-specs/+/842015 | |
| 09:48:44 | Uggla | Could someone help me in the beginning of the afternoon. I have issues with 3 unit tests but can not figure out why. | |
| 09:49:51 | sean-k-mooney | proably have you posted the failing patch for review or the output of the tox results somewhere | |
| 09:53:13 | Uggla | sean-k-mooney, not yet | |
| 09:53:49 | Uggla | sean-k-mooney, doing it right now | |
| 09:54:55 | Uggla | sean-k-mooney, this is the tests issues: https://paste.openstack.org/show/bkFrl9uor71V8QBjPcF1/ | |
| 09:55:40 | Uggla | sean-k-mooney, code is here: https://github.com/uggla/nova/tree/bp/manila_shares_attachments | |
| 09:57:01 | Uggla | It seems something is wrong with the placement API. | |
| 10:07:23 | sean-k-mooney | those dont sound like they shoudl be related to your patches | |
| 10:07:36 | sean-k-mooney | you might want to rebase your patches to the tip of master and see if they go away | |
| 10:08:35 | gibi | yepp the schema validation error might be related to the recent bump on jsonschema | |
| 10:08:39 | gibi | and that is fixed on master | |
| 10:09:45 | sean-k-mooney | recent as in merged on friday i think | |
| 10:16:58 | Uggla | sean-k-mooney, I think about it and rebase. But it is still present. | |
| 10:18:55 | sean-k-mooney | try add -r | |
| 10:19:04 | sean-k-mooney | to have tox recreate teh venv | |
| 10:19:19 | sean-k-mooney | ill try running it myself with master shortly | |
| 10:19:25 | sean-k-mooney | just getting something to drink | |
| 10:43:04 | gibi | Uggla: I pulled your branch but the unit tests, you see failing, are passing for me locally | |
| 10:45:37 | gibi | (I added the os_trats change manually to the venv) | |
| 10:49:40 | sean-k-mooney | doing the same currently | |
| 10:51:07 | sean-k-mooney | ====== | |
| 10:51:10 | sean-k-mooney | Totals | |
| 10:51:12 | sean-k-mooney | ====== | |
| 10:51:14 | sean-k-mooney | Ran: 16546 tests in 76.0306 sec. | |
| 10:51:16 | sean-k-mooney | - Passed: 16487 | |
| 10:51:18 | sean-k-mooney | - Skipped: 58 | |
| 10:51:20 | sean-k-mooney | - Expected Fail: 1 | |
| 10:51:22 | sean-k-mooney | - Unexpected Success: 0 | |
| 10:51:24 | sean-k-mooney | - Failed: 0 | |
| 10:51:26 | sean-k-mooney | Sum of execute time for each test: 3190.1976 sec. | |
| 10:51:45 | sean-k-mooney | so ya with clean envs if i install the os-traits repo into the env | |
| 10:51:50 | sean-k-mooney | then it seams to work fine | |
| 10:52:00 | sean-k-mooney | at least on ubuntu 22.04 | |
| 10:52:37 | sean-k-mooney | Uggla: if that is the only issue you have and you have fixed up any other patch comments i would push it and see if the ci complains | |
| 11:27:03 | sean-k-mooney | sfinucan: can you swap the order of the first two patches https://review.opendev.org/c/openstack/nova/+/850006/1 | |
| 11:27:25 | sean-k-mooney | the dataclasses dep seams to be causing issues on the first patch | |
| 11:29:39 | sfinucan | sure | |
| 11:30:35 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unused requirement https://review.opendev.org/c/openstack/nova/+/850006 | |
| 11:30:35 | opendevreview | Stephen Finucane proposed openstack/nova master: Fix compatibility with jsonschema 4.x https://review.opendev.org/c/openstack/nova/+/849867 | |
| 11:30:36 | opendevreview | Stephen Finucane proposed openstack/nova master: Bump jsonschema minimum to 4.0.0 https://review.opendev.org/c/openstack/nova/+/850021 | |
| 11:30:43 | sfinucan | sean-k-mooney: done | |
| 11:30:47 | sean-k-mooney | thanks | |
| 11:30:53 | sean-k-mooney | ill read my +2 to those | |
| 11:31:02 | sean-k-mooney | *readd | |
| 11:32:17 | sean-k-mooney | stephenfin: can you weigh in on https://review.opendev.org/c/openstack/os-traits/+/844336/1/os_traits/compute/viommu.py#24 by the way | |
| 11:32:33 | sean-k-mooney | do we need an auto trait? it might be nice but im not sure we do | |
| 11:33:12 | sean-k-mooney | it would simplfy the prefileter so im inclined ot leave it but drop none | |
| 11:38:47 | opendevreview | Merged openstack/os-traits master: Add 'COMPUTE_STORAGE_VIRTIO_FS', 'COMPUTE_MEM_BACKING_FILE' https://review.opendev.org/c/openstack/os-traits/+/832769 | |
| 11:40:30 | sean-k-mooney | Uggla: ill propose a release of os-traits later this week there is one other patch i want to land | |
| 11:41:08 | sean-k-mooney | but that will allow you to use it form pypi once we have done the release and unblock the nova series form merging. | |
| 11:43:09 | sean-k-mooney | well its still WIP but more it will unblock the unit/functional tests form passign in the gate | |
| 11:43:28 | Uggla | sean-k-mooney, cool thx | |
| 11:46:55 | opendevreview | sean mooney proposed openstack/nova master: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/832330 | |
| 11:51:39 | opendevreview | sean mooney proposed openstack/nova master: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/832330 | |
| 11:56:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: Gracefully ERROR in _init_instance if vnic_type changed https://review.opendev.org/c/openstack/nova/+/850003 | |
| 11:58:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: Gracefully ERROR in _init_instance if vnic_type changed https://review.opendev.org/c/openstack/nova/+/850003 | |
| 12:01:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move uwsgi-gmr reno to the proper place https://review.opendev.org/c/openstack/nova/+/850152 | |
| 12:22:31 | Uggla | gibi, just get a new fresh repo from my code and still having the unit tests failing.... becoming crazy... | |
| 12:23:00 | sean-k-mooney | what version of python are you using and what os | |
| 12:23:43 | Uggla | 3.10.5 | |
| 12:23:52 | Uggla | Fedora 36 | |
| 12:23:59 | sean-k-mooney | i see well that could be the issue | |
| 12:24:06 | sean-k-mooney | 3.10 support is experimental | |
| 12:24:18 | sean-k-mooney | ill see if i get the same issue with 3.10 | |
| 12:24:38 | sean-k-mooney | oh hum | |
| 12:24:43 | sean-k-mooney | i was using 3.10.4 | |
| 12:24:52 | Uggla | but within tox 3.9 is used no ? | |
| 12:25:13 | sean-k-mooney | only if you run tox -e py39 | |
| 12:25:52 | sean-k-mooney | i ran with 3.10.4 so its not 3.10 | |
| 12:26:07 | Uggla | yep this is what I run. | |
| 12:26:44 | sean-k-mooney | well this seams to work on ubuntu so it might be a fedora dep issue | |
| 12:26:56 | sean-k-mooney | although im not sure what would cause that to fail | |
| 12:30:21 | Uggla | sean-k-mooney, it seems it happened to me on Friday after a rebase to split patches... I thought I did something wrong... | |
| 13:07:27 | fungi | sean-k-mooney: did you see my question on https://launchpad.net/bugs/1981813 (just trying to confirm the bug type change was intentional before i add an ossa bugtask)? | |