-
Notifications
You must be signed in to change notification settings - Fork 1
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: Updates to book based on usage #51
Conversation
run_time: 4 | ||
image: keramik/runner:dev | ||
imagePullPolicy: IfNotPresent | ||
``` |
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.
This section should not be deleted. Maybe it can live in a different section be this topic is not covered anywhere else. This section explains how to configure either the Network or Simulation resources to use a local build of the runner.
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.
Re-added in its own section now with a couple of links to it https://github.com/3box/keramik/pull/51/files#diff-68393f8f40dbae2e7f5a2f43d4cd93a4d9338f305017f5775a327825db9b7e23
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.
lgtm
@@ -8,6 +8,7 @@ To analyze the results of a simulation first copy the metrics-TIMESTAMP.parquet | |||
First restart opentelemetry-0 pod so it writes out the parquet file footer. | |||
|
|||
kubectl delete pod opentelemetry-0 | |||
sleep 5 # Give pod time to restart |
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.
Just a note, you can use the api the know when the pod is ready
kubectl wait --for=condition=Ready pod/opentelemetry-0
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.
Thanks, will add that in the next round of book changes
No description provided.