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

Dockerfile generation fails with new EmPy 4.0.0 #1014

Open
mikaelarguedas opened this issue Dec 1, 2023 · 2 comments
Open

Dockerfile generation fails with new EmPy 4.0.0 #1014

mikaelarguedas opened this issue Dec 1, 2023 · 2 comments

Comments

@mikaelarguedas
Copy link
Contributor

EmPy 4.0.0 was released on November 30

Jobs relying on ros_buildfarm started failing with the following error:

  File "/usr/local/lib/python3.12/site-packages/ros_buildfarm/templates/__init__.py", line 165, in create_dockerfile
    content = expand_template(template_name, data)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/ros_buildfarm/templates/__init__.py", line 102, in expand_template
    interpreter.string(content, template_path, locals=data)
TypeError: Interpreter.string() got multiple values for argument 'locals'

I didnt find a test or simple reproducing example on this repository.

An example of such failing job: https://github.com/osrf/docker_images/actions/runs/7056402886/job/19208326490

It can be reproduced with:

cd /tmp
git clone git@github.com:osrf/docker_images.git -b empy_400_error
cd docker_images/docker
make build
./run.sh bash
cd ros
./create_dockerfolders.py dir -d . -a

When reverting empy to v 3.3.4 we can see that the error doesnt occur

@tfoote
Copy link
Member

tfoote commented Mar 15, 2024

This is something we need to resolve

On rocker I did this: osrf/rocker#261
There's a more complicated system with migration on colcon that @cottsay is working on https://github.com/colcon/colcon-core/pull/627/files

@tfoote
Copy link
Member

tfoote commented Mar 15, 2024

This is pinned down for CI in #1015 which should be reverted when this is resolved.

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

No branches or pull requests

2 participants