Skip to content
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

Only tagged version push to docker.io #116

Merged
merged 42 commits into from
Jan 19, 2024
Merged
Show file tree
Hide file tree
Changes from 37 commits
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
395a676
Only tagged version push to docker.io
ywkim312 Jan 17, 2024
e207511
removed test line
ywkim312 Jan 17, 2024
6bfeb47
test dockerhub login
ywkim312 Jan 17, 2024
0f777c4
redo whole thing
ywkim312 Jan 17, 2024
ebd0080
test github action post
ywkim312 Jan 17, 2024
02246bc
modified version grab process
ywkim312 Jan 17, 2024
4085532
added print outs
ywkim312 Jan 17, 2024
2d62fa9
test github.repository
ywkim312 Jan 17, 2024
17f7593
test 1
ywkim312 Jan 17, 2024
c8f75cf
test 2
ywkim312 Jan 17, 2024
695861e
test 3
ywkim312 Jan 17, 2024
1207c5d
test 5
ywkim312 Jan 17, 2024
5f2fc4a
test github posting
ywkim312 Jan 17, 2024
1fbd495
test dockerhub posting
ywkim312 Jan 17, 2024
df5d069
change container name
ywkim312 Jan 17, 2024
ebb3817
revert back to original
ywkim312 Jan 17, 2024
517fe8f
added docker hub push only to main
ywkim312 Jan 17, 2024
7262ca7
added branch info to env variable
ywkim312 Jan 17, 2024
7b80c13
test1
ywkim312 Jan 17, 2024
ef9827f
test2
ywkim312 Jan 17, 2024
d724b1e
test3
ywkim312 Jan 17, 2024
e49277c
test4
ywkim312 Jan 17, 2024
1afd3d5
test4
ywkim312 Jan 17, 2024
dd5e493
test5
ywkim312 Jan 17, 2024
c886731
test6
ywkim312 Jan 17, 2024
ecbbf1a
test7
ywkim312 Jan 17, 2024
54bb311
test8
ywkim312 Jan 17, 2024
403a1c2
test9
ywkim312 Jan 17, 2024
4303b64
test10
ywkim312 Jan 17, 2024
7aca136
test11
ywkim312 Jan 17, 2024
9984f17
test12
ywkim312 Jan 17, 2024
9d28e7b
test13
ywkim312 Jan 17, 2024
9554d98
test15
ywkim312 Jan 17, 2024
e942f66
test16
ywkim312 Jan 17, 2024
347d45f
test17
ywkim312 Jan 17, 2024
4ec9b0a
test18
ywkim312 Jan 17, 2024
4eced99
modified changelog
ywkim312 Jan 17, 2024
732d72c
removed cheking for posting to github
ywkim312 Jan 18, 2024
ab5a7b9
Merge branch 'develop' into 106-only-push-tagged-version-to-dockerio
ywkim312 Jan 18, 2024
9c43660
test 1
ywkim312 Jan 19, 2024
7486ba1
retreived to original
ywkim312 Jan 19, 2024
c58156b
fixed the issue in dockerhub
ywkim312 Jan 19, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 21 additions & 5 deletions .github/workflows/smile-server.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -70,18 +70,30 @@ jobs:
VERSION="$BRANCH"
tags="$BRANCH"
fi

echo "GITHUB_BRANCH=${BRANCH}" >> $GITHUB_ENV

# should we push to dockerhub, and is there a README
DOCKERHUB_PUSH="false"
DOCKERHUB_README="false"
if [ "${{ github.repository }}" == "${{ env.MAIN_REPO }}" ]; then
if [ "${{ secrets.DOCKERHUB_USERNAME }}" != "" -a "${{ secrets.DOCKERHUB_PASSWORD }}" != "" ]; then
DOCKERHUB_PUSH="true"
if [ -e "README.md" ]; then
DOCKERHUB_README="true"
if [ ${BRANCH} == "main" ]; then
if [ "${{ secrets.DOCKERHUB_USERNAME }}" != "" -a "${{ secrets.DOCKERHUB_PASSWORD }}" != "" ]; then
DOCKERHUB_PUSH="true"
if [ -e "README.md" ]; then
DOCKERHUB_README="true"
fi
fi
fi
fi

# should we push to Github
GITHUB_PUSH="false"
if [ "${{ github.repository }}" == "${{ env.MAIN_REPO }}" ]; then
if [ ${BRANCH} == "develop" ]; then
longshuicy marked this conversation as resolved.
Show resolved Hide resolved
GITHUB_PUSH="true"
fi
fi

# should we push to NCSA
NCSAHUB_PUSH="false"
Expand All @@ -101,7 +113,9 @@ jobs:
if [ "$NCSAHUB_PUSH" == "true" ]; then
IMAGES="${IMAGES}${NCSAHUB}/${IMAGE}:${tag},"
fi
IMAGES="${IMAGES}ghcr.io/${OWNER_LC}/${IMAGE}:${tag},"
if [ "$GITHUB_PUSH" == "true" ]; then
IMAGES="${IMAGES}ghcr.io/${OWNER_LC}/${IMAGE}:${tag},"
fi
done
IMAGES="${IMAGES%,*}"

Expand All @@ -111,6 +125,7 @@ jobs:
echo "DOCKERHUB_README=${DOCKERHUB_README}"
echo "DOCKERHUB_PUSH=${DOCKERHUB_PUSH}"
echo "NCSAHUB_PUSH=${NCSAHUB_PUSH}"
echo "GITHUB_PUSH=${GITHUB_PUSH}"
echo "IMAGES=${IMAGES}"

echo "BRANCH=${BRANCH}" >> $GITHUB_ENV
Expand Down Expand Up @@ -161,6 +176,7 @@ jobs:

# build the docker images
- name: Build and push docker
if: env.DOCKERHUB_PUSH == 'true' || env.NCSAHUB_PUSH == 'true' || env.GITHUB_PUSH == 'true'
uses: docker/build-push-action@v3
with:
context: ./www
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## [Unreleased]
### Added
- Google Analytics 4 tag for user analysis [#107](https://github.com/ncsa/standalone-smm-smile/issues/107)
- Github action to publish to different docker registry [#106](https://github.com/ncsa/standalone-smm-smile/issues/106)

## [0.3.1] - 2023-10-12
### Changed
Expand Down