Deprecate Config, replace with StackConfigDir

CHANGELOG_PENDING.md

@@ -15,6 +15,9 @@
   edit the code to successfully run.
   [#8922](https://github.com/pulumi/pulumi/pull/8922)
 
+- [cli/config] Rename the `config` property in `Pulumi.yaml` to `stacksDirectory`. The `config` key will continue to be supported.
+  [#9145](https://github.com/pulumi/pulumi/pull/9145)
+
 ### Bug Fixes
 
 - [sdk/python] - Fix build warnings. See

sdk/go/common/workspace/paths.go

@@ -1,4 +1,4 @@
-// Copyright 2016-2018, Pulumi Corporation.
+// Copyright 2016-2022, Pulumi Corporation.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -98,8 +98,28 @@ func DetectProjectStackPath(stackName tokens.QName) (string, error) {
 		return "", err
 	}
 
-	return filepath.Join(filepath.Dir(projPath), proj.Config, fmt.Sprintf("%s.%s%s", ProjectFile, qnameFileName(stackName),
-		filepath.Ext(projPath))), nil
+	fileName := fmt.Sprintf("%s.%s%s", ProjectFile, qnameFileName(stackName), filepath.Ext(projPath))
+
+	// Back compat: StacksDirectory used to be called Config.
+	configValue, hasConfigValue := proj.Config.(string)
+	hasConfigValue = hasConfigValue && configValue != ""
+
+	if proj.StacksDirectory != "" {
+		// If config and stacksDirectory are both set return an error
+		if hasConfigValue {
+			return "", fmt.Errorf("can not set `config` and `stacksDirectory`, remove the `config` entry")
+		}
+
+		return filepath.Join(filepath.Dir(projPath), proj.StacksDirectory, fileName), nil
+	}
+
+	// Back compat: If StacksDirectory is not present and Config is given and it's a non-empty string use it
+	// for the stacks directory.
+	if hasConfigValue {
+		return filepath.Join(filepath.Dir(projPath), configValue, fileName), nil
+	}
+
+	return filepath.Join(filepath.Dir(projPath), fileName), nil
 }
 
 // DetectProjectPathFrom locates the closest project from the given path, searching "upwards" in the directory

sdk/go/common/workspace/paths_test.go

@@ -0,0 +1,104 @@
+// Copyright 2016-2022, Pulumi Corporation.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package workspace
+
+import (
+	"io/ioutil"
+	"os"
+	"path/filepath"
+	"testing"
+
+	"github.com/pulumi/pulumi/sdk/v3/go/common/tokens"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestDetectProjectAndPath(t *testing.T) {
+	tmpDir, err := ioutil.TempDir("", "projecttest")
+	assert.NoError(t, err)
+	cwd, err := os.Getwd()
+	assert.NoError(t, err)
+	defer func() { err := os.Chdir(cwd); assert.NoError(t, err) }()
+	err = os.Chdir(tmpDir)
+	assert.NoError(t, err)
+
+	yamlPath := filepath.Join(tmpDir, "Pulumi.yaml")
+	yamlContents :=
+		"name: some_project\ndescription: Some project\nruntime: nodejs\n"
+
+	err = os.WriteFile(yamlPath, []byte(yamlContents), 0600)
+	assert.NoError(t, err)
+
+	project, path, err := DetectProjectAndPath()
+	assert.NoError(t, err)
+	assert.Equal(t, yamlPath, path)
+	assert.Equal(t, tokens.PackageName("some_project"), project.Name)
+	assert.Equal(t, "Some project", *project.Description)
+	assert.Equal(t, "nodejs", project.Runtime.name)
+}
+
+func TestProjectStackPath(t *testing.T) {
+	expectedPath := func(expectedPath string) func(t *testing.T, projectDir, path string, err error) {
+		return func(t *testing.T, projectDir, path string, err error) {
+			assert.NoError(t, err)
+			assert.Equal(t, filepath.Join(projectDir, expectedPath), path)
+		}
+	}
+
+	tests := []struct {
+		name         string
+		yamlContents string
+		validate     func(t *testing.T, projectDir, path string, err error)
+	}{{
+		"WithoutStacksDirectory",
+		"name: some_project\ndescription: Some project\nruntime: nodejs\n",
+		expectedPath("Pulumi.my_stack.yaml"),
+	}, {
+		"WithStacksDirectory",
+		"name: some_project\ndescription: Some project\nruntime: nodejs\nstacksDirectory: stacks\n",
+		expectedPath(filepath.Join("stacks", "Pulumi.my_stack.yaml")),
+	}, {
+		"WithConfig",
+		"name: some_project\ndescription: Some project\nruntime: nodejs\nconfig: stacks\n",
+		expectedPath(filepath.Join("stacks", "Pulumi.my_stack.yaml")),
+	}, {
+		"WithBoth",
+		"name: some_project\ndescription: Some project\nruntime: nodejs\nconfig: stacks\nstacksDirectory: stacks\n",
+		func(t *testing.T, projectDir, path string, err error) {
+			assert.Error(t, err)
+		},
+	}}
+
+	for _, tt := range tests {
+		tt := tt
+		t.Run(tt.name, func(t *testing.T) {
+			tmpDir, err := ioutil.TempDir("", "projecttest")
+			assert.NoError(t, err)
+			cwd, err := os.Getwd()
+			assert.NoError(t, err)
+			defer func() { err := os.Chdir(cwd); assert.NoError(t, err) }()
+			err = os.Chdir(tmpDir)
+			assert.NoError(t, err)
+
+			err = os.WriteFile(
+				filepath.Join(tmpDir, "Pulumi.yaml"),
+				[]byte(tt.yamlContents),
+				0600)
+			assert.NoError(t, err)
+
+			path, err := DetectProjectStackPath("my_stack")
+			tt.validate(t, tmpDir, path, err)
+		})
+	}
+}

sdk/go/common/workspace/project.go

@@ -1,4 +1,4 @@
-// Copyright 2016-2018, Pulumi Corporation.
+// Copyright 2016-2022, Pulumi Corporation.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -87,8 +87,12 @@ type Project struct {
 	// License is the optional license governing this project's usage.
 	License *string `json:"license,omitempty" yaml:"license,omitempty"`
 
-	// Config indicates where to store the Pulumi.<stack-name>.yaml files, combined with the folder Pulumi.yaml is in.
-	Config string `json:"config,omitempty" yaml:"config,omitempty"`
+	// Config has been renamed to StacksDirectory.
+	Config interface{} `json:"config,omitempty" yaml:"config,omitempty"`
+
+	// StacksDirectory indicates where to store the Pulumi.<stack-name>.yaml files, combined with the folder
+	// Pulumi.yaml is in.
+	StacksDirectory string `json:"stacksDirectory,omitempty" yaml:"stacksDirectory,omitempty"`
 
 	// Template is an optional template manifest, if this project is a template.
 	Template *ProjectTemplate `json:"template,omitempty" yaml:"template,omitempty"`