Failed: apache/httpd#1974 (trunk - 1032155)

2021-09-23 Thread Travis CI
Build Update for apache/httpd
-

Build: #1974
Status: Failed

Duration: 22 mins and 5 secs
Commit: 1032155 (trunk)
Author: Christophe Jaillet
Message: Tentative update to reduce noise in travis log (i.e. tons of AH02637).

Just switch 2 tests in 'dialup_handler()' when deciding to DECLINE something.

If mod_dialup is not configured, there is no need lo log anything about an
mpm that doesn't support something that is finally not used.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1893582 
13f79535-47bb-0310-9956-ffa450edef68

View the changeset: 
https://github.com/apache/httpd/compare/61a5af6ac2c2...1032155c5720

View the full build log and details: 
https://app.travis-ci.com/github/apache/httpd/builds/238377854?utm_medium=notification_source=email


--

You can unsubscribe from build emails from the apache/httpd repository going to 
https://app.travis-ci.com/account/preferences/unsubscribe?repository=16806660_medium=notification_source=email.
Or unsubscribe from *all* email updating your settings at 
https://app.travis-ci.com/account/preferences/unsubscribe?utm_medium=notification_source=email.
Or configure specific recipients for build notifications in your .travis.yml 
file. See https://docs.travis-ci.com/user/notifications.



Errored: apache/httpd#1972 (trunk - 61a5af6)

2021-09-23 Thread Travis CI
Build Update for apache/httpd
-

Build: #1972
Status: Errored

Duration: 34 mins and 25 secs
Commit: 61a5af6 (trunk)
Author: Stefan Eissing
Message: mod_http2: fixed speling error.


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1893564 
13f79535-47bb-0310-9956-ffa450edef68

View the changeset: 
https://github.com/apache/httpd/compare/ebf7cf2faf3f...61a5af6ac2c2

View the full build log and details: 
https://app.travis-ci.com/github/apache/httpd/builds/238345076?utm_medium=notification_source=email


--

You can unsubscribe from build emails from the apache/httpd repository going to 
https://app.travis-ci.com/account/preferences/unsubscribe?repository=16806660_medium=notification_source=email.
Or unsubscribe from *all* email updating your settings at 
https://app.travis-ci.com/account/preferences/unsubscribe?utm_medium=notification_source=email.
Or configure specific recipients for build notifications in your .travis.yml 
file. See https://docs.travis-ci.com/user/notifications.



Re: svn commit: r1893563 - in /httpd/httpd/trunk/modules/http2: h2_version.h h2_workers.c

2021-09-23 Thread Christopher Schultz

Stefan,

On 9/23/21 09:52, ste...@eissing.org wrote:




Am 23.09.2021 um 15:51 schrieb Ruediger Pluem :



On 9/23/21 3:44 PM, ic...@apache.org wrote:

Author: icing
Date: Thu Sep 23 13:44:58 2021
New Revision: 1893563

URL: http://svn.apache.org/viewvc?rev=1893563=rev
Log:
  * mod_http2: fix version suffix after sync with github
Make shutdown worker wait loop robust against timed wait
interruptions and report the correct seconds waited.


Modified:
httpd/httpd/trunk/modules/http2/h2_version.h
httpd/httpd/trunk/modules/http2/h2_workers.c




Modified: httpd/httpd/trunk/modules/http2/h2_workers.c
URL: 
http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_workers.c?rev=1893563=1893562=1893563=diff
==
--- httpd/httpd/trunk/modules/http2/h2_workers.c (original)
+++ httpd/httpd/trunk/modules/http2/h2_workers.c Thu Sep 23 13:44:58 2021
@@ -319,9 +319,9 @@ static void workers_abort_idle(h2_worker
static apr_status_t workers_pool_cleanup(void *data)
{
 h2_workers *workers = data;
-apr_time_t timout = apr_time_from_sec(1);
+apr_time_t end, timout = apr_time_from_sec(1);


timeout instead of timout ? :-)


It's a short timeout. ;-)


No it's not: apt_time_t is a 64-bit value, so it's a long long timeout. ;)

-chris


Broken: apache/httpd#1967 (trunk - 505c4fc)

2021-09-23 Thread Travis CI
Build Update for apache/httpd
-

Build: #1967
Status: Broken

Duration: 10 mins and 41 secs
Commit: 505c4fc (trunk)
Author: Ruediger Pluem
Message: * t/modules/http2.t is gone since 1893547. So do not use it any longer 
in
  HTTP/2 tests and remove the dependency on Protocol::HTTP2::Client. 
  Update test/travis_run_linux.sh to handle NO_TEST_FRAMEWORK.


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1893549 
13f79535-47bb-0310-9956-ffa450edef68

View the changeset: 
https://github.com/apache/httpd/compare/0557043c0244...505c4fcb9a3f

View the full build log and details: 
https://app.travis-ci.com/github/apache/httpd/builds/238331945?utm_medium=notification_source=email


--

You can unsubscribe from build emails from the apache/httpd repository going to 
https://app.travis-ci.com/account/preferences/unsubscribe?repository=16806660_medium=notification_source=email.
Or unsubscribe from *all* email updating your settings at 
https://app.travis-ci.com/account/preferences/unsubscribe?utm_medium=notification_source=email.
Or configure specific recipients for build notifications in your .travis.yml 
file. See https://docs.travis-ci.com/user/notifications.



Re: svn commit: r1893563 - in /httpd/httpd/trunk/modules/http2: h2_version.h h2_workers.c

2021-09-23 Thread ste...@eissing.org



> Am 23.09.2021 um 15:51 schrieb Ruediger Pluem :
> 
> 
> 
> On 9/23/21 3:44 PM, ic...@apache.org wrote:
>> Author: icing
>> Date: Thu Sep 23 13:44:58 2021
>> New Revision: 1893563
>> 
>> URL: http://svn.apache.org/viewvc?rev=1893563=rev
>> Log:
>>  * mod_http2: fix version suffix after sync with github
>>Make shutdown worker wait loop robust against timed wait
>>interruptions and report the correct seconds waited.
>> 
>> 
>> Modified:
>>httpd/httpd/trunk/modules/http2/h2_version.h
>>httpd/httpd/trunk/modules/http2/h2_workers.c
>> 
> 
>> Modified: httpd/httpd/trunk/modules/http2/h2_workers.c
>> URL: 
>> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_workers.c?rev=1893563=1893562=1893563=diff
>> ==
>> --- httpd/httpd/trunk/modules/http2/h2_workers.c (original)
>> +++ httpd/httpd/trunk/modules/http2/h2_workers.c Thu Sep 23 13:44:58 2021
>> @@ -319,9 +319,9 @@ static void workers_abort_idle(h2_worker
>> static apr_status_t workers_pool_cleanup(void *data)
>> {
>> h2_workers *workers = data;
>> -apr_time_t timout = apr_time_from_sec(1);
>> +apr_time_t end, timout = apr_time_from_sec(1);
> 
> timeout instead of timout ? :-)

It's a short timeout. ;-)

> 
>> apr_status_t rv;
>> -int i, n = 5;
>> +int n, wait_sec = 5;
>> 
>> ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, workers->s,
>>  "h2_workers: cleanup %d workers idling",
> 
> 
> Regards
> 
> Rüdiger



Re: svn commit: r1893563 - in /httpd/httpd/trunk/modules/http2: h2_version.h h2_workers.c

2021-09-23 Thread Ruediger Pluem



On 9/23/21 3:44 PM, ic...@apache.org wrote:
> Author: icing
> Date: Thu Sep 23 13:44:58 2021
> New Revision: 1893563
> 
> URL: http://svn.apache.org/viewvc?rev=1893563=rev
> Log:
>   * mod_http2: fix version suffix after sync with github
> Make shutdown worker wait loop robust against timed wait
> interruptions and report the correct seconds waited.
> 
> 
> Modified:
> httpd/httpd/trunk/modules/http2/h2_version.h
> httpd/httpd/trunk/modules/http2/h2_workers.c
> 

> Modified: httpd/httpd/trunk/modules/http2/h2_workers.c
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http2/h2_workers.c?rev=1893563=1893562=1893563=diff
> ==
> --- httpd/httpd/trunk/modules/http2/h2_workers.c (original)
> +++ httpd/httpd/trunk/modules/http2/h2_workers.c Thu Sep 23 13:44:58 2021
> @@ -319,9 +319,9 @@ static void workers_abort_idle(h2_worker
>  static apr_status_t workers_pool_cleanup(void *data)
>  {
>  h2_workers *workers = data;
> -apr_time_t timout = apr_time_from_sec(1);
> +apr_time_t end, timout = apr_time_from_sec(1);

timeout instead of timout ? :-)

>  apr_status_t rv;
> -int i, n = 5;
> +int n, wait_sec = 5;
>  
>  ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, workers->s,
>   "h2_workers: cleanup %d workers idling",


Regards

Rüdiger


Re: svn commit: r1893549 - in /httpd/httpd/trunk: .travis.yml test/travis_run_linux.sh

2021-09-23 Thread ste...@eissing.org
Thanks, Rüdiger. I was about to go looking for that...

> Am 23.09.2021 um 13:12 schrieb rpl...@apache.org:
> 
> Author: rpluem
> Date: Thu Sep 23 11:12:25 2021
> New Revision: 1893549
> 
> URL: http://svn.apache.org/viewvc?rev=1893549=rev
> Log:
> * t/modules/http2.t is gone since 1893547. So do not use it any longer in
>  HTTP/2 tests and remove the dependency on Protocol::HTTP2::Client. 
>  Update test/travis_run_linux.sh to handle NO_TEST_FRAMEWORK.
> 
> Modified:
>httpd/httpd/trunk/.travis.yml
>httpd/httpd/trunk/test/travis_run_linux.sh
> 
> Modified: httpd/httpd/trunk/.travis.yml
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/trunk/.travis.yml?rev=1893549=1893548=1893549=diff
> ==
> --- httpd/httpd/trunk/.travis.yml (original)
> +++ httpd/httpd/trunk/.travis.yml Thu Sep 23 11:12:25 2021
> @@ -271,7 +271,7 @@ jobs:
>   env: APR_VERSION=1.7.0
>APU_VERSION=1.6.1 APU_CONFIG="--with-crypto"
>CONFIG="--enable-mods-shared=reallyall --with-mpm=event 
> --enable-mpms-shared=event"
> -   TESTS="t/modules/http2.t" TEST_INSTALL=1 TEST_H2=1
> +   NO_TEST_FRAMEWORK=1 TEST_INSTALL=1 TEST_H2=1
>   addons:
> apt:
>   sources:
> @@ -382,7 +382,7 @@ before_install:
>   - if [ "$TRAVIS_OS_NAME" = "linux" -a ! -v SKIP_TESTING ]; then
> if [ -v CLEAR_CACHE ]; then rm -rf ~/perl5; fi;
> cpanm --local-lib=~/perl5 local::lib && eval $(perl -I 
> ~/perl5/lib/perl5/ -Mlocal::lib);
> -CC=gcc cpanm --notest Net::SSL LWP::Protocol::https ExtUtils::Embed 
> Test::More AnyEvent DateTime HTTP::DAV Protocol::HTTP2::Client FCGI 
> AnyEvent::WebSocket::Client Apache::Test;
> +CC=gcc cpanm --notest Net::SSL LWP::Protocol::https ExtUtils::Embed 
> Test::More AnyEvent DateTime HTTP::DAV FCGI AnyEvent::WebSocket::Client 
> Apache::Test;
> fi
> 
> before_script:
> 
> Modified: httpd/httpd/trunk/test/travis_run_linux.sh
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/trunk/test/travis_run_linux.sh?rev=1893549=1893548=1893549=diff
> ==
> --- httpd/httpd/trunk/test/travis_run_linux.sh (original)
> +++ httpd/httpd/trunk/test/travis_run_linux.sh Thu Sep 23 11:12:25 2021
> @@ -94,22 +94,24 @@ if ! test -v SKIP_TESTING; then
> # Try to keep all potential coredumps from all processes
> sudo sysctl -w kernel.core_uses_pid=1 2>/dev/null || true
> 
> -if test -v WITH_TEST_SUITE; then
> -make check TESTS="${TESTS}" TEST_CONFIG="${TEST_ARGS}"
> -RV=$?
> -else
> -test -v TEST_INSTALL || make install
> -pushd test/perl-framework
> -perl Makefile.PL -apxs $PREFIX/bin/apxs
> -make test APACHE_TEST_EXTRA_ARGS="${TEST_ARGS} ${TESTS}"
> +if ! test -v NO_TEST_FRAMEWORK; then
> +if test -v WITH_TEST_SUITE; then
> +make check TESTS="${TESTS}" TEST_CONFIG="${TEST_ARGS}"
> RV=$?
> -popd
> -fi
> +else
> +test -v TEST_INSTALL || make install
> +pushd test/perl-framework
> +perl Makefile.PL -apxs $PREFIX/bin/apxs
> +make test APACHE_TEST_EXTRA_ARGS="${TEST_ARGS} ${TESTS}"
> +RV=$?
> +popd
> +fi
> 
> -# Skip further testing if a core dump was created during the test
> -# suite run above.
> -if test $RV -eq 0 && test -n "`ls test/perl-framework/t/core{,.*} 
> 2>/dev/null`"; then
> -RV=4
> +# Skip further testing if a core dump was created during the test
> +# suite run above.
> +if test $RV -eq 0 && test -n "`ls test/perl-framework/t/core{,.*} 
> 2>/dev/null`"; then
> +RV=4
> +fi
> fi
> 
> if test -v TEST_SSL -a $RV -eq 0; then
> 
> 



Fixed: apache/httpd#1959 (trunk - 0557043)

2021-09-23 Thread Travis CI
Build Update for apache/httpd
-

Build: #1959
Status: Fixed

Duration: 11 mins and 41 secs
Commit: 0557043 (trunk)
Author: Yann Ylavic
Message: mod_rewrite: Follow up to r1893519: Handle potential 
"unix://[authority]/".

is_absolute_uri() should return the offset to the authority part, if any.



git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1893532 
13f79535-47bb-0310-9956-ffa450edef68

View the changeset: 
https://github.com/apache/httpd/compare/8720881b0634...0557043c0244

View the full build log and details: 
https://app.travis-ci.com/github/apache/httpd/builds/238315502?utm_medium=notification_source=email


--

You can unsubscribe from build emails from the apache/httpd repository going to 
https://app.travis-ci.com/account/preferences/unsubscribe?repository=16806660_medium=notification_source=email.
Or unsubscribe from *all* email updating your settings at 
https://app.travis-ci.com/account/preferences/unsubscribe?utm_medium=notification_source=email.
Or configure specific recipients for build notifications in your .travis.yml 
file. See https://docs.travis-ci.com/user/notifications.



Re: svn commit: r1893542 [1/12] - in /httpd/httpd/branches/2.4.x: ./ test/modules/ test/modules/http2/ test/modules/http2/conf/ test/modules/http2/data/ test/modules/http2/htdocs/ test/modules/http2/h

2021-09-23 Thread ste...@eissing.org



> Am 23.09.2021 um 11:32 schrieb Yann Ylavic :
> 
> On Thu, Sep 23, 2021 at 10:56 AM  wrote:
>> 
>> Author: icing
>> Date: Thu Sep 23 08:56:33 2021
>> New Revision: 1893542
>> 
>> URL: http://svn.apache.org/viewvc?rev=1893542=rev
>> Log:
>> * test: adding the modules/http2 test suite for easy availability
>>   in 2.4.x.
>> * configure.in: adding the test/modules/http2 .in files to process
> 
> Is this a backport or a special tests for 2.4.x?
> (wondering why there is no mergeinfo)
> 
> Cheers;
> Yann.
> 
> PS: sorry for being picky, but later merging gets easier in the end if
> we track merges..

Ok, ok. 


Re: svn commit: r1893543 - /httpd/httpd/branches/2.4.x/STATUS

2021-09-23 Thread ste...@eissing.org



> Am 23.09.2021 um 11:28 schrieb Yann Ylavic :
> 
> On Thu, Sep 23, 2021 at 11:14 AM  wrote:
>> 
>> Author: icing
>> Date: Thu Sep 23 09:13:59 2021
>> New Revision: 1893543
>> 
>> URL: http://svn.apache.org/viewvc?rev=1893543=rev
>> Log:
>> Reload of proposed mod_http2 backport of server graceful handling,
>> including all the tiny code changes that were never proposed.
>> 
>> 
>> Modified:
>>httpd/httpd/branches/2.4.x/STATUS
>> 
>> Modified: httpd/httpd/branches/2.4.x/STATUS
>> URL: 
>> http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1893543=1893542=1893543=diff
>> ==
>> --- httpd/httpd/branches/2.4.x/STATUS (original)
>> +++ httpd/httpd/branches/2.4.x/STATUS Thu Sep 23 09:13:59 2021
>> @@ -166,12 +166,14 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
>>   *) mod_http2: when a server is restarted gracefully, any idle h2 worker
>>  threads are shut down immediately.
>>  Also, change OpenSSL API use for deprecations in OpenSLL 3.0.
>> + Adds all other, never proposed code changes to make a clean
>> + sync of http2 sources.
>>  trunk patch: http://svn.apache.org/r1893214
>>   http://svn.apache.org/r1893215
>>   http://svn.apache.org/r1893220
>> - 2.4.x patch: svn merge -c 1893214,1893215,1893220 ^/httpd/httpd/trunk .
>> +  and other never proposed code changes
>> + PR: https://github.com/apache/httpd/pull/270
> 
> This means that the never proposed code changes are in the PR only and
> not in trunk?

The code in 2.4.x would after this be identical to trunk, including all the
tiny things over the years that have never been proposed.

> Cheers;
> Yann.



Re: svn commit: r1893542 [1/12] - in /httpd/httpd/branches/2.4.x: ./ test/modules/ test/modules/http2/ test/modules/http2/conf/ test/modules/http2/data/ test/modules/http2/htdocs/ test/modules/http2/h

2021-09-23 Thread Yann Ylavic
On Thu, Sep 23, 2021 at 10:56 AM  wrote:
>
> Author: icing
> Date: Thu Sep 23 08:56:33 2021
> New Revision: 1893542
>
> URL: http://svn.apache.org/viewvc?rev=1893542=rev
> Log:
>  * test: adding the modules/http2 test suite for easy availability
>in 2.4.x.
>  * configure.in: adding the test/modules/http2 .in files to process

Is this a backport or a special tests for 2.4.x?
(wondering why there is no mergeinfo)

Cheers;
Yann.

PS: sorry for being picky, but later merging gets easier in the end if
we track merges..


Re: svn commit: r1893543 - /httpd/httpd/branches/2.4.x/STATUS

2021-09-23 Thread Yann Ylavic
On Thu, Sep 23, 2021 at 11:14 AM  wrote:
>
> Author: icing
> Date: Thu Sep 23 09:13:59 2021
> New Revision: 1893543
>
> URL: http://svn.apache.org/viewvc?rev=1893543=rev
> Log:
> Reload of proposed mod_http2 backport of server graceful handling,
> including all the tiny code changes that were never proposed.
>
>
> Modified:
> httpd/httpd/branches/2.4.x/STATUS
>
> Modified: httpd/httpd/branches/2.4.x/STATUS
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1893543=1893542=1893543=diff
> ==
> --- httpd/httpd/branches/2.4.x/STATUS (original)
> +++ httpd/httpd/branches/2.4.x/STATUS Thu Sep 23 09:13:59 2021
> @@ -166,12 +166,14 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
>*) mod_http2: when a server is restarted gracefully, any idle h2 worker
>   threads are shut down immediately.
>   Also, change OpenSSL API use for deprecations in OpenSLL 3.0.
> + Adds all other, never proposed code changes to make a clean
> + sync of http2 sources.
>   trunk patch: http://svn.apache.org/r1893214
>http://svn.apache.org/r1893215
>http://svn.apache.org/r1893220
> - 2.4.x patch: svn merge -c 1893214,1893215,1893220 ^/httpd/httpd/trunk .
> +  and other never proposed code changes
> + PR: https://github.com/apache/httpd/pull/270

This means that the never proposed code changes are in the PR only and
not in trunk?

Cheers;
Yann.


Re: svn commit: r1893530 - /httpd/httpd/branches/2.4.x/STATUS

2021-09-23 Thread ste...@eissing.org



> Am 23.09.2021 um 09:25 schrieb rpl...@apache.org:
> 
> Author: rpluem
> Date: Thu Sep 23 07:25:24 2021
> New Revision: 1893530
> 
> URL: http://svn.apache.org/viewvc?rev=1893530=rev
> Log:
> * Comment [skip ci]
> 
> Modified:
>httpd/httpd/branches/2.4.x/STATUS
> 
> Modified: httpd/httpd/branches/2.4.x/STATUS
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1893530=1893529=1893530=diff
> ==
> --- httpd/httpd/branches/2.4.x/STATUS (original)
> +++ httpd/httpd/branches/2.4.x/STATUS Thu Sep 23 07:25:24 2021
> @@ -163,6 +163,7 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
>   http://svn.apache.org/1893220
>  2.4.x patch: svn merge -c 1893214,1893215,1893220 ^/httpd/httpd/trunk .
>  +1: icing,
> + rpluem says: This one has merge conflicts.

Arg. modules/http2 in 2.4.x and trunk have been gone out of sync
and I am wading through changesets that have never been backported
and cause the newer changes to give merge conflicts.

FTR: I hate our trunk development model.




Re: svn commit: r1893437 - in /httpd/test/framework/trunk/t: conf/extra.conf.in modules/heartbeat.t

2021-09-23 Thread Ruediger Pluem



On 9/19/21 1:49 PM, jaillet...@apache.org wrote:
> Author: jailletc36
> Date: Sun Sep 19 11:49:33 2021
> New Revision: 1893437
> 
> URL: http://svn.apache.org/viewvc?rev=1893437=rev
> Log:
> Add basic tests for mod_heartbeat and mod_heartmonitor
> 
> They need multicast to work, so maybe, some extra logic is needed to
> avoid expected failures on some plateforms/configurations.
> 
> Added:
> httpd/test/framework/trunk/t/modules/heartbeat.t
> Modified:
> httpd/test/framework/trunk/t/conf/extra.conf.in

Looks like this breaks the prefork mpm test in Travis not always, but on a 
regular basis.

Regards

Rüdiger