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: migrate CI to CNCF machines in release-x branch #5952

Merged
merged 1 commit into from
May 4, 2023

Conversation

chivalryq
Copy link
Member

@chivalryq chivalryq commented May 4, 2023

Description of your changes

🤖 Generated by Copilot at 92d8ffa

Summary

♻️🏃🔧

This pull request switches the runners for several GitHub Actions workflows from aliyun to self-hosted to improve the stability and reliability of the end-to-end tests, the code quality check, and the periodic clean-up task. It also consolidates the end-to-end tests that require multiple clusters into one workflow, e2e-multicluster-test.yml.

aliyun runner
too unstable for testing
self-hosted instead

Walkthrough

  • Switch the runner for the end-to-end tests that involve multiple clusters from aliyun to self-hosted to improve stability and reliability (link, link, link)
  • Consolidate the end-to-end tests that require multiple clusters into one workflow in .github/workflows/e2e-multicluster-test.yml to simplify the configuration and execution (link, link)
  • Switch the runner for the code quality check and the periodic image cleanup task from aliyun to self-hosted to avoid network issues and timeouts (link, link)

I have:

  • Read and followed KubeVela's contribution process.
  • Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

Signed-off-by: Qiaozp <qiaozhongpei.qzp@alibaba-inc.com>
@chivalryq chivalryq requested a review from wonderflow as a code owner May 4, 2023 06:36
@chivalryq chivalryq changed the title Migrate CI to CNCF machines in release-x branch Fix: migrate CI to CNCF machines in release-x branch May 4, 2023
@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Patch coverage has no change and project coverage change: +5.09 🎉

Comparison is base (92be5b3) 55.88% compared to head (92d8ffa) 60.98%.

Additional details and impacted files
@@               Coverage Diff               @@
##           release-1.8    #5952      +/-   ##
===============================================
+ Coverage        55.88%   60.98%   +5.09%     
===============================================
  Files              178      217      +39     
  Lines            27404    30776    +3372     
===============================================
+ Hits             15315    18768    +3453     
+ Misses           10578    10291     -287     
- Partials          1511     1717     +206     
Flag Coverage Δ
core-unittests 55.90% <ø> (+0.01%) ⬆️
e2e-multicluster-test 25.65% <ø> (?)
e2e-rollout-tests 22.43% <ø> (?)
e2etests 26.45% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 118 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@chivalryq chivalryq merged commit f3a0098 into kubevela:release-1.8 May 4, 2023
27 of 28 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

2 participants