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

Use Background context #808

Closed
wants to merge 2 commits into from
Closed

Use Background context #808

wants to merge 2 commits into from

Conversation

ckotzbauer
Copy link
Member

close #234

@ckotzbauer ckotzbauer added this to the 1.14.0 milestone Aug 4, 2023
Signed-off-by: Christian Kotzbauer <[email protected]>
Signed-off-by: Christian Kotzbauer <[email protected]>
@ckotzbauer ckotzbauer force-pushed the cleanup/todo-context branch from d39a904 to 59cbea5 Compare August 14, 2023 17:08
Copy link
Collaborator

@jackfrancis jackfrancis left a comment

Choose a reason for hiding this comment

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

I don't think we wanna bother with this. My thoughts here:

@ckotzbauer ckotzbauer removed this from the 1.14.0 milestone Aug 19, 2023
@github-actions
Copy link

This PR was automatically considered stale due to lack of activity. Please refresh it and/or join our slack channels to highlight it, before it automatically closes (in 7 days).

@ckotzbauer ckotzbauer added keep This won't be closed by the stale bot. and removed no-pr-activity labels Oct 21, 2023
Copy link

This PR was automatically considered stale due to lack of activity. Please refresh it and/or join our slack channels to highlight it, before it automatically closes (in 7 days).

Copy link

This PR was automatically considered stale due to lack of activity. Please refresh it and/or join our slack channels to highlight it, before it automatically closes (in 7 days).

leonnicolas added a commit to leonnicolas/kured that referenced this pull request Mar 4, 2024
Propagate a context from the main function to wherever it is needed.

Use `github.com/oklog/run` run groups to handle the life cycle of the go
routines running the rebooter and metrics server.

Ref: kubereboot#234
and kubereboot#808

Signed-off-by: leonnicolas <[email protected]>
@leonnicolas leonnicolas mentioned this pull request Mar 4, 2024
Copy link

This PR was automatically considered stale due to lack of activity. Please refresh it and/or join our slack channels to highlight it, before it automatically closes (in 7 days).

@github-actions github-actions bot closed this May 12, 2024
leonnicolas added a commit to leonnicolas/kured that referenced this pull request May 16, 2024
Propagate a context from the main function to wherever it is needed.

Use `github.com/oklog/run` run groups to handle the life cycle of the go
routines running the rebooter and metrics server.

Ref: kubereboot#234
and kubereboot#808

Signed-off-by: leonnicolas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement keep This won't be closed by the stale bot. no-pr-activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use of context.TODO()
2 participants