Skip to content

[Feature Complete]Inspection datasources, docs, acceptance tests#327

Merged
ramya-bangera merged 2 commits intomainfrom
feat/inspections
Jan 11, 2024
Merged

[Feature Complete]Inspection datasources, docs, acceptance tests#327
ramya-bangera merged 2 commits intomainfrom
feat/inspections

Conversation

@GilTeraSky
Copy link
Copy Markdown
Contributor

@GilTeraSky GilTeraSky commented Nov 3, 2023

  1. What this PR does / why we need it:
    This PR includes the entire implementation of the cluster inspection data sources including tests & docs
  2. Which issue(s) this PR fixes
  3. Additional information
    Acceptance tests
Screenshot 2024-01-09 at 3 37 03 PM

Terraform output:
Screenshot 2024-01-09 at 3 37 55 PM

Screenshot 2024-01-09 at 3 38 09 PM
  1. Special notes for your reviewer

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8e66181) 25.09% compared to head (c6bc92e) 25.10%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #327   +/-   ##
=======================================
  Coverage   25.09%   25.10%           
=======================================
  Files         191      191           
  Lines       16480    16482    +2     
=======================================
+ Hits         4135     4137    +2     
  Misses      12131    12131           
  Partials      214      214           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

vrabbi
vrabbi previously approved these changes Nov 12, 2023
Copy link
Copy Markdown

@vrabbi vrabbi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@GilTeraSky GilTeraSky changed the title Feat/Inspections -- Data Sources Feat/Inspections -- Feature Complete Nov 16, 2023
Copy link
Copy Markdown
Contributor

@shobha2626 shobha2626 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Please squash the commits accordingly to meaningful messages
  2. Please attach the screenshot of the acceptance test

@shobha2626
Copy link
Copy Markdown
Contributor

Please resolve the conflicts

@ramya-bangera ramya-bangera changed the title Feat/Inspections -- Feature Complete [Feature Complete]Inspection datasources, docs, acceptance tests Jan 9, 2024
@shobha2626
Copy link
Copy Markdown
Contributor

shobha2626 commented Jan 10, 2024

Overall LGTM. Please squash the commits into one meaningful message

@ramya-bangera
Copy link
Copy Markdown
Contributor

Overall LGTM. Please squash the commits into one meaningful message

Done

… rebasing and updated the copyright to 2024

Signed-off-by: Ramya Bangera <bangerar@vmware.com>
@ramya-bangera ramya-bangera merged commit 7a83d88 into main Jan 11, 2024
@tenthirtyam tenthirtyam deleted the feat/inspections branch November 13, 2024 06:30
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants