Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

fix: flakiness integ setup #727

Merged
merged 2 commits into from
Sep 29, 2021
Merged

fix: flakiness integ setup #727

merged 2 commits into from
Sep 29, 2021

Conversation

SanketD92
Copy link
Contributor

Issue #, if available:
Integ tests flaky with undefined setup instance.

Description of changes:
Added a retry logic for initializing setup instance.

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you successfully tested with your changes locally?

AS review ticket id:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@SanketD92 SanketD92 requested a review from a team as a code owner September 29, 2021 14:29
@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

Merging #727 (9ce70cf) into develop (4d6f972) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #727   +/-   ##
========================================
  Coverage    49.17%   49.17%           
========================================
  Files          252      252           
  Lines        12802    12802           
  Branches      2066     2066           
========================================
  Hits          6296     6296           
  Misses        5691     5691           
  Partials       815      815           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d6f972...9ce70cf. Read the comment docs.

@SanketD92 SanketD92 merged commit 65ea43d into develop Sep 29, 2021
@SanketD92 SanketD92 deleted the fix-flakiness-integ-setup branch September 29, 2021 15:19
jxuamazon pushed a commit to jxuamazon/service-workbench-on-aws that referenced this pull request Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants