Skip to content

Commit 5e7085a

Browse files
committed
refactor(manifest): Rename Environments field in Manifest to SelectedEnvironments
This makes it clear what is contained here: the subset of environments from the manifest actually selected for use.
1 parent 0af2528 commit 5e7085a

25 files changed

+79
-76
lines changed

cmd/monaco/completion/completion.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ func loadEnvironmentsFromManifest(manifestPath string) ([]string, cobra.ShellCom
9797
ManifestPath: manifestPath,
9898
})
9999

100-
return maps.Keys(man.Environments), cobra.ShellCompDirectiveDefault
100+
return maps.Keys(man.SelectedEnvironments), cobra.ShellCompDirectiveDefault
101101
}
102102

103103
func AccountsByManifestFlag(cmd *cobra.Command, _ []string, _ string) ([]string, cobra.ShellCompDirective) {

cmd/monaco/delete/command.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ func GetDeleteCommand(fs afero.Fs) (deleteCmd *cobra.Command) {
8181
return fmt.Errorf("encountered errors while parsing %s: %w", deleteFile, err)
8282
}
8383

84-
return Delete(cmd.Context(), manifest.Environments, entriesToDelete)
84+
return Delete(cmd.Context(), manifest.SelectedEnvironments, entriesToDelete)
8585
},
8686
ValidArgsFunction: completion.DeleteCompletion,
8787
}

cmd/monaco/deploy/deploy.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func deployConfigs(ctx context.Context, fs afero.Fs, manifestPath string, enviro
5151
return err
5252
}
5353

54-
ok := verifyEnvironmentGen(ctx, loadedManifest.Environments, dryRun)
54+
ok := verifyEnvironmentGen(ctx, loadedManifest.SelectedEnvironments, dryRun)
5555
if !ok {
5656
return fmt.Errorf("unable to verify Dynatrace environment generation")
5757
}
@@ -61,21 +61,21 @@ func deployConfigs(ctx context.Context, fs afero.Fs, manifestPath string, enviro
6161
return err
6262
}
6363

64-
if err := validateProjectsWithEnvironments(ctx, loadedProjects, loadedManifest.Environments); err != nil {
64+
if err := validateProjectsWithEnvironments(ctx, loadedProjects, loadedManifest.SelectedEnvironments); err != nil {
6565
return err
6666
}
6767

6868
logging.LogProjectsInfo(loadedProjects)
69-
logging.LogEnvironmentsInfo(loadedManifest.Environments)
69+
logging.LogEnvironmentsInfo(loadedManifest.SelectedEnvironments)
7070

71-
err = validateAuthenticationWithProjectConfigs(loadedProjects, loadedManifest.Environments)
71+
err = validateAuthenticationWithProjectConfigs(loadedProjects, loadedManifest.SelectedEnvironments)
7272
if err != nil {
7373
formattedErr := fmt.Errorf("manifest auth field misconfigured: %w", err)
7474
report.GetReporterFromContextOrDiscard(ctx).ReportLoading(report.StateError, formattedErr, "", nil)
7575
return formattedErr
7676
}
7777

78-
clientSets, err := dynatrace.CreateEnvironmentClients(ctx, loadedManifest.Environments, dryRun)
78+
clientSets, err := dynatrace.CreateEnvironmentClients(ctx, loadedManifest.SelectedEnvironments, dryRun)
7979
if err != nil {
8080
formattedErr := fmt.Errorf("failed to create API clients: %w", err)
8181
report.GetReporterFromContextOrDiscard(ctx).ReportLoading(report.StateError, formattedErr, "", nil)

cmd/monaco/download/download_configs.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ func (d DefaultCommand) DownloadConfigsBasedOnManifest(ctx context.Context, fs a
9090
return err
9191
}
9292

93-
env, found := m.Environments[cmdOptions.specificEnvironmentName]
93+
env, found := m.SelectedEnvironments[cmdOptions.specificEnvironmentName]
9494
if !found {
9595
return fmt.Errorf("environment %q was not available in manifest %q", cmdOptions.specificEnvironmentName, cmdOptions.manifestFile)
9696
}

cmd/monaco/generate/dependencygraph/dependencygraph.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ func writeGraphFiles(ctx context.Context, fs afero.Fs, manifestPath string, envi
104104
})
105105
}
106106

107-
graphs := graph.New(projects, m.Environments.Names(), opts...)
107+
graphs := graph.New(projects, m.SelectedEnvironments.Names(), opts...)
108108

109109
folderPath, err := filepath.Abs(outputFolder)
110110
if err != nil {
@@ -128,7 +128,7 @@ func writeGraphFiles(ctx context.Context, fs afero.Fs, manifestPath string, envi
128128
}
129129
}
130130

131-
for _, e := range m.Environments.Names() {
131+
for _, e := range m.SelectedEnvironments.Names() {
132132
b, err := graphs.EncodeToDOT(e)
133133
if err != nil {
134134
return ExportError{

cmd/monaco/integrationtest/assert.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,9 @@ func AssertAllConfigsAvailability(t *testing.T, fs afero.Fs, manifestPath string
7171

7272
projects := LoadProjects(t, fs, manifestPath, loadedManifest)
7373

74-
envNames := make([]string, 0, len(loadedManifest.Environments))
74+
envNames := make([]string, 0, len(loadedManifest.SelectedEnvironments))
7575

76-
for _, env := range loadedManifest.Environments {
76+
for _, env := range loadedManifest.SelectedEnvironments {
7777
envNames = append(envNames, env.Name)
7878
}
7979

@@ -100,7 +100,7 @@ func AssertAllConfigsAvailability(t *testing.T, fs afero.Fs, manifestPath string
100100

101101
for envName, configs := range sortedConfigs {
102102

103-
env := loadedManifest.Environments[envName]
103+
env := loadedManifest.SelectedEnvironments[envName]
104104

105105
clients := CreateDynatraceClients(t, env)
106106

cmd/monaco/integrationtest/v2/delete_integration_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@ environmentGroups:
254254
assert.Empty(t, errs)
255255

256256
envName := "environment"
257-
env := man.Environments[envName]
257+
env := man.SelectedEnvironments[envName]
258258
clientSet := integrationtest.CreateDynatraceClients(t, env)
259259

260260
// check the setting was deleted
@@ -339,7 +339,7 @@ configs:
339339
assert.Empty(t, errs)
340340

341341
envName := "environment"
342-
env := man.Environments[envName]
342+
env := man.SelectedEnvironments[envName]
343343
clientSet := integrationtest.CreateDynatraceClients(t, env)
344344
apis := api.NewAPIs()
345345

cmd/monaco/integrationtest/v2/diff_project_diff_ext_id_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ func TestSettingsInDifferentProjectsGetDifferentExternalIDs(t *testing.T) {
4747

4848
var manifestPath = diffProjectDiffExtIDFolderManifest
4949
loadedManifest := integrationtest.LoadManifest(t, fs, manifestPath, "")
50-
environment := loadedManifest.Environments["platform_env"]
50+
environment := loadedManifest.SelectedEnvironments["platform_env"]
5151
projects := integrationtest.LoadProjects(t, fs, manifestPath, loadedManifest)
5252
sortedConfigs, _ := graph.SortProjects(projects, []string{"platform_env"})
5353

cmd/monaco/integrationtest/v2/documents_integration_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ func TestDocuments(t *testing.T) {
5757
assert.Empty(t, errs)
5858

5959
// check isPrivate == false
60-
clientSet := integrationtest.CreateDynatraceClients(t, man.Environments[environment])
60+
clientSet := integrationtest.CreateDynatraceClients(t, man.SelectedEnvironments[environment])
6161
result, err := clientSet.DocumentClient.List(t.Context(), fmt.Sprintf("name='my-notebook_%s'", testContext.suffix))
6262
assert.NoError(t, err)
6363
assert.Len(t, result.Responses, 1)

cmd/monaco/integrationtest/v2/scoped_configs_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ func assertOverallDashboardSharedState(t *testing.T, fs afero.Fs, testContext Te
6767
})
6868
assert.Empty(t, errs)
6969

70-
environmentDefinition := man.Environments[environment]
70+
environmentDefinition := man.SelectedEnvironments[environment]
7171
clientSet := integrationtest.CreateDynatraceClients(t, environmentDefinition)
7272
apis := api.NewAPIs()
7373

cmd/monaco/integrationtest/v2/segments_integration_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ func createSegmentsClient(t *testing.T, fs afero.Fs, manifestPath string, enviro
166166
})
167167
assert.Empty(t, errs)
168168

169-
clientSet := integrationtest.CreateDynatraceClients(t, man.Environments[environment])
169+
clientSet := integrationtest.CreateDynatraceClients(t, man.SelectedEnvironments[environment])
170170
return clientSet.SegmentClient
171171
}
172172

cmd/monaco/integrationtest/v2/settings_acl_integration_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ func TestSettingsWithACL(t *testing.T) {
8181
require.NoError(t, err)
8282

8383
loadedManifest := integrationtest.LoadManifest(t, fs, manifestPath, environment)
84-
environmentDefinition := loadedManifest.Environments[environment]
84+
environmentDefinition := loadedManifest.SelectedEnvironments[environment]
8585
client := createSettingsClientPlatform(t, environmentDefinition)
8686

8787
coord := coordinate.Coordinate{

cmd/monaco/integrationtest/v2/settings_integration_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ func TestOldExternalIDGetsUpdated(t *testing.T) {
9090
loadedManifest := integrationtest.LoadManifest(t, fs, manifestPath, env)
9191
projects := integrationtest.LoadProjects(t, fs, manifestPath, loadedManifest)
9292
sortedConfigs, _ := graph.SortProjects(projects, []string{env})
93-
environment := loadedManifest.Environments[env]
93+
environment := loadedManifest.SelectedEnvironments[env]
9494
configToDeploy := sortedConfigs[env][0]
9595

9696
defer func() {
@@ -210,7 +210,7 @@ func TestOrderedSettings(t *testing.T) {
210210
integrationtest.AssertAllConfigsAvailability(t, fs, manifestPath, []string{"project"}, "platform_env", true)
211211

212212
loadedManifest := integrationtest.LoadManifest(t, fs, manifestPath, "platform_env")
213-
environment := loadedManifest.Environments["platform_env"]
213+
environment := loadedManifest.SelectedEnvironments["platform_env"]
214214
settingsClient := createSettingsClient(t, environment)
215215

216216
results, err := settingsClient.List(t.Context(), "builtin:processavailability", dtclient.ListSettingsOptions{
@@ -235,7 +235,7 @@ func TestOrderedSettings(t *testing.T) {
235235
integrationtest.AssertAllConfigsAvailability(t, fs, manifestPath, []string{"project"}, "platform_env", true)
236236

237237
loadedManifest := integrationtest.LoadManifest(t, fs, manifestPath, "platform_env")
238-
environment := loadedManifest.Environments["platform_env"]
238+
environment := loadedManifest.SelectedEnvironments["platform_env"]
239239
settingsClient := createSettingsClient(t, environment)
240240

241241
results, err := settingsClient.List(t.Context(), "builtin:processavailability", dtclient.ListSettingsOptions{
@@ -267,7 +267,7 @@ func TestOrderedSettingsCrossProjects(t *testing.T) {
267267
integrationtest.AssertAllConfigsAvailability(t, fs, manifestPath, []string{"source"}, "platform_env", true)
268268

269269
loadedManifest := integrationtest.LoadManifest(t, fs, manifestPath, "platform_env")
270-
environment := loadedManifest.Environments["platform_env"]
270+
environment := loadedManifest.SelectedEnvironments["platform_env"]
271271
settingsClient := createSettingsClient(t, environment)
272272
results, err := settingsClient.List(t.Context(), schema, dtclient.ListSettingsOptions{
273273
DiscardValue: true,
@@ -556,7 +556,7 @@ func createSettingsClientFromManifest(t *testing.T, fs afero.Fs, manifestPath st
556556
})
557557
assert.Empty(t, errs)
558558

559-
clientSet := integrationtest.CreateDynatraceClients(t, man.Environments[environment])
559+
clientSet := integrationtest.CreateDynatraceClients(t, man.SelectedEnvironments[environment])
560560
return clientSet.SettingsClient
561561
}
562562

cmd/monaco/integrationtest/v2/skip_e2e_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ func TestSkip(t *testing.T) {
103103
loadedManifest := integrationtest.LoadManifest(t, afero.OsFs{}, manifest, "")
104104
clients := make(map[string]client.SettingsClient)
105105

106-
for name, def := range loadedManifest.Environments {
106+
for name, def := range loadedManifest.SelectedEnvironments {
107107
set := integrationtest.CreateDynatraceClients(t, def)
108108
clients[name] = set.SettingsClient
109109
}

cmd/monaco/integrationtest/v2/slo-v2_integration_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ func createSloV2Client(t *testing.T, fs afero.Fs, manifestPath string, environme
118118
})
119119
assert.Empty(t, errs)
120120

121-
clientSet := integrationtest.CreateDynatraceClients(t, man.Environments[environment])
121+
clientSet := integrationtest.CreateDynatraceClients(t, man.SelectedEnvironments[environment])
122122
return clientSet.ServiceLevelObjectiveClient
123123
}
124124

cmd/monaco/purge/purge.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ func purge(ctx context.Context, fs afero.Fs, deploymentManifestPath string, envi
5757
return errors.New("error while loading manifest")
5858
}
5959

60-
return purgeConfigs(ctx, maps.Values(mani.Environments), apis)
60+
return purgeConfigs(ctx, maps.Values(mani.SelectedEnvironments), apis)
6161
}
6262

6363
func purgeConfigs(ctx context.Context, environments []manifest.EnvironmentDefinition, apis api.APIs) error {

pkg/download/download_writer.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ func writeToDisk(fs afero.Fs, writerContext WriterContext) error {
7070

7171
manifest := manifest.Manifest{
7272
Projects: projectDefinition,
73-
Environments: map[string]manifest.EnvironmentDefinition{
73+
SelectedEnvironments: map[string]manifest.EnvironmentDefinition{
7474
writerContext.ProjectToWrite.Id: {
7575
Name: writerContext.ProjectToWrite.Id,
7676
URL: writerContext.EnvironmentUrl,

pkg/manifest/loader/manifest_loader.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -202,9 +202,9 @@ func Load(context *Context) (manifest.Manifest, []error) {
202202
}
203203

204204
return manifest.Manifest{
205-
Projects: projectDefinitions,
206-
Environments: environmentDefinitions,
207-
Accounts: accounts,
205+
Projects: projectDefinitions,
206+
SelectedEnvironments: environmentDefinitions,
207+
Accounts: accounts,
208208
}, nil
209209
}
210210

0 commit comments

Comments
 (0)