| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-10-13 | |||
| 12:48:46 | gibi | sean-k-mooney: seems test_description_errors can fail and leave running greenthreads behind: https://paste.opendev.org/show/809966/ | |
| 12:49:00 | gibi | I mean can produce an error without the testcase failing | |
| 12:49:09 | gibi | then leaking running greanthreads | |
| 12:49:10 | sean-k-mooney | i see | |
| 12:49:26 | sean-k-mooney | ok and i guess that can somehow impact other tests | |
| 12:50:02 | gibi | yeah it is still strange how these tests interact. there is a global state somewhere | |
| 12:50:17 | sean-k-mooney | could there somehow be sharing betwen the fake_impl for oslo.messaging | |
| 12:51:28 | sean-k-mooney | or the db fixture. it seams like somehost when those finally time out it trigres the db to be cleaned up | |
| 12:53:09 | sean-k-mooney | or i guess the notifcation fixture but really im just speculating | |
| 12:53:35 | gibi | I dig forward.. | |
| 12:53:39 | sean-k-mooney | at least we have something to investigate | |
| 12:54:50 | gibi | yeah | |
| 13:03:54 | opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | |
| 13:04:22 | opendevreview | Federico Ressi proposed openstack/nova master: Check Nova project changes with Tobiko scenario test cases https://review.opendev.org/c/openstack/nova/+/806853 | |
| 13:18:50 | kashyap | sean-k-mooney: I haven't checked the specs and the Git logs yet - do you recall what's the status of vDPA in upstream Nova? | |
| 13:19:33 | sean-k-mooney | we have partial support | |
| 13:19:42 | sean-k-mooney | we do not support any move operations | |
| 13:19:50 | sean-k-mooney | and we only support it for netwok devices | |
| 13:19:53 | kashyap | Ah, right - no move support | |
| 13:20:02 | sean-k-mooney | with ovs hardware offload | |
| 13:21:03 | sean-k-mooney | e.g. we do not support vdpa in legacy mode only in switchdev mode and the sriovnic_agent only support legacy mode | |
| 13:21:33 | sean-k-mooney | so you cant use vdpa with openstack without ovs as the contol plane using tcflower to instll flow into the nic | |
| 13:21:56 | sean-k-mooney | kashyap: was there a resaond you wanted to know | |
| 13:22:24 | sean-k-mooney | for our product it will be tech previwe in 17 until we can actully support at least some of the move ops | |
| 13:22:38 | kashyap | sean-k-mooney: Not for product - I was curious from an upstream PoV | |
| 13:23:03 | sean-k-mooney | i may add more support this cycle but hardware has been a limiting factor to working on this | |
| 13:23:04 | kashyap | The reason was just I was reading about vDPA and it reminded me of it | |
| 13:23:25 | sean-k-mooney | ack | |
| 14:23:33 | sean-k-mooney | does this look familar to anyone | |
| 14:23:35 | sean-k-mooney | https://paste.opendev.org/show/809977/ | |
| 14:23:46 | sean-k-mooney | ] GET /placement/resource_providers?in_tree=4838f356-678a-477c-a2b8-7a743ad0842c => generated 162 bytes in 5 msecs (HTTP/1.1 404) | |
| 14:24:11 | sean-k-mooney | but in the debug logs the path is wrong | |
| 14:24:12 | sean-k-mooney | 92.168.1.10 "GET /placemen//resource_providers?in_tree=4838f356-678a-477c-a2b8-7a743ad0842c" status: 404 | |
| 14:25:39 | sean-k-mooney | the apach2 config just has ProxyPass "/placement" "unix:/var/run/uwsgi/placement-api.socket|uwsgi://uwsgi-uds-placement-api/" retry=0 | |
| 14:26:41 | sean-k-mooney | this is causeing "/usr/local/bin/nova-status --config-file /etc/nova/nova.conf upgrade check" to fail which breaks devstack | |
| 14:26:57 | alecorps | hello | |
| 14:27:44 | alecorps | is someone can have a look to the PR 808791 ? | |
| 14:28:14 | sean-k-mooney | PR as in pull request | |
| 14:28:21 | gibi | sean-k-mooney: you have wrong config in you apache proxy | |
| 14:28:24 | gibi | sean-k-mooney: in devstack | |
| 14:28:32 | gibi | sean-k-mooney: searching for the commit... | |
| 14:28:49 | gibi | sean-k-mooney: https://review.opendev.org/c/openstack/devstack/+/811303 | |
| 14:28:51 | sean-k-mooney | gibi: i guess i need to pull devstack? | |
| 14:28:59 | gibi | probably yes | |
| 14:29:15 | gibi | you need the above fix is you have a new enough apache | |
| 14:29:24 | sean-k-mooney | oh ok | |
| 14:29:31 | sean-k-mooney | it removed the / on the socket | |
| 14:30:37 | sean-k-mooney | alecorps: what is PR 808791 | |
| 14:30:56 | alecorps | https://review.opendev.org/c/openstack/nova/+/808791 | |
| 14:31:00 | alecorps | yes pull request :) | |
| 14:31:06 | alecorps | sorry | |
| 14:31:15 | sean-k-mooney | alecorps: ok that is not a pull request | |
| 14:31:21 | sean-k-mooney | its a gerrit reivew | |
| 14:31:25 | alecorps | yes sorry | |
| 14:31:46 | sean-k-mooney | they are diffenet code review systems we used ot somtiem get PRs to the github mirror | |
| 14:32:11 | sean-k-mooney | ok so that is a new feature | |
| 14:32:22 | sean-k-mooney | dos it have a approve spec or blueprint | |
| 14:33:13 | sean-k-mooney | https://blueprints.launchpad.net/nova/+spec/vmware-fcd | |
| 14:33:40 | sean-k-mooney | ok so that is not approved currently | |
| 14:34:17 | sean-k-mooney | so ideally we need to review that at the next team meeting and either approve as a specless blueprint or request a spec if its invalice | |
| 14:34:33 | alecorps | ok, do i need to be there ? | |
| 14:34:34 | sean-k-mooney | it looks likke its self contained to the vmware code | |
| 14:34:52 | sean-k-mooney | it just need to be put on the adgenda | |
| 14:35:04 | sean-k-mooney | hovever we might not have next weeks meeting because of the ptg | |
| 14:35:07 | sean-k-mooney | so the meeting after | |
| 14:35:18 | alecorps | ok cool | |
| 14:35:25 | sean-k-mooney | i can take a look at it in the mean time but procetully its blocked until the paper work is done | |
| 14:35:34 | sean-k-mooney | you could just add it to the ptg adgenda too | |
| 14:35:39 | alecorps | understood, thanks | |
| 14:37:37 | alecorps | I iwll add it in the agenda | |
| 14:47:22 | alecorps | i added it in open discussion on the wiki | |
| 14:47:31 | alecorps | let me know if it's ok for you ? thanks | |
| 14:49:34 | sean-k-mooney | yep that shoudl be fine | |
| 14:50:04 | sean-k-mooney | if your going to attend the PTG next week you could also add it to https://etherpad.opendev.org/p/nova-yoga-ptg | |
| 14:50:31 | sean-k-mooney | otherwise we will likely cover it tuseday week as i expect next weeks meeing is cancled for the ptg | |
| 14:53:23 | gmann | gibi: ack, as i have removed the use of tox_extra_args now but yes in future if we use then this is something to note down. | |
| 14:54:16 | alecorps | thanks Sean, not sure for ptg, i will try | |
| 15:38:06 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move the implemented specs for the xena release https://review.opendev.org/c/openstack/nova-specs/+/812248 | |
| 15:39:01 | bauzas | gibi: stephenfin (if you're around) : approval for the xena implemented specs would be appreciated ^ | |
| 15:39:19 | bauzas | i just rebased my own change into brinzhang's change as he was missing a few bits | |
| 15:57:11 | stephenfin | bauzas: done | |
| 15:58:39 | bauzas | stephenfin: ta | |
| 15:59:06 | bauzas | sean-k-mooney: I moved your PTG point about rbac for novaclient during the rbac popup team meeting session we plan | |
| 15:59:26 | bauzas | but we can put it off this slot and just discuss between us | |
| 15:59:27 | sean-k-mooney | ok | |
| 16:00:07 | bauzas | or we can leave it there during the meeting and we don't have time to go thru it, we can try to find another time | |
| 16:00:15 | sean-k-mooney | well basically i think we shoudl be deprecating the novaclinet cli so i dont think we shoudl add the project id passthough feature to novaclinet for rbac | |
| 16:00:26 | bauzas | sean-k-mooney: sorry, I messed up your colors by pasting, btw. | |
| 16:00:36 | sean-k-mooney | its fine | |
| 16:00:46 | sean-k-mooney | move things as you see fit | |
| 16:04:43 | bauzas | I'm just doing a first round | |
| 16:05:07 | bauzas | probably something like "paperwork nova", then "general nova", then others | |
| 16:22:05 | opendevreview | Merged openstack/nova-specs master: Move the implemented specs for the xena release https://review.opendev.org/c/openstack/nova-specs/+/812248 | |
| 16:26:06 | stephenfin | melwitt: good spot on https://review.opendev.org/c/openstack/nova/+/812144 | |
| 16:31:19 | stephenfin | dansmith: So nova-compute doesn't access the DB and iirc, you're expected to upgrade all services at once, right? i.e. the N and N-1 in a single deployment only applies to nova-compute N-1 | |
| 16:31:51 | sean-k-mooney | stephenfin: nova-compute only accesses the db via the conductor | |
| 16:31:55 | dansmith | stephenfin: you're expected to upgrade all the non-compute services at once, yes, largely because of schema but not only for that | |
| 16:32:25 | stephenfin | Okay, so that being the case, why do we insist that DB columns are removed in a later cycle than the corresponding SQLAlchemy model fields? | |
| 16:32:44 | opendevreview | Merged openstack/nova-specs master: Re-propose Unified Limits in Nova https://review.opendev.org/c/openstack/nova-specs/+/809020 | |
| 16:32:45 | dansmith | so you can apply new schema before you roll any of that new code | |