From 268c318bc5ff767c0d7fcd082cdcd48d592b7cc3 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Sun, 15 May 2022 15:47:13 -0400 Subject: [PATCH] fix: ensure we do not filter apps for certain commands and triggers Some things must pull *all* apps vs filtered apps (mostly things that write out config files or install triggers). Using an unflitered list is best. Also, we default to filtering, which should be what most usage needs. --- plugins/checks/install | 4 ++-- plugins/common/triggers.go | 2 +- plugins/cron/functions.go | 2 +- plugins/domains/install | 2 +- plugins/git/install | 2 +- plugins/logs/functions.go | 2 +- plugins/network/triggers.go | 2 +- plugins/nginx-vhosts/functions | 2 +- plugins/nginx-vhosts/install | 2 +- plugins/ps/triggers.go | 2 +- plugins/scheduler/triggers.go | 2 +- 11 files changed, 12 insertions(+), 12 deletions(-) diff --git a/plugins/checks/install b/plugins/checks/install index 1497ca5d4..f6321f86b 100755 --- a/plugins/checks/install +++ b/plugins/checks/install @@ -9,7 +9,7 @@ migrate_checks_vars_0_5_0() { local GLOBAL_SKIP_ALL_CHECKS=$(config_get --global DOKKU_SKIP_ALL_CHECKS || true) local GLOBAL_SKIP_DEFAULT_CHECKS=$(config_get --global DOKKU_SKIP_DEFAULT_CHECKS || true) - for app in $(dokku_apps); do + for app in $(dokku_apps "false"); do local APP_SKIP_ALL_CHECKS=$(config_get "$app" DOKKU_SKIP_ALL_CHECKS || true) local APP_SKIP_DEFAULT_CHECKS=$(config_get "$app" DOKKU_SKIP_DEFAULT_CHECKS || true) @@ -35,7 +35,7 @@ migrate_checks_vars_0_5_0() { migrate_checks_vars_0_6_0() { declare desc="migrates CHECKS config variables from 0.5.x to support fully-disabled zero-downtime checks" - for app in $(dokku_apps); do + for app in $(dokku_apps "false"); do local APP_DOKKU_CHECKS_ENABLED=$(config_get "$app" DOKKU_CHECKS_ENABLED || true) if [[ $APP_DOKKU_CHECKS_ENABLED ]]; then dokku_log_info1 "Migrating zero downtime env variables to 0.6.x. The following variables will be migrated" diff --git a/plugins/common/triggers.go b/plugins/common/triggers.go index 5915ccf83..e904c525a 100644 --- a/plugins/common/triggers.go +++ b/plugins/common/triggers.go @@ -36,7 +36,7 @@ func TriggerInstall() error { return fmt.Errorf("Unable to install the common plugin: %s", err.Error()) } - apps, err := DokkuApps() + apps, err := UnfilteredDokkuApps() if err != nil { return nil } diff --git a/plugins/cron/functions.go b/plugins/cron/functions.go index a3733d0b7..0a3aee197 100644 --- a/plugins/cron/functions.go +++ b/plugins/cron/functions.go @@ -99,7 +99,7 @@ func deleteCrontab() error { } func writeCronEntries() error { - apps, _ := common.DokkuApps() + apps, _ := common.UnfilteredDokkuApps() commands := []templateCommand{} for _, appName := range apps { scheduler := common.GetAppScheduler(appName) diff --git a/plugins/domains/install b/plugins/domains/install index 05e19a657..cbdc6c332 100755 --- a/plugins/domains/install +++ b/plugins/domains/install @@ -8,7 +8,7 @@ trigger-domains-install() { declare trigger="install" shopt -s nullglob - for app in $(dokku_apps); do + for app in $(dokku_apps "false"); do domains_setup "$app" done } diff --git a/plugins/git/install b/plugins/git/install index 738e2f99b..565301253 100755 --- a/plugins/git/install +++ b/plugins/git/install @@ -26,7 +26,7 @@ migrate_git_vars_0_12_0() { DOKKU_QUIET_OUTPUT=1 config_unset --global DOKKU_DEPLOY_BRANCH || true fi - for app in $(dokku_apps); do + for app in $(dokku_apps "false"); do DOKKU_DEPLOY_BRANCH=$(config_get "$app" DOKKU_DEPLOY_BRANCH || true) if [[ -n "$DOKKU_DEPLOY_BRANCH" ]]; then fn-plugin-property-write "git" "$app" "deploy-branch" "$DOKKU_DEPLOY_BRANCH" diff --git a/plugins/logs/functions.go b/plugins/logs/functions.go index 1b696b4a0..0682aec37 100644 --- a/plugins/logs/functions.go +++ b/plugins/logs/functions.go @@ -157,7 +157,7 @@ func sinkValueToConfig(appName string, sinkValue string) (vectorSink, error) { } func writeVectorConfig() error { - apps, _ := common.DokkuApps() + apps, _ := common.UnfilteredDokkuApps() data := vectorConfig{ Sources: map[string]vectorSource{}, Sinks: map[string]vectorSink{}, diff --git a/plugins/network/triggers.go b/plugins/network/triggers.go index 2906c1428..fbdc8d510 100644 --- a/plugins/network/triggers.go +++ b/plugins/network/triggers.go @@ -33,7 +33,7 @@ func TriggerInstall() error { return fmt.Errorf("Unable to install the network plugin: %s", err.Error()) } - apps, err := common.DokkuApps() + apps, err := common.UnfilteredDokkuApps() if err != nil { return nil } diff --git a/plugins/nginx-vhosts/functions b/plugins/nginx-vhosts/functions index 1abe7ae1f..f7514161d 100755 --- a/plugins/nginx-vhosts/functions +++ b/plugins/nginx-vhosts/functions @@ -109,7 +109,7 @@ validate_nginx() { verify_app_name "$APP" nginx_vhosts_validate_single_func "$APP" "$FLAG" else - for app in $(dokku_apps); do + for app in $(dokku_apps "false"); do nginx_vhosts_validate_single_func "$app" "$FLAG" done fi diff --git a/plugins/nginx-vhosts/install b/plugins/nginx-vhosts/install index a9a1f0fd2..4a00206a3 100755 --- a/plugins/nginx-vhosts/install +++ b/plugins/nginx-vhosts/install @@ -95,7 +95,7 @@ trigger-nginx-vhosts-install() { [[ -f /etc/logrotate.d/nginx ]] && sed -i -e 's/invoke-rc.d/service/g' /etc/logrotate.d/nginx # @TODO: Remove this after a few versions - for app in $(dokku_apps); do + for app in $(dokku_apps "false"); do nginx_port="$(config_get "$app" DOKKU_NGINX_PORT || true)" nginx_ssl_port="$(config_get "$app" DOKKU_NGINX_SSL_PORT || true)" if [[ -n "$nginx_port" ]] || [[ -n "$nginx_ssl_port" ]]; then diff --git a/plugins/ps/triggers.go b/plugins/ps/triggers.go index 88c590ef0..918f447ae 100644 --- a/plugins/ps/triggers.go +++ b/plugins/ps/triggers.go @@ -74,7 +74,7 @@ func TriggerInstall() error { return err } - apps, err := common.DokkuApps() + apps, err := common.UnfilteredDokkuApps() if err != nil { return nil } diff --git a/plugins/scheduler/triggers.go b/plugins/scheduler/triggers.go index d5ae16b1d..234d3be1a 100644 --- a/plugins/scheduler/triggers.go +++ b/plugins/scheduler/triggers.go @@ -31,7 +31,7 @@ func TriggerInstall() error { return fmt.Errorf("Unable to install the scheduler plugin: %s", err.Error()) } - apps, err := common.DokkuApps() + apps, err := common.UnfilteredDokkuApps() if err != nil { return nil }