| opendevreview | Merged openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | 00:52 |
|---|---|---|
| opendevreview | Merged openstack/nova master: Detect port-resource-request-groups neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | 00:52 |
| opendevreview | Merged openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | 00:52 |
| opendevreview | Merged openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | 00:53 |
| opendevreview | Merged openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | 01:35 |
| opendevreview | Merged openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | 01:35 |
| opendevreview | Merged openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | 01:35 |
| opendevreview | Merged openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | 01:35 |
| opendevreview | Merged openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | 01:35 |
| opendevreview | Merged openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | 01:35 |
| opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to unified limits https://review.opendev.org/c/openstack/nova/+/713498 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | 05:34 |
| opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | 05:34 |
| bauzas | good morning Nova | 07:03 |
| * bauzas is eventually released from his PTOs :p | 07:03 | |
| gibi | bauzas: welcome back | 07:04 |
| gibi | bauzas: have you managed to recharge your batteries? | 07:04 |
| gthiemonge | Hi Folks, do you know some recent issues with placement? the Octavia gates have been failing since the end of last week, with some exceptions in the placement-api logs: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d26/761195/19/check/octavia-v2-act-stdby-dsvm-scenario/d26bf59/controller/logs/screen-placement-api.txt | 07:05 |
| bauzas | gibi: let me see | 07:05 |
| bauzas | gibi: 80% :p | 07:05 |
| gibi | bauzas: that looks good :) | 07:05 |
| gthiemonge | logstash shows that it started on 2021-08-25 in many projects: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22placement.exception.ProjectNotFound%3A%20No%20such%20project%5C%22 | 07:06 |
| bauzas | gibi: you as well ? | 07:06 |
| bauzas | gthiemonge: looking | 07:07 |
| gibi | gthiemonge: looking | 07:07 |
| gibi | bauzas: sure, I had a nice weekend with a good book | 07:08 |
| gibi | gthiemonge: 2/3rd of the hits are actually successful job runs | 07:09 |
| gibi | gthiemonge: are you sure this is the root cause of the failed ones? | 07:09 |
| gthiemonge | gibi: yeah, for Octavia at least | 07:10 |
| gibi | gthiemonge: could you link an octavia failre please? | 07:10 |
| gthiemonge | gibi: yes sure | 07:12 |
| gthiemonge | octavia worker logs: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d26/761195/19/check/octavia-v2-act-stdby-dsvm-scenario/d26bf59/controller/logs/screen-o-cw.txt | 07:12 |
| bauzas | gibi: do you know why we persist the project UUIDs ? for the unified-limits or for the consumer types ? | 07:12 |
| gthiemonge | error is at Aug 30 13:07:06.582519 with a "{'code': 500, 'created': '2021-08-30T13:07:05Z', 'message': 'No valid host was found. There are not enough hosts available.'," | 07:13 |
| gibi | bauzas: we have projecti and user id for a log time in placement | 07:13 |
| gthiemonge | nova scheduler: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d26/761195/19/check/octavia-v2-act-stdby-dsvm-scenario/d26bf59/controller/logs/screen-n-sch.txt | 07:13 |
| gthiemonge | at Aug 30 13:07:04.212245 there's a Internal Server Error | 07:13 |
| gibi | bauzas: but I don't think placement uses it directly | 07:13 |
| gibi | bauzas: so possible yes, for unified limits so you can group usages per project | 07:14 |
| gibi | gthiemonge: thanks, looking | 07:14 |
| gthiemonge | gibi: if that helps: it fails on Octavia when we request 2 server creations at the same time, it doesn't fail when creating one server | 07:15 |
| gibi | gthiemonge: thanks, this will be a race | 07:15 |
| gibi | we changed transactional behavior of allocation update in https://review.opendev.org/q/topic:bp/support-consumer-types | 07:16 |
| bauzas | gibi: surely for unified limits, but l looked at the series and nothing merged yet | 07:17 |
| bauzas | gibi: yeah, hence my confusion : we merged stuff for consumer types but none of them touched the projects table | 07:18 |
| gibi | bauzas: here you can query usages per project https://docs.openstack.org/api-ref/placement/?expanded=list-usages-detail#list-usages | 07:18 |
| bauzas | oh now I remember | 07:19 |
| bauzas | gibi: gotcha, thanks | 07:19 |
| gibi | bauzas: but consumer_types changes how the allocation transaction handling is done | 07:19 |
| gibi | bauzas: before consumer_type we had actually two transactions per allocation update, after consumer_types we have one | 07:19 |
| gibi | the duplicate project entry (due to a race) makes the new single transaction session invalid and later operations are failed in the same transaction hence the sqlalchemy.exc.InvalidRequestError: This session is in 'inactive' state, due to the SQL transaction being rolled back; no further SQL can be emitted within this transaction. | 07:21 |
| gibi | that is my running assumption now | 07:21 |
| bauzas | yup, I agree | 07:22 |
| gibi | I have to go offline for ~2 hours | 07:23 |
| bauzas | I need to filter 100+ emails but I guess you found something | 07:23 |
| gibi | gthiemonge: could you please file a bug for placement about it? | 07:23 |
| gthiemonge | gibi: yes, thanks! | 07:23 |
| gibi | bauzas: https://github.com/openstack/placement/blob/48f31d446be5dd8743392e6d1e45ed8183a9ce1b/placement/handlers/util.py#L56 this is the first exception | 07:25 |
| gibi | and that is the second Aug 30 13:07:04.208104 nested-virt-ubuntu-focal-ovh-bhs1-0026183953 devstack@placement-api.service[89049]: ERROR placement.fault_wrap proj = project_obj.Project.get_by_external_id(ctx, project_id) | 07:26 |
| gibi | sorry wrong buffer | 07:26 |
| gibi | so the second https://github.com/openstack/placement/blob/48f31d446be5dd8743392e6d1e45ed8183a9ce1b/placement/handlers/util.py#L58 | 07:26 |
| gibi | this is strange I don't think consumer types changed this exact logic | 07:26 |
| gibi | https://review.opendev.org/c/openstack/placement/+/679441/25/placement/handlers/util.py#b65 yepp the logic is the same in the from state https://review.opendev.org/c/openstack/placement/+/679441/25/placement/handlers/util.py#b65 | 07:28 |
| gthiemonge | gibi: https://storyboard.openstack.org/#!/story/2009159 | 07:29 |
| gibi | but yes, the Project.create() and the Project.get_by_external_id was two separate transaction before the consumer_types | 07:30 |
| gibi | this is the start of the new and only transaction https://review.opendev.org/c/openstack/placement/+/679441/25/placement/handlers/allocation.py#424 | 07:30 |
| gibi | gthiemonge: thanks | 07:30 |
| gibi | right now I don't know how to solve this. as we need one big transaction but we should not stop due to a project creation race | 07:35 |
| gibi | maybe we need to re-try the whole transaction from the start if the race is caugth | 07:35 |
| gibi | and now I go offline, back around 11:30 CEST | 07:36 |
| opendevreview | Federico Ressi proposed openstack/nova master: Write requests failure details to log https://review.opendev.org/c/openstack/nova/+/806683 | 07:42 |
| opendevreview | Federico Ressi proposed openstack/nova master: [WIP] Investigate on Nova quota limits problem https://review.opendev.org/c/openstack/nova/+/806683 | 08:03 |
| *** whoami-rajat is now known as Guest5879 | 08:10 | |
| *** whoami-rajat__ is now known as whoami-rajat | 08:10 | |
| opendevreview | Merged openstack/nova master: docs: admin/networking rename neutron_tunneled to neutron_tunnel https://review.opendev.org/c/openstack/nova/+/806197 | 08:37 |
| *** akekane_ is now known as abhishekk | 09:02 | |
| * gibi is back | 09:37 | |
| gibi | bauzas, melwitt: I will try to add a reproduction test for https://storyboard.openstack.org/#!/story/2009159 | 09:46 |
| bauzas | ack | 09:47 |
| gibi | stephenfin, lyarwood: a quick re-review on the pps series would be much appreciated :) (starting here https://review.opendev.org/c/openstack/nova/+/800086 ) | 09:49 |
| lyarwood | ack done | 09:54 |
| kevinz | kashyap: sbauza: Hi, bother again :-) could you help to review this live migration patch?live migration on arm64 patch, https://review.opendev.org/c/openstack/nova/+/763928, the comments has been addressed. really appreciated! | 09:58 |
| kevinz | bauzas: ^^, thanks for reviewing | 10:01 |
| gibi | lyarwood: thanks | 10:02 |
| *** bhagyashris_ is now known as bhagyashris | 10:07 | |
| kashyap | kevinz: Hi | 10:13 |
| kashyap | kevinz: Just catching up after PTO; will look today | 10:14 |
| bauzas | kevinz: ditto as kashyap, I'm just back from a long period of inactivity | 10:23 |
| bauzas | this reminds me actually... core reviews would be appreciated for https://review.opendev.org/c/openstack/nova/+/802918 | 10:24 |
| bauzas | and the series | 10:24 |
| kevinz | kashyap: bauzas: Thanks! Really appreciated :-) | 10:30 |
| opendevreview | Merged openstack/nova master: nova-manage: Introduce volume show, refresh, get_connector commands https://review.opendev.org/c/openstack/nova/+/800634 | 10:54 |
| opendevreview | Merged openstack/placement master: Refactor consumer type methods for readability https://review.opendev.org/c/openstack/placement/+/806035 | 10:54 |
| opendevreview | Merged openstack/nova stable/wallaby: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803092 | 10:55 |
| gibi | /12 | 11:21 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add reproducer for Project creation race bug https://review.opendev.org/c/openstack/placement/+/806730 | 11:46 |
| gibi | melwitt, bauzas: ^^ it is the reproducer to the issue octavia folks reported today. I could not create a real race in the func test env so I cheated a but | 11:47 |
| gibi | bit | 11:47 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Address nits for configurable-instance-hostnames series https://review.opendev.org/c/openstack/nova/+/806735 | 12:22 |
| gibi | sean-k-mooney: what is the expected behavior of a sriov interface attach in an env where the PCI devices are part of numa0 but the instance cpus are part of numa1? | 12:24 |
| sean-k-mooney | it depend on if the vm has a numa toplogy and the numa policy | 12:25 |
| sean-k-mooney | if the vm has no numa toplogy its fine it should boot | 12:25 |
| sean-k-mooney | if it it has cpu pinning it should not boot unless you use pci_numa_toplogy=prefer | 12:25 |
| sean-k-mooney | which will prefer a host where affinity can be acive but not enforce it on the host | 12:26 |
| gibi | sean-k-mooney: what about pci_numa_toplogy=legacy? | 12:26 |
| sean-k-mooney | legacy means if the device report numa info its enforced but if it does not it allowed regardless of the cpu selected | 12:27 |
| sean-k-mooney | so legacy which is the default rquired numa affinity if we have numa info for the device and the cpus/memory | 12:27 |
| gibi | OK, so In my internal case the VM has topology due to cpu pinning and they not specify policy so they get the legacy one | 12:28 |
| sean-k-mooney | yep | 12:28 |
| gibi | and the pci device has numa info so the basically get strict affinity | 12:29 |
| gibi | with hw:pci_numa_affinity_policy=preferred they would get best effort affinity | 12:29 |
| sean-k-mooney | yep | 12:29 |
| sean-k-mooney | we have test for this in whitebox by the way | 12:29 |
| sean-k-mooney | https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/api/compute/test_sriov.py#L221-L432 | 12:29 |
| sean-k-mooney | we also have func tests for this i think in nova | 12:30 |
| sean-k-mooney | preffer is basically currenlty implemtned as a weigher | 12:30 |
| sean-k-mooney | i have a very long running todo to make prefer prefer numa nodes on the comptue too | 12:30 |
| sean-k-mooney | its a simple chagne just never got around to do it | 12:31 |
| sean-k-mooney | to implement prefer on the compute node we litrally just need an else on https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L2201-L2205 that sorts in teh reverse order | 12:32 |
| sean-k-mooney | gibi: for backwards compatiablity reasons we made legacy the default but prefer is arguable a more user friendly default | 12:33 |
| gibi | ack thanks | 12:33 |
| gibi | sean-k-mooney: do we have the per port numa affinity supported by nova? | 12:34 |
| gibi | i.e. the nova side of https://bugs.launchpad.net/neutron/+bug/1886798 | 12:34 |
| sean-k-mooney | yes | 12:34 |
| sean-k-mooney | i need to check which release but we do in wallaby | 12:34 |
| sean-k-mooney | maybe victoria | 12:34 |
| gibi | so for interface attach the user could set the policy on the port if the instace was booted without the flavor extra sepc | 12:35 |
| gibi | spec | 12:35 |
| sean-k-mooney | https://github.com/openstack/nova-specs/blob/master/specs/wallaby/implemented/port-scoped-sriov-numa-affinity.rst | 12:35 |
| gibi | cool, thanks | 12:35 |
| sean-k-mooney | ya i think we covered that usecase when you added sriov attach support too | 12:35 |
| sean-k-mooney | both feature were added in wallaby and im 99% sure that we pass the posicy when trying to claim the pci device on attach and will reject it if the numa affintiy cant be achived | 12:37 |
| sean-k-mooney | but you can relax the policy with the neutron port | 12:37 |
| sean-k-mooney | so the vm can default to legacy or strict and then you can add a port with prefer | 12:37 |
| gibi | cool. I do remember passing policy around in sriov attach impl | 12:39 |
| sean-k-mooney | yep we do https://github.com/openstack/nova/blob/649b2452da07c25e1251c6185552278f91b7c517/nova/compute/manager.py#L7788-L7794 | 12:40 |
| sean-k-mooney | hehe ya was just finding that in the code | 12:40 |
| sean-k-mooney | that gets the flavor/image policy then we override that with the port policy if set https://github.com/openstack/nova/blob/649b2452da07c25e1251c6185552278f91b7c517/nova/network/neutron.py#L2023-L2026 | 12:43 |
| sean-k-mooney | https://github.com/openstack/nova/blob/649b2452da07c25e1251c6185552278f91b7c517/nova/network/neutron.py#L2084 | 12:43 |
| sean-k-mooney | https://github.com/openstack/nova/blob/649b2452da07c25e1251c6185552278f91b7c517/nova/network/neutron.py#L2162-L2164 | 12:43 |
| sean-k-mooney | gibi: i rememebr discussing the precednece of these two during the spec review | 12:44 |
| opendevreview | sean mooney proposed openstack/nova master: Add 'hw:vif_multiqueue_enabled' flavor extra spec https://review.opendev.org/c/openstack/nova/+/792356 | 12:47 |
| opendevreview | sean mooney proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | 12:47 |
| opendevreview | sean mooney proposed openstack/nova master: Move 'hw:pmu', 'hw_pmu' parsing to nova.virt.hardware https://review.opendev.org/c/openstack/nova/+/792364 | 12:47 |
| sean-k-mooney | stephenfin: i have adressed my nits regarding the acidental cahnge form 403 to 500 and resolved the merge conflict otherwise this is as you left it | 12:48 |
| sean-k-mooney | i think those should be good to merge now if people have time to review | 12:48 |
| kashyap | kevinz: Looked at it; I see you've also done real tests with it. I don't see anything glaring in there. Also the conditional is AArch64 specific. Thx for reworking. | 12:59 |
| bauzas | sean-k-mooney: correct me if I'm wrong, but I see a last revision of generic mdevs that is only due for fixing pep8 mypy issues because we were not explicitely returning None ? | 13:02 |
| bauzas | plus the fact you moved the documentation bit out of the last change into a specific WIP change, right? | 13:02 |
| sean-k-mooney | bauzas: yes | 13:02 |
| sean-k-mooney | stephenfin: was ok with adressing the doc update in a seperate change so suggested we split it out to not block the feature | 13:03 |
| sean-k-mooney | since the doc update can happen after FF | 13:03 |
| sean-k-mooney | bauzas: effectivly what stephenfin would like is if we have a stuf file that explains vgpu are now implemented in terms of generic mdevs and then links to mdev doc | 13:04 |
| sean-k-mooney | bauzas: if we want to have specific vgpu docs we can keep them int vgpu doc | 13:04 |
| sean-k-mooney | bauzas: but ya i did not actully change any of your codes functionality form where you left it just the mypy update | 13:05 |
| bauzas | sean-k-mooney: no worries at all, my question was just for understanding the touched bits, and to make sure I wasn't missing crucial bits | 13:08 |
| bauzas | sean-k-mooney: thanks for having followed up while I was working on my tan | 13:09 |
| sean-k-mooney | i ment to get to it sooner but i did not see the pep8 failure until recently | 13:09 |
| bauzas | (which was a difficult effort as I discovered to having traveled to the worst raining place in France apparently) | 13:09 |
| bauzas | a perfect Irish weather for the first week | 13:10 |
| sean-k-mooney | haha we hit 27+ a few days while you were off | 13:13 |
| gibi | now that is perfect ^^ :) | 13:13 |
| kevinz | kashyap: Thanks a lot! Good to hear that! | 13:14 |
| sean-k-mooney | its been a pretty good summber all things considered in ireland this year at least on teh weather front | 13:14 |
| bauzas | sean-k-mooney: are you honestly saying I got an irish weather while you were getting a mediterrean weather ? | 13:15 |
| bauzas | life is unfair | 13:15 |
| sean-k-mooney | very much so | 13:15 |
| bauzas | pro-tip : we usually joke about french Britain rainy weather | 13:15 |
| bauzas | but please reconsider the basque country as more wet than britain :) | 13:16 |
| sean-k-mooney | with that said my tomatoes got hit by blight last week and are all starting to die but we still got some of the harvest | 13:16 |
| sean-k-mooney | its the result of having heat then lots of humitity for 2 and ah half weeks | 13:17 |
| bauzas | sean-k-mooney: hah, I'm jealous | 13:17 |
| bauzas | we got a ridiculous harvest this year for our tomatoes | 13:17 |
| bauzas | but, let me ask one question : any series to look at now besides kevinz's ask ? | 13:18 |
| sean-k-mooney | i think we have got like 2-3 kg form 20-25 plants which is not greate but i know some have just had there entire lot of plants wiped out | 13:18 |
| bauzas | gibi: I guess I'm all done for you with your pps series, right? | 13:18 |
| sean-k-mooney | bauzas: https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 13:18 |
| opendevreview | Federico Ressi proposed openstack/nova master: [WIP] Investigate on Nova quota limits problem https://review.opendev.org/c/openstack/nova/+/806683 | 13:19 |
| sean-k-mooney | i just put a few on the meeting adgenda | 13:19 |
| sean-k-mooney | bauzas: the review priortiy lable merged a few days ago so im suggeesting we start using that to track them | 13:19 |
| gibi | bauzas: I have my reviews for the pps so you can focus on other things | 13:19 |
| bauzas | sean-k-mooney: excellent | 13:20 |
| bauzas | gibi: that's the question I'm asking, I don't know what other things to do | 13:20 |
| opendevreview | Federico Ressi proposed openstack/nova master: [WIP] Investigate on Nova quota limits problem https://review.opendev.org/c/openstack/nova/+/806683 | 13:20 |
| bauzas | unified limits would get my interest, but we're two days from FF, and this is unfortunate I didn't had time to look at it before | 13:21 |
| gibi | bauzas: things in NeedsCodeReview state are a good candidates https://launchpad.net/nova/+milestone/xena-3 | 13:21 |
| bauzas | will find candidates, no worries | 13:21 |
| gibi | ack | 13:21 |
| sean-k-mooney | oh thats a good point | 13:22 |
| gibi | and now I remember to update the meeting agenda ... :D | 13:22 |
| opendevreview | Federico Ressi proposed openstack/nova master: [WIP] Investigate on Nova quota limits problem https://review.opendev.org/c/openstack/nova/+/806683 | 13:23 |
| opendevreview | Merged openstack/nova master: Provide the mdev class for every PCI device https://review.opendev.org/c/openstack/nova/+/802918 | 13:44 |
| opendevreview | Merged openstack/nova master: Provide and use other RCs for mdevs if needed https://review.opendev.org/c/openstack/nova/+/803233 | 13:44 |
| gibi | sean-k-mooney: thanks for the review priority links on the agenda. I move that to a new section right after the Release Planning topic | 13:47 |
| sean-k-mooney | gibi: ack that was also going to be somethign i brought up. shoudl it be in its own section so cool. | 13:48 |
| gibi | gmann: if you have time, could you please check back to the instance hostname series https://review.opendev.org/c/openstack/nova/+/778550 I think it is ready to land | 13:54 |
| gmann | gibi: sure, +w. | 14:03 |
| gibi | gmann: thanks, +1 feature for Xena \o/ | 14:03 |
| gmann | +1 | 14:03 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | 14:10 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | 14:11 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | 14:13 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Reno for qos-minimum-guaranteed-packet-rate https://review.opendev.org/c/openstack/nova/+/805046 | 14:13 |
| *** whoami-rajat__ is now known as whoami-rajat | 14:15 | |
| sean-k-mooney | stephenfin: just an fyi im going to rebase https://review.opendev.org/c/openstack/nova/+/669411 and adress gibi comment, this looks ok to me so should be an easy win | 14:30 |
| sean-k-mooney | ill see how hard it is to add that final check after binding | 14:31 |
| gibi | I only skimmed that patch, but if you make it fresh then I can try to look at the | 14:32 |
| sean-k-mooney | its most of the way there if it misses its not the end of the world but also its small and low risk so if we can get it done then its a nice to have | 14:32 |
| sean-k-mooney | im currently goign trhough the set of blueprint and patches that seam close and seeing if i can get them over the line | 14:33 |
| sean-k-mooney | but this is lower on my list of priorities | 14:33 |
| gibi | ack | 14:35 |
| opendevreview | Merged openstack/nova master: Remove module level caching https://review.opendev.org/c/openstack/nova/+/806394 | 14:52 |
| spatel | is anyone running DPDK on Intel X710 nic, i would like to buy so looking for some good feedback ? | 15:04 |
| spatel | currently i am using Intel® 82599 10 Gigabit Ethernet but it has limitation of rx/tx queue which is max=2 per VF | 15:10 |
| gibi | fyi nova weekly meeting starts in 10 minutes here in the channel | 15:50 |
| sean-k-mooney | spatel: x710 used ot be the primaly nic that intel used for dpdk development so its fully supported but its a few years old now and they have proably moveed tothe 800 seriese they lauched last year | 15:51 |
| sean-k-mooney | spatel: this is the dpdk docs for that nic https://doc.dpdk.org/guides/nics/i40e.html | 15:52 |
| spatel | do you know how many rx/tx queue does x710 support ? | 15:52 |
| spatel | x710 is older so may be cheaper for me to get it up and running with dpdk | 15:52 |
| sean-k-mooney | x710 is much much newer then 82599 | 15:52 |
| sean-k-mooney | but i know the queue count is actul configurable in the driver | 15:53 |
| sean-k-mooney | im not sure what its set too by default for VFs | 15:53 |
| spatel | 82599 has hard limit for 2 rx/tx queue per VF | 15:53 |
| sean-k-mooney | ah Reserved number of Queues per VF (default 4) | 15:53 |
| spatel | Look like x710 has 16 per VF (default 4) but we can change it so that is good | 15:53 |
| sean-k-mooney | https://doc.dpdk.org/guides/nics/i40e.html#runtime-config-options | 15:54 |
| spatel | nic look like x710 is good choice for me in low cost | 15:54 |
| sean-k-mooney | the 72x revision are nic if you need 25G or think will upgrade at some point | 15:55 |
| sean-k-mooney | they are not much more expensive and also are slightly more power effeinct | 15:55 |
| spatel | 10G is more then enough, my whole requirement is to run DPDK and remove SRIOV dependency which doesn't support security-group and bonding | 15:56 |
| spatel | hmm | 15:56 |
| spatel | let me see what i can get because all depend on budget but good to know | 15:57 |
| spatel | sean-k-mooney i have successfully deploy SRIOV with OVN and its working great | 15:57 |
| sean-k-mooney | good to know | 15:58 |
| sean-k-mooney | do not use it for PF passhtouhg however | 15:58 |
| sean-k-mooney | we need to remove that at some point | 15:58 |
| spatel | remove it from where? NOVA? | 15:58 |
| sean-k-mooney | ovn | 15:59 |
| sean-k-mooney | it allows vnic_type=direct-physical but does not manage it properly | 15:59 |
| spatel | I have question for you how does OVN provide redundancy for network node? currently my compute nodes are network node so lets say i have 10 compute nodes then how OVN schedule gateway chassis? | 15:59 |
| bauzas | 8 seconds before the meeting ;) | 16:00 |
| bauzas | so, wait a bit for the answer :p | 16:00 |
| spatel | :) | 16:00 |
| sean-k-mooney | that proably better asked in neutron too | 16:00 |
| bauzas | gibi: you're late | 16:00 |
| spatel | thank you! | 16:00 |
| gibi | #startmeeting nova | 16:01 |
| opendevmeet | Meeting started Tue Aug 31 16:01:25 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
| opendevmeet | The meeting name has been set to 'nova' | 16:01 |
| gibi | I was just reading the scrollback about the nics | 16:01 |
| gibi | :) | 16:01 |
| gibi | o/ | 16:01 |
| gmann | o/ | 16:01 |
| elodilles | o/ | 16:02 |
| dansmith | o/ | 16:02 |
| gibi | #topic Bugs (stuck/critical) | 16:02 |
| gibi | we have one bug marked critical | 16:02 |
| gibi | https://bugs.launchpad.net/nova/+bug/1940555 around SQLAlchemy URL handling and the fix is going through the gate: https://review.opendev.org/c/openstack/nova/+/805663 | 16:02 |
| bauzas | \o | 16:03 |
| gibi | it is bounced since I wrote the agenda but I requeued it | 16:03 |
| sean-k-mooney | ack | 16:03 |
| gibi | #link 17 new untriaged bugs (+6 since the last meeting): #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New | 16:03 |
| gibi | I had no time to check the untriaged bugs in the last couple of days | 16:04 |
| gibi | we have 0 bugs marked with xena-rc-potential tag #link https://bugs.launchpad.net/nova/+bugs?field.tag=xena-rc-potential | 16:04 |
| gibi | please start marking release critical bugs with xena-rc-potential tag | 16:04 |
| bauzas | tbc, those need to be regression bugs | 16:04 |
| gibi | yepp | 16:04 |
| bauzas | ie. something that worked before Xena but now has issues with this release | 16:05 |
| bauzas | we only have 2 weeks before RC1 in order to merge bugfixes that *aren't* regressions | 16:05 |
| bauzas | or those would be fixed in the next Yoga release | 16:05 |
| gibi | and backported :) | 16:05 |
| sean-k-mooney | i dont think we currenlty have any critical bugs that would need the rc flag | 16:06 |
| bauzas | so, pings are appreciated besides classic feature review requests | 16:06 |
| bauzas | we have an interesting race condition issue with placement | 16:06 |
| gibi | yepp I saved that for the gate issue topic | 16:06 |
| gibi | lets go there | 16:06 |
| bauzas | this isn't only impacting the gate, right? | 16:06 |
| gibi | it is a new race so so far we only have infor from the gate about it | 16:07 |
| bauzas | but yeah, let's discuss this bug on the gate section | 16:07 |
| gibi | #topic Gate status | 16:07 |
| gibi | Nova gate bugs #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure | 16:07 |
| gibi | We had the allocation deletion conflict bug #link https://bugs.launchpad.net/nova/+bug/1836754 that hopefully resolved when we landed #link https://review.opendev.org/c/openstack/nova/+/688802 yesterday | 16:07 |
| gibi | We have a project creation race in allocation update bug as well #link https://storyboard.openstack.org/#!/story/2009159 that is affecting the those gate jobs that are booting more than on VM at the same time. Octavia is heavily affected. I've pushed a reproducer #link https://review.opendev.org/c/openstack/placement/+/806730 but I don't know how to fix the actual bug. | 16:08 |
| gibi | this is the one bauzas mentioned above ^^ | 16:08 |
| bauzas | if the race isn't occurring a lot of times, we should maybe just call the transaction again | 16:09 |
| gibi | in short if two VM for the same project is scheduled at the same time then the PUT /allocations call could cause a race in the project creation in placement | 16:09 |
| gibi | bauzas: that was my first idea, but looking the code I'm not sure about the amount of surgery needed for this | 16:10 |
| dansmith | it must be the *first* two vms right? | 16:10 |
| gibi | if somebody has deeper sqlAlchemy knowledge than me then help is appreciated | 16:10 |
| bauzas | dansmith: it *should* be yeah | 16:10 |
| gibi | dansmith: first two VMs for a project yes | 16:10 |
| sean-k-mooney | gibi: sorry im a bit confusted by the term project createion in a placment context | 16:10 |
| dansmith | that definitely seems uncool | 16:10 |
| bauzas | I guess we could hit the problem often with multicreate | 16:11 |
| gibi | sean-k-mooney: placement stores the project_id and user_id of each consumer so that usages can be aggregated per project | 16:11 |
| bauzas | ie. server create --max 2 | 16:11 |
| sean-k-mooney | why does placement need to be project wawre beyond storigng the project/user id in the allocation? | 16:11 |
| bauzas | sean-k-mooney: because we use it from an API query | 16:11 |
| dansmith | sean-k-mooney: it's relational so it has a table for it I think | 16:12 |
| sean-k-mooney | yes im aware it there for usage | 16:12 |
| bauzas | you can filter per project | 16:12 |
| gibi | https://docs.openstack.org/api-ref/placement/?expanded=list-usages-detail#list-usages | 16:12 |
| sean-k-mooney | but im not sure what resouce in the db we would need to create that would race | 16:12 |
| bauzas | this was my question earlier this EU morning time :) | 16:12 |
| gibi | sean-k-mooney: it is the row in the projects table | 16:12 |
| dansmith | sean-k-mooney: the first time that entry goes into the table for the project? | 16:12 |
| gibi | yes ^^ | 16:12 |
| bauzas | correct | 16:12 |
| sean-k-mooney | right im wondering why we need a proejct tbale at all | 16:12 |
| dansmith | ...because it's relational :) | 16:13 |
| gibi | sean-k-mooney: for the /usages?project_id= query | 16:13 |
| sean-k-mooney | i assuem its just for forine key constriants | 16:13 |
| sean-k-mooney | dansmith: right but we dont actuly do that in other dbs in openstack | 16:13 |
| dansmith | we could create the project first in a separate transaction, which is easy to retry, but that's definitely surgery | 16:13 |
| dansmith | sean-k-mooney: not because it's a good idea :) | 16:13 |
| gibi | dansmith: actually I think that is what happened before consumer_types | 16:14 |
| gibi | dansmith: project_id user_id and consumer creation was each in a separate transaction, then a single transaction created / updated the allocation | 16:14 |
| dansmith | gibi: ah, so this is more fallout from that work? | 16:15 |
| gibi | it seems so | 16:15 |
| dansmith | okay | 16:15 |
| gibi | dansmith: this changed, now the whole PUT /allocations is a single DB transaction | 16:15 |
| dansmith | for something like the project record, it makes sense to do that in a separate transaction I think | 16:15 |
| gibi | and when that transaction founds a duplicate project it dies even if we catch the exception from sqla | 16:15 |
| sean-k-mooney | i wonder if we shoudl drop those tables in the future and just store the uuids direcly in the allcoation table | 16:15 |
| dansmith | gibi: further explains the large-ness of the transaction being a problem I guess | 16:15 |
| bauzas | dansmith: that's why I nearly consider this as a regression | 16:16 |
| dansmith | bauzas: definitely sounds like a regression | 16:16 |
| bauzas | we stepped into this mess in Xena | 16:16 |
| opendevreview | Merged openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | 16:16 |
| dansmith | sean-k-mooney: only if we want it to be slower and more bloated I think :) | 16:16 |
| gibi | the reason this become a single transaction is that consumer_type can be updated during PUT /allocations and if that happens in a separate transaction then we leak the update | 16:17 |
| sean-k-mooney | well right now we are going to have to do a select by uuid to get the id in the project/user tabel then join based on that to the allcoation | 16:17 |
| bauzas | I have no idea because I haven't looked at code yet, but is it crazy to switch back to the 2-step transactional model we had before ? | 16:17 |
| dansmith | gibi: but we're just recording projects for relational purposes, so even if the allocation put doesn't succeed, it's fine that we've created the project record right? | 16:17 |
| sean-k-mooney | so im not sure it would be slow to just to a select againt the allcoation table directly with an index on the user_id/project_id columns | 16:17 |
| bauzas | ie. get the projet or create it, first, then do the allocation update | 16:18 |
| gibi | dansmith: yes, for project and user it is fine, for consumer_type it is not fine | 16:18 |
| bauzas | into two transactions | 16:18 |
| dansmith | gibi: right | 16:18 |
| gibi | so we can move back to many transactions but then we need to figure out how to move the consumer_type update to the last big transaction | 16:19 |
| dansmith | yeah, which is surgery for sure | 16:19 |
| dansmith | was melwitt the one that wrote the consumer type patches? | 16:19 |
| dansmith | maybe we should get her consult | 16:19 |
| gibi | it was melwitt who updated it after cdent left | 16:19 |
| dansmith | well, right | 16:20 |
| gibi | so she knows more | 16:20 |
| dansmith | $current_owner = 'melwitt' is what I meant :) | 16:20 |
| gibi | yeah | 16:20 |
| gibi | OK, I guess that is it for now about this bug I will ping melwitt about it and record this discussion inthe bug | 16:20 |
| gibi | moving on | 16:20 |
| gibi | is there any other nova bug (gate or not) we need to discuss? | 16:21 |
| bauzas | did we put the hot potato into someone's hand who isn't there ? excellent outcome of this discussion :p | 16:21 |
| opendevreview | Merged openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | 16:21 |
| gibi | bauzas: I can still feel the responsibility until I can talk to melwitt | 16:21 |
| dansmith | wasn't it in her hands to begin with? I think it's okay :P | 16:21 |
| * bauzas is happy to be eventually back, then :p | 16:21 | |
| gibi | :D | 16:21 |
| gibi | I see there is no other bug to talk about great :D | 16:22 |
| bauzas | moving on | 16:22 |
| gibi | Placement periodic job status #link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly | 16:22 |
| gibi | they are greeen | 16:22 |
| gibi | and last but not least be prepared that the gate will be slow this week due to Milestone 3 | 16:22 |
| gibi | which brings us to | 16:23 |
| gibi | #topic Release Planning | 16:23 |
| gibi | Release tracking etherpad #link https://etherpad.opendev.org/p/nova-xena-rc-potential | 16:23 |
| gibi | Feature freeze is 3rd of Sept which is end of this week. | 16:23 |
| gibi | Also we have Milestone 3 end of this week that marks the last release of our client libs. Release patches: | 16:23 |
| gibi | novaclient https://review.opendev.org/c/openstack/releases/+/806592 | 16:23 |
| gibi | osc-placement https://review.opendev.org/c/openstack/releases/+/806580 | 16:23 |
| gibi | I'm not tracking any open novaclient patches | 16:23 |
| gibi | but for osc-placement the https://review.opendev.org/c/openstack/osc-placement/+/804458 consumer_types support client patch would be nice | 16:24 |
| gibi | the client release should happen this week so we still have a bit of time | 16:24 |
| bauzas | I can review it | 16:25 |
| gibi | We are tracking the release TODOs in #link https://etherpad.opendev.org/p/nova-xena-rc-potential | 16:25 |
| gibi | bauzas: thanks, I'm also plannig to get to it | 16:25 |
| gibi | there are two open todos for the release | 16:25 |
| gibi | We need to produce the cycle highlights | 16:25 |
| gibi | review is there: #link https://review.opendev.org/c/openstack/releases/+/800755 | 16:26 |
| gibi | I will do a bp status cleanup tomorrow and along that I will update the highlights patch. But you can help me with the highlight by bringing up features in the comments. | 16:26 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add support for 'hostname' parameter https://review.opendev.org/c/openstack/nova/+/778550 | 16:26 |
| opendevreview | Stephen Finucane proposed openstack/nova master: policy: Deprecate field from 'os-extended-server-attributes' policy https://review.opendev.org/c/openstack/nova/+/806131 | 16:26 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Speed up 'servers' API tests https://review.opendev.org/c/openstack/nova/+/778732 | 16:26 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Address nits for configurable-instance-hostnames series https://review.opendev.org/c/openstack/nova/+/806735 | 16:26 |
| gibi | the other todo is | 16:26 |
| gibi | We need a volunteer to write the release notes prelude (deadline for the prelude is RC1) | 16:26 |
| dansmith | gosh I wonder who will do that | 16:27 |
| * gibi turning the general direction of France :) | 16:27 | |
| * gibi turning to the general direction of France :) | 16:27 | |
| bauzas | hmmm | 16:27 |
| * bauzas wonders who gibi is looking at | 16:27 | |
| gibi | :D | 16:28 |
| bauzas | but ok, I can volunteer for it | 16:28 |
| gibi | thanks | 16:28 |
| gibi | anything else about the release before we jump to open bps? | 16:28 |
| gibi | #topic Review priorities | 16:29 |
| gibi | this is a new topic as the gerrit patch enabling the new Review-Priority label has been landed | 16:30 |
| gibi | you can query the label with something like https://review.opendev.org/q/project:openstack/nova+label:Review-Priority%252B1 | 16:30 |
| gibi | the doc about the label is here https://review.opendev.org/c/openstack/nova/+/792357 | 16:31 |
| gibi | anyhow as we are close the FF sean-k-mooney collected couple of bps from our list that we quickl agree on to mark as priority | 16:31 |
| gibi | instance host name https://review.opendev.org/q/topic:%22bp%252Fconfigurable-instance-hostnames%22+(status:open%20OR%20status:merged) | 16:32 |
| gibi | this is basically approved | 16:32 |
| gibi | as far as I see | 16:32 |
| bauzas | let me ask a stupid question | 16:32 |
| bauzas | do we have a formal process on how to use this flag ? | 16:32 |
| gmann | or we can run this to filter already +W one #link https://review.opendev.org/q/project:openstack/nova+Review-Priority:1+label:Verified%253D1++NOT+label:Workflow%253C%253D-1 | 16:32 |
| bauzas | I remember some change I reviewed a while ago | 16:32 |
| gibi | bauzas: this is where we discussing that https://review.opendev.org/c/openstack/nova/+/792357 | 16:32 |
| bauzas | but did we get into some direction ? | 16:33 |
| sean-k-mooney | bauzas: yes | 16:33 |
| gibi | I see you have negative comments there :) | 16:33 |
| sean-k-mooney | well it was +w'd so i assuems we had agreed | 16:33 |
| bauzas | I can see some consensus | 16:33 |
| bauzas | so I can rereview this | 16:33 |
| gibi | sure I have to back to that too | 16:34 |
| gibi | lets then just quickly check some of the open bps that close to lend and ignore the new label until we agree on the doc | 16:34 |
| gibi | s/lend/land | 16:34 |
| gibi | so instance hostname is a basically done we can move on | 16:35 |
| gibi | vhost multi queue flavor extra spec https://review.opendev.org/q/topic:%22bp%252Fmultiqueue-flavor-extra-spec%22+(status:open%20OR%20status:merged) | 16:35 |
| sean-k-mooney | well i would prefer to use the new lable and adjust our useage if we need too in a follow up t be hosest but sure lets proced | 16:35 |
| gibi | sean-k-mooney: I don't want to hold up the meeting agreeing on the usage as we have FF this week | 16:36 |
| sean-k-mooney | gibi sure we can move on | 16:36 |
| gibi | so the multique series is a simple one but still lack reviews | 16:36 |
| sean-k-mooney | ya tl;dr that is supporting a mutlique via the flaovr in addtion to the existing image property | 16:36 |
| bauzas | gibi: I was looking into the multiqueue series before the meeting | 16:37 |
| sean-k-mooney | its pretty straigt forward but need reviews | 16:37 |
| bauzas | so I could mark it with Review-Prio flag :p | 16:37 |
| gibi | bauzas: you can :P | 16:37 |
| gibi | if nobody else shows up I can read that too | 16:38 |
| gibi | moving on | 16:38 |
| gibi | generic mdevs https://review.opendev.org/q/topic:%22bp%252Fgeneric-mdevs%22+(status:open%20OR%20status:merged) | 16:38 |
| gibi | as far as I see the code is approved | 16:38 |
| gibi | the doc patch is open | 16:38 |
| bauzas | thanks all | 16:38 |
| gibi | we can land doc only patches after FF so this is in good shape | 16:38 |
| bauzas | I can work on the doc change btw. | 16:38 |
| bauzas | but my priority for this weeks is reviews. | 16:39 |
| gibi | that sounds good to me | 16:39 |
| gibi | moving on | 16:39 |
| gibi | pps qos https://review.opendev.org/q/topic:%2522bp/qos-minimum-guaranteed-packet-rate%2522+(status:open+OR+status:merged)+project:openstack/nova | 16:39 |
| gibi | this is easy as we don't have to rush | 16:39 |
| gibi | the neturon API extension enabling the new logic probably wont land in Xena | 16:39 |
| gibi | (but already works on in my env) | 16:40 |
| sean-k-mooney | ah ok so this is moving to yoga then? | 16:40 |
| gibi | the neutron impact seems so | 16:40 |
| gibi | the nova impact is good to land now but it wont be triggered if neutron does not have the new api extension | 16:41 |
| gibi | so it is dead code | 16:41 |
| sean-k-mooney | i mean if its dead code its also going to have little or no impact on people deploying xena but sure | 16:41 |
| gibi | yes | 16:41 |
| gibi | moving on | 16:42 |
| gibi | melwitt updated the unified limits https://review.opendev.org/q/topic:%22bp%252Funified-limits-nova%22+(status:open%20OR%20status:merged) sries | 16:42 |
| gibi | dansmith: you had comments before. Will you have time this week to look back to this? | 16:42 |
| dansmith | I saw her note about not reaching into limit internals but haven't gone back to look | 16:42 |
| dansmith | I will, but are we really expecting to land this? if we are, then I'll try to prioritize, but if not, I'd focus on other things that are time-critical | 16:43 |
| dansmith | seems like there are a lot of patches yet to land for thursday, | 16:43 |
| dansmith | and I know some have had some review, but I don't think all (but could be wrong) | 16:43 |
| bauzas | agreed with dansmith | 16:43 |
| bauzas | I'd love this series to merge, but this is very late in time | 16:44 |
| gibi | dansmith: it was open for a long time so I guess it is not that time critical | 16:45 |
| dansmith | lots of odd gate fail on the bottom few, which I'm sure is not related to those patches, but just in terms of likelihood for merging all of that by thursday seems unlikely | 16:46 |
| gibi | I don't see that much thing to close to land in nova but I guess you have commitments outside of nova too | 16:46 |
| dansmith | yeah I meant other stuff | 16:46 |
| dansmith | we should recheck the bottom three that are +W at least I guess | 16:47 |
| gibi | yepp | 16:47 |
| dansmith | the API ones at the top of the set are some I want to look closer at, and I haven't looked at the tempest tests that are available yet | 16:47 |
| gibi | ack. if you not get to it the it will be moved to yoga, no hard feelings :) | 16:48 |
| gibi | moving on | 16:48 |
| gibi | here are the rest of the open bps https://launchpad.net/nova/+milestone/xena-3 | 16:48 |
| gibi | (some of the should be closed which is will do tomorrow)_ | 16:48 |
| gibi | is there any of those somebody want to get attention? | 16:49 |
| gmann | remove tenant-id is still not in shape for Xena, we are still need all code up before we start merging first patch | 16:49 |
| gmann | may be for Yoga | 16:49 |
| gibi | gmann: thanks for the info. that is also a long time open series. | 16:49 |
| gmann | yeah. | 16:50 |
| gmann | we are reviewing it on time but just all the code should be up before we start merging it | 16:50 |
| gibi | yeah, that is a big change in a single microversion. | 16:50 |
| gibi | any other bp? | 16:51 |
| gibi | the moving on | 16:51 |
| gibi | #topic PTG Planning | 16:51 |
| gibi | every info is in the PTG etherpad #link https://etherpad.opendev.org/p/nova-yoga-ptg | 16:51 |
| gibi | please add topics ^^ | 16:51 |
| gibi | If you see a need for a specific cross project section then please let me know | 16:51 |
| gibi | any question about the PTG? | 16:51 |
| gibi | #topic Stable Branches | 16:52 |
| gibi | periodic-stable jobs are not failing | 16:52 |
| gibi | every stable gate seem to be working and unblocked | 16:52 |
| gibi | EOM( elodilles ) | 16:52 |
| gibi | anything else about stable? | 16:52 |
| gibi | #topic Sub/related team Highlights | 16:53 |
| elodilles | nothing exciting compared to FF :X | 16:53 |
| gibi | Libvirt (bauzas) | 16:53 |
| gibi | elodilles: :) | 16:53 |
| bauzas | well, I pledge guilty, I was off for 3 weeks | 16:53 |
| gibi | good for you :) | 16:53 |
| gibi | #topic Open discussion | 16:54 |
| gibi | (gibi) PTL election: as you probably saw from Yoga we will have a new PTL, bauzas. I'm not going anywhere and I will help to make the transition smooth. | 16:54 |
| gibi | welcome bauzas :) | 16:54 |
| bauzas | thanks, just hoping I won't get a revolution, as in my country | 16:54 |
| bauzas | don't cut my head, please. | 16:55 |
| gibi | dont put your head near to sharp object :D | 16:55 |
| sean-k-mooney | hehe. all hail the new hearder of cat :P | 16:55 |
| * gibi need more cats | 16:55 | |
| gibi | anything else for today? | 16:56 |
| bauzas | fun fact, I have no cats but a dog | 16:56 |
| bauzas | (and 2 kids, but apparently we can't count them as animals) | 16:56 |
| gmann | bauzas: :) | 16:57 |
| gibi | would you like to? :) | 16:57 |
| gibi | if nothing else then lets close this meeting | 16:58 |
| bauzas | well, they bark, always ask for food and sleep | 16:58 |
| gibi | :D | 16:58 |
| dansmith | are we done here? :) | 16:58 |
| bauzas | I guess | 16:58 |
| gibi | #endmeeting | 16:58 |
| opendevmeet | Meeting ended Tue Aug 31 16:58:50 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-08-31-16.01.html | 16:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-08-31-16.01.txt | 16:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-08-31-16.01.log.html | 16:58 |
| gibi | I'm dropping of for today see you tomorrow o/ | 17:00 |
| gmann | o/, GN | 17:01 |
| gmann | stephenfin: can you please check this review https://review.opendev.org/c/openstack/nova/+/806294 | 18:09 |
| opendevreview | Merged openstack/nova stable/ussuri: Centralize sqlite FK constraint enforcement https://review.opendev.org/c/openstack/nova/+/785070 | 19:02 |
| opendevreview | Merged openstack/nova stable/ussuri: Add functional test for bug 1837995 https://review.opendev.org/c/openstack/nova/+/785071 | 19:02 |
| mloza | hello, i given a user a reader role at system level but unable to list all projects and instances | 20:05 |
| opendevreview | Merged openstack/nova stable/ussuri: Dynamically archive FK related records in archive_deleted_rows https://review.opendev.org/c/openstack/nova/+/785072 | 20:14 |
| opendevreview | Merged openstack/nova master: db: Handle parameters in DB strings https://review.opendev.org/c/openstack/nova/+/805663 | 20:47 |
| opendevreview | Merged openstack/nova stable/victoria: Fix 1vcpu error with multiqueue and vif_type=tap https://review.opendev.org/c/openstack/nova/+/806004 | 21:15 |
| opendevreview | Merged openstack/nova master: api: Add support for 'hostname' parameter https://review.opendev.org/c/openstack/nova/+/778550 | 21:15 |
| opendevreview | Rodrigo Barbieri proposed openstack/nova stable/ussuri: Fix 1vcpu error with multiqueue and vif_type=tap https://review.opendev.org/c/openstack/nova/+/806811 | 21:29 |
| opendevreview | Merged openstack/nova master: policy: Deprecate field from 'os-extended-server-attributes' policy https://review.opendev.org/c/openstack/nova/+/806131 | 21:37 |
| opendevreview | Merged openstack/nova master: tests: Speed up 'servers' API tests https://review.opendev.org/c/openstack/nova/+/778732 | 21:38 |
| opendevreview | Merged openstack/nova master: tests: Address nits for configurable-instance-hostnames series https://review.opendev.org/c/openstack/nova/+/806735 | 21:38 |
| opendevreview | Merged openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | 21:38 |
| opendevreview | Merged openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | 21:38 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!