Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(main): add apis node #57

Merged
merged 1 commit into from
Oct 19, 2024
Merged

feat(main): add apis node #57

merged 1 commit into from
Oct 19, 2024

Conversation

cuisongliu
Copy link
Owner

@cuisongliu cuisongliu commented Oct 19, 2024

This pull request includes significant changes to the automq-operator project, focusing on workflow improvements, refactoring the main application, and enhancing end-to-end tests. The most important changes include renaming jobs in the GitHub Actions workflow, refactoring the main application to use a centralized API registry, and adding new end-to-end tests.

Workflow Improvements:

  • .github/workflows/test.yml: Renamed job2 to job1 and vice versa, added a step to log disk space before freeing it up, and modified the job to install Kubernetes and applications instead of pruning the OS. [1] [2] [3] [4]

Application Refactoring:

End-to-End Testing Enhancements:

Copy link

codecov bot commented Oct 19, 2024

Codecov Report

Attention: Patch coverage is 69.35484% with 19 lines in your changes missing coverage. Please review.

Project coverage is 36.31%. Comparing base (11e5b41) to head (add936b).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
e2e/httplib.go 69.35% 16 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
+ Coverage   30.48%   36.31%   +5.83%     
==========================================
  Files           4        5       +1     
  Lines         351      413      +62     
==========================================
+ Hits          107      150      +43     
- Misses        224      240      +16     
- Partials       20       23       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cuisongliu cuisongliu force-pushed the e2e_controller branch 9 times, most recently from 333e91a to 3fd9492 Compare October 19, 2024 03:04
Signed-off-by: cuisongliu <cuisongliu@qq.com>
@cuisongliu cuisongliu merged commit d832db1 into main Oct 19, 2024
8 checks passed
@cuisongliu cuisongliu deleted the e2e_controller branch October 19, 2024 03:45
@github-actions github-actions bot mentioned this pull request Oct 19, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant