Skip to content

Conversation

MarcelKoch
Copy link
Member

@MarcelKoch MarcelKoch commented Oct 2, 2024

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.

@MarcelKoch MarcelKoch added this to the Ginkgo 1.9.0 milestone Oct 2, 2024
@MarcelKoch MarcelKoch requested a review from a team October 2, 2024 13:36
@MarcelKoch MarcelKoch self-assigned this Oct 2, 2024
@ginkgo-bot ginkgo-bot added the reg:ci-cd This is related to the continuous integration system. label Oct 2, 2024
@MarcelKoch MarcelKoch added the 1:ST:no-changelog-entry Skip the wiki check for changelog update label Oct 2, 2024
@upsj
Copy link
Member

upsj commented Oct 2, 2024

This should probably be rebased. Otherwise LGTM

@MarcelKoch MarcelKoch force-pushed the disable-horeka-jobs branch from 0db634a to 2de3a42 Compare October 2, 2024 14:11
@MarcelKoch
Copy link
Member Author

@upsj thanks for catching that. Maybe there is an option to make rebasing the default when updating a branch?

@upsj
Copy link
Member

upsj commented Oct 2, 2024

Not AFAIK, but I recently talked to somebody from GitHub, I'll bring it up with him.

Copy link
Member

@upsj upsj left a 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.

Copy link
Member

@yhmtsai yhmtsai left a 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

@MarcelKoch MarcelKoch force-pushed the disable-horeka-jobs branch from c829d6d to 0b38132 Compare October 8, 2024 06:38
@MarcelKoch MarcelKoch added the 1:ST:ready-to-merge This PR is ready to merge. label Oct 8, 2024
@MarcelKoch MarcelKoch force-pushed the disable-horeka-jobs branch 3 times, most recently from 55fae31 to acbd78b Compare October 9, 2024 07:55
Copy link

sonarqubecloud bot commented Oct 9, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Copy link

codecov bot commented Oct 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.00%. Comparing base (7cf1b31) to head (acbd78b).
Report is 141 commits behind head on develop.

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.
📢 Have feedback on the report? Share it here.

@MarcelKoch MarcelKoch merged commit a6cf607 into develop Oct 10, 2024
6 of 11 checks passed
@MarcelKoch MarcelKoch deleted the disable-horeka-jobs branch October 10, 2024 07:00
@ginkgo-bot
Copy link
Member

Error: PR already merged!

MarcelKoch added a commit to MarcelKoch/ginkgo that referenced this pull request Dec 2, 2024
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1:ST:no-changelog-entry Skip the wiki check for changelog update 1:ST:ready-to-merge This PR is ready to merge. reg:ci-cd This is related to the continuous integration system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants