[GitHub] [zookeeper] tisonkun opened a new pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun opened a new pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826 So far, I add a simplest test executing `bin/zkCli.sh sync /` for testing connectivity. You can comment to enrich the test script. This command itself is enough for proceeding the

[GitHub] [zookeeper] tisonkun commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r813935061 ## File path: .github/workflows/e2e.yaml ## @@ -0,0 +1,106 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [zookeeper] eolivelli opened a new pull request #1825: ZOOKEEPER-4479: Tests: C client test TestOperations.cc testTimeoutCausedByWatches1 is very flaky on CI

2022-02-24 Thread GitBox
eolivelli opened a new pull request #1825: URL: https://github.com/apache/zookeeper/pull/1825 This is a temporary patch that disables the assertion, that is slowing down a lot the CI Pull Request merge workflow -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zookeeper] eolivelli commented on pull request #1824: ZOOKEEPER-4478: Suppress OWASP false positives zookeeper-jute-3.8.0-SNAPSHOT.jar: CVE-2021-29425, CVE-2021-28164, CVE-2021-34429

2022-02-24 Thread GitBox
eolivelli commented on pull request #1824: URL: https://github.com/apache/zookeeper/pull/1824#issuecomment-1049725080 @symat I believe we can merge this patch even if C tests failed. I created a issue https://issues.apache.org/jira/browse/ZOOKEEPER-4479 and a patch to disable the

[GitHub] [zookeeper] ztzg opened a new pull request #1827: ZOOKEEPER-4479: C tests: Avoid some jitter which results in flaky tests

2022-02-24 Thread GitBox
ztzg opened a new pull request #1827: URL: https://github.com/apache/zookeeper/pull/1827 With these commits, the various tests in `TestOperations.cc` set the `last_sent` and `last_recv` fields of the "force-connected" ZooKeeper handle to deterministic values (which match the initial value

[GitHub] [zookeeper] tisonkun commented on pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun commented on pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#issuecomment-1050452622 @eolivelli updated. -- 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

[GitHub] [zookeeper] tisonkun commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814432991 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [zookeeper] sonatype-lift[bot] commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
sonatype-lift[bot] commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814433000 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation

[GitHub] [zookeeper] tisonkun commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814435497 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [zookeeper] sonatype-lift[bot] commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
sonatype-lift[bot] commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814435523 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation

[GitHub] [zookeeper] sonatype-lift[bot] commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
sonatype-lift[bot] commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814430039 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation

[GitHub] [zookeeper] tisonkun commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814432485 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation (ASF) under

[GitHub] [zookeeper] sonatype-lift[bot] commented on a change in pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
sonatype-lift[bot] commented on a change in pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#discussion_r814432522 ## File path: tools/ci/test-connectivity.py ## @@ -0,0 +1,48 @@ +#!/usr/bin/env python3 + +# Licensed to the Apache Software Foundation

[GitHub] [zookeeper] tisonkun edited a comment on pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun edited a comment on pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#issuecomment-1050461863 I don't know why build-cppunit-tests failed, obviously I don't touch any files about it :) -- This is an automated message from the Apache Git Service. To respond

[GitHub] [zookeeper] tisonkun commented on pull request #1826: ZOOKEEPER-4480: Introduce end to end compatibility tests

2022-02-24 Thread GitBox
tisonkun commented on pull request #1826: URL: https://github.com/apache/zookeeper/pull/1826#issuecomment-1050461863 I don't know why build-cppunit-tests failed, obviously I don't touch any files about it :) -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zookeeper] eolivelli closed pull request #1824: ZOOKEEPER-4478: Suppress OWASP false positives zookeeper-jute-3.8.0-SNAPSHOT.jar: CVE-2021-29425, CVE-2021-28164, CVE-2021-34429

2022-02-24 Thread GitBox
eolivelli closed pull request #1824: URL: https://github.com/apache/zookeeper/pull/1824 -- 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: