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.

Sign up for GitHub

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

Jump to bottom

setting remote config doesn't respect Ctrl+C #1242

Open
Aergonus opened this issue Jul 29, 2021 · 1 comment · May be fixed by #2214
Open

setting remote config doesn't respect Ctrl+C #1242

Aergonus opened this issue Jul 29, 2021 · 1 comment · May be fixed by #2214
Assignees
@sagnik3788
Labels
good first issue A good first issue to get started with. help wanted Need some extra hands to get this done. status/ready Issue ready to be worked on. type/bug Issue that reports an unexpected behaviour.

Comments

@Aergonus
Copy link

Aergonus commented Jul 29, 2021 •
edited
Loading

Description

Crtl+C does not abort hanging query

$ pack config default-builder myremotehub.com/buildpacks/builder:latest -v
Verifying local image...
Verifying remote image...


^C^C^C^C^C^C^C^C^C^C^C^CERROR: validating that builder myremotehub.com/buildpacks/builder:latest exists: failed to inspect remote image myremotehub.com/buildpacks/builder:latest: fetching builder image: connect to repo store "myremotehub.com/buildpacks/builder:latest": Get "https://myremotehub.com/v2/": net/http: TLS handshake timeout

Proposed solution

Add handler for SIGINT to abort quickly

Describe alternatives you've considered

Add a note to pull down the image in the error logs

Additional context

$ pack --version
0.19.0+git-360dbae.build-2550

#1145 relates, pulling does work if I have access

to simulate this, set your proxy to forward to /dev/null
@Aergonus Aergonus added status/triage Issue or PR that requires contributor attention. type/enhancement Issue that requests a new feature or improvement. labels Jul 29, 2021
@natalieparellano natalieparellano added type/bug Issue that reports an unexpected behaviour. status/ready Issue ready to be worked on. and removed type/enhancement Issue that requests a new feature or improvement. status/triage Issue or PR that requires contributor attention. labels Mar 15, 2022
@natalieparellano natalieparellano added help wanted Need some extra hands to get this done. good first issue A good first issue to get started with. labels Aug 21, 2023
@sagnik3788
Copy link

@natalieparellano may i give it a try

@sagnik3788 sagnik3788 linked a pull request Jul 14, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagnik3788 sagnik3788

Labels
good first issue A good first issue to get started with. help wanted Need some extra hands to get this done. status/ready Issue ready to be worked on. type/bug Issue that reports an unexpected behaviour.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Handle SIGINT to abort hanging queries in ValidateBuilderExists sagnik3788/pack
3 participants
@Aergonus @natalieparellano @sagnik3788