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

Docs: http protocol spec for producer plugin #171

Merged
merged 2 commits into from
Sep 16, 2022
Merged

Conversation

ericpassmore
Copy link
Contributor

Description

Complete update of plugin_producer HTTP protocol documentation. One file changed.

Fixes

resolved #87

Summary

  • corrected return codes, filling in 400 errors when missing
  • corrected schemas both provided and returned
  • updated descriptions
  • added get_account_ram_corrections which was missing from docs

@ericpassmore ericpassmore added the documentation Improvements or additions to documentation label Sep 15, 2022
@ericpassmore
Copy link
Contributor Author

Exceeded CPU limit and that failed transferring sys tokens to account2. Should we re-run the tests, or is more debugging required?

2022-09-15T20:56:19.829943 cmd: ['programs/cleos/cleos', '--url', 'http://localhost:8890', '--wallet-url', 'http://localhost:9899', '--no-auto-keosd', 'push', 'transaction', '-j', '{"actions":[{"account":"eosio.token","name":"transfer","authorization":[{"actor":"account2","permission":"active"}],"data":{"from":"account2","to":"account1","quantity":"10.0001 SYS","memo":"tx-4"},"compression":"none"}]}'] 2022-09-15T20:56:19.849818 ERROR: b'\x1b[31mError 3080004: Transaction exceeded the current CPU usage limit imposed on the transaction\x1b[0m\n\x1b[33mError Details:\ntransaction d1c73b53e575595b1e5577638b085069205d99b64f3454f7f43e54a6483b72e0 exceeded failure limit for account account2\x1b[0m\n' 2022-09-15T20:56:19.849958 ERROR: Exception during push transaction. cmd Duration=0.020 sec. Error 3080004: Transaction exceeded the current CPU usage limit imposed on the transaction Error Details: transaction d1c73b53e575595b1e5577638b085069205d99b64f3454f7f43e54a6483b72e0 exceeded failure limit for account account2

@heifner
Copy link
Member

heifner commented Sep 15, 2022

Exceeded CPU limit and that failed transferring sys tokens to account2. Should we re-run the tests, or is more debugging required?

This is a real issue. Would you mind creating a github issue for it. Should be an easy fix.

@ericpassmore
Copy link
Contributor Author

git diff --name-only 2aefa0d3c 44ec05bfc
plugins/producer_api_plugin/producer.swagger.yaml

@ericpassmore
Copy link
Contributor Author

tracking failure in issue #174

Copy link
Contributor

@lparisc lparisc left a comment

Choose a reason for hiding this comment

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

Looks great. Just a small typo in the description field of most endpoints: Retreives --> Retrieves

@ericpassmore ericpassmore merged commit 9b49989 into main Sep 16, 2022
@ericpassmore ericpassmore deleted the docs-plugin-producer branch September 16, 2022 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"params" object is not required for set_whitelist_blacklist, add_greylist_accounts, remove_greylist_accounts
3 participants