github/dokku
0
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity

fix: run global env migration before listing apps and set DOKKU_LIB_ROOT in tests

Browse Source 
- Move migrateGlobalEnv() before UnfilteredDokkuApps() so the global
  config directory is created even on fresh installs with no apps
- Add DOKKU_LIB_ROOT to Go test env vars in tests.mk
This commit is contained in:
Jose Diaz-Gonzalez
2026-04-25 03:26:42 -04:00
parent d846324fd5
commit 17c549ee63
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
plugins/config/triggers.go
View File

@@ -99,15 +99,15 @@ func TriggerInstall() error {
return fmt.Errorf("Unable to install the config plugin: %s", err.Error()) return fmt.Errorf("Unable to install the config plugin: %s", err.Error())
} }
if err := migrateGlobalEnv(); err != nil {
return fmt.Errorf("Unable to migrate global environment: %s", err.Error())
}
apps, err := common.UnfilteredDokkuApps() apps, err := common.UnfilteredDokkuApps()
if err != nil { if err != nil {
return nil return nil
} }
if err := migrateGlobalEnv(); err != nil {
return fmt.Errorf("Unable to migrate global environment: %s", err.Error())
}
// migrate all app ENV files to config path // migrate all app ENV files to config path
for _, appName := range apps { for _, appName := range apps {
if err := common.PropertySetupApp("config", appName); err != nil { if err := common.PropertySetupApp("config", appName); err != nil {