[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

rhtyd commented on issue #2302: CLOUDSTACK-10122: Unrelated error message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-341006082
 
 
   @blueorangutan test


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>Priority: Major
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

blueorangutan commented on issue #2302: CLOUDSTACK-10122: Unrelated error 
message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-341006155
 
 
   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>Priority: Major
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

blueorangutan commented on issue #2302: CLOUDSTACK-10122: Unrelated error 
message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-341005621
 
 
   Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1187


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>Priority: Major
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

blueorangutan commented on issue #2302: CLOUDSTACK-10122: Unrelated error 
message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-341001888
 
 
   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>Priority: Major
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

rhtyd commented on issue #2302: CLOUDSTACK-10122: Unrelated error message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-341001855
 
 
   @blueorangutan package


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>Priority: Major
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10012) Migrate to Embedded Jetty based mgmt server

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10012:
-

rhtyd commented on issue #2226: CLOUDSTACK-10012: Towards CloudStack fat jar
URL: https://github.com/apache/cloudstack/pull/2226#issuecomment-341001277
 
 
   @borisstoyanov thanks, with embedded Jetty until the management server 
starts the embedded Jetty server does not start to listen on http/https ports 
which is why you get connection refused quite promptly.
   Have you also performed in-place upgrade tests, i.e. say upgrading from 
4.9.3 to 4.11/fatjar? We can leave reviewing other PRs until this is well 
tested and merged first. Thanks again.


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:
us...@infra.apache.org


> Migrate to Embedded Jetty based mgmt server
> ---
>
> Key: CLOUDSTACK-10012
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10012
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Rohit Yadav
>Assignee: Rohit Yadav
>Priority: Major
> Fix For: Future, 4.11.0.0
>
>
> CloudStack installation has huge dependency on distro provided tomcat, by 
> moving to embedded jetty (like we do use jetty for development) we can sync 
> how developers develop/test mgmt servers and users use it. In addition, it 
> will be easier to publish pkgs without depending on distro provided 
> dependencies.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (CLOUDSTACK-10126) Separate Subnet for CPVM and SSVM

2017-10-31 Thread Nicolas Vazquez (JIRA)
Nicolas Vazquez created CLOUDSTACK-10126:


 Summary: Separate Subnet for CPVM and SSVM
 Key: CLOUDSTACK-10126
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10126
 Project: CloudStack
  Issue Type: Improvement
  Security Level: Public (Anyone can view this level - this is the default.)
Affects Versions: 4.11.0.0
Reporter: Nicolas Vazquez
Assignee: Nicolas Vazquez


Separate Management Subnet for CPVM and SSVM



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-9832) Do not assign public IP NIC to the VPC VR when the VPC offering does not contain VpcVirtualRouter as a SourceNat provider

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-9832:


fmaximus commented on issue #2004: CLOUDSTACK-9832: Do not assign public IP NIC 
to the VPC VR when the VPC offering does not contain VpcVirtualRouter as a 
SourceNat provider
URL: https://github.com/apache/cloudstack/pull/2004#issuecomment-340018338
 
 
   @remibergsma To summarize:
   Currently VPN on VR only works if VR is also the gateway/sourcenat provider.
   So checking on VPN becomes necessary once VR provides VPN, 
   but sourcenat is implemented by another provider. 
   In this case more changes will be necessary to get VPN working, 
   like adding static routes and adding iptables rukes i.c.o. VR not providing 
Acl. 
   
   Another point to take into account: for isolated networks a similar check 
already exists, and that also only checks SourceNat.


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:
us...@infra.apache.org


> Do not assign public IP NIC to the VPC VR when the VPC offering does not 
> contain VpcVirtualRouter as a SourceNat provider
> -
>
> Key: CLOUDSTACK-9832
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9832
> Project: CloudStack
>  Issue Type: Improvement
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: Management Server
>Reporter: Mani Prashanth Varma Manthena
>Assignee: Frank Maximus
> Fix For: 4.11.0.0
>
>
> When the VPC offering does not contain VpcVirtualRouter as a SourceNat 
> provider, then we will not add the interface in the public network to the 
> VpcVR. Thus, conserving Public IPs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

nvazquez commented on issue #2302: CLOUDSTACK-10122: Unrelated error message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-340842685
 
 
   Done. Thanks guys for your input @rhtyd @borisstoyanov @rafaelweingartner 
@marcaurele @GabrielBrascher 


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (CLOUDSTACK-9751) if a public IP is assigned to a VM when VR is in starting state, it does not get applied to the vport in Nuage VSD

2017-10-31 Thread Frank Maximus (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Maximus updated CLOUDSTACK-9751:
--
Fix Version/s: 4.9.3.0

> if a public IP is assigned to a VM when VR is in starting state, it does not 
> get applied to the vport in Nuage VSD
> --
>
> Key: CLOUDSTACK-9751
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9751
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.9.0, 4.10.0.0
>Reporter: Raf Smeets
>Assignee: Frank Maximus
> Fix For: 4.9.3.0
>
>
> if a public IP is assigned to a VM when VR is in starting state, it does not 
> get applied to the vport in Nuage VSD.
> Steps to reproduce:-
> 1) Create a Isolated Network with Dns Network Offering.
> 2) Start a vm in it, it will start a VR
> 3) At this time accquire a Public IP and enable static nat (assigned it to VM 
> , Till this time no vport is created for it on the Nuage VSD). No exception 
> come cloudstack says successfully.
> 4) After the VM is in running state, the Public Ip in cloudstack is not 
> applied to the Nuage VSD.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Closed] (CLOUDSTACK-9751) if a public IP is assigned to a VM when VR is in starting state, it does not get applied to the vport in Nuage VSD

2017-10-31 Thread Frank Maximus (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Maximus closed CLOUDSTACK-9751.
-

> if a public IP is assigned to a VM when VR is in starting state, it does not 
> get applied to the vport in Nuage VSD
> --
>
> Key: CLOUDSTACK-9751
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9751
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.9.0, 4.10.0.0
>Reporter: Raf Smeets
>Assignee: Frank Maximus
> Fix For: 4.9.3.0
>
>
> if a public IP is assigned to a VM when VR is in starting state, it does not 
> get applied to the vport in Nuage VSD.
> Steps to reproduce:-
> 1) Create a Isolated Network with Dns Network Offering.
> 2) Start a vm in it, it will start a VR
> 3) At this time accquire a Public IP and enable static nat (assigned it to VM 
> , Till this time no vport is created for it on the Nuage VSD). No exception 
> come cloudstack says successfully.
> 4) After the VM is in running state, the Public Ip in cloudstack is not 
> applied to the Nuage VSD.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Resolved] (CLOUDSTACK-9751) if a public IP is assigned to a VM when VR is in starting state, it does not get applied to the vport in Nuage VSD

2017-10-31 Thread Frank Maximus (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Maximus resolved CLOUDSTACK-9751.
---
Resolution: Fixed

> if a public IP is assigned to a VM when VR is in starting state, it does not 
> get applied to the vport in Nuage VSD
> --
>
> Key: CLOUDSTACK-9751
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9751
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.9.0, 4.10.0.0
>Reporter: Raf Smeets
>Assignee: Frank Maximus
> Fix For: 4.9.3.0
>
>
> if a public IP is assigned to a VM when VR is in starting state, it does not 
> get applied to the vport in Nuage VSD.
> Steps to reproduce:-
> 1) Create a Isolated Network with Dns Network Offering.
> 2) Start a vm in it, it will start a VR
> 3) At this time accquire a Public IP and enable static nat (assigned it to VM 
> , Till this time no vport is created for it on the Nuage VSD). No exception 
> come cloudstack says successfully.
> 4) After the VM is in running state, the Public Ip in cloudstack is not 
> applied to the Nuage VSD.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Closed] (CLOUDSTACK-10081) CloudUtils getDevInfo function only checks for KVM bridgePort and not OVS

2017-10-31 Thread Frank Maximus (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-10081?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Maximus closed CLOUDSTACK-10081.
--

> CloudUtils getDevInfo function only checks for KVM bridgePort and not OVS
> -
>
> Key: CLOUDSTACK-10081
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10081
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: cloudstack-agent
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
> Fix For: 4.11.0.0
>
>
> CloudUtils getDevInfo function only checks for KVM bridgePort and not OVS. In 
> case you provide an ovsbridge, getDevInfo(dev) will say it's a device instead 
> of a bridge.
> h2. Scenario
> h3. Expected behaviour
> *Given* a KVM Host with openvswitch networking
> *and* kvmnetworklabel of the guest traffic type specifying the name of an 
> existing OVS bridge.
> *When* cloudstack-setup-agent is run on the host
> *Then* the existing openvswitch bridge is used.
> h3. Actual (incorrect) behaviour
> A new bridge cloudbr0 is created in openvswitch.
> and the networking scripts define the new bridge as OVS_BRIDGE in the ifcfg 
> of the existing bridge.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Resolved] (CLOUDSTACK-10081) CloudUtils getDevInfo function only checks for KVM bridgePort and not OVS

2017-10-31 Thread Frank Maximus (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-10081?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Maximus resolved CLOUDSTACK-10081.

   Resolution: Fixed
Fix Version/s: 4.11.0.0

> CloudUtils getDevInfo function only checks for KVM bridgePort and not OVS
> -
>
> Key: CLOUDSTACK-10081
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10081
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: cloudstack-agent
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
> Fix For: 4.11.0.0
>
>
> CloudUtils getDevInfo function only checks for KVM bridgePort and not OVS. In 
> case you provide an ovsbridge, getDevInfo(dev) will say it's a device instead 
> of a bridge.
> h2. Scenario
> h3. Expected behaviour
> *Given* a KVM Host with openvswitch networking
> *and* kvmnetworklabel of the guest traffic type specifying the name of an 
> existing OVS bridge.
> *When* cloudstack-setup-agent is run on the host
> *Then* the existing openvswitch bridge is used.
> h3. Actual (incorrect) behaviour
> A new bridge cloudbr0 is created in openvswitch.
> and the networking scripts define the new bridge as OVS_BRIDGE in the ifcfg 
> of the existing bridge.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Closed] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread Frank Maximus (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-10057?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Frank Maximus closed CLOUDSTACK-10057.
--

> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10012) Migrate to Embedded Jetty based mgmt server

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10012:
-

marcaurele commented on issue #2226: CLOUDSTACK-10012: Towards CloudStack fat 
jar
URL: https://github.com/apache/cloudstack/pull/2226#issuecomment-340789765
 
 
   LGTM - thanks @rhtyd for all the extra work


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:
us...@infra.apache.org


> Migrate to Embedded Jetty based mgmt server
> ---
>
> Key: CLOUDSTACK-10012
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10012
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Rohit Yadav
>Assignee: Rohit Yadav
> Fix For: Future, 4.11.0.0
>
>
> CloudStack installation has huge dependency on distro provided tomcat, by 
> moving to embedded jetty (like we do use jetty for development) we can sync 
> how developers develop/test mgmt servers and users use it. In addition, it 
> will be easier to publish pkgs without depending on distro provided 
> dependencies.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-9602) Add resource type name in response

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-9602:


marcaurele commented on issue #1554: CLOUDSTACK-9602: API: improve resource 
limits comprehension
URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-340786236
 
 
   @fmaximus comment has been addressed


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:
us...@infra.apache.org


> Add resource type name in response
> --
>
> Key: CLOUDSTACK-9602
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9602
> Project: CloudStack
>  Issue Type: Improvement
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Reporter: Marc-Aurèle Brothier
>Assignee: Marc-Aurèle Brothier
>
> Output the resource type name along with its id to be more human readable for 
> all API call returning a resource.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10065) Optimize SQL queries in listTemplate API to improve performance.

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10065:
-

PranaliM commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in 
listTemplate API to improve performance
URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-340772630
 
 
   @marcaurele, since this change is related to database queries, a Marvin test 
will help because the actual result of the db query can be verified. In case of 
a Unit test case, the db won't be actually queried. So, as per me Marvin test 
case could be useful in this scenario. 
   Please let me know if you think otherwise.


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:
us...@infra.apache.org


> Optimize SQL queries in listTemplate API to improve performance. 
> -
>
> Key: CLOUDSTACK-10065
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10065
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Pranali Mande
>
> *Description*:
> The db queries could be optimized to get unique results from the database 
> which could help in reducing the listTemplate API response time.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10065) Optimize SQL queries in listTemplate API to improve performance.

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10065:
-

PranaliM commented on a change in pull request #2260: CLOUDSTACK-10065: 
Optimize SQL queries in listTemplate API to improve performance
URL: https://github.com/apache/cloudstack/pull/2260#discussion_r148003324
 
 

 ##
 File path: framework/db/src/com/cloud/utils/db/GenericDaoBase.java
 ##
 @@ -1927,6 +1935,54 @@ public Integer getDistinctCount(SearchCriteria sc) {
 }
 }
 
+public Integer getDistinctCount(SearchCriteria sc, String[] 
distinctColumns) {
+String clause = sc != null ? sc.getWhereClause() : null;
+if (clause != null && clause.length() == 0) {
+clause = null;
+}
+
+final StringBuilder str = createDistinctSelect(sc, clause != null, 
distinctColumns);
+if (clause != null) {
+str.append(clause);
+}
+
+Collection>> joins = null;
+if (sc != null) {
+joins = sc.getJoins();
+if (joins != null) {
+addJoins(str, joins);
+}
+}
+
+final TransactionLegacy txn = TransactionLegacy.currentTxn();
+final String sql = "SELECT COUNT(*) FROM (" + str.toString() + ") AS 
tmp";
+
+PreparedStatement pstmt = null;
+try {
 
 Review comment:
   Refactored to use try-with-resource


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:
us...@infra.apache.org


> Optimize SQL queries in listTemplate API to improve performance. 
> -
>
> Key: CLOUDSTACK-10065
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10065
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Pranali Mande
>
> *Description*:
> The db queries could be optimized to get unique results from the database 
> which could help in reducing the listTemplate API response time.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10065) Optimize SQL queries in listTemplate API to improve performance.

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10065:
-

PranaliM commented on a change in pull request #2260: CLOUDSTACK-10065: 
Optimize SQL queries in listTemplate API to improve performance
URL: https://github.com/apache/cloudstack/pull/2260#discussion_r148003391
 
 

 ##
 File path: framework/db/src/com/cloud/utils/db/SqlGenerator.java
 ##
 @@ -680,4 +680,21 @@ public String buildDistinctIdSql() {
 
 return sql.append("SELECT DISTINCT id FROM 
").append(buildTableReferences()).append(" WHERE 
").append(buildDiscriminatorClause().first()).toString();
 }
+
+public String buildDistinctSql(String[] distinctColumnNames) {
+StringBuilder sbColumn = new StringBuilder();
+
+if(distinctColumnNames != null && distinctColumnNames.length > 0) {
+for (String columnName : distinctColumnNames) {
+sbColumn.append(columnName).append(", ");
+}
+sbColumn.delete(sbColumn.length() - 2, sbColumn.length());
+}
+else{
 
 Review comment:
   Fixed


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:
us...@infra.apache.org


> Optimize SQL queries in listTemplate API to improve performance. 
> -
>
> Key: CLOUDSTACK-10065
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10065
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Pranali Mande
>
> *Description*:
> The db queries could be optimized to get unique results from the database 
> which could help in reducing the listTemplate API response time.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10065) Optimize SQL queries in listTemplate API to improve performance.

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10065:
-

PranaliM commented on a change in pull request #2260: CLOUDSTACK-10065: 
Optimize SQL queries in listTemplate API to improve performance
URL: https://github.com/apache/cloudstack/pull/2260#discussion_r148003170
 
 

 ##
 File path: framework/db/src/com/cloud/utils/db/GenericDaoBase.java
 ##
 @@ -1927,6 +1935,54 @@ public Integer getDistinctCount(SearchCriteria sc) {
 }
 }
 
+public Integer getDistinctCount(SearchCriteria sc, String[] 
distinctColumns) {
+String clause = sc != null ? sc.getWhereClause() : null;
+if (clause != null && clause.length() == 0) {
 
 Review comment:
   Replaced as suggested.


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:
us...@infra.apache.org


> Optimize SQL queries in listTemplate API to improve performance. 
> -
>
> Key: CLOUDSTACK-10065
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10065
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Pranali Mande
>
> *Description*:
> The db queries could be optimized to get unique results from the database 
> which could help in reducing the listTemplate API response time.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10012) Migrate to Embedded Jetty based mgmt server

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10012:
-

DaanHoogland commented on issue #2226: CLOUDSTACK-10012: Towards CloudStack fat 
jar
URL: https://github.com/apache/cloudstack/pull/2226#issuecomment-340768660
 
 
   @rhtyd @marcaurele thanks to you both. We might just need a tad more testing 
but LGTM otherwise


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:
us...@infra.apache.org


> Migrate to Embedded Jetty based mgmt server
> ---
>
> Key: CLOUDSTACK-10012
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10012
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: Rohit Yadav
>Assignee: Rohit Yadav
> Fix For: Future, 4.11.0.0
>
>
> CloudStack installation has huge dependency on distro provided tomcat, by 
> moving to embedded jetty (like we do use jetty for development) we can sync 
> how developers develop/test mgmt servers and users use it. In addition, it 
> will be easier to publish pkgs without depending on distro provided 
> dependencies.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

marcaurele commented on issue #2302: CLOUDSTACK-10122: Unrelated error message
URL: https://github.com/apache/cloudstack/pull/2302#issuecomment-340767053
 
 
   @nvazquez can you change the PR to remove the try/catch block all together?


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (CLOUDSTACK-10125) Usage Server shows incorrect amount of RAM

2017-10-31 Thread Vladimir Melnik (JIRA)

 [ 
https://issues.apache.org/jira/browse/CLOUDSTACK-10125?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vladimir Melnik updated CLOUDSTACK-10125:
-
Description: 
I don't know, how to reproduce it, but the usage server reports incorrect 
amount of RAM for one of several hundreds VMs: it has 4 GB of RAM, but 8 GB are 
being reported:

{noformat}

  a4d03771-7267-4281-b615-d5cbae6c6e39
  VM-a4d03771-7267-4281-b615-d5cbae6c6e39
  VM-a4d03771-7267-4281-b615-d5cbae6c6e39
  admin
  87417805-cc3c-4244-bdde-ce35d5f83d19
  admin
  e681b2ae-7d10-44d9-befc-c0facc34cf64
  A201***
  2017-02-22T15:58:56+0200
  Running
  false
  c8d773fa-76ca-4637-8ecf-88656444fc86
  z2.tucha13.net
  ed5a7c16-adf3-453a-9238-23c9e2697eb5
  h3.c1.p1.z2.tucha13.net
  3a8dfd95-96c1-4d86-8365-9645a9ea7996
  windows-server-2012-r2-x64-russian
  Microsoft Windows Server 2012 R2 x64 
Russian
  false
  5248afa9-f896-4608-bf3b-316262c21b9d
  custom-ssd-a1
  a7c8489e-214f-41b0-90fc-1a0f5ac537cb
  custom-tier-a
  2
  2499
  4096
  1.39%
  188561
  112379
  4930471
  3153415
  4194304
  4235640
  4194304
  231433
  128710
  b5490e1c-bd31-11e6-b74f-06973a00088a
  0
  ROOT
  
29e08f6e-8649-4b1b-bda2-a7872f69485a
e4a97b82-c7c1-420d-b77f-987881064e0b
NET-PUB-193.151.***.0-24
255.255.255.0
193.151.***.***
193.151.***.***
vlan://***
vlan://***
Guest
Shared
true
06:b9:d0***:***:***
  
  KVM
  i-1251-386-VM
  true
  false
  168

{noformat}

{noformat}

  1
  
admin
10aa1795-e315-46e7-a09a-3712acb51962
e681b2ae-7d10-44d9-befc-c0facc34cf64
c8d773fa-76ca-4637-8ecf-88656444fc86
VM-a4d03771-7267-4281-b615-d5cbae6c6e39 allocated 
(ServiceOffering: 231) (Template: 203)
24 Hrs
2
24
a4d03771-7267-4281-b615-d5cbae6c6e39
VM-a4d03771-7267-4281-b615-d5cbae6c6e39
5248afa9-f896-4608-bf3b-316262c21b9d
3a8dfd95-96c1-4d86-8365-9645a9ea7996
a4d03771-7267-4281-b615-d5cbae6c6e39
KVM
2
2400
8192
2017-10-30'T'00:00:00+00:00
2017-10-30'T'23:59:59+00:00
  

{noformat}

Are there any data that I could provide with more information that would help 
to find and fix the issue? I'd be glad to!

  was:
I don't know, how to reproduce it, but the usage server reports incorrect 
amount of RAM for one of several hundreds VMs: it has 4 GB of RAM, but 8 GB are 
being reported:

{{
  a4d03771-7267-4281-b615-d5cbae6c6e39
  VM-a4d03771-7267-4281-b615-d5cbae6c6e39
  VM-a4d03771-7267-4281-b615-d5cbae6c6e39
  admin
  87417805-cc3c-4244-bdde-ce35d5f83d19
  admin
  e681b2ae-7d10-44d9-befc-c0facc34cf64
  A201***
  2017-02-22T15:58:56+0200
  Running
  false
  c8d773fa-76ca-4637-8ecf-88656444fc86
  z2.tucha13.net
  ed5a7c16-adf3-453a-9238-23c9e2697eb5
  h3.c1.p1.z2.tucha13.net
  3a8dfd95-96c1-4d86-8365-9645a9ea7996
  windows-server-2012-r2-x64-russian
  Microsoft Windows Server 2012 R2 x64 
Russian
  false
  5248afa9-f896-4608-bf3b-316262c21b9d
  custom-ssd-a1
  a7c8489e-214f-41b0-90fc-1a0f5ac537cb
  custom-tier-a
  2
  2499
  4096
  1.39%
  188561
  112379
  4930471
  3153415
  4194304
  4235640
  4194304
  231433
  128710
  b5490e1c-bd31-11e6-b74f-06973a00088a
  0
  ROOT
  
29e08f6e-8649-4b1b-bda2-a7872f69485a
e4a97b82-c7c1-420d-b77f-987881064e0b
NET-PUB-193.151.***.0-24
255.255.255.0
193.151.***.***
193.151.***.***
vlan://***
vlan://***
Guest
Shared
true
06:b9:d0***:***:***
  
  KVM
  i-1251-386-VM
  true
  false
  168

}}

{{

  1
  
admin
10aa1795-e315-46e7-a09a-3712acb51962
e681b2ae-7d10-44d9-befc-c0facc34cf64
c8d773fa-76ca-4637-8ecf-88656444fc86
VM-a4d03771-7267-4281-b615-d5cbae6c6e39 allocated 
(ServiceOffering: 231) (Template: 203)
24 Hrs
2
24
a4d03771-7267-4281-b615-d5cbae6c6e39
VM-a4d03771-7267-4281-b615-d5cbae6c6e39
5248afa9-f896-4608-bf3b-316262c21b9d
3a8dfd95-96c1-4d86-8365-9645a9ea7996
a4d03771-7267-4281-b615-d5cbae6c6e39
KVM
2
2400
8192
2017-10-30'T'00:00:00+00:00
2017-10-30'T'23:59:59+00:00
  

}}

Could I provide with more information that would help to find and fix the 
issue? I'd be glad to!


> Usage Server shows incorrect amount of RAM
> --
>
> Key: CLOUDSTACK-10125
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10125
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: Usage
>Affects Versions: 4.9.0
> Environment: CentOS-6.8-x64
>Reporter: Vladimir Melnik
>Priority: Critical
>  Labels: usage
>
> I don't know, how to reproduce it, but the usage server reports incorrect 
> amount of RAM for one of several hundreds VMs: it has 4 GB of RAM, but 8 GB 
> are being reported:
> {noformat}
> 
>   a4d03771-7267-4281-b615-d5cbae6c6e39
>  

[jira] [Created] (CLOUDSTACK-10125) Usage Server shows incorrect amount of RAM

2017-10-31 Thread Vladimir Melnik (JIRA)
Vladimir Melnik created CLOUDSTACK-10125:


 Summary: Usage Server shows incorrect amount of RAM
 Key: CLOUDSTACK-10125
 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10125
 Project: CloudStack
  Issue Type: Bug
  Security Level: Public (Anyone can view this level - this is the default.)
  Components: Usage
Affects Versions: 4.9.0
 Environment: CentOS-6.8-x64
Reporter: Vladimir Melnik
Priority: Critical


I don't know, how to reproduce it, but the usage server reports incorrect 
amount of RAM for one of several hundreds VMs: it has 4 GB of RAM, but 8 GB are 
being reported:

{{
  a4d03771-7267-4281-b615-d5cbae6c6e39
  VM-a4d03771-7267-4281-b615-d5cbae6c6e39
  VM-a4d03771-7267-4281-b615-d5cbae6c6e39
  admin
  87417805-cc3c-4244-bdde-ce35d5f83d19
  admin
  e681b2ae-7d10-44d9-befc-c0facc34cf64
  A201***
  2017-02-22T15:58:56+0200
  Running
  false
  c8d773fa-76ca-4637-8ecf-88656444fc86
  z2.tucha13.net
  ed5a7c16-adf3-453a-9238-23c9e2697eb5
  h3.c1.p1.z2.tucha13.net
  3a8dfd95-96c1-4d86-8365-9645a9ea7996
  windows-server-2012-r2-x64-russian
  Microsoft Windows Server 2012 R2 x64 
Russian
  false
  5248afa9-f896-4608-bf3b-316262c21b9d
  custom-ssd-a1
  a7c8489e-214f-41b0-90fc-1a0f5ac537cb
  custom-tier-a
  2
  2499
  4096
  1.39%
  188561
  112379
  4930471
  3153415
  4194304
  4235640
  4194304
  231433
  128710
  b5490e1c-bd31-11e6-b74f-06973a00088a
  0
  ROOT
  
29e08f6e-8649-4b1b-bda2-a7872f69485a
e4a97b82-c7c1-420d-b77f-987881064e0b
NET-PUB-193.151.***.0-24
255.255.255.0
193.151.***.***
193.151.***.***
vlan://***
vlan://***
Guest
Shared
true
06:b9:d0***:***:***
  
  KVM
  i-1251-386-VM
  true
  false
  168

}}

{{

  1
  
admin
10aa1795-e315-46e7-a09a-3712acb51962
e681b2ae-7d10-44d9-befc-c0facc34cf64
c8d773fa-76ca-4637-8ecf-88656444fc86
VM-a4d03771-7267-4281-b615-d5cbae6c6e39 allocated 
(ServiceOffering: 231) (Template: 203)
24 Hrs
2
24
a4d03771-7267-4281-b615-d5cbae6c6e39
VM-a4d03771-7267-4281-b615-d5cbae6c6e39
5248afa9-f896-4608-bf3b-316262c21b9d
3a8dfd95-96c1-4d86-8365-9645a9ea7996
a4d03771-7267-4281-b615-d5cbae6c6e39
KVM
2
2400
8192
2017-10-30'T'00:00:00+00:00
2017-10-30'T'23:59:59+00:00
  

}}

Could I provide with more information that would help to find and fix the 
issue? I'd be glad to!



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10051) Mouse Scrolling is not working in instance VM console

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10051:
-

rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse 
Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r147935075
 
 

 ##
 File path: systemvm/js/jquery.flot.navigate.js
 ##
 @@ -0,0 +1,336 @@
+/*
 
 Review comment:
   @vedulasantosh sure, add this new library dependency in maven exclusion 
list, however, then remove the Apache license unless you authored this library.


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:
us...@infra.apache.org


> Mouse Scrolling is not working in instance VM console
> -
>
> Key: CLOUDSTACK-10051
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10051
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: vedula santosh
>
> When mouse is scrolled in VM Console it is not working.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10051) Mouse Scrolling is not working in instance VM console

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10051:
-

rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse 
Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r147934828
 
 

 ##
 File path: systemvm/js/ajaxviewer.js
 ##
 @@ -1230,6 +1230,23 @@ AjaxViewer.prototype = {
e.stopPropagation();
return false;
});
+
+   target.bind('mousewheel DOMMouseScroll',function(e) {
+   if(!ajaxViewer.ptInPanel(e.pageX, e.pageY))
 
 Review comment:
   Can you consider adding braces to `if () {}`


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:
us...@infra.apache.org


> Mouse Scrolling is not working in instance VM console
> -
>
> Key: CLOUDSTACK-10051
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10051
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: vedula santosh
>
> When mouse is scrolled in VM Console it is not working.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

rhtyd commented on a change in pull request #2302: CLOUDSTACK-10122: Unrelated 
error message
URL: https://github.com/apache/cloudstack/pull/2302#discussion_r147934443
 
 

 ##
 File path: 
engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
 ##
 @@ -1423,7 +1423,7 @@ public void prepare(VirtualMachineProfile vm, 
DeployDestination dest) throws Sto
 }
 }
 catch (Throwable e){
-s_logger.debug("[NSX_PLUGIN_LOG] ERROR: " + 
e.getMessage());
+s_logger.error("ERROR: " + e.getMessage());
 
 Review comment:
   LGTM


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10051) Mouse Scrolling is not working in instance VM console

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10051:
-

rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse 
Scrolling is not working in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#discussion_r147934703
 
 

 ##
 File path: systemvm/js/ajaxviewer.js
 ##
 @@ -1230,6 +1230,23 @@ AjaxViewer.prototype = {
e.stopPropagation();
return false;
});
+
+   target.bind('mousewheel DOMMouseScroll',function(e) {
+   if(!ajaxViewer.ptInPanel(e.pageX, e.pageY))
+   return true;
+
+   var modifiers = ajaxViewer.getKeyModifiers(e);
+   var sd = e.originalEvent.detail/3;
+
+   var pt = ajaxViewer.pageToPanel(e.clientX, e.clientY);
+
 
 Review comment:
   Remove extra newline.


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:
us...@infra.apache.org


> Mouse Scrolling is not working in instance VM console
> -
>
> Key: CLOUDSTACK-10051
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10051
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: vedula santosh
>
> When mouse is scrolled in VM Console it is not working.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10051) Mouse Scrolling is not working in instance VM console

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10051:
-

rhtyd commented on issue #2240: CLOUDSTACK-10051 Mouse Scrolling is not working 
in instance VM console
URL: https://github.com/apache/cloudstack/pull/2240#issuecomment-340709548
 
 
   Thanks @vedulasantosh for your PR, since these are purely UI changes we'll 
need an additional review/LGTM testing that can confirm the UI fix. Regression 
testing is not strictly required to merge this change.


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:
us...@infra.apache.org


> Mouse Scrolling is not working in instance VM console
> -
>
> Key: CLOUDSTACK-10051
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10051
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Reporter: vedula santosh
>
> When mouse is scrolled in VM Console it is not working.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10057:
-

rhtyd commented on issue #2250: CLOUDSTACK-10057: listNetworkOfferings now 
returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250#issuecomment-340708220
 
 
   LGTM, sure @fmaximus @krissterckx we can merge this.


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:
us...@infra.apache.org


> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10057:
-

rhtyd closed pull request #2250: CLOUDSTACK-10057: listNetworkOfferings now 
returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java 
b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
index 4dd52632338..6af8723afe3 100755
--- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
+++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java
@@ -4793,7 +4793,7 @@ protected void validateNtwkOffDetails(final Map details, final M
 // Now apply pagination
 final List wPagination = 
StringUtils.applyPagination(supportedOfferings, cmd.getStartIndex(), 
cmd.getPageSizeVal());
 if (wPagination != null) {
-final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, offerings.size());
+final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, supportedOfferings.size());
 return listWPagination;
 }
 return new Pair, 
Integer>(supportedOfferings, supportedOfferings.size());
diff --git a/server/test/com/cloud/configuration/ConfigurationManagerTest.java 
b/server/test/com/cloud/configuration/ConfigurationManagerTest.java
index 34f91c4e27c..80c184b0bf9 100644
--- a/server/test/com/cloud/configuration/ConfigurationManagerTest.java
+++ b/server/test/com/cloud/configuration/ConfigurationManagerTest.java
@@ -17,40 +17,34 @@
 
 package com.cloud.configuration;
 
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.anyBoolean;
-import static org.mockito.Matchers.anyInt;
-import static org.mockito.Matchers.anyLong;
-import static org.mockito.Matchers.anyString;
-import static org.mockito.Mockito.doNothing;
-import static org.mockito.Mockito.doThrow;
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.when;
-
 import java.lang.reflect.Field;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Random;
 import java.util.UUID;
 
-import com.cloud.user.User;
-import org.apache.cloudstack.engine.subsystem.api.storage.ZoneScope;
-import org.apache.cloudstack.storage.datastore.db.ImageStoreDao;
-import org.apache.cloudstack.storage.datastore.db.ImageStoreVO;
 import org.apache.log4j.Logger;
 import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
+import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
+
 import org.apache.cloudstack.api.command.admin.vlan.DedicatePublicIpRangeCmd;
 import org.apache.cloudstack.api.command.admin.vlan.ReleasePublicIpRangeCmd;
+import org.apache.cloudstack.api.command.user.network.ListNetworkOfferingsCmd;
 import org.apache.cloudstack.context.CallContext;
 import 
org.apache.cloudstack.engine.orchestration.service.NetworkOrchestrationService;
+import org.apache.cloudstack.engine.subsystem.api.storage.ZoneScope;
+import org.apache.cloudstack.framework.config.dao.ConfigurationDao;
+import org.apache.cloudstack.storage.datastore.db.ImageStoreDao;
+import org.apache.cloudstack.storage.datastore.db.ImageStoreVO;
 
 import com.cloud.configuration.Resource.ResourceType;
 import com.cloud.dc.AccountVlanMapVO;
@@ -72,13 +66,17 @@
 import com.cloud.host.dao.HostDao;
 import com.cloud.network.IpAddressManager;
 import com.cloud.network.Network;
-import com.cloud.network.NetworkModel;
 import com.cloud.network.Network.Capability;
+import com.cloud.network.NetworkModel;
+import com.cloud.network.Networks;
 import com.cloud.network.dao.FirewallRulesDao;
 import com.cloud.network.dao.IPAddressDao;
 import com.cloud.network.dao.IPAddressVO;
 import com.cloud.network.dao.PhysicalNetworkDao;
 import com.cloud.network.dao.PhysicalNetworkVO;
+import com.cloud.offering.NetworkOffering;
+import com.cloud.offerings.NetworkOfferingVO;
+import com.cloud.offerings.dao.NetworkOfferingDao;
 import com.cloud.projects.ProjectManager;
 import com.cloud.storage.VolumeVO;
 import com.cloud.storage.dao.VolumeDao;
@@ -86,18 +84,34 @@
 import com.cloud.user.AccountManager;
 import com.cloud.user.AccountVO;
 import com.cloud.user.ResourceLimitService;
+import com.cloud.user.User;
 import com.cloud.user.UserVO;
 import com.cloud.user.dao.AccountDao;
+import com.clo

[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF subversion and git services (JIRA)

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

ASF subversion and git services commented on CLOUDSTACK-10057:
--

Commit 587b66db71dd417c957f59e80a01e7e08e60be1f in cloudstack's branch 
refs/heads/master from [~sgoeminn]
[ https://gitbox.apache.org/repos/asf?p=cloudstack.git;h=587b66d ]

CLOUDSTACK-10057: listNetworkOfferings now returns the correct number of 
offerings (#2250)

Fix paginated response to send correct total resource counts.

> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10057:
-

rhtyd commented on a change in pull request #2250: CLOUDSTACK-10057: 
listNetworkOfferings now returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250#discussion_r147934160
 
 

 ##
 File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
 ##
 @@ -4788,7 +4788,7 @@ protected void validateNtwkOffDetails(final Map details, final M
 // Now apply pagination
 final List wPagination = 
StringUtils.applyPagination(supportedOfferings, cmd.getStartIndex(), 
cmd.getPageSizeVal());
 if (wPagination != null) {
-final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, offerings.size());
+final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, supportedOfferings.size());
 
 Review comment:
   @sgoeminn thanks.


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:
us...@infra.apache.org


> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10057:
-

sgoeminn commented on a change in pull request #2250: CLOUDSTACK-10057: 
listNetworkOfferings now returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250#discussion_r147928964
 
 

 ##
 File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
 ##
 @@ -4788,7 +4788,7 @@ protected void validateNtwkOffDetails(final Map details, final M
 // Now apply pagination
 final List wPagination = 
StringUtils.applyPagination(supportedOfferings, cmd.getStartIndex(), 
cmd.getPageSizeVal());
 if (wPagination != null) {
-final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, offerings.size());
+final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, supportedOfferings.size());
 
 Review comment:
   @rhtyd we always want to return the number of total found offerings and not 
only the number of offerings in the page that is returned. This is the same 
behavior as all the other list cmds.


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:
us...@infra.apache.org


> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-9590) KVM + CentOS 7.2 + Agent in Alert State for long time

2017-10-31 Thread Rajani Karuturi (JIRA)

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

Rajani Karuturi commented on CLOUDSTACK-9590:
-

Hi @sven.vogel and @dag.Sonstebo 
I am facing a similar issue with adding KVM host.
Which java is downgraded? The MS java version or host java version? Do you know 
any reason for this to happen? Is it java issue or cloudstack issue?

> KVM + CentOS 7.2 + Agent in Alert State for long time
> -
>
> Key: CLOUDSTACK-9590
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9590
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: cloudstack-agent
>Affects Versions: 4.9.0
> Environment: entOS Linux release 7.2.1511 (Core)
> cloudstack-agent-4.9.0-1.el7.centos.x86_64
>Reporter: Sven Vogel
> Fix For: 4.9.1.0
>
> Attachments: agent.log, cloudstack-startup.log, management-server.zip
>
>
> Hi,
> When i add a new host to cloudstack management server it take some time to 
> get host out from alert state.
> 1. i add the host and host add not possible
> 2. values are correct set to agent.properties, restart cloustack agent
> 3. agent says connected to server
> 4. management server says "alert"
> management-server.log
> 2016-11-10 13:23:06,783 DEBUG [c.c.h.Status] 
> (AgentConnectTaskPool-49:ctx-c3b72839) (logid:5a86e1fb) Transition:[Resource 
> state = Enabled, Agent event = AgentDisconnected, Host
> id = 51, name = kvm02.oscloud.local]
> 2016-11-10 13:23:06,798 DEBUG [c.c.a.m.ClusteredAgentManagerImpl] 
> (AgentConnectTaskPool-49:ctx-c3b72839) (logid:5a86e1fb) Notifying other nodes 
> of to disconnect
> 2016-11-10 13:23:06,806 DEBUG [c.c.a.m.AgentManagerImpl] 
> (AgentConnectTaskPool-49:ctx-c3b72839) (logid:5a86e1fb) Failed to handle host 
> connection: com.cloud.exception.Connection
> Exception: Unable to get an answer to the CheckNetworkCommand from agent: 51
> is there any way to speed up the alert state? is it normal that it take so 
> long?
> thanks
> Sven



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

marcaurele commented on a change in pull request #2302: CLOUDSTACK-10122: 
Unrelated error message
URL: https://github.com/apache/cloudstack/pull/2302#discussion_r147920880
 
 

 ##
 File path: 
engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
 ##
 @@ -1423,7 +1423,7 @@ public void prepare(VirtualMachineProfile vm, 
DeployDestination dest) throws Sto
 }
 }
 catch (Throwable e){
-s_logger.debug("[NSX_PLUGIN_LOG] ERROR: " + 
e.getMessage());
 
 Review comment:
   Agree with @rafaelweingartner - it should be removed all together. It would 
be hard anyway to track the problem since the log output will only output the 
message and not the stack trace. If we're trying to catch any error, the stack 
is very important to understand the issue. This would normally be done at a 
higher level if the exception isn't caught here.


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10122) Unrelated error message

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10122:
-

rafaelweingartner commented on a change in pull request #2302: 
CLOUDSTACK-10122: Unrelated error message
URL: https://github.com/apache/cloudstack/pull/2302#discussion_r147916282
 
 

 ##
 File path: 
engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
 ##
 @@ -1423,7 +1423,7 @@ public void prepare(VirtualMachineProfile vm, 
DeployDestination dest) throws Sto
 }
 }
 catch (Throwable e){
-s_logger.debug("[NSX_PLUGIN_LOG] ERROR: " + 
e.getMessage());
 
 Review comment:
   I would remove the try/catch block. I really did not seen any relevant 
exception in that piece of code to be dealt if there.


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:
us...@infra.apache.org


> Unrelated error message
> ---
>
> Key: CLOUDSTACK-10122
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10122
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>Affects Versions: 4.11.0.0
>Reporter: Nicolas Vazquez
>Assignee: Nicolas Vazquez
>
> Remove unrelated text "NSX_PLUGIN_LOG" from error message



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10057:
-

rhtyd commented on a change in pull request #2250: CLOUDSTACK-10057: 
listNetworkOfferings now returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250#discussion_r147909203
 
 

 ##
 File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
 ##
 @@ -4788,7 +4788,7 @@ protected void validateNtwkOffDetails(final Map details, final M
 // Now apply pagination
 final List wPagination = 
StringUtils.applyPagination(supportedOfferings, cmd.getStartIndex(), 
cmd.getPageSizeVal());
 if (wPagination != null) {
-final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, offerings.size());
+final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, supportedOfferings.size());
 
 Review comment:
   @sgoeminn why not use `wPagination.size()`, or is it that you want to expose 
the count value equal to total found/supported offerings however only return 
the offerings in the current page?


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:
us...@infra.apache.org


> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (CLOUDSTACK-10057) ListNetworkOfferingsCmd does not return the correct count

2017-10-31 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CLOUDSTACK-10057:
-

rhtyd commented on a change in pull request #2250: CLOUDSTACK-10057: 
listNetworkOfferings now returns the correct number  of offerings.
URL: https://github.com/apache/cloudstack/pull/2250#discussion_r147909203
 
 

 ##
 File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java
 ##
 @@ -4788,7 +4788,7 @@ protected void validateNtwkOffDetails(final Map details, final M
 // Now apply pagination
 final List wPagination = 
StringUtils.applyPagination(supportedOfferings, cmd.getStartIndex(), 
cmd.getPageSizeVal());
 if (wPagination != null) {
-final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, offerings.size());
+final Pair, Integer> 
listWPagination = new Pair, 
Integer>(wPagination, supportedOfferings.size());
 
 Review comment:
   @sgoeminn why not use `wPagination.size()` ?


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:
us...@infra.apache.org


> ListNetworkOfferingsCmd does not return the correct count
> -
>
> Key: CLOUDSTACK-10057
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10057
> Project: CloudStack
>  Issue Type: Bug
>  Security Level: Public(Anyone can view this level - this is the 
> default.) 
>  Components: API
>Affects Versions: 4.11.0.0
>Reporter: Sigert Goeminne
>Assignee: Sigert Goeminne
>
> ListNetworkOfferingsCmd returns a count value which is incorrect (no network 
> offerings are returned but the count is set to a value higher then 0).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)