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

Fixed nightly and weekly builds which use gfbuild.sh #23832

Merged
merged 1 commit into from
Feb 26, 2022

Conversation

pzygielo
Copy link
Contributor

@pzygielo pzygielo commented Feb 26, 2022

@pzygielo pzygielo marked this pull request as ready for review February 26, 2022 07:34
@pzygielo
Copy link
Contributor Author

pzygielo commented Feb 26, 2022

So this working

glassfish/Jenkinsfile

Lines 265 to 266 in a85f4d2

# temporary build of external snapshot dependencies
mvn clean install -f ./snapshots/pom.xml
(without settings.xml used)
regardless of glassfish_build-and-publish-to-eclipse-download failing, just confirms to me that

glassfish/Jenkinsfile

Lines 164 to 165 in a85f4d2

- name: maven-repo-shared-storage
mountPath: /home/jenkins/.m2/repository
can be at least dangerously misleading as for build result.
Re-staged artifacts (as I verified also in #23830) are not refreshed.

I just hope that for release build cache is not used, but this https://ci.eclipse.org/glassfish/view/GlassFish/job/glassfish_1-build-and-stage/25/consoleText

   - mountPath: "/home/jenkins/.m2/repository"
     name: "volume-3"
     readOnly: false

is not reassuring +

with several releases since.

@arjantijms
Copy link
Contributor

Re-staged artifacts (as I verified also in #23830) are not refreshed.

That's indeed a defect. Just the other day I encountered the same issue with re-staging a Mojarra artefact.

Perhaps we need to add the prune-maven-cache functionality to the 1_stage job as an option, and mention it should be enabled at least for releases going to central.

Copy link
Contributor

@arjantijms arjantijms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@arjantijms arjantijms merged commit 9758319 into eclipse-ee4j:master Feb 26, 2022
@pzygielo pzygielo deleted the gfbuild branch February 27, 2022 06:31
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