Buildbot build failing for a long time

[Gavin McDonald]

Hi All,

https://ci2.apache.org/#/builders/72

Has been failing for a long time. can someone investigate please, if
necessary open an INFRA ticket if something is required our end.

Thanks

-- 

*Gavin McDonald*
Systems Administrator
ASF Infrastructure Team

Re: Buildbot build failing for a long time

[Sai Boorlagadda]

Thanks Gavin,

This nightly build has been failing for quite a while. I will see
whats going on.

Devs, Do we know about this nightly build? Any info on the setup and how it
is configured?

Sai

On Mon, 23 Jan 2023 at 00:58, Gavin McDonald  wrote:

> Hi All,
>
> https://ci2.apache.org/#/builders/72
>
> Has been failing for a long time. can someone investigate please, if
> necessary open an INFRA ticket if something is required our end.
>
> Thanks
>
> --
>
> *Gavin McDonald*
> Systems Administrator
> ASF Infrastructure Team
>

Re: Pullrequest required checks

[Michael Oleske]

I believe the required checks are setup in the .asf.yaml file 
here.  I would hope 
just deleting those lines would remove the existing checks and in your migrate 
to github actions pr you could change the required checks.  I would hope this 
doesn't need a vote as those current required check's pipelines seem to all 
have been removed.

-michael

From: Sai Boorlagadda 
Sent: Sunday, January 22, 2023 08:28
To: dev@geode.apache.org 
Subject: Pullrequest required checks

!! External Email

Could someone have info regarding how the required checks are enabled on a
Pull Request?

In the process of setting up the Github Actions, I see there are still
Concourse CI checks as "required" eg: on this PR[1]

[1] 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7870&data=05%7C01%7Cmoleske%40vmware.com%7C64eff0f8ae934a67e2b608dafc95eb87%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638100018086696005%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=4w0XLfg0ks8bVEwQRYIfsl5q1CKGpXy0pAUl7KUO2io%3D&reserved=0

Thanks
Sai

!! External Email: This email originated from outside of the organization. Do 
not click links or open attachments unless you recognize the sender.

Re: Pullrequest required checks

[Michael Oleske]

Quick addendum, it looks like you would have to update the master branch as 
well as develop.  As the master 
branch has more 
required checks for reason.

-michael

From: Michael Oleske 
Sent: Monday, January 23, 2023 09:20
To: dev@geode.apache.org 
Subject: Re: Pullrequest required checks

!! External Email

I believe the required checks are setup in the .asf.yaml file 
here.
  I would hope just deleting those lines would remove the existing checks and 
in your migrate to github actions pr you could change the required checks.  I 
would hope this doesn't need a vote as those current required check's pipelines 
seem to all have been removed.

-michael

From: Sai Boorlagadda 
Sent: Sunday, January 22, 2023 08:28
To: dev@geode.apache.org 
Subject: Pullrequest required checks

!! External Email

Could someone have info regarding how the required checks are enabled on a
Pull Request?

In the process of setting up the Github Actions, I see there are still
Concourse CI checks as "required" eg: on this PR[1]

[1] 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7870&data=05%7C01%7Cmoleske%40vmware.com%7Ca08669bc99d24832dd5e08dafd662876%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638100912455129730%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=4OndODLKhyumfbU330712kWLSgrc%2FkcGRh%2Fu4GQeLcs%3D&reserved=0

Thanks
Sai

!! External Email: This email originated from outside of the organization. Do 
not click links or open attachments unless you recognize the sender.

Re: Buildbot build failing for a long time

[Gavin McDonald]

On 2023/01/23 14:59:56 Sai Boorlagadda wrote:
> Thanks Gavin,
> 
> This nightly build has been failing for quite a while. I will see
> whats going on.
> 
> Devs, Do we know about this nightly build? Any info on the setup and how it
> is configured?

Your config file lives at:-

https://github.com/apache/infrastructure-bb2/blob/master/geode.py 

any changes to it should go live immediately.

I placed the file there as it was transitioned from the old Buildbot server to 
the new.

You can either try and fix it, or delete the file which will remove it from the 
Buildbot config.

Happy to help in any way I can if you feel you would like it fixed.

Gav...

> 
> Sai
> 
> On Mon, 23 Jan 2023 at 00:58, Gavin McDonald  wrote:
> 
> > Hi All,
> >
> > https://ci2.apache.org/#/builders/72
> >
> > Has been failing for a long time. can someone investigate please, if
> > necessary open an INFRA ticket if something is required our end.
> >
> > Thanks
> >
> > --
> >
> > *Gavin McDonald*
> > Systems Administrator
> > ASF Infrastructure Team
> >
> 

Github actions - Integration tests

[Sai Boorlagadda]

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://github.com/apache/geode/actions/runs/3983609395/jobs/6829148633

Sai

Re: Pullrequest required checks

[Sai Boorlagadda]

Thanks, Owen and Michael. Will check the .asf.yaml file.

On Mon, 23 Jan 2023 at 10:05, Michael Oleske 
wrote:

> Quick addendum, it looks like you would have to update the master branch
> as well as develop.  As the master branch<
> https://github.com/apache/geode/blob/master/.asf.yaml#L39> has more
> required checks for reason.
>
> -michael
> 
> From: Michael Oleske 
> Sent: Monday, January 23, 2023 09:20
> To: dev@geode.apache.org 
> Subject: Re: Pullrequest required checks
>
> !! External Email
>
> I believe the required checks are setup in the .asf.yaml file here<
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fblob%2Fdevelop%2F.asf.yaml%23L35&data=05%7C01%7Cmoleske%40vmware.com%7Ca08669bc99d24832dd5e08dafd662876%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638100912455129730%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=5bKCtDt7LEO1WxYwtH5rtFCch5wJzjuk6uHGx3yQ9lY%3D&reserved=0>.
> I would hope just deleting those lines would remove the existing checks and
> in your migrate to github actions pr you could change the required checks.
> I would hope this doesn't need a vote as those current required check's
> pipelines seem to all have been removed.
>
> -michael
> 
> From: Sai Boorlagadda 
> Sent: Sunday, January 22, 2023 08:28
> To: dev@geode.apache.org 
> Subject: Pullrequest required checks
>
> !! External Email
>
> Could someone have info regarding how the required checks are enabled on a
> Pull Request?
>
> In the process of setting up the Github Actions, I see there are still
> Concourse CI checks as "required" eg: on this PR[1]
>
> [1]
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F7870&data=05%7C01%7Cmoleske%40vmware.com%7Ca08669bc99d24832dd5e08dafd662876%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C638100912455129730%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=4OndODLKhyumfbU330712kWLSgrc%2FkcGRh%2Fu4GQeLcs%3D&reserved=0
>
> Thanks
> Sai
>
> !! External Email: This email originated from outside of the organization.
> Do not click links or open attachments unless you recognize the sender.
>

isolated test plugin

[Sai Boorlagadda]

Could someone provide context on how integration & dunit tests are run in
CI?

I remember geode had dockerized test plugin that isolates test execution.
But looking at this PR[1] it seems we removed usage of that plugin and have
a customized mechanism to allocate ports so that we don't need to run tests
in docker containers.

But the CI source (execute_tests.sh) seems to still pass docker
image -PdunitDockerUser=geode -PdunitDockerImage=\$(docker images. So do we
need to run these tests in docker containers?

1. https://github.com/apache/geode/pull/6720

Sai