Github user wido commented on the issue:
https://github.com/apache/cloudstack/pull/1645
Ok, this code is a bit rusty for me. But I think that the qemu-img
converter is started at the back to do the conversion, right?
---
If your project is set up for it, you can reply to this email a
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1650
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 f
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1651
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 fe
Github user rhtyd closed the pull request at:
https://github.com/apache/cloudstack/pull/1648
---
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/1652
LGTM, the Travis test failures are not related to this PR but with tearDown
issues fixed here: https://github.com/apache/cloudstack/pull/1648
---
If your project is set up for it, you can reply t
GitHub user rhtyd reopened a pull request:
https://github.com/apache/cloudstack/pull/1648
test/integration: fix tearDown order in list_acl_ tests
This fixes tearDown issues because of which few Travis tests have failed
such as:
https://travis-ci.org/apache/cloudstack/job
On the web page, the date is updated to Sept 29-30, is this correct? If so,
it should be advertised.
--
Erik
On Tue, Aug 16, 2016 at 5:32 PM, Marco Sinhoreli <
marco.sinhor...@shapeblue.com> wrote:
> All,
>
>
>
> I am very sorry to have to say that we have had to postpone the upcoming
> CCC con
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1641
Ran tests, failure before the fix:
2016-08-22 14:11:09,496 DEBUG [c.c.u.d.ScriptRunner] (main:null) (logid:)
UPDATE `cloud`.`ntwk_offering_service_map` SET Provider='VpcVirtualRouter'
WHE
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1641
LGTM - tested against a client db which initially failed.
---
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 n
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1641
Based on tests and review, merging 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 asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1641
---
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/1620
Trillian test results wrt oobm:
Tests out-of-band management background powerstate sync ... === TestName:
test_oobm_background_powerstate_sync | Status : SUCCESS ===
ok
Tests out-o
GitHub user NuxRo opened a pull request:
https://github.com/apache/cloudstack/pull/1653
Prevent DNS reflection attacks
At least in SG zones DNS on the VR is publicly accessible and as such
susceptible to DNS amplification/reflection attacks. This fixes it as per
https://issues.apac
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1652
@PaulAngus Shouldn't the product code be fixed instead of the test? If the
isDefault property on VPC offerings in not working then that needs to be fixed.
I checked the vpc_offerings table i
Github user murali-reddy commented on the issue:
https://github.com/apache/cloudstack/pull/1620
LGTM on code review, have not tested 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 f
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1620
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1653
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 t
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1651
@PaulAngus Refer to #1563. In that PR it was mentioned that ping response
varies based on OS. So hard coding based on some specific environment is not
the right way to fix this.
---
If you
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1620
Trillian test result (trillian-pr1638-29-xenserver-65sp1-cs410): Test
completed. 41 look ok, 11 have errors
+-+-+---
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1620
Trillian test result (trillian-pr1638-29-xenserver-65sp1-cs410): Test |
Result | Time
--- | --- | ---
test_DeployVmAntiAffinityGroup_in_project | Success | 282.569
test_create_p
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1620
Merging based on review and test results above, thanks @blueorangutan.
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1593
@karuturi a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been
kicked to run smoke tests against packages at
http://packages.shapeblue.com/cloudstack/pr/1593
---
If your pro
Filters on action are already used for vm's.
See vmActionfilter in instances.js.
Also for the sections I can find a sectionPreFilter in cloudStack.js
That seems to check the response of listCapabilities for
'allowusercreateprojects'.
Kind Regards,
Frank
*Frank Maximus *
Senior Software Develop
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1620
---
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/1621
@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 so
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1621
@rhtyd a Trillian-Jenkins test job (centos6 mgmt + vmware-55u3) has been
kicked to run smoke tests against packages at
http://packages.shapeblue.com/cloudstack/pr/1621
---
If your projec
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
LGTM, we need to also test for users who connect to VPN on that network
(will DNS work for them as well?).
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1648
The specific tests run with simulator only, so travis test results should
be acceptable.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1648
@rhtyd could you please amend the commit message to explain the defect
being fixed? It's hard to review the code without understanding the scope of
the problem being addressed.
---
If your p
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1648
LGTM based on code review, the order of cleanup is fixed so that the
resources are released in right order.
---
If your project is set up for it, you can reply to this email and have
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1648
@jburwell thanks fixed.
---
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 NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1653
I've tested this in Adv zone with Security Groups and it works. Can't vouch
for "pure" Adv zones. Perhaps @ustcweizhou can say more.
---
If your project is set up for it, you can reply to this em
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1649
This change is limited to the migration script and does not impact
CloudStack. Manual tests confirmed the fix works with both new and old
commands.properties.
---
If your project is set up for i
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1649
Test run:
python scripts/util/migrate-dynamicroles.py -d -f commands.properties
Apache CloudStack Role Permission Migration Tool
(c) Apache CloudStack Authors and the ASF, under the
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1649
---
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/1653
@NuxRo if you can change the base-branch of the PR to 4.9, I can initiate
some tests on VR/trillian.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1647
@wido with this change will we have to host two versions of deb repository
-- one built on ubuntu without systemd (such as 14.04) and one built on ubuntu
with systemd (16.04+)?
---
If your proj
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1634
@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 so
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1634
@rhtyd a Trillian-Jenkins test job (centos6 mgmt + vmware-55u3) has been
kicked to run smoke tests against packages at
http://packages.shapeblue.com/cloudstack/pr/1634
---
If your projec
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1643
@abhinandanprateek can you reopen/edit the PR for 4.9? thanks.
LGTM. /cc @jburwell @karuturi
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Rohit, done, go right ahead.
---
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/1630
Manually tested, LGTM. We can merge this, the failing Travis tests are not
related to the change (which is purely UI).
---
If your project is set up for it, you can reply to this email and have y
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Thanks @NuxRo can you rebase against latest 4.9, looks like some more
commits came in.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1652
Without full understanding of the Nuage plugin, fixing the test seemed the
safest path, one hopes that the nuage offering was configured that way for a
good reason.
---
If your project is se
Github user NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Done?
---
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 fe
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo there are still several commits which have come from master; can you
export the commit (git format-patch -1) and reset --hard origin/4.9 and
re-apply the commit (git am ) and then git push
Github user NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Rohit unfortunately I do not have git setup properly, I am limited to what
the github interface offers me.
---
If your project is set up for it, you can reply to this email and have your
reply ap
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo okay, can you grant me push access on your fork:
https://github.com/NuxRo/cloudstack/tree/patch-4 and I can help fix it for you?
---
If your project is set up for it, you can reply to this
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@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 rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo thanks, fixed the branch, you may remove my access now :)
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1651
The best way to fix this would be a ping function that gets called.
However, that's not something that i can personally do and this will work
for (at least) Ubuntu, CentOS/RHEL and macchin
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1653
Job ID-94
---
If your project is set up for it, you can reply to this email an
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@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/1653
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs56sp1, centos7 mgmt
+ vmware55u3, ubuntu mgmt + kvmcentos7) has been kicked to run smoke tests
against packages at http://packages.sh
I believe that is the old date. It should probably be updated to TBD.
On Aug 22, 2016, at 2:39 AM, Erik Weber
mailto:terbol...@gmail.com>> wrote:
On the web page, the date is updated to Sept 29-30, is this correct? If so, it
should be advertised.
--
Erik
On Tue, Aug 16, 2016 at 5:32 PM, Marco
Github user nathanejohnson commented on the issue:
https://github.com/apache/cloudstack/pull/1645
@wido that seems to be the case, yes.
https://github.com/myENA/cloudstack/blob/44220a41c119020404c55e7a82a808ddee828116/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storag
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1651
Debian returns: 1 packets transmitted, 1 packets received, 0% packet loss
CentOs returns: 1 packets transmitted, 1 received, 0% packet loss, time 4ms
Between those 2 we could be simpl
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1646
@rhtyd could you please squash the commits?
---
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 jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1653#discussion_r75690054
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -452,9 +452,9 @@ def fw_vpcrouter(self):
self.fw.append(
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo is it possible to create/update a Marvin test case to verify this
change?
---
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 NuxRo commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@jburwell I am afraid I do not have the coding skills to do what you want
re that variable, I am just submitting @ustcweizhou's code from that Jira issue.
Right now basic/sg zones are vuln
Hi, I am unable to compile latest source due to the following error, please can
someone help me with this?
[ERROR] Failed to execute goal com.mycila:license-maven-plugin:2.11:check
(cloudstack-checklicence) on project cloud-plugin-network-nvp: Execution
cloudstack-checklicence of goal com.mycil
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo understood. I am working to get you some assistance. That work for
you?
---
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 fmaximus commented on the issue:
https://github.com/apache/cloudstack/pull/1652
As Nuage plugin developer, I can't come up with any good reason why it
should be default.
Also I see that Redundant VPC Offering is set to be a default.
Both are most likely just copied
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo np, I'll help with the refactoring fixes etc.
---
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 t
Github user rhtyd commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1653#discussion_r75715632
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -452,9 +452,9 @@ def fw_vpcrouter(self):
self.fw.append(
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1653
@NuxRo can you give me push permission on your repository once again to
push the refactoring change, thanks.
---
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/1646
@jburwell I've kept the changes into two commits (1) for pom.xml related
version changes and (2) for db upgrade path, as when doing fwd-merging this
will cause conflict with other pom files so on
Hi B, how are you building CloudStack? Which branch is it?
Can you try a full clean build: mvn clean install -Dnoredist -Dsimulator -P
developer,systemvm
You may install nonoss dependencies from here:
https://github.com/rhtyd/cloudstack-nonoss
Regards.
From:
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1651
that works for me. I'll adjust 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 have this featur
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1634
Trillian test result (trillian-pr1634-33-vmware-55u3-cs49):
Test completed. 39 look ok, 13 have errors
Test | Result | Time
--- | --- | ---
test_DeployVmAntiAffinityGroup_in
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Trillian test result (trillian-pr1653-34-xenserver-65sp1-cs49):
Test completed. 41 look ok, 11 have errors
Test | Result | Time
--- | --- | ---
test_DeployVmAntiAffinityGrou
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1652
Thanks frank, would you make that change to the Nuage Network offering or
shall I?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1621
Trillian test result (trillian-pr1621-31-vmware-55u3-cs49):
Test completed. 40 look ok, 12 have errors
Test | Result | Time
--- | --- | ---
test_DeployVmAntiAffinityGroup_in
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1653
Trillian test result (trillian-pr1653-36-vmware-55u3-cs49):
Test completed. 32 look ok, 20 have errors
Test | Result | Time
--- | --- | ---
test_DeployVmAntiAffinityGroup_in
Github user serg38 commented on the pull request:
https://github.com/apache/cloudstack/commit/db854a35790c77099e9f82a4e2dd77edd5be3eed#commitcomment-18732552
Don't you need to put a space in from of 0 to exclude e.g 100% packet loss ?
---
If your project is set up for it, you can rep
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1593
@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 s
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1640#discussion_r75758946
--- Diff: server/src/com/cloud/ha/HighAvailabilityManagerImpl.java ---
@@ -497,7 +498,7 @@ protected Long restart(final HaWorkVO work) {
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1640
@marcaurele I agree that an ``UNKNOWN`` state is proper way to handle a
network partition. As an example, out-of-band management uses this approach
which the management server loses connectivi
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1640
@marcaurele per @koushik-das, what is the issue with re-pointing this PR to
the 4.9 release branch? When the PR is merged, it will be forward merged to
master. Therefore, the concern you expr
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1634
@rhtyd the test results from blueorangutan are looking pretty good. A
couple of observations:
1. The markdown for the table seems to be invalid as it is not rendering to
a table
2
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1642
@mike-tutkowski determining the required tests to run for a PR is at the
top of my TODO list. For now, I would say running all smoke tests and
associated component tests on hardware would be
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1599
@rhtyd thanks for the expanded commit message. The change makes complete
sense to me now. LGTM for code.
---
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/1647
@wido While I understand that it is quite old and presents maintenance
challenges, it remains supported until [April
2017](https://wiki.ubuntu.com/Releases). Therefore, it would be reasonable
All,
PR 1647 [1] proposes dropping support for Ubuntu 12.04 from 4.9.2.0+. The
primary motivation for its removal is that the age of its libvirt and qemu
versions greatly complicate maintenance of the KVM integration. However,
Ubuntu 12.04 will be supported until April 2017 [2]. What would be
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1635
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1635
Job ID-95
---
If your project is set up for it, you can reply to this email an
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1643
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1643
Job ID-96
---
If your project is set up for it, you can reply to this email an
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1645
Packaging result: âcentos6 âcentos7 âdebian repo:
http://packages.shapeblue.com/cloudstack/pr/1645
Job ID-97
---
If your project is set up for it, you can reply to this email an
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1648
@rhtyd thanks for the clarification of the commit message. However, the
Travis failed for the commit. Could you please investigate and fix?
---
If your project is set up for it, you can repl
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1653#discussion_r75794286
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -452,9 +452,9 @@ def fw_vpcrouter(self):
self.fw.append(
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1646
@rhtyd I understand the reasoning for two commits, and it makes sense.
Since you understand the process required for the forward merge, I think it
makes sense for you to perform it.
L
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1593#discussion_r75795653
--- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java ---
@@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase
checkCase) throws SQ
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1593
@nvazquez could you please squash your commits?
---
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 thi
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1605#discussion_r75795908
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -2042,49 +2043,46 @@ protected void
postVideo
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1605#discussion_r75796008
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -2042,49 +2043,46 @@ protected void
postVideo
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1605
@nvazquez could you please create Marvin test case to verify the use case
in the PR description?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user serg38 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1605#discussion_r75796954
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -2042,49 +2043,46 @@ protected void
postVideoCa
Github user jburwell commented on the issue:
https://github.com/apache/cloudstack/pull/1650
@abhinandanprateek are these two test suites where we should be using
``vm.instance`` config variable instead specifying the template directly in the
test suite?
---
If your project is set up
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1605
@jburwell The support for passing vGPU parameters is already implemented
via PR1310 which has a test case. This PR is only performance optimization by
reducing an extra reconfgtask which save a
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1650
We tried macchinina template during test cases on ESX with E1000 adapter
and VMs doesn't come with eth0 causing test_internal_lb.py and test_vpc_vpn.py
to fail on a real hypervisor test. Has anyo
1 - 100 of 118 matches
Mail list logo
