Skip to content

[prebuild] stuck in PENDING if image build fails (due to error in dockerfile) #13207

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

Closed
AlexTugarev opened this issue Sep 22, 2022 · 2 comments · Fixed by #13296
Closed

[prebuild] stuck in PENDING if image build fails (due to error in dockerfile) #13207

AlexTugarev opened this issue Sep 22, 2022 · 2 comments · Fixed by #13296
Assignees
Labels
feature: prebuilds team: webapp Issue belongs to the WebApp team type: bug Something isn't working

Comments

@AlexTugarev
Copy link
Member

Bug description

see #13204 (comment) for context. while the linked issue is a dupe, this follow up seems to be a valid in itself.

if the image build exits with error code > 0, e.g. failure during apt-get install zzz, the workspace instance is marked as stopped, but the prebuild workspace won't be marked as failed. this means the state of the prebuild remains "PENDING" indefinitely.

we've the bridge in place to cleanup workspace instances (and derived resources) periodically, but in this case the WSI is stopped properly already. I think it wont cover this state.

cc. @geropl, please correct me if I'm wrong.

Steps to reproduce

see #13204

Workspace affected

No response

Expected behavior

prebuild workspace should be marked as failed.

this is quite important, because due to the lack of persisted image build logs, it's completely unclear what the status of a prebuild is.

Example repository

No response

Anything else?

No response

@AlexTugarev AlexTugarev added type: bug Something isn't working feature: prebuilds team: webapp Issue belongs to the WebApp team labels Sep 22, 2022
@AlexTugarev AlexTugarev self-assigned this Sep 22, 2022
@geropl
Copy link
Member

geropl commented Sep 22, 2022

@geropl, please correct me if I'm wrong.

@AlexTugarev Correct. If we stop the instance, the contract is to also stop all derived prebuilds. That contract is broken 💯 .

@AlexTugarev
Copy link
Member Author

The fix is buried in #13113. Going to extract it later on.

@geropl geropl moved this to In Progress in 🍎 WebApp Team Sep 26, 2022
Repository owner moved this from In Progress to In Validation in 🍎 WebApp Team Sep 26, 2022
@AlexTugarev AlexTugarev moved this from In Validation to Done in 🍎 WebApp Team Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: prebuilds team: webapp Issue belongs to the WebApp team type: bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants