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

tools: search/replace indexer/conduit in block-generator #5321

Merged
merged 16 commits into from Apr 28, 2023

Conversation

tzaffi
Copy link
Contributor

@tzaffi tzaffi commented Apr 20, 2023

Summary

Replacing mentions of "indexer" with "conduit" and some other small tweaks.

Test Plan

Locally tested by running

cp test_config.yml scenario.yml
./run_runner.sh ./conduit scenario.yml

inside of tools/block-generator (after copying over a conduit binary)

@tzaffi tzaffi marked this pull request as ready for review April 20, 2023 21:26
RunnerCmd.Flags().StringVarP(&runnerArgs.LogLevel, "log-level", "l", "error", "LogLevel to use when starting Indexer. [panic, fatal, error, warn, info, debug, trace]")
RunnerCmd.Flags().StringVarP(&runnerArgs.CPUProfilePath, "cpuprofile", "", "", "Path where Indexer writes its CPU profile.")
RunnerCmd.Flags().StringVarP(&runnerArgs.LogLevel, "log-level", "l", "error", "LogLevel to use when starting Conduit. [panic, fatal, error, warn, info, debug, trace]")
RunnerCmd.Flags().StringVarP(&runnerArgs.CPUProfilePath, "cpuprofile", "", "", "Path where Conduit writes its CPU profile.")
Copy link
Contributor

Choose a reason for hiding this comment

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

making a note to myself that I need to add this to the config template.

shiqizng
shiqizng previously approved these changes Apr 21, 2023
@tzaffi tzaffi requested a review from shiqizng April 21, 2023 17:02
@tzaffi tzaffi changed the title tools: copy/replace indexer/conduit in block-generator tools: search/replace indexer/conduit in block-generator Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants