-
Notifications
You must be signed in to change notification settings - Fork 211
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] - Removed API to update poet servers from SmesherService #5209
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #5209 +/- ##
=========================================
- Coverage 78.1% 78.0% -0.1%
=========================================
Files 264 264
Lines 31825 31802 -23
=========================================
- Hits 24870 24831 -39
- Misses 5454 5474 +20
+ Partials 1501 1497 -4
|
bors try |
tryBuild succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
bors merge |
## Motivation Remove the API to live-update poet servers ## Changes - updated the api dep to spacemeshos/api#280, merge after this - removed relevant code - updated the system tests using this API - fixed a problem in system tests where multiple `smesher-0` nodes are created all of which but 1 are destroyed - I believe this causes recent flakiness of systests. ## Test Plan Existing tests pass
Build failed:
|
Bors merge |
## Motivation Remove the API to live-update poet servers ## Changes - updated the api dep to spacemeshos/api#280, merge after this - removed relevant code - updated the system tests using this API - fixed a problem in system tests where multiple `smesher-0` nodes are created all of which but 1 are destroyed - I believe this causes recent flakiness of systests. ## Test Plan Existing tests pass
Pull request successfully merged into develop. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page.
|
Clean up leftovers missed in #5209.
Clean up leftovers missed in #5209.
Motivation
Remove the API to live-update poet servers
Changes
smesher-0
nodes are created all of which but 1 are destroyed - I believe this causes recent flakiness of systests.Test Plan
Existing tests pass