Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat(coderd): add provisioner_daemons to /debug/health endpoint#11393

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
johnstcn merged 18 commits intomainfromcj/provisionerd-healthcheck
Jan 8, 2024

Conversation

johnstcn
Copy link
Member

@johnstcnjohnstcn commentedJan 3, 2024
edited
Loading

Adds a healthcheck for provisioner daemons to /debug/health endpoint.

Part of#10676

@johnstcnjohnstcn self-assigned thisJan 3, 2024
@johnstcnjohnstcnforce-pushed thecj/provisionerd-healthcheck branch from1b2bc9a to53ed901CompareJanuary 4, 2024 16:33
@johnstcnjohnstcn changed the base branch frommain tocj/util-apiversionJanuary 4, 2024 16:34
Base automatically changed fromcj/util-apiversion tomainJanuary 5, 2024 10:22
@johnstcnjohnstcnforce-pushed thecj/provisionerd-healthcheck branch from53ed901 tob83013bCompareJanuary 5, 2024 12:19
@johnstcnjohnstcn changed the titleWIP: add version healthcheck for provisioner daemonsfeat(coderd): add provisioner_daemons to /debug/health endpointJan 5, 2024
@johnstcnjohnstcn marked this pull request as ready for reviewJanuary 5, 2024 12:31
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you understand why this keeps getting marked as changed?

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

I think it's the timestamp on the files changing; I haven't yet figured out a way to ignore this.

continue
}
// Daemon has gone away, skip.
ifnow.Sub(daemon.LastSeenAt.Time)> (opts.StaleInterval) {
Copy link
Member

Choose a reason for hiding this comment

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

Thinking out loud, is it possible that a daemon reports an error, but after that, the last seen isn't updated. Then after stale interval the error disappears, and perhaps nobody ever notices it?

Also, don't we want to apply the same rules tor.ProvisionerDaemons?

johnstcn reacted with thumbs up emoji
Copy link
MemberAuthor

Choose a reason for hiding this comment

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

is it possible that a daemon reports an error, but after that, the last seen isn't updated. Then after stale interval the error disappears, and perhaps nobody ever notices it?

Yes, and this is by design. If a provisioner daemon connects at some point, has a transient error, and then never heartbeats, I don't think it makes sense to warn about this.

Copy link
Member

Choose a reason for hiding this comment

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

That's fair. I was entertaining the possibility of some error state resulting in the provisioner exiting, crashing or plain stopping communication with server. But I didn't have anything concrete in mind, so this is fine.

ProvisionerDaemons []codersdk.ProvisionerDaemon`json:"provisioner_daemons"`
}

typeProvisionerDaemonsReportDepsstruct {
Copy link
Member

Choose a reason for hiding this comment

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

This feels more likeConfig instead ofDeps, but I'll leave it up to you.

@johnstcnjohnstcnforce-pushed thecj/provisionerd-healthcheck branch from62d42e8 to19e896aCompareJanuary 8, 2024 09:10
@johnstcnjohnstcn merged commit04fd96a intomainJan 8, 2024
@johnstcnjohnstcn deleted the cj/provisionerd-healthcheck branchJanuary 8, 2024 09:29
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsJan 8, 2024
@mtojek
Copy link
Member

@johnstcn Nice contribution! Could you please link it with the umbrella issue and fill in the PR description?

Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@spikecurtisspikecurtisspikecurtis left review comments

@mafredrimafredrimafredri approved these changes

@mtojekmtojekAwaiting requested review from mtojek

Assignees

@johnstcnjohnstcn

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@johnstcn@mtojek@mafredri@spikecurtis

[8]ページ先頭

©2009-2025 Movatter.jp