Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1947
@karuturi this is ready for merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1950
LGTM, I'll test it manually as soon as I get some free time.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1945
@karuturi this is ready for merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1946
LGTM.
@karuturi this is ready for merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1770
@karuturi this is ready for merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1896
@karuturi pure translation changes, we need one more LGTM, otherwise this
is ready for merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1829
This is ready for merge @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/926
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1212
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1945
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1903
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1941
@blueorangutan test centos7 xenserver-65sp1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/843
@blueorangutan test centos7 xenserver-65sp1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/843
@blueorangutan help
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1951
Requesting for review - @DaanHoogland @abhinandanprateek @karuturi
@koushik-das @syed
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
Requesting for review - @abhinandanprateek @DaanHoogland @rashmidixit
@karuturi @koushik-das
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1960
CLOUDSTACK-9782: Host HA and KVM HA provider
Host-HA offers investigation, fencing and recovery mechanisms for host that
for
any reason are malfunctioning. It uses Activity and Health checks
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1951
Thanks @borisstoyanov
@karuturi ping
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@borisstoyanov can you please identify the failures against Ubuntu
environment, thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1948
@borisstoyanov cloud-scripts tarball in the systemvm.iso will have the
router/scripts. We can test it.
@blueorangutan test
---
If your project is set up for it, you can reply to this email
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1949
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1829
@karuturi let's merge this, thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1779
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/843
Thanks @jayapalu makes sense.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1847
@borisstoyanov can you have a look at ^^, thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1880
@blueorangutan test centos7 vmware-55u3
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1875
@blueorangutan test centos7 vmware-55u3
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@blueorangutan test centos7 vmware-55u3
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1880
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1875
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1879
Sure.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/847
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1917
@jayapalu thanks, yes it seems an intermittent issue with `ipmisim`,
`pyghmi` library that is used by the test to simulate a live ipmi server/board.
I'll try to fix it soon, meanwhile yo
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1829
LGTM, code and (regression) tests.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1951
@borisstoyanov an explicit integration test is not necessary if Travis
passes, since all changes are around db upgrade/version.
---
If your project is set up for it, you can reply to this email
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
Thanks @borisstoyanov test results LGTM.
/cc @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1951#discussion_r101735670
--- Diff: setup/db/db/schema-4920to4930.sql ---
@@ -0,0 +1,22 @@
+-- Licensed to the Apache Software Foundation (ASF) under one
+-- or more
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1951
CLOUDSTACK-9792: Add upgrade path for 4.9.3.0
This adds an upgrade path from 4.9.2.0 to 4.9.3.0, this also includes
changes from PR https://github.com/apache/cloudstack/pull/1928 that adds
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1916
@ustcweizhou makes sense, in that case please close this PR and open a new
PR for 4.10/master that adds support for cloudstack mgmt server on ubuntu
16.04, dropping support for 12.04 etc
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1916
@ustcweizhou I think it would be good to add support for 16.04/systemd to
run mgmt server. But Given 12.04 will EOL in next 2.5 months, let's have
support for 16.04 in 4.9 as well, drop su
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1856
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1770
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1856
LGTM.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1901
LGTM, @ustcweizhou can you add a marvin test to validate api changes?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1903
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1916
@borisstoyanov can you check Trillian failures?
@ustcweizhou can you check Jenkins/Travis failures? I'm in favour of
removing 12.04 support (for mgmt server) in 4.9+, since the suppor
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1947
LGTM.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1946
This might affect other/all APIs therefore would require regression
testing, but LGTM (did not test this).
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1770
LGTM, did not test it.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1928
@syed I'll work on the upgrade path today and include your fix, thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/847
LGTM, @kishankavala please squash your changes and amend git summary with
jira id.
/cc @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1913
LGTM. /cc @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1829
LGTM. /cc @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
Thanks for testing/reviewing @borisstoyanov
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1927
@karuturi blueorangutan by default can run all the smoke/component tests --
but by default only in an advanced zone (advanced network). If your question
was around testing this in basic zone
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1826
Thanks @rashmidixit both the PRs may be closed given the new performance
improvement PR #1944 which also includes some case fixes from this PR.
---
If your project is set up for it, you can
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1884
This is not needed with PR #1944, though changes around capability
refactoring can be separated.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/847
Thanks @kishankavala
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1826
@rashmidixit I've taken your feedback and improved the feature here
https://github.com/apache/cloudstack/pull/1944 -- this reimplements the
frontend logic at the backend, improving the ov
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1944
CLOUDSTACK-9783: Improve metrics view performance
This improves the metrics view feature by improving the rendering
performance
of metrics view tables, by re-implementing the logic at the
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1941
@anshul1886 can you check and comment on template related failure?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1941
@blueorangutan test centos7 xenserver-65sp1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1941
@blueorangutan help
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1927
LGTM, let's merge this blocker @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1941
Thanks @anshul1886
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/672
@anshul1886 yes given you've created a new PR, you may close this one.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/672
Thanks @anshul1886 kindly also squash your changes and fix the commit
message/jira id etc
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1836
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/672
@anshul1886 I would suggest, if your PR already has @syed 's fix along with
other useful fixes -- please rebase against 4.9, fix the conflicts and we can
help review/test your PR if you
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1836
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1928
@syed simply moving this to a file is not enough. Please see
Upgrade4910to4920.java and the class usage to see how upgrade paths are
implemented. We'll need a Upgrade4920to4930 upgrade
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1741
Thanks @swill @jayapalu for the work. I'll build a new systemvmtemplate to
use with upcoming Trilian tests on master/4.10+.
---
If your project is set up for it, you can reply to this emai
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/672
@anshul1886 can you retarget this PR for 4.9? Also rebase against 4.9, fix
conflicts thanks. /cc @karuturi @syed @koushik-das
---
If your project is set up for it, you can reply to this email
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1829
LGTM. @karuturi this can be merged.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1884
I'm working on improving the metrics view feature by implementing the logic
as backend APIs, will post a PR this/next week. Thanks.
---
If your project is set up for it, you can reply to
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1914
LGTM, /cc @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1928
@syed since 4.9.2.0 is already released, please created 4.9.2.0->4.9.3.0
upgrade path and move the changes to the suggested sql file. This needs to go
into master/4.10 as well /cc @karut
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1913
@blueorangutan test centos7 vmware-60u2
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1659
@murali-reddy @abhinandanprateek @DaanHoogland can you comment on this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1928#discussion_r99288392
--- Diff: setup/db/db/schema-4910to4920.sql ---
@@ -236,3 +236,5 @@ CALL
`cloud`.`IDEMPOTENT_INSERT_GUESTOS_HYPERVISOR_MAPPING`('Xenserver
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1913
@karuturi please package first
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1923
Thanks @resmo can you re-target this PR for 4.9 branch? Let's get this in
both 4.9 and master.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1871
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1871
@karuturi looks like it failed, I'll re-kick it
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1794
Ping, @karuturi can you squash your changes and rebase? Let's add the
support in 4.10.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1889
LGTM, did not test it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1741
Ping, update on this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1920
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1921
LGTM, did not test it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the pull request:
https://github.com/apache/cloudstack/commit/e6cc78f531d0201aed004fd9e003a1ae2ab9a816#commitcomment-20657567
In tools/docker/Dockerfile:
In tools/docker/Dockerfile on line 25:
Sure, thanks @pdion891
---
If your project is set
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1916
@ustcweizhou can you check/fix the failures?
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1836
@blueorangutan test matrix
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1836
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1837
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1888
@swill build el7 (centos7) packages on centos7, and el6 (centos6) packages
on centos6. You may use my docker images, this is what Trillian/blueorangutan
uses for building (PR) packages: https
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1888
@syed yes, if the systemvmtemplates were built from master after this PR
was merged. There is no secret sauce in the systemvmtemplate that I've uploaded
on packages.shapeblue.com S3 b
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1888
@serg38 I've been maintaining the non-oss libs here:
https://github.com/rhtyd/cloudstack-nonoss
You can get the latest v12.1 library that is Java8 compatible from above,
it also include
201 - 300 of 2445 matches
Mail list logo