merge acs-api-commands project into tools - #11098
Conversation
added 4.9 and updated readme
Updated README and added 4.10 commands
There was a problem hiding this comment.
Pull Request Overview
This PR merges the existing acs-api-commands project into the tools/apidoc-versions-diff directory (preserving its history), removes the old license file, and updates the release notes accordingly.
- Added a subtree for tools/apidoc-versions-diff with diff.txt and diff.json for versions 4.19→4.20 (and earlier).
- Removed the old license file and updated release notes.
- No functional changes to existing code; purely a build/CI and tooling import.
Reviewed Changes
Copilot reviewed 31 out of 52 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tools/apidoc-versions-diff/diff-419-420/diff.txt | Imported raw API commands diff for CloudStack 4.19→4.20 |
| tools/apidoc-versions-diff/diff-419-420/diff.json | Imported structured JSON diff for CloudStack 4.19→4.20 |
Comments suppressed due to low confidence (2)
tools/apidoc-versions-diff/diff-419-420/diff.txt:17
- [nitpick] Command descriptions should start with an uppercase verb in third-person singular (e.g.,
Deletes a backup repository).
deleteBackupRepository (delete a backup repository)
tools/apidoc-versions-diff/diff-419-420/diff.txt:29
- The command name uses
Nsxrather thanNSX; consider renaming todeleteNSXControllerto reflect the acronym consistently.
deleteNsxController (delete NSX Controller to CloudStack)
|
|
||
| purgeExpungedResources (Purge expunged resources) | ||
|
|
||
| deleteNsxController (delete NSX Controller to CloudStack) |
There was a problem hiding this comment.
The description reads 'to CloudStack' but should say 'from CloudStack' to accurately reflect removal.
| deleteNsxController (delete NSX Controller to CloudStack) | |
| deleteNsxController (delete NSX Controller from CloudStack) |
|
@pdion891 , are you still going through with this? |
|
Hi @pdion891 is this PR still in progress? |
|
Hi @nvazquez , I have not kept track of this PR recently. I will rebase the PR branch... |
- .codespellrc stops the spelling check from failing on the generated files' typos.
|
recent changes to get the ci-cd pipeline to work:
|
|
@DaanHoogland @nvazquez , I think this PR is ready. but please review my last comment. |
|
@weizhouapache @winterhazel you are the current RMs (but we can wait till @RosiKyu / @nvazquez ‘ss turn ;) @pdion891 some release procedure changes on the wiki needed as well? |
|
@DaanHoogland , the only reference I found in the wiki would be in this page: https://cwiki.apache.org/confluence/spaces/CLOUDSTACK/pages/30738117/Release+Procedure it also refer to a tool from @swill. Would there be anything more to add to this PR ? |
I guess if that’s what you found it should be good, tnx @pdion891 |
|
@blueorangutan package |
|
@nvazquez 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. |
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 19140 |
|
Thanks @pdion891 merging this PR - will report back if any issues with the new versioning (hopefully none 😄 ) |
Description
As suggested in the mailing list, here is the merge of acs-api-commands/
which is use to create CloudStack Release Notes to extract new or update api's.
Project renamed into folder
tools/apidoc-versions-diffonce this PR merge, the project acs-api-commands/ will be mark as archived/deprecated with a link to this folder.
Perform using git subtree which kept previous commit histories;
Types of changes
Feature/Enhancement Scale
Bug Severity
How Has This Been Tested?
No.
How did you try to break this feature and the system with this change?
Did not.