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

Fil/sky 1060 wait for consensus sync before full setup #330

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

firyx
Copy link
Collaborator

@firyx firyx commented Jun 22, 2022

PULL REQUEST

Overview

Current playbooks wait in the following order:

  • wait for Sia finished full setup
  • wait for Sia consensus sync

This PR swaps order of waits so that portal operator is more accurately informed what happened when the wait fails.

Example for Visual Changes

Checklist

  • All git commits are signed. (REQUIRED)
  • All new methods or updated methods have clear docstrings.
  • Testing added or updated for new methods.
  • Verified if any changes impact the WebPortal Health Checks.
  • Appropriate documentation updated.
  • Changelog file created.

Issues Closed

@firyx firyx requested a review from MSevey as a code owner June 22, 2022 12:21
@linear
Copy link

linear bot commented Jun 22, 2022

SKY-1060 Wait for consensus sync before wait for sia full setup

In the playbook there is a wait for "Wait until Sia finished full setup".

We should check that consensus is synced before the above check.

@firyx firyx self-assigned this Jun 22, 2022
@firyx firyx requested review from kwypchlo and ro-tex June 22, 2022 12:21
Copy link
Contributor

@MSevey MSevey left a comment

Choose a reason for hiding this comment

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

skyd will complete a full set up on a fresh server before the consensus is sync. So this change is an anti-pattern.

What was the reason behind this change?

@firyx
Copy link
Collaborator Author

firyx commented Jun 22, 2022

@firyx firyx closed this Jun 22, 2022
@firyx firyx reopened this Jun 22, 2022
@firyx
Copy link
Collaborator Author

firyx commented Jun 22, 2022

skyd will complete a full set up on a fresh server before the consensus is sync. So this change is an anti-pattern.

What was the reason behind this change?

I'm observing it otherwise:

  • sia full setup doesn't finish until consensus is synced

@firyx firyx requested a review from MSevey June 22, 2022 15:41
@MSevey
Copy link
Contributor

MSevey commented Jun 23, 2022

I'm observing it otherwise:

Is this on repeated runs of the setup playbook?

if so then we can figure out what is causing it to not finish the setup.

a syncing consensus on its own is not the blocking action. So we shouldn't switch these. We should figure out which action is blocking the setup due to the consensus syncing and see if that needs to be blocking.

Copy link
Contributor

@MSevey MSevey left a comment

Choose a reason for hiding this comment

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

Pending longer discussion on root issue.

@kwypchlo kwypchlo removed their request for review October 3, 2022 14:28
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.

2 participants