Re: [VOTE] Fluo 1.0.0-incubating-rc1

2016-09-23 Thread Christopher
Technically, this vote doesn't expire until 1930EDT, but I don't plan on
voting on this one, and it doesn't look like it'll pass as-is. I'll create
an RC2 on Monday, if this vote fails.

On Fri, Sep 23, 2016 at 5:35 PM Keith Turner  wrote:

> On Wed, Sep 21, 2016 at 11:13 PM, Josh Elser  wrote:
> > +1 (binding)
> >
> > First off, interesting use of the fetch.sh script to circumvent any extra
> > licensing in Fluo. I feel like this kind of goes against the intent of
> the
> > LICENSE and NOTICE files (they're meant to be there so that users know
> your
> > product depends on and those licenses), but I can't cite a reason why
> it's
> > against policy.
> >
> > * xsum/sigs OK
> > * no binary files in source-release
> > * apache-rat:check passes
> > * mvn verify passes (sans GarbageCollectionIteratorIT[1])
> > * DISCLAIMER is present
> > * L are fine
> > * KEYS present and contains the signing key
> > * SHA1 present in repo
> >
> > * I feel like you could easily add the license header to the
> docs/**/*.md,
> > but I know I've seen this go both ways.
>
> Just made this change
>
> > * I would like to see apache-fluo in the tarball names and directories
> > contained in the tarballs, but I've also argued at length with
> Christopher
> > about this in the past. I will probably continue to cite it as something
> I'd
> > like to see :)
> > * I would guess some people in IPMC would like to see your artifacats
> staged
> > to dist.a.o/repos/dist/dev/incubator/fluo but nothing wrong with
> references
> > from repo.a.o AFAIK
> >
> > Made a pass over the website and things look OK too. Will be happy to see
> > the two reference to "pre-ASF life" things removed promptly after
> > 1.0.0-incubating is released.
> >
> >
> > - Josh
> >
> > [1]
> >
> > ```
> > Running org.apache.fluo.integration.impl.GarbageCollectionIteratorIT
> > Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 113.007
> sec
> > <<< FAILURE! - in
> > org.apache.fluo.integration.impl.GarbageCollectionIteratorIT
> >
> testGetOldestTimestamp(org.apache.fluo.integration.impl.GarbageCollectionIteratorIT)
> > Time elapsed: 21.67 sec  <<< FAILURE!
> > java.lang.AssertionError: expected:<0> but was:<2>
> > at
> >
> org.apache.fluo.integration.impl.GarbageCollectionIteratorIT.testGetOldestTimestamp(GarbageCollectionIteratorIT.java:194)
> > ```
> >
> > Christopher wrote:
> >>
> >> Fluo Developers,
> >>
> >> Please consider the following candidate for Fluo 1.0.0-incubating.
> >>
> >> Git Commit:
> >>  b07fbf8b755e8f555d06fe2d57db2868f9b20c1c
> >> Branch:
> >>  1.0.0-incubating-rc1
> >>
> >> If this vote passes, a gpg-signed tag will be created using:
> >>  git tag -f -m 'Apache Fluo 1.0.0-incubating' -s
> >> rel/fluo-1.0.0-incubating \
> >>  b07fbf8b755e8f555d06fe2d57db2868f9b20c1c
> >>
> >> Staging repo:
> >> https://repository.apache.org/content/repositories/orgapachefluo-1012
> >> Source (official release artifact):
> >>
> >>
> https://repository.apache.org/content/repositories/orgapachefluo-1012/org/apache/fluo/fluo/1.0.0-incubating/fluo-1.0.0-incubating-source-release.tar.gz
> >> Binary:
> >>
> >>
> https://repository.apache.org/content/repositories/orgapachefluo-1012/org/apache/fluo/fluo/1.0.0-incubating/fluo-1.0.0-incubating-bin.tar.gz
> >> (Append ".sha1", ".md5", or ".asc" to download the signature/hash for a
> >> given artifact.)
> >>
> >> All artifacts were built and staged with:
> >>  mvn release:prepare&&  mvn release:perform
> >>
> >> Signing keys are available at
> >> https://www.apache.org/dist/incubator/fluo/KEYS
> >> (Expected fingerprint: 8CC4F8A2B29C2B040F2B835D6F0CDAE700B6899D)
> >>
> >> Release notes (in progress) can be found at:
> >> https://fluo.apache.org/.../1.0.0-incubating
> >>
> >> Please vote one of:
> >> [ ] +1 - I have verified and accept...
> >> [ ] +0 - I have reservations, but not strong enough to vote against...
> >> [ ] -1 - Because..., I do not accept...
> >> ... these artifacts as the 1.0.0-incubating release of Apache Fluo.
> >>
> >> This vote will end on Fri Sep 23 23:30:00 UTC 2016
> >> (Fri Sep 23 19:30:00 EDT 2016 / Fri Sep 23 16:30:00 PDT 2016)
> >>
> >> Thanks!
> >>
> >> P.S. Hint: download the whole staging repo with
> >>  wget -erobots=off -r -l inf -np -nH \
> >>
> >> https://repository.apache.org/content/repositories/orgapachefluo-1012/
> >>  # note the trailing slash is needed
> >>
> >
>


Re: [VOTE] Fluo 1.0.0-incubating-rc1

2016-09-21 Thread Josh Elser

+1 (binding)

First off, interesting use of the fetch.sh script to circumvent any 
extra licensing in Fluo. I feel like this kind of goes against the 
intent of the LICENSE and NOTICE files (they're meant to be there so 
that users know your product depends on and those licenses), but I can't 
cite a reason why it's against policy.


* xsum/sigs OK
* no binary files in source-release
* apache-rat:check passes
* mvn verify passes (sans GarbageCollectionIteratorIT[1])
* DISCLAIMER is present
* L are fine
* KEYS present and contains the signing key
* SHA1 present in repo

* I feel like you could easily add the license header to the 
docs/**/*.md, but I know I've seen this go both ways.
* I would like to see apache-fluo in the tarball names and directories 
contained in the tarballs, but I've also argued at length with 
Christopher about this in the past. I will probably continue to cite it 
as something I'd like to see :)
* I would guess some people in IPMC would like to see your artifacats 
staged to dist.a.o/repos/dist/dev/incubator/fluo but nothing wrong with 
references from repo.a.o AFAIK


Made a pass over the website and things look OK too. Will be happy to 
see the two reference to "pre-ASF life" things removed promptly after 
1.0.0-incubating is released.



- Josh

[1]

```
Running org.apache.fluo.integration.impl.GarbageCollectionIteratorIT
Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 113.007 
sec <<< FAILURE! - in 
org.apache.fluo.integration.impl.GarbageCollectionIteratorIT
testGetOldestTimestamp(org.apache.fluo.integration.impl.GarbageCollectionIteratorIT) 
 Time elapsed: 21.67 sec  <<< FAILURE!

java.lang.AssertionError: expected:<0> but was:<2>
	at 
org.apache.fluo.integration.impl.GarbageCollectionIteratorIT.testGetOldestTimestamp(GarbageCollectionIteratorIT.java:194)

```

Christopher wrote:

Fluo Developers,

Please consider the following candidate for Fluo 1.0.0-incubating.

Git Commit:
 b07fbf8b755e8f555d06fe2d57db2868f9b20c1c
Branch:
 1.0.0-incubating-rc1

If this vote passes, a gpg-signed tag will be created using:
 git tag -f -m 'Apache Fluo 1.0.0-incubating' -s
rel/fluo-1.0.0-incubating \
 b07fbf8b755e8f555d06fe2d57db2868f9b20c1c

Staging repo:
https://repository.apache.org/content/repositories/orgapachefluo-1012
Source (official release artifact):
https://repository.apache.org/content/repositories/orgapachefluo-1012/org/apache/fluo/fluo/1.0.0-incubating/fluo-1.0.0-incubating-source-release.tar.gz
Binary:
https://repository.apache.org/content/repositories/orgapachefluo-1012/org/apache/fluo/fluo/1.0.0-incubating/fluo-1.0.0-incubating-bin.tar.gz
(Append ".sha1", ".md5", or ".asc" to download the signature/hash for a
given artifact.)

All artifacts were built and staged with:
 mvn release:prepare&&  mvn release:perform

Signing keys are available at
https://www.apache.org/dist/incubator/fluo/KEYS
(Expected fingerprint: 8CC4F8A2B29C2B040F2B835D6F0CDAE700B6899D)

Release notes (in progress) can be found at:
https://fluo.apache.org/.../1.0.0-incubating

Please vote one of:
[ ] +1 - I have verified and accept...
[ ] +0 - I have reservations, but not strong enough to vote against...
[ ] -1 - Because..., I do not accept...
... these artifacts as the 1.0.0-incubating release of Apache Fluo.

This vote will end on Fri Sep 23 23:30:00 UTC 2016
(Fri Sep 23 19:30:00 EDT 2016 / Fri Sep 23 16:30:00 PDT 2016)

Thanks!

P.S. Hint: download the whole staging repo with
 wget -erobots=off -r -l inf -np -nH \
 https://repository.apache.org/content/repositories/orgapachefluo-1012/
 # note the trailing slash is needed