Re: RFC new doc section for size format

2023-05-17 Thread Christopher Faulet

Le 5/17/23 à 17:16, Daniel Epperson a écrit :

I have attached the patch.


Thanks Daniel ! Too late for the dev12 but now merged :)

--
Christopher Faulet




Re: RFC new doc section for size format

2023-05-17 Thread Willy Tarreau
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 
> > 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 = 8640ms
> > 
> > 
> > -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



Re: RFC new doc section for size format

2023-05-17 Thread Daniel Epperson

I have attached the patch.


On 5/17/2023 6:02 AM, 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 
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 = 8640ms


-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





Daniel, could you send a real patch to ease the merge ?


--
Daniel Epperson
Sr. Systems Engineer
https://www.haproxy.com/
From 3652208cbe7b4521c8e219081a14647f32454129 Mon Sep 17 00:00:00 2001
From: Daniel Epperson 
Date: Mon, 15 May 2023 12:45:27 -0700
Subject: [PATCH] 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.
---
 doc/configuration.txt | 21 +++--
 1 file changed, 19 insertions(+), 2 deletions(-)

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 = 8640ms
 
 
-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
-- 
2.34.1



Re: RFC new doc section for size format

2023-05-17 Thread Christopher Faulet

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 
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 = 8640ms


-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




Daniel, could you send a real patch to ease the merge ?

--
Christopher Faulet




Re: RFC new doc section for size format

2023-05-17 Thread Christopher Faulet

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 
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 = 8640ms


-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 ?

--
Christopher Faulet