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
[INFRA-1953] Make pipeline-library#runATH step runs with old ATH sources and images #84
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I trust you it's working, just wondering if we could simplify the instructions to make it more accessible to new-comers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alecharp proposal and better messages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Keep the containerArgs as it was, add some extra information and fail the build if java11 required but set-java.sh not found
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Failure on Java11 without set-env.sh and just a message on Java8. @olivergondza @batmat
@olivergondza @raul-arabaolaza Does these changes look good to you? Thank you. |
@MRamonLeon it seems good, but why did you change it to invoke each command at a time instead of all of them together? Just out of curiosity. |
@varyvol No specific reason. Probably I forgot to add the && after the if. Good point. |
See Make pipeline-library#runATH step runs with old ATH sources and images
This PR allows you to use the runATH step with old ATH sources and images. With this, your builds will not fail if the pipeline-library is not on sync with the ATH sources and images used.
@batmat @alecharp @raul-arabaolaza @olivergondza