FWIW, I will stage in SVN before IPMC vote. I've also been thinking of ways
to automate that part for the future. :)

On Wed, Sep 21, 2016, 23:13 Josh Elser <els...@apache.org> 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&N 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
> >
>

Reply via email to