Merge pull request #1152 from Liujingfang1/patchfix

fix the bug for patching CRDs
This commit is contained in:
Kubernetes Prow Robot
2019-06-04 11:53:54 -07:00
committed by GitHub
3 changed files with 6 additions and 7 deletions

View File

@@ -97,8 +97,8 @@ func (tf *transformer) Transform(baseResourceMap resmap.ResMap) error {
return err return err
} }
} }
base.SetName(baseName)
baseResourceMap[id].SetMap(merged) baseResourceMap[id].SetMap(merged)
base.SetName(baseName)
} }
return nil return nil
} }

View File

@@ -295,7 +295,7 @@ spec:
if err != nil { if err != nil {
t.Fatalf("Err: %v", err) t.Fatalf("Err: %v", err)
} }
// TODO(#669): Bee's name should be "prod-x-bee", not "prod-bee".
th.AssertActualEqualsExpected(m, ` th.AssertActualEqualsExpected(m, `
apiVersion: v1 apiVersion: v1
data: data:
@@ -310,14 +310,14 @@ metadata:
name: prod-x-mykind name: prod-x-mykind
spec: spec:
beeRef: beeRef:
name: prod-bee name: prod-x-bee
secretRef: secretRef:
name: prod-x-crdsecret name: prod-x-crdsecret
--- ---
apiVersion: v1beta1 apiVersion: v1beta1
kind: Bee kind: Bee
metadata: metadata:
name: prod-bee name: prod-x-bee
spec: spec:
action: makehoney action: makehoney
`) `)

View File

@@ -274,15 +274,14 @@ spec:
if err != nil { if err != nil {
t.Fatalf("Err: %v", err) t.Fatalf("Err: %v", err)
} }
// TODO(#669): The name of AnimalPark should be x-o-sandiego,
// not o-sandiego, since AnimalPark appears in the base.
th.AssertActualEqualsExpected(m, ` th.AssertActualEqualsExpected(m, `
kind: AnimalPark kind: AnimalPark
metadata: metadata:
labels: labels:
app: myApp app: myApp
movie: planetOfTheApes movie: planetOfTheApes
name: o-sandiego name: o-x-sandiego
spec: spec:
food: food:
- mimosa - mimosa