Merge pull request #244 from mengqiy/multitransformer

add a constructor for multitransformer
This commit is contained in:
k8s-ci-robot 2018-02-02 10:05:11 -08:00 committed by GitHub
commit c8be5e44f5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 3 deletions

View File

@ -56,5 +56,5 @@ func DefaultTransformer(m *manifest.Manifest) (Transformer, error) {
if nrt != nil {
transformers = append(transformers, nrt)
}
return &MultiTransformer{transformers}, nil
return NewMultiTransformer(transformers), nil
}

View File

@ -23,14 +23,22 @@ import (
// MultiTransformer contains a list of transformers.
type MultiTransformer struct {
Transformers []Transformer
transformers []Transformer
}
var _ Transformer = &MultiTransformer{}
// NewMultiTransformer constructs a MultiTransformer.
func NewMultiTransformer(t []Transformer) Transformer {
r := &MultiTransformer{
transformers: make([]Transformer, len(t))}
copy(r.transformers, t)
return r
}
// Transform prepends the name prefix.
func (o *MultiTransformer) Transform(m map[gvkn.GroupVersionKindName]*unstructured.Unstructured) error {
for _, t := range o.Transformers {
for _, t := range o.transformers {
err := t.Transform(m)
if err != nil {
return err