Earlier  
Posted Nick Remark
#openstack-nova - 2021-04-28
12:27:30 sean-k-mooney sorry i tought i had said that at the PTG but maybe i was not very clear
12:27:30 sean-k-mooney sorry i tought i had said that at the PTG but maybe i was not very clear
12:27:58 gibi sean-k-mooney: no it is my bad, I mixied the QoS policy part with the resource classes
12:27:58 gibi sean-k-mooney: no it is my bad, I mixied the QoS policy part with the resource classes
12:28:16 gibi next step
12:28:16 gibi next step
12:28:27 sean-k-mooney but ya its a little more complet for the tenant but simpler in code
12:28:27 sean-k-mooney but ya its a little more complet for the tenant but simpler in code
12:28:32 sean-k-mooney at least i think it is
12:28:32 sean-k-mooney at least i think it is
12:28:38 gibi yepp
12:29:54 gibi so next step, when we have the directionless QoS policy and later we want to implement data plane enforcement for that, then how we map the direction less min guarantee to direction aware pps rate limit rules to do enforcement?
12:29:54 gibi so next step, when we have the directionless QoS policy and later we want to implement data plane enforcement for that, then how we map the direction less min guarantee to direction aware pps rate limit rules to do enforcement?
12:31:06 gibi this is the packet rate limiter work https://bugs.launchpad.net/neutron/+bug/1912460 from Liu
12:31:09 openstack Launchpad bug 1912460 in neutron "[RFE] [QoS] add qos rule type packet per second (pps)" [Wishlist,In progress] - Assigned to LIU Yulong (dragon889)
12:31:09 gibi this is the packet rate limiter work https://bugs.launchpad.net/neutron/+bug/1912460 from Liu
12:31:09 openstack Launchpad bug 1912460 in neutron "[RFE] [QoS] add qos rule type packet per second (pps)" [Wishlist,In progress] - Assigned to LIU Yulong (dragon889)
12:31:42 gibi and as I understood from ralonso the basic solution for the guarante is to set up max limits := min guarante for all the traffic
12:31:42 gibi and as I understood from ralonso the basic solution for the guarante is to set up max limits := min guarante for all the traffic
12:31:51 sean-k-mooney that is a goog question we have 3 choices i guess, 1 dont, 2 split it evenly between each direction(assuming its a share resouces), 3 assume they are indepented so allwo it for both
12:31:51 sean-k-mooney that is a goog question we have 3 choices i guess, 1 dont, 2 split it evenly between each direction(assuming its a share resouces), 3 assume they are indepented so allwo it for both
12:33:19 sean-k-mooney gibi: ya so like badnwith, the mins for ovs are not enfroced by ovs so you would use max ppp instead to do enfrocement
12:33:20 sean-k-mooney gibi: ya so like badnwith, the mins for ovs are not enfroced by ovs so you would use max ppp instead to do enfrocement
12:33:44 sean-k-mooney for any backend that can enforce it then min should be enough
12:33:44 sean-k-mooney for any backend that can enforce it then min should be enough
12:33:57 sean-k-mooney and you should not need max rules
12:33:57 sean-k-mooney and you should not need max rules
12:34:09 gibi OK so in case of normal OVS where we have the directionless QoS we know that it is not an independent resource and therefore we would need to split it but splitting it equally might not what the user intended originally
12:34:09 gibi OK so in case of normal OVS where we have the directionless QoS we know that it is not an independent resource and therefore we would need to split it but splitting it equally might not what the user intended originally
12:35:28 sean-k-mooney gibi: well since the enforcement will be doen by the max policy the could use a directionless min policy with ad directionful? pair of max policies
12:35:28 sean-k-mooney gibi: well since the enforcement will be doen by the max policy the could use a directionless min policy with ad directionful? pair of max policies
12:36:08 sean-k-mooney so min_totall_pps=1000 max_ingress_pps=200 max_egress_pps=800
12:36:08 sean-k-mooney so min_totall_pps=1000 max_ingress_pps=200 max_egress_pps=800
12:36:22 gibi so we are sure that OVS will never have the capability to enforce min alone? so we alway need to manually define both min and max
12:36:22 gibi so we are sure that OVS will never have the capability to enforce min alone? so we alway need to manually define both min and max
12:37:19 sean-k-mooney well intel wehere trying to get this into ovs-dpdk for a few year but could never get around the fact they would basically have to dedicate 1 core to doing the min enforcement
12:37:19 sean-k-mooney well intel wehere trying to get this into ovs-dpdk for a few year but could never get around the fact they would basically have to dedicate 1 core to doing the min enforcement
12:37:31 sean-k-mooney i dont think the kernel implemation has made any progress either
12:37:31 sean-k-mooney i dont think the kernel implemation has made any progress either
12:37:42 sean-k-mooney so i dont know
12:37:42 sean-k-mooney so i dont know
12:37:53 gibi OK, thanks I think I got the full picture
12:37:53 gibi OK, thanks I think I got the full picture
12:38:07 gibi I will go and document it now in the neutron spec
12:38:07 gibi I will go and document it now in the neutron spec
12:39:01 gibi I really appreciate you help
12:39:01 gibi I really appreciate you help
12:40:40 sean-k-mooney just did some googleing mabe they have some support https://mail.openvswitch.org/pipermail/ovs-dev/2016-July/317678.html
12:40:40 sean-k-mooney just did some googleing mabe they have some support https://mail.openvswitch.org/pipermail/ovs-dev/2016-July/317678.html
12:41:47 sean-k-mooney i dont actully see min suport
12:41:47 sean-k-mooney i dont actully see min suport
12:42:39 sean-k-mooney althoug its in the exaMPLES http://www.openvswitch.org/support/dist-docs/ovs-vsctl.8.txt
12:42:39 sean-k-mooney althoug its in the exaMPLES http://www.openvswitch.org/support/dist-docs/ovs-vsctl.8.txt
12:43:04 sean-k-mooney min-rate is bandwith though not pps
12:43:04 sean-k-mooney min-rate is bandwith though not pps
12:45:01 gibi OK
12:55:07 sean-k-mooney gibi: so looking at https://www.openvswitch.org/support/dist-docs/ovs-vswitchd.conf.db.5.html the queue table does suppoirt min-rate now but if i rememebr correctly it is very challaging to use that correctly
12:55:07 sean-k-mooney gibi: so looking at https://www.openvswitch.org/support/dist-docs/ovs-vswitchd.conf.db.5.html the queue table does suppoirt min-rate now but if i rememebr correctly it is very challaging to use that correctly
12:55:22 sean-k-mooney other_config : min-rate: optional string, containing an integer, at
12:55:22 sean-k-mooney other_config : min-rate: optional string, containing an integer, at
12:55:24 sean-k-mooney least 1
12:55:24 sean-k-mooney least 1
12:55:26 sean-k-mooney Minimum guaranteed bandwidth, in bit/s.
12:55:26 sean-k-mooney Minimum guaranteed bandwidth, in bit/s.
12:55:45 gibi yapp that just bandwidth not packet rate
12:55:45 gibi yapp that just bandwidth not packet rate
12:56:07 sean-k-mooney that sound greate but to use this you have to manually create quees on each port and then instead of output to the prot or using the normal action you have to output to the queue explictly
12:56:07 sean-k-mooney that sound greate but to use this you have to manually create quees on each port and then instead of output to the prot or using the normal action you have to output to the queue explictly
12:57:05 sean-k-mooney neutron could use that for traffic ingressing to the vm or but only if it stopped using the normal action in ml2/ovs
12:57:05 sean-k-mooney neutron could use that for traffic ingressing to the vm or but only if it stopped using the normal action in ml2/ovs
12:57:45 sean-k-mooney ovn could use that for ingress to the vm but on egress from the vm it would not really work
12:57:45 sean-k-mooney ovn could use that for ingress to the vm but on egress from the vm it would not really work
12:58:25 sean-k-mooney and ya its bandwith not pps so it wont help in your case
12:58:25 sean-k-mooney and ya its bandwith not pps so it wont help in your case
12:58:37 gibi ack
12:58:41 gibi ack
13:20:36 openstackgerrit Daniel Bengtsson proposed openstack/nova master: Use the new type HostDomainOpt. https://review.opendev.org/c/openstack/nova/+/788240
14:51:56 lyarwood elod / bauzas / melwitt ; https://review.opendev.org/c/openstack/nova/+/787943 - If you have time today can I get reviews on this backport please.
14:51:56 lyarwood elod / bauzas / melwitt ; https://review.opendev.org/c/openstack/nova/+/787943 - If you have time today can I get reviews on this backport please.
14:52:35 bauzas lyarwood: we have our meeting in 9 mins and then I'll need to go to a blood donating but I'll try
14:52:35 bauzas lyarwood: we have our meeting in 9 mins and then I'll need to go to a blood donating but I'll try
14:53:45 openstackgerrit Stephen Finucane proposed openstack/nova master: Add functional regression test for bug 1853009 https://review.opendev.org/c/openstack/nova/+/695012
14:53:47 openstack bug 1853009 in OpenStack Compute (nova) ussuri "Ironic node rebalance race can lead to missing compute nodes in DB" [High,In progress] https://launchpad.net/bugs/1853009 - Assigned to Mark Goddard (mgoddard)
14:53:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Add functional regression test for bug 1853009 https://review.opendev.org/c/openstack/nova/+/695012
14:53:47 openstack bug 1853009 in OpenStack Compute (nova) ussuri "Ironic node rebalance race can lead to missing compute nodes in DB" [High,In progress] https://launchpad.net/bugs/1853009 - Assigned to Mark Goddard (mgoddard)
14:53:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/c/openstack/nova/+/695187
14:53:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/c/openstack/nova/+/695187
14:53:56 lyarwood bauzas: yup no issues if you can't get to it
14:53:57 lyarwood bauzas: yup no issues if you can't get to it
14:54:08 openstackgerrit Stephen Finucane proposed openstack/nova master: Invalidate provider tree when compute node disappears https://review.opendev.org/c/openstack/nova/+/695188
14:54:08 openstackgerrit Stephen Finucane proposed openstack/nova master: Invalidate provider tree when compute node disappears https://review.opendev.org/c/openstack/nova/+/695188
14:54:34 openstackgerrit Stephen Finucane proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/c/openstack/nova/+/694802
14:54:34 openstackgerrit Stephen Finucane proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/c/openstack/nova/+/694802
14:54:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/c/openstack/nova/+/695189
14:54:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/c/openstack/nova/+/695189
14:54:52 bauzas lyarwood: /me clicks anyway
14:54:52 bauzas lyarwood: /me clicks anyway
14:55:21 lyarwood ah crap the patch below hasn't actually landed
14:55:22 lyarwood ah crap the patch below hasn't actually landed
14:55:32 lyarwood sorry I thought it had, this isn't urgent in that case

Earlier   Later