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

Persistent worker failed to start the task: Parallels isolation failed #373

Closed
real-or-random opened this issue Apr 29, 2021 · 4 comments · Fixed by #375
Closed

Persistent worker failed to start the task: Parallels isolation failed #373

real-or-random opened this issue Apr 29, 2021 · 4 comments · Fixed by #375
Assignees

Comments

@fkorotkov fkorotkov transferred this issue from cirruslabs/macos-image-templates Apr 29, 2021
@fkorotkov
Copy link
Contributor

It does seem like a false alarm. @edigaryev seems it's hitting this case https://github.com/golang/crypto/blob/83a5a9bb288b4c9523e182ed4f621411a11de3df/ssh/session.go#L456-L460.

I've fixed the issue on the server side: now Cirrus ignores failures from the persistent worker in case of a task being already in a final state. In @real-or-random cases all the tasks finished successfully but then CLI reported the failure.

@edigaryev once you are back from PTO next week let's think if CLI should just ignore ExitMissingError.

@real-or-random
Copy link
Author

Thanks for the quick fixes again!

Sorry, here's another failure:
https://cirrus-ci.com/task/6378104290541568
Not sure if this is related or not. The error message is different but it shows the same behavior of marking a task failed even though it has run successfully.

Maybe this is already fixed here but the fixed is simply not deployed? If yes, ignore me.

@fkorotkov
Copy link
Contributor

@real-or-random this is a bit different issue. Seems there was an issue while updating clone command status (this is why it's reported as failed). It caused a confusion between service that is responsible for status updates of tasks/builds and the service that runs persistent workers.

I've fixed the issue. Sorry for the inconvenience!

@real-or-random
Copy link
Author

I've fixed the issue. Sorry for the inconvenience!

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants