---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/#review25127
---
An issue related to this patch:
On Aug. 14, 2013, 7:48 a.m., Wei Zhou wrote:
An issue related to this patch:
https://issues.apache.org/jira/browse/CLOUDSTACK-4314
and patch:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=eb59c3c
Testing ok on devcloud.
indeed I did not test with the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/#review25005
---
Hi Reviewers,
Any feedback on this patch? :)
- Laszlo Hornyak
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/#review25036
---
Ship it!
This is very straightforward patch. Java filesystem api
Laszlo,
I was only concerned about the lack of asserts in the original test case. That
has been updated, so it looks good to me now.
Thanks,
-John
On Aug 12, 2013, at 5:43 PM, Frank Zhang frank.zh...@citrix.com wrote:
---
This is an
: Monday, August 12, 2013 2:43 PM
To: John Burwell; Frank Zhang
Cc: Laszlo Hornyak; cloudstack
Subject: Re: Review Request 11942: ProcessUtil cleanup
---
This is an automatically generated e-mail. To reply, visit:
https
; John Burwell
Cc: Laszlo Hornyak
Subject: RE: Review Request 11942: ProcessUtil cleanup
Is this really needed for 4.2? 4.2 is in limited updates and only
blocker/critical
bug fixes and doc and test updates are allowed. All commits to 4.2 should have
associated BUG id.
-Original Message
We should keep it for master
-Original Message-
From: Frank Zhang
Sent: Monday, August 12, 2013 5:40 PM
To: Animesh Chaturvedi; dev@cloudstack.apache.org; John Burwell
Cc: Laszlo Hornyak
Subject: RE: Review Request 11942: ProcessUtil cleanup
Emm. Actually it's not a blocker
: Review Request 11942: ProcessUtil cleanup
We should keep it for master
-Original Message-
From: Frank Zhang
Sent: Monday, August 12, 2013 5:40 PM
To: Animesh Chaturvedi; dev@cloudstack.apache.org; John Burwell
Cc: Laszlo Hornyak
Subject: RE: Review Request 11942: ProcessUtil
I reverted it from 4.2. now is adding the Apache header to master
-Original Message-
From: Animesh Chaturvedi
Sent: Monday, August 12, 2013 5:56 PM
To: dev@cloudstack.apache.org; Frank Zhang; John Burwell
Cc: Laszlo Hornyak
Subject: RE: Review Request 11942: ProcessUtil cleanup
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/
---
(Updated July 24, 2013, 4:42 a.m.)
Review request for cloudstack, Frank Zhang
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/#review23615
---
Review 11942 PASSED the build test
The url of build
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/
---
(Updated July 20, 2013, 8:01 p.m.)
Review request for cloudstack.
Changes
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11942/
---
(Updated June 29, 2013, 3:51 p.m.)
Review request for cloudstack.
Changes
14 matches
Mail list logo