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

Match homeassistant core prometheus-client and faust-cchardet dependency version #3

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

meichthys
Copy link

@meichthys meichthys commented Feb 23, 2023

Currently https://github.com/meichthys/uptime_kuma which relies on pyuptimekuma is causing DependencyResolution errors when trying to install/run uptime_kuma on HomeAssistant 2023.3

It appears to be a conflict with homeassistant core which requires an older version of prometheus-client than what we currently require here.

Do you think this module would work with an older version, or do you know another way to resolve meichthys/uptime_kuma#37 without relaxing our dependency requirement here? (I doubt Homeassistant will be willing to bump their dependency just for us without significant testing).

@meichthys
Copy link
Author

Also, maybe not the right spot to report it, but pypy has v0.0.6 while github only has a v0.0.4 release.

@bdraco
Copy link

bdraco commented Mar 4, 2023

You'll need to drop cchardet as well
https://github.com/home-assistant/core/blob/34f8e94ca9be9d660751cb259dea1045ffe39a5d/homeassistant/package_constraints.txt#L79

or switch it to faust-cchardet>=2.1.18

cchardet==2.1.7

Also bump faust-chardet to2.1.18 to match HomeAssistant core
@meichthys
Copy link
Author

@jayakornk Could you take a look at this when you get a chance?
Currently the uptkimekuma integration is broken after a recent HA update due to dependency conflicts.

@meichthys meichthys changed the title Match homeassistant core prometheus-client dependency version Match homeassistant core prometheus-client and faust-cchardet dependency version Mar 6, 2023
@meichthys
Copy link
Author

FYI - I've released a temporary patch (pyuptimekuma-hass) until we can get this merged/fixed.

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.

Requirements for uptime_kuma not found: ['pyuptimekuma==0.0.6'].
2 participants