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

Re-add testnet API to Hive client #57

Merged
merged 13 commits into from
Oct 10, 2022
Merged

Conversation

brianoflondon
Copy link
Collaborator

Testing against Hive post HF26

Defaults now match those held by @podping account on Hive as of 2022-08-26
Allow `hive_operation_period` to be set by ENV on startup and ignore overrides from control account
Fixes #54
Signed-off-by: Brian of London <[email protected]>
Signed-off-by: Brian of London <[email protected]>
Signed-off-by: Brian of London <[email protected]>
@brianoflondon
Copy link
Collaborator Author

Fixes #56

Signed-off-by: Brian of London <[email protected]>
Signed-off-by: Brian of London <[email protected]>
Signed-off-by: Brian of London <[email protected]>
Signed-off-by: Brian of London <[email protected]>
Signed-off-by: Brian of London <[email protected]>
@brianoflondon
Copy link
Collaborator Author

I've done extensive testing of this release against the post HardFork HF26 version of Hive using the dedicated fakenet APIs.

This update also includes a new test which looks for and uses a fakenet/testnet if one exists and performs at least one test on it.

@brianoflondon brianoflondon linked an issue Sep 28, 2022 that may be closed by this pull request
Signed-off-by: Brian of London <[email protected]>
@agates agates changed the base branch from develop to hotfix/1.2.7 October 10, 2022 18:33
@agates agates changed the title Issue56 hard fork hf26 Re-add testnet API to Hive client Oct 10, 2022
Copy link
Collaborator

@agates agates left a comment

Choose a reason for hiding this comment

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

Needs some minor changes for logging/integration tests but overall good. Will handle them after merge.

@agates agates merged commit 2ec6d25 into hotfix/1.2.7 Oct 10, 2022
@agates agates deleted the issue56-hard-fork-hf26 branch October 10, 2022 18:47
agates added a commit that referenced this pull request Oct 15, 2022
* Re-add testnet API to Hive client (#57)

* Update defaults to match current values

Defaults now match those held by @podping account on Hive as of 2022-08-26

* Fixes #54

Allow `hive_operation_period` to be set by ENV on startup and ignore overrides from control account
Fixes #54

* testing and preparing for HF26

Signed-off-by: Brian of London <[email protected]>

* fix warning for depreciated poetry option

Signed-off-by: Brian of London <[email protected]>

* linting

Signed-off-by: Brian of London <[email protected]>

* hard coded test against a testnet if one exists

Signed-off-by: Brian of London <[email protected]>

* bump version to 1.2.7

Signed-off-by: Brian of London <[email protected]>

* poetry updates

Signed-off-by: Brian of London <[email protected]>

* fix for stupidity reading envs

Signed-off-by: Brian of London <[email protected]>

* improve error handling in logging

Signed-off-by: Brian of London <[email protected]>

* fix test #51

Signed-off-by: Brian of London <[email protected]>

* update test_name in test #56

Signed-off-by: Brian of London <[email protected]>

* poetry updates

Signed-off-by: Brian of London <[email protected]>

Signed-off-by: Brian of London <[email protected]>

* Update CLI docs

* Lighthive/zmq/tooling updates

* Use new lighthive load balancing/node selection

* Fix missing property on Config

* Fix tests, refactor error handling/logging, catch more API node failures

* More specific error handling in listen_for_custom_json_operations

* Handle RC depletion

Changes to handle RC depletion by retrying.

Separate to this software, Brianoflondon will be running a service which gives RC delegations to any podping sending account.

This should be fast enough to fix RC depletion so the correct response from HiveWriter will be to re-try.

* Bump version

Signed-off-by: Brian of London <[email protected]>

* Limit nodes to exclude pharesim and others.

Signed-off-by: Brian of London <[email protected]>

* Removed https://hived.emre.sh/

Signed-off-by: Brian of London <[email protected]>

* Catch and handle RC Depletion

This introduces a sleep timer which gives time for the RC delegation script to delegate more RCs to the server.

* Catch RC exhaustion

Uses secrets for random wait time with exponential back off.

* Test for NotEnoughResourceCredits error

* lint

Signed-off-by: Brian of London <[email protected]>

* version back to 1.2.7

Signed-off-by: Brian of London <[email protected]>

* Remove random time in wait for RC Delegation

Signed-off-by: Brian of London <[email protected]>

* Unit test for not enough RCs

Signed-off-by: Brian of London <[email protected]>

* Fix issue caused by git merge

Signed-off-by: Brian of London <[email protected]>

* remove failing resource checks on startup

Signed-off-by: Brian of London <[email protected]>

* Add send_notification_iris to #48 regression tests

* Make startup error handling more clear, construct_operation non-async

* Catch TooManyCustomJsonsPerBlock for backoff

* Separate exception catch for TooManyCustomJsonsPerBlock, sleep one block

Signed-off-by: Brian of London <[email protected]>
Co-authored-by: Alecks Gates <[email protected]>
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.

Dealing with next Hive Hard Fork 26 and testing in advance
2 participants