Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1907
code 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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1972
code 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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1950
updated.
changes
(1) add init-system-helpers version
(2) add ExecStartPre to avoid the issue caused by JAVA_HOME
---
If your project is set up for it, you can reply to this email
please use the criteria mentioned at http://acspr.ena.net/help.html to tag
PRs as merge ready.
~Rajani
http://cloudplatform.accelerite.com/
On Thu, Feb 23, 2017 at 4:24 AM, Sergey Levitskiy <
sergey.levits...@autodesk.com> wrote:
> I would like to get this added
> https://github.com/apache/cloud
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@rhtyd The permission issue seems to be caused by
./scripts/vm/systemvm/injectkeys.sh
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1935
@rhtyd @karuturi To fix errors in B.O in
test_02_list_snapshots_with_removed_data_store we need to merge PR1961 and then
adjust test_data on B.O side to have correct mapping for nfs2 label
---
Github user yvsubhash commented on the issue:
https://github.com/apache/cloudstack/pull/1942
LGTM for code 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 project does not have this feature
enabled and wis
Github user marcaurele commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1944#discussion_r103143033
--- Diff: plugins/metrics/pom.xml ---
@@ -0,0 +1,55 @@
+
+http://maven.apache.org/POM/4.0.0";
+ xmlns:xsi="http://www.w3.org/2001/
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1935
@nvazquez can you check if the failures in above test results are related
to your changes, thanks.
---
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/1954
Thanks @dmabry
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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1954
@dmabry can we have this in 4.9 as well?
---
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/1915
LGTM @karuturi this is ready for merge
Ping @abhinandanprateek in case you want to review?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1212
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 feature
en
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1907
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 feature
en
Hi will,
I believe, we didn’t get to close ‘getting a mirror on Apache’ because we
needed someone on the Apache Infra side to close this. BTW,
cloudstack-apt.get.eu (I think Nux manages this?) has all/most of the content.
Once we can close on the Apache mirror for hosting the content, I can h
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1946
LGTM. @karuturi let's merge 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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1944
Thanks @rashmidixit awaiting your review
---
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/1945
LGTM. @karuturi let's merge 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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1903
LGTM. @karuturi let's merge 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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1950
@ustcweizhou can you check the permissions issue I hit in my test, as share
in my comment above?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1896
LGTM. @karuturi let's merge 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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1829
Okay @syed either PR need to be merged to solve the issue of restart bug.
---
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 pr
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1941
LGTM. @karuturi let's merge 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
Github user shwetaag commented on the issue:
https://github.com/apache/cloudstack/pull/1949
@rhtyd any updates on this PR .. it shows Travis build failed ..Does this
need any action on my part ?
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1886
Code changes 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 wishe
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1252
### ACS CI BVT Run
**Sumarry:**
Build Number 397
Hypervisor xenserver
NetworkType Advanced
Passed=102
Failed=3
Skipped=7
_Link to logs Folder (searc
Github user niteshsarda commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1972#discussion_r103138362
--- Diff: engine/schema/src/com/cloud/network/dao/LoadBalancerDaoImpl.java
---
@@ -75,4 +84,40 @@ protected LoadBalancerDaoImpl() {
ret
Github user anshul1886 commented on the issue:
https://github.com/apache/cloudstack/pull/669
tag:mergeready
---
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 SudharmaJain commented on the issue:
https://github.com/apache/cloudstack/pull/1869
LGTM on the code 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 project does not have this feature
enabled
Github user sudhansu7 closed the pull request at:
https://github.com/apache/cloudstack/pull/1795
---
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 feature
Github user sudhansu7 commented on the issue:
https://github.com/apache/cloudstack/pull/1795
Closing this PR as this is already fixed in master.
---
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 hav
Github user jayapalu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1886#discussion_r103136343
--- Diff: core/src/com/cloud/network/resource/TrafficSentinelResource.java
---
@@ -228,6 +239,7 @@ private DirectNetworkUsageAnswer
getPublicIpBytesSe
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1253
@sureshanaparti Can you please investigate the two failures listed in
results from @cloudmonger?
---
If your project is set up for it, you can reply to this email and have your
rep
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1257
@karuturi @koushik This has 1 LGTM and test results without any failures.
Can you please review this?
---
If your project is set up for it, you can reply to this email and have
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1958
Trillian test result (tid-899)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31181 seconds
Marvin logs:
https://github.com/blueoranguta
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1253
### ACS CI BVT Run
**Sumarry:**
Build Number 396
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=2
Skipped=7
_Link to logs Folder (searc
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1257
### ACS CI BVT Run
**Sumarry:**
Build Number 395
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=7
_Link to logs Folder (searc
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1958
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1958
@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 w
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1951
@syed yes there is no problem I see, with `INSERT IGNORE` we can have this
db change in both paths resulting in same final state.
---
If your project is set up for it, you can reply to this email
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1965
### ACS CI BVT Run
**Sumarry:**
Build Number 394
Hypervisor xenserver
NetworkType Advanced
Passed=104
Failed=1
Skipped=7
_Link to logs Folder (searc
41 matches
Mail list logo
