Skip to content

Commit 92e3581

Browse files
simplify env vars
1 parent e82679a commit 92e3581

File tree

1 file changed

+7
-12
lines changed

1 file changed

+7
-12
lines changed

.github/workflows/precommit.yml

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,9 @@ env:
3737
TESTS_FILTER: 'checkstyle,javac,javadoc,pylint,shellcheck,shelldocs,blanks,perlcritic,ruby-lint,rubocop'
3838
SUREFIRE_FIRST_PART_FORK_COUNT: '1C'
3939
SUREFIRE_SECOND_PART_FORK_COUNT: '1C'
40-
SUREFIRE_REUSE_FORKS: 'true'
4140
ASF_NIGHTLIES: 'https://nightlies.apache.org'
41+
EXCLUDE_TESTS_URL: 'https://ci-hbase.apache.org/job/HBase-Find-Flaky-Tests/job/${{ github.base_ref }}/lastSuccessfulBuild/artifact/output/excludes'
42+
BUILD_URL_ARTIFACTS: '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}'
4243

4344
jobs:
4445
precommit-checks:
@@ -72,32 +73,26 @@ jobs:
7273
- name: Setup environment paths
7374
id: setup-env
7475
run: |
75-
WORKDIR="${GITHUB_WORKSPACE}/${{ matrix.workdir }}"
76-
echo "WORKDIR=${WORKDIR}" >> $GITHUB_ENV
77-
echo "YETUSDIR=${WORKDIR}/${YETUS_REL}" >> $GITHUB_ENV
76+
WORKDIR="${{ github.workspace }}/${{ matrix.workdir }}"
77+
7878
echo "SOURCEDIR=${WORKDIR}/${SRC_REL}" >> $GITHUB_ENV
7979
echo "PATCHDIR=${WORKDIR}/${PATCH_REL}" >> $GITHUB_ENV
8080
echo "DOCKERFILE=${WORKDIR}/${SRC_REL}/dev-support/docker/Dockerfile" >> $GITHUB_ENV
81+
echo "YETUSDIR=${WORKDIR}/yetus" >> $GITHUB_ENV
8182
echo "CACHE_APACHE_ARTIFACT=${WORKDIR}/${SRC_REL}/dev-support/jenkins-scripts/cache-apache-project-artifact.sh" >> $GITHUB_ENV
8283
echo "YETUS_DRIVER=${WORKDIR}/${SRC_REL}/dev-support/jenkins_precommit_github_yetus.sh" >> $GITHUB_ENV
8384
echo "HIDE_OLD_COMMENTS=${WORKDIR}/${SRC_REL}/dev-support/gh_hide_old_comments.sh" >> $GITHUB_ENV
8485
echo "ASF_NIGHTLIES_CHECK_BASE=${ASF_NIGHTLIES}/hbase/${{ github.repository }}/${{ github.run_id }}/${{ matrix.workdir }}/${PATCH_REL}" >> $GITHUB_ENV
85-
echo "JENKINS_URL=https://ci-hbase.apache.org" >> $GITHUB_ENV
86-
echo "EXCLUDE_TESTS_URL=${JENKINS_URL}/job/HBase-Find-Flaky-Tests/job/${{ github.base_ref }}/lastSuccessfulBuild/artifact/output/excludes"
87-
88-
# Set matrix-specific environment variables
8986
echo "PLUGINS=${{ matrix.plugins }}" >> $GITHUB_ENV
9087
echo "SET_JAVA_HOME=${{ matrix.java-home }}" >> $GITHUB_ENV
9188
echo "HADOOP_PROFILE=${{ matrix.hadoop-profile }}" >> $GITHUB_ENV
92-
echo "WORKDIR_REL=${{ matrix.workdir }}" >> $GITHUB_ENV
89+
echo "WORKDIR_REL=${WORKDIR}" >> $GITHUB_ENV
9390
echo "SKIP_ERRORPRONE=${{ matrix.skip-errorprone }}" >> $GITHUB_ENV
94-
echo "BUILD_URL_ARTIFACTS=${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}" >> $GITHUB_ENV
9591
96-
# Create working directories
9792
mkdir -p "${WORKDIR}"
9893
mkdir -p "${WORKDIR}/${SRC_REL}"
99-
mkdir -p "${WORKDIR}/${YETUS_REL}"
10094
mkdir -p "${WORKDIR}/${PATCH_REL}"
95+
mkdir -p "${WORKDIR}/${YETUS_REL}"
10196
10297
- name: Download and extract Apache Yetus
10398
working-directory: ${{ env.YETUSDIR }}

0 commit comments

Comments
 (0)