Update coverage information (#4808)
* Try merging with the queue * Provide github token * Fix syntax * Get GITHUB_TOKEN from vault * Add the PR label * update coverage information --------- Co-authored-by: Arseniy Zaostrovnykh <necto.ne@gmail.com> Co-authored-by: SonarTech <sonartech@sonarsource.com>
This commit is contained in:
parent
dd52d59602
commit
e240ba5d73
142
.github/workflows/update_coverage.yml
vendored
142
.github/workflows/update_coverage.yml
vendored
@ -31,89 +31,99 @@ jobs:
|
|||||||
path: 'rspec'
|
path: 'rspec'
|
||||||
token: ${{ fromJSON(steps.secrets.outputs.vault).coverage_github_token }}
|
token: ${{ fromJSON(steps.secrets.outputs.vault).coverage_github_token }}
|
||||||
|
|
||||||
- uses: actions/setup-python@v4
|
# - uses: actions/setup-python@v4
|
||||||
with:
|
# with:
|
||||||
python-version: '3.9'
|
# python-version: '3.9'
|
||||||
|
|
||||||
- name: 'Install Pipenv'
|
# - name: 'Install Pipenv'
|
||||||
run: pip install pipenv
|
# run: pip install pipenv
|
||||||
|
|
||||||
- name: 'Install coverage script dependencies'
|
# - name: 'Install coverage script dependencies'
|
||||||
working-directory: 'rspec/rspec-tools'
|
# working-directory: 'rspec/rspec-tools'
|
||||||
run: |
|
# run: |
|
||||||
pipenv --python python3.9 install
|
# pipenv --python python3.9 install
|
||||||
|
|
||||||
- name: 'Regenerate coverage information'
|
# - name: 'Regenerate coverage information'
|
||||||
env:
|
# env:
|
||||||
GITHUB_TOKEN: ${{ fromJSON(steps.secrets.outputs.vault).coverage_github_token }}
|
# GITHUB_TOKEN: ${{ fromJSON(steps.secrets.outputs.vault).coverage_github_token }}
|
||||||
id: gen-coverage
|
# id: gen-coverage
|
||||||
working-directory: 'rspec/rspec-tools'
|
# working-directory: 'rspec/rspec-tools'
|
||||||
run: |
|
# run: |
|
||||||
pipenv run rspec-tools update-coverage --rulesdir ../rules
|
# pipenv run rspec-tools update-coverage --rulesdir ../rules
|
||||||
mv ./covered_rules.json ../frontend/public/covered_rules.json
|
# mv ./covered_rules.json ../frontend/public/covered_rules.json
|
||||||
if git diff --exit-code ../frontend/public/covered_rules.json; then
|
# if git diff --exit-code ../frontend/public/covered_rules.json; then
|
||||||
echo "new_coverage=false" >> "$GITHUB_OUTPUT"
|
# echo "new_coverage=false" >> "$GITHUB_OUTPUT"
|
||||||
else
|
# else
|
||||||
echo "new_coverage=true" >> "$GITHUB_OUTPUT"
|
# echo "new_coverage=true" >> "$GITHUB_OUTPUT"
|
||||||
fi
|
# fi
|
||||||
|
|
||||||
- name: 'Cancel if coverage did not change'
|
# - name: 'Cancel if coverage did not change'
|
||||||
if: steps.gen-coverage.outputs.new_coverage != 'true'
|
# if: steps.gen-coverage.outputs.new_coverage != 'true'
|
||||||
uses: andymckay/cancel-action@0.2
|
# uses: andymckay/cancel-action@0.2
|
||||||
|
|
||||||
- name: 'Push the updated coverage file to a new branch'
|
- name: 'Push the updated coverage file to a new branch'
|
||||||
id: create-temp-branch
|
id: create-temp-branch
|
||||||
if: steps.gen-coverage.outputs.new_coverage == 'true'
|
# if: steps.gen-coverage.outputs.new_coverage == 'true'
|
||||||
working-directory: 'rspec'
|
working-directory: 'rspec'
|
||||||
run: |
|
run: |
|
||||||
git config --global user.name "SonarTech"
|
git config --global user.name "SonarTech"
|
||||||
git config --global user.email "sonartech@sonarsource.com"
|
git config --global user.email "sonartech@sonarsource.com"
|
||||||
git checkout -b $TMP_BRANCH
|
git checkout -b $TMP_BRANCH
|
||||||
|
echo "" >> frontend/public/covered_rules.json # TODO remove this line
|
||||||
git add frontend/public/covered_rules.json
|
git add frontend/public/covered_rules.json
|
||||||
git commit -m "update coverage information"
|
git commit -m "update coverage information"
|
||||||
git push --force-with-lease origin $TMP_BRANCH
|
git push --force-with-lease origin $TMP_BRANCH
|
||||||
|
|
||||||
- name: 'Wait for CI to succeed'
|
# - name: 'Wait for CI to succeed'
|
||||||
if: steps.gen-coverage.outputs.new_coverage == 'true'
|
# if: steps.gen-coverage.outputs.new_coverage == 'true'
|
||||||
uses: fountainhead/action-wait-for-check@v1.0.0
|
# uses: fountainhead/action-wait-for-check@v1.0.0
|
||||||
id: wait-for-build
|
# id: wait-for-build
|
||||||
with:
|
# with:
|
||||||
token: ${{ secrets.GITHUB_TOKEN }}
|
# token: ${{ secrets.GITHUB_TOKEN }}
|
||||||
checkName: all_required_checks
|
# checkName: all_required_checks
|
||||||
ref: ${{ env.TMP_BRANCH }}
|
# ref: ${{ env.TMP_BRANCH }}
|
||||||
timeoutSeconds: 2400
|
# timeoutSeconds: 2400
|
||||||
intervalSeconds: 30
|
# intervalSeconds: 30
|
||||||
|
|
||||||
- name: 'Push the updated coverage to master'
|
- name: 'Create a PR'
|
||||||
if: |
|
id: crate-github-pr
|
||||||
steps.gen-coverage.outputs.new_coverage == 'true' &&
|
|
||||||
steps.wait-for-build.outputs.conclusion == 'success' &&
|
|
||||||
(github.event_name != 'workflow_dispatch' || github.ref == format('refs/heads/{0}', github.event.repository.default_branch))
|
|
||||||
working-directory: 'rspec'
|
working-directory: 'rspec'
|
||||||
run: |
|
|
||||||
git checkout master
|
|
||||||
git merge $TMP_BRANCH
|
|
||||||
git push origin master
|
|
||||||
|
|
||||||
- name: 'Delete the temporary branch'
|
|
||||||
if: always() && steps.create-temp-branch.conclusion == 'success'
|
|
||||||
uses: dawidd6/action-delete-branch@v3
|
|
||||||
with:
|
|
||||||
github_token: ${{ secrets.GITHUB_TOKEN }}
|
|
||||||
branches: ${{ env.TMP_BRANCH}}
|
|
||||||
|
|
||||||
- name: 'Fail if the change breaks CI'
|
|
||||||
if: |
|
|
||||||
steps.gen-coverage.outputs.new_coverage == 'true' &&
|
|
||||||
steps.wait-for-build.outputs.conclusion != 'success'
|
|
||||||
run: exit 1
|
|
||||||
|
|
||||||
- name: 'Notify on slack about the failure'
|
|
||||||
if: ${{ failure() }}
|
|
||||||
env:
|
env:
|
||||||
SLACK_API_TOKEN: ${{ fromJSON(steps.secrets.outputs.vault).slack_token }}
|
GH_TOKEN: ${{ fromJSON(steps.secrets.outputs.vault).coverage_github_token }}
|
||||||
working-directory: 'rspec/rspec-tools'
|
|
||||||
run: |
|
run: |
|
||||||
pipenv run rspec-tools notify-failure-on-slack \
|
PR_URL=$(gh pr create --head ${{ env.TMP_BRANCH }} --title "Update coverage information" --body "" --label "rspec system")
|
||||||
--message "ERROR: failed to update rule coverage. See https://github.com/SonarSource/rspec/actions/runs/$GITHUB_RUN_ID" \
|
gh pr merge $PR_URL
|
||||||
--channel team-analysis-rspec
|
|
||||||
|
# - name: 'Push the updated coverage to master'
|
||||||
|
# if: |
|
||||||
|
# steps.gen-coverage.outputs.new_coverage == 'true' &&
|
||||||
|
# steps.wait-for-build.outputs.conclusion == 'success' &&
|
||||||
|
# (github.event_name != 'workflow_dispatch' || github.ref == format('refs/heads/{0}', github.event.repository.default_branch))
|
||||||
|
# working-directory: 'rspec'
|
||||||
|
# run: |
|
||||||
|
# git checkout master
|
||||||
|
# git merge $TMP_BRANCH
|
||||||
|
# git push origin master
|
||||||
|
|
||||||
|
# - name: 'Delete the temporary branch'
|
||||||
|
# if: always() && steps.create-temp-branch.conclusion == 'success'
|
||||||
|
# uses: dawidd6/action-delete-branch@v3
|
||||||
|
# with:
|
||||||
|
# github_token: ${{ secrets.GITHUB_TOKEN }}
|
||||||
|
# branches: ${{ env.TMP_BRANCH}}
|
||||||
|
|
||||||
|
# - name: 'Fail if the change breaks CI'
|
||||||
|
# if: |
|
||||||
|
# steps.gen-coverage.outputs.new_coverage == 'true' &&
|
||||||
|
# steps.wait-for-build.outputs.conclusion != 'success'
|
||||||
|
# run: exit 1
|
||||||
|
|
||||||
|
# - name: 'Notify on slack about the failure'
|
||||||
|
# if: ${{ failure() }}
|
||||||
|
# env:
|
||||||
|
# SLACK_API_TOKEN: ${{ fromJSON(steps.secrets.outputs.vault).slack_token }}
|
||||||
|
# working-directory: 'rspec/rspec-tools'
|
||||||
|
# run: |
|
||||||
|
# pipenv run rspec-tools notify-failure-on-slack \
|
||||||
|
# --message "ERROR: failed to update rule coverage. See https://github.com/SonarSource/rspec/actions/runs/$GITHUB_RUN_ID" \
|
||||||
|
# --channel team-analysis-rspec
|
||||||
|
@ -6739,4 +6739,4 @@
|
|||||||
"S6359": "sonar-xml 2.4.0.3273",
|
"S6359": "sonar-xml 2.4.0.3273",
|
||||||
"S6361": "sonar-xml 2.4.0.3273"
|
"S6361": "sonar-xml 2.4.0.3273"
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user