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(ci): add cleanup step #8301

Closed
wants to merge 1 commit into from
Closed

Conversation

joshua-goldstein
Copy link
Contributor

@joshua-goldstein joshua-goldstein commented Sep 16, 2022

Problem

Dgraph CI tests were failing overnight during the Github actions/checkout setup. This looks like a small, common issue. See here and here.

Solution

Add a cleanup job before running the actual tests.

@joshua-goldstein joshua-goldstein marked this pull request as ready for review September 16, 2022 12:53
@coveralls
Copy link

coveralls commented Sep 16, 2022

Coverage Status

Coverage remained the same at 37.103% when pulling 43709c7 on joshua/fix-dgraph-ci into c1be93b on main.

@joshua-goldstein joshua-goldstein marked this pull request as draft September 16, 2022 14:18
@skrdgraph
Copy link
Contributor

@joshua-goldstein don't close this PR - till we are very sure everything is consistent. This PR may come in handy.

@skrdgraph
Copy link
Contributor

This was a good fix, we are accommodating this into the load test PR

@skrdgraph skrdgraph closed this Sep 16, 2022
@skrdgraph skrdgraph deleted the joshua/fix-dgraph-ci branch September 16, 2022 18:17
skrdgraph added a commit that referenced this pull request Sep 16, 2022
skrdgraph added a commit that referenced this pull request Sep 16, 2022
* add cleanup at start

* bring in changes from #8301

* cleanup
dshekhar95 pushed a commit that referenced this pull request Sep 19, 2022
* add cleanup at start

* bring in changes from #8301

* cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants