Re: [PR] Upgrade to alpine linux 3.16.9 [trafficserver-ingress-controller]

2024-03-22 Thread via GitHub
shukitchan merged PR #206: URL: https://github.com/apache/trafficserver-ingress-controller/pull/206 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: gith

[PR] Upgrade to alpine linux 3.16.9 [trafficserver-ingress-controller]

2024-03-22 Thread via GitHub
shukitchan opened a new pull request, #206: URL: https://github.com/apache/trafficserver-ingress-controller/pull/206 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comm

Re: [PR] I cannot get the nghttp2 to work on centos:7 [trafficserver-ci]

2024-03-22 Thread via GitHub
bneradt merged PR #328: URL: https://github.com/apache/trafficserver-ci/pull/328 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@traf

[PR] I cannot get the nghttp2 to work on centos:7 [trafficserver-ci]

2024-03-22 Thread via GitHub
bneradt opened a new pull request, #328: URL: https://github.com/apache/trafficserver-ci/pull/328 gmake[2]: Leaving directory '/root/build_h3_tools/nghttp2/third-party' Making all in src gmake[2]: Entering directory '/root/build_h3_tools/nghttp2/src' Makefile:1237: ../tests/munit/.deps/n

[PR] build_h3_tools.sh updates from latest ATS master [trafficserver-ci]

2024-03-22 Thread via GitHub
bneradt opened a new pull request, #327: URL: https://github.com/apache/trafficserver-ci/pull/327 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscrib

Re: [PR] build_h3_tools.sh updates from latest ATS master [trafficserver-ci]

2024-03-22 Thread via GitHub
bneradt merged PR #327: URL: https://github.com/apache/trafficserver-ci/pull/327 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@traf

Re: [PR] Fix a build issue with autest plugins [trafficserver]

2024-03-22 Thread via GitHub
bneradt commented on PR #11165: URL: https://github.com/apache/trafficserver/pull/11165#issuecomment-2015735392 [approve ci autest fedora] -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spec

[PR] Clean up unused docker file and image references [trafficserver-ci]

2024-03-22 Thread via GitHub
bneradt opened a new pull request, #326: URL: https://github.com/apache/trafficserver-ci/pull/326 fedora:32 is long since dead, and so is centos:6. Cleaning up the noise in our lists. We can always git revert if we need them back for some reason. -- This is an automated message from the A

Re: [PR] Clean up unused docker file and image references [trafficserver-ci]

2024-03-22 Thread via GitHub
bneradt merged PR #326: URL: https://github.com/apache/trafficserver-ci/pull/326 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@traf

Re: [PR] H3 deps build script update. [trafficserver]

2024-03-22 Thread via GitHub
brbzull0 merged PR #11174: URL: https://github.com/apache/trafficserver/pull/11174 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@tr

Re: [PR] Work around a GCC bug for __sync_val_compare_and_swap on ARMv8.1+ [trafficserver]

2024-03-22 Thread via GitHub
bryancall merged PR #11147: URL: https://github.com/apache/trafficserver/pull/11147 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@t

Re: [PR] H3 deps build script update. [trafficserver]

2024-03-22 Thread via GitHub
bneradt commented on code in PR #11174: URL: https://github.com/apache/trafficserver/pull/11174#discussion_r1535789065 ## tools/build_h3_tools.sh: ## @@ -144,11 +144,7 @@ echo "Building quiche" QUICHE_BASE="${BASE:-/opt}/quiche" [ ! -d quiche ] && git clone --recursive https:

Re: [PR] H3 deps build script update. [trafficserver]

2024-03-22 Thread via GitHub
brbzull0 commented on PR #11174: URL: https://github.com/apache/trafficserver/pull/11174#issuecomment-2014890804 > No quiche update on this PR? Although the latest release does not have quictls support yet, there were security fixes. https://github.com/cloudflare/quiche/releases/tag/0.20.1

Re: [PR] H3 deps build script update. [trafficserver]

2024-03-22 Thread via GitHub
brbzull0 commented on code in PR #11174: URL: https://github.com/apache/trafficserver/pull/11174#discussion_r1535400103 ## tools/build_h3_tools.sh: ## @@ -186,8 +186,9 @@ LDFLAGS=${LDFLAGS:-"-Wl,-rpath,${OPENSSL_LIB}"} # Then nghttp3 echo "Building nghttp3..." -[ ! -d nghttp

Re: [PR] H3 deps build script update. [trafficserver]

2024-03-22 Thread via GitHub
brbzull0 commented on PR #11174: URL: https://github.com/apache/trafficserver/pull/11174#issuecomment-2014832810 > Thanks for your work on this. > > Do we have other updates we need now that your quictls change has landed in quiche? We do not have an offical release just yet, b

Re: [PR] Update HTTP3 test to use hostname instead of numeric address. [trafficserver]

2024-03-22 Thread via GitHub
brbzull0 merged PR #11175: URL: https://github.com/apache/trafficserver/pull/11175 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@tr

Re: [PR] SSL Diags - Use boringssl API to fetch a proper error description. [trafficserver]

2024-03-22 Thread via GitHub
brbzull0 merged PR #11176: URL: https://github.com/apache/trafficserver/pull/11176 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@tr

Re: [PR] rate_limit: Set initialized flag of SieveLru when loading yaml is done [trafficserver]

2024-03-22 Thread via GitHub
masaori335 commented on code in PR #11181: URL: https://github.com/apache/trafficserver/pull/11181#discussion_r1535155321 ## plugins/experimental/rate_limit/ip_reputation.cc: ## @@ -135,6 +135,8 @@ SieveLru::parseYaml(const YAML::Node &node) Dbg(dbg_ctl, "\twith perma-block r

[PR] rate_limit: Set initialized flag of SieveLru when loading yaml is done [trafficserver]

2024-03-22 Thread via GitHub
masaori335 opened a new pull request, #11181: URL: https://github.com/apache/trafficserver/pull/11181 The rate_limit plugin keep saying "no IP reputation" with any config. It looks like the `SieveLru::_initialized` flag never be true. ``` DIAG: (rate_limit) CLIENT_HELLO on localh