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

Fix issue #1221 #1750

Closed
wants to merge 3 commits into from
Closed

Conversation

alistanis
Copy link

This was the only place I could find that might be affected by a sudden shutdown during a write. If there are any I've missed, please let me know and I'll add them to this PR.

Christopher Cooper added 2 commits February 24, 2016 00:34
Merge from hashicorp master
…test.go. calls writeFileAtomic from command/agent/agent.go in (a *Agent) persistCheckState
@multani
Copy link

multani commented Jun 8, 2016

You may want to reconsider the formatting changes in your commits! :)

@slackpad
Copy link
Contributor

Hi @alistanis sorry for the delay on this. Can you please rebase and run go fmt on your changes? Thanks!

@alistanis
Copy link
Author

@slackpad No problem! I'll take care of this tonight when I get home. I always run go fmt (or goimports) so I'm not totally sure what happened.

Also thanks @multani, looks like I missed your comment the first time around.

@kyhavlov
Copy link
Contributor

kyhavlov commented Nov 17, 2016

Closing as this was incorporated into #2480; thanks for the PR and sorry this took so long to get to!

@kyhavlov kyhavlov closed this Nov 17, 2016
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.

4 participants