Skip to content
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

Warn if Bindle or Hippo not responding #596

Merged
merged 1 commit into from
Jun 27, 2022

Conversation

itowlson
Copy link
Contributor

Fixes #595.

Signed-off-by: itowlson ivan.towlson@fermyon.com

Signed-off-by: itowlson <ivan.towlson@fermyon.com>
@itowlson itowlson requested a review from michelleN June 22, 2022 01:03
@lann
Copy link
Collaborator

lann commented Jun 23, 2022

This looks similar to warn_if_slothful from the engine crate, but deduplicating isn't obvious.

@itowlson
Copy link
Contributor Author

It is, but I didn't want to make it part of the engine crate API. We should discuss where to put little utility functions like this.

@lann
Copy link
Collaborator

lann commented Jun 27, 2022

We should discuss where to put little utility functions like this.

spin-util? 😬 It's good enough for cargo...

@itowlson
Copy link
Contributor Author

chuckle Do you want that done as part of this PR or as a separate refactoring?

Copy link
Collaborator

@lann lann left a comment

Choose a reason for hiding this comment

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

Woops, meant to approve

@itowlson itowlson merged commit fc80306 into fermyon:main Jun 27, 2022
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.

Deploy: seems to hang if AWS install has been torn down
2 participants