DaanHoogland commented on a change in pull request #2710: comply with api key
constraint
URL: https://github.com/apache/cloudstack/pull/2710#discussion_r195356322
##########
File path: server/src/com/cloud/user/AccountManagerImpl.java
##########
@@ -1734,6 +1734,8 @@ public Boolean doInTransaction(TransactionStatus status)
{
UserVO newUser = new UserVO(user);
Review comment:
We are not changing db id which is used for foreign key relations. and we
are copying the uuid to the external identity for future reference. through the
new user we can then also find any 'real external' ids.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
