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

Check if setup task is actually still running #874

Merged
merged 2 commits into from
Sep 4, 2024
Merged

Conversation

agners
Copy link
Collaborator

@agners agners commented Sep 4, 2024

On first setup, we create a task and the task will remain in the task list. If a node gets offline and rediscovered on mDNS again, we will need to setup the node again. In this case we'll have that previous task, but it will be done. So check if the task is done and if so allow starting another setup task.

Fixes introduced in #873.

On first setup, we create a task and the task will remain in the task
list. If a node gets offline and rediscovered on mDNS again, we will
need to setup the node again. In this case we'll have that previous
task, but it will be done. So check if the task is done and if so allow
starting another setup task.
@agners agners added the bugfix Pull request that fixes a (known) issue/bug label Sep 4, 2024
This avoids stale tasks to be stored.
@agners agners merged commit 5c130ec into main Sep 4, 2024
4 checks passed
@agners agners deleted the fix-second-setup-task branch September 4, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull request that fixes a (known) issue/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants