Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1727#discussion_r84839712
--- Diff:
engine/storage/snapshot/src/org/apache/cloudstack/storage/vmsnapshot/DefaultVMSnapshotStrategy.java
---
@@ -128,7 +128,7 @@ public VMSnaps
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1727#discussion_r84839559
--- Diff: engine/schema/src/com/cloud/vm/snapshot/VMSnapshotVO.java ---
@@ -248,4 +252,9 @@ public void setRemoved(Date removed) {
public Cla
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1727#discussion_r84841375
--- Diff: server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java ---
@@ -639,10 +648,57 @@ else if (jobResult instanceof Throwable)
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1727#discussion_r84839190
--- Diff: core/src/com/cloud/agent/api/VMSnapshotTO.java ---
@@ -35,6 +35,7 @@
private VMSnapshotTO parent;
private List volumes;
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1727#discussion_r84841681
--- Diff: server/src/com/cloud/vm/snapshot/VMSnapshotManagerImpl.java ---
@@ -610,7 +615,11 @@ public UserVm revertToSnapshot(Long vmSnapshotId)
thro
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1727#discussion_r84839433
--- Diff: core/src/com/cloud/agent/api/VMSnapshotTO.java ---
@@ -123,4 +125,12 @@ public boolean getQuiescevm() {
public void setQuiescevm(bo
Github user murali-reddy closed the pull request at:
https://github.com/apache/cloudstack/pull/1724
---
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 feat
GitHub user murali-reddy reopened a pull request:
https://github.com/apache/cloudstack/pull/1724
CLOUDSTACK-9511: fix test_privategw_acl.py to handle multiple physical
network
fix to ensure only physical network with guest traffic is picked up for
creating a private network
Github user yvsubhash commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1726#discussion_r84859378
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2199,15 +2199,20 @@ public void cleanupDownloadUrls(){
if(do
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1726#discussion_r84862649
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2199,15 +2199,20 @@ public void cleanupDownloadUrls(){
if(
Github user yvsubhash commented on the issue:
https://github.com/apache/cloudstack/pull/1726
@koushik-das Added the null check
---
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
ena
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1724
I saw there seems to be an issue with the fwd-merge, I'll fix that
@murali-reddy for now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1724
---
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 is
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1615
@karuturi there was a capacity issue at the backend, the tests have been
re-fired.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1451
Code LGTM.
BVTs are good(iso failures are URL access issues and not related to this
PR).
merging this now
---
If your project is set up for it, you can reply to this email an
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1451
---
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 is
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1729
CLOUDSTACK-9564: Fix memory leaks in VmwareContextPool
In a recent management server crash, it was found that the largest
contributor
to memory leak was in VmwareContextPool where a registry
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@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 appe
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1451
### ACS CI BVT Run
**Sumarry:**
Build Number 121
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=2
Skipped=5
_Link to logs Folder (searc
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@rhtyd I understand these words: "help", "hello", "thanks", "package",
"test"
Test run usage: test (mgmt server, one of: centos6, centos7, ubuntu)
(hypervisor, one of: kvm-centos6, kvm
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
Packaging result: âcentos6 âcentos7 âdebian. JID-94
---
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 do
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@blueorangutan test centos6 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
e
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@rhtyd a Trillian-Jenkins test job (centos6 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 ap
GitHub user marcaurele opened a pull request:
https://github.com/apache/cloudstack/pull/1730
VMTemplateZone needs some love
It's incorrect to use the findIncludingRemovedBy and
listIncludingRemovedBy for the common list and find operation. Moreover
the DAO is trying to update
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1642
Trillian test result (tid-178)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 25671 seconds
Marvin logs:
https://github.com/blueoranguta
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1729
CLOUDSTACK-9564: Fix memory leaks in VmwareContextPool
In a recent management server crash, it was found that the largest
contributor
to memory leak was in VmwareContextPool where a registr
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1729
---
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 is
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1726
Trillian test result (tid-179)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24929 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1728
Trillian test result (tid-180)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26449 seconds
Marvin logs:
https://github.com/blueoranguta
Github user syed commented on the issue:
https://github.com/apache/cloudstack/pull/1711
@PaulAngus would it be possible to run another round of tests 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 proje
@rohit
Can you set a BlueO job off from this please?
Kind regards,
Paul Angus
paul.an...@shapeblue.com
www.shapeblue.com
53 Chandos Place, Covent Garden, London WC2N 4HSUK
@shapeblue
-Original Message-
From: syed [mailto:g...@git.apache.org]
Sent: 25 October 2016 17:27
To: dev
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@rhtyd @murali-reddy @PaulAngus @jburwell Thanks for fixing all outstanding
Marvin test issues. We have now 100% passing score on Marvin integration tests
on RHEL 6 management server , advanced n
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1677
LGTM for testing
```
2016-10-24 20:52:55 47 test files ro run
2016-10-24 20:52:55 Running test file:
test/integration/smoke/test_affinity_groups_projects.py
2016-10-24 20:56:07 Run
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1642
@rhtyd @karuturi I think we are good to go on this PR now. Do you agree?
---
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 NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1700
Wido,
Will do, but need to wait until next week to get some help re getting IPv6
on my test rig. As soon as I have feedback, I'll post here.
Lucian
--
Sent from the D
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@jburwell a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1677
@blueorangutan test centos7 vmware55u3
---
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/1729
Trillian test result (tid-181)
Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 6
Total time taken: 39964 seconds
Marvin logs:
https://github.com/blueoranguta
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
Trillian test result (tid-182)
Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26183 seconds
Marvin logs:
https://github.com/blueoranguta
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1711
@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 wi
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1711
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt
+ vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests
---
If your project is set up for it
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@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 appe
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
Packaging result: âcentos6 âcentos7 âdebian. JID-95
---
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 do
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@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
e
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1729
@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 ap
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1642
yes. I am merging this 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 an
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1642
LGTM. The one failing test is due to environment issue, I've fixed it and
will send a PR shortly.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1615
@rhtyd any update on the 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
enable
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1642
---
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 is
GitHub user rhtyd opened a pull request:
https://github.com/apache/cloudstack/pull/1731
CLOUDSTACK-9565: Fix intermittent failure in test_oobm_zchange_password
Fixes intermittent integration smoke test failures caused in
test_oobm_zchange_password test.
The scope is limi
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1615
@karuturi tests are about to finish
---
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
enabl
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1716
@yvsubhash Can you check why travis failed? (force push to re run it if its
a false alarm)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1711#discussion_r85050747
--- Diff: setup/db/db/schema-4910to41000.sql ---
@@ -35,4 +35,192 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,
hypervisor_type, hypervi
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1711
One small comment, rest of the 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 f
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1726
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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1701
Is this good to merge, purely UI change? /cc @murali-reddy
@abhinandanprateek @karuturi
---
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/1712
This should be good to merge, purely UI change? /cc @murali-reddy
@abhinandanprateek @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1615
ok. Thank you.
---
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 mike-tutkowski opened a pull request:
https://github.com/apache/cloudstack/pull/1732
Switched to the official SolidFire SDK for Java
SolidFire has recently released an official SDK for Java.
I have converted the SolidFire storage plug-ins to make use of this new
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1732
TestAddRemoveHosts.py
test_add_remove_host_with_solidfire_plugin_1
(TestAddRemoveHosts.TestAddRemoveHosts) ... === TestName:
test_add_remove_host_with_solidfire_plugin_1 | Status
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1732
TestSnapshots.py: The failure in test 04 makes sense as that test case is
actually dependent upon #1600, which has not yet been merged in.
test_01_create_volume_snapshot_using_sf_
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1732
TestVMSnapshots.py
test_01_take_VM_snapshot (TestVMSnapshots.TestVMSnapshots) ... ===
TestName: test_01_take_VM_snapshot | Status : SUCCESS ===
ok
test_02_take_VM_snapshot
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1732
TestVMMigrationWithStorage
test_01_storage_migrate_root_and_data_disks
(TestVMMigrationWithStorage.TestVMMigrationWithStorage) ... === TestName:
test_01_storage_migrate_root_and_
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1732
TestVolumes (resigning)
test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName:
test_00_check_template_cache | Status : SUCCESS ===
ok
Attach a volume to
Github user mike-tutkowski commented on the issue:
https://github.com/apache/cloudstack/pull/1732
TestVolumes.py (no resigning)
test_00_check_template_cache (TestVolumes.TestVolumes) ... === TestName:
test_00_check_template_cache | Status : SUCCESS ===
ok
Attach a volu
Github user romain-dartigues commented on the issue:
https://github.com/apache/cloudstack/pull/1604
@rhtyd < JIRA issue
[#9568](https://issues.apache.org/jira/browse/CLOUDSTACK-9568)
I'll rebase as soon as possible.
---
If your project is set up for it, you can reply to this
68 matches
Mail list logo
