Thanks Vivek!
I have updated the weather wiki with the information.
I renamed the service back to the name in the initial patchset, as “load-balancer” looks more a generic term, and is
likely to get confused with other lb implementations.
Thanks,
Faseela
*From:* N Vivekanandan
*Sent:* Wednesday, April 04, 2018 6:17 PM
*To:* Faseela K <[email protected]>; Vivek Srivastava V <[email protected]>;
[email protected]
*Cc:* odl netvirt dev <[email protected]>; Frederick Kautz <[email protected]>;
[email protected]; [email protected]
*Subject:* RE: [netvirt-dev] [genius-dev] Genius Weather - Change L3 and ELAN
service index to accommodate new service
Hi Faseela,
The table name seems to represent a ‘Load Balancer Service’ and service name is
also not filled in the review.
*L3_LB_SERVICE_INDEX=180*
LoadBalancer is already an existing Openstack Neutron sub-service and it is
easy for folks to confuse with that name.
If you are using it for CoE_KubeProxy, can you please name this service
appropriately.
Please but a brief in the weather item on the responsibility of this service
and how it is different from Openstack LBaaS.
--
Thanks,
Vivek
*From:* Faseela K
*Sent:* Wednesday, April 04, 2018 3:32 PM
*To:* N Vivekanandan <[email protected] <mailto:[email protected]>>; Vivek Srivastava V
<[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Cc:* odl netvirt dev <[email protected] <mailto:[email protected]>>; Frederick Kautz
<[email protected] <mailto:[email protected]>>; [email protected] <mailto:[email protected]>;
[email protected] <mailto:[email protected]>
*Subject:* RE: [netvirt-dev] [genius-dev] Genius Weather - Change L3 and ELAN
service index to accommodate new service
Sure Vivek! That can be done.
Do you see any problem with the change in service index, for L3VPN and ELAN?
This is with the assumption that we will have replay based upgrades in Fluorine, and no need to worry about the older
flows in FRM.
Thanks,
Faseela
*From:* N Vivekanandan
*Sent:* Wednesday, April 04, 2018 3:30 PM
*To:* Faseela K <[email protected] <mailto:[email protected]>>; Vivek Srivastava V
<[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Cc:* odl netvirt dev <[email protected] <mailto:[email protected]>>; Frederick Kautz
<[email protected] <mailto:[email protected]>>; [email protected] <mailto:[email protected]>;
[email protected] <mailto:[email protected]>
*Subject:* RE: [netvirt-dev] [genius-dev] Genius Weather - Change L3 and ELAN
service index to accommodate new service
Hi Faseela,
Can’t we choose tables with higher numbers for COE?
Why Table #30 when space around that is already sparse..
Please choose 180 – 200 as it will help you to expand in the future also if
required.
--
Thanks,
Vivek
*From:* [email protected] <mailto:[email protected]>
[mailto:[email protected]] *On Behalf Of *Faseela K
*Sent:* Wednesday, April 04, 2018 2:57 PM
*To:* Vivek Srivastava V <[email protected] <mailto:[email protected]>>;
[email protected] <mailto:[email protected]>
*Cc:* odl netvirt dev <[email protected] <mailto:[email protected]>>; Frederick Kautz
<[email protected] <mailto:[email protected]>>; [email protected] <mailto:[email protected]>;
[email protected] <mailto:[email protected]>
*Subject:* Re: [netvirt-dev] [genius-dev] Genius Weather - Change L3 and ELAN
service index to accommodate new service
+netvirt-dev and sfc-dev
Hi Vivek,
Please find the weather item :
https://wiki.opendaylight.org/view/Weather#New_service_index_for_COE_Kube_Proxy_in_Genius
And the updated patch : https://git.opendaylight.org/gerrit/#/c/70191/
Thanks,
Faseela
*From:* Vivek Srivastava V
*Sent:* Tuesday, April 03, 2018 12:33 PM
*To:* Faseela K <[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Cc:* Frederick Kautz <[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Subject:* RE: [genius-dev] New OpenFlow Table Request for COE Service
Implementation
Hi Faseela,
If you want to insert a service in between existing services, you will need to change service-index for all services
after ACL and below.
Please create a weather report for this change and get approval from those
services.
Regards,
Vivek
*From:* Faseela K
*Sent:* Monday, April 02, 2018 12:10 PM
*To:* Vivek Srivastava V <[email protected] <mailto:[email protected]>>;
[email protected] <mailto:[email protected]>
*Cc:* Frederick Kautz <[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Subject:* RE: [genius-dev] New OpenFlow Table Request for COE Service
Implementation
Thanks for reminding this Vivek.
We need some service index lesser than L3VPN and higher than ACL, but I see that all service indices from 0 till 8 are
occupied. Any suggestions in this case?
Thanks,
Faseela
*From:* Vivek Srivastava V
*Sent:* Thursday, March 29, 2018 11:27 PM
*To:* Vivek Srivastava V <[email protected] <mailto:[email protected]>>; Faseela K
<[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Cc:* Frederick Kautz <[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Subject:* RE: [genius-dev] New OpenFlow Table Request for COE Service
Implementation
What about service-index?
*From:* [email protected] <mailto:[email protected]>
[mailto:[email protected]] *On Behalf Of *Vivek Srivastava V
*Sent:* Thursday, March 29, 2018 11:25 PM
*To:* Faseela K <[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Cc:* Frederick Kautz <[email protected] <mailto:[email protected]>>; [email protected]
<mailto:[email protected]>
*Subject:* Re: [genius-dev] New OpenFlow Table Request for COE Service
Implementation
Hi Faseela,
Looks okay to me. I have given +1, Sam can review and merge.
Regards,
Vivek
*From:* Faseela K
*Sent:* Thursday, March 29, 2018 10:30 PM
*To:* [email protected] <mailto:[email protected]>; Vivek Srivastava V
<[email protected] <mailto:[email protected]>>
*Cc:* [email protected] <mailto:[email protected]>; Mohamed El-Serngawy
<[email protected] <mailto:[email protected]>>; Frederick Kautz <[email protected] <mailto:[email protected]>>
*Subject:* New OpenFlow Table Request for COE Service Implementation
Hi Vivek,
We need a new table for COE Service Implementation,
Would like to reserve Table 30 for the same.
Please find the patch for the same below :
https://git.opendaylight.org/gerrit/#/c/70191/
Thanks,
Faseela
_______________________________________________
netvirt-dev mailing list
[email protected]
https://lists.opendaylight.org/mailman/listinfo/netvirt-dev
