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

Extend everest docker image with OCPP 2.0.1 compatibility #129

Merged
merged 19 commits into from
Jul 26, 2024

Conversation

hikinggrass
Copy link
Contributor

No description provided.

@barsnick
Copy link
Contributor

This should fix the issue with the current Dockerfile, that Debian 11 can no longer compile libevse-security:
EVerest/libevse-security#75

I just drafted my own updated Debian 12 Dockerfile, just to figure out you already have the exact changes in this branch (notably in e13daae1eb76a00399871bba7eda45eaa371b9eb).

My Dockerfile had this issue:

5.311   DEPRECATION: netifaces is being installed using the legacy 'setup.py install' method, because it does not have a 'pyproject.toml' and the 'wheel' package is not installed. pip 23.1 will enforce this behaviour change. A possible replacement is to enable the '--use-pep517' option. Discussion can be found at https://github.com/pypa/pip/issues/8559
5.312   Running setup.py install for netifaces: started
5.459   Running setup.py install for netifaces: finished with status 'error'

Your branch does build fine.

Signed-off-by: Kai-Uwe Hermann <[email protected]>
This uses the init_device_model_db.py from libocpp and associated files, so copy over the libocpp/config/v201 directory into the resulting image as well

Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Kai-Uwe Hermann <[email protected]>
This can remove the "--ssh default" parameter from docker build

Signed-off-by: Kai-Uwe Hermann <[email protected]>
This makes the container runtime used for building and saving the image selectable
Defaults to "docker" but can be used to e.g. select "podman" as well

Signed-off-by: Kai-Uwe Hermann <[email protected]>
This allows additional CMake parameters to be passed to the EVerest build

Signed-off-by: Kai-Uwe Hermann <[email protected]>
This logs the build date, used container runtime and used additional CMake parameters

Signed-off-by: Kai-Uwe Hermann <[email protected]>
Signed-off-by: Kai-Uwe Hermann <[email protected]>
@hikinggrass hikinggrass force-pushed the feature/everest-docker-image-ocpp201-compatibility branch from e13daae to 62b290b Compare July 26, 2024 11:49
@hikinggrass hikinggrass marked this pull request as ready for review July 26, 2024 12:05
@hikinggrass hikinggrass merged commit b1d74b3 into main Jul 26, 2024
6 of 7 checks passed
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.

3 participants