GabrielBrascher opened a new pull request #5763:
URL: https://github.com/apache/cloudstack/pull/5763


   ### Description
   
   This PR proposes setting a length for domain id parameters that allow 
passing a list of UUIDs separated by a comma. Allowing to run the API command 
in use-cases where there are many domain admins.
   
   Currently when calling API command with multiple domain ids (for example 
`updateServiceOffering`) it leads to the following API error:
   ```
   Error: (HTTP 431, error code 9999) Unable to execute API command 
updateserviceoffering due to invalid value.
   Value greater than max allowed length 255 for param: domainIds"
   ```
   I am targeting this implementation at 4.16.1.0 and considering it a bug due 
to the fact that we could not relly on the API to execute a few critical 
updates. 
   
   Admins end up either calling multiple times the API or (:warning: don't go 
this way) doing DB changes.
   
   <!--- Describe your changes in DETAIL - And how has behaviour functionally 
changed. -->
   
   <!-- For new features, provide link to FS, dev ML discussion etc. -->
   <!-- In case of bug fix, the expected and actual behaviours, steps to 
reproduce. -->
   
   <!-- When "Fixes: #<id>" is specified, the issue/PR will automatically be 
closed when this PR gets merged -->
   <!-- For addressing multiple issues/PRs, use multiple "Fixes: #<id>" -->
   <!-- Fixes: # -->
   
   <!--- 
*********************************************************************************
 -->
   <!--- NOTE: AUTOMATATION USES THE DESCRIPTIONS TO SET LABELS AND PRODUCE 
DOCUMENTATION. -->
   <!--- PLEASE PUT AN 'X' in only **ONE** box -->
   <!--- 
*********************************************************************************
 -->
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [X] 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)
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [ ] Minor
   
   #### Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [ ] Minor
   - [X] Trivial
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to