Re: [openstack-dev] Change in openstack/networking-sfc[master]: Networking-sfc / OVN Driver

2016-06-28 Thread Na Zhu 
Thanks your response.




Regards,
Juno Zhu
IBM China Development Labs (CDL) Cloud IaaS Lab
Email: na...@cn.ibm.com
5F, Building 10, 399 Keyuan Road, Zhangjiang Hi-Tech Park, Pudong New 
District, Shanghai, China (201203)



From:   Cathy Zhang 
To: Na Zhu/China/IBM@IBMCN
Cc: Farhad Sunavala , John McDowall 
, Henry Fourie , 
Kyle Mestery , "OpenStack Development Mailing List 
(not for usage questions)" , Russell 
Bryant , Ryan Moats , Richard Theis 
, Stephen Wong , "Vikram 
Choudhary" 
Date:   2016/06/29 09:21
Subject:RE: Change in openstack/networking-sfc[master]: 
Networking-sfc / OVN Driver



Please see inline.
 
Cathy
 
From: Na Zhu [mailto:na...@cn.ibm.com] 
Sent: Monday, June 27, 2016 9:56 PM
To: Cathy Zhang
Cc: Farhad Sunavala; John McDowall; Henry Fourie; Kyle Mestery; OpenStack 
Development Mailing List (not for usage questions); Russell Bryant; Ryan 
Moats; Richard Theis; Stephen Wong; Vikram Choudhary
Subject: RE: Change in openstack/networking-sfc[master]: Networking-sfc / 
OVN Driver
 
Hi Cathy,

Thanks your response.
Another question, how to create different port-chains for different 
tenants, and these chains consist of the same port pair group. In my test 
scenario, port-pair-group created by tenant A is not visible for tenant B.
 
Cathy> Each port pair group has an associated tenant ID, so it belongs to 
a tenant, that is why it is not visible. The same tenant can have multiple 
chains, eg. One for voice, one for video, one for data, and these chains 
can share the port pair group. But different tenants can not share the 
port pair group. 





Regards,
Juno Zhu
IBM China Development Labs (CDL) Cloud IaaS Lab
Email: na...@cn.ibm.com
5F, Building 10, 399 Keyuan Road, Zhangjiang Hi-Tech Park, Pudong New 
District, Shanghai, China (201203)



From:Cathy Zhang 
To:Na Zhu/China/IBM@IBMCN, Henry Fourie , 
"OpenStack Development Mailing List (not for usage questions)" <
openstack-dev@lists.openstack.org>
Cc:Ryan Moats , Kyle Mestery <
mest...@mestery.com>, Russell Bryant , Richard Theis <
rth...@us.ibm.com>, Stephen Wong , Vikram 
Choudhary , Farhad Sunavala , "John McDowall" 
Date:2016/06/28 12:29
Subject:RE: Change in openstack/networking-sfc[master]: 
Networking-sfc / OVN Driver




Hi Na,

Please see inline for my reply.

Cathy

-Original Message-
From: Na Zhu (Code Review) [mailto:rev...@openstack.org] 
Sent: Monday, June 27, 2016 8:08 PM
To: Henry Fourie
Cc: Ryan Moats; Na Zhu; Kyle Mestery; Russell Bryant; Richard Theis; 
Stephen Wong; Cathy Zhang; Vikram Choudhary; Farhad Sunavala; John 
McDowall
Subject: Change in openstack/networking-sfc[master]: Networking-sfc / OVN 
Driver

Na Zhu has posted comments on this change.

Change subject: Networking-sfc / OVN Driver 
..


Patch Set 1:

(1 comment)

https://review.openstack.org/#/c/333172/1/doc/source/sfc_ovn_driver.rst
File doc/source/sfc_ovn_driver.rst:

Line 88: +-+   +-+ outport +===+
> Agree that it is better to clarify these points. 
Hi Cathy,
I try to create multiple port-chains with the same port-pair-group, it 
failed. I think it is not allowed to do what you said, right?
(neutron) port-chain-create --flow-classifier fc --port-pair-group pg1 pc1 
Port Pair Group(s) [u'17c9a0a5-a38f-4a75-834e-9aa213cd431f'] in use by 
Port Chain f3af530f-210a-4c51-9a02-f1835d5b1d85.
Neutron server returns request_ids: 
['req-47e6a39b-677a-4ce3-9976-5dfcee0ac47f']
(neutron)

Cathy> when a port pair group is shared by multiple chains, these chains 
should be different, which means these chains should consist of different 
sequences of port pair groups. For example, chain 1 consists of 
 and chain 2 consists of 
"port-pair-group1, port-pair-group3, port-pair-group4>. But if the two 
chains are the same, i.e. they consist of the same sequence of 
port-pair-groups (e.g. the two chains both consist of ), 
then it is not allowed since it does not make sense to create the same 
chain twice. Maybe your test scenario falls into the second case?



-- 
To view, visit https://review.openstack.org/333172
To unsubscribe, visit https://review.openstack.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c1827dcc81e48983a41120ec08983c571c5b2e9
Gerrit-PatchSet: 1
Gerrit-Project: openstack/networking-sfc
Gerrit-Branch: master
Gerrit-Owner: Louis Fourie 
Gerrit-Reviewer: Farhad

Re: [openstack-dev] Change in openstack/networking-sfc[master]: Networking-sfc / OVN Driver

2016-06-28 Thread Cathy Zhang 
Please see inline.

Cathy

From: Na Zhu [mailto:na...@cn.ibm.com]
Sent: Monday, June 27, 2016 9:56 PM
To: Cathy Zhang
Cc: Farhad Sunavala; John McDowall; Henry Fourie; Kyle Mestery; OpenStack 
Development Mailing List (not for usage questions); Russell Bryant; Ryan Moats; 
Richard Theis; Stephen Wong; Vikram Choudhary
Subject: RE: Change in openstack/networking-sfc[master]: Networking-sfc / OVN 
Driver

Hi Cathy,

Thanks your response.
Another question, how to create different port-chains for different tenants, 
and these chains consist of the same port pair group. In my test scenario, 
port-pair-group created by tenant A is not visible for tenant B.

Cathy> Each port pair group has an associated tenant ID, so it belongs to a 
tenant, that is why it is not visible. The same tenant can have multiple 
chains, eg. One for voice, one for video, one for data, and these chains can 
share the port pair group. But different tenants can not share the port pair 
group.





Regards,
Juno Zhu
IBM China Development Labs (CDL) Cloud IaaS Lab
Email: na...@cn.ibm.com
5F, Building 10, 399 Keyuan Road, Zhangjiang Hi-Tech Park, Pudong New District, 
Shanghai, China (201203)



From:Cathy Zhang 
>
To:Na Zhu/China/IBM@IBMCN, Henry Fourie 
>, "OpenStack 
Development Mailing List (not for usage questions)" 
>
Cc:Ryan Moats >, Kyle 
Mestery >, Russell Bryant 
>, Richard Theis 
>, Stephen Wong 
>, Vikram Choudhary 
>, Farhad 
Sunavala >, "John McDowall" 
>
Date:2016/06/28 12:29
Subject:RE: Change in openstack/networking-sfc[master]: Networking-sfc 
/ OVN Driver




Hi Na,

Please see inline for my reply.

Cathy

-Original Message-
From: Na Zhu (Code Review) [mailto:rev...@openstack.org]
Sent: Monday, June 27, 2016 8:08 PM
To: Henry Fourie
Cc: Ryan Moats; Na Zhu; Kyle Mestery; Russell Bryant; Richard Theis; Stephen 
Wong; Cathy Zhang; Vikram Choudhary; Farhad Sunavala; John McDowall
Subject: Change in openstack/networking-sfc[master]: Networking-sfc / OVN Driver

Na Zhu has posted comments on this change.

Change subject: Networking-sfc / OVN Driver 
..


Patch Set 1:

(1 comment)

https://review.openstack.org/#/c/333172/1/doc/source/sfc_ovn_driver.rst
File doc/source/sfc_ovn_driver.rst:

Line 88: +-+   +-+ outport +===+
> Agree that it is better to clarify these points.
Hi Cathy,
I try to create multiple port-chains with the same port-pair-group, it failed. 
I think it is not allowed to do what you said, right?
(neutron) port-chain-create --flow-classifier fc --port-pair-group pg1 pc1 Port 
Pair Group(s) [u'17c9a0a5-a38f-4a75-834e-9aa213cd431f'] in use by Port Chain 
f3af530f-210a-4c51-9a02-f1835d5b1d85.
Neutron server returns request_ids: ['req-47e6a39b-677a-4ce3-9976-5dfcee0ac47f']
(neutron)

Cathy> when a port pair group is shared by multiple chains, these chains should 
be different, which means these chains should consist of different sequences of 
port pair groups. For example, chain 1 consists of  and chain 2 consists of "port-pair-group1, port-pair-group3, 
port-pair-group4>. But if the two chains are the same, i.e. they consist of the 
same sequence of port-pair-groups (e.g. the two chains both consist of 
), then it is not allowed since it does not make sense to 
create the same chain twice. Maybe your test scenario falls into the second 
case?



--
To view, visit https://review.openstack.org/333172
To unsubscribe, visit https://review.openstack.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c1827dcc81e48983a41120ec08983c571c5b2e9
Gerrit-PatchSet: 1
Gerrit-Project: openstack/networking-sfc
Gerrit-Branch: master
Gerrit-Owner: Louis Fourie 
>
Gerrit-Reviewer: Farhad Sunavala >
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: John McDowall 
>
Gerrit-Reviewer: Kyle Mestery >
Gerrit-Reviewer: Louis Fourie 
>
Gerrit-Reviewer: Na Zhu >
Gerrit-Reviewer: Richard Theis

Re: [openstack-dev] Change in openstack/networking-sfc[master]: Networking-sfc / OVN Driver

2016-06-27 Thread Na Zhu 
Hi Cathy,

Thanks your response.
Another question, how to create different port-chains for different 
tenants, and these chains consist of the same port pair group. In my test 
scenario, port-pair-group created by tenant A is not visible for tenant B.




Regards,
Juno Zhu
IBM China Development Labs (CDL) Cloud IaaS Lab
Email: na...@cn.ibm.com
5F, Building 10, 399 Keyuan Road, Zhangjiang Hi-Tech Park, Pudong New 
District, Shanghai, China (201203)



From:   Cathy Zhang 
To: Na Zhu/China/IBM@IBMCN, Henry Fourie , 
"OpenStack Development Mailing List (not for usage questions)" 

Cc: Ryan Moats , Kyle Mestery 
, Russell Bryant , Richard Theis 
, Stephen Wong , Vikram 
Choudhary , Farhad Sunavala 
, "John McDowall" 
Date:   2016/06/28 12:29
Subject:RE: Change in openstack/networking-sfc[master]: 
Networking-sfc / OVN Driver



Hi Na,

Please see inline for my reply.

Cathy

-Original Message-
From: Na Zhu (Code Review) [mailto:rev...@openstack.org] 
Sent: Monday, June 27, 2016 8:08 PM
To: Henry Fourie
Cc: Ryan Moats; Na Zhu; Kyle Mestery; Russell Bryant; Richard Theis; 
Stephen Wong; Cathy Zhang; Vikram Choudhary; Farhad Sunavala; John 
McDowall
Subject: Change in openstack/networking-sfc[master]: Networking-sfc / OVN 
Driver

Na Zhu has posted comments on this change.

Change subject: Networking-sfc / OVN Driver 
..


Patch Set 1:

(1 comment)

https://review.openstack.org/#/c/333172/1/doc/source/sfc_ovn_driver.rst
File doc/source/sfc_ovn_driver.rst:

Line 88: +-+   +-+ outport +===+
> Agree that it is better to clarify these points. 
Hi Cathy,
I try to create multiple port-chains with the same port-pair-group, it 
failed. I think it is not allowed to do what you said, right?
(neutron) port-chain-create --flow-classifier fc --port-pair-group pg1 pc1 
Port Pair Group(s) [u'17c9a0a5-a38f-4a75-834e-9aa213cd431f'] in use by 
Port Chain f3af530f-210a-4c51-9a02-f1835d5b1d85.
Neutron server returns request_ids: 
['req-47e6a39b-677a-4ce3-9976-5dfcee0ac47f']
(neutron)

Cathy> when a port pair group is shared by multiple chains, these chains 
should be different, which means these chains should consist of different 
sequences of port pair groups. For example, chain 1 consists of 
 and chain 2 consists of 
"port-pair-group1, port-pair-group3, port-pair-group4>. But if the two 
chains are the same, i.e. they consist of the same sequence of 
port-pair-groups (e.g. the two chains both consist of ), 
then it is not allowed since it does not make sense to create the same 
chain twice. Maybe your test scenario falls into the second case?



-- 
To view, visit https://review.openstack.org/333172
To unsubscribe, visit https://review.openstack.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c1827dcc81e48983a41120ec08983c571c5b2e9
Gerrit-PatchSet: 1
Gerrit-Project: openstack/networking-sfc
Gerrit-Branch: master
Gerrit-Owner: Louis Fourie 
Gerrit-Reviewer: Farhad Sunavala 
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: John McDowall 
Gerrit-Reviewer: Kyle Mestery 
Gerrit-Reviewer: Louis Fourie 
Gerrit-Reviewer: Na Zhu 
Gerrit-Reviewer: Richard Theis 
Gerrit-Reviewer: Russell Bryant 
Gerrit-Reviewer: Ryan Moats 
Gerrit-Reviewer: Stephen Wong 
Gerrit-Reviewer: cathy 
Gerrit-Reviewer: vikram.choudhary 
Gerrit-HasComments: Yes



__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Re: [openstack-dev] Change in openstack/networking-sfc[master]: Networking-sfc / OVN Driver

2016-06-27 Thread Cathy Zhang 
Hi Na,

Please see inline for my reply.

Cathy

-Original Message-
From: Na Zhu (Code Review) [mailto:rev...@openstack.org] 
Sent: Monday, June 27, 2016 8:08 PM
To: Henry Fourie
Cc: Ryan Moats; Na Zhu; Kyle Mestery; Russell Bryant; Richard Theis; Stephen 
Wong; Cathy Zhang; Vikram Choudhary; Farhad Sunavala; John McDowall
Subject: Change in openstack/networking-sfc[master]: Networking-sfc / OVN Driver

Na Zhu has posted comments on this change.

Change subject: Networking-sfc / OVN Driver 
..


Patch Set 1:

(1 comment)

https://review.openstack.org/#/c/333172/1/doc/source/sfc_ovn_driver.rst
File doc/source/sfc_ovn_driver.rst:

Line 88: +-+   +-+ outport +===+
> Agree that it is better to clarify these points. 
Hi Cathy,
I try to create multiple port-chains with the same port-pair-group, it failed. 
I think it is not allowed to do what you said, right?
(neutron) port-chain-create --flow-classifier fc --port-pair-group pg1 pc1 Port 
Pair Group(s) [u'17c9a0a5-a38f-4a75-834e-9aa213cd431f'] in use by Port Chain 
f3af530f-210a-4c51-9a02-f1835d5b1d85.
Neutron server returns request_ids: ['req-47e6a39b-677a-4ce3-9976-5dfcee0ac47f']
(neutron)

Cathy> when a port pair group is shared by multiple chains, these chains should 
be different, which means these chains should consist of different sequences of 
port pair groups. For example, chain 1 consists of  and chain 2 consists of "port-pair-group1, port-pair-group3, 
port-pair-group4>. But if the two chains are the same, i.e. they consist of the 
same sequence of port-pair-groups (e.g. the two chains both consist of 
), then it is not allowed since it does not make sense to 
create the same chain twice. Maybe your test scenario falls into the second 
case?



-- 
To view, visit https://review.openstack.org/333172
To unsubscribe, visit https://review.openstack.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5c1827dcc81e48983a41120ec08983c571c5b2e9
Gerrit-PatchSet: 1
Gerrit-Project: openstack/networking-sfc
Gerrit-Branch: master
Gerrit-Owner: Louis Fourie 
Gerrit-Reviewer: Farhad Sunavala 
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: John McDowall 
Gerrit-Reviewer: Kyle Mestery 
Gerrit-Reviewer: Louis Fourie 
Gerrit-Reviewer: Na Zhu 
Gerrit-Reviewer: Richard Theis 
Gerrit-Reviewer: Russell Bryant 
Gerrit-Reviewer: Ryan Moats 
Gerrit-Reviewer: Stephen Wong 
Gerrit-Reviewer: cathy 
Gerrit-Reviewer: vikram.choudhary 
Gerrit-HasComments: Yes
__
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev