> -Original Message-
> From: Prasanna Santhanam [mailto:t...@apache.org]
> Sent: Tuesday, July 23, 2013 9:32 PM
> To: dev@cloudstack.apache.org
> Subject: Re: New Components on JIRA
>
> On Tue, Jul 23, 2013 at 05:23:03PM +, Ram Ganesh wrote:
> > > -Original Message-
> > > Fro
Folks
We are now just 1 week from ACS 4.2 code freeze on 7/29. For the past 2 - 3
week there has been vibrant activity in filing and fixing issues. We have
managed to contain open defects to less than 400 but blockers and critical are
still in 90+. For the days remaining to code freeze, le
Hi all,
I've been working on re-factoring the dedication UI code in these
days and just finished.
I discussed the requirements with Pranav before. The changes are listed as
following:
1. zone wizard: (JIRA ticket :
https://issues.apache.org/jira/browse/CLOUDSTACK-3700)
Add a dedicati
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12886/#review23745
---
I think you should increase the number of expunge.workers in your de
Not there in 4.2?
-Original Message-
From: Isaac Chiang [mailto:isaacchi...@gmail.com]
Sent: Wednesday, July 24, 2013 3:12 PM
To: Pranav Saxena
Cc: dev@cloudstack.apache.org
Subject: Re: Git Push Summary
Hi all,
I've been working on re-factoring the dedication UI code in these d
Thanks Isaac for making the relevant code changes there. I think you
probably committed it only to master . You would need to cherry-pick it up
to asf/4.2 as well . Otherwise the changes you have made look good and in
accordance with the workflow we discussed .
Regards,
Pranav
On Wed, Jul 24, 2
Hi Chiradeep,
Before I explore further, can I ask where "StartupSecondaryStorageCommand"
mentioned in the defect https://issues.apache.org/jira/browse/CLOUDSTACK-3377
gets used?
DL
> -Original Message-
> From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com]
> Sent: 24 July 2013
> -Original Message-
> From: Prasanna Santhanam [mailto:t...@apache.org]
> Sent: 24 July 2013 10:02
> To: dev@cloudstack.apache.org
> Subject: Re: New Components on JIRA
>
> On Tue, Jul 23, 2013 at 05:23:03PM +, Ram Ganesh wrote:
> > > -Original Message-
> > > From: Prasanna Sa
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12896/
---
Review request for cloudstack, Abhinandan Prateek and Sebastien Goasguen.
Repos
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12810/
---
(Updated July 24, 2013, 10:28 a.m.)
Review request for cloudstack, Alena Prokha
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12896/#review23746
---
Review 12896 failed the build test : FAILURE
The url of build clouds
Thanks Frank and John,
I've been strugling with the enums in Networks.java. A discussion on those
is needed I think. BroadcastDomainType seems to be a hybrid of a stable and
a plugin extendible enum. It breaks my mind and my every change.
regards,
Daan
On Tue, Jul 23, 2013 at 8:37 PM, Frank Zha
Hi,
I uploaded a patch to review board for a specific branch, ldapplugin.
Jenkins attempted to take the patch and apply it to master, resulting
in a failed result.
https://reviews.apache.org/r/12896/
http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/29/console
there has been some discussion on the proper use of enums recently.
I've been strugling with the enums in Networks.java. A discussion on those
is needed I think. BroadcastDomainType seems to be a hybrid of a stable and
a plugin extendible enum. It breaks my mind and my every change. There are
more
> On July 24, 2013, 10:30 a.m., Jenkins Cloudstack.org wrote:
> > Review 12896 failed the build test : FAILURE
> > The url of build cloudstack-master-with-patch #29 is :
> > http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/29/
Jenkins attempted to patch master... patch is for ldap
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12897/
---
Review request for cloudstack and Prasanna Santhanam.
Bugs: CLOUDSTACK-3594
R
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12811/
---
(Updated July 24, 2013, 10:53 a.m.)
Review request for cloudstack, Alena Prokha
I had an offline conversation with Chiradeep to sort this out.
Provided the latest SystemVM code is built, there's no difficulty. E.g. using
"mvn clean install -P developer,systemvm"
See
https://cwiki.apache.org/confluence/display/CLOUDSTACK/How+to+build+on+master+branch
That said, our instr
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12858/#review23749
---
Commit 2b8f1bde8f8e3c399baf44695c10866798e1fe45 in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12897/#review23748
---
Commit b8a38ca7bd866db32dba644f8fc1be5d3406e16c in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12897/#review23751
---
Commit 4a7f5d59d6f77fdeb5623937f4efa2542f891573 in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12893/#review23750
---
Commit 8e30ee3f1a6de50c7ae6599d62e53c291d443103 in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12893/#review23753
---
Commit 605a31eb70e8998b4476ffadf2514c969b83f7d7 in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12858/#review23752
---
Commit 74ac8575b09427c7511272bdd6592df092b55112 in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12897/#review23754
---
Review 12897 PASSED the build test
The url of build cloudstack-maste
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12809/
---
(Updated July 24, 2013, 11:43 a.m.)
Review request for cloudstack, Abhinandan P
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12899/
---
Review request for cloudstack, SrikanteswaraRao Talluri and Prasanna Santhanam.
+1
So are BroadcastScheme, IsolationType, TrafficType.
--Alex
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Wednesday, July 24, 2013 3:38 AM
> To: dev
> Subject: [DISCUSS} enums in Networks,.java
>
> there has been some discussion on the proper use
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12898/
---
Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
Bugs: CLOU
Parnav, Jessica or Brian, can you help Ian here ?
thx
On Jul 22, 2013, at 9:51 AM, Ian Duffy wrote:
> Hi,
>
> I'm having an issue with the below gisted snippets. For whatever
> reason they do not get validated. Any idea why?
>
> https://gist.github.com/imduffy15/6053967
>
> Thanks,
> Ian
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12899/#review23755
---
Review 12899 PASSED the build test
The url of build cloudstack-maste
Daan and Frank,
First, I think its important to separate when and where an enumeration should
be used from how to best employ enums when their use is appropriate. Enums are
a finite set of values defined at compile-time. Therefore, they shouldn't
employed in situations where extensibility of
Sorry for missing this thread .Seems like the mail was automatically sent
to my citrix ID which doesn't exist anymore. Anyways, I gave a quick look
at what Ian is trying to do here and realized that he has probably
forgotten to add the "required" class in the form structure . Ian , you
might want t
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12811/#review23756
---
Commit 2b9d9cf0360b22a703f27600292f22711f51c62f in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12852/#review23757
---
Commit 135a2e66c565044b6597e9178c3cf3d544712cca in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12852/#review23760
---
Ship it!
commit d89115b5cff9013c917c8d2e984c596698389cfe
- Kishan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12811/#review23758
---
Commit 743d35cae4985a8d898da06e5768ae282b32adb2 in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12852/#review23759
---
Commit d89115b5cff9013c917c8d2e984c596698389cfe in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12811/#review23761
---
Ship it!
commit 743d35cae4985a8d898da06e5768ae282b32adb2
- Kishan
Hi Pranav,
The "Required: true" in the creates that span required field
automatically. The following is the result of its generated html:
(Just using account as an example, account isn't required in practice)
*Account:
But it still allows the f
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12898/#review23762
---
Review 12898 PASSED the build test
The url of build cloudstack-maste
And as an example: the way the net should be isolated or the broadcast
should be constructed are typically behavioral issues, whilst the fact that
it is being used for storage or private network or routing can be caught in
a constant/enum value. right?
On Wed, Jul 24, 2013 at 2:37 PM, John Burwel
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12896/#review23763
---
Ship it!
Ship It!
- Abhinandan Prateek
On July 24, 2013, 10:26 a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12775/#review23764
---
Ship it!
So it took me some time to get around reviewing this, but
Wei.
Yes, I recreated the systemvms by destroying them. I have no agents, if you
mean KVM/Xen agents, I'm running vSphere.
Regards
Ilya
> -Original Message-
> From: Wei ZHOU [mailto:ustcweiz...@gmail.com]
> Sent: Wednesday, July 24, 2013 2:37 AM
> To: dev@cloudstack.apache.org
> Subject
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12897/#review23765
---
Ship it!
Ship It!
- Prasanna Santhanam
On July 24, 2013, 10:51 a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12874/#review23766
---
I think you forgot to attach the diff here.
- Prasanna Santhanam
On Tue, Jul 23, 2013 at 06:12:31PM +, Mathias Mullins wrote:
> Prasanna,
>
> If we broke these tests down into smaller test specific test cases, could
> we automate them into one specific end-to-end test together?
I think this is called 'sequencing'. You have a bunch of individual
operations
Hi,
I reverted this commit in the 4.1 branch:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=a069f31f51d0ef1f255d85ef32d7649a5ec659c1
There has been no discussion about changing this API in the 4.1 series,
I even think we can't simply do that for 4.2 either.
You CAN
Noah,
Did you ever review / report / re-vote this?
Thanks,
Matt
On 6/25/13 11:17 AM, "Noah Slater" wrote:
>Thanks for the feedback, Matt.
>
>Anyone else got any feedback on this? Might cut a new vote.
>
>
>On 24 June 2013 05:12, Mathias Mullins wrote:
>
>> Noah,
>>
>> I agree that there ne
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12853/#review23771
---
There is no bug in current Java code, and the java part of this revi
RB doesn’t accept the patch. Keeps failing. Here's the patch:
From 02238e6903d3d399d402886513efd6cd2664696a Mon Sep 17 00:00:00 2001
From: Sowmya Krishnan
Date: Tue, 23 Jul 2013 22:29:10 +0530
Subject: [PATCH] Add wait while Netscaler recognizes service to be up
---
test/integration/component/t
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12893/
---
(Updated July 24, 2013, 5:12 p.m.)
Review request for cloudstack and Prasanna S
Yes, definitely shouldn't be a 4.1.x change. I don't think we want people
to have to go through major pains on a minor point release unless there's
some major showstopper bug.
4.2, I don't know. I was always under the impression that going from 4.x to
4.y WAS a major release, per the versioning st
Thank you all guys!
If I am understanding this correctly, using "mvn clean install -P
developer,systemvm" will build the latest management server and systemvm
code.
This latest management server is using fully qualified class name which
requires the latest agent and system vm template to work
> On July 24, 2013, 5:02 p.m., Alena Prokharchyk wrote:
> > There is no bug in current Java code, and the java part of this review
> > ticket shouldn't be checked in.
> >
> >
> > In the method cleanupDomain(), we try to remove every account before
> > checking if accounts marked for cleanup,
Nope. Sorry. Feel free to run with it. If not, I can see about doing
another vote in the next few days.
On 24 July 2013 18:02, Mathias Mullins wrote:
> Noah,
>
> Did you ever review / report / re-vote this?
>
> Thanks,
> Matt
>
>
> On 6/25/13 11:17 AM, "Noah Slater" wrote:
>
> >Thanks for the
Daan,
Sorry for the late reply. Now, it kinda goes into code/design philosophy.
I'll tell you what mine is.
There's one school of thought that code should cover all bases. To me having
undecided is in that school because it asks "could it be undecided" and the
answer is of course yes becaus
You can look at the code yourself and see if it's any different than what we
copied.
http://grepcode.com/file/repo1.maven.org/maven2/org.kuali.rice/rice-impl/1.0.3.1-BX/org/apache/commons/httpclient/contrib/ssl/EasySSLProtocolSocketFactory.java
--Alex
> -Original Message-
> From: Amogh
Hi All,
I've created a 4.1.1 release, with the following artifacts up for a vote:
Git Branch and Commit SH:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.1
Commit: 8fe3505cba45756a51e9e9ee97cd09bf1e71c79e
Source release (checksums and signatures are available
It's only two files and they were not that huge if I recall correctly from my
quick glance.
--Alex
> -Original Message-
> From: Alex Huang
> Sent: Wednesday, July 24, 2013 10:22 AM
> To: dev@cloudstack.apache.org
> Subject: RE: Inclusion of commons-httpclient-contrib jar
>
> You can loo
For now I think cloud-utils is the best option.
> -Original Message-
> From: Amogh Vasekar [mailto:amogh.vase...@citrix.com]
> Sent: Wednesday, July 24, 2013 10:48 PM
> To: dev@cloudstack.apache.org
> Subject: Re: Inclusion of commons-httpclient-contrib jar
>
> Hi Alex,
>
> Thanks! I had
Hi Alex,
Thanks! I had looked at that initially, but found this disclaimer on the
source website :
"Warning:
not-yet-commons-ssl-0.3.11 should be considered to be of "Alpha" quality!"
I do not see a code update in it for the past 4 years, and thus was
skeptical.
Please let me know if we should g
So I'm not even a committer yet, but this is an idea on how I think I
would want to be voted in.
For Committer - 2/3 Lazy
This makes sure that at least 2 people basically nominated, and seconded
and the votes were 2:1 in favor of the person coming in.
For PMC - 3/4 Lazy
This is the leadership of
I'm not a committer so I don't want really to take this one on the Bylaws
side. :-)
Matt
On 7/24/13 1:19 PM, "Noah Slater" wrote:
>Nope. Sorry. Feel free to run with it. If not, I can see about doing
>another vote in the next few days.
>
>
>On 24 July 2013 18:02, Mathias Mullins wrote:
>
>>
Can you rebase your branch and try to create a new patch; file that you
modified might have got updated.
--Sanjay
> -Original Message-
> From: Sowmya Krishnan [mailto:sowmya.krish...@citrix.com]
> Sent: Wednesday, July 24, 2013 10:37 PM
> To: dev@cloudstack.apache.org; Prasanna Santhanam
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12893/#review23775
---
Commit cd51303f2086fc1fe6d21495c9514ef59450ea0d in branch refs/heads
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12893/#review23776
---
Commit 5009d3b82af4fa385490517185d071e1326cffaa in branch refs/heads
Actually I just applied it. So it's okay.
On Wed, Jul 24, 2013 at 05:34:27PM +, Sanjay Tripathi wrote:
> Can you rebase your branch and try to create a new patch; file that you
> modified might have got updated.
>
> --Sanjay
>
> > -Original Message-
> > From: Sowmya Krishnan [mailto
The resources of those accounts might not be removed yet. Removing the domain
while these resources are still present, will introduce tons of NPEs when we
try to access the domain info for them.
So removing the domain should be done only after all the accounts are removed.
-Alena.
From: Sanjay
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12723/
---
(Updated July 24, 2013, 5:42 p.m.)
Review request for cloudstack and Prasanna S
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12905/
---
Review request for cloudstack and Sheng Yang.
Bugs: Cloudstack-3753
Repositor
Speaking of system vm upgrades, would you how to force an upgrade without
recreating the vm?
Thinking worse case would be to mount systemvm.iso and execute one of the
init.d scripts, but not certain.
We should not have to recreate system vms, in order for them to get new code.
> -Original
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12907/
---
Review request for cloudstack, Abhinandan Prateek, Brian Federle, Jessica Wang,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12907/
---
(Updated July 24, 2013, 5:52 p.m.)
Review request for cloudstack, Abhinandan Pr
No it's not.
I used to use commons-httpclient-contrib jar, it's separate
> -Original Message-
> From: Alex Huang [mailto:alex.hu...@citrix.com]
> Sent: Tuesday, July 23, 2013 10:25 PM
> To: dev@cloudstack.apache.org
> Subject: RE: Inclusion of commons-httpclient-contrib jar
>
> Isn't thi
Its correct that there is the possibility that resource of these accounts might
not be removed; but if we take the same scenario with deleteAccount API, where
we always return true as success response irrespective of the resources (except
project) are still attached to this account or not. If th
Agree. Amogh actually has done it in cloud-utils
> -Original Message-
> From: Koushik Das [mailto:koushik@citrix.com]
> Sent: Wednesday, July 24, 2013 10:26 AM
> To: dev@cloudstack.apache.org
> Subject: RE: Inclusion of commons-httpclient-contrib jar
>
> For now I think cloud-utils is
Thanks, it will have to be part of a bigger change to also change the UI to
update the Network when the Zone DNS Domain is updated.
-Soheil
From: Chiradeep Vittal [chiradeep.vit...@citrix.com]
Sent: Tuesday, July 23, 2013 9:56 PM
To: dev@cloudstack.apache.o
Nobody has bothered to deal with the LICENSE file for the file described
below.
Anyone want to take this on, or at least give me the information
required?
On Wed, Jul 24, 2013 at 05:59:59PM +, Frank Zhang wrote:
> Agree. Amogh actually has done it in cloud-utils
>
> > -Original Message-
+1, I did QA of 4.1.1 to best of my abilities.
> -Original Message-
> From: Musayev, Ilya [mailto:imusa...@webmd.net]
> Sent: Wednesday, July 24, 2013 1:23 PM
> To: dev@cloudstack.apache.org
> Subject: [VOTE] Apache Cloudstack 4.1.1
>
> Hi All,
>
> I've created a 4.1.1 release, with the
It's going to be different for various things, depending on what is
being changed. Is it a conf for a service, that will need to be
restarted? Or the cloud-early-config init script itself, requiring a
reboot? Internally, we've written a plugin (upgradeRouterScripts
routerid=) that copies the cloud
Amogh has checked it's Apache 2. I am not sure if any process we need to go
through
> -Original Message-
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Wednesday, July 24, 2013 11:24 AM
> To: dev@cloudstack.apache.org
> Subject: Re: Inclusion of commons-httpclient-contr
Soheil,
About network domain update. Changing network domain on the
zone/account/domain/global level, will affect newly created networks only. It
should stay this way. Once the network domain is determined and set for the
network (during the network creation), it can be changed just by modifyin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12810/#review23779
---
Ship it!
Nice work!
patches/systemvm/debian/config/root/dnsmasq.s
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12912/
---
Review request for cloudstack, Koushik Das, Hugo Trippaers, and Wido den
Holland
We just need to include it in the legal docs. It's compatible, but all
external code and dependencies need to be accounted for.
I see that this is from the ASF actually, so adding it is easy. I'll take
a look at doing it shortly.
On Wed, Jul 24, 2013 at 2:28 PM, Frank Zhang wrote:
> Amogh ha
Ok,
I don't like changing this enum. I'd rather throw it out and start over but
you are answering the question by sharing your views on school of
programming, i think. BroadcastDomainType by its name implies it could be
unknown but never undecided. Do you agree?
I can comply with any school.
Op 24
Daan,
Agreed. If it's unknown, the variable of that type can just be null which is
an accepted practice for saying the business logic hasn't determined the value
yet.
--Alex
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: Wednesday, July 24, 2013 12:41 PM
To: Alex Huang
Cc: cloudst
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12905/
---
(Updated July 24, 2013, 8:02 p.m.)
Review request for cloudstack and Sheng Yang
FYI, I just pushed a patch that contains a new router script. Existing
routers in existing KVM dev environments will need to be rebooted after you
pull this in.
Thanks Chip. Let me know once you done so I can get Amogh's patch in
> -Original Message-
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Wednesday, July 24, 2013 12:19 PM
> To:
> Subject: Re: Inclusion of commons-httpclient-contrib jar
>
> We just need to include it in
I don't think you need to wait. I actually need the files in the repo
to pick up on them anyway for the docs.
On Wed, Jul 24, 2013 at 08:13:59PM +, Frank Zhang wrote:
> Thanks Chip. Let me know once you done so I can get Amogh's patch in
>
> > -Original Message-
> > From: Chip Child
Hi,
The files are currently in : http://bit.ly/18Ae7zH
Thanks,
Amogh
On 7/24/13 1:22 PM, "Chip Childers" wrote:
>I don't think you need to wait. I actually need the files in the repo
>to pick up on them anyway for the docs.
>
>
>On Wed, Jul 24, 2013 at 08:13:59PM +, Frank Zhang wrote:
>>
On Tue, Jul 23, 2013 at 11:54 PM, Prasanna Santhanam wrote:
> On Tue, Jul 23, 2013 at 11:20:22PM -0700, Sheng Yang wrote:
> > About the patch I committed, in fact it's not moving. It's fixing.
> Because:
> > 1. The logical reason is, currently in many cases(which I committed the
> > patches) the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12877/#review23782
---
Ship it!
shipped in both 4.2/master
- Frank Zhang
On July 23, 20
Right, that's what I thought. Frank, what were you saying you were
waiting for?
On Wed, Jul 24, 2013 at 08:38:36PM +, Amogh Vasekar wrote:
> Hi,
>
> The files are currently in : http://bit.ly/18Ae7zH
>
> Thanks,
> Amogh
>
> On 7/24/13 1:22 PM, "Chip Childers" wrote:
>
> >I don't think yo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12242/#review23783
---
Commit d09a088093568f167921597825110d5517ac535b in branch refs/heads
No. I have checked in the patch
> -Original Message-
> From: Chip Childers [mailto:chip.child...@sungard.com]
> Sent: Wednesday, July 24, 2013 1:46 PM
> To: dev@cloudstack.apache.org
> Subject: Re: Inclusion of commons-httpclient-contrib jar
>
> Right, that's what I thought. Frank, what
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12883/#review23785
---
After thinking about this some more I think a better way to fix this
1 - 100 of 126 matches
Mail list logo