Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test(connector-corda): fix flaky v4.8 test #1469

Open
petermetz opened this issue Oct 21, 2021 · 0 comments
Open

test(connector-corda): fix flaky v4.8 test #1469

petermetz opened this issue Oct 21, 2021 · 0 comments
Labels
bug Something isn't working Corda Developer_Experience Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) good-first-issue Good for newcomers good-first-issue-400-expert Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. help wanted Extra attention is needed Tests Anything related to tests be that automatic or manual, integration or unit, etc.

Comments

@petermetz
Copy link
Member

Describe the bug

We've got a flaky one at:
packages/cactus-plugin-ledger-connector-corda/src/test/typescript/integration/deploy-cordapp-jars-to-nodes-v4.8.test.ts

One of the corda nodes within the container occasionally don't boot up (hangs and then the test waits for an hour before giving up).

To Reproduce

Keep running the test and eventually you'll see it.

Expected behavior

Tests are stable, not flaky.

Logs/Stack traces

The CI logs from the GHA runner as of this morning:
logs_10155.zip

Cloud provider or hardware configuration:

GHA runner

Operating system name, version, build:

Ubuntu 20.04

Hyperledger Cactus release version or commit (git rev-parse --short HEAD):

1.0.0-rc.1

Hyperledger Cactus Plugins/Connectors Used

Corda

Additional context

I'll send a separate PR that skips this test to avoid flakiness in the meantime.
@petermetz petermetz added bug Something isn't working good-first-issue Good for newcomers help wanted Extra attention is needed Corda Developer_Experience Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. good-first-issue-400-expert Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) Tests Anything related to tests be that automatic or manual, integration or unit, etc. labels Oct 21, 2021
petermetz added a commit to petermetz/cacti that referenced this issue Oct 21, 2021
@petermetz
test(connector-corda): skip deploy-cordapp-jars-to-nodes-v4.8
ba38c59 
Related to hyperledger#1469

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
This was referenced Oct 21, 2021
Merged
Open
petermetz added a commit that referenced this issue Oct 22, 2021
@petermetz
test(connector-corda): skip deploy-cordapp-jars-to-nodes-v4.8
db0941e 
Related to #1469

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
Leeyoungone pushed a commit to Leeyoungone/cactus that referenced this issue Oct 26, 2021
@petermetz @Leeyoungone
test(connector-corda): skip deploy-cordapp-jars-to-nodes-v4.8
8403dd9 
Related to hyperledger#1469

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
@petermetz petermetz mentioned this issue Oct 27, 2021
Open
RafaelAPB pushed a commit to RafaelAPB/blockchain-integration-framework that referenced this issue Mar 9, 2022
@petermetz @RafaelAPB
test(connector-corda): skip deploy-cordapp-jars-to-nodes-v4.8
cec1d02 
Related to hyperledger#1469

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
outSH added a commit to outSH/cactus that referenced this issue Jul 4, 2022
@outSH
test(cactus): remove flaky tests from main CI execution
c34e857 
- Remove flaky tests from main CI execution.
- Add separate TAP and jest configuration for running only flaky tests.
- Add test:*:flaky scripts in the main package.json
- Add new step to github CI script to run flaky tests as optional
  (i.e. don't fail if flaky test fails)

Closes: hyperledger#2112

Related: hyperledger#2104
Related: hyperledger#1626
Related: hyperledger#2019
Related: hyperledger#1625
Related: hyperledger#1543
Related: hyperledger#1598
Related: hyperledger#1528
Related: hyperledger#1521
Related: hyperledger#1495
Related: hyperledger#1485
Related: hyperledger#1473
Related: hyperledger#1471
Related: hyperledger#1469
Related: hyperledger#1150

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
@outSH outSH mentioned this issue Jul 4, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working Corda Developer_Experience Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) good-first-issue Good for newcomers good-first-issue-400-expert Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. help wanted Extra attention is needed Tests Anything related to tests be that automatic or manual, integration or unit, etc.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@petermetz