GitHub user harikrishna-patnala opened a pull request:
https://github.com/apache/cloudstack/pull/1990
CLOUDSTACK-9674: Baremetal deployment improvements
With the current implementation of baremetal with cloudstack works but
there are lot of improvements and bug fixes to be done.
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1792
code LGTM.
@nitin-maharana Please initiate internal CI run on @cloudmonger.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1983
@rhtyd I've tried the upgrade from 4.5 and ended up with failing at the
point when upgrading templates ids:
```
2017-03-06 08:53:17,365 DEBUG [c.c.u.d.ScriptRunner] (main:null)
Github user yvsubhash commented on the issue:
https://github.com/apache/cloudstack/pull/1726
@ustcweizhou Please 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 a
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@nvazquez @serg38 Please review 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
Github user harikrishna-patnala commented on the issue:
https://github.com/apache/cloudstack/pull/1880
LGTM for 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
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1726#discussion_r104370668
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2263,27 +2264,43 @@ public void cleanupDownloadUrls(){
// C
Github user sureshanaparti commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1875#discussion_r104373535
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
---
@@ -707,6 +709,23 @@ private String setu
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@wido Can you please update the systemvm templates at
http://cloudstack.apt-get.eu/systemvm ?
You can use the one from
https://builds.cloudstack.org/job/build-master-systemvm/148/
---
If
Github user yvsubhash commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1726#discussion_r104374914
--- Diff: server/src/com/cloud/storage/StorageManagerImpl.java ---
@@ -2263,27 +2264,43 @@ public void cleanupDownloadUrls(){
// Cle
about ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-9821
this seems to be caused by commit 175c8d83b8a628566a4c443db0de587874718c8c
Author: Jayapal
AuthorDate: Mon Feb 20 18:29:14 2017 +0530
Commit: Jayapal
CommitDate: Mon Feb 20 18:34:13 2017 +0530
CLOUDSTACK-8871: fix
Github user priyankparihar commented on the issue:
https://github.com/apache/cloudstack/pull/1813
Hi, @serg38 and @borisstoyanov soon test_00_deploy_vm_root_resize will be
updated.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Thanks Wei Zhow. Will ping Jayapal.
~ Rajani
http://cloudplatform.accelerite.com/
On March 6, 2017 at 3:25 PM, Wei ZHOU (ustcweiz...@gmail.com)
wrote:
about ticket:
https://issues.apache.org/jira/browse/CLOUDSTACK-9821
this seems to be caused by commit
175c8d83b8a628566a4c443db0de587874718c8c
Github user nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1792
@cloudmonger
---
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 nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1792
@karuturi : Currently running. Will post once completes.
---
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 niteshsarda commented on the issue:
https://github.com/apache/cloudstack/pull/1725
I have tested this and **LGTM** for 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 fe
GitHub user jayapalu opened a pull request:
https://github.com/apache/cloudstack/pull/1991
CLOUDSTACK-9821: Fixed issue in deploying vm in basic zone
Fixed issue in deploying vm in basic zone.
There is issue in ipset command with xenserver 6.5. In util.pread2 ipset
and -N is pa
Created the PR.
https://github.com/apache/cloudstack/pull/1991
Thanks,
Jayapal
On Mar 6, 2017, at 3:58 PM, Rajani Karuturi
mailto:raj...@apache.org>> wrote:
Thanks Wei Zhow. Will ping Jayapal.
~ Rajani
http://cloudplatform.accelerite.com/
On March 6, 2017 at 3:25 PM, Wei ZHOU
(ustcweiz...@gm
Github user prashanthvarma commented on the issue:
https://github.com/apache/cloudstack/pull/1981
@borisstoyanov yes, I also did cross check the test failures. Thanks for
the investigation.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ustcweizhou commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1991#discussion_r104393162
--- Diff: scripts/vm/hypervisor/xenserver/vmops ---
@@ -1267,9 +1267,9 @@ def cache_ipset_keyword():
type = getIpsetType()
tmpname
Github user jayapalu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1991#discussion_r104405081
--- Diff: scripts/vm/hypervisor/xenserver/vmops ---
@@ -1267,9 +1267,9 @@ def cache_ipset_keyword():
type = getIpsetType()
tmpname = '
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@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 an
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@borisstoyanov 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
re
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1991
code LGTM. not tested.
---
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 jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1991
I am testing it again. Please wait till I update
---
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 th
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1982
Packaging result: âcentos6 âcentos7 âdebian. JID-562
---
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 d
Github user jayapalu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1991#discussion_r104414236
--- Diff: scripts/vm/hypervisor/xenserver/vmops ---
@@ -1267,9 +1267,9 @@ def cache_ipset_keyword():
type = getIpsetType()
tmpname = '
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1991
Updated the patch. Tested it no issues found.
Please review this 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
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1961
@nvazquez @serg38 I'm happy to merge this change I've run the tests with
the NFS2 trillian branch and it passed. As soon as we get the trillian branch
merged as well we'll have this passin
Sorry for late reply @daan.
My id is sachin931350
On Fri, Mar 3, 2017 at 11:35 AM, Daan Hoogland
wrote:
> On Thu, Mar 2, 2017 at 11:22 PM, Syed Ahmed wrote:
> > I've talked to folks here and I don't think I can give write access to
> you
> > for the Wiki. I will create a google form and lets' u
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1982
I've tested the upgrade and it has passed from 4.5 to 4.10. Will kick
smoketests now
@blueorangutan test
---
If your project is set up for it, you can reply to this email and have you
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@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
On Mon, Mar 6, 2017 at 2:59 PM, sachin patil wrote:
> sachin931350
you have access now
--
Daan
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1832
### ACS CI BVT Run
**Sumarry:**
Build Number 431
Hypervisor xenserver
NetworkType Advanced
Passed=104
Failed=1
Skipped=7
_Link to logs Folder (searc
Hi all,
I am seeing the following issue while trying to dedicate a vlan range to an
account:
(local) 🐵 > dedicate guestvlanrange account=admin
domainid=efd478ff-0145-11e7-ba73-faaca6101800 vlanrange=100-300
physicalnetworkid=7c658a8b-6e24-4a2d-ad58-75f318815dfa
Error 530: Index: 0, Size: 0
cserro
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1961
That's great! Thanks @borisstoyanov for your 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
Github user nathanejohnson commented on the issue:
https://github.com/apache/cloudstack/pull/1955
@borisstoyanov I have reworked the test so that it should now use the host
IP versus the hostname. Could you please re-run the blue orangutan test?
Thank you!
---
If your project is s
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1278
@anshul1886 sorry, but I still do not understand you.
Your changes affect `NetworkHelperImpl.java` class. Do we agree on that?
You changed lines 334 and 512. You change
Github user nathanejohnson commented on the issue:
https://github.com/apache/cloudstack/pull/1955
Also, I just made a commit, I have not squashed it because I wanted to get
some feedback. I noticed that vmware uses details to pass rootDiskController
for the bus type, and there was ev
@daan Thank you!!
On Mon, Mar 6, 2017 at 7:58 PM, Daan Hoogland
wrote:
> On Mon, Mar 6, 2017 at 2:59 PM, sachin patil
> wrote:
> > sachin931350
>
>
> you have access now
>
> --
> Daan
>
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1961
LGTM
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 wish
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@abhinandanprateek I am curious if this PR will allow deploying VM with
Data disk attached?
---
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/1879
LGTM @nvazquez Can we test it with shared networking?
---
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 cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1792
### ACS CI BVT Run
**Sumarry:**
Build Number 432
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/1982
Trillian test result (tid-932)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 27819 seconds
Marvin logs:
https://github.com/blueoranguta
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/914
### ACS CI BVT Run
**Sumarry:**
Build Number 433
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=2
Skipped=7
_Link to logs Folder (search
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1879
Sure @serg38, after enabling HA on cluster, stopping VR and the starting VR
we got this exception on ESXi 6:
2017-03-06 12:18:14,654 ERROR [cloud.vm.VmWorkJobDispatcher]
(Work-Job-
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@serg38 Yes, the aim of the PR is to enable deployment from templates with
multiple data disks.
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1655
### ACS CI BVT Run
**Sumarry:**
Build Number 434
Hypervisor xenserver
NetworkType Advanced
Passed=102
Failed=1
Skipped=7
_Link to logs Folder (searc
Github user nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1792
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 wishe
Github user nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1859
@rajesh-battala : Will update the same in wiki page.
---
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 rajesh-battala commented on the issue:
https://github.com/apache/cloudstack/pull/1859
Thanks.
Once its update please point the same here. So that every one knows it.
Thanks
Rajesh Battala
On Tue, Mar 7, 2017 at 9:27 AM, Nitin Kumar Maharana <
no
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1908
@ProjectMoon Can you please review 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 featur
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1873
Two LGTMs and no test failures from cloudmonger test results.
@ustcweizhou waited for your comments for this, I am putting the mergeready
tag. If you have anything to bring up let me know.
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@kishankavala can you use the templates from
http://cloudstack.apt-get.eu/systemvm/4.10/ ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1991
@borisstoyanov can you start the 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 project does not have th
PRs are ready for the blockers. Waiting for reviews and test
results. Once they are ready, I will merge them(and a few more
bug fixes) and create RC2 (probably tomorrow, Wednesday)
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On March 3, 2017 at 4:30 PM, Rajani Karuturi (raj...@apache.
Github user bvbharatk commented on the issue:
https://github.com/apache/cloudstack/pull/1969
will move the changes to new schema file, when the new master branch is
created after release.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1970
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 if
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@nvazquez @serg38 I tested this fix on VMware vSphere 5.5 and 6.0. I'll
re-verify this on 6.0. Can you please share the build# of ESXi 6.
---
If your project is set up for it, you can r
Github user jayapalu commented on the issue:
https://github.com/apache/cloudstack/pull/1965
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 wishes s
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@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
enable
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@abhinandanprateek 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 you
Github user kishankavala commented on the issue:
https://github.com/apache/cloudstack/pull/1982
@karuturi updated template URLs and md5sum
---
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
I seem to have found another blocker:
https://issues.apache.org/jira/browse/CLOUDSTACK-9822
On 3/6/17, 9:51 PM, "Rajani Karuturi" wrote:
PRs are ready for the blockers. Waiting for reviews and test
results. Once they are ready, I will merge them(and a few more
bug fixes) and create
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1989
Packaging result: âcentos6 âcentos7 âdebian. JID-563
---
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 d
I believe this is another blocker, but probably related to CLOUDSTACK-9822
(which I just opened):
https://issues.apache.org/jira/browse/CLOUDSTACK-9823
On 3/6/17, 11:32 PM, "Tutkowski, Mike" wrote:
I seem to have found another blocker:
https://issues.apache.org/jira/browse/CLOUDST
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@blueorangutan test vmware
---
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/1989
@abhinandanprateek unsupported parameters provided. Supported mgmt server
os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6,
kvm-centos7, kvm-ubuntu, xenserver-65
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@blueorangutan test 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 featu
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@abhinandanprateek unsupported parameters provided. Supported mgmt server
os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6,
kvm-centos7, kvm-ubuntu, xenserver-65
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@abhinandanprateek 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 y
Github user abhinandanprateek commented on the issue:
https://github.com/apache/cloudstack/pull/1989
@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 th
Github user anshul1886 commented on the issue:
https://github.com/apache/cloudstack/pull/1278
@rafaelweingartner, Basically CallContext.current().getCallingAccount(); is
setting the proper context which is later used. If you have observed in code
all the api calls have a similar call
Github user SudharmaJain commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1875#discussion_r104600403
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
---
@@ -707,6 +709,23 @@ private String setupM
Github user nitin-maharana commented on the issue:
https://github.com/apache/cloudstack/pull/1859
Please find the test plan executed from below link(Updated in wiki page).
https://cwiki.apache.org/confluence/display/CLOUDSTACK/NCC+Integration+with+CloudStack+Test+Plan
---
If you
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1991
basic zone is not supported currently with BO @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 do
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1965
@bvbharatk I think the new password should be saved to one of the running
routers, not all running routers.
I disagree with change line 797.
---
If your project is set up for it, you
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1873
@jayapalu LGTM
I have two questions
(1) Is it possible to disable the check ?
(2) Could you please fix CLOUDSTACK-9413 (with this PR or in new PR) ?
---
If your project is
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1970
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
80 matches
Mail list logo
