Skip to content

Commit

Permalink
api/krusty: Ensure sort ordering works with CLI flag and kustomization
Browse files Browse the repository at this point in the history
Sort order can be defined in two places:
- (new) kustomization file
- (old) CLI flag
We want the kustomization file to take precedence over the CLI flag.

Eventually, we may want to move away from having a CLI flag altogether:
kubernetes-sigs#3947

Case 1: Sort order set in kustomization file AND in CLI flag.
Print a warning and let the kustomization file take precedence.

Case 2: Sort order set in CLI flag only or not at all.
Follow the CLI flag (defaults to legacy) and reorder at the end.

Case 3: Sort order set in kustomization file only.
Simply build the kustomization.

Signed-off-by: Yannis Zarkadas <yanniszark@gmail.com>
  • Loading branch information
yanniszark committed Jun 19, 2022
1 parent db76383 commit 2fa7d57
Showing 1 changed file with 29 additions and 2 deletions.
31 changes: 29 additions & 2 deletions api/krusty/kustomizer.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ package krusty

import (
"fmt"
"log"

"sigs.k8s.io/kustomize/api/internal/builtins"
pLdr "sigs.k8s.io/kustomize/api/internal/plugins/loader"
Expand All @@ -18,6 +19,7 @@ import (
"sigs.k8s.io/kustomize/api/types"
"sigs.k8s.io/kustomize/kyaml/filesys"
"sigs.k8s.io/kustomize/kyaml/openapi"
"sigs.k8s.io/kustomize/kyaml/resid"
)

// Kustomizer performs kustomizations.
Expand Down Expand Up @@ -90,8 +92,33 @@ func (b *Kustomizer) Run(
if err != nil {
return nil, err
}
if b.options.DoLegacyResourceSort {
err = builtins.NewLegacyOrderTransformerPlugin().Transform(m)
// Sort order can be defined in two places:
// - (new) kustomization file
// - (old) CLI flag
//
// We want the kustomization file to take precedence over the CLI flag.
// Eventually, we may want to move away from having a CLI flag altogether:
// https://github.com/kubernetes-sigs/kustomize/issues/3947

// Case 1: Sort order set in kustomization file AND in CLI flag.
if b.options.SortModeSetExplicitly && kt.Kustomization().SortOptions != nil {
log.Println("Warning: Sorting order is set both in 'kustomization.yaml'" +
" ('sortOptions') and in a CLI flag ('--reorder'). Using the" +
" kustomization file over the CLI flag.")
}

// Case 2: Sort order set in CLI flag only or not at all.
if b.options.DoLegacyResourceSort && kt.Kustomization().SortOptions == nil {
pl := &builtins.SortOrderTransformerPlugin{
SortOptions: &types.SortOptions{
Order: types.LegacySortOrder,
LegacySortOptions: &types.LegacySortOptions{
OrderFirst: resid.OrderFirst,
OrderLast: resid.OrderLast,
},
},
}
err = pl.Transform(m)
if err != nil {
return nil, err
}
Expand Down

0 comments on commit 2fa7d57

Please sign in to comment.