Health 1.0 Final by COB tomorrow (CET)

[Heiko Braun]

We just finished the health conference call to review 1.0 RC1 and the only issue identified was security, which we agreed to leave out for 1.0. https://github.com/eclipse/microprofile-health/pull/99 addresses this concern by all security requirements from the spec.

We will leave the RC1 open until tomorrow COB (6PM CET) and then roll the 1.0 Final release. Please let us know if anything jumps to your eye that requires changes for 1.0.

Regards,

Heiko

[Heiko Braun]

Apologies, it should read "by removing all security requirements from the spec".

[John Clingan]

This implies that each vendor can address security in their own way until we follow up with a Health Check.next release. That's an acceptable approach.

Thanks Heiko.

[John D. Ament]

+1

[Kevin Sutter]

Sounds great, Heiko!  Please be aware of the other threads discussing the requirements for MP 1.2 prep...

Thanks, Kevin


On Tuesday, August 29, 2017 at 12:13:45 PM UTC-5, John D. Ament wrote:

+1

[Heiko Braun]

Thanks for the notice Kevin. I've tagged 1.0.0 RC2 in preparation for the IP review and such. If we get a positive conformation, we will use this version to derive 1.0.0

[Kevin Sutter]

Heiko,
As I am prepping for the IP review for the Health component, I see that you have a dual dependency on testng at different levels...  In your tck folder, your pom.xml references both the 6.9.9 and the 6.8.21 versions.  Which one is correct?  Hopefully, you can get by with 6.9.9 since that one is already approved for use.  The 6.8.21 will take an extra item in the CQ process.  Thanks!

<testng.version>6.9.9</testng.version>

<dependency>

<groupId>org.testng</groupId>

<artifactId>testng</artifactId>

<version>${testng.version}</version>

<scope>compile</scope>

</dependency>

<dependency>

<groupId>org.testng</groupId>

<artifactId>testng</artifactId>

<version>6.8.21</version>

</dependency> --  Kevin

[Kevin Sutter]

FYI, Here's the CQ I created for the Health component.  I only found Build and Test dependencies.  We should still figure out the proper answer for the testng version...  Thanks.

https://dev.eclipse.org/ipzilla/show_bug.cgi?id=14059

--  Kevin

[sst...@redhat.com]

I have verified that the TCK runs fine with just the testng 6.9.9 version dependency, and so I pushed that change. I'll leave it to Heiko to verify it with his latest changes to make sure there is no issue.

[Heiko Braun]

Thanks for doing the CQ Kevin. I did struggle wth that yesterday. I'll verify the approved testng version, but wouldn't expect any issues. And thanks Scott, for doing the pre-flight check.

[Heiko Braun]

I've verified that  testng 6.9.9 works as expected.

[Kevin Sutter]

Excellent! Thanks everyone for the quick follow up. 

-- Kevin 

On Aug 31, 2017 02:50, "'Heiko Braun' via Eclipse MicroProfile" <microp...@googlegroups.com> wrote:

I've verified that  testng 6.9.9 works as expected.

--
You received this message because you are subscribed to a topic in the Google Groups "Eclipse MicroProfile" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/microprofile/fLzOysDStvU/unsubscribe.
To unsubscribe from this group and all its topics, send an email to microprofile+unsubscribe@googlegroups.com.
To post to this group, send email to microp...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/microprofile/bbe19f6f-80b1-41d3-80d4-caccd04d398a%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.