search

davidusb-geek / emhass

emhass: Energy Management for Home Assistant, is a Python module designed to optimize your home energy interfacing with Home Assistant.
MIT License
260 stars 51 forks source link

Docker Standalone Publish Workspace Fix #207

Closed GeoDerp closed 4 months ago

GeoDerp commented 4 months ago

https://github.com/davidusb-geek/emhass/issues/205

GeoDerp commented 4 months ago

Work in progress, still testing.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.63%. Comparing base (9d85851) to head (20cc256).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #207 +/- ## ======================================= Coverage 87.63% 87.63% ======================================= Files 6 6 Lines 1707 1707 ======================================= Hits 1496 1496 Misses 211 211 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

GeoDerp commented 4 months ago

@davidusb-geek , Im trying to test the DockerHub integration, however having some issues. I may need you to test this one your end.

davidusb-geek commented 4 months ago

Ok. I just manually launched the docker publish workflow and it is still failing with some authentication issue.

GeoDerp commented 4 months ago

Thanks. I think I might know why. Working it out now 👍

GeoDerp commented 4 months ago

Considering how long arm v7 takes to build, I may finish testing this tomorrow. (I think I Just have to check the tags work appropriately now).

So public_docker.yaml has been modified using: https://docs.docker.com/build/ci/github-actions/multi-platform/ as an example.

This workflow is beneficial, as it distributes all the architectures to different runners (making it faster and similar to Emhass-Add-on's matrix).

docker-build-test.yaml file has also been created as an subset of this that works to test the build of each architecture (just like the Add-on version on EMHASS-Add-on )

GeoDerp commented 4 months ago

@davidusb-geek , this has been tested. Should be good to go.