Message ID | 20171130183809.24301-1-blp@ovn.org |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev] sflow: Correctly document setup command. | expand |
On 11/30/2017 10:38 AM, Ben Pfaff wrote: > Reported-by: Shivaram Mysore <shivaram.mysore@gmail.com> > Signed-off-by: Ben Pfaff <blp@ovn.org> > --- > Documentation/howto/sflow.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/howto/sflow.rst b/Documentation/howto/sflow.rst > index 1152a7db567a..66f4856b3340 100644 > --- a/Documentation/howto/sflow.rst > +++ b/Documentation/howto/sflow.rst > @@ -105,7 +105,7 @@ Still on `host1`, run the following command to create an sFlow configuration > and attach it to bridge br0:: > > $ ovs-vsctl -- --id=@sflow create sflow agent=${AGENT_IP} \ > - target="${COLLECTOR_IP}:${COLLECTOR_PORT}" header=${HEADER_BYTES} \ > + target='"${COLLECTOR_IP}:${COLLECTOR_PORT}"' header=${HEADER_BYTES} \ > sampling=${SAMPLING_N} polling=${POLLING_SECS} \ > -- set bridge br0 sflow=@sflow > Looks good. Reviewed-by: Greg Rose <gvrose8192@gmail.com>
On Mon, Dec 04, 2017 at 08:32:28AM -0800, Gregory Rose wrote: > On 11/30/2017 10:38 AM, Ben Pfaff wrote: > >Reported-by: Shivaram Mysore <shivaram.mysore@gmail.com> > >Signed-off-by: Ben Pfaff <blp@ovn.org> > >--- > > Documentation/howto/sflow.rst | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > >diff --git a/Documentation/howto/sflow.rst b/Documentation/howto/sflow.rst > >index 1152a7db567a..66f4856b3340 100644 > >--- a/Documentation/howto/sflow.rst > >+++ b/Documentation/howto/sflow.rst > >@@ -105,7 +105,7 @@ Still on `host1`, run the following command to create an sFlow configuration > > and attach it to bridge br0:: > > $ ovs-vsctl -- --id=@sflow create sflow agent=${AGENT_IP} \ > >- target="${COLLECTOR_IP}:${COLLECTOR_PORT}" header=${HEADER_BYTES} \ > >+ target='"${COLLECTOR_IP}:${COLLECTOR_PORT}"' header=${HEADER_BYTES} \ > > sampling=${SAMPLING_N} polling=${POLLING_SECS} \ > > -- set bridge br0 sflow=@sflow > > Looks good. > > Reviewed-by: Greg Rose <gvrose8192@gmail.com> > Thanks, applied to master. I changed '"..."' to "\"...\"" because I realized that using '"..."' will prevent the intended shell variable expansion.
diff --git a/Documentation/howto/sflow.rst b/Documentation/howto/sflow.rst index 1152a7db567a..66f4856b3340 100644 --- a/Documentation/howto/sflow.rst +++ b/Documentation/howto/sflow.rst @@ -105,7 +105,7 @@ Still on `host1`, run the following command to create an sFlow configuration and attach it to bridge br0:: $ ovs-vsctl -- --id=@sflow create sflow agent=${AGENT_IP} \ - target="${COLLECTOR_IP}:${COLLECTOR_PORT}" header=${HEADER_BYTES} \ + target='"${COLLECTOR_IP}:${COLLECTOR_PORT}"' header=${HEADER_BYTES} \ sampling=${SAMPLING_N} polling=${POLLING_SECS} \ -- set bridge br0 sflow=@sflow
Reported-by: Shivaram Mysore <shivaram.mysore@gmail.com> Signed-off-by: Ben Pfaff <blp@ovn.org> --- Documentation/howto/sflow.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)