Re: [netmod] I-D Action: draft-ietf-netmod-acl-model-16.txt

2018-02-07 Thread Kristian Larsson
On 2018-02-06 19:36, Mahesh Jethanandani wrote: Kristian, As I commented on the PR, putting the ‘container’ inside of the ‘choice’ statement allows me to collapse the ‘container’ and the ‘case’ statement into a single ‘container’ statement. With your changes, I see an additional ‘case’ stat

Re: [netmod] I-D Action: draft-ietf-netmod-acl-model-16.txt

2018-02-06 Thread Mahesh Jethanandani
Kristian, As I commented on the PR, putting the ‘container’ inside of the ‘choice’ statement allows me to collapse the ‘container’ and the ‘case’ statement into a single ‘container’ statement. With your changes, I see an additional ‘case’ statement, bloating the model in four places. Cheers.

Re: [netmod] I-D Action: draft-ietf-netmod-acl-model-16.txt

2018-02-06 Thread Kristian Larsson
Mahesh, I suppose, since you posted the update Friday night, that I missed my chance of prettifying the source/destination port choice/container structure that was just added. If not, it's in a PR towards your repo - https://github.com/mjethanandani/acl-model/pull/4 Kind regards, Kristian

Re: [netmod] I-D Action: draft-ietf-netmod-acl-model-16.txt

2018-02-02 Thread Mahesh Jethanandani
This update addresses the comments that were received as part of LC. For those of you who commented on the draft during the LC, please verify that your comments have been addressed. Thanks. > On Feb 2, 2018, at 5:26 PM, internet-dra...@ietf.org wrote: > > > A New Internet-Draft is available f