[prev in list] [next in list] [prev in thread] [next in thread]
List: incubator-general
Subject: Re: [VOTE] Release Apache SDAP (incubating) 1.0.0-rc2
From: larry mccay <lmccay () apache ! org>
Date: 2022-12-30 23:39:13
Message-ID: CACRbFygE8gKUjiYYDYuQzU7QRGdC7KrKZ9B44sFq4Xk=Cyh3qg () mail ! gmail ! com
[Download RAW message or body]
I wanted to lend a hand with the sdap 1.0.0 release review and wanted to
let you know that it seems a bit cumbersome to review.
Perhaps my experience with other projects is limited to similar
conventions, though.
The fact that the src artifacts aren't under a project specific root ended
up polluting my releases directory (where I review releases). This was my
fault for not noticing the missing root but clearly outside my
expectations.
I also notice that each of what I would expect to just be separate modules
have their own tar balls and even github mirrors which is also awkward for
how I expect to review things - though not technically related to typical
review requirements.
These may be seen as less relevant to the specific release at hand and more
for the overall project structure and conventions but they would certainly
help in reviewing.
I note that there are numerous files [1][2][3] and many more that may
require Apache License headers.
You can use something like the following in python files:
#
# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements. See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to You under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#
The NOTICE file has a copyright of 2017-2022 still and unless this goes out
in the next couple days that will be out of date.
Duplicate and separate files like NOTICE, LICENSE, DISCLAIMER access
ingester, nexus and nexusproto as well as CONTRIBUTING.md which is in 2 but
not the other point to some organization issues. There should be some
effort put in to possibly make these modules under a common project rather
than what seem like separate projects.
Some of the above make it pretty cumbersome to review even if not
technically blockers.
Addressing these sorts of things early will help contributors feel more
easily able to help out and can help build your community.
It will also help you get releases out easier.
Hope the above is useful for you.
Happy to see a release coming together for the SDAP community and
congratulate you on tackling this milestone!
It isn't clear to me that these should block your initial release - my vote
would be as follows:
-0 for my vote at this time.
1.
https://github.com/apache/incubator-sdap-ingester/blob/dev/collection_manager/requirements.txt
2.
https://github.com/apache/incubator-sdap-ingester/blob/dev/collection_manager/setup.py
3.
https://github.com/apache/incubator-sdap-ingester/blob/dev/collection_manager/collection_manager/main.py
On Fri, Dec 30, 2022 at 4:30 PM Julian Hyde <jhyde@apache.org> wrote:
> Good catch. But could you (and others) continue the process of
> reviewing and voting on the release.
>
> It is established that an official ASF release must not contain
> gradle-wrapper.jar [1], but in my opinion this is a problem that could
> be solved in the next incubating release. If a few IPMC members
> surface several issues this time around, the second release should be
> much cleaner.
>
> Julian
>
> [1] https://issues.apache.org/jira/browse/LEGAL-570
>
> On Thu, Dec 29, 2022 at 5:44 PM Calvin Kirs <kirs@apache.org> wrote:
> >
> > Hi,
> >
> > There has a compiled code (gradle-wrapper jar) in the source release.
> >
> > [1]
> https://dist.apache.org/repos/dist/dev/incubator/sdap/apache-sdap-1.0.0-rc2/apache-sdap-nexusproto-1.0.0-src.tar.gz
>
> >
> > On Sat, Dec 24, 2022 at 5:46 AM Riley Kuttruff <rkk@apache.org> wrote:
> > >
> > > Hello,
> > >
> > > In consideration of the upcoming holiday weekend, let's extend this
> vote to last to a week from now.
> > >
> > > Thank you,
> > > Riley
> > >
> > > On 2022/12/23 03:19:10 Riley Kuttruff wrote:
> > > > Hello everyone,
> > > >
> > > > This is a call for a vote to release Apache SDAP (incubating)
> version 1.0.0-rc2.
> > > >
> > > > The Apache SDAP community has voted to approve release of Apache
> SDAP (incubating) version 1.0.0-rc2.
> > > >
> > > > We now request the Incubator PMC review and vote on this release.
> > > >
> > > > SDAP community vote thread:
> > > > https://lists.apache.org/thread/6nmknw8n1flwsqp4n3cromnrck21xmh9
> > > >
> > > > Instructions for building docker images from source can be found
> here:
> > > > https://incubator-sdap-nexus.readthedocs.io/en/latest/build.html
> > > > Instructions for deploying locally to test can be found here:
> > > >
> https://incubator-sdap-nexus.readthedocs.io/en/latest/quickstart.html
> > > >
> > > > The tags to be voted on are 1.0.0-rc2:
> > > >
> > > > - nexusproto:
> > > > - URL:
> https://github.com/apache/incubator-sdap-nexusproto/tree/1.0.0-rc2
> > > > - Commit: f3c726e69997436dd9ee6f9a993d4c95ee490b6f
> > > > - ingester:
> > > > - URL:
> https://github.com/apache/incubator-sdap-ingester/tree/1.0.0-rc2
> > > > - Commit: 2c8a9d35320f11a5db2534802f43366ddaff2fea
> > > > - nexus:
> > > > - URL:
> https://github.com/apache/incubator-sdap-nexus/tree/1.0.0-rc2
> > > > - Commit: f60e98331ac29b57ca40322f8f96e98a5c69e553
> > > >
> > > > The release files, including signatures, digests, etc. can be found
> at:
> > > >
> https://dist.apache.org/repos/dist/dev/incubator/sdap/apache-sdap-1.0.0-rc2/
> > > >
> > > > For verification, the hashes of the .tar.gz artifacts we are voting
> on are as follows:
> > > >
> > > > apache-sdap-ingester-1.0.0-src.tar.gz:
> > > >
> db380c3002351e65ec3581a3ff0cacbb7092f46292b0a656a78a919e1409738442e845731050a2160a556a7818f22b2ab4bb6da437200bc0a62419332de49ad7
>
> > > >
> > > > apache-sdap-nexus-1.0.0-src.tar.gz:
> > > >
> da34e9830cf5126bac498180d7cdd95bc2cfc13535c7b4355ba51de64e3589a385a969e0cf1a89fa77e436b30c99a48abffab4b4ef03b23982aa33c2e83e8dbd
>
> > > >
> > > > apache-sdap-nexusproto-1.0.0-src.tar.gz:
> > > >
> 05dc8e96d9c49207750c1da13f630493e8d08f5e68733742c157ac6c85ede29aca25f3c94cd706acbbe21cee41ebc052a880b4d6ce859bf510bcc2d66d8f88b2
>
> > > >
> > > > Release artifacts are signed with the following key:
> > > >
> https://keyserver.ubuntu.com/pks/lookup?op=get&search=0x4e98c4a32026656e14e0b570fc20035a010e3b7b
>
> > > >
> > > > Please vote on releasing this package as Apache SDAP (incubating)
> 1.0.0.
> > > >
> > > > The vote is open for 72 hours or until at least 3 "+1" PMC votes are
> cast and passes if at least 3 +1 PMC votes are
> > > > cast.
> > > >
> > > > [ ] +1 Release this package as Apache SDAP (incubating) 1.0.0
> > > > [ ] +0 No opinion
> > > > [ ] -1 Do not release this package because ...
> > > >
> > > > Thank you,
> > > > Riley
> > > >
> > > > ---------------------------------------------------------------------
> > > > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> > > > For additional commands, e-mail: general-help@incubator.apache.org
> > > >
> > > >
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> > > For additional commands, e-mail: general-help@incubator.apache.org
> > >
> >
> >
> > --
> > Best wishes！
> > CalvinKirs
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: general-help@incubator.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic