-
Notifications
You must be signed in to change notification settings - Fork 2
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
DAS-1937 - Add GitHub workflows for unit tests and Docker image publication. #2
Merged
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,83 @@ | ||
# This workflow will run when changes are detected in the `main` branch, which | ||
# must include an update to the `docker/service_version.txt` file. The workflow | ||
# can also be manually triggered by a repository maintainer. This workflow will | ||
# first trigger the reusable workflow in `.github/workflows/run_tests.yml`, | ||
# which runs the `unittest` suite. If that workflow is successful, the latest | ||
# version of the service Docker image is pushed to ghcr.io, a tag is added to | ||
# the latest git commit, and a GitHub release is created with the release notes | ||
# from the latest version of HOSS. | ||
name: Publish Harmony OPeNDAP SubSetter (HOSS) Docker image | ||
|
||
on: | ||
push: | ||
branches: [ main ] | ||
paths: docker/service_version.txt | ||
workflow_dispatch: | ||
|
||
env: | ||
IMAGE_NAME: ${{ github.repository }} | ||
REGISTRY: ghcr.io | ||
|
||
jobs: | ||
run_tests: | ||
uses: ./.github/workflows/run_tests.yml | ||
|
||
build_and_publish_image: | ||
needs: run_tests | ||
runs-on: ubuntu-latest | ||
environment: release | ||
permissions: | ||
# write permission is required to create a GitHub release | ||
contents: write | ||
id-token: write | ||
packages: write | ||
strategy: | ||
fail-fast: false | ||
|
||
steps: | ||
- name: Checkout harmony-opendap-subsetter repository | ||
uses: actions/checkout@v3 | ||
with: | ||
lfs: true | ||
|
||
- name: Extract semantic version number | ||
run: echo "semantic_version=$(cat docker/service_version.txt)" >> $GITHUB_ENV | ||
|
||
- name: Extract release version notes | ||
run: | | ||
version_release_notes=$(./bin/extract-release-notes.sh) | ||
echo "RELEASE_NOTES<<EOF" >> $GITHUB_ENV | ||
echo "${version_release_notes}" >> $GITHUB_ENV | ||
echo "EOF" >> $GITHUB_ENV | ||
|
||
- name: Log-in to ghcr.io registry | ||
uses: docker/login-action@v2 | ||
with: | ||
registry: ${{ env.REGISTRY }} | ||
username: ${{ github.actor }} | ||
password: ${{ secrets.GITHUB_TOKEN }} | ||
|
||
- name: Add tags to the Docker image | ||
id: meta | ||
uses: docker/metadata-action@v4 | ||
with: | ||
images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }} | ||
tags: | | ||
type=semver,pattern={{version}},value=${{ env.semantic_version }} | ||
|
||
- name: Push Docker image | ||
uses: docker/build-push-action@v3 | ||
with: | ||
context: . | ||
file: docker/service.Dockerfile | ||
push: true | ||
tags: ${{ steps.meta.outputs.tags }} | ||
labels: ${{ steps.meta.outputs.labels }} | ||
|
||
- name: Publish GitHub release | ||
uses: ncipollo/release-action@v1 | ||
with: | ||
body: ${{ env.RELEASE_NOTES }} | ||
commit: main | ||
name: Version ${{ env.semantic_version }} | ||
tag: ${{ env.semantic_version }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
# This workflow will build the service and test Docker images for HOSS, then | ||
# run the Python `unittest` suite within a test Docker container, reporting | ||
# test results and code coverage as artefacts. It will be called by the | ||
# workflow that run tests against new PRs and as a first step in the workflow | ||
# that publishes new Docker images. | ||
name: Run Python unit tests | ||
|
||
on: | ||
workflow_call | ||
|
||
jobs: | ||
build_and_test: | ||
runs-on: ubuntu-latest | ||
strategy: | ||
fail-fast: false | ||
|
||
steps: | ||
- name: Checkout harmony-opendap-subsetter repository | ||
uses: actions/checkout@v3 | ||
with: | ||
lfs: true | ||
|
||
- name: Build service image | ||
run: ./bin/build-image | ||
|
||
- name: Build test image | ||
run: ./bin/build-test | ||
|
||
- name: Run test image | ||
run: ./bin/run-test | ||
|
||
- name: Archive test results | ||
uses: actions/upload-artifact@v3 | ||
with: | ||
name: Test results | ||
path: test-reports/ | ||
|
||
- name: Archive coverage report | ||
uses: actions/upload-artifact@v3 | ||
with: | ||
name: Coverage report | ||
path: coverage/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
# This workflow will run when a PR is opened against the `main` branch. It will | ||
# trigger the reusable workflow in `.github/workflows/run_tests.yml`, which | ||
# builds the service and test Docker images, and runs the `unittest` suite in a | ||
# Docker container built from the test image. | ||
name: Run Python unit tests for pull requests against main | ||
|
||
on: | ||
pull_request: | ||
branches: [ main ] | ||
|
||
jobs: | ||
build_and_test: | ||
uses: ./.github/workflows/run_tests.yml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
#!/bin/bash | ||
############################################################################### | ||
# | ||
# A bash script to extract only the notes related to the most recent version of | ||
# HOSS from CHANGELOG.md | ||
# | ||
# 2023-06-16: Created. | ||
# 2023-10-10: Copied from earthdata-varinfo repository to HOSS. | ||
# | ||
############################################################################### | ||
|
||
CHANGELOG_FILE="CHANGELOG.md" | ||
VERSION_PATTERN="^## v" | ||
# Count number of versions in version file: | ||
number_of_versions=$(grep -c "${VERSION_PATTERN}" ${CHANGELOG_FILE}) | ||
|
||
if [ ${number_of_versions} -gt 1 ] | ||
then | ||
grep -B 9999 -m 2 "${VERSION_PATTERN}" ${CHANGELOG_FILE} | sed '$d' | sed '$d' | ||
else | ||
cat ${CHANGELOG_FILE} | ||
fi |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
packages
permission is new in comparison to HGA and NetCDF-to-Zarr, however, I could not get my fork to create the package without it. I got guidance from StackOverflow and also the GitHub documentation.