| opendevreview | OpenStack Proposal Bot proposed openstack/networking-odl master: Imported Translations from Zanata https://review.opendev.org/c/openstack/networking-odl/+/857997 | 03:07 |
|---|---|---|
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: DNM: just a test for distributed dhcp jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859019 | 07:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Remove "dhcp_agent_scheduler" API extension from common list https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859020 | 07:36 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-odl stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/networking-odl/+/859021 | 07:49 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-odl stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/networking-odl/+/859022 | 07:49 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-odl master: Update master for stable/zed https://review.opendev.org/c/openstack/networking-odl/+/859023 | 07:49 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-odl master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/networking-odl/+/859024 | 07:49 |
| opendevreview | Merged openstack/networking-odl master: Update master for stable/zed https://review.opendev.org/c/openstack/networking-odl/+/859023 | 08:29 |
| opendevreview | Merged openstack/networking-odl stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/networking-odl/+/859021 | 08:29 |
| opendevreview | Merged openstack/networking-odl stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/networking-odl/+/859022 | 08:29 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/xena: Fix healthMonitor events affecting to unrelated LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/855633 | 08:55 |
| ralonsoh | lajoskatona, https://review.opendev.org/c/openstack/networking-odl/+/859024/1 | 09:08 |
| ralonsoh | I think you missed this one | 09:08 |
| lajoskatona | ralonsoh: zuul not finished when I last checked, thanks :-) | 09:09 |
| opendevreview | Merged openstack/networking-odl master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/networking-odl/+/859024 | 09:33 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Capture port deleted event associated to HM ServiceMonitor https://review.opendev.org/c/openstack/ovn-octavia-provider/+/859063 | 10:06 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Capture port deleted event associated to HM ServiceMonitor https://review.opendev.org/c/openstack/ovn-octavia-provider/+/859064 | 10:07 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Capture port deleted event associated to HM ServiceMonitor https://review.opendev.org/c/openstack/ovn-octavia-provider/+/859065 | 10:08 |
| ralonsoh | slaweq, if you have 1 min | 10:15 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron/+/858890 | 10:15 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron/+/858663 | 10:15 |
| ralonsoh | thanks in advance | 10:15 |
| slaweq | ralonsoh sure, looking now | 10:15 |
| slaweq | ralonsoh both approved | 10:17 |
| ralonsoh | slaweq, thanks! | 10:17 |
| slaweq | ralonsoh lajoskatona ykarel if You have some time, please look at https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859020 | 10:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Accept a port deletion with missing port binding information https://review.opendev.org/c/openstack/neutron/+/857490 | 10:18 |
| slaweq | thx in advance | 10:18 |
| ralonsoh | sure | 10:18 |
| ralonsoh | slaweq, qq, why did you add "network_api_extensions_openvswitch"? | 10:19 |
| ralonsoh | is that related? | 10:19 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859020/1/zuul.d/master_jobs.yaml#427 | 10:19 |
| slaweq | yes, it's related | 10:20 |
| ralonsoh | ok, thanks | 10:20 |
| slaweq | this job inherits from neutron-tempest-plugin-openvswitch | 10:20 |
| slaweq | and in neutron-tempest-plugin-openvswitch in that network_api_extensions_openvswitch there is now dhcp-agent-scheduler extension added | 10:21 |
| slaweq | but in that -distributed-dhcp jobs I want to override it to not have that extension enabled | 10:21 |
| ralonsoh | ahhhhh | 10:21 |
| ralonsoh | now I get it | 10:21 |
| slaweq | when that job was first introduced, it was running only scenario tests, as for api we had different job | 10:21 |
| slaweq | and it was fine | 10:22 |
| ralonsoh | slaweq, why "qos-bw-minimum-ingress"? | 10:22 |
| slaweq | but now we run also api tests in those jobs and 4 tests were failing all the time | 10:22 |
| ralonsoh | L429 | 10:22 |
| ralonsoh | instead of logging | 10:22 |
| slaweq | because it is alo in original openvswitch job in L243 | 10:23 |
| ralonsoh | ufff, I was reading the other parent | 10:23 |
| ralonsoh | thanks again! | 10:23 |
| slaweq | :) | 10:23 |
| ykarel | slaweq, ack | 10:50 |
| stephenfin | ralonsoh: slaweq: Two questions on quotas in neutron. Firstly, do you have the idea of quota classes? Secondly, is there any way to set the default quotas via the API rather than in neutron.conf? | 10:58 |
| stephenfin | I think the answer is no for both but would like to confirm :) | 10:59 |
| ralonsoh | stephenfin, what is a quota class? | 11:04 |
| ralonsoh | and as you said, we can set the default values only in the config file, not via API | 11:05 |
| lajoskatona | stephenfin, ralonsoh: quotaclasses are described here am I right: https://docs.openstack.org/project-team-guide/technical-guides/unified-limits.html ? | 11:06 |
| lajoskatona | stephenfin, ralonsoh: this was something done by nova already, so it can be something that we also adopt in next cycles, if it is really useful | 11:07 |
| ralonsoh | lajoskatona, we define the quotas in runtime | 11:08 |
| ralonsoh | some components are marked as tracked | 11:09 |
| ralonsoh | and during when the API is reading all the routes, we build those paths depending on the registered quotas | 11:09 |
| stephenfin | ralonsoh: Quota classes appear to be a half implemented concept in nova and cinder. I think the idea of them was to allow you to set different types of "default" without having to explicitly set quotas for each project | 11:10 |
| lajoskatona | ralonsoh: yes, the quota API, but this would be a centralized store for it in keystone if I understand well, to tell the truth I am not convinced that this quota class feature is really worth the work, but I just read few things about it :-) | 11:10 |
| lajoskatona | stephenfin: thanks | 11:10 |
| stephenfin | So you could have bronze, silver and gold quota classes, and assign different projects to different classes depending on price plan or whatever | 11:10 |
| stephenfin | like I said though, it seems to be half implemented. From what I can tell, both nova and cinder only support one quota class - "default" :) | 11:11 |
| ralonsoh | ok so from Neutron point of view, we should allow to modify the available Neutron quotas via the API | 11:11 |
| ralonsoh | just to be able to create a new project with the required default quota values | 11:12 |
| ralonsoh | that is "easy" | 11:12 |
| stephenfin | lajoskatona: I'm not sure if quota classes are anything to do with the oslo.limit work. I think that's purely moving where the information is stored | 11:12 |
| lajoskatona | stephenfin: ahh, ok, that I missed | 11:12 |
| ralonsoh | does it mean we are not longer storing this info in the Neutron DB? | 11:13 |
| stephenfin | ralonsoh: In nova, the order of precedence is: nova.conf configured quotas < API configured quotas < project configured quotas | 11:13 |
| ralonsoh | makes sense | 11:13 |
| ralonsoh | we can implement this | 11:13 |
| stephenfin | ralonsoh: I think so, yes | 11:13 |
| stephenfin | as in yes, you would no longer store stuff in neutron | 11:14 |
| ralonsoh | hmmmm, bad yuyu | 11:14 |
| lajoskatona | stephenfin: and the store is in keystone? | 11:14 |
| stephenfin | If you were to implement unified limits, you actually wouldn't want to make quotas API configurable (at least not from neutron) | 11:14 |
| stephenfin | because once you switched to unified limits, the neutron API would be a proxy | 11:15 |
| stephenfin | and proxy APIs are bad | 11:15 |
| stephenfin | :) | 11:15 |
| stephenfin | lajoskatona: This is my understanding. johnthetubaguy and melwitt were driving this work | 11:15 |
| stephenfin | maybe that work has merged by now... | 11:15 |
| * stephenfin looks | 11:15 | |
| lajoskatona | stephenfin: thanks, perhaps we can have for this a discussion on the PTG also, to have a common view | 11:16 |
| ralonsoh | right | 11:16 |
| stephenfin | yes https://review.opendev.org/q/topic:bp%252Funified-limits-nova | 11:16 |
| stephenfin | I knew I'd reviewed it. I didn't remember if it had merged though | 11:16 |
| stephenfin | lajoskatona: That would make sense. It's still experimental in nova and I don't know if melwitt or anyone else is pushing to make it non-experimental yet but it does seem like a good long-term plan | 11:18 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: docs: Remove errant indentation, mark up literals https://review.opendev.org/c/openstack/neutron/+/859077 | 11:18 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: db: Remove checks for old single branch head files https://review.opendev.org/c/openstack/neutron/+/859078 | 11:18 |
| stephenfin | random doc fixes I spotted while reading the quota docs btw ^ (plus a change I was carrying locally for a few months) | 11:19 |
| lajoskatona | stephenfin: thanks :-) | 11:20 |
| ralonsoh | stephenfin, one qq (now reading the patches) | 11:23 |
| ralonsoh | any time we have a new resource request, do we need to check this external DB? | 11:23 |
| stephenfin | I don't know for sure, but I think so, yes | 11:25 |
| stephenfin | however, oslo.limit does some level of caching https://github.com/openstack/oslo.limit/blob/master/oslo_limit/limit.py | 11:26 |
| opendevreview | Merged openstack/neutron stable/zed: Fix openstack-tox-py39-with-oslo-master job https://review.opendev.org/c/openstack/neutron/+/858890 | 11:47 |
| opendevreview | Merged openstack/neutron master: Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler" https://review.opendev.org/c/openstack/neutron/+/858663 | 11:47 |
| opendevreview | Merged openstack/neutron master: Move the "ovn_hash_ring" clean up to maintenance worker https://review.opendev.org/c/openstack/neutron/+/858542 | 12:09 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Rate limit the "Disallow caching" log from hash ring https://review.opendev.org/c/openstack/neutron/+/858731 | 12:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler" https://review.opendev.org/c/openstack/neutron/+/859084 | 13:18 |
| opendevreview | Merged openstack/networking-ovn stable/train: [OVN] Rate limit the "Disallow caching" log from hash ring https://review.opendev.org/c/openstack/networking-ovn/+/858735 | 13:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler" https://review.opendev.org/c/openstack/neutron/+/859091 | 13:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler" https://review.opendev.org/c/openstack/neutron/+/859092 | 13:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Use "OVNMechDriver" instance in "TestOvn[Nb|Sb]IdlNotifyHandler" https://review.opendev.org/c/openstack/neutron/+/859093 | 13:56 |
| *** dasm|off is now known as dasm | 14:02 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Move the "ovn_hash_ring" clean up to maintenance worker" https://review.opendev.org/c/openstack/neutron/+/858908 | 14:09 |
| opendevreview | Merged openstack/neutron stable/victoria: [OVN] Rate limit the "Disallow caching" log from hash ring https://review.opendev.org/c/openstack/neutron/+/858732 | 15:32 |
| opendevreview | Merged openstack/neutron stable/zed: [OVN] Rate limit the "Disallow caching" log from hash ring https://review.opendev.org/c/openstack/neutron/+/858728 | 15:32 |
| opendevreview | Merged openstack/neutron stable/zed: Handle several dhcp agents for metadata over ipv6 https://review.opendev.org/c/openstack/neutron/+/858902 | 15:32 |
| opendevreview | Merged openstack/neutron stable/yoga: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/858469 | 15:32 |
| opendevreview | Merged openstack/neutron stable/victoria: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/858619 | 15:32 |
| opendevreview | Merged openstack/neutron stable/wallaby: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/858618 | 15:32 |
| opendevreview | Merged openstack/neutron stable/ussuri: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/858620 | 15:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Open Antelope DB branch https://review.opendev.org/c/openstack/neutron/+/859111 | 15:43 |
| opendevreview | Merged openstack/neutron master: docs: Remove errant indentation, mark up literals https://review.opendev.org/c/openstack/neutron/+/859077 | 16:23 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Rate limit the "Disallow caching" log from hash ring https://review.opendev.org/c/openstack/neutron/+/858730 | 18:49 |
| opendevreview | Fernando Royo proposed openstack/neutron master: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/859143 | 19:05 |
| opendevreview | Merged openstack/neutron stable/xena: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/858617 | 19:07 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Fix missed six library items in .pylintrc https://review.opendev.org/c/openstack/neutron-lib/+/858990 | 19:21 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add missing comma in .pylintrc https://review.opendev.org/c/openstack/neutron/+/859146 | 19:24 |
| melwitt | stephenfin, lajoskatona: fwiw "quota classes" are a super legacy thing where the idea was that some System outside of nova would call in to the nova API with a "quota class" tag and nova would apply quota limits that are defined in that class. and if that class doesn't exist they get the "default" quota class limits | 19:35 |
| melwitt | i.e. they stand up a proxy in front of nova and that's what slaps on the quota class. only ever used by rax that we know of and we want ultimately to remove it | 19:36 |
| melwitt | for the notion of "gold", "silver", and "bronze" type of quotas, those would be represented by CUSTOM_ resource classes in placement. and tag relevant flavors with that class in the extra specs | 19:37 |
| melwitt | you get kind of a quota by flavor if you do that | 19:38 |
| melwitt | (^ using unified limits) | 19:39 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Remove "dhcp_agent_scheduler" API extension from common list https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859020 | 19:45 |
| opendevreview | Merged openstack/neutron stable/yoga: [OVN] Rate limit the "Disallow caching" log from hash ring https://review.opendev.org/c/openstack/neutron/+/858729 | 19:45 |
| stephenfin | melwitt++ thanks for the info | 20:21 |
| opendevreview | Merged openstack/neutron stable/yoga: Handle several dhcp agents for metadata over ipv6 https://review.opendev.org/c/openstack/neutron/+/858898 | 20:38 |
| *** dasm is now known as dasm|off | 21:10 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!