DaanHoogland commented on code in PR #7545:
URL: https://github.com/apache/cloudstack/pull/7545#discussion_r1334104226
##########
api/src/main/java/org/apache/cloudstack/api/command/admin/network/UpdateNetworkOfferingCmd.java:
##########
@@ -55,7 +55,7 @@ public class UpdateNetworkOfferingCmd extends BaseCmd {
private String displayText;
@Parameter(name = ApiConstants.AVAILABILITY, type = CommandType.STRING,
description = "the availability of network offering."
- + " Default value is Required for Guest Virtual network offering;
Optional for Guest Direct network offering")
+ + " The default value is Required for Guest Virtual network
offering, Optional for Guest Direct network offering. Only one network offering
can have the value Required ")
Review Comment:
```suggestion
+ " The value is Required makes this network offering default
for Guest Virtual Networks. Only one network offering can have the value
Required ")
```
I do not know the significance of the term `Guest Direct network offering`
in the description so I am proposing this change @weizhouapache @kiranchavala .
Please enhance if needed.
--
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]