Skip to content

Remove awsapi leftovers - #13712

Open
winterhazel wants to merge 4 commits into
apache:mainfrom
winterhazel:remove-awsapi
Open

winterhazel wants to merge 4 commits into
apache:mainfrom
winterhazel:remove-awsapi

Conversation

@winterhazel

@winterhazel winterhazel commented Jul 27, 2026 •

Copy link
Copy Markdown
Member

Description

This PR removes a few leftovers from awsapi.

Column default_zone_id was added to account solely for the integration, and is not used anymore. API markDefaultZoneForAccount was created to set the value of this unused column.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • Build/CI
  • Test (unit or integration test code)

How Has This Been Tested?

Tests are still pending.

@codecov

codecov Bot commented Jul 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.78%. Comparing base (f8f3848) to head (96577ae).

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #13712   +/-   ##
=========================================
  Coverage     20.78%   20.78%           
- Complexity    21940    21943    +3     
=========================================
  Files          6496     6495    -1     
  Lines        591822   591748   -74     
  Branches      72615    72609    -6     
=========================================
+ Hits         123015   123020    +5     
+ Misses       455452   455371   -81     
- Partials      13355    13357    +2     
Flag Coverage Δ
uitests 3.88% <ø> (ø)
unittests 22.10% <ø> (+<0.01%) ⬆️

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DaanHoogland DaanHoogland moved this from Backlog to Ready in CloudStack Testing Sep 1, 2026
@winterhazel
winterhazel marked this pull request as ready for review October 7, 2026 13:13
@winterhazel

Copy link
Copy Markdown
Member Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@winterhazel a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✖️ debian ✔️ suse15. SL-JID 19981

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Ready

Development

Successfully merging this pull request may close these issues.

5 participants