Skip to content

Run all dev actions in containers #10803

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

Merged
merged 12 commits into from
Feb 23, 2022
Merged

Run all dev actions in containers #10803

merged 12 commits into from
Feb 23, 2022

Conversation

di
Copy link
Member

@di di commented Feb 22, 2022

Fixes #4948, fixes #10447. Closes #10469.

@di di requested a review from ewdurbin February 22, 2022 04:18
@di di force-pushed the fix/4948 branch 2 times, most recently from abffe45 to c765102 Compare February 22, 2022 23:12
Copy link
Contributor

@ewjoachim ewjoachim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments, but overall, 👍 from me

@ewdurbin
Copy link
Member

Everything seems to work except translations. I did a full purge/clean etc and

ee-imacpro:warehouse ee$ make translations
docker-compose run --rm web env -i  bin/translations
Creating warehouse_web_run ... done
bin/translations: line 7: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8): No such file or directory
+ make -C warehouse/locale/ translations
make: Entering directory '/opt/warehouse/src/warehouse/locale'
cd ../../; \
PYTHONPATH=/opt/warehouse/src pybabel extract \
	-F babel.cfg \
	--omit-header \
	--output="warehouse/locale/messages.pot" \
	warehouse
/bin/sh: 2: pybabel: not found
make: *** [Makefile:4: compile-pot] Error 127
make: Leaving directory '/opt/warehouse/src/warehouse/locale'
ERROR: 2
make: *** [translations] Error 2

@di di merged commit 3521a34 into pypi:main Feb 23, 2022
@di di deleted the fix/4948 branch February 23, 2022 19:04
@abitrolly
Copy link
Contributor

Tests on dev still fail #10052

miketheman added a commit to miketheman/warehouse that referenced this pull request Mar 20, 2022
When moving all development actions to containers in pypi#10803, this step
remained outside on the host OS.

As `xz-utils` are installed in the `web` image already, use those
instead of relying on the host to install them.

Refs: pypi#5872
Closes: pypi#5980

Signed-off-by: Mike Fiedler <[email protected]>
di pushed a commit that referenced this pull request Mar 20, 2022
When moving all development actions to containers in #10803, this step
remained outside on the host OS.

As `xz-utils` are installed in the `web` image already, use those
instead of relying on the host to install them.

Refs: #5872
Closes: #5980

Signed-off-by: Mike Fiedler <[email protected]>
domdfcoding pushed a commit to domdfcoding/warehouse that referenced this pull request Jun 7, 2022
* Run docs build in container

* Run translations build in container

* Run depchecker in container

* Run requirements build in container

* Run license check in container

* Clean up Makefile

* Update docs

* Remove unused COMMAND_ARGS, use double quotes

* Add missing semicolons

* Move github-actions-deps out of Makefile

* Set ENCODING and PATH in dev/environment

* Fix typo with translations build
domdfcoding pushed a commit to domdfcoding/warehouse that referenced this pull request Jun 7, 2022
When moving all development actions to containers in pypi#10803, this step
remained outside on the host OS.

As `xz-utils` are installed in the `web` image already, use those
instead of relying on the host to install them.

Refs: pypi#5872
Closes: pypi#5980

Signed-off-by: Mike Fiedler <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Circular dependency in Makefile Run everything in containers
4 participants