Frawless commented on code in PR #26908:
URL: https://github.com/apache/flink/pull/26908#discussion_r2323037105
##########
.github/workflows/template.flink-ci.yml:
##########
@@ -286,6 +286,20 @@ jobs:
if: ${{ !cancelled() && (failure() || !steps.docker-cache.cache.hit) }}
run: ./tools/azure-pipelines/cache_docker_images.sh save
+ flink-kubernetes:
+ name: "Build & test flink-kubernetes with multiple HTTP clients"
+ needs: compile
+ strategy:
+ fail-fast: false
+ matrix:
+ http-client: [ "okhttp", "jdk", "jetty", "vertx" ]
+ uses: ./.github/workflows/template.flink-kubernetes-ci.yml
+ with:
+ http-client: ${{ matrix.http-client }}
Review Comment:
In my [fork](https://github.com/Frawless/flink/actions/runs/16989320117) it
was 4 parallel jobs, around 25 minutes per each and it finished like 1h before
current jobs for e2e tests that were running in parallel with these new ones. I
basically followed the same approach as was implemented in operator repo as
part of https://github.com/apache/flink-kubernetes-operator/pull/881
Regarding Azure vs GHA, my understanding was that Flink community is aiming
to move from Azure to GHA, that's the reason why I implemented it there. If we
agree that Azure is a preferable way, I am happy to extend Azure pipelines. I
understand this might need longer discussion so implementation in this PR can
be used more like as a PoC.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
