Welcome! Log In Create A New Profile

Advanced

[PATCH] REGTEST/MINOR: Wrong URI syntax.

Posted by Frederic Lecaille 
Frederic Lecaille
[PATCH] REGTEST/MINOR: Wrong URI syntax.
July 12, 2018 11:10AM
This is a patch to fix the issue reported by Ilya Shipitsin in this thread.

Fred.
Willy Tarreau
Re: [PATCH] REGTEST/MINOR: Wrong URI syntax.
July 12, 2018 06:00PM
On Thu, Jul 12, 2018 at 11:05:30AM +0200, Frederic Lecaille wrote:
> This is a patch to fix the issue reported by Ilya Shipitsin in this thread.

Applied, thank you Fred.

Willy
Илья Шипицин
Re: [PATCH] REGTEST/MINOR: Wrong URI syntax.
July 13, 2018 10:00AM
sorry, I did not test it on centos 7

https://gitlab.com/chipitsine/haproxy/-/jobs/81501288


(I could not find out what's that, error message is strange, I'll try to
investigate on separate vm)

чт, 12 июл. 2018 г. в 14:08, Frederic Lecaille <[email protected]>:

> This is a patch to fix the issue reported by Ilya Shipitsin in this thread.
>
> Fred.
>
Frederic Lecaille
Re: [PATCH] REGTEST/MINOR: Wrong URI syntax.
July 13, 2018 10:20AM
On 07/13/2018 09:53 AM, Илья Шипицин wrote:
> sorry, I did not test it on centos 7
>
> https://gitlab.com/chipitsine/haproxy/-/jobs/81501288
>
>
> (I could not find out what's that, error message is strange, I'll try to
> investigate on separate vm)
>
> чт, 12 июл. 2018 г. в 14:08, Frederic Lecaille <[email protected]
> <mailto:[email protected]>>:
>
> This is a patch to fix the issue reported by Ilya Shipitsin in this
> thread.
>
> Fred.
>

This is becoming boring. This is why we should not use external programs
like curl. We will perhaps have to disable this vtc file.

Try one of these patches with a preference for reg-tests.2.diff.

They both work on my linux host with curl 7.38.0.
Илья Шипицин
Re: [PATCH] REGTEST/MINOR: Wrong URI syntax.
July 13, 2018 10:30AM
пт, 13 июл. 2018 г. в 13:08, Frederic Lecaille <[email protected]>:

> On 07/13/2018 09:53 AM, Илья Шипицин wrote:
> > sorry, I did not test it on centos 7
> >
> > https://gitlab.com/chipitsine/haproxy/-/jobs/81501288
> >
> >
> > (I could not find out what's that, error message is strange, I'll try to
> > investigate on separate vm)
> >
> > чт, 12 июл. 2018 г. в 14:08, Frederic Lecaille <[email protected]
> > <mailto:[email protected]>>:
> >
> > This is a patch to fix the issue reported by Ilya Shipitsin in this
> > thread.
> >
> > Fred.
> >
>
> This is becoming boring. This is why we should not use external programs
> like curl. We will perhaps have to disable this vtc file.
>


once CI is up and running it will not be boring anymore.


>
> Try one of these patches with a preference for reg-tests.2.diff.
>

I confirm test is green now


>
> They both work on my linux host with curl 7.38.0.
>
>
Frederic Lecaille
Re: [PATCH] REGTEST/MINOR: Wrong URI syntax.
July 13, 2018 10:30AM
On 07/13/2018 10:17 AM, Илья Шипицин wrote:
>
>
> пт, 13 июл. 2018 г. в 13:08, Frederic Lecaille <[email protected]
> <mailto:[email protected]>>:
>
> On 07/13/2018 09:53 AM, Илья Шипицин wrote:
> > sorry, I did not test it on centos 7
> >
> > https://gitlab.com/chipitsine/haproxy/-/jobs/81501288
> >
> >
> > (I could not find out what's that, error message is strange, I'll
> try to
> > investigate on separate vm)
> >
> > чт, 12 июл. 2018 г. в 14:08, Frederic Lecaille
> <[email protected] <mailto:[email protected]>
> > <mailto:[email protected] <mailto:[email protected]m>>>:
> >
> >     This is a patch to fix the issue reported by Ilya Shipitsin
> in this
> >     thread.
> >
> >     Fred.
> >
>
> This is becoming boring. This is why we should not use external
> programs
> like curl. We will perhaps have to disable this vtc file.
>
>
>
> once CI is up and running it will not be boring anymore.
>
>
> Try one of these patches with a preference for reg-tests.2.diff.
>
>
> I confirm test is green now


Thank you for your good job Ilya.

Regards.
Frederic Lecaille
[PATCH] REGTEST/MINOR: Unexpected curl URL globling.
July 13, 2018 11:00AM
On 07/12/2018 05:52 PM, Willy Tarreau wrote:
> On Thu, Jul 12, 2018 at 11:05:30AM +0200, Frederic Lecaille wrote:
>> This is a patch to fix the issue reported by Ilya Shipitsin in this thread.
>
> Applied, thank you Fred.
>
> Willy
>

Another patch to fix a remaining issue with this boring VTC file
reported by Ilya.

Fred.
Willy Tarreau
Re: [PATCH] REGTEST/MINOR: Unexpected curl URL globling.
July 13, 2018 11:10AM
On Fri, Jul 13, 2018 at 10:51:07AM +0200, Frederic Lecaille wrote:
> Another patch to fix a remaining issue with this boring VTC file reported by
> Ilya.

Applied, thanks Fred!

willy
Sorry, only registered users may post in this forum.

Click here to login