BryanMLima commented on PR #7378: URL: https://github.com/apache/cloudstack/pull/7378#issuecomment-1773307322
> > Hi @BryanMLima > > The feature is nice > > Can the following api's have an additional parameter of "ipaddress" > > So that the user can easily provide the ip address instead of providing the id > > ``` > > 1. update quarantinedip > > > > 2. remove quarantinedip > > ``` > > > > > > Similar to The associateIPaddress api call which has ipaddress parameter > > https://cloudstack.apache.org/api/apidocs-4.18/apis/associateIpAddress.html > > good request @kiranchavala but I think this can be put in the scope of your related request at #8125 instead of in this PR, would you agree? cc @BryanMLima @shwstppr @DaanHoogland That's a good enhancement, the plan is to add a UI for this feature as well to make it even easier to manage the IPs in quarantine. I agree with @DaanHoogland to put this enhancement in issue #8125. I can add it to my tasks and adjust both quarantine APIs and the `disassociateIpAddress` API in another PR. -- 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]
