Skip to content

Commit 2801041

Browse files
authored
Make fields of Resource Filter Flags accessible outside the package (#941)
Signed-off-by: Rohit Aggarwal <[email protected]> Co-authored-by: Rohit Aggarwal <[email protected]>
1 parent 32b2b6f commit 2801041

File tree

1 file changed

+20
-20
lines changed

1 file changed

+20
-20
lines changed

pkg/kapp/cmd/tools/resource_filter_flags.go

+20-20
Original file line numberDiff line numberDiff line change
@@ -12,23 +12,23 @@ import (
1212
)
1313

1414
type ResourceFilterFlags struct {
15-
age string
16-
rf ctlres.ResourceFilter
17-
bf string
15+
Age string
16+
Rf ctlres.ResourceFilter
17+
Bf string
1818
}
1919

2020
func (s *ResourceFilterFlags) Set(cmd *cobra.Command) {
21-
cmd.Flags().StringVar(&s.age, "filter-age", "", "Set age filter (example: 5m-, 500h+, 10m-)")
21+
cmd.Flags().StringVar(&s.Age, "filter-age", "", "Set age filter (example: 5m-, 500h+, 10m-)")
2222

23-
cmd.Flags().StringSliceVar(&s.rf.Kinds, "filter-kind", nil, "Set kinds filter (example: Pod) (can repeat)")
24-
cmd.Flags().StringSliceVar(&s.rf.Namespaces, "filter-ns", nil, "Set namespace filter (example: knative-serving) (can repeat)")
25-
cmd.Flags().StringSliceVar(&s.rf.Names, "filter-name", nil, "Set name filter (example: controller) (can repeat)")
26-
cmd.Flags().StringSliceVar(&s.rf.KindNames, "filter-kind-name", nil, "Set kind-name filter (example: Pod/controller) (can repeat)")
27-
cmd.Flags().StringSliceVar(&s.rf.KindNamespaces, "filter-kind-ns", nil, "Set kind-namespace filter (example: Pod/, Pod/knative-serving) (can repeat)")
28-
cmd.Flags().StringSliceVar(&s.rf.KindNsNames, "filter-kind-ns-name", nil, "Set kind-namespace-name filter (example: Deployment/knative-serving/controller) (can repeat)")
29-
cmd.Flags().StringSliceVar(&s.rf.Labels, "filter-labels", nil, "Set label filter (example: x=y)")
23+
cmd.Flags().StringSliceVar(&s.Rf.Kinds, "filter-kind", nil, "Set kinds filter (example: Pod) (can repeat)")
24+
cmd.Flags().StringSliceVar(&s.Rf.Namespaces, "filter-ns", nil, "Set namespace filter (example: knative-serving) (can repeat)")
25+
cmd.Flags().StringSliceVar(&s.Rf.Names, "filter-name", nil, "Set name filter (example: controller) (can repeat)")
26+
cmd.Flags().StringSliceVar(&s.Rf.KindNames, "filter-kind-name", nil, "Set kind-name filter (example: Pod/controller) (can repeat)")
27+
cmd.Flags().StringSliceVar(&s.Rf.KindNamespaces, "filter-kind-ns", nil, "Set kind-namespace filter (example: Pod/, Pod/knative-serving) (can repeat)")
28+
cmd.Flags().StringSliceVar(&s.Rf.KindNsNames, "filter-kind-ns-name", nil, "Set kind-namespace-name filter (example: Deployment/knative-serving/controller) (can repeat)")
29+
cmd.Flags().StringSliceVar(&s.Rf.Labels, "filter-labels", nil, "Set label filter (example: x=y)")
3030

31-
cmd.Flags().StringVar(&s.bf, "filter", "", `Set filter (example: {"and":[{"not":{"resource":{"kinds":["foo%"]}}},{"resource":{"kinds":["!foo"]}}]})`)
31+
cmd.Flags().StringVar(&s.Bf, "filter", "", `Set filter (example: {"and":[{"not":{"resource":{"kinds":["foo%"]}}},{"resource":{"kinds":["!foo"]}}]})`)
3232
}
3333

3434
func (s *ResourceFilterFlags) ResourceFilter() (ctlres.ResourceFilter, error) {
@@ -37,12 +37,12 @@ func (s *ResourceFilterFlags) ResourceFilter() (ctlres.ResourceFilter, error) {
3737
return ctlres.ResourceFilter{}, err
3838
}
3939

40-
rf := s.rf
40+
rf := s.Rf
4141
rf.CreatedAtAfterTime = createdAtAfterTime
4242
rf.CreatedAtBeforeTime = createdAtBeforeTime
4343

44-
if len(s.bf) > 0 {
45-
boolFilter, err := ctlres.NewBoolFilterFromString(s.bf)
44+
if len(s.Bf) > 0 {
45+
boolFilter, err := ctlres.NewBoolFilterFromString(s.Bf)
4646
if err != nil {
4747
return ctlres.ResourceFilter{}, err
4848
}
@@ -54,21 +54,21 @@ func (s *ResourceFilterFlags) ResourceFilter() (ctlres.ResourceFilter, error) {
5454
}
5555

5656
func (s *ResourceFilterFlags) Times() (*time.Time, *time.Time, error) {
57-
if len(s.age) == 0 {
57+
if len(s.Age) == 0 {
5858
return nil, nil, nil
5959
}
6060

6161
var ageStr string
6262
var ageOlder bool
6363

64-
lastIdx := len(s.age) - 1
64+
lastIdx := len(s.Age) - 1
6565

66-
switch string(s.age[lastIdx]) {
66+
switch string(s.Age[lastIdx]) {
6767
case "+":
68-
ageStr = s.age[:lastIdx]
68+
ageStr = s.Age[:lastIdx]
6969
ageOlder = true
7070
case "-":
71-
ageStr = s.age[:lastIdx]
71+
ageStr = s.Age[:lastIdx]
7272
}
7373

7474
dur, err := time.ParseDuration(ageStr)

0 commit comments

Comments
 (0)