diff options
author | Joel Hans <joel@netdata.cloud> | 2020-01-27 14:26:27 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-01-27 14:26:27 -0700 |
commit | 439f334e678baf92253315d65d374b67dd4b9a16 (patch) | |
tree | 63fc394f948da156c9604da85a3ae24d0d7e1335 /tests | |
parent | a4223457b49497f57bd199dfaf5cf35c4fb86997 (diff) |
Docs: Overhaul of installation documentation (#7841)
* Initial commit to bring new branch up to speed with previous work
* Initial commit to bring new branch up to speed with previous work
* Pass through for grammar and typos
* Pass through for grammar and typos
* First additions
* Improvements to installation page plus new separate kickstart page
* Lots of new pages, lots of improvements
* Continued work
* Fixing the install grid
* Added methods to nav
* Fix typo and add kickstart to nav
* CSS cleanup
* Various cleanup
* Cleanup on update/uninstall pages
* Add responsiveness to install grid
* Update checking of the kickstart files MD5 checksusm.
This updates the CI script used to verify the MD5 checksums of the
kickstart files as being correct in the documentation to use the new
locations for the respective checksums.
This is more involved than a simple path update because the existing
script assumes that both checksums are listed in the same file, which is
no longer the case.
Any future updates that move the location of the checksums just need to
modify the lines in tests/installer/checksums.sh that start with
`check_checksum` to point to the correct files.
* Added Alpine package for James
* Add packages to top of installation page
* Fix for Chris
* Telemetry fixes
* Trying to fix CI
* Changing checksums
* Fix CI checks for kickstart checksums.
The changed wording was confusing the code that parsed the checksum out
of the documentation, this fixes the code to handle this new wording
correctly.
* Update kickstart-static64 checksum
* Update 64 checksum
Co-authored-by: Austin S. Hemmelgarn <ahferroin7@gmail.com>
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/installer/checksums.sh | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/tests/installer/checksums.sh b/tests/installer/checksums.sh index 18cad074e4..3fbbfa3864 100755 --- a/tests/installer/checksums.sh +++ b/tests/installer/checksums.sh @@ -7,6 +7,7 @@ # # Author : Pawel Krupa (pawel@netdata.cloud) # Author : Pavlos Emm. Katsoulakis (paul@netdata.cloud) +# Author : Austin S. Hemmelgarn (austin@netdata.cloud) set -e # If we are not in netdata git repo, at the top level directory, fail @@ -18,14 +19,12 @@ if [ -n "$CWD" ] || [ "${TOP_LEVEL}" != "netdata" ]; then exit 1 fi -README_DOC="packaging/installer/README.md" - -for file in kickstart.sh kickstart-static64.sh; do - README_MD5=$(grep "$file" $README_DOC | grep md5sum | cut -d '"' -f2) - KICKSTART_URL="https://my-netdata.io/$file" - KICKSTART="packaging/installer/$file" +check_file() { + README_MD5=$(grep "$1" "$2" | grep md5sum | grep curl | cut -d '"' -f2) + KICKSTART_URL="https://my-netdata.io/$1" + KICKSTART="packaging/installer/$1" KICKSTART_MD5="$(md5sum "${KICKSTART}" | cut -d' ' -f1)" - CALCULATED_MD5="$(curl -Ss ${KICKSTART_URL} | md5sum | cut -d ' ' -f 1)" + CALCULATED_MD5="$(curl -Ss "${KICKSTART_URL}" | md5sum | cut -d ' ' -f 1)" # Conditionally run the website validation if [ -z "${LOCAL_ONLY}" ]; then @@ -38,15 +37,18 @@ for file in kickstart.sh kickstart-static64.sh; do fi fi - echo "Validating documentation for $file" + echo "Validating documentation for $1" if [ "$KICKSTART_MD5" != "$README_MD5" ]; then - echo "Invalid checksum for $file in $README_DOC." + echo "Invalid checksum for $1 in $2." echo "checksum in docs: $README_MD5" echo "current checksum: $KICKSTART_MD5" exit 2 else - echo "$file MD5Sum is well documented" + echo "$1 MD5Sum is well documented" fi +} + +check_file kickstart.sh packaging/installer/methods/kickstart.md +check_file kickstart-static64.sh packaging/installer/methods/kickstart-64.md -done echo "No problems found, exiting succesfully!" |