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.

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

Modifying travis to add retry logic #444

Merged
merged 2 commits into from
Aug 25, 2020
Merged

Modifying travis to add retry logic #444

merged 2 commits into from
Aug 25, 2020

Conversation

anishakj
Copy link
Contributor

@anishakj anishakj commented Aug 25, 2020

Signed-off-by: anisha.kj [email protected]

Change log description

In travis, we were waiting for 5 min for cluster creation to complete. But in some cases cluster creation takes more time.Add a retry logic, and check whether the cluster state is active.

Purpose of the change

Add retry mechanism to check cluster is active.

What the code does

Add retry mechanism to check cluster is active.

How to verify it

Make sure cluster creation is happening.

Signed-off-by: anisha.kj <[email protected]>
@anishakj anishakj requested a review from Prabhaker24 August 25, 2020 07:42
Copy link
Contributor

@Prabhaker24 Prabhaker24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anishakj anishakj merged commit 7662f09 into master Aug 25, 2020
@anishakj anishakj deleted the issue-travis-cncf branch August 25, 2020 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants