-
Notifications
You must be signed in to change notification settings - Fork 123
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
Update client/swagger.yaml from 0.21.0 to b922c7d #233
Conversation
Let me terminate the EC2 instance (i-0b150441a0ae63644). It seems unhealthy. https://buildkite.com/firecracker-microvm/firecracker-go-sdk/builds/873 |
0ecfdb0
to
f1cdac6
Compare
0fa8f41
to
6303571
Compare
And "make generate" to generate files. Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
The SDK's master no longer works with 0.21.x. Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
…ork pre-boot These methods use PATCH internally, which no longer work pre-boot. Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
machine.go
Outdated
} | ||
|
||
func (m *Machine) setupMetrics(ctx context.Context) error { | ||
fmt.Println("setupMetrics") |
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.
m.logger.printf if we still want this message?
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.
Right. Will fix.
Previously Firecracker had a single operation to set logging and metrics, and the fields specifically mentioned "FIFO". Now the operation is broken into two and it doesn't mention "FIFO" anymore. This change keeps MetricsFifo and LogFifo to be backward-compatible. Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com> fix
Issue #, if available:
NA
Description of changes:
Firecracker is making a few breaking changes. I'd like to estimate how much changes we may need, before their official release.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.