-
Notifications
You must be signed in to change notification settings - Fork 8
DAS-2070 - Implement GitHub workflows CI/CD. #1
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
Merged
Merged
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
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 hidden or 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,17 @@ | ||
## Description | ||
|
||
A short description of the changes in this PR. | ||
|
||
## Jira Issue ID | ||
|
||
[DAS-XXXX](https://bugs.earthdata.nasa.gov/browse/DAS-XXXX) | ||
|
||
## Local Test Steps | ||
|
||
|
||
## PR Acceptance Checklist | ||
* [ ] Jira ticket acceptance criteria met. | ||
* [ ] `CHANGELOG.md` updated to include high level summary of PR changes. | ||
* [ ] `docker/service_version.txt` updated if publishing a release. | ||
* [ ] Tests added/updated and passing. | ||
* [ ] Documentation updated (if needed). |
This file contains hidden or 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 HyBIG. | ||
name: Publish Harmony Browse Image Generator (HyBIG) 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-browse-image-generator 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 hidden or 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 HyBIG, 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-browse-image-generator 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 hidden or 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 hidden or 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 hidden or 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,47 @@ | ||
# Contributing to HyBIG | ||
|
||
Thanks for contributing! | ||
|
||
## Making Changes | ||
|
||
To allow us to incorporate your changes, please use the | ||
[Fork-and-Pull](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/getting-started/about-collaborative-development-models#fork-and-pull-model) | ||
development model: | ||
|
||
1. Fork this repository to your personal account. | ||
2. Create a branch and make your changes. | ||
3. Test the changes locally/in your personal fork. | ||
4. Submit a pull request to open a discussion about your proposed changes. | ||
5. The maintainers will talk with you about it and decide to merge or request | ||
additional changes. | ||
|
||
For larger items, consider contacting the maintainers first to coordinate | ||
development efforts. | ||
|
||
## Commits | ||
|
||
Our ticketing and CI/CD tools are configured to sync statuses amongst each | ||
other. Commits play an important role in this process. Please start all commits | ||
with the Jira ticket number associated with your feature, task, or bug. All | ||
commit messages should follow the format | ||
"[Jira Project]-XXXX - [Your commit message here]" | ||
|
||
## General coding practices: | ||
|
||
This repository adheres to Python coding style recommendations from | ||
[PEP8](https://peps.python.org/pep-0008/). Additionally, type hints are | ||
encouraged in all function signatures. | ||
|
||
When adding or updating functionality, please ensure unit tests are added to | ||
the existing `unittest` suite in the `tests` directory, which cover each branch | ||
of the code. | ||
|
||
## Disclaimer | ||
|
||
HyBIG maintainers will review all pull requests submitted. Only requests that | ||
meet the standard of quality set forth by existing code, following the patterns | ||
set forth by existing code, and adhering to existing design patterns will be | ||
considered and/or accepted. | ||
|
||
For general tips on open source contributions, see | ||
[Contributing to Open Source on GitHub](https://guides.github.com/activities/contributing-to-open-source/). |
This file contains hidden or 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 |
---|---|---|
@@ -1,14 +1,15 @@ | ||
# Harmony Browse Image Generator (HyBIG) backend service. | ||
|
||
This Harmony backend service is designed to produce browse imagery, with | ||
default behaviour to produce browse imagery that is compatible with Global | ||
Image Browse Services (GIBS). | ||
default behaviour to produce browse imagery that is compatible with the NASA | ||
Global Image Browse Services ([GIBS](https://www.earthdata.nasa.gov/eosdis/science-system-description/eosdis-components/gibs)). | ||
|
||
## Repository structure: | ||
|
||
``` | ||
|- .pre-commit-config.yaml | ||
|- CHANGELOG.md | ||
|- CONTRIBUTING.md | ||
|- README.md | ||
|- bin | ||
|- conda_requirements.txt | ||
|
@@ -25,6 +26,8 @@ Image Browse Services (GIBS). | |
* `CHANGELOG.md` - This file contains a record of changes applied to each new | ||
release of a service Docker image. Any release of a new service version | ||
should have a record of what was changed in this file. | ||
* `CONTRIBUTING.md` - This file contains guidance for making contributions to | ||
HyBIG, including recommended git best practices. | ||
* `README.md` - This file, containing guidance on developing the service. | ||
* `bin` - A directory containing utility scripts to build the service and test | ||
images. This also includes scripts that Bamboo uses to deploy new service | ||
|
@@ -40,6 +43,9 @@ Image Browse Services (GIBS). | |
* `harmony_browse_image_generator` - The directory containing Python source code | ||
for the HyBIG. `adapter.py` contains the `BrowseImageGeneratorAdapter` | ||
class that is invoked by calls to the service. | ||
* `legacy-CHANGELOG.md` - Notes for each version that was previously released | ||
internally to EOSDIS, prior to open-source publication of the code and Docker | ||
image. | ||
* `pip_requirements.txt` - A list of service Python package dependencies. | ||
* `tests` - A directory containing the service unit test suite. | ||
|
||
|
@@ -99,11 +105,36 @@ updated: | |
* CHANGELOG.md - Notes should be added to capture the changes to the service. | ||
* docker/service_version.txt - The semantic version number should be updated. | ||
|
||
## Docker image publication: | ||
## CI/CD: | ||
|
||
Initially service Docker images will be hosted in AWS Elastic Container | ||
Registry (ECR). When this repository is migrated to the NASA GitHub | ||
organisation, service images will be published to ghcr.io, instead. | ||
The CI/CD for HyBIG is contained in GitHub workflows in the | ||
`.github/workflows` directory: | ||
|
||
* `run_tests.yml` - A reusable workflow that builds the service and test Docker | ||
images, then runs the Python unit test suite in an instance of the test | ||
Docker container. | ||
* `run_tests_on_pull_requests.yml` - Triggered for all PRs against the `main` | ||
branch. It runs the workflow in `run_tests.yml` to ensure all tests pass for | ||
the new code. | ||
* `publish_docker_image.yml` - Triggered either manually or for commits to the | ||
`main` branch that contain changes to the `docker/service_version.txt` file. | ||
|
||
The `publish_docker_image.yml` workflow will: | ||
|
||
* Run the full unit test suite, to prevent publication of broken code. | ||
* Extract the semantic version number from `docker/service_version.txt`. | ||
* Extract the released notes for the most recent version from `CHANGELOG.md`. | ||
* Create a GitHub release that will also tag the related git commit with the | ||
semantic version number. | ||
|
||
Before triggering a release, ensure both the `docker/service_version.txt` and | ||
`CHANGELOG.md` files are updated. The `CHANGELOG.md` file requires a specific | ||
format for a new release, as it looks for the following string to define the | ||
newest relate of the code (starting at the top of the file). | ||
|
||
``` | ||
## vX.Y.Z | ||
``` | ||
|
||
## Releasing a new version of the service: | ||
|
||
|
@@ -112,3 +143,11 @@ that service version can be released to a Harmony environment by updating the | |
main Harmony Bamboo deployment project. Find the environment you wish to | ||
release the service version to and update the associated environment variable | ||
to update the semantic version tag at the end of the full Docker image name. | ||
|
||
## Get in touch: | ||
|
||
You can reach out to the maintainers of this repository via email: | ||
|
||
* [email protected] | ||
* [email protected] | ||
* [email protected] |
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 | ||
# HyBIG from CHANGELOG.md | ||
# | ||
# 2023-06-16: Created. | ||
# 2023-10-10: Copied from earthdata-varinfo repository to HOSS. | ||
# 2024-01-03: Copied from HOSS repository to the Swath Projector. | ||
# 2024-01-23: Copied and modified from Swath Projector repository to HyBIG. | ||
# | ||
############################################################################### | ||
|
||
CHANGELOG_FILE="CHANGELOG.md" | ||
VERSION_PATTERN="^## v" | ||
|
||
# Read the file and extract text between the first two occurrences of the | ||
# VERSION_PATTERN | ||
result=$(awk "/$VERSION_PATTERN/{c++; if(c==2) exit;} c==1" "$CHANGELOG_FILE") | ||
|
||
# Print the result | ||
echo "$result" | grep -v "^#" |
This file contains hidden or 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 hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.