Skip to content

feat(cluster): add cluster analysis commands#666

Merged
pggb25 merged 1 commit into
mainfrom
feat/cluster-analysis-cli
Jun 25, 2026
Merged

feat(cluster): add cluster analysis commands#666
pggb25 merged 1 commit into
mainfrom
feat/cluster-analysis-cli

Conversation

@pggb25

@pggb25 pggb25 commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@pggb25 pggb25 force-pushed the feat/cluster-analysis-cli branch 3 times, most recently from 18c670c to 0d1bfcc Compare June 25, 2026 10:20
@fabienfleureau fabienfleureau requested a review from Copilot June 25, 2026 13:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a new qovery cluster analysis command group to start and inspect cluster “analysis” jobs (e.g., cost recommendation, deprecated API checks), and bumps qovery-client-go to a version that exposes the required API endpoints/types.

Changes:

  • Add qovery cluster analysis command group with subcommands to run analyses, list past analyses, and fetch analysis logs.
  • Implement a shared runner that can optionally watch analysis completion and print the resulting report logs.
  • Update github.com/qovery/qovery-client-go dependency to a newer pseudo-version.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
go.mod Bumps qovery-client-go to support cluster analysis APIs.
go.sum Updates module checksums for the bumped qovery-client-go version.
cmd/cluster_analysis.go Adds the cluster analysis command group plus shared helpers (output parsing, status helpers, HTTP error formatting, log rendering).
cmd/cluster_analysis_runner.go Adds the common “start + optional watch + print logs” runner and shared flags.
cmd/cluster_analysis_logs.go Adds cluster analysis logs to fetch and print a past analysis report/log stream (text or JSON).
cmd/cluster_analysis_list.go Adds cluster analysis list to list prior analyses for a cluster (table or JSON).
cmd/cluster_analysis_deprecated_api.go Adds cluster analysis deprecated-api runner subcommand.
cmd/cluster_analysis_cost_recommendation.go Adds cluster analysis cost-recommendation runner subcommand.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmd/cluster_analysis.go
Comment thread cmd/cluster_analysis_cost_recommendation.go
Comment thread cmd/cluster_analysis_deprecated_api.go
Comment thread cmd/cluster_analysis.go
@pggb25 pggb25 force-pushed the feat/cluster-analysis-cli branch 2 times, most recently from e1ff66a to b6b1f8c Compare June 25, 2026 13:47
@pggb25 pggb25 force-pushed the feat/cluster-analysis-cli branch from b6b1f8c to e262e30 Compare June 25, 2026 13:57
@pggb25 pggb25 merged commit 085e5a1 into main Jun 25, 2026
6 checks passed
@pggb25 pggb25 deleted the feat/cluster-analysis-cli branch June 25, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants