Skip to content

Commit c6dc108

Browse files
authored
fix: do not drop name field from platform manifests (#705)
1 parent e4b1756 commit c6dc108

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

src/repo.ts

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -412,19 +412,20 @@ export function renderManifest(fileMap: FileMap, jsonPath: jsonpath.PathComponen
412412
if (fileMap.kind === 'AplTeamWorkloadValues') {
413413
return { values: data.values }
414414
}
415-
const manifest = {
415+
let spec = data
416+
const labels = {}
417+
if (fileMap.resourceGroup === 'team') {
418+
spec = omit(data, ['id', 'name', 'teamId'])
419+
labels['apl.io/teamId'] = getTeamNameFromJsonPath(jsonPath)
420+
}
421+
return {
416422
kind: fileMap.kind,
417423
metadata: {
418424
name: getResourceName(fileMap, jsonPath, data),
419-
labels: {},
425+
labels,
420426
},
421-
spec: omit(data, ['id', 'teamId', 'name']),
427+
spec,
422428
}
423-
if (fileMap.resourceGroup === 'team') {
424-
manifest.metadata.labels['apl.io/teamId'] = getTeamNameFromJsonPath(jsonPath)
425-
}
426-
427-
return manifest
428429
}
429430

430431
export function renderManifestForSecrets(fileMap: FileMap, resourceName: string, data: Record<string, any>) {

0 commit comments

Comments
 (0)