Re: [PROPOASAL] backport GEODE-8144

2020-05-28 Thread Bill Burcham
+1

On Thu, May 28, 2020 at 11:49 AM Ernie Burghardt 
wrote:

> +1
>
> On 5/27/20, 1:35 PM, "Bruce Schuchardt"  wrote:
>
> This ticket has two PRs.  One passed all normal CI runs but then we
> hit a faulty test that failed on a Windows machine.  There’s a new PR that
> fixes that test & has been merged.
>
> The PRs fixe endpoint verification problems in servers and locators.
> Without this fix it’s not possible to boot a locator/server with endpoint
> verification enabled.
>
>
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5164data=02%7C01%7Cburghardte%40vmware.com%7Ca40e55b9f1e54af3b44c08d8027d8349%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637262085382722407sdata=GBAPvBI8qMtsCObX6GYrowVlV9%2FmWq%2BV0yQLLKfpA%2BQ%3Dreserved=0
> fixes the failing test
>
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5131data=02%7C01%7Cburghardte%40vmware.com%7Ca40e55b9f1e54af3b44c08d8027d8349%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637262085382722407sdata=X1TE4qizOKk400%2BR%2FUaOA%2FDm6FeVrJPAPMn%2FIl1ooqo%3Dreserved=0
> is the original PR
>
> both have been merged to develop
>
>
>
>


Re: [PROPOASAL] backport GEODE-8144

2020-05-28 Thread Ernie Burghardt
+1

On 5/27/20, 1:35 PM, "Bruce Schuchardt"  wrote:

This ticket has two PRs.  One passed all normal CI runs but then we hit a 
faulty test that failed on a Windows machine.  There’s a new PR that fixes that 
test & has been merged.

The PRs fixe endpoint verification problems in servers and locators.  
Without this fix it’s not possible to boot a locator/server with endpoint 
verification enabled.


https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5164data=02%7C01%7Cburghardte%40vmware.com%7Ca40e55b9f1e54af3b44c08d8027d8349%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637262085382722407sdata=GBAPvBI8qMtsCObX6GYrowVlV9%2FmWq%2BV0yQLLKfpA%2BQ%3Dreserved=0
 fixes the failing test

https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5131data=02%7C01%7Cburghardte%40vmware.com%7Ca40e55b9f1e54af3b44c08d8027d8349%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637262085382722407sdata=X1TE4qizOKk400%2BR%2FUaOA%2FDm6FeVrJPAPMn%2FIl1ooqo%3Dreserved=0
 is the original PR

both have been merged to develop





Re: [PROPOASAL] backport GEODE-8144

2020-05-28 Thread Dave Barnes
I'm going to spend a release-manager +1 to put this proposal over the top.
Please merge this fix into support/1.13, Bruce.
Thanks,
Dave

On Thu, May 28, 2020 at 7:52 AM Udo Kohlmeyer  wrote:

> +1
> On May 27, 2020, 1:35 PM -0700, Bruce Schuchardt ,
> wrote:
> This ticket has two PRs. One passed all normal CI runs but then we hit a
> faulty test that failed on a Windows machine. There’s a new PR that fixes
> that test & has been merged.
>
> The PRs fixe endpoint verification problems in servers and locators.
> Without this fix it’s not possible to boot a locator/server with endpoint
> verification enabled.
>
>
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5164data=02%7C01%7Cudo%40vmware.com%7C0796e6c983d444e87feb08d8027d7ce8%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637262085284295283sdata=sqhirE3DudI00gOpLHlGYTHG9T8GPNVaJK7GB6rSl%2Bc%3Dreserved=0
> fixes the failing test
>
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5131data=02%7C01%7Cudo%40vmware.com%7C0796e6c983d444e87feb08d8027d7ce8%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637262085284305283sdata=BruRleV2p%2BRVg5mWBEt8LPWrCsMKUClogFX8VInanmc%3Dreserved=0
> is the original PR
>
> both have been merged to develop
>
>
>


Re: [PROPOASAL] backport GEODE-8144

2020-05-28 Thread Udo Kohlmeyer
+1
On May 27, 2020, 1:35 PM -0700, Bruce Schuchardt , wrote:
This ticket has two PRs. One passed all normal CI runs but then we hit a faulty 
test that failed on a Windows machine. There’s a new PR that fixes that test & 
has been merged.

The PRs fixe endpoint verification problems in servers and locators. Without 
this fix it’s not possible to boot a locator/server with endpoint verification 
enabled.

https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5164data=02%7C01%7Cudo%40vmware.com%7C0796e6c983d444e87feb08d8027d7ce8%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637262085284295283sdata=sqhirE3DudI00gOpLHlGYTHG9T8GPNVaJK7GB6rSl%2Bc%3Dreserved=0
 fixes the failing test
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5131data=02%7C01%7Cudo%40vmware.com%7C0796e6c983d444e87feb08d8027d7ce8%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637262085284305283sdata=BruRleV2p%2BRVg5mWBEt8LPWrCsMKUClogFX8VInanmc%3Dreserved=0
 is the original PR

both have been merged to develop




Re: [PROPOASAL] backport GEODE-8144

2020-05-28 Thread Joris Melchior
+1

From: Bruce Schuchardt 
Sent: May 27, 2020 16:35
To: dev@geode.apache.org 
Subject: [PROPOASAL] backport GEODE-8144

This ticket has two PRs.  One passed all normal CI runs but then we hit a 
faulty test that failed on a Windows machine.  There’s a new PR that fixes that 
test & has been merged.

The PRs fixe endpoint verification problems in servers and locators.  Without 
this fix it’s not possible to boot a locator/server with endpoint verification 
enabled.

https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5164data=02%7C01%7Cjmelchior%40vmware.com%7C9bfb066e00084476b4df08d8027d7d39%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637262085285945188sdata=wjue8o5303%2FgfCWHRSsDTNRj8uJGshhi1XTTtUM59Ps%3Dreserved=0
 fixes the failing test
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5131data=02%7C01%7Cjmelchior%40vmware.com%7C9bfb066e00084476b4df08d8027d7d39%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637262085285945188sdata=kKtob9T0vSphqUgKIKZ4XCAKDMDlC9HXnFX3wKIDXNE%3Dreserved=0
 is the original PR

both have been merged to develop




[PROPOASAL] backport GEODE-8144

2020-05-27 Thread Bruce Schuchardt
This ticket has two PRs.  One passed all normal CI runs but then we hit a 
faulty test that failed on a Windows machine.  There’s a new PR that fixes that 
test & has been merged.

The PRs fixe endpoint verification problems in servers and locators.  Without 
this fix it’s not possible to boot a locator/server with endpoint verification 
enabled.

https://github.com/apache/geode/pull/5164 fixes the failing test
https://github.com/apache/geode/pull/5131 is the original PR

both have been merged to develop