Record file debugging tool #217
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
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Ivo-Yankov
added
the
New Feature
Ivo-Yankov
requested review from
georgi-l95,
dimitrovmaksim,
natanasow and
nathanklick
# Conflicts: # src/utils/cliOptions.js
georgi-l95
approved these changes
Nov 14, 2022
cli.js
Outdated
| @@ -62,6 +62,16 @@ yargs(hideBin(process.argv)) | |||
| await HederaUtils.generateAccounts(console, argv.accounts); | |||
| } | |||
| ) | |||
| .command( | |||
| "debug [timestamp]", | |||
| "Parses and prints the contents of the record file that has been created during the selected timestamp [default: latest]", | |||
There was a problem hiding this comment.
nit: doesn't work if no timestamp is passed or if latest is passed.
hedera-local-node % node cli.js debug
cli.js debug [timestamp]
Parses and prints the contents of the record file that has been created during
the selected timestamp [default: latest]
Options:
--help Show help [boolean]
--version Show version number [boolean]
--timestamp Record file timestamp [string] [required]
Missing required argument: timestamp
hedera-local-node % node cli.js debug latest
Invalid timestamp string. Accepted formats are: 0000000000.000000000 and 0000000000-000000000
hedera-local-node %
There was a problem hiding this comment.
The tooltip was outdated. I decided against implementing the option to parse the latest record file, since that information wouldn't really be usefull due to the constant and fast creation of newer record files.
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
Signed-off-by: Ivo Yankov <ivo@devlabs.bg>
|
Kudos, SonarCloud Quality Gate passed!
|
Kalina-Todorova
pushed a commit
that referenced
this pull request
Nov 17, 2022
* Add smoke tests for each network Signed-off-by: nikolay <n.atanasow94@gmail.com> * Edit ci Signed-off-by: nikolay <n.atanasow94@gmail.com> * Bump versions Signed-off-by: nikolay <n.atanasow94@gmail.com> * Edit ci Signed-off-by: nikolay <n.atanasow94@gmail.com> * Edit ci Signed-off-by: nikolay <n.atanasow94@gmail.com> * Fix local node occasionally fails to start (#203) * wait before generating accounts Signed-off-by: georgi-l95 <glazarov95@gmail.com> * bump sdk version Signed-off-by: georgi-l95 <glazarov95@gmail.com> * bump sdk version Signed-off-by: georgi-l95 <glazarov95@gmail.com> * test Signed-off-by: georgi-l95 <glazarov95@gmail.com> * check if network node is started Signed-off-by: georgi-l95 <glazarov95@gmail.com> Signed-off-by: georgi-l95 <glazarov95@gmail.com> * Add environment readiness test flow (#188) Add environment readiness test flow Signed-off-by: georgi-l95 <glazarov95@gmail.com> * Add feature flag for atomicCryptoTransfer (#210) Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: nikolay <n.atanasow94@gmail.com> * bump version 2.1.3 Signed-off-by: georgi-l95 <glazarov95@gmail.com> * feat: add a flag to control relay rate limit (#206) Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * feat: add option to enable dev mode (#214) Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: add missing env vars (#221) Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * Record file debugging tool (#217) * wip: add a record parsing command Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * wip: parse records from inside network node Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * chore: temporarily add java build files Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * chore: code cleanup Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * chore: re-ignore compile directory Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * feat: input validation Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * nit: resolve code smells Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: debug tooltip Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: changed record file selection logic Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: load env vars at correct time Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * Official hedera explorer image (#222) * chore: bump version Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * update explorer image registry Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: georgi-l95 <glazarov95@gmail.com> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Co-authored-by: nikolay <n.atanasow94@gmail.com> Co-authored-by: georgi-l95 <glazarov95@gmail.com>
georgi-l95
added a commit
that referenced
this pull request
Dec 2, 2022
* Add smoke tests for each network Signed-off-by: nikolay <n.atanasow94@gmail.com> * Edit ci Signed-off-by: nikolay <n.atanasow94@gmail.com> * Bump versions Signed-off-by: nikolay <n.atanasow94@gmail.com> * Edit ci Signed-off-by: nikolay <n.atanasow94@gmail.com> * Edit ci Signed-off-by: nikolay <n.atanasow94@gmail.com> * Fix local node occasionally fails to start (#203) * wait before generating accounts Signed-off-by: georgi-l95 <glazarov95@gmail.com> * bump sdk version Signed-off-by: georgi-l95 <glazarov95@gmail.com> * bump sdk version Signed-off-by: georgi-l95 <glazarov95@gmail.com> * test Signed-off-by: georgi-l95 <glazarov95@gmail.com> * check if network node is started Signed-off-by: georgi-l95 <glazarov95@gmail.com> Signed-off-by: georgi-l95 <glazarov95@gmail.com> * Add environment readiness test flow (#188) Add environment readiness test flow Signed-off-by: georgi-l95 <glazarov95@gmail.com> * Add feature flag for atomicCryptoTransfer (#210) Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: nikolay <n.atanasow94@gmail.com> * bump version 2.1.3 Signed-off-by: georgi-l95 <glazarov95@gmail.com> * feat: add a flag to control relay rate limit (#206) Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * feat: add option to enable dev mode (#214) Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: add missing env vars (#221) Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * Record file debugging tool (#217) * wip: add a record parsing command Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * wip: parse records from inside network node Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * chore: temporarily add java build files Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * chore: code cleanup Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * chore: re-ignore compile directory Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * feat: input validation Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * nit: resolve code smells Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: debug tooltip Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: changed record file selection logic Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * fix: load env vars at correct time Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * Official hedera explorer image (#222) * chore: bump version Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * update explorer image registry Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> * Fix ports mapping (#234) Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: nikolay <n.atanasow94@gmail.com> * Bump the relay image version (#235) Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: nikolay <n.atanasow94@gmail.com> * audit fix Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: georgi-l95 <glazarov95@gmail.com> Signed-off-by: Ivo Yankov <ivo@devlabs.bg> Co-authored-by: nikolay <n.atanasow94@gmail.com> Co-authored-by: Ivo Yankov <ivo@devlabs.bg>
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.
Description:
Adds a new command for reading the contents of a record file from a provided timestamp:
Related issue(s):
Fixes #
Notes for reviewer:
Checklist