launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #31939
Re: [Merge] ~lgp171188/launchpad-bastion-scripts:add-verify-production into launchpad-bastion-scripts:main
Diff comments:
> diff --git a/verify-production b/verify-production
> new file mode 100755
> index 0000000..b6c9e78
> --- /dev/null
> +++ b/verify-production
> @@ -0,0 +1,11 @@
> +#! /bin/bash
> +set -e
> +set -o pipefail
> +
> +# Ignore haproxy queue depth checks for deployment purposes: they may need
> +# to be handled by IS, but are unlikely to relate very closely to code
> +# deployments.
> +skip_checks=check_haproxy_queue
> +
> +mkdir -p "$HOME/logs"
> +ssh prod-launchpad@is-bastion-ps5.internal autodeploy-trigger prod-launchpad verify EXTRA_SKIP_CHECKS="$skip_checks" 2>&1 | tee -a "$HOME/logs/$(date +%F)-production.log"
This is just a minor variant of the 'upgrade-production' script. But since this is just a one-liner, it is not really helpful to deduplicate.
--
https://code.launchpad.net/~lgp171188/launchpad-bastion-scripts/+git/launchpad-bastion-scripts/+merge/477497
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad-bastion-scripts:add-verify-production into launchpad-bastion-scripts:main.
References