Thanks, Sandeep and Phil! I have verified the double configuration param issue and have a patch ready to go. I will cancel this VOTE and spin a new RC with the fix.
On Wed, Jul 25, 2018 at 2:44 PM, Phil Zampino <pzamp...@apache.org> wrote: > -1 > > I've discovered that conf/gateway-site.xml defines the > gateway.dispatch.whitelist property twice, each with different values. > This may result in the expected behavior, but it's technically incorrect > (properties should not be defined twice) and it's certainly confusing. > > Less concerning is that the CHANGES file does not include any changes from > rc1 --> rc2. > > * Verified GPG/MD5/SHA signatures for Knox source, Knox and KnoxShell > release > packages (zip files) > * Downloaded and built from source (with Java 1.8.0_144) > * Checked LICENSE and NOTICE files > * Checked ISSUES file > * Checked CHANGES file > * Ran KnoxShell samples > * Exercised the Admin UI (provider configuration wizard, descriptor wizard, > etc...) > * Verified the Admin UI issues discovered in RC1 (URL field validation, > topology list refreshing) have been addressed > * Tested service discovery and topology generation > > > On Tue, Jul 24, 2018 at 8:21 PM larry mccay <lmc...@apache.org> wrote: > > > All - > > > > A number of issues were identified and subsequently fixed based > > on testing of RC 1. > > > > Release candidate #2 for the Apache Knox 1.1.0 is available at: > > > > https://dist.apache.org/repos/dist/dev/knox/knox-1.1.0/ > > > > The release candidate is a zip archive of the sources in: > > > > https://git-wip-us.apache.org/repos/asf/knox.git > > Branch v1.1.0 (git checkout -b v1.1.0) > > Tag is v1.1.0-rc2 > > > > The KEYS file for signature validation is available at: > > https://dist.apache.org/repos/dist/release/knox/KEYS > > > > Please vote on releasing this package as Apache Knox 1.1.0. > > The vote is open for the next 72 hours and passes if a majority of at > > least three +1 Apache Knox PMC votes are cast. > > > > [ ] +1 Release this package as Apache Knox 1.1.0 > > [ ] -1 Do not release this package because... > > > > Thanks, > > > > --larry > > > > >
