Skip to content

feat: expose service statuses in cli#134

Merged
michaelshin merged 5 commits into
mainfrom
mshin/expose-detailed-status-in-cli
May 21, 2026
Merged

feat: expose service statuses in cli#134
michaelshin merged 5 commits into
mainfrom
mshin/expose-detailed-status-in-cli

Conversation

@michaelshin
Copy link
Copy Markdown
Contributor

To properly show why service statuses are occurring, the CLI has been updated to expose the correct error message

@michaelshin michaelshin requested a review from anandj91 May 5, 2026 15:33
Comment thread centml/compiler/prediction/__init__.py Outdated
Comment thread centml/cli/cluster.py Outdated
@michaelshin michaelshin changed the title feat: expose status_v3 in cli feat: expose service statuses in cli May 14, 2026
Comment thread centml/sdk/api.py Outdated
Comment thread centml/sdk/api.py Outdated
@anandj91
Copy link
Copy Markdown
Contributor

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bc4e6ef3dc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread centml/sdk/api.py Outdated
@michaelshin
Copy link
Copy Markdown
Contributor Author

@codex re-review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@michaelshin michaelshin requested a review from anandj91 May 20, 2026 02:25
@anandj91
Copy link
Copy Markdown
Contributor

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 097216d127

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread centml/cli/cluster.py Outdated
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 097216d127

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread centml/sdk/api.py
Comment thread centml/sdk/api.py
@michaelshin
Copy link
Copy Markdown
Contributor Author

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 16a8799a08

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread centml/sdk/api.py
@michaelshin
Copy link
Copy Markdown
Contributor Author

@codex re-review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

https://github.com/CentML/centml-python-client/blob/16a8799a0827e24bcfbf0e191dae689e6a71b79d/centml/compiler/prediction/__init__.py#L1
P1 Badge Restore centml.compiler.prediction package shim

Deleting this file removes the centml.compiler.prediction package from the installed distribution, so any existing code that still imports centml.compiler.prediction now fails immediately with ModuleNotFoundError. This is a runtime compatibility break introduced by this commit (and the same import path was previously restored to fix that exact failure), so the empty package marker should be kept.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread centml/sdk/api.py
@michaelshin michaelshin merged commit 13c7886 into main May 21, 2026
5 checks passed
@michaelshin michaelshin deleted the mshin/expose-detailed-status-in-cli branch May 21, 2026 18:31
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.

2 participants