Skip to content

Add pip dependency to resolve hanging custom_work_dependency test #15429

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
wants to merge 3 commits into from

Conversation

otaj
Copy link
Contributor

@otaj otaj commented Oct 31, 2022

Hopefully resolves hang happening in custom_work_dependency test, that is blocking app framework.

The related issue pypa/pip#11387
cc @carmocca @akihironitta @Borda @manskx

@otaj otaj self-assigned this Oct 31, 2022
@otaj otaj added this to the v1.8 milestone Oct 31, 2022
@github-actions github-actions bot added the app (removed) Generic label for Lightning App package label Oct 31, 2022
@otaj otaj added the ci Continuous Integration label Oct 31, 2022
@Borda
Copy link
Member

Borda commented Oct 31, 2022

@akihironitta
Copy link
Contributor

Is this PR a patch for #15417? If so, it's just been fixed by #15431. See its CI run in master: https://dev.azure.com/Lightning-AI/lightning/_build/results?buildId=111417&view=results

@Borda
Copy link
Member

Borda commented Oct 31, 2022

I think that #15431 was a hotfix to unblock the master but not addressing the underlying issue...

@justusschock
Copy link
Member

@akihironitta mind checking this PR if it is still relevant? Would greatly appreciate if you could take it over or just close it.

@justusschock justusschock mentioned this pull request Nov 9, 2022
12 tasks
@akihironitta
Copy link
Contributor

Closing this PR as the reason for the hang was because of the very outdated docker image used in CI as I confimed with Mansy last week in #15417.

@akihironitta akihironitta deleted the ci/e2e_custom_work branch November 9, 2022 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app (removed) Generic label for Lightning App package ci Continuous Integration
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants