Browse Source

Merge pull request #10006 from jfrazelle/docs-release-fixes

Update docs release script so we can have autodeploys
Jessie Frazelle 10 years ago
parent
commit
1781fed730
3 changed files with 12 additions and 6 deletions
  1. 1 0
      .gitignore
  2. 2 2
      Makefile
  3. 9 4
      docs/release.sh

+ 1 - 0
.gitignore

@@ -28,3 +28,4 @@ docs/AWS_S3_BUCKET
 docs/GIT_BRANCH
 docs/GIT_BRANCH
 docs/VERSION
 docs/VERSION
 docs/GITCOMMIT
 docs/GITCOMMIT
+docs/changed-files

+ 2 - 2
Makefile

@@ -30,7 +30,7 @@ DOCKER_DOCS_IMAGE := docker-docs$(if $(GIT_BRANCH),:$(GIT_BRANCH))
 
 
 DOCKER_RUN_DOCKER := docker run --rm -it --privileged $(DOCKER_ENVS) $(DOCKER_MOUNT) "$(DOCKER_IMAGE)"
 DOCKER_RUN_DOCKER := docker run --rm -it --privileged $(DOCKER_ENVS) $(DOCKER_MOUNT) "$(DOCKER_IMAGE)"
 
 
-DOCKER_RUN_DOCS := docker run --rm -it $(DOCS_MOUNT) -e AWS_S3_BUCKET
+DOCKER_RUN_DOCS := docker run --rm -it $(DOCS_MOUNT) -e AWS_S3_BUCKET -e NOCACHE
 
 
 # for some docs workarounds (see below in "docs-build" target)
 # for some docs workarounds (see below in "docs-build" target)
 GITCOMMIT := $(shell git rev-parse --short HEAD 2>/dev/null)
 GITCOMMIT := $(shell git rev-parse --short HEAD 2>/dev/null)
@@ -83,7 +83,7 @@ build: bundles
 	docker build -t "$(DOCKER_IMAGE)" .
 	docker build -t "$(DOCKER_IMAGE)" .
 
 
 docs-build:
 docs-build:
-	git diff --name-status upstream/release..upstream/docs docs/ > docs/changed-files
+	( git remote | grep -v upstream ) || git diff --name-status upstream/release..upstream/docs docs/ > docs/changed-files
 	cp ./VERSION docs/VERSION
 	cp ./VERSION docs/VERSION
 	echo "$(GIT_BRANCH)" > docs/GIT_BRANCH
 	echo "$(GIT_BRANCH)" > docs/GIT_BRANCH
 	echo "$(AWS_S3_BUCKET)" > docs/AWS_S3_BUCKET
 	echo "$(AWS_S3_BUCKET)" > docs/AWS_S3_BUCKET

+ 9 - 4
docs/release.sh

@@ -78,6 +78,11 @@ upload_current_documentation() {
 	src=site/
 	src=site/
 	dst=s3://$BUCKET$1
 	dst=s3://$BUCKET$1
 
 
+	cache=max-age=3600
+	if [ "$NOCACHE" ]; then
+		cache=no-cache
+	fi
+
 	echo
 	echo
 	echo "Uploading $src"
 	echo "Uploading $src"
 	echo "  to $dst"
 	echo "  to $dst"
@@ -90,7 +95,7 @@ upload_current_documentation() {
 	#  versions.html_fragment
 	#  versions.html_fragment
 		include="--recursive --include \"*.$i\" "
 		include="--recursive --include \"*.$i\" "
 		echo "uploading *.$i"
 		echo "uploading *.$i"
-		run="aws s3 cp $src $dst $OPTIONS --profile $BUCKET --cache-control \"max-age=3600\" --acl public-read $include"
+		run="aws s3 cp $src $dst $OPTIONS --profile $BUCKET --cache-control $cache --acl public-read $include"
 		echo "======================="
 		echo "======================="
 		echo "$run"
 		echo "$run"
 		echo "======================="
 		echo "======================="
@@ -114,7 +119,7 @@ invalidate_cache() {
 
 
 	len=${#files[@]}
 	len=${#files[@]}
 
 
-	echo "aws cloudfront  create-invalidation --profile docs.docker.com --distribution-id $DISTRIBUTION_ID --invalidation-batch '" > batchfile
+	echo "aws cloudfront  create-invalidation --profile $AWS_S3_BUCKET --distribution-id $DISTRIBUTION_ID --invalidation-batch '" > batchfile
 	echo "{\"Paths\":{\"Quantity\":$len," >> batchfile
 	echo "{\"Paths\":{\"Quantity\":$len," >> batchfile
 	echo "\"Items\": [" >> batchfile
 	echo "\"Items\": [" >> batchfile
 
 
@@ -150,7 +155,7 @@ if [ "$BUILD_ROOT" == "yes" ]; then
 	echo "Building root documentation"
 	echo "Building root documentation"
 	build_current_documentation
 	build_current_documentation
 	upload_current_documentation
 	upload_current_documentation
-	invalidate_cache
+	[ "$NOCACHE" ] || invalidate_cache
 fi
 fi
 
 
 #build again with /v1.0/ prefix
 #build again with /v1.0/ prefix
@@ -158,4 +163,4 @@ sed -i "s/^site_url:.*/site_url: \/$MAJOR_MINOR\//" mkdocs.yml
 echo "Building the /$MAJOR_MINOR/ documentation"
 echo "Building the /$MAJOR_MINOR/ documentation"
 build_current_documentation
 build_current_documentation
 upload_current_documentation "/$MAJOR_MINOR/"
 upload_current_documentation "/$MAJOR_MINOR/"
-invalidate_cache "/$MAJOR_MINOR"
+[ "$NOCACHE" ] || invalidate_cache "/$MAJOR_MINOR"