weizhouapache commented on a change in pull request #5769:
URL: https://github.com/apache/cloudstack/pull/5769#discussion_r778615529
##########
File path: api/src/main/java/com/cloud/network/GuestVlan.java
##########
@@ -16,17 +16,26 @@
// under the License.
package com.cloud.network;
-import org.apache.cloudstack.api.Identity;
import org.apache.cloudstack.api.InternalIdentity;
-public interface GuestVlan extends InternalIdentity, Identity {
+import java.util.Date;
+
+public interface GuestVlan extends InternalIdentity {
@Override
public long getId();
- public long getAccountId();
+ Date getTakenAt();
+
+ String getVnet();
+
+ String getReservationId();
+
+ Long getAccountId();
+
+ long getDataCenterId();
- public String getGuestVlanRange();
+ long getPhysicalNetworkId();
- public long getPhysicalNetworkId();
+ Long getAccountGuestVlanMapId();
Review comment:
@DaanHoogland
they are both optional. if guest vlan is not taken , both are null.
--
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]
