Pearl1594 commented on code in PR #11587:
URL: https://github.com/apache/cloudstack/pull/11587#discussion_r2362708097


##########
engine/schema/src/main/java/com/cloud/storage/SnapshotPolicyVO.java:
##########
@@ -59,14 +59,20 @@ public class SnapshotPolicyVO implements SnapshotPolicy {
     @Column(name = "uuid")
     String uuid;
 
+    @Column(name = "account_id")
+    long accountId;
+
+    @Column(name = "domain_id")
+    long domainId;
+

Review Comment:
   actually, I did not consider this case.
   We do not allow policies to have different owners than their resource 
counterpart i.e., volume / VM . It has to be the same.



-- 
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