Retrieve changes form STF_685_MEC_IPE

  • Update .golangci.yml to v2 format (add version, move goimports to formatters, remove deprecated linters)
  • Replace deprecated ioutil calls with os/io equivalents
  • Remove ineffectual err = nil assignments in utility functions
  • Refactor if/else chains to switch statements for better readability
  • Replace strings.Replace with strings.ReplaceAll where applicable
  • Fix error string capitalization to comply with Go conventions
  • Apply goimports formatting fixes across all modified files
  • Fix CRLF line endings in go-apps/meepctl/install.sh

Merge request reports

Loading
+13 −0
Changes for .devcontainer/devcontainer.json: 13 added lines, 0 removed lines.
Original line number Diff line number Diff line
{
    "name": "Go",
    "image": "mcr.microsoft.com/devcontainers/go:1.25-bookworm",
    "customizations": {
        "vscode": {
            "extensions": [
                "golang.go"
            ]
        }
    },
    "postCreateCommand": "go version",
    "remoteUser": "vscode"
}
 No newline at end of file
+9 −8
Changes for charts/kube-state-metrics/values.yaml: 9 added lines, 8 removed lines.
Original line number Diff line number Diff line
@@ -2,7 +2,7 @@
meepOrigin: core
prometheusScrape: true
image:
  repository: quay.io/coreos/kube-state-metrics
  repository: rancher/mirrored-coreos-kube-state-metrics
  tag: v1.9.7
  pullPolicy: IfNotPresent

@@ -124,23 +124,23 @@ podDisruptionBudget: {}
# Available collectors for kube-state-metrics. By default all available
# collectors are enabled.
collectors:
  certificatesigningrequests: true
  certificatesigningrequests: false
  configmaps: true
  cronjobs: true
  cronjobs: false
  daemonsets: true
  deployments: true
  endpoints: true
  horizontalpodautoscalers: true
  ingresses: true
  horizontalpodautoscalers: false
  ingresses: false
  jobs: true
  limitranges: true
  mutatingwebhookconfigurations: true
  mutatingwebhookconfigurations: false
  namespaces: true
  networkpolicies: true
  nodes: true
  persistentvolumeclaims: true
  persistentvolumes: true
  poddisruptionbudgets: true
  poddisruptionbudgets: false
  pods: true
  replicasets: true
  replicationcontrollers: true
@@ -149,10 +149,11 @@ collectors:
  services: true
  statefulsets: true
  storageclasses: true
  validatingwebhookconfigurations: true
  validatingwebhookconfigurations: false
  verticalpodautoscalers: false
  volumeattachments: true
 

# Enabling kubeconfig will pass the --kubeconfig argument to the container
kubeconfig:
  enabled: false
+3 −3
Changes for charts/meep-tinyiot-in-cse/templates/_helpers.tpl: 3 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -2,7 +2,7 @@
{{/*
Expand the name of the chart.
*/}}
{{- define "meep-tinyiot-in-cse.name" -}}
{{- define "meep-tiny-in-cse.name" -}}
{{- default .Chart.Name .Values.nameOverride | trunc 63 | trimSuffix "-" -}}
{{- end -}}

@@ -11,7 +11,7 @@ Create a default fully qualified app name.
We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
If release name contains chart name it will be used as a full name.
*/}}
{{- define "meep-tinyiot-in-cse.fullname" -}}
{{- define "meep-tiny-in-cse.fullname" -}}
{{- if .Values.fullnameOverride -}}
{{- .Values.fullnameOverride | trunc 63 | trimSuffix "-" -}}
{{- else -}}
@@ -27,6 +27,6 @@ If release name contains chart name it will be used as a full name.
{{/*
Create chart name and version as used by the chart label.
*/}}
{{- define "meep-tinyiot-in-cse.chart" -}}
{{- define "meep-tiny-in-cse.chart" -}}
{{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" -}}
{{- end -}}
+2 −2
Changes for charts/meep-tinyiot-in-cse/templates/clusterrolebinding.yaml: 2 added lines, 2 removed lines.
Original line number Diff line number Diff line
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: "{{ .Release.Namespace }}:{{ template "meep-tinyiot-in-cse.fullname" . }}"
  name: "{{ .Release.Namespace }}:{{ template "meep-tiny-in-cse.fullname" . }}"
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cluster-admin
subjects:
- kind: ServiceAccount
  name: {{ template "meep-tinyiot-in-cse.fullname" . }}
  name: {{ template "meep-tiny-in-cse.fullname" . }}
  namespace: {{ .Release.Namespace }}
  
 No newline at end of file
+5 −5
Changes for charts/meep-tinyiot-in-cse/templates/codecov-pv.yaml: 5 added lines, 5 removed lines.
Original line number Diff line number Diff line
@@ -2,9 +2,9 @@
kind: PersistentVolume
apiVersion: v1
metadata:
  name: meep-tinyiot-in-cse-codecov-pv
  name: meep-tiny-in-cse-codecov-pv
spec:
  storageClassName: meep-tinyiot-in-cse-codecov-sc
  storageClassName: meep-tiny-in-cse-codecov-sc
  capacity:
    storage: 100Mi
  accessModes:
@@ -17,16 +17,16 @@ spec:
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: meep-tinyiot-in-cse-codecov-sc
  name: meep-tiny-in-cse-codecov-sc
provisioner: kubernetes.io/no-provisioner
volumeBindingMode: WaitForFirstConsumer
---
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: meep-tinyiot-in-cse-codecov-pvc
  name: meep-tiny-in-cse-codecov-pvc
spec:
  storageClassName: meep-tinyiot-in-cse-codecov-sc
  storageClassName: meep-tiny-in-cse-codecov-sc
  accessModes:
    - ReadWriteOnce
  resources:
Loading
Loading