-
Notifications
You must be signed in to change notification settings - Fork 153
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
Release 0.15.0 #122
Release 0.15.0 #122
Conversation
doc/release-notes.md
Outdated
@@ -1,6 +1,23 @@ | |||
# Release 0.14.13 |
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.
@andrewpmartinez Release 0.15.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.
@andrewpmartinez i have a pr to this branch out there that updates the readme to 0.15.0 as edward points out. also shouldn't the databaser snapshot stuff get put into 0.15.0?
…the usage and error from cobra in favor of ours
} | ||
|
||
requestMsg := channel2.NewMessage(int32(mgmt_pb.ContentType_SnapshotDbRequestType), nil) | ||
responseMsg, err := ch.SendAndWaitWithTimeout(requestMsg, 5*time.Second) |
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.
i mean- 5 seconds should be enough but is there any harm in waiting longer? just in case? if the timeout hits can we cancel the request to backup somehow?
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.
I wouldn't worry too much about it for now. ziti-fabric will go away in the short to medium term.
Nicer enroller output
…mission in development/example config.
* Add names to fabric service/routers. Add snapshot-db command
update refs and doc
No description provided.