Re: Github actions - Integration tests
Thanks Dan. I choose to remove the test for now. Here is the PR that enables integration tests on linux platform. https://github.com/apache/geode/pull/7872 On Wed, 8 Feb 2023 at 08:34, Dan Smith wrote: > That ConcurrentTestRunnerTest is testing a test utility to see if it can > catch race conditions. We didn't end up using that utility as much as I > thought we might. You could consider just ignoring/deleting that test. > > -Dan > > From: Sai Boorlagadda > Sent: Tuesday, February 7, 2023 5:56 PM > To: dev@geode.apache.org > Subject: Re: Github actions - Integration tests > > !! External Email > > Integration tests pass consistently after configuring the > `--max-workers=12` and by not setting the `testMaxParallelForks` (so it > defaults to max-workers/4), except for one test[1] which is a test called > ConcurrentTestRunnerTest to test the ConcurrentTestRunner. > > This test is expected to fail when an atomic int is modified in parallel > threads. This ensures ConcurrentTestRunner is running in parallel, but > unfortunately, Github actions may be running with a single core > (availableProcessors = 2). The logs are available in the summary section > and saved for up to 5 days. > > Though the available processors are 2 and it seems the atomic int is never > got executed in parallel. Any thoughts? > > @Dan Tagging you as you were the original author to little bit understand > about this test. > > Available processors: 2 > Available processors: 2 > Available processors: 2 > Available processors: 2 > > > [1] > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Factions%2Fruns%2F4119919022=05%7C01%7Cdasmith%40vmware.com%7C7b2cf42318694513e67208db0977c63d%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638114182273485483%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=unVvHQh5KW2a9ELRAduFXhGS%2B3ahspCpq1NGP9IFAq8%3D=0 > > > > On Mon, 23 Jan 2023 at 11:05, Sai Boorlagadda > wrote: > > > After working through unit tests and the builds are passing. I am working > > on adding an integration step to the build pipeline. > > > > Looks like there are a few (4 tests) failing[1] on the integration suite. > > Let me know if anyone interested to look into them. It would really good > to > > get integration tests enabled on develop. > > > > Below 4 tests needs to be analyzed and fixed: > > 1) LocatorLauncherJmxSerialFilterPropertyBlankIntegrationTest > > > startDoesNotConfigureJmxSerialFilter_whenPropertyIsBlank_onJava8 > > 2) LocatorLauncherGlobalSerialFilterPropertyEmptyIntegrationTest > > > startDoesNotConfigureGlobalSerialFilter_whenPropertyIsEmpty > > 3) VersionCommandJUnitTest > initializationError > > 4) LoadClusterConfigFromDirInt > canStartWithDeployedJarInClusterConfig > > > > 1. > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Factions%2Fruns%2F3983609395%2Fjobs%2F6829148633=05%7C01%7Cdasmith%40vmware.com%7C7b2cf42318694513e67208db0977c63d%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638114182273485483%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=jIvQuxFsYNFyXEyUuuMVHPdrodXymM9IGYaw4lWIZ2k%3D=0 > > > > Sai > > > > !! External Email: This email originated from outside of the organization. > Do not click links or open attachments unless you recognize the sender. >
Re: Github actions - Integration tests
That ConcurrentTestRunnerTest is testing a test utility to see if it can catch race conditions. We didn't end up using that utility as much as I thought we might. You could consider just ignoring/deleting that test. -Dan From: Sai Boorlagadda Sent: Tuesday, February 7, 2023 5:56 PM To: dev@geode.apache.org Subject: Re: Github actions - Integration tests !! External Email Integration tests pass consistently after configuring the `--max-workers=12` and by not setting the `testMaxParallelForks` (so it defaults to max-workers/4), except for one test[1] which is a test called ConcurrentTestRunnerTest to test the ConcurrentTestRunner. This test is expected to fail when an atomic int is modified in parallel threads. This ensures ConcurrentTestRunner is running in parallel, but unfortunately, Github actions may be running with a single core (availableProcessors = 2). The logs are available in the summary section and saved for up to 5 days. Though the available processors are 2 and it seems the atomic int is never got executed in parallel. Any thoughts? @Dan Tagging you as you were the original author to little bit understand about this test. Available processors: 2 Available processors: 2 Available processors: 2 Available processors: 2 [1] https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Factions%2Fruns%2F4119919022=05%7C01%7Cdasmith%40vmware.com%7C7b2cf42318694513e67208db0977c63d%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638114182273485483%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=unVvHQh5KW2a9ELRAduFXhGS%2B3ahspCpq1NGP9IFAq8%3D=0 On Mon, 23 Jan 2023 at 11:05, Sai Boorlagadda wrote: > After working through unit tests and the builds are passing. I am working > on adding an integration step to the build pipeline. > > Looks like there are a few (4 tests) failing[1] on the integration suite. > Let me know if anyone interested to look into them. It would really good to > get integration tests enabled on develop. > > Below 4 tests needs to be analyzed and fixed: > 1) LocatorLauncherJmxSerialFilterPropertyBlankIntegrationTest > > startDoesNotConfigureJmxSerialFilter_whenPropertyIsBlank_onJava8 > 2) LocatorLauncherGlobalSerialFilterPropertyEmptyIntegrationTest > > startDoesNotConfigureGlobalSerialFilter_whenPropertyIsEmpty > 3) VersionCommandJUnitTest > initializationError > 4) LoadClusterConfigFromDirInt > canStartWithDeployedJarInClusterConfig > > 1. > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Factions%2Fruns%2F3983609395%2Fjobs%2F6829148633=05%7C01%7Cdasmith%40vmware.com%7C7b2cf42318694513e67208db0977c63d%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638114182273485483%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C=jIvQuxFsYNFyXEyUuuMVHPdrodXymM9IGYaw4lWIZ2k%3D=0 > > Sai > !! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
Re: [DRAFT] Geode Board Report For February 2023
Hey Mark, Thanks for this. Do we have commit and mailing list statistics (I usually at least look at them for the Pivot report)? It would help me to assess the differences since the last report. Thanks, ~Roger Whitcomb > On Feb 7, 2023, at 7:33 AM, Mark Bretl wrote: > > Please take a look at the board report draft, feedback is welcome. I will > be sending this to the board EOD Wednesday. Thanks! > > --Mark > > ## Description: > The mission of Apache Geode is the creation and maintenance of software > related > to a data management platform that provides real-time, consistent access to > data-intensive applications throughout widely distributed cloud > architectures. > > ## Issues: > No immediate issues. The PMC now has enough members to keep the project > going and out of the attic. > > ## Membership Data: > Apache Geode was founded 2016-11-15 (6 years ago) > There are currently 118 committers and 31 PMC members in this project. > The Committer-to-PMC ratio is roughly 4:1. > > Community changes, past quarter: > - Calvin Kirs was added to the PMC on 2022-11-14 > - Roger Lee Whitcomb was added to the PMC on 2022-11-13 > - Calvin Kirs was added as committer on 2022-11-15 > - Roger Lee Whitcomb was added as committer on 2022-11-14 > > ## Project Activity: > 1.15.1 was released on 2022-10-10. > > ## Community Health: > With the exit of the two corporate sponsors back in November, the community > is beginning the rebuilding process. There has been good response from the > Apache Membership to join the project and help through the transition. The > community is currently focused on transitioning continuous integration > tooling from a previously sponsored Concourse CI to GitHub Actions. This > migration will also help remove dependencies on sponsored infrastructure in > the future as well.