Skip to content

Commit 28b3872

Browse files
authored
Merge pull request #2600 from lizardruss/devspace-space
feat: add variables to support migrating DevSpace Cloud users
2 parents a98ad6a + 15405ca commit 28b3872

File tree

4 files changed

+8
-1
lines changed

4 files changed

+8
-1
lines changed

e2e/tests/hooks/hooks.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -221,6 +221,9 @@ var _ = DevSpaceDescribe("hooks", func() {
221221
// check namespace hook
222222
framework.ExpectLocalFileContentsImmediately("namespace.txt", ns)
223223

224+
// check space hook
225+
framework.ExpectLocalFileContentsImmediately("space.txt", ns)
226+
224227
// stop second command
225228
cancel2()
226229

e2e/tests/hooks/testdata/once/devspace.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ deployments:
1414
hooks:
1515
- command: |
1616
echo -n $DEVSPACE_NAMESPACE > namespace.txt
17+
echo -n $DEVSPACE_SPACE > space.txt
1718
events: ["after:deploy"]
1819
- command: |
1920
mkdir -p /app

pkg/devspace/config/loader/variable/predefined_variable.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,9 @@ func init() {
138138
predefinedVars["devspace.git.commit"] = predefinedVars["DEVSPACE_GIT_COMMIT"]
139139
predefinedVars["devspace.context"] = predefinedVars["DEVSPACE_CONTEXT"]
140140
predefinedVars["devspace.namespace"] = predefinedVars["DEVSPACE_NAMESPACE"]
141+
142+
// support migrating DevSpace Cloud users
143+
predefinedVars["DEVSPACE_SPACE"] = predefinedVars["DEVSPACE_NAMESPACE"]
141144
}
142145

143146
func IsPredefinedVariable(name string) bool {

pkg/devspace/config/loader/variable/resolver.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import (
2020
"github.com/pkg/errors"
2121
)
2222

23-
var AlwaysResolvePredefinedVars = []string{"DEVSPACE_NAME", "DEVSPACE_EXECUTABLE", "DEVSPACE_KUBECTL_EXECUTABLE", "DEVSPACE_TMPDIR", "DEVSPACE_VERSION", "DEVSPACE_RANDOM", "DEVSPACE_PROFILE", "DEVSPACE_PROFILES", "DEVSPACE_USER_HOME", "DEVSPACE_TIMESTAMP", "devspace.context", "DEVSPACE_CONTEXT", "devspace.namespace", "DEVSPACE_NAMESPACE"}
23+
var AlwaysResolvePredefinedVars = []string{"DEVSPACE_NAME", "DEVSPACE_EXECUTABLE", "DEVSPACE_KUBECTL_EXECUTABLE", "DEVSPACE_TMPDIR", "DEVSPACE_VERSION", "DEVSPACE_RANDOM", "DEVSPACE_PROFILE", "DEVSPACE_PROFILES", "DEVSPACE_USER_HOME", "DEVSPACE_TIMESTAMP", "devspace.context", "DEVSPACE_CONTEXT", "devspace.namespace", "DEVSPACE_NAMESPACE", "DEVSPACE_SPACE"}
2424

2525
// NewResolver creates a new resolver that caches resolved variables in memory and in the provided cache
2626
func NewResolver(localCache localcache.Cache, predefinedVariableOptions *PredefinedVariableOptions, flags []string, log log.Logger) (Resolver, error) {

0 commit comments

Comments
 (0)