Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

[CAD-2178] The retryCount from the /errors endpoint is not correctly incremented. #125

Merged
merged 2 commits into from
Dec 3, 2020

Conversation

ksaric
Copy link
Contributor

@ksaric ksaric commented Dec 1, 2020

@ksaric ksaric force-pushed the ksaric/CAD-2178 branch 2 times, most recently from 5c91496 to da43777 Compare December 3, 2020 12:00
@ksaric
Copy link
Contributor Author

ksaric commented Dec 3, 2020

bors try

iohk-bors bot added a commit that referenced this pull request Dec 3, 2020
@ksaric
Copy link
Contributor Author

ksaric commented Dec 3, 2020

Please note that it is possible that a user submits the same pool hash several times which will result in the fetch being done for each of the entries initially. We do this in order to make sure that the metadata wasn't fixed in the meantime, but after the initial fetch, we keep retrying just the latest fetch error (since the errors persisted, that means that not a single entry fixed all the issues).

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Dec 3, 2020

try

Build failed:

@ksaric
Copy link
Contributor Author

ksaric commented Dec 3, 2020

bors try

iohk-bors bot added a commit that referenced this pull request Dec 3, 2020
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Dec 3, 2020

try

Build succeeded:

@ksaric
Copy link
Contributor Author

ksaric commented Dec 3, 2020

Also tested with the previous version syncing up to (less then tip) mainnet and switching to this branch to see if the migration works correctly.

@ksaric ksaric merged commit 9d5efc8 into master Dec 3, 2020
@iohk-bors iohk-bors bot deleted the ksaric/CAD-2178 branch December 3, 2020 13:04
This was referenced Dec 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant