blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344905603
Packaging result: ?centos6 ?centos7 ?debian. JID-1264
This is an automated message
resmo opened a new pull request #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330
Prevent NPE for querying a deleted project.
This is an automated message from
GabrielBrascher commented on issue #2326: Re-work method
QuotaResponseBuilderImpl.startOfNextDay and its test cases
URL: https://github.com/apache/cloudstack/pull/2326#issuecomment-344917958
Thanks Rafael, code LGTM.
This is
ustcweizhou commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344919959
code LGTM.
However, could you please add some code to release vlan and ip ranges while
delete a project ?
If it
blueorangutan commented on issue #2324: Remove annotation and "depends-on"
declaration not needed at cloud-engine-storage-image
URL: https://github.com/apache/cloudstack/pull/2324#issuecomment-344967093
Trillian test result (tid-1655)
Environment: kvm-centos7 (x2), Advanced Networking
rafaelweingartner commented on a change in pull request #2331: CLOUDSTACK-10145
Mouse Double Click is not working in instance VM console where there is latency
in network
URL: https://github.com/apache/cloudstack/pull/2331#discussion_r151459184
##
File path:
fmaximus closed pull request #2005: CLOUDSTACK-9450: Network Offering for VPC
based on DB flag
URL: https://github.com/apache/cloudstack/pull/2005
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:
This is an automated email from the ASF dual-hosted git repository.
fmaximus pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.
from a1008f9 CLOUDSTACK-10129: Show account, network info in VR list view
(#2327)
add d467a6b
vedulasantosh opened a new pull request #2331: CLOUDSTACK-10145 Mouse Double
Click is not working in instance VM console where there is latency in network
URL: https://github.com/apache/cloudstack/pull/2331
Mouse Double Click is not working in instance VM console where there is
latency in
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344900335
@blueorangutan package
This is an automated message from the Apache Git Service.
To
anshul1886 commented on issue #2315: A comprehensive solution for
#CLOUDSTACK-9025.
URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-344901475
@rafaelweingartner I hope one thing is clear here that in case of
non-managed storage pool snapshots reside on secondary storage
blueorangutan commented on issue #2318: [4.9] Smoketest health checkrun baseline
URL: https://github.com/apache/cloudstack/pull/2318#issuecomment-344914782
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
rhtyd closed pull request #2327: CLOUDSTACK-10129: Show account, network info
in VR list view
URL: https://github.com/apache/cloudstack/pull/2327
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:
This is an automated email from the ASF dual-hosted git repository.
bhaisaab pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/master by this push:
new a1008f9 CLOUDSTACK-10129: Show account,
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344900447
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
borisstoyanov commented on issue #2318: [4.9] Smoketest health checkrun baseline
URL: https://github.com/apache/cloudstack/pull/2318#issuecomment-344914525
@blueorangutan test
This is an automated message from the Apache Git
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344920896
@marcaurele I found a workaround to use the old 9.2.x jetty-maven-plugin as
the newer one did not work for me (and Travis). Please review the changes.
resmo commented on issue #2327: CLOUDSTACK-10129: Show account, network info in
VR list view
URL: https://github.com/apache/cloudstack/pull/2327#issuecomment-344917667
LGTM
This is an automated message from the Apache Git
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344902497
@marcaurele how do you start the jetty server with mvn, when I run `mvn -pl
:cloud-client-ui jetty:run -Dnoredist -Dsimulator
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344928527
Packaging result: ?centos6 ?centos7 ?debian. JID-1265
This is an automated message
This is an automated email from the ASF dual-hosted git repository.
marcaurele pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git
commit 869c527aabe33b142ebae2c5460d9558eb7c87d1
Merge: 8c801a4 637e853
Author: Marc-Aurèle Brothier
anshul1886 commented on issue #2315: A comprehensive solution for
#CLOUDSTACK-9025.
URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-344902027
@rafaelweingartner Also take care, host which is getting used is enabled.
Only when no enabled host is available use disabled
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344900300
@marcaurele I've fixed the cloudian unit test failures and the session
timeout to 30 (mins).
marcaurele commented on issue #2324: Remove annotation and "depends-on"
declaration not needed at cloud-engine-storage-image
URL: https://github.com/apache/cloudstack/pull/2324#issuecomment-344839965
LGTM - ~the `@Component` cannot work due to the system of spring sub modules
created for
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344920945
@blueorangutan package
This is an automated message from the Apache Git Service.
To
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344921157
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
ustcweizhou commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344931408
@resmo agreed. let's merge this at first.
It would be nice to fix the issue you said.
resmo commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344921452
rollback transaction on releasing vlan and ip ranges while delete a project
was the root cause getting into this state. Unsure
anshul1886 commented on a change in pull request #2315: A comprehensive
solution for #CLOUDSTACK-9025.
URL: https://github.com/apache/cloudstack/pull/2315#discussion_r151392089
##
File path: engine/schema/src/com/cloud/host/dao/HostDaoImpl.java
##
@@ -1194,4 +1178,26 @@
blueorangutan commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344921395
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
rhtyd commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344921281
@blueorangutan package
This is an automated message from the
blueorangutan commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344928818
Packaging result: ?centos6 ?centos7 ?debian. JID-1266
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-345010973
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been
kicked to run smoke tests
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-345010688
@blueorangutan test
This is an automated message from the Apache Git Service.
To respond
This is an automated email from the ASF dual-hosted git repository.
bhaisaab pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/master by this push:
new 7f6ae15 Remove annotation not needed at
rhtyd commented on issue #2324: Remove annotation and "depends-on" declaration
not needed at cloud-engine-storage-image
URL: https://github.com/apache/cloudstack/pull/2324#issuecomment-345000974
Ignoring intermittent failures not caused by this PR, I'll proceed with
merging this. Thanks
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-345010088
Packaging result: ?centos6 ?centos7 ?debian. JID-1268
This is an automated message
blueorangutan commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-345010073
Packaging result: ?centos6 ?centos7 ?debian. JID-1267
rhtyd commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-344999763
@blueorangutan package
This is an automated message from the
rhtyd commented on issue #2324: Remove annotation and "depends-on" declaration
not needed at cloud-engine-storage-image
URL: https://github.com/apache/cloudstack/pull/2324#issuecomment-345000236
Tests LGTM
This is an
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-345000224
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
This is an automated email from the ASF dual-hosted git repository.
bhaisaab pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git
The following commit(s) were added to refs/heads/master by this push:
new cb246f5 Typo fix in download url (1
blueorangutan commented on issue #1981: CLOUDSTACK-9806: Nuage domain template
selection per VPC
URL: https://github.com/apache/cloudstack/pull/1981#issuecomment-345029232
Trillian test result (tid-1656)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time
blueorangutan commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-34456
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-34510
@blueorangutan package
This is an automated message from the Apache Git Service.
To
rhtyd closed pull request #2324: Remove annotation and "depends-on" declaration
not needed at cloud-engine-storage-image
URL: https://github.com/apache/cloudstack/pull/2324
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
This is an automated email from the ASF dual-hosted git repository.
bhaisaab pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/master by this push:
new 55c059c CLOUDSTACK-10144: fix possible
rhtyd commented on issue #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330#issuecomment-345143037
Simple npe check, merging based to Travis and packaging validations.
rhtyd closed pull request #2330: CLOUDSTACK-10144: fix possible NPE in
listVlanIpRanges
URL: https://github.com/apache/cloudstack/pull/2330
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
rhtyd commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey
based approach for reading 'ping' configua?
URL: https://github.com/apache/cloudstack/pull/2292#discussion_r151601804
##
File path: engine/schema/src/com/cloud/host/dao/HostDaoImpl.java
##
@@
blueorangutan commented on issue #2298: CLOUDSTACK-9620: Enhancements for
managed storage
URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-345143813
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed
storage
URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-345143722
@blueorangutan package
This is an automated message from the
blueorangutan commented on issue #2298: CLOUDSTACK-9620: Enhancements for
managed storage
URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-345146393
Packaging result: ?centos6 ?centos7 ?debian. JID-1269
This
blueorangutan commented on issue #1981: CLOUDSTACK-9806: Nuage domain template
selection per VPC
URL: https://github.com/apache/cloudstack/pull/1981#issuecomment-344887369
Packaging result: ?centos6 ?centos7 ?debian. JID-1262
blueorangutan commented on issue #1981: CLOUDSTACK-9806: Nuage domain template
selection per VPC
URL: https://github.com/apache/cloudstack/pull/1981#issuecomment-344887776
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
borisstoyanov commented on issue #1981: CLOUDSTACK-9806: Nuage domain template
selection per VPC
URL: https://github.com/apache/cloudstack/pull/1981#issuecomment-344887676
@blueorangutan test
This is an automated message
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344888577
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
rhtyd commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344888394
@blueorangutan package
This is an automated message from the Apache Git Service.
To
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344892584
Packaging result: ?centos6 ?centos7 ?debian. JID-1263
This is an automated message
rhtyd commented on issue #2327: CLOUDSTACK-10129: Show account, network info in
VR list view
URL: https://github.com/apache/cloudstack/pull/2327#issuecomment-344893553
Can you re-review @wido @ustcweizhou @borisstoyanov thanks.
rafaelweingartner commented on a change in pull request #2327:
CLOUDSTACK-10129: Show account, network info in VR list view
URL: https://github.com/apache/cloudstack/pull/2327#discussion_r151386526
##
File path: ui/scripts/system.js
##
@@ -29,9 +29,19 @@
var
rhtyd commented on a change in pull request #2327: CLOUDSTACK-10129: Show
account, network info in VR list view
URL: https://github.com/apache/cloudstack/pull/2327#discussion_r151388752
##
File path: ui/scripts/system.js
##
@@ -29,9 +29,19 @@
var mapRouterType =
blueorangutan commented on issue #2318: [4.9] Smoketest health checkrun baseline
URL: https://github.com/apache/cloudstack/pull/2318#issuecomment-345050405
Trillian test result (tid-1657)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26631
marcaurele opened a new pull request #2329: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329
Upgrading to jetty 9.4 along with:
- new gzip handler
- redirect `/` to the context path
- change for cloudian tests
ustcweizhou commented on issue #2327: CLOUDSTACK-10129: Show account, network
info in VR list view
URL: https://github.com/apache/cloudstack/pull/2327#issuecomment-344846455
@rhtyd I suggest to show 'Network' or 'VPC' in 'Type' field, and show the
network/vpc name in 'Network' field.
marcaurele commented on issue #2328: CLOUDSTACK-10012: Upgrade Jetty to 9.3.x
URL: https://github.com/apache/cloudstack/pull/2328#issuecomment-344847572
@wido The problem upgrading to jetty 9.3 or 9.4 is the wiremock dependency
which forces the use of jetty 9.2. The dependency is only used
marcaurele commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-344848923
superseed #2328 (Jetty 9.4 is Java8 compatible)
cc @rhtyd @wido
This is an
This is an automated email from the ASF dual-hosted git repository.
gabriel pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git
commit 6db8413a525e30bd13a18f45597a635c8d598ba1
Merge: cb246f5 f94070e
Author: Gabriel Beims Bräscher
This is an automated email from the ASF dual-hosted git repository.
gabriel pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git.
from cb246f5 Typo fix in download url (1 was inserted) (#40)
add f94070e Use 'latest' instead of last
borisstoyanov commented on issue #1981: CLOUDSTACK-9806: Nuage domain template
selection per VPC
URL: https://github.com/apache/cloudstack/pull/1981#issuecomment-344881413
@blueorangutan package
This is an automated message
rhtyd commented on issue #2327: CLOUDSTACK-10129: Show account, network info in
VR list view
URL: https://github.com/apache/cloudstack/pull/2327#issuecomment-344884522
@ustcweizhou thanks, I see your point. The router type is not something
return by the API, but set by the UI limited to
rafaelweingartner commented on issue #2324: Remove annotation and "depends-on"
declaration not needed at cloud-engine-storage-image
URL: https://github.com/apache/cloudstack/pull/2324#issuecomment-344886007
@marcaurele yes, it is not working because these modules do not inherit the
blueorangutan commented on issue #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#issuecomment-345153109
Trillian test result (tid-1658)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42208 seconds
This is an automated email from the ASF dual-hosted git repository.
marcaurele pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git
The following commit(s) were added to refs/heads/master by this push:
new 3b777b1 Version 4.10 should be
blueorangutan commented on issue #2328: CLOUDSTACK-10012: Upgrade Jetty to 9.3.x
URL: https://github.com/apache/cloudstack/pull/2328#issuecomment-344860967
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted
as I make progress.
rhtyd commented on issue #2328: CLOUDSTACK-10012: Upgrade Jetty to 9.3.x
URL: https://github.com/apache/cloudstack/pull/2328#issuecomment-344860812
@wido yes, building a package and testing that management server works as
expected is a valid regression test.
Thanks @marcaurele -- last
rhtyd closed pull request #2328: CLOUDSTACK-10012: Upgrade Jetty to 9.3.x
URL: https://github.com/apache/cloudstack/pull/2328
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
blueorangutan commented on issue #2328: CLOUDSTACK-10012: Upgrade Jetty to 9.3.x
URL: https://github.com/apache/cloudstack/pull/2328#issuecomment-344866823
Packaging result: ?centos6 ?centos7 ?debian. JID-1261
This is an
rhtyd commented on a change in pull request #2329: CLOUDSTACK-10012: Jetty 9.4
URL: https://github.com/apache/cloudstack/pull/2329#discussion_r151355734
##
File path: client/conf/server.properties.in
##
@@ -24,6 +24,9 @@ context.path=/client
# The HTTP port to be used by
79 matches
Mail list logo
