Skip to content

Add CLI tool#59

Merged
mohit-sheth merged 4 commits intocloud-bulldozer:mainfrom
jtaleric:cli-tool
Aug 28, 2025
Merged

Add CLI tool#59
mohit-sheth merged 4 commits intocloud-bulldozer:mainfrom
jtaleric:cli-tool

Conversation

@jtaleric
Copy link
Member

@jtaleric jtaleric commented Aug 18, 2025

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

CLI tool which allows us to get cluster metadata. This can be used in other tools like e2e-benchmkaring, avoiding the need to maintain two different ways to get cluster metadata

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please describe the System Under Test.
  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

CLI tool which allows us to get cluster metadata. This can be used in
other tools like e2e-benchmkaring, avoiding the need to maintain two
different ways to get cluster metadata

Signed-off-by: rook <joe.talerico@gmail.com>
@codecov
Copy link

codecov bot commented Aug 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.74%. Comparing base (717a817) to head (31e1617).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #59   +/-   ##
=======================================
  Coverage   80.74%   80.74%           
=======================================
  Files          10       10           
  Lines         483      483           
=======================================
  Hits          390      390           
  Misses         71       71           
  Partials       22       22           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jtaleric jtaleric added the enhancement New feature or request label Aug 19, 2025
@josecastillolema
Copy link
Member

On a BM cluster:

Field                Value
-----                -----
Cluster Name         vlan604-7rpcj
Platform             BareMetal
Cluster Type         self-managed
OCP Version          4.18.21
OCP Major Version    4.18
K8s Version          v1.31.10
Region
SDN Type             OVNKubernetes
Total Nodes          5
Master Nodes         3 ()
Worker Nodes         2 ()
Infra Nodes          0 ()
Other Nodes          0
Worker Arch          amd64
Control Plane Arch   amd64
FIPS                 false
Publish              External
IPSec                false
IPSec Mode           Disabled

On a vanila k8s cluster:

Field                Value
-----                -----
Cluster Name
Platform
Cluster Type
OCP Version
OCP Major Version
K8s Version          v1.32.2
Region
SDN Type
Total Nodes          3
Master Nodes         1 ()
Worker Nodes         2 ()
Infra Nodes          0 ()
Other Nodes          0
Worker Arch
Control Plane Arch
FIPS                 false
Publish
IPSec                false
IPSec Mode

Minor nit, @jtaleric can you please make it so ocp-metadata reads the $KUBECONFIG var when defined? I think that will ease consumption from the CPT pipelines.

@jtaleric
Copy link
Member Author

Thanks for the feedback! Addressed! @josecastillolema @vishnuchalla

Signed-off-by: rook <joe.talerico@gmail.com>
Copy link
Collaborator

@vishnuchalla vishnuchalla left a comment

Choose a reason for hiding this comment

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

lgtm

Signed-off-by: rook <joe.talerico@gmail.com>
@rsevilla87 rsevilla87 self-requested a review August 28, 2025 17:26
Copy link
Member

@rsevilla87 rsevilla87 left a comment

Choose a reason for hiding this comment

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

thanks, lgtm

@mohit-sheth mohit-sheth merged commit e43a8c5 into cloud-bulldozer:main Aug 28, 2025
6 checks passed
@jtaleric jtaleric deleted the cli-tool branch August 28, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants