On Wed, May 17, 2023 at 09:49:01AM +0200, Christopher Faulet wrote:
> Le 5/15/23 à 21:48, Daniel Epperson a écrit :
> > Hello,
> > 
> > I filed #2153 to add a section to the manual, now I'm looking for
> > comments to implement the fix via a patch. This is what I have so far.
> > If it is acceptable, please merge, or let me know what to correct.
> > 
> > Author: Daniel Epperson <depper...@haproxy.com>
> > Date:   Mon May 15 12:45:27 2023 -0700
> > 
> >       DOC: add size format section to manual
> > 
> >       The manual refers to an HAProxy size format but does not define it.
> >       This patch adds a section to the manual to define the HAProxy size
> >       format.
> > 
> > diff --git a/doc/configuration.txt b/doc/configuration.txt
> > index 43e4bffeb..ae7f7ad65 100644
> > --- a/doc/configuration.txt
> > +++ b/doc/configuration.txt
> > @@ -42,7 +42,8 @@ Summary
> >    2.3.      Environment variables
> >    2.4.      Conditional blocks
> >    2.5.      Time format
> > -2.6.      Examples
> > +2.6.      Size format
> > +2.7.      Examples
> > 
> >    3.    Global parameters
> >    3.1.      Process management and security
> > @@ -980,7 +981,23 @@ for every keyword. Supported units are :
> >      - d  : days.    1d = 24h = 1440m = 86400s = 86400000ms
> > 
> > 
> > -2.6. Examples
> > +2.6. Size format
> > +----------------
> > +
> > +Some parameters involve values representing size, such as bandwidth limits.
> > +These values are generally expressed in bytes (unless explicitly stated
> > +otherwise) but may be expressed in any other unit by suffixing the unit
> > to the
> > +numeric value. It is important to consider this because it will not be
> > repeated
> > +for every keyword. Supported units are case insensitive :
> > +
> > +  - k : kilobytes. 1 kilobyte = 1024 bytes
> > +  - m : megabytes. 1 megabyte = 1048576 bytes
> > +  - g : gigabytes. 1 gigabyte = 1073741824 bytes
> > +
> > +Both time and size formats require integers, decimal notation is not
> > allowed.
> > +
> > +
> > +2.7. Examples
> >    -------------
> > 
> >        # Simple configuration for an HTTP proxy listening on port 80 on all
> > 
> > 
> 
> Thanks Daniel,
> 
> It seems good to me. Any comment before I merge it ?

It looks good to me as well, thank you both!

Willy

Reply via email to