Skip to content

Commit

Permalink
[AWSRESET-365]: workflow updated
Browse files Browse the repository at this point in the history
  • Loading branch information
rahul-dvsa committed Oct 9, 2023
1 parent 212abf5 commit be618c1
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/php-base-image-push-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,7 @@ jobs:
echo "Not pushing - $repoName, build parameter equal to false"
fi
done
echo "::set-output name=imageTag::${$AWS_ACCOUNT_ID.dkr.ecr.$AWS_REGION.amazonaws.com/$registry/$repoName:$tag-$SHA}"


- name: Sign base images
run: |
jq -c '.[]' build.json | while read -r results; do
Expand Down
11 changes: 10 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,11 @@
# dvsa-docker-images
Docker Images for use in DVSA projects
Docker base Images for use in DVSA projects

The images will be used by all the base images used in the project. The github repo is structured to cater the multiple docker builds in a single repo

`build` - Contains the the repo name for instance `vol-php-fpm` , if you want to create another docker image, create another folder and have your dockerfile in that folder along with any required files etc.
`scripts` - Scripts used in the workflow
`.github/workflows` - It contians 2 workflows,
`php-base-image-build-pr-main` runs on the PR request to main branch and perform the linting, security check and docker build
`php-base-image-push-main` runs on the PR merge to main branch and performs the image signing, docker build and docker push
`build.json` - This is the file used by the workflow to perform the operation on multiple docker build operations, the workflow loop through the file and look for the parameter `build` if set to `true` the workflow performs the operation on only those images. so, please make sure to set this parameter as either `true | false`

0 comments on commit be618c1

Please sign in to comment.