Volume path parsing seems broken

[pkarakal]

In the docker-compose.yml, under the volumes I have the following

volumes:
      - ./environment/configuration/app.json:/usr/share/nginx/html/assets/config/app.json

When running podman-compose up I see that the path of the volume gets tangled up and instead of getting $(pwd)/environment/configuration/app.json, I see $(pwd)/./environment/configuration/app.json and this way the container cannot cannot start.

Environment

OS/Distro: Fedora 32
Podman version:

podman --version
podman version 2.0.2

podman-compose version:

Name         : podman-compose
Version      : 0.1.5

Maybe it's something worth taking a look into.
When running docker-compose up it seems to start the container just fine.

[elmariachi111]

what happens when you add a :Z at the end?

volumes:
      - ./environment/configuration/app.json:/usr/share/nginx/html/assets/config/app.json:Z

?

[pkarakal]

what happens when you add a :Z at the end?

volumes:
      - ./environment/configuration/app.json:/usr/share/nginx/html/assets/config/app.json:Z

?

@elmariachi111 I don't see any change. This still does the same thing when parsing the volume path unfortunately.

[pkarakal]

replacing the ./ with ${PWD}/ seems to eliminate the weird parsing.

I also added a bind-propagation parameter ./environment/configuration/app.json:/usr/share/nginx/html/assets/config/app.json:rshared and it seems to be fine

[JonasKs]

Please don't close the issue because a workaround has been found, when the issue is still valid.

I've encountered the same now, and the fix was indeed replacing ./ with ${PWD}.

[pkarakal]

The issue on the code seems to get fixed when using os.path.realpath when parsing the volumes. (This is a TODO here https://github.com/containers/podman-compose/blob/devel/podman_compose.py#L90) .

[muayyad-alsadi]

would you please confirm that this bug is still there in latest code?

[pkarakal]

would you please confirm that this bug is still there in latest code?

yeap. The bug still persists. tried it with the latest master branch.

[muayyad-alsadi]

the following are equivalent and ./ or multiple /// are valid

$(pwd)/environment/configuration/app.json
$(pwd)/./environment/configuration/app.json
$(pwd)/////environment/configuration/app.json

despite that I'll fix it