Merge pull request #1 from raito-io/RAITO-1817-config-map-metadata-in…

…formation-aws-organisation

Upgrade to CLI v0.50.0

.github/workflows/build.yml

@@ -18,7 +18,7 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v3
         with:
-          go-version: '1.20'
+          go-version: '1.21'
           cache: true
 
       - name: Configure git for private modules
@@ -33,7 +33,7 @@ jobs:
       - name: Lint
         uses: golangci/golangci-lint-action@v3
         with:
-          version: v1.52.2
+          version: v1.54.2
 
       - name: Test
         run: make test-coverage

.github/workflows/release.yml

@@ -23,7 +23,7 @@ jobs:
       - name: Install Go
         uses: actions/setup-go@v3
         with:
-          go-version: '1.20'
+          go-version: '1.21'
           cache: true
 
       - name: Configure git for private modules

.golangci.yml

@@ -43,6 +43,7 @@ linters-settings:
   gosec:
     excludes:
       - G404 # To be checked: Insecure random number source (rand)
+      - G602
 
   stylecheck:
     checks: [ "all", "-ST1003"] # ST1003: No underscores in package names should be fixed and removed?
@@ -54,14 +55,22 @@ linters-settings:
     ignored-functions:
       - 'math.*' # Magic number generation, should be fixed and removed?
 
+  cyclop:
+    max-complexity: 20
+    skip-tests: true
+    package-average: 0.0
+
 linters:
   disable-all: true
   enable:
+    - asasalint
     - bodyclose
-    - deadcode
-    - depguard
+    - cyclop #To be enabled in the future
+    # - depguard #We should create a correct config to enabe depguard
     - dogsled
     - dupl
+    - dupword
+    - durationcheck
     - errcheck
     - exportloopref
     - exhaustive
@@ -71,7 +80,7 @@ linters:
     - gofmt
     - goimports
     #- gomnd # Magic numbers should be fixed and enable gomnd,
-    #- gocyclo
+    - gocyclo
     - gosec
     - gosimple
     - govet
@@ -83,14 +92,13 @@ linters:
     - predeclared
     - revive
     - staticcheck
-    - structcheck
     - stylecheck
     - thelper
     - tparallel
     - typecheck
     - unconvert
     - unparam
-    - varcheck
+    - unused
     - whitespace
     # - wrapcheck # 3th party errors should be wrapped to not lose context
     - wsl

aws/identity_store.go

@@ -24,7 +24,7 @@ func NewIdentityStoreSyncer() *IdentityStoreSyncer {
 	return &IdentityStoreSyncer{repoProvider: newRepoProvider}
 }
 
-func (s *IdentityStoreSyncer) GetIdentityStoreMetaData(ctx context.Context) (*is.MetaData, error) {
+func (s *IdentityStoreSyncer) GetIdentityStoreMetaData(_ context.Context, _ *config.ConfigMap) (*is.MetaData, error) {
 	logger.Debug("Returning meta data for AWS identity store")
 
 	return &is.MetaData{