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

Further reduce sqlsmith loops in nightly runs #6238

Merged
merged 1 commit into from
Oct 27, 2023

Conversation

svenklemm
Copy link
Member

@svenklemm svenklemm commented Oct 27, 2023

In the nightly CI runs the sqlsmith job seems to fail quite often with GitHub Actions 3 lost communication with the server.. This can happen when the runner process is starved for CPU/Memory. To reduce the likelyhood this patch reduces the number of loops for sqlsmith during nightly CI runs further.

Disable-check: force-changelog-file

In the nightly CI runs the sqlsmith job seems to fail quite often
with `GitHub Actions 3 lost communication with the server.`.
This can happen when the runner process is starved for CPU/Memory.
To reduce the likelyhood this patch reduces the number of loops for
sqlsmith during nightly CI runs further.
@github-actions
Copy link

@fabriziomello, @jnidzwetzki: please review this pull request.

Powered by pull-review

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Merging #6238 (73aad8c) into main (f3b3e55) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #6238      +/-   ##
==========================================
- Coverage   65.10%   65.05%   -0.06%     
==========================================
  Files         246      246              
  Lines       57066    57020      -46     
  Branches    12651    12632      -19     
==========================================
- Hits        37155    37095      -60     
- Misses      18044    18055      +11     
- Partials     1867     1870       +3     

see 16 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@svenklemm svenklemm merged commit aa81f17 into timescale:main Oct 27, 2023
41 of 42 checks passed
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.

None yet

3 participants