Skip to content

Commit cda232e

Browse files
committed
Changed codes nits
Signed-off-by: Yuri Sa <[email protected]>
1 parent 1971b71 commit cda232e

File tree

4 files changed

+5
-7
lines changed

4 files changed

+5
-7
lines changed

cmd/check.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -89,8 +89,8 @@ func init() {
8989
"specifies the catalogsource to test against")
9090
flags.StringVar(&checkflags.CatalogSourceNamespace, "catalogsourcenamespace", "openshift-marketplace",
9191
"specifies the namespace where the catalogsource exists")
92-
flags.StringSliceVar(&checkflags.AuditPlan, "auditplan", defaultAuditPlan, "audit plan is the ordered list of operator test functions to be called during a capability audit.")
92+
flags.StringSliceVar(&checkflags.AuditPlan, "auditplan", defaultAuditPlan, "audit plan is the ordered list of operator test functions to be called during a capability audit")
9393
flags.BoolVar(&checkflags.ListPackages, "list-packages", false, "list packages in the catalog.")
94-
flags.StringSliceVar(&checkflags.FilterPackages, "filter-packages", []string{}, "a list of package(s) which limits audits and/or other flag(s) output.")
95-
flags.BoolVar(&checkflags.AllInstallModes, "all-installmodes", false, "when set, all install modes supported by an operator will be tested.")
94+
flags.StringSliceVar(&checkflags.FilterPackages, "filter-packages", []string{}, "a list of package(s) which limits audits and/or other flag(s) output")
95+
flags.BoolVar(&checkflags.AllInstallModes, "all-installmodes", false, "when set, all install modes supported by an operator will be tested")
9696
}

internal/capability/operand_cleanup.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import (
1414

1515
// OperandCleanup removes the operand from the OCP cluster in the ca.namespace
1616
func (ca *CapAudit) OperandCleanUp() error {
17-
zap.S().Debug("CleaningUp operand for operator", "package", ca.Subscription.Package, "channel", ca.Subscription.Channel, "installmode", ca.Subscription.InstallModeType)
17+
zap.S().Debug("cleaning up operand for operator", "package", ca.Subscription.Package, "channel", ca.Subscription.Channel, "installmode", ca.Subscription.InstallModeType)
1818

1919
if len(ca.CustomResources) > 0 {
2020
for _, cr := range ca.CustomResources {

internal/capability/operand_install.go

-2
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,7 @@ func (ca *CapAudit) getAlmExamples() error {
5252
// OperandInstall installs the operand from the ALMExamples in the ca.namespace
5353
func (ca *CapAudit) OperandInstall() error {
5454
zap.S().Debug("Installing operand for operator", "package", ca.Subscription.Package, "channel", ca.Subscription.Channel, "installmode", ca.Subscription.InstallModeType)
55-
5655
ca.getAlmExamples()
57-
5856
// TODO: we need a stratergy to select which CR to select from ALMExamplesList
5957
if len(ca.CustomResources) > 0 {
6058
for _, cr := range ca.CustomResources {

internal/logger/logger.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import (
88
)
99

1010
var (
11-
envLog = os.Getenv("OPCAP_LOG")
11+
envLog = os.Getenv("OPCAP_LOG_LEVEL")
1212
cfg zap.Config
1313
)
1414

0 commit comments

Comments
 (0)