-
Notifications
You must be signed in to change notification settings - Fork 57
Switched che_version extraction from static master to the source branch #1700
Switched che_version extraction from static master to the source branch #1700
Conversation
@ScrewTSW PR check [build 1969] failed.Depending on failure reason, the artifacts or deployment may not be present. |
ff864e3
to
970856f
Compare
@ScrewTSW PR check [build 1970] failed.Depending on failure reason, the artifacts or deployment may not be present. |
970856f
to
80e3af4
Compare
@ScrewTSW RH-Che [build 1971] has been successfully deployed and tested.deployment: http://rhche-prcheck-1700.devtools-dev.ext.devshift.net |
@@ -37,7 +37,7 @@ echo "Installing all dependencies lasted $instal_dep_duration seconds." | |||
|
|||
export PROJECT_NAMESPACE=prcheck-${RH_PULL_REQUEST_ID} | |||
export DOCKER_IMAGE_TAG="${RH_TAG_DIST_SUFFIX}"-"${RH_PULL_REQUEST_ID}" | |||
CHE_VERSION=$(curl -s https://raw.githubusercontent.com/redhat-developer/rh-che/master/pom.xml | xq -r '.project.parent.version') | |||
CHE_VERSION=$(getVersionFromPom) |
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.
how does getVersionFromPom
work?
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.
the method is being sourced from .ci/functional_tests_util.sh
it's calling
scl enable rh-maven33 "mvn -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn org.apache.maven.plugins:maven-help-plugin:evaluate -q -Dexpression=project.parent.version -DforceStdout"
to get the version from the pom in the currently checked out branch of the project.
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.
If yq
is installed, then xq -r '.project.parent.version' pom.xml
could be a replacement as well.
What does this PR do?
Changed logic for extracting Che version to use the source branch of the PR instead of the master when running prcheck script
What issues does this PR fix or reference?
hotfix - deploying incorrect version of registries for prcheck
How have you tested this PR?
manual testing