+1. Thanks Owen

On Mon, Dec 3, 2018, 10:07 Owen Nichols <onich...@pivotal.io wrote:

> > 2. No zip file for the geode, just .tgz.
>
> I believe this was changed a few months ago to simplify our build and
> release process.  Distributing as both .zip and .tgz is a relic of a time
> before WinZip, WinRar, 7-Zip, and most other popular zip utilities gained
> native support for .tgz archives.  If there is still a segment of the Geode
> user community that would suffer hardship due to lack of .zip packaging, we
> should revisit this decision.
>
> -Owen
>
>
> > On Dec 3, 2018, at 9:52 AM, Dan Smith <dsm...@pivotal.io> wrote:
> >
> > I see a few things with the artifacts that I think should be tweaked
> > 1. No pgp signature for the sources!
> > 2. No zip file for the geode, just .tgz. Older releases on our website
> have
> > both zip and tgz. See the differences between [1] and [2]
> > 3. pgp signature for the native source is not ascii armored. See [3]
> >
> > Regarding SHA512 vs SHA256 - we should probably just move everything to
> > SHA512 in the future.
> >
> > [1] https://dist.apache.org/repos/dist/dev/geode/1.8.0.RC1/
> > [2] https://www.apache.org/dist/geode/1.7.0/
> > [3] https://www.apache.org/dev/release-signing.html#signing-basics
> >
> > On Mon, Dec 3, 2018 at 9:24 AM Alexander Murmann <amurm...@pivotal.io>
> > wrote:
> >
> >> Thanks for taking such a detailed a look, Nabarun! That's awesome input.
> >>
> >> 1. Is there a reason why geode-native is signed with SHA512 while all
> the
> >>> rest are signed with SHA256?
> >>
> >> Not really. I used the defaults provided by the Gradle signing task in
> the
> >> case of the core codebase and the GPG tool's default when signing the
> >> native code. I noticed that GPG's default was larger, but figured more
> bits
> >> are better bits and come at pretty much no additional cost. If this is
> >> confusing, I am happy to sign with a smaller hash or at least at
> >> documentation for doing so more consistently in the next release.
> >> Any opinions on how to proceed with this?
> >>
> >> 2. Are there any directions / documentation on how to verify the
> >>> geode-native release components?
> >>
> >> I unpacked the release and followed the instructions in building.md.
> >> Someone who has contributed more to the Native code base might have much
> >> better steps in mind. Pleas chime in!
> >>
> >> On Mon, Dec 3, 2018 at 8:45 AM Nabarun Nag <n...@apache.org> wrote:
> >>
> >>> Following checks completed:
> >>> - checked signatures
> >>> - checked SHA's
> >>> - builds from source [geode]
> >>> - run gfsh - start locator, server - create region - do put and get -
> >>> execute OQL query
> >>> - examples run cleanly [geode-examples]
> >>> - the correct version in gfsh command version
> >>>
> >>> Questions:
> >>> 1. Is there a reason why geode-native is signed with SHA512 while all
> the
> >>> rest are signed with SHA256?
> >>> 2. Are there any directions / documentation on how to verify the
> >>> geode-native release components?
> >>>
> >>>
> >>> File Differences:
> >>> 1. Files KEYS and gradlew.bat are present in the github repo for
> >>> rel/v1.8.0.RC1 but not present in the source release
> >> apache-geode-1.8.0-src
> >>> 2. gradlew file differs in the rel/v1.8.0.RC1 repo and the source
> release
> >>> apache-geode-1.8.0-src.
> >>>
> >>> Apologies if these changes are expected.
> >>>
> >>> Regards
> >>> Nabarun Nag
> >>>
> >>>
> >>>
> >>>
> >>> On Fri, Nov 30, 2018 at 5:38 PM Alexander Murmann <amurm...@pivotal.io
> >
> >>> wrote:
> >>>
> >>>> Hi everyone,
> >>>>
> >>>> Per above discussion the release now contains Geode Native.
> >>>> Here is the updated release information:
> >>>>
> >>>> Apache Geode:
> >>>> https://github.com/apache/geode/tree/rel/v1.8.0.RC1
> >>>> Apache Geode examples:
> >>>> https://github.com/apache/geode-examples/tree/rel/v1.8.0.RC1
> >>>> Apache Geode Native:
> >>>> https://github.com/apache/geode-native/tree/rel/v1.8.0.RC1
> >>>>
> >>>> Commit IDs:
> >>>> Apache Geode: 671671b5e81acde2684df3331aedf176cc927e6e
> >>>> Apache Geode Examples: aee3794f1302ffab51b4ca5d02657598420b7a00
> >>>> Apache Geode Native: 32d71d13087b5c1a36417693cf8da9a8819edbdf
> >>>>
> >>>> Source and binary files:
> >>>> https://dist.apache.org/repos/dist/dev/geode/1.8.0.RC1/
> >>>>
> >>>> Maven staging repo:
> >>>>
> https://repository.apache.org/content/repositories/orgapachegeode-1048
> >>>>
> >>>> Geode's KEYS file containing PGP keys we use to sign the release:
> >>>> https://github.com/apache/geode/blob/develop/KEYS
> >>>>
> >>>> Signed the release with fingerprint:
> >>>> rsa4096 2018-09-01 [SC]
> >>>> D5C5C950D61898EDE8928820D6048392BDFB7797
> >>>>
> >>>>
> >>>> On Fri, Nov 30, 2018 at 9:44 AM Anthony Baker <aba...@pivotal.io>
> >> wrote:
> >>>>
> >>>>> Because this is confusing, let me clarify our current approach again:
> >>>>>
> >>>>> When we do a release of the Geode Project, it will include all the
> >>>>> constituent pieces we deem appropriate regardless of the repo the
> >>> source
> >>>>> comes from.  Currently this includes:
> >>>>>
> >>>>> - geode
> >>>>> - geode-examples
> >>>>> - geode-native
> >>>>>
> >>>>> Perhaps in the future, we would include geode-benchmarks.
> >>>>>
> >>>>> In order to create a distinct release with a separate lifecycle we
> >>> would
> >>>>> need to spawn a subproject with a separate PMC and a viable
> >> community.
> >>>>>
> >>>>> @Alexander, I don’t think you need to issue a new release candidate.
> >>>> Just
> >>>>> add the geode-native source distribution and update the VOTE email.
> >>>>>
> >>>>>
> >>>>> Anthony
> >>>>>
> >>>>>
> >>>>>> On Nov 30, 2018, at 8:08 AM, Alexander Murmann <
> >> amurm...@pivotal.io>
> >>>>> wrote:
> >>>>>>
> >>>>>> Sorry, I was unaware that we were planning on releasing
> >> geode-native
> >>> as
> >>>>>> part of the same release and not as a separate release that goes
> >> out
> >>>> at a
> >>>>>> similar time.
> >>>>>>
> >>>>>> I am happy to work on a new candidate that includes geode-native.
> >>>>>>
> >>>>>> On Fri, Nov 30, 2018 at 6:39 AM Anthony Baker <aba...@pivotal.io>
> >>>> wrote:
> >>>>>>
> >>>>>>> Is there a reason the geode-native repo was not included in the
> >>>> release?
> >>>>>>>
> >>>>>>> Anthony
> >>>>>>>
> >>>>>>>
> >>>>>>>> On Nov 29, 2018, at 11:15 PM, Alexander Murmann <
> >>> amurm...@pivotal.io
> >>>>>
> >>>>>>> wrote:
> >>>>>>>>
> >>>>>>>> Hello Geode dev community!
> >>>>>>>>
> >>>>>>>> I am happy to announce the first release candidate for Apache
> >> Geode
> >>>>>>> 1.8.0!
> >>>>>>>> Thanks to all the community members for their contributions to
> >> this
> >>>>>>>> release!
> >>>>>>>>
> >>>>>>>> Please review and give your feedback! The deadline is the end of
> >>> day
> >>>>> Dec.
> >>>>>>>> 4th 2018.
> >>>>>>>>
> >>>>>>>> It resolves 162 issues on Apache Geode JIRA system. Release notes
> >>> can
> >>>>> be
> >>>>>>>> found at:
> >>>>>>>>
> >>>>>>>
> >>>>>
> >>>>
> >>>
> >>
> https://cwiki.apache.org/confluence/display/GEODE/Release+Notes#ReleaseNotes-1.8.0
> >>>>>>>>
> >>>>>>>> Please note that we are voting upon the source tags:
> >> rel/v1.8.0.RC1
> >>>>>>>> Apache Geode:
> >>>>>>>> https://github.com/apache/geode/tree/rel/v1.8.0.RC1
> >>>>>>>> Apache Geode examples:
> >>>>>>>> https://github.com/apache/geode-examples/tree/rel/v1.8.0.RC1
> >>>>>>>>
> >>>>>>>> Commit IDs:
> >>>>>>>> Apache Geode: 671671b5e81acde2684df3331aedf176cc927e6e
> >>>>>>>> Apache Geode Examples: aee3794f1302ffab51b4ca5d02657598420b7a00
> >>>>>>>>
> >>>>>>>> Source and binary files:
> >>>>>>>> https://dist.apache.org/repos/dist/dev/geode/1.8.0.RC1/
> >>>>>>>>
> >>>>>>>> Maven staging repo:
> >>>>>>>>
> >>>>
> https://repository.apache.org/content/repositories/orgapachegeode-1048
> >>>>>>>>
> >>>>>>>> Geode's KEYS file containing PGP keys we use to sign the release:
> >>>>>>>> https://github.com/apache/geode/blob/develop/KEYS
> >>>>>>>>
> >>>>>>>> Signed the release with fingerprint:
> >>>>>>>> rsa4096 2018-09-01 [SC]
> >>>>>>>> D5C5C950D61898EDE8928820D6048392BDFB7797
> >>>>>>>>
> >>>>>>>> PS: Command to run geode-examples: ./gradlew -PgeodeReleaseUrl=
> >>>>>>>> https://dist.apache.org/repos/dist/dev/geode/1.8.0.RC1
> >>>>>>> -PgeodeRepositoryUrl=
> >>>>>>>>
> >>>>
> https://repository.apache.org/content/repositories/orgapachegeode-1048
> >>>>>>>> build runAll
> >>>>>>>>
> >>>>>>>> Thank you!
> >>>>>>>> Alexander
> >>>>>>>
> >>>>>>>
> >>>>>
> >>>>>
> >>>>
> >>>
> >>
>
>

Reply via email to