Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/2032
---
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 koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/2032
LGTM. Build is broken due to 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
GitHub user karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/2032
CLOUDSTACK-9783: corrected the version number in metrics pom
master is broken after fwd-merging metrics PR #1944 from 4.9
This is due to the incorrect parent version number in the metrics
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/2018
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1944
master is broken after the merge. Pom version 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
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/2018
Understood! Looks good to me.
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
Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/2023
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 nitin-maharana opened a pull request:
https://github.com/apache/cloudstack/pull/2031
CLOUDSTACK - 8672 : Test PR
Currently running with few commit. Please don't review this PR.
According to the result of this PR, I will update in original PR(1859).
You can merge
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/2025
Trillian test result (tid-974)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28997 seconds
Marvin logs:
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1935
Trillian test result (tid-973)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 29259 seconds
Marvin logs:
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/2027
Thanks @marcaurele that's a good catch! Can you push -f and see why Travis
failed, if your fix works let's reenable this test! Also, can you edit the PR
and send it for 4.9?
---
If your project
Github user marcaurele commented on the issue:
https://github.com/apache/cloudstack/pull/2027
@rhtyd I found one issue with the test and `NioConnection` class. This kind
of intermittent problem are always hard to search for a root cause, but after
lots of logging I finally found why.
Jayapal: Just an FYI that I haven't had a chance to investigate this yet.
I did notice, however, that the same behavior exists in 4.5.
> On Apr 2, 2017, at 10:13 PM, Jayapal Uradi
> wrote:
>
> Hi Mike,
>
> Can you please check the following
> 1. In kvm host
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1889
---
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 rashmidixit reopened a pull request:
https://github.com/apache/cloudstack/pull/1889
CLOUDSTACK-9718: Revamp the dropdown showing lists of hosts available for
migration in a Zone
Refer to
[CLOUDSTACK-9718](https://issues.apache.org/jira/browse/CLOUDSTACK-9718) for
more
Github user abhinandanprateek commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/2030#discussion_r110145877
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
---
@@ -550,15 +552,21 @@ public boolean
Github user abhinandanprateek commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/2030#discussion_r110145761
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
---
@@ -128,6 +129,7 @@
public
Github user ProjectMoon commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1908#discussion_r110113725
--- Diff:
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java
---
@@ -625,15 +627,20 @@ protected
Github user jayapalu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1908#discussion_r110112007
--- Diff: server/src/com/cloud/network/router/CommandSetupHelper.java ---
@@ -848,13 +849,37 @@ public int compare(final PublicIpAddress o1, final
Github user jayapalu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1908#discussion_r11089
--- Diff: server/src/com/cloud/network/router/CommandSetupHelper.java ---
@@ -848,13 +849,37 @@ public int compare(final PublicIpAddress o1, final
Github user rashmidixit commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@karuturi I have rebased with master and pushed. I still see failures in
Travis but they dont seem to be with my changes. Can you please check?
---
If your project is set up for it, you
GitHub user DaanHoogland opened a pull request:
https://github.com/apache/cloudstack/pull/2030
WIP: CLOUDSTACK-9864 cleanup stale worker VMs after job expiry time
You can merge this pull request into a Git repository by running:
$ git pull
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been
kicked to run smoke tests
---
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/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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1879
Packaging result: âcentos6 âcentos7 âdebian. JID-621
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/2026
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been
kicked to run smoke tests
---
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/2026
LGTM (did not test it).
@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
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/2026#discussion_r110098624
--- Diff: setup/db/db/schema-4920to41000.sql ---
@@ -232,3 +232,5 @@ WHERE (o.cpu is null AND o.speed IS NULL AND o.ram_size
IS NULL) AND
--
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/2001
@LGTM on code 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
enabled
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
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/1879
@sureshanaparti 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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/2027
@marcaurele the test makes use of threads to run various clients and server
instances and is known to consume ports for the same, for these reasons it
might fail in some test/build environments.
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/2024
LGTM, @karuturi this is ready for merging.
---
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 abhinandanprateek closed the pull request at:
https://github.com/apache/cloudstack/pull/2029
---
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
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/2024
@rhtyd @karuturi here is the cherry picked PR on 4.9
https://github.com/apache/cloudstack/pull/2029
---
If your project is set up for it, you can reply to this email and have your
GitHub user abhinandanprateek opened a pull request:
https://github.com/apache/cloudstack/pull/2029
CLOUDSTACK-9857: With this change if agent dies the systemd will catcâ¦
â¦h it properly and show process as exited
You can merge this pull request into a Git repository by running:
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1864
@wido no, I'm not working on this presently. Though, I've kept it open as a
reminder that we need to indeed work on updating the ssvm code to Debian8. Part
of the issue is between Debian7 and
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1282
@anshul1886 I've left a comment, the code to check and remove snapshot on
allocated state is already in that method?
---
If your project is set up for it, you can reply to this email and have
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1282#discussion_r110092518
--- Diff:
engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java
---
@@ -245,6 +245,12 @@ public boolean
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1282#discussion_r110092478
--- Diff:
engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java
---
@@ -245,6 +245,12 @@ public boolean
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1951
Ping @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 so,
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1980
@karuturi this is ready for merge, has the screenshots and lgtms
---
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/1944
Thanks @nvazquez @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
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/2022
Thanks @karuturi the failures are not related to the PR.
---
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
Syed,
Can you try this and let me know if it works better?
http://jenkins.openvm.eu/cloudstack/centos/x86_64/centos-7-vmware.ova
I've removed NetworkManager
--
Sent from the Delta quadrant using Borg technology!
Nux!
www.nux.ro
- Original Message -
> From: "Nux!"
>
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1880
---
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 sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1880
The above tests failure are not related to this PR. These are failing most
for the other PRs.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1282
@swill @rhtyd can you 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
enabled
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1944
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1944
merging now
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/2022
@rhtyd can you check on the failures if they are related?
---
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 karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1889
@rashmidixit travis is failing. Can you rebase with master and force push?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
52 matches
Mail list logo