[VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

Dan Kirkwood-2
Hello Incubator PMC,

The Apache Traffic Control community has voted on and approved a
proposal to release Apache Traffic Control 1.8.1-incubating. We now
kindly request that the Incubator PMC members review and vote on this
incubator release.

The VOTE RESULT is here:

https://lists.apache.org/thread.html/c43fe387586c44e2ecbac1f968a146f9a402a328be0f6e9b296fc3c9@%3Cusers.trafficcontrol.apache.org%3E

The draft release notes (along with links to artifacts,
signatures/checksums, and updated documentation) can be found here:

http://trafficcontrol.incubator.apache.org/downloads/

The git tag for the repository is "RELEASE-1.8.1-RC0":
https://github.com/apache/incubator-trafficcontrol/releases/tag/RELEASE-1.8.1-RC0

The source distribution (also linked in the release notes) is here:

https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/1.8.1/RC0/

Build instructions are included in the BUILD.md file which is included
in the source artifact.

Artifacts have been signed with the "[hidden email]" key listed in:

https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS

Please review and vote:

[ ] +1 Approve the release
[ ] -1 Don't approve the release (please provide specific comments)

This vote will be open for at least 72 hours.

Thanks,

 Dan Kirkwood <[hidden email]>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

sebb-2-2
On 5 June 2017 at 22:12, Dan Kirkwood <[hidden email]> wrote:

DROPPED user@ list - see below.

> Hello Incubator PMC,
>
> The Apache Traffic Control community has voted on and approved a
> proposal to release Apache Traffic Control 1.8.1-incubating. We now
> kindly request that the Incubator PMC members review and vote on this
> incubator release.
>
> The VOTE RESULT is here:
>
> https://lists.apache.org/thread.html/c43fe387586c44e2ecbac1f968a146f9a402a328be0f6e9b296fc3c9@%3Cusers.trafficcontrol.apache.org%3E
>
> The draft release notes (along with links to artifacts,
> signatures/checksums, and updated documentation) can be found here:
>
> http://trafficcontrol.incubator.apache.org/downloads/

This is a *public* URL.
However the release has not yet been approved, so it must *not* be published.
For the same reason, the email regarding the RC should not be sent to
the user list.

Also the download page points to

https://dist.apache.org/repos/dist/

The dist repo is a staging repo only, and must not be used for public URLs.

> The git tag for the repository is "RELEASE-1.8.1-RC0":
> https://github.com/apache/incubator-trafficcontrol/releases/tag/RELEASE-1.8.1-RC0
>
> The source distribution (also linked in the release notes) is here:
>
> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/1.8.1/RC0/

That URL is fine for RC vote mails

> Build instructions are included in the BUILD.md file which is included
> in the source artifact.
>
> Artifacts have been signed with the "[hidden email]" key listed in:
>
> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS

That URL is fine for RC vote mails

> Please review and vote:
>
> [ ] +1 Approve the release
> [ ] -1 Don't approve the release (please provide specific comments)
>
> This vote will be open for at least 72 hours.
>
> Thanks,
>
>  Dan Kirkwood <[hidden email]>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

Dan Kirkwood-2
Thanks for that feedback -- I'll make sure our release instructions
are clear on those points and remove users@ from any future votes.

> The draft release notes (along with links to artifacts,
> signatures/checksums, and updated documentation) can be found here:
>
> http://trafficcontrol.incubator.apache.org/downloads/

To clarify,  this is misstated: the artifacts for this release are not
there -- they will be placed at that location once the release is
approved and not before.  The previous (approved) release artifacts
are at that location.

-dan


On Mon, Jun 5, 2017 at 3:49 PM, sebb <[hidden email]> wrote:

> On 5 June 2017 at 22:12, Dan Kirkwood <[hidden email]> wrote:
>
> DROPPED user@ list - see below.
>
>> Hello Incubator PMC,
>>
>> The Apache Traffic Control community has voted on and approved a
>> proposal to release Apache Traffic Control 1.8.1-incubating. We now
>> kindly request that the Incubator PMC members review and vote on this
>> incubator release.
>>
>> The VOTE RESULT is here:
>>
>> https://lists.apache.org/thread.html/c43fe387586c44e2ecbac1f968a146f9a402a328be0f6e9b296fc3c9@%3Cusers.trafficcontrol.apache.org%3E
>>
>> The draft release notes (along with links to artifacts,
>> signatures/checksums, and updated documentation) can be found here:
>>
>> http://trafficcontrol.incubator.apache.org/downloads/
>
> This is a *public* URL.
> However the release has not yet been approved, so it must *not* be published.
> For the same reason, the email regarding the RC should not be sent to
> the user list.
>
> Also the download page points to
>
> https://dist.apache.org/repos/dist/
>
> The dist repo is a staging repo only, and must not be used for public URLs.
>
>> The git tag for the repository is "RELEASE-1.8.1-RC0":
>> https://github.com/apache/incubator-trafficcontrol/releases/tag/RELEASE-1.8.1-RC0
>>
>> The source distribution (also linked in the release notes) is here:
>>
>> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/1.8.1/RC0/
>
> That URL is fine for RC vote mails
>
>> Build instructions are included in the BUILD.md file which is included
>> in the source artifact.
>>
>> Artifacts have been signed with the "[hidden email]" key listed in:
>>
>> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS
>
> That URL is fine for RC vote mails
>
>> Please review and vote:
>>
>> [ ] +1 Approve the release
>> [ ] -1 Don't approve the release (please provide specific comments)
>>
>> This vote will be open for at least 72 hours.
>>
>> Thanks,
>>
>>  Dan Kirkwood <[hidden email]>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [hidden email]
>> For additional commands, e-mail: [hidden email]
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

sebb-2-2
On 5 June 2017 at 23:20, Dan Kirkwood <[hidden email]> wrote:

> Thanks for that feedback -- I'll make sure our release instructions
> are clear on those points and remove users@ from any future votes.
>
>> The draft release notes (along with links to artifacts,
>> signatures/checksums, and updated documentation) can be found here:
>>
>> http://trafficcontrol.incubator.apache.org/downloads/
>
> To clarify,  this is misstated: the artifacts for this release are not
> there -- they will be placed at that location once the release is
> approved and not before.  The previous (approved) release artifacts
> are at that location.

OK, that's a relief.

Is there a copy of the proposed download page?
It would be useful to link to that.

> -dan
>
>
> On Mon, Jun 5, 2017 at 3:49 PM, sebb <[hidden email]> wrote:
>> On 5 June 2017 at 22:12, Dan Kirkwood <[hidden email]> wrote:
>>
>> DROPPED user@ list - see below.
>>
>>> Hello Incubator PMC,
>>>
>>> The Apache Traffic Control community has voted on and approved a
>>> proposal to release Apache Traffic Control 1.8.1-incubating. We now
>>> kindly request that the Incubator PMC members review and vote on this
>>> incubator release.
>>>
>>> The VOTE RESULT is here:
>>>
>>> https://lists.apache.org/thread.html/c43fe387586c44e2ecbac1f968a146f9a402a328be0f6e9b296fc3c9@%3Cusers.trafficcontrol.apache.org%3E
>>>
>>> The draft release notes (along with links to artifacts,
>>> signatures/checksums, and updated documentation) can be found here:
>>>
>>> http://trafficcontrol.incubator.apache.org/downloads/
>>
>> This is a *public* URL.
>> However the release has not yet been approved, so it must *not* be published.
>> For the same reason, the email regarding the RC should not be sent to
>> the user list.
>>
>> Also the download page points to
>>
>> https://dist.apache.org/repos/dist/
>>
>> The dist repo is a staging repo only, and must not be used for public URLs.
>>
>>> The git tag for the repository is "RELEASE-1.8.1-RC0":
>>> https://github.com/apache/incubator-trafficcontrol/releases/tag/RELEASE-1.8.1-RC0
>>>
>>> The source distribution (also linked in the release notes) is here:
>>>
>>> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/1.8.1/RC0/
>>
>> That URL is fine for RC vote mails
>>
>>> Build instructions are included in the BUILD.md file which is included
>>> in the source artifact.
>>>
>>> Artifacts have been signed with the "[hidden email]" key listed in:
>>>
>>> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS
>>
>> That URL is fine for RC vote mails
>>
>>> Please review and vote:
>>>
>>> [ ] +1 Approve the release
>>> [ ] -1 Don't approve the release (please provide specific comments)
>>>
>>> This vote will be open for at least 72 hours.
>>>
>>> Thanks,
>>>
>>>  Dan Kirkwood <[hidden email]>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [hidden email]
>>> For additional commands, e-mail: [hidden email]
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [hidden email]
>> For additional commands, e-mail: [hidden email]
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

John D. Ament-2
I generally agree with Sebb's sentiments.  Its confusing that you list a
downloads page, but it's not listed (though I feel its better that its not
shown based on last release's issues).

For next release, I would like the output artifact to be
"apache-trafficcontrol-VERSION-incubating" (think back to my
presentation).  This better aligns with your overall branding.

You're mentioning the font licenses in NOTICE, but they better apply to the
LICENSE file.

Here's my +1 as your release is better.  Please try to address these for
next release.

John

On Mon, Jun 5, 2017 at 6:52 PM sebb <[hidden email]> wrote:

> On 5 June 2017 at 23:20, Dan Kirkwood <[hidden email]> wrote:
> > Thanks for that feedback -- I'll make sure our release instructions
> > are clear on those points and remove users@ from any future votes.
> >
> >> The draft release notes (along with links to artifacts,
> >> signatures/checksums, and updated documentation) can be found here:
> >>
> >> http://trafficcontrol.incubator.apache.org/downloads/
> >
> > To clarify,  this is misstated: the artifacts for this release are not
> > there -- they will be placed at that location once the release is
> > approved and not before.  The previous (approved) release artifacts
> > are at that location.
>
> OK, that's a relief.
>
> Is there a copy of the proposed download page?
> It would be useful to link to that.
>
> > -dan
> >
> >
> > On Mon, Jun 5, 2017 at 3:49 PM, sebb <[hidden email]> wrote:
> >> On 5 June 2017 at 22:12, Dan Kirkwood <[hidden email]> wrote:
> >>
> >> DROPPED user@ list - see below.
> >>
> >>> Hello Incubator PMC,
> >>>
> >>> The Apache Traffic Control community has voted on and approved a
> >>> proposal to release Apache Traffic Control 1.8.1-incubating. We now
> >>> kindly request that the Incubator PMC members review and vote on this
> >>> incubator release.
> >>>
> >>> The VOTE RESULT is here:
> >>>
> >>>
> https://lists.apache.org/thread.html/c43fe387586c44e2ecbac1f968a146f9a402a328be0f6e9b296fc3c9@%3Cusers.trafficcontrol.apache.org%3E
> >>>
> >>> The draft release notes (along with links to artifacts,
> >>> signatures/checksums, and updated documentation) can be found here:
> >>>
> >>> http://trafficcontrol.incubator.apache.org/downloads/
> >>
> >> This is a *public* URL.
> >> However the release has not yet been approved, so it must *not* be
> published.
> >> For the same reason, the email regarding the RC should not be sent to
> >> the user list.
> >>
> >> Also the download page points to
> >>
> >> https://dist.apache.org/repos/dist/
> >>
> >> The dist repo is a staging repo only, and must not be used for public
> URLs.
> >>
> >>> The git tag for the repository is "RELEASE-1.8.1-RC0":
> >>>
> https://github.com/apache/incubator-trafficcontrol/releases/tag/RELEASE-1.8.1-RC0
> >>>
> >>> The source distribution (also linked in the release notes) is here:
> >>>
> >>>
> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/1.8.1/RC0/
> >>
> >> That URL is fine for RC vote mails
> >>
> >>> Build instructions are included in the BUILD.md file which is included
> >>> in the source artifact.
> >>>
> >>> Artifacts have been signed with the "[hidden email]" key listed
> in:
> >>>
> >>> https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS
> >>
> >> That URL is fine for RC vote mails
> >>
> >>> Please review and vote:
> >>>
> >>> [ ] +1 Approve the release
> >>> [ ] -1 Don't approve the release (please provide specific comments)
> >>>
> >>> This vote will be open for at least 72 hours.
> >>>
> >>> Thanks,
> >>>
> >>>  Dan Kirkwood <[hidden email]>
> >>>
> >>> ---------------------------------------------------------------------
> >>> To unsubscribe, e-mail: [hidden email]
> >>> For additional commands, e-mail: [hidden email]
> >>>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: [hidden email]
> >> For additional commands, e-mail: [hidden email]
> >>
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [hidden email]
> > For additional commands, e-mail: [hidden email]
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

Justin Mclean
In reply to this post by Dan Kirkwood-2
Hi,

+1 binding

I checked:
- incubating in release name
- signatures and hashes good
- disclaimer exists
- In NOTICE there no need to list the fonts
- LICENSE is good
- Source files have apache headers
- No unexpected binaries files
- Didn’t build a that requires docker

For completeness you may want to include these Apache licensed [3] font files in LICENSE. [1][2]

Thanks,
Justin

1. trafficcontrol-incubating-1.8.1/docs/source/_themes/sphinx_rtd_theme/static/fonts/RobotoSlab-Bold.ttf
2. trafficcontrol-incubating-1.8.1/docs/source/_themes/sphinx_rtd_theme/static/fonts/RobotoSlab-Regular.ttf
3. https://fonts.google.com/specimen/Roboto+Slab


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Traffic Control 1.8.1-incubating (RC0)

Phil Sorber
+1 binding.

Agree with statements made up-thread.

Built all artifacts using docker.

-rw-r--r--. 1 root root  986K Jun 13 00:13 build-traffic_monitor.log
-rw-r--r--. 1 root root  130K Jun 13 00:12 build-traffic_ops.log
-rw-r--r--. 1 root root  205K Jun 13 00:13 build-traffic_portal.log
-rw-r--r--. 1 root root 1019K Jun 13 00:13 build-traffic_router.log
-rw-r--r--. 1 root root   18K Jun 13 00:12 build-traffic_stats.log
-rw-r--r--. 1 root root   51M Jun 13 00:13
traffic_monitor-1.8.1-4642.cc56e763.el7.x86_64.rpm
-rw-r--r--. 1 root root  810K Jun 13 00:12
traffic_ops-1.8.1-4642.cc56e763.el7.src.rpm
-rw-r--r--. 1 root root  5.1M Jun 13 00:12
traffic_ops-1.8.1-4642.cc56e763.el7.x86_64.rpm
-rw-r--r--. 1 root root   22K Jun 13 00:12
traffic_ops_ort-1.8.1-4642.cc56e763.el7.src.rpm
-rw-r--r--. 1 root root   21K Jun 13 00:12
traffic_ops_ort-1.8.1-4642.cc56e763.el7.x86_64.rpm
-rw-r--r--. 1 root root  143K Jun 13 00:13
traffic_portal-1.8.1-4642.cc56e763.el7.src.rpm
-rw-r--r--. 1 root root  3.6M Jun 13 00:13
traffic_portal-1.8.1-4642.cc56e763.el7.x86_64.rpm
-rw-r--r--. 1 root root   19M Jun 13 00:13
traffic_router-1.8.1-4642.cc56e763.el7.x86_64.rpm
-rw-r--r--. 1 root root   38K Jun 13 00:12
traffic_stats-1.8.1-4642.cc56e763.el7.src.rpm
-rw-r--r--. 1 root root  5.0M Jun 13 00:12
traffic_stats-1.8.1-4642.cc56e763.el7.x86_64.rpm

Thanks.

On Sat, Jun 10, 2017 at 6:48 PM Justin Mclean <[hidden email]>
wrote:

> Hi,
>
> +1 binding
>
> I checked:
> - incubating in release name
> - signatures and hashes good
> - disclaimer exists
> - In NOTICE there no need to list the fonts
> - LICENSE is good
> - Source files have apache headers
> - No unexpected binaries files
> - Didn’t build a that requires docker
>
> For completeness you may want to include these Apache licensed [3] font
> files in LICENSE. [1][2]
>
> Thanks,
> Justin
>
> 1.
> trafficcontrol-incubating-1.8.1/docs/source/_themes/sphinx_rtd_theme/static/fonts/RobotoSlab-Bold.ttf
> 2.
> trafficcontrol-incubating-1.8.1/docs/source/_themes/sphinx_rtd_theme/static/fonts/RobotoSlab-Regular.ttf
> 3. https://fonts.google.com/specimen/Roboto+Slab
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>