Skip to content
This repository has been archived by the owner on Nov 22, 2024. It is now read-only.

CI/CD for container image #25

Merged
merged 3 commits into from
Aug 24, 2023

Conversation

johnandersen777
Copy link

@johnandersen777 johnandersen777 commented Mar 30, 2023

$ docker build -t ghcr.io/scitt-community/scitt-api-emulator:main --progress plain .
$ docker run --rm -ti -w /src/src/scitt-api-emulator -v $PWD:/src/src/scitt-api-emulator -p 8000:8000 ghcr.io/scitt-community/scitt-api-emulator:main

Signed-off-by: John Andersen <johnandersenpdx@gmail.com>
@johnandersen777 johnandersen777 force-pushed the ci_cd_container_image branch 5 times, most recently from 2344ad3 to d9539f0 Compare March 30, 2023 05:15
johnandersen777 pushed a commit to intel/dffml that referenced this pull request Mar 30, 2023
…ocker-compose: Add scitt API emulator

Related: scitt-community/scitt-api-emulator#25
Related: ietf-scitt/use-cases#14
Related: https://codeberg.org/forgejo/discussions/issues/12
Related: ietf-scitt/use-cases#18
Signed-off-by: John Andersen <johnandersenpdx@gmail.com>
@johnandersen777 johnandersen777 force-pushed the ci_cd_container_image branch 2 times, most recently from dcd3d8d to 4670828 Compare March 30, 2023 06:00
OR13
OR13 previously approved these changes Mar 31, 2023
Copy link
Contributor

@OR13 OR13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

ci-cd-build-and-push-image-container:
name: CI/CD (container)
runs-on: ubuntu-latest
if: ${{ github.ref == 'refs/heads/main' }}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Force pushed due to originally missing final } on this line

Signed-off-by: John Andersen <johnandersenpdx@gmail.com>
Copy link
Author

@johnandersen777 johnandersen777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Realized we should probably build the container on pull requests after seeing and fixing the missing trailing } on the ${{ github.ref == 'refs/heads/main' }} check. This now skips push unless on main branch.

@johnandersen777 johnandersen777 requested a review from OR13 March 31, 2023 07:49
Signed-off-by: John Andersen <johnandersenpdx@gmail.com>
@OR13 OR13 merged commit c1c3454 into scitt-community:main Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants