New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Added ReadyConditions
type which is used in ServiceConfig
for defining how to check services readiness
#151
Conversation
Deploying with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add some tests for ReadyConditions?
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
...server/api_container/server/startosis_engine/kurtosis_types/service_config/service_config.go
Show resolved
Hide resolved
@gbouv this guy is ready for a second review, thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good!
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
...rver/api_container/server/startosis_engine/kurtosis_types/service_config/ready_conditions.go
Outdated
Show resolved
Hide resolved
core/server/api_container/server/startosis_engine/recipe/http_request_recipe.go
Show resolved
Hide resolved
@leoporoli are we tracking to add docs about this too? |
No description provided.