[GitHub] [cloudstack] nvazquez commented on a diff in pull request #7455: Fix Tungsten Fabric errors on adding a new zone due to duplicated entries on ntwk_service_map

[via GitHub]

nvazquez commented on code in PR #7455:
URL: https://github.com/apache/cloudstack/pull/7455#discussion_r1195798254


##########
plugins/network-elements/tungsten/src/main/java/org/apache/cloudstack/network/tungsten/api/command/ConfigTungstenFabricServiceCmd.java:
##########
@@ -132,6 +135,21 @@ public void doInTransactionWithoutResult(final 
TransactionStatus status) {
 
     private void persistDefaultSystemNetwork() {
         Transaction.execute(new TransactionCallbackNoReturn() {
+
+            private void persistNetworkServiceMapAvoidingDuplicates(Network 
network,
+                                                                    
Network.Service service,
+                                                                    
Network.Provider provider,
+                                                                    
NetworkServiceMapVO mapVO) {

Review Comment:
   Thanks @shwstppr, fixed



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org