Do realize that this way you will need to have concensus on removing
it as you will have donated to apache (so in fact you have set the
type of license).
have fun implementing,
Daan
On Tue, Apr 15, 2014 at 7:48 AM, Damoder Reddy damoder.re...@citrix.com wrote:
Ok to start with I will include
On Tue, Apr 15, 2014 at 6:01 AM, Mandar Barve
mandar.ba...@sungardas.com wrote:
1. The pilot patch I posted already does this of checking the command level
flag first and only for flagged commands check the list of sensitive
parameters. I was wondering if that code itself, building list of
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20343/
---
(Updated April 15, 2014, 6:13 a.m.)
Review request for cloudstack.
On April 15, 2014, 5:11 a.m., Rajani Karuturi wrote:
utils/src/com/cloud/utils/servlet/CacheControlUtil.java, line 24
https://reviews.apache.org/r/20343/diff/1/?file=557372#file557372line24
Hi Laszlo,
Is there any reason to disable browser caching for IE?
I think we
On 15/04/14 3:03 AM, Kelven Yang kelven.y...@citrix.com wrote:
I was the original author for AsyncJob management and I wasn¹t aware of
that there is a hook-up to EventBus?
Changes related to below commit gone missing in 4.3, 4.4 and master.
commit 70f33efa7250d1ac8022f4390cddcbcb62ba92f8
Oh all I meant was stuffing the existing search string that cleanString
uses with all identified sensitive keywords. And I guess if it starts
getting GIANT it won't make sense.
On Tue, Apr 15, 2014 at 11:36 AM, Daan Hoogland daan.hoogl...@gmail.comwrote:
On Tue, Apr 15, 2014 at 6:01 AM,
hi,
i'm using new devcloud for 4.3, and configured the basic zone using
devcloud python scripts.
when we launch a the downloaded tiny linux template, it throws the
following exception :
Router requires upgrade. Unable to send command to router
Unable to send command. Upgrade in progress. Please
Ok, so you mean generating an or including all names of annotated
methods/params? or do you mean to include package and class names? It
can not work if you don't. A name maybe used different in different
classes.
On Tue, Apr 15, 2014 at 8:25 AM, Mandar Barve
mandar.ba...@sungardas.com wrote:
Oh
Folks,
I just applied a patch from Tim Mackey:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=748b575af8a66c58b0d52e7457e4d4e1e875231f
commit: 748b575af8a66c58b0d52e7457e4d4e1e875231f
This followed a [PROPOSAL] thread [1]
This was pushed into a separate branch:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20315/#review40359
---
Commit b78aee8f117f9e89a533afba86b3ff91da19744e in cloudstack's
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20313/#review40358
---
Commit d45ee3ef21fd55c8a1cf1373225cab81389ad8c7 in cloudstack's
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20311/#review40357
---
Commit 36103633c2764fe33374f5cbfb1ac0d5bf24ed1b in cloudstack's
On 19.01.2014 18:00, Nux! wrote:
Hi,
Opened https://issues.apache.org/jira/browse/CLOUDSTACK-5907 for this.
There doesn't seem to be any lack of functionality with these
volumes, but it seems wrong to display OVM there instead of KVM and
may cause other issues in the future.
Anyone aware of
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19917/#review40362
---
Ship it!
Ship It!
- daan Hoogland
On April 5, 2014, 3:16 a.m.,
On April 15, 2014, 8:28 a.m., daan Hoogland wrote:
Ship It!
c031eb7d38200d680da85ef57367b21df3483c41
- daan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19917/#review40362
HI Daan,
Thanks a lot for committing it! Sorry for the troubles, and I will do what you
said next time if I upload the patch (thanks for the suggestions)!
Cheers!
Ding
On Apr 15, 2014, at 1:52 AM, Daan Hoogland daan.hoogl...@gmail.com wrote:
it applies,
I won't push it like this though. The
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20365/
---
Review request for cloudstack, Devdeep Singh and Rajesh Battala.
Bugs:
As I said in the previous threaed, I'm +1 on the principle. It will avoid
confusion between straight Xen and XenServer. It will also allow us to offer a
non-XenServer Xen implementation.
Sebastien, as all the filenames have changed, it's not clear from the diff
whether this is just a straight
I use google labels and thread display, the rest is mostly manual. I
check with markmail if the questions are really not answered. There is
a responsiveness report page on the wiki that explains my process and
that I hardly get any input on so please flame me.
On Tue, Apr 15, 2014 at 10:56 AM,
Hi,
I am trying to login in to CS 4.3 though login API.
I am passing MD5 hash (1st) in password that works fine with CS 4.0.2 but
same doesn't works well with CS 4.3. Then I try to pass password in plain
text ( 2nd) it worked, Is this a bug?
Both APIs are given below,
1)
On 15/04/14 3:08 AM, Alena Prokharchyk alena.prokharc...@citrix.com
wrote:
Murali,
I have a question to you. In NetworkOrchestrator, implementNetwork code,
when we update the state for the network, why don¹t we use state machine
for Shared networks, but instead update it directly in the DB?
if
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20311/#review40370
---
Ship it!
Ship It!
- Devdeep Singh
On April 14, 2014, 8:28 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20311/#review40367
---
Commit 0aa6cc60c73698a7be960d2503a5abceaf16d8c7 in cloudstack's
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20313/#review40368
---
Commit 3ec7497e1ebcaa65a8cbd24c27326356355ac8cc in cloudstack's
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20313/#review40371
---
Ship it!
Ship It!
- Devdeep Singh
On April 14, 2014, 8:41 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20315/#review40369
---
Commit 55f53e13e82179c2cbde3f667869495ec944f496 in cloudstack's
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20315/#review40372
---
Ship it!
Ship It!
- Devdeep Singh
On April 15, 2014, 5:12 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20366/
---
Review request for cloudstack, Abhinandan Prateek and daan Hoogland.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20367/
---
Review request for cloudstack and Abhinandan Prateek.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19616/
---
(Updated April 15, 2014, 10:18 a.m.)
Review request for cloudstack and daan
Sanjeev,
Please see answers inline. You may want to also check out
https://cwiki.apache.org/confluence/display/CLOUDSTACK/OVS+Tunnel+Manager+f
or+CloudStack for general limitation etc.
Some of the question are good points that needed to be in FS, I will add
them where ever appropriate.
On
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/236/
--
[...truncated 32989 lines...]
24/24 KB
Downloaded:
http://repo.maven.apache.org/maven2/org/apache/maven/maven-parent/8/maven-parent-8.pom
(24 KB at 444.7 KB/sec)
On Apr 15, 2014, at 4:46 AM, Stephen Turner stephen.tur...@citrix.com wrote:
As I said in the previous threaed, I'm +1 on the principle. It will avoid
confusion between straight Xen and XenServer. It will also allow us to offer
a non-XenServer Xen implementation.
Sebastien, as all the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20250/#review40378
---
Ship it!
Ship It!
- Koushik Das
On April 11, 2014, 2:37 a.m.,
In my crazy head I sometimes wish that we would have a big red button on our
website.
It would be a twilio app that would forward the call 24/7 to members of our
list willing to help out.
A volunteer based 24/7 hot-line for cloudstack !
-sebastien
On Apr 15, 2014, at 6:34 AM, Daan Hoogland
hihi, I think you are stampeding. amok.
On Tue, Apr 15, 2014 at 12:38 PM, Sebastien Goasguen run...@gmail.com wrote:
In my crazy head I sometimes wish that we would have a big red button on our
website.
It would be a twilio app that would forward the call 24/7 to members of our
list
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/237/
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20366/#review40380
---
please test on other platforms to make sure it doesn't break
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/238/
--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20123/
---
(Updated April 15, 2014, 11:50 a.m.)
Review request for cloudstack, Jayapal
On April 14, 2014, 8:58 a.m., Santhosh Edukulla wrote:
systemvm/scripts/utils.sh, line 9
https://reviews.apache.org/r/20123/diff/1/?file=552596#file552596line9
Where is the variable CLOUD_COM_HOME defined?
This is a typo. I have fixed this with the new patch file.
On April 14,
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/239/
--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/240/
--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/241/
--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/242/
--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20123/
---
(Updated April 15, 2014, 12:13 p.m.)
Review request for cloudstack, Jayapal
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/243/
--
Started by user talluri
[EnvInject] - Loading node environment variables.
Building remotely on devcloud-continuous-tests (devcloud) in workspace
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18677/
---
(Updated April 15, 2014, 12:37 p.m.)
Review request for cloudstack, Chiradeep
See http://jenkins.buildacloud.org/job/build-systemvm64-4.2/245/
As long as yours is the first number in the rotation :)
On Apr 15, 2014 6:39 AM, Sebastien Goasguen run...@gmail.com wrote:
In my crazy head I sometimes wish that we would have a big red button on
our website.
It would be a twilio app that would forward the call 24/7 to members of
our list
At the hackathon of CCCNA14 with Sebastien, we made few tests the RTD for
documentation versioning.
Look like we can use branch instead of tag for versioning and we can also
select which branches are published on RTD.org
So, in order to have doc version match product version, would it make sense
Sachchidanand,
It helps to find a dev that has been to the code before and add them
to the reviewers list. I'll have a quick look to see if I can help.
Daan
On Mon, Apr 14, 2014 at 8:19 PM, Sachchidanand Vaidya
vaidy...@juniper.net wrote:
Hi,
Can someone pls review these changes.
Thanks
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19892/#review40386
---
please remove trailing spaces in the json definitions
- daan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20365/#review40388
---
makes sense! the only behavior that needs to be taken into account is
that of any publication scripts that we might write. So it seems to me
this is the best result we have from this version of the hackathon :(
:)
On Tue, Apr 15, 2014 at 2:54 PM, Pierre-Luc Dion pd...@cloudops.com wrote:
At the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19616/#review40390
---
server/src/com/cloud/storage/VolumeApiServiceImpl.java
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19912/#review40393
---
It would seem that this is a behavioral change that would break
Hi All,
Latest 4.2 template built from jenkins.buildacloud.org , is corrupted.
Even though Jenkins job
http://jenkins.buildacloud.org/view/4.2/job/build-systemvm-4.2 complete
successfully, template is not useful; I cannot see any files under
/opt/cloud/bin and SSVM and CPVM never getting
Hi all,
The resource counts for primary and secondary storage listed in GBs, and I
haven't seen these in decimals. They are always in int.
What happens if suppose I register an ISO of smaller size (less than 1 GB)?
Shouldn't it show the resource count in decimals?
I am currently seeing the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20357/#review40394
---
server/src/com/cloud/api/ApiDBUtils.java
To add to what Pierre-Luc said:
Readthedocs has something they call releases but those are in fact builds
that point to a branch. Not a specific tag.
So the release version of the doc we would see on the website will be the live
state of the release branch, not a tag that we could vote on.
On April 15, 2014, 2:52 p.m., daan Hoogland wrote:
server/src/com/cloud/api/ApiDBUtils.java, line 1088
https://reviews.apache.org/r/20357/diff/1/?file=557802#file557802line1088
I don't fully understand the logic of querying .next() and then
.previous()
It seems like
On April 15, 2014, 2:52 p.m., daan Hoogland wrote:
server/src/com/cloud/api/ApiDBUtils.java, line 1088
https://reviews.apache.org/r/20357/diff/1/?file=557802#file557802line1088
I don't fully understand the logic of querying .next() and then
.previous()
It seems like
I think it's better if we make one patch per bug..
On Apr 15, 2014, at 5:59 PM, Tanner Danzey arkan...@gmail.com wrote:
On April 15, 2014, 2:52 p.m., daan Hoogland wrote:
server/src/com/cloud/api/ApiDBUtils.java, line 1088
Thank Chip,
Yes, I was referring to those two modules in my email.
csclient appears to be broken while cloudstack module looks ok.
Cheers,
Madan
On 14 April 2014 at 10:49:08 pm, Chip Childers (chipchild...@apache.org) wrote:
https://nodejsmodules.org/tags/cloudstack
I have not used them
So I think it makes sense for most documents to point to feature
version (e.g. 4.3 branch, 4.4 branch.) E.g. docs for 4.3.1 should be
materially the same as 4.3.0 from a docs standpoint. Release notes are
the exception here, though perhaps they could be dealt with in an
additive way.
--David
On
Done, thank you, Murali
https://issues.apache.org/jira/browse/CLOUDSTACK-6420
On 4/15/14, 2:13 AM, Murali Reddy murali.re...@citrix.com wrote:
On 15/04/14 3:08 AM, Alena Prokharchyk alena.prokharc...@citrix.com
wrote:
Murali,
I have a question to you. In NetworkOrchestrator, implementNetwork
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19584/#review40414
---
Ship it!
Ship It!
- Rajesh Battala
On April 14, 2014, 7:10
I am still running into this issue. Alex reverted complete change
replacing xapi.jar, which was also resulting XmlRpc exceptions but system
VM's were launching fine after revert. I may be wrong but looking at the
stack trace and recent commits, commit
9f44909e63a19b2b8034baa13b92e24ea2eb0227 (use
Punit,
There is a change in the template.
Download it from http://jenkins.buildacloud.org/job/build-systemvm64-master/
You can replace the base system vm template in devcloud and use it.
Thanks
Rajesh Battala
-Original Message-
From: Punith S [mailto:punit...@cloudbyte.com]
Sent:
Sorry - I picked that up but must have been after I'd done the patch. Will
amend _again_!
Alex Hitchins | 07788 423 969 | 01892 523 587
-
-Original Message-
From: daan Hoogland [mailto:nore...@reviews.apache.org] On Behalf Of daan
Hoogland
Hi Murali,
It is caused by the commit, I tested the commit on XS 6.2 FOX, which doesn't
use copy_vhd_from_secondarystorage, that's why I didn't see this issue.
Just disabled XAPI Event in 4.4 branch.
Thanks,
Anthony
-Original Message-
From: Murali Reddy
Sent: Tuesday, April 15,
All,
I am facing an issue when trying to get devcloud2 running with latest code from
4.4 branch. There was a similar email to this a few weeks back on master, but I
did not see any resolution.
I'm running everything in side of devcloud2 (mgmt server, mysql, xen).
Seeing the following errors
Hi Anthony,
Can you go into a bit more depth about copy_vhd_from_secondarystorage not
being used with XenServer 6.2?
Does that mean public Answer copyTemplateToPrimaryStorage(CopyCommand
cmd) in XenServerStorageProcessor is not called when XenServer 6.2 is in
use?
I'm testing out a feature of
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19892/
---
(Updated April 15, 2014, 1:18 p.m.)
Review request for cloudstack.
Changes
On April 15, 2014, 7:23 a.m., daan Hoogland wrote:
It would seem that this is a behavioral change that would break backwards
compatibility, is it? Right now empty service names are accepted. What is
the consequence and why change the behavior, can you shed light on the
subject?
All,
Has anyone run across issues when standing up a new instances with the
simulator on 4.3? Configuration goes fine, but whenever I try to add a new
instance I get the following errors:
ERROR [c.c.v.VirtualMachineManagerImpl] (Job-Executor-1:ctx-8ed90afb
ctx-7310e6f4) Failed to start
Anthony,
Can you remove instead of commenting out code? We do have git.
Op 15 apr. 2014 20:43 schreef anthon...@apache.org:
Repository: cloudstack
Updated Branches:
refs/heads/4.4 b78aee8f1 - 46f6b26d4
disable XAPI event
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
CloudStack changed password encoding in 4.2 (I believe, but I seem to
remember, but can't find documentation that it changed in 4.1) such that it
no longer assumes that passwords are MD5s coming in. It instead expects
that passwords are passed as plain text and are encoded by CS internally
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20387/
---
Review request for cloudstack, Chiradeep Vittal and Rohit Yadav.
Repository:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20389/
---
Review request for cloudstack, Animesh Chaturvedi, Frank Zhang, and Nitin Mehta.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20390/
---
Review request for cloudstack, Chiradeep Vittal and Rohit Yadav.
Repository:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20391/
---
Review request for cloudstack, Brian Federle and Jessica Wang.
Bugs:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20389/#review40468
---
Ship it!
Looks like the code change to get the snapshot from the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20389/
---
(Updated April 15, 2014, 11:02 p.m.)
Review request for cloudstack, Animesh
On April 15, 2014, 10:54 p.m., Prachi Damle wrote:
Looks like the code change to get the snapshot from the correct branch -
4.2 in this case, looks ok. This changes is to be applied on 4.2 only.
Yes, this required only in 4.2
64-bit template script already updated in below check-in
Hi,
Prior to 4.4, if I built a CS cluster with one XenServer host (that was not
in a XenServer resource pool) and then later wanted to add another
XenServer host to the CS cluster, CS would put both hosts in a XenServer
resource pool automatically and then show them both in the CS cluster.
In
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20389/#review40480
---
Commit 0610dbf803d1b1dc37b41b3a6319cfcd94cf6bbe in cloudstack's
This was a little more than a straight renaming, and I'll post all the
changes to the wiki for review in the morning. At a high level this patch
did the following:
- Move the old plugins/hypervisors/xen to /plugins/hypervisors/xenserver
and change all namespaces to reflect the move (bulk of the
Hi,
I have a problem that - at first - I thought was related to the XAPI
changes (although they were reverted and this problem seems to remain).
I am no longer able to create an SR in CitrixResourceBase (getIscsiSR
method) as XmlRpcException is being thrown with the following text:
Failed to
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20357/
---
(Updated April 16, 2014, 1:31 a.m.)
Review request for cloudstack.
Changes
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20357/
---
(Updated April 16, 2014, 1:34 a.m.)
Review request for cloudstack.
Changes
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20357/
---
(Updated April 16, 2014, 3:23 a.m.)
Review request for cloudstack.
Changes
Hi Nux,
The paste links are does not exist.
Can you please upload the logs again. Also upload rules/logs specific to system
rules are not set.
Thanks,
Jayapal
On 11-Apr-2014, at 9:10 PM, Nux! n...@li.nux.ro wrote:
Hello,
I'm on 4.3 right now, CentOS6.5 + KVM and SG ADV zone.
What
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20357/#review40494
---
It seems getHypervisorTypeFromFormat method is only called from
Please take a look.
-- Forwarded message --
From: Sonal Ojha sonal.o...@sungard.com
Date: Tue, Apr 15, 2014 at 6:07 PM
Subject: Re: Review Request 18677: CLOUDSTACK-3272
To: Murali Reddy muralimmre...@gmail.com, Chiradeep Vittal
chirade...@gmail.com
Cc: Sonal Ojha
Folks
FYI I will be out on PTO for 2 weeks starting 4/17
Thanks
Animesh
Just an FYI that this is not only a problem for managed storage (as I
expected it wouldn't be).
If you try to add primary storage to CloudStack for XenServer using an
iSCSI target, it fails. You have to first create the SR from XenCenter (or
xe) and then use the PreSetup protocol to introduce the
In 4.3 you need to set global config router.version.check to false to
workaround this issue in simulator. This is fixed in 4.4/master by commit
38b8677e01fe1ec89df43abe9fac26295597080f.
-Koushik
-Original Message-
From: David Grizzanti [mailto:david.grizza...@sungard.com]
Sent:
On April 16, 2014, 4:02 a.m., Yoshikazu Nojima wrote:
It seems getHypervisorTypeFromFormat method is only called from
VolumeJoinDaoImpl#newVolumeResponse.
It should be fixed, but does it affect snapshot feature?
Yes. Without this fix, if you go into the Storage UI panel the volumes
1 - 100 of 106 matches
Mail list logo
