-
Notifications
You must be signed in to change notification settings - Fork 99
Disable horeka CI jobs #1685
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
Disable horeka CI jobs #1685
Conversation
This should probably be rebased. Otherwise LGTM |
0db634a
to
2de3a42
Compare
@upsj thanks for catching that. Maybe there is an option to make rebasing the default when updating a branch? |
Not AFAIK, but I recently talked to somebody from GitHub, I'll bring it up with him. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should remove the jobs instead of disabling them? Any future configuration I had in mind would not involve the split test/build configuration, so the jobs would no longer be necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we do not have project quota issue in the future, then simple is always better.
I think you can change the title, if you feel the changes does not fit the title
c829d6d
to
0b38132
Compare
55fae31
to
acbd78b
Compare
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1685 +/- ##
===========================================
- Coverage 91.60% 90.00% -1.61%
===========================================
Files 761 763 +2
Lines 61199 62866 +1667
===========================================
+ Hits 56064 56580 +516
- Misses 5135 6286 +1151 ☔ View full report in Codecov by Sentry. |
acbd78b
to
af84817
Compare
Error: PR already merged! |
This merge removes CI jobs on horeka. The QoL jobs are now executed on nla-gpu. In addition, it also deletes the windows github jobs. Related PR: ginkgo-project#1685
This PR disables CI jobs on horeka. In time for the release I would like to have our CI pass all stages.
Edit:
This also deletes the windows github jobs, although it doesn't fit the original PR topic ideally.