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

[Merged by Bors] - Wait for ATX-synced before selecting commitment ATX #5423

Closed

Conversation

poszu
Copy link
Contributor

@poszu poszu commented Jan 10, 2024

Motivation

We need to wait to be ATX-synced before selecting a commitment ATX to select the best possible one.

Changes

  • moved waiting for ATX-synced to the function that finds the best ATX for commitment ATX
  • post supervisor doesn't always wait for ATX-synced before it starts post-service

Test Plan

Existing tests pass

Copy link

codecov bot commented Jan 10, 2024

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (5651ada) 77.4% compared to head (f5095f5) 77.4%.
Report is 1 commits behind head on develop.

Files Patch % Lines
activation/post.go 84.6% 2 Missing ⚠️
activation/post_supervisor.go 90.0% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           develop   #5423   +/-   ##
=======================================
  Coverage     77.4%   77.4%           
=======================================
  Files          265     265           
  Lines        30880   30890   +10     
=======================================
+ Hits         23929   23937    +8     
- Misses        5430    5432    +2     
  Partials      1521    1521           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

activation/post_supervisor_test.go Outdated Show resolved Hide resolved
activation/post_supervisor_test.go Outdated Show resolved Hide resolved
@poszu poszu requested a review from fasmat January 11, 2024 08:31
@poszu
Copy link
Contributor Author

poszu commented Jan 11, 2024

bors merge

spacemesh-bors bot pushed a commit that referenced this pull request Jan 11, 2024
## Motivation
We need to wait to be ATX-synced before selecting a commitment ATX to select the best possible one.

## Changes
- moved waiting for ATX-synced to the function that finds the best ATX for commitment ATX
- post supervisor doesn't always wait for ATX-synced before it starts post-service

## Test Plan
Existing tests pass
@spacemesh-bors
Copy link

Build failed (retrying...):

  • ci-status

spacemesh-bors bot pushed a commit that referenced this pull request Jan 11, 2024
## Motivation
We need to wait to be ATX-synced before selecting a commitment ATX to select the best possible one.

## Changes
- moved waiting for ATX-synced to the function that finds the best ATX for commitment ATX
- post supervisor doesn't always wait for ATX-synced before it starts post-service

## Test Plan
Existing tests pass
@spacemesh-bors
Copy link

Pull request successfully merged into develop.

Build succeeded:

@spacemesh-bors spacemesh-bors bot changed the title Wait for ATX-synced before selecting commitment ATX [Merged by Bors] - Wait for ATX-synced before selecting commitment ATX Jan 11, 2024
@spacemesh-bors spacemesh-bors bot closed this Jan 11, 2024
@spacemesh-bors spacemesh-bors bot deleted the wait-for-atx-sync-before-selecting-commitment-atx branch January 11, 2024 13:10
poszu added a commit that referenced this pull request Jan 16, 2024
## Motivation
We need to wait to be ATX-synced before selecting a commitment ATX to select the best possible one.

## Changes
- moved waiting for ATX-synced to the function that finds the best ATX for commitment ATX
- post supervisor doesn't always wait for ATX-synced before it starts post-service

## Test Plan
Existing tests pass
poszu added a commit that referenced this pull request Jan 17, 2024
## Motivation
We need to wait to be ATX-synced before selecting a commitment ATX to select the best possible one.

## Changes
- moved waiting for ATX-synced to the function that finds the best ATX for commitment ATX
- post supervisor doesn't always wait for ATX-synced before it starts post-service

## Test Plan
Existing tests pass
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