[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-8832?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14956708#comment-14956708
 ] 

ASF GitHub Bot commented on CLOUDSTACK-8832:
--------------------------------------------

Github user nlivens commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/801#discussion_r41980442
  
    --- Diff: 
plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
 ---
    @@ -303,13 +422,17 @@ protected boolean canHandle(Network network, Service 
service) {
             }
     
             if (!_networkModel.isProviderForNetwork(getProvider(), 
network.getId())) {
    -            s_logger.debug("NuageElement is not a provider for network " + 
network.getDisplayText());
    +            if (s_logger.isDebugEnabled()) {
    +                s_logger.debug("NuageElement is not a provider for network 
" + network.getDisplayText());
    +            }
                 return false;
             }
     
             if (service != null) {
                 if 
(!_ntwkSrvcDao.canProviderSupportServiceInNetwork(network.getId(), service, 
getProvider())) {
    -                s_logger.debug("NuageElement can't provide the " + 
service.getName() + " service on network " + network.getDisplayText());
    +                if (s_logger.isDebugEnabled()) {
    +                    s_logger.debug("NuageElement can't provide the " + 
service.getName() + " service on network " + network.getDisplayText());
    --- End diff --
    
    Changed <code>NuageElement</code> to <code>NuageVsp</code> to reflect the 
name of the provider. No additional context information needed.


> Update Nuage VSP plugin to work with Nuage VSP release 3.2
> ----------------------------------------------------------
>
>                 Key: CLOUDSTACK-8832
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8832
>             Project: CloudStack
>          Issue Type: Improvement
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Management Server
>    Affects Versions: 4.6.0
>            Reporter: Nick Livens
>            Assignee: Nick Livens
>         Attachments: nuageVspMarvinLogs.tar.gz
>
>
> Nuage VSP 3.2 is being released, we want to bring the plugin up to date for 
> this release



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to