DaanHoogland commented on issue #7776: URL: https://github.com/apache/cloudstack/issues/7776#issuecomment-1653057263
> @DaanHoogland yes error can be thrown, I think this is a design thought. Last time I just logged a message, didn't want to break the backward compatibility. > > https://github.com/apache/cloudstack/blob/41bbedb530e2e94016525adc67e6642cb48ef2c8/server/src/main/java/com/cloud/network/NetworkServiceImpl.java#L1405-L1406 that message is an info in case "either projectId or accountname and domainId are not provided". I think we can split between that case and when `domainid` *is* provided but `projectId or accountname` are not. This is a minor issue, but it makes sense to return an error in those cases. (if we all agree!) -- 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]
