The insights-results-aggregator-cleaner's discuss from redhatinsights

Setup Golint GH Action for this repository

Ability to display actual configuration when CLI option is used

Bump-up Go version to 1.16 on CI

cleaner.go:92:20:3 other occurrence(s) of "input" found in: cleaner.go:90:19 cleaner.go:136:19 cleaner.go:138:20

cleaner.go:143:17:1 other occurrence(s) of "improper cluster entries" found in: cleaner.go:97:17

gocyclo needs to be run on CI

Ensure that cloud.redhat.com address is not wired in the code

cleaner.go:97:73:3 other occurrence(s) of "Cluster list finished" found in: cleaner.go:96:71 cleaner.go:142:71 cleaner.go:143:73

New gosec-related issue detected

[/home/tester/ccx/insights-results-aggregator-cleaner/cleaner.go:138-143] - G307 (CWE-703): Deferring unsafe method "Close" on type "*os.File" (Confidence: HIGH, Severity: MEDIUM)
    137:        }
  > 138:        defer func() {
  > 139:                err := file.Close()
  > 140:                if err != nil {
  > 141:                        log.Err(err).Msg("File close failed")
  > 142:                }
  > 143:        }()
    144:

cleaner.go:90:41:1 other occurrence(s) of "Proper cluster ID" found in: cleaner.go:136:38

cleaner.go:136:38:1 other occurrence(s) of "Proper cluster ID" found in: cleaner.go:90:41

Find out sensitive data used on CI

Ability to display authors when CLI option is used

database.go:268:10:1 other occurrence(s) of "table" found in: database.go:263:10

Prepare unit tests for the cleaner

cleaner.go:143:73:3 other occurrence(s) of "Cluster list finished" found in: cleaner.go:96:71 cleaner.go:97:73 cleaner.go:142:71

Figure out which environment variables are used on CI

Rename `database.go` to `storage.go` for better consistency with other Insights repositories

Rename database.go to storage.go for better consistency with other Insights repositories

cleaner.go:142:17:1 other occurrence(s) of "number of clusters to delete" found in: cleaner.go:96:17

database.go:263:10:1 other occurrence(s) of "table" found in: database.go:268:10

Fix all minor issues found by linters

Refactoring: use `cliFlags` data structure to store all command line flags and arguments

Refactoring: use cliFlags data structure to store all command line flags and arguments

cleaner.go:138:20:3 other occurrence(s) of "input" found in: cleaner.go:90:19 cleaner.go:92:20 cleaner.go:136:19

Update pull request template - add an option for bump-up dependency

Add package manifest into documentation

Possible error in shell script

In bdd_tests/run_tests.sh line 7:
    virtualenv -p python3 venv && source venv/bin/activate && python3 `which pip3` install -r requirements.txt
                                                                      ^-- SC2046: Quote this to prevent word splitting.
                                                                      ^-- SC2006: Use $(..) instead of legacy `..`.

cleaner.go:90:19:3 other occurrence(s) of "input" found in: cleaner.go:92:20 cleaner.go:136:19 cleaner.go:138:20

Update to newer gosec version

cleaner.go:96:17:1 other occurrence(s) of "number of clusters to delete" found in: cleaner.go:142:17

database.go:176:18:1 other occurrence(s) of "cluster" found in: database.go:269:10

Bump-up insights-operator-utils to 1.14

Ability to display version info when CLI option is used

Add Table of Content into the README.md file

Function with zero code coverage detected: cleaner.go:100: IsValidUUID

cleaner.go:142:71:3 other occurrence(s) of "Cluster list finished" found in: cleaner.go:96:71 cleaner.go:97:73 cleaner.go:143:73

Make gosec installation verbose

Make gosec installation verbose to detect installation errors

Install gosec the official way - through binaries

Possible error in shell script

In bdd_tests/run_tests.sh line 12:
PYTHONDONTWRITEBYTECODE=1 python3 "`which behave`" --tags=-skip -D dump_errors=true @feature_list.txt $@
                                   ^-- SC2006: Use $(..) instead of legacy `..`.
                                                                                                      ^-- SC2068: Double quote array expansions to avoid re-splitting elements.

cleaner.go:96:71:3 other occurrence(s) of "Cluster list finished" found in: cleaner.go:97:73 cleaner.go:142:71 cleaner.go:143:73

redhatinsights / insights-results-aggregator-cleaner Goto Github PK

insights-results-aggregator-cleaner's Issues

Recommend Projects

Recommend Topics

Recommend Org