Skip to content

sd_notify ready status when accepting API requests#3060

Merged
creack merged 1 commit into
masterfrom
sd_notify
Dec 5, 2013
Merged

sd_notify ready status when accepting API requests#3060
creack merged 1 commit into
masterfrom
sd_notify

Conversation

@vieux

@vieux vieux commented Dec 5, 2013

Copy link
Copy Markdown
Contributor

Fixes #2322

This is a rebase of @stephenmw's work

@vieux vieux mentioned this pull request Dec 5, 2013
Comment thread systemd/sd_notify.go Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be exported?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Probably..?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@crosbymichael

Copy link
Copy Markdown
Contributor

LGTM

@vieux

vieux commented Dec 5, 2013

Copy link
Copy Markdown
Contributor Author

ping @creack

@creack

creack commented Dec 5, 2013

Copy link
Copy Markdown
Contributor

LGTM

creack added a commit that referenced this pull request Dec 5, 2013
sd_notify ready status when accepting API requests
@creack
creack merged commit 066b961 into master Dec 5, 2013
@creack
creack deleted the sd_notify branch December 5, 2013 18:33
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.

sd_notify ready status when accepting API requests

3 participants