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

chore(ci): test docker build on pull request #5588

Merged
merged 5 commits into from
Dec 17, 2024

Conversation

roblambell
Copy link
Contributor

@roblambell roblambell commented Dec 9, 2024

Adds GHA workflow coverage for docker image build on PRs

Looks like we had bump #5335, revert #5379, bump #5400, and open PR to fix #5447 - which this should help to avoid 🤞

@roblambell roblambell changed the title chore: test docker build on pull request chore(ci): test docker build on pull request Dec 9, 2024
@mvilanova mvilanova added docker Pull requests that update Docker code enhancement New feature or request labels Dec 12, 2024
@rialg
Copy link
Contributor

rialg commented Dec 16, 2024

#8 0.254 ModuleNotFoundError: No module named 'distutils'
#8 ERROR: process "/bin/bash -o pipefail -o errexit -c YARN_CACHE_FOLDER=\"$(mktemp -d)\"     && export YARN_CACHE_FOLDER     && pushd /usr/src/dispatch     && python setup.py bdist_wheel     && rm -r \"$YARN_CACHE_FOLDER\"     && mv /usr/src/dispatch/dist /dist" did not complete successfully: exit code: 1
------

⬆️ The test seems to be working. @mvilanova , as commented in #5447, either replace the distutils module or lower the Python version in use. That PR fix the issue by lowering the version to 3.11.10

@whitdog47
Copy link
Contributor

@roblambell please update this branch with the latest when you get a chance

@whitdog47 whitdog47 merged commit d86bebd into Netflix:master Dec 17, 2024
9 checks passed
@roblambell roblambell deleted the chore/test_docker_build branch December 17, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Pull requests that update Docker code enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants