Skip to content

Conversation

@markbackman
Copy link
Contributor

@markbackman markbackman commented Dec 17, 2025

No description provided.

@markbackman markbackman changed the title Mb/fix deployment status check fix: deployment ready check uses service and deployment readiness Dec 17, 2025
Copy link
Contributor

@mattshep mattshep left a comment

Choose a reason for hiding this comment

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

I think this is good, but we might want to also consider the changes in https://github.com/daily-co/pipecat-cloud-sandbox/pull/335 - that adds available (can handle requests) to the response, and does a better job of surfacing operator level errors (like failed image pulls).

@markbackman
Copy link
Contributor Author

I think this is good, but we might want to also consider the changes in daily-co/pipecat-cloud-sandbox#335 - that adds available (can handle requests) to the response, and does a better job of surfacing operator level errors (like failed image pulls).

I'm going to move forward with this change now and then once that update is made, we can improve this code to use the available status. That will cover even more cases.

Can you make that change to the CLI once it's in the PCC server?

@markbackman markbackman force-pushed the mb/fix-deployment-status-check branch from fcd88c6 to ee128e1 Compare December 18, 2025 14:09
@markbackman markbackman merged commit bf6c715 into main Dec 18, 2025
1 of 2 checks passed
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