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

fix(tests): wait for license to be applied before trying to login #8744

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

mangalaman93
Copy link
Member

once the zero cluster is up, it applies a trial license. Until the license is not applied, health check for alpha returns that the cluster is an OSS cluster.

This PR ensures that the t driver code now waits for the license to be proposed and applied in the cluster. Then, it waits for the login to succeed before running the tests.

@dgraph-bot dgraph-bot added area/testing Testing related issues go Pull requests that update Go code labels Mar 9, 2023
once the zero cluster is up, it applies a trial license. Until the
license is applied, health check for alpha returns that the cluster
is an OSS cluster.

This PR ensures that the t driver code now waits for the license
to be proposed and applied in the cluster. Then, it waits for the
login to succeed before running the tests if ACL is enabled.
@mangalaman93 mangalaman93 merged commit 47c6b68 into main Mar 10, 2023
@mangalaman93 mangalaman93 deleted the aman/testfix branch March 10, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing Testing related issues go Pull requests that update Go code
Development

Successfully merging this pull request may close these issues.

None yet

4 participants