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

rpk/start: Reduce startup time #1185

Merged
merged 1 commit into from
Apr 17, 2021

Conversation

0x5d
Copy link
Contributor

@0x5d 0x5d commented Apr 16, 2021

Disable all checks when --check=false to reduce startup time.

Before:

0.09s user 0.06s system 6% cpu 2.195 total

After

0.07s user 0.07s system 13% cpu 1.040 total

Measured with:

time rpk redpanda start --check=false  -- --unknown-rp-flag

--unknown-rp-flag is passed so redpanda exits upon encountering an invalid flag.

Fixes: #1158

@0x5d 0x5d added the area/rpk label Apr 16, 2021
@0x5d 0x5d requested a review from emaxerrno April 16, 2021 23:18
@0x5d 0x5d self-assigned this Apr 16, 2021
@0x5d 0x5d requested a review from mmaslankaprv as a code owner April 16, 2021 23:18
emaxerrno
emaxerrno previously approved these changes Apr 17, 2021
@0x5d 0x5d force-pushed the rpk-start-disable-vendor-check branch from b41a97e to bb1ec2e Compare April 17, 2021 14:36
Disable all checks when `--check=false` to reduce startup time.

Before:

0.09s user 0.06s system 6% cpu 2.195 total

After

0.07s user 0.07s system 13% cpu 1.040 total

Measured with:

time rpk redpanda start --check=false  -- --unknown-rp-flag

--unknown-rp-flag is passed so redpanda exits upon encountering
an invalid flag.
@0x5d 0x5d force-pushed the rpk-start-disable-vendor-check branch from bb1ec2e to 87d06c1 Compare April 17, 2021 14:37
@emaxerrno emaxerrno merged commit 8f38139 into redpanda-data:dev Apr 17, 2021
@0x5d 0x5d deleted the rpk-start-disable-vendor-check branch April 19, 2021 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

testing: rpk --check=false should skip all testing straight into execvpe()
2 participants