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

Fix e2e integration test transient failure #3540

Closed
longquanzheng opened this issue Sep 22, 2020 · 1 comment
Closed

Fix e2e integration test transient failure #3540

longquanzheng opened this issue Sep 22, 2020 · 1 comment
Labels
up-for-grabs Issues that are good entry points for those new to Cadence that want to contribute

Comments

@longquanzheng
Copy link
Contributor

longquanzheng commented Sep 22, 2020 •
edited
Loading

Describe the bug
The bug will cause test failing so we have to disable the assertion for now.

Edited on Apr 6, 2021 -- it's now also failing for Cassandra sometimes.

To Reproduce
Is the issue reproducible?

  • Yes, once you allow the assertion. more than 50% it will fail.

Steps to reproduce the behavior:
A clear and concise description of the reproduce steps.
Remove the three lines under the TODO

Expected behavior
A clear and concise description of what you expected to happen.
Fix the test and letting test pass with the assertion.

Screenshots
N/A

Additional context
#3532
@longquanzheng longquanzheng added up-for-grabs Issues that are good entry points for those new to Cadence that want to contribute good first issue Up for grab as first issue to contribute to Cadence project labels Sep 22, 2020
@longquanzheng longquanzheng changed the title Fix e2e integration test for Postgres Fix e2e integration test transient failure for Postgres Sep 22, 2020
This was referenced Apr 3, 2021
Closed
Merged
@longquanzheng longquanzheng removed the good first issue Up for grab as first issue to contribute to Cadence project label Apr 6, 2021
@longquanzheng longquanzheng changed the title Fix e2e integration test transient failure for Postgres Fix e2e integration test transient failure Apr 7, 2021
@longquanzheng
Copy link
Contributor Author

https://buildkite.com/uberopensource/cadence-server/builds/10120#2be36cee-631a-4254-856c-d37b3e105ded


--- FAIL: TestIntegrationSuite/TestCronWorkflow (8.51s)
--
  | integrationTest.go:1251:
  | Error Trace:	integrationTest.go:1251
  | Error:      	Not equal:
  | expected: 0
  | actual  : 2
  | Test:       	TestIntegrationSuite/TestCronWorkflow
  | Messages:   	Still Flaky?: backoffSeconds: 2 ((1617766143031000000-1617766140927000000) - (1617766141031000000-1617766141030000000)), targetBackoffSeconds: 3

@longquanzheng longquanzheng mentioned this issue Apr 7, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
up-for-grabs Issues that are good entry points for those new to Cadence that want to contribute
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@longquanzheng