Uploaded image for project: 'OpenShift Monitoring'
  1. OpenShift Monitoring
  2. MON-3144

Enables static code analysis in CMO

XMLWordPrintable

    • CMO static analysis
    • False
    • None
    • False
    • Not Selected
    • NEW
    • To Do
    • MON-3159Technical Debt
    • NEW
    • 0% To Do, 0% In Progress, 100% Done

      Epic Goal

      • Enable static code analysis in cluster-monitoring-operator
      • Create a suitable config for the selected analyzers to allow for ignoring issues that are deemed safe or ignoring portions of the code (like tests)
      • set up PR checks

      Why is this important?

      • static code analylsis can reduce certain classes of bugs
      • highlight unused code
      • enforces consistent code quality

       

      We should run at least https://github.com/golangci/golangci-lint. 

      https://github.com/securego/gosec could be interesting.

      We also have an internal team: https://gitlab.cee.redhat.com/covscan/covscan/-/wikis/home. Maybe there are additional scanners we can possibly run.

      Acceptance Criteria

      • CI - set up PR checks
      • Run at least golangci-lint
      • Fix existing issues or create exceptions in the relevant config files.

              dmellado1@redhat.com Daniel Mellado Area
              jfajersk@redhat.com Jan Fajerski
              Junqi Zhao Junqi Zhao
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated:
                Resolved: