feat: add step and aggregation fields for kubectl get KeptnMetric

[AryanSharma9917]

This PR closes : #2100

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (16e1f86) 85.68% compared to head (d24265a) 85.75%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2556      +/-   ##
==========================================
+ Coverage   85.68%   85.75%   +0.06%     
==========================================
  Files         160      160              
  Lines       10164    10164              
==========================================
+ Hits         8709     8716       +7     
+ Misses       1174     1169       -5     
+ Partials      281      279       -2     

Files	Coverage Δ
metrics-operator/api/v1beta1/keptnmetric_types.go	100.00% <ø> (ø)

... and 2 files with indirect coverage changes

Flag	Coverage Δ
certificate-operator	67.42% <ø> (ø)
component-tests	57.12% <ø> (+1.03%)	⬆️
lifecycle-operator	84.72% <ø> (ø)
metrics-operator	87.65% <ø> (ø)
scheduler	36.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[netlify]

✅ Deploy Preview for keptn-lifecycle-toolkit ready!

Name	Link
🔨 Latest commit	f8a6f17
🔍 Latest deploy log	https://app.netlify.com/sites/keptn-lifecycle-toolkit/deploys/6573630b94e241000720d5bb
😎 Deploy Preview	https://deploy-preview-2556--keptn-lifecycle-toolkit.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[prakrit55]

After these lines you put your texts that you have added, with properly aligned.

lifecycle-toolkit/.github/scripts/.helm-tests/default/result.yaml

Line 7036 in 84f5588

type: string

lifecycle-toolkit/.github/scripts/.helm-tests/metrics-only/result.yaml

Line 1399 in 84f5588

type: string

lifecycle-toolkit/.github/scripts/.helm-tests/metrics-with-certs/result.yaml

Line 1580 in 84f5588

type: string

[odubajDT]

Hi @AryanSharma9917 do you have the capacity to finish this PR? Thanks!

[odubajDT]

can you also adapt this file please? .github/scripts/.helm-tests/metrics-only-with-apiservice-disabled/result.yaml

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[AryanSharma9917]

can you also adapt this file please? .github/scripts/.helm-tests/metrics-only-with-apiservice-disabled/result.yaml

Done..!!

[odubajDT]

can you also adapt this file please? .github/scripts/.helm-tests/metrics-only-with-apiservice-disabled/result.yaml

Done..!!

The fields were added in a wrong place, can you please adapt it?

--- ./.github/scripts/.helm-tests/metrics-only-with-apiservice-disabled/helm_tests_output.yaml	2023-12-11 06:32:30.315336491 +0000
+++ ./.github/scripts/.helm-tests/metrics-only-with-apiservice-disabled/result.yaml	2023-12-11 06:32:25.895373723 +0000
@@ -1397,15 +1397,15 @@
     - jsonPath: .spec.range.interval
       name: Interval
       type: string
+    - jsonPath: .status.value
+      name: Value
+      type: string
     - jsonPath: .spec.range.step
       name: Step
       type: string
     - jsonPath: .spec.range.aggregation
       name: Aggregation
       type: string
-    - jsonPath: .status.value
-      name: Value
-      type: string
     name: v1alpha3
     schema:
       openAPIV3Schema:

[rakshitgondwal]

Hi @AryanSharma9917, are you still working on this?

[odubajDT]

Hi @AryanSharma9917 any updates here ?

[odubajDT]

Since there was no response from Dec 8th, I am taking overt this PR.

[sonarqubecloud]

Quality Gate failed

Failed conditions

15.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud