yang: Corrected range format in YANG

[y-bharath14]

The range statement should specify a continuous range of values

[donaldsharp]

The range of values is 0 or 10000-max. so the original is correct. 0 means use the default and 10000...max is the allowed range for configuration. This change seems wrong

[y-bharath14]

The range of values is 0 or 10000-max. so the original is correct. 0 means use the default and 10000...max is the allowed range for configuration. This change seems wrong

Hello Donald Sharp,

How about including the below change..

leaf required-echo-receive-interval {
  type uint32 {
      range "10000..max"; <---- Allow user to configure in the range.
  }
  default 0;   <---- Default value is set it to 0.

}

Thanks & Regards,
Bharath

[mjstapp]

I ... don't think that would work: the "default" value would be out of the valid range, and it would be impossible ever to revert the config to the default value - right?

Hello Donald Sharp,

How about including the below change..

leaf required-echo-receive-interval {
  type uint32 {
      range "10000..max"; <---- Allow user to configure in the range.
  }
  default 0;   <---- Default value is set it to 0.

}

Thanks & Regards, Bharath

[y-bharath14]

I ... don't think that would work: the "default" value would be out of the valid range, and it would be impossible ever to revert the config to the default value - right?

Hello Donald Sharp,
How about including the below change..

leaf required-echo-receive-interval {
  type uint32 {
      range "10000..max"; <---- Allow user to configure in the range.
  }
  default 0;   <---- Default value is set it to 0.

}
Thanks & Regards, Bharath

Hello Mark Stapp, Donald Sharp

Yes, you are correct . '0' should be part of range(Original one).
If the default value for "required-echo-receive-interval" is "0", then default 0 can be used. Otherwise, we can discard this PR.

Thanks & Regards,
Bharath

[choppsv1]

.

.

[choppsv1]

I don't think the problem is understood here. Rather than talk about a solution, can you state what the problem you are seeing is?