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(test): adjust frequency #8340

Merged
merged 3 commits into from
Sep 30, 2022
Merged

fix(test): adjust frequency #8340

merged 3 commits into from
Sep 30, 2022

Conversation

skrdgraph
Copy link
Contributor

@skrdgraph skrdgraph commented Sep 30, 2022 •
edited
Loading

Problem

  • Increase test frequency to capture more run results before RC
  • add sleep post docker container cleanup
  • adjust the gh-runner cruft cleanup bug

Solution

  • change to */30 * * * *
  • introduce wait after cleanup
  • make the gh-runner cruft run every 1s

@skrdgraph skrdgraph marked this pull request as ready for review September 30, 2022 22:09
@skrdgraph skrdgraph merged commit dff8415 into main Sep 30, 2022
@skrdgraph skrdgraph deleted the skrdgraph/adjust_test_freq branch September 30, 2022 22:13
@coveralls
Copy link

coveralls commented Sep 30, 2022 •
edited
Loading

Coverage Status

Coverage remained the same at 37.18% when pulling a30ebdd on skrdgraph/adjust_test_freq into 56df012 on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akon-dey akon-dey Awaiting requested review from akon-dey

@gcxml gcxml Awaiting requested review from gcxml

@darkn3rd darkn3rd Awaiting requested review from darkn3rd

@meghalims meghalims Awaiting requested review from meghalims

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@skrdgraph @coveralls