add timeout to websock, resolves hanging issue #1609
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Eugene-hu
approved these changes
Dec 6, 2023
Closed
ifrit98
added a commit
that referenced
this pull request
Dec 7, 2023
* add timeout to websock, resolves hanging issue * add timeout set try/catch
Merged
ifrit98
added a commit
that referenced
this pull request
Dec 11, 2023
* add timeout to websock, resolves hanging issue (#1609) * add timeout to websock, resolves hanging issue * add timeout set try/catch * add ability to turn off verbose logging for btcli cmds (#1608) * add ability to turn off verbose logging for btcli cmds * black * Add taostats explorer to map (#1607) * add taostats explorer to map and printed list of links for transfers * update test for explorer * add polkascan explorer endpoint * fix endpoint tests * table set take command This is intended as a fix for squash+merge issues that arise when merging master back into staging. Then PRs on github (not git) show the unsquashed commits when attempting to create a new release branch. * update changelog, version * don't log in process pool for stake command * Sudo/hyperparams pallet update (#1612) * Update hyperparams set extrinsic to use AdminUtils pallet * update metadata call --------- Co-authored-by: Ayden Brewer <dalegribble@riseup.net> Co-authored-by: philanthrope <ifrit98@gmail.com> * update changelog 1-sentence info * fix ful changelog diff * fix full changelog diff * fix changelog for 6.4.2 * fix readme typos * Hyperparams get update (#1614) * fixes `btcli sudo get` * black --------- Co-authored-by: Eugene-hu <85906264+Eugene-hu@users.noreply.github.com> Co-authored-by: Ayden Brewer <dalegribble@riseup.net>
ifrit98
added a commit
that referenced
this pull request
Dec 12, 2023
* add timeout to websock, resolves hanging issue * add timeout set try/catch
ifrit98
added a commit
that referenced
this pull request
Dec 12, 2023
Release/6.4.2 (#1611) * add timeout to websock, resolves hanging issue (#1609) * add timeout to websock, resolves hanging issue * add timeout set try/catch * add ability to turn off verbose logging for btcli cmds (#1608) * add ability to turn off verbose logging for btcli cmds * black * Add taostats explorer to map (#1607) * add taostats explorer to map and printed list of links for transfers * update test for explorer * add polkascan explorer endpoint * fix endpoint tests * table set take command This is intended as a fix for squash+merge issues that arise when merging master back into staging. Then PRs on github (not git) show the unsquashed commits when attempting to create a new release branch. * update changelog, version * don't log in process pool for stake command * Sudo/hyperparams pallet update (#1612) * Update hyperparams set extrinsic to use AdminUtils pallet * update metadata call --------- * update changelog 1-sentence info * fix ful changelog diff * fix full changelog diff * fix changelog for 6.4.2 * fix readme typos * Hyperparams get update (#1614) * fixes `btcli sudo get` * black --------- Co-authored-by: Eugene-hu <85906264+Eugene-hu@users.noreply.github.com> Co-authored-by: Ayden Brewer <dalegribble@riseup.net>
unconst
pushed a commit
that referenced
this pull request
Dec 14, 2023
* Release/6.4.2 (#1611) * add timeout to websock, resolves hanging issue (#1609) * add timeout to websock, resolves hanging issue * add timeout set try/catch * add ability to turn off verbose logging for btcli cmds (#1608) * add ability to turn off verbose logging for btcli cmds * black * Add taostats explorer to map (#1607) * add taostats explorer to map and printed list of links for transfers * update test for explorer * add polkascan explorer endpoint * fix endpoint tests * table set take command This is intended as a fix for squash+merge issues that arise when merging master back into staging. Then PRs on github (not git) show the unsquashed commits when attempting to create a new release branch. * update changelog, version * don't log in process pool for stake command * Sudo/hyperparams pallet update (#1612) * Update hyperparams set extrinsic to use AdminUtils pallet * update metadata call --------- Co-authored-by: Ayden Brewer <dalegribble@riseup.net> Co-authored-by: philanthrope <ifrit98@gmail.com> * update changelog 1-sentence info * fix ful changelog diff * fix full changelog diff * fix changelog for 6.4.2 * fix readme typos * Hyperparams get update (#1614) * fixes `btcli sudo get` * black --------- Co-authored-by: Eugene-hu <85906264+Eugene-hu@users.noreply.github.com> Co-authored-by: Ayden Brewer <dalegribble@riseup.net> * Add an api for metadata commitments to extrinsics.serving * Change extrinsics.serving.publish_metadata argument hash to data * add docstring to publish-metadata * Add QoL function to get commitment data for netuid,uid * Return storage struct directly instead of scale object * add maxlen str * blacks --------- Co-authored-by: Eugene-hu <85906264+Eugene-hu@users.noreply.github.com> Co-authored-by: Ayden Brewer <dalegribble@riseup.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a timeout parameter to substrate so we avoid hanging calls for more than 600 sec (10 min).