Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip git gc on aix and master/built-in #21034

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

AdamBrousseau
Copy link
Contributor

Issue #20346

@AdamBrousseau
Copy link
Contributor Author

@pshipton
Copy link
Member

Should I be concerned about all the repack errors in the test run?

@pshipton
Copy link
Member

Also, should we test this on AIX?

@AdamBrousseau
Copy link
Contributor Author

The test run was on aix.
I reran another one one the failed machines with CLEAN_CACHE first set.
https://openj9-jenkins.osuosl.org/job/Update-Reference-Repos-Linux-x86/24/consoleFull

@pshipton
Copy link
Member

pshipton commented Jan 28, 2025

Why did the 24 job fail?

Can you pls explain how this works, wrt running on master. When does it run on master and when doesn't it?

@AdamBrousseau
Copy link
Contributor Author

24 failed on aix-11. Not sure why. It was during the fetch. Might have been git out of memory.
Rerun on aix-11 here https://openj9-jenkins.osuosl.org/job/Update-Reference-Repos-Linux-x86/25/consoleFull passed.

For master, we will run the cache update but no git gc since it appears that the gc is what is causing jenkins to fall over. Also no git gc on aix since it doesn't seem to work well either.

Build 23 shows a sucessful run on master without a gc
https://openj9-jenkins.osuosl.org/job/Update-Reference-Repos-Linux-x86/23/consoleFull

@pshipton
Copy link
Member

I understand the logic of the change, what I don't understand is when we run on master, or not. Is master just one of the x86 machines we run on when updating x86?

@AdamBrousseau
Copy link
Contributor Author

Master falls under the SETUP nodes list (worker). We only run the setup nodes (true) on the xlinux job ever since we divided by platform.

@pshipton pshipton merged commit 28c9282 into eclipse-openj9:master Jan 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants