mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-09-02 17:48:50 -04:00
Remove unused targets and variables from Makefile
Remove `cf-build` and `cf-build-with-docker` as they are not being used Remove `build-codedeploy-artifact` in favor of `build-paas-artifact` Remove `upload-codedeploy-artifact` in favor of `upload-paas-artifact` Remove `deploy`, `check-aws-vars`, `deploy-suspend-autoscaling-processes`, `deploy-resume-autoscaling-processes`, `deploy-check-autoscaling-processes` as they are remains of the pre-paas era. Consequently some variables became obsolete, namely: `CODEDEPLOY_PREFIX` `CODEDEPLOY_APP_NAME`, `DNS_NAME`, `AWS_ACCESS_KEY_ID` and `AWS_SECRET_ACCESS_KEY` and they are removed.
This commit is contained in:
53
Makefile
53
Makefile
@@ -19,9 +19,6 @@ BUILD_URL ?=
|
|||||||
|
|
||||||
DOCKER_CONTAINER_PREFIX = ${USER}-${BUILD_TAG}
|
DOCKER_CONTAINER_PREFIX = ${USER}-${BUILD_TAG}
|
||||||
|
|
||||||
CODEDEPLOY_PREFIX ?= notifications-admin
|
|
||||||
CODEDEPLOY_APP_NAME ?= notify-admin
|
|
||||||
|
|
||||||
CF_API ?= api.cloud.service.gov.uk
|
CF_API ?= api.cloud.service.gov.uk
|
||||||
CF_ORG ?= govuk-notify
|
CF_ORG ?= govuk-notify
|
||||||
CF_SPACE ?= ${DEPLOY_ENV}
|
CF_SPACE ?= ${DEPLOY_ENV}
|
||||||
@@ -42,32 +39,25 @@ venv/bin/activate:
|
|||||||
.PHONY: check-env-vars
|
.PHONY: check-env-vars
|
||||||
check-env-vars: ## Check mandatory environment variables
|
check-env-vars: ## Check mandatory environment variables
|
||||||
$(if ${DEPLOY_ENV},,$(error Must specify DEPLOY_ENV))
|
$(if ${DEPLOY_ENV},,$(error Must specify DEPLOY_ENV))
|
||||||
$(if ${DNS_NAME},,$(error Must specify DNS_NAME))
|
|
||||||
$(if ${AWS_ACCESS_KEY_ID},,$(error Must specify AWS_ACCESS_KEY_ID))
|
|
||||||
$(if ${AWS_SECRET_ACCESS_KEY},,$(error Must specify AWS_SECRET_ACCESS_KEY))
|
|
||||||
|
|
||||||
.PHONY: sandbox
|
.PHONY: sandbox
|
||||||
sandbox: ## Set environment to sandbox
|
sandbox: ## Set environment to sandbox
|
||||||
$(eval export DEPLOY_ENV=sandbox)
|
$(eval export DEPLOY_ENV=sandbox)
|
||||||
$(eval export DNS_NAME="cloudapps.digital")
|
|
||||||
@true
|
@true
|
||||||
|
|
||||||
.PHONY: preview
|
.PHONY: preview
|
||||||
preview: ## Set environment to preview
|
preview: ## Set environment to preview
|
||||||
$(eval export DEPLOY_ENV=preview)
|
$(eval export DEPLOY_ENV=preview)
|
||||||
$(eval export DNS_NAME="notify.works")
|
|
||||||
@true
|
@true
|
||||||
|
|
||||||
.PHONY: staging
|
.PHONY: staging
|
||||||
staging: ## Set environment to staging
|
staging: ## Set environment to staging
|
||||||
$(eval export DEPLOY_ENV=staging)
|
$(eval export DEPLOY_ENV=staging)
|
||||||
$(eval export DNS_NAME="staging-notify.works")
|
|
||||||
@true
|
@true
|
||||||
|
|
||||||
.PHONY: production
|
.PHONY: production
|
||||||
production: ## Set environment to production
|
production: ## Set environment to production
|
||||||
$(eval export DEPLOY_ENV=production)
|
$(eval export DEPLOY_ENV=production)
|
||||||
$(eval export DNS_NAME="notifications.service.gov.uk")
|
|
||||||
@true
|
@true
|
||||||
|
|
||||||
.PHONY: dependencies
|
.PHONY: dependencies
|
||||||
@@ -87,55 +77,22 @@ build: dependencies generate-version-file ## Build project
|
|||||||
npm run build
|
npm run build
|
||||||
. venv/bin/activate && PIP_ACCEL_CACHE=${PIP_ACCEL_CACHE} pip-accel install -r requirements.txt
|
. venv/bin/activate && PIP_ACCEL_CACHE=${PIP_ACCEL_CACHE} pip-accel install -r requirements.txt
|
||||||
|
|
||||||
.PHONY: cf-build
|
.PHONY: build-paas-artifact
|
||||||
cf-build: dependencies generate-version-file ## Build project
|
build-paas-artifact: ## Build the deploy artifact for PaaS
|
||||||
npm run build
|
|
||||||
|
|
||||||
.PHONY: build-codedeploy-artifact
|
|
||||||
build-codedeploy-artifact: ## Build the deploy artifact for CodeDeploy
|
|
||||||
rm -rf target
|
rm -rf target
|
||||||
mkdir -p target
|
mkdir -p target
|
||||||
zip -y -q -r -x@deploy-exclude.lst target/notifications-admin.zip ./
|
zip -y -q -r -x@deploy-exclude.lst target/notifications-admin.zip ./
|
||||||
|
|
||||||
.PHONY: upload-codedeploy-artifact ## Upload the deploy artifact for CodeDeploy
|
|
||||||
upload-codedeploy-artifact: check-env-vars
|
|
||||||
$(if ${DEPLOY_BUILD_NUMBER},,$(error Must specify DEPLOY_BUILD_NUMBER))
|
|
||||||
aws s3 cp --region eu-west-1 --sse AES256 target/notifications-admin.zip s3://${DNS_NAME}-codedeploy/${CODEDEPLOY_PREFIX}-${DEPLOY_BUILD_NUMBER}.zip
|
|
||||||
|
|
||||||
.PHONY: build-paas-artifact
|
|
||||||
build-paas-artifact: build-codedeploy-artifact ## Build the deploy artifact for PaaS
|
|
||||||
|
|
||||||
.PHONY: upload-paas-artifact ## Upload the deploy artifact for PaaS
|
.PHONY: upload-paas-artifact ## Upload the deploy artifact for PaaS
|
||||||
upload-paas-artifact:
|
upload-paas-artifact:
|
||||||
$(if ${DEPLOY_BUILD_NUMBER},,$(error Must specify DEPLOY_BUILD_NUMBER))
|
$(if ${DEPLOY_BUILD_NUMBER},,$(error Must specify DEPLOY_BUILD_NUMBER))
|
||||||
$(if ${JENKINS_S3_BUCKET},,$(error Must specify JENKINS_S3_BUCKET))
|
$(if ${JENKINS_S3_BUCKET},,$(error Must specify JENKINS_S3_BUCKET))
|
||||||
aws s3 cp --region eu-west-1 --sse AES256 target/notifications-admin.zip s3://${JENKINS_S3_BUCKET}/build/${CODEDEPLOY_PREFIX}/${DEPLOY_BUILD_NUMBER}.zip
|
aws s3 cp --region eu-west-1 --sse AES256 target/notifications-admin.zip s3://${JENKINS_S3_BUCKET}/build/notifications-admin/${DEPLOY_BUILD_NUMBER}.zip
|
||||||
|
|
||||||
.PHONY: test
|
.PHONY: test
|
||||||
test: venv ## Run tests
|
test: venv ## Run tests
|
||||||
./scripts/run_tests.sh
|
./scripts/run_tests.sh
|
||||||
|
|
||||||
.PHONY: deploy
|
|
||||||
deploy: check-env-vars ## Upload deploy artifacts to S3 and trigger CodeDeploy
|
|
||||||
aws deploy create-deployment --application-name ${CODEDEPLOY_APP_NAME} --deployment-config-name CodeDeployDefault.OneAtATime --deployment-group-name ${CODEDEPLOY_APP_NAME} --s3-location bucket=${DNS_NAME}-codedeploy,key=${CODEDEPLOY_PREFIX}-${DEPLOY_BUILD_NUMBER}.zip,bundleType=zip --region eu-west-1
|
|
||||||
|
|
||||||
.PHONY: check-aws-vars
|
|
||||||
check-aws-vars: ## Check if AWS access keys are set
|
|
||||||
$(if ${AWS_ACCESS_KEY_ID},,$(error Must specify AWS_ACCESS_KEY_ID))
|
|
||||||
$(if ${AWS_SECRET_ACCESS_KEY},,$(error Must specify AWS_SECRET_ACCESS_KEY))
|
|
||||||
|
|
||||||
.PHONY: deploy-suspend-autoscaling-processes
|
|
||||||
deploy-suspend-autoscaling-processes: check-aws-vars ## Suspend launch and terminate processes for the auto-scaling group
|
|
||||||
aws autoscaling suspend-processes --region eu-west-1 --auto-scaling-group-name ${CODEDEPLOY_APP_NAME} --scaling-processes "Launch" "Terminate"
|
|
||||||
|
|
||||||
.PHONY: deploy-resume-autoscaling-processes
|
|
||||||
deploy-resume-autoscaling-processes: check-aws-vars ## Resume launch and terminate processes for the auto-scaling group
|
|
||||||
aws autoscaling resume-processes --region eu-west-1 --auto-scaling-group-name ${CODEDEPLOY_APP_NAME} --scaling-processes "Launch" "Terminate"
|
|
||||||
|
|
||||||
.PHONY: deploy-check-autoscaling-processes
|
|
||||||
deploy-check-autoscaling-processes: check-aws-vars ## Returns with the number of instances with active autoscaling events
|
|
||||||
@aws autoscaling describe-auto-scaling-groups --region eu-west-1 --auto-scaling-group-names ${CODEDEPLOY_APP_NAME} | jq '.AutoScalingGroups[0].Instances|map(select(.LifecycleState != "InService"))|length'
|
|
||||||
|
|
||||||
.PHONY: coverage
|
.PHONY: coverage
|
||||||
coverage: venv ## Create coverage report
|
coverage: venv ## Create coverage report
|
||||||
. venv/bin/activate && coveralls
|
. venv/bin/activate && coveralls
|
||||||
@@ -180,10 +137,6 @@ endef
|
|||||||
build-with-docker: prepare-docker-build-image ## Build inside a Docker container
|
build-with-docker: prepare-docker-build-image ## Build inside a Docker container
|
||||||
$(call run_docker_container,build,gosu hostuser make build)
|
$(call run_docker_container,build,gosu hostuser make build)
|
||||||
|
|
||||||
.PHONY: cf-build-with-docker
|
|
||||||
cf-build-with-docker: prepare-docker-build-image ## Build inside a Docker container
|
|
||||||
$(call run_docker_container,build,gosu hostuser make cf-build)
|
|
||||||
|
|
||||||
.PHONY: test-with-docker
|
.PHONY: test-with-docker
|
||||||
test-with-docker: prepare-docker-build-image ## Run tests inside a Docker container
|
test-with-docker: prepare-docker-build-image ## Run tests inside a Docker container
|
||||||
$(call run_docker_container,test,gosu hostuser make test)
|
$(call run_docker_container,test,gosu hostuser make test)
|
||||||
|
|||||||
Reference in New Issue
Block a user