| 14 Jun 2021 |
 etu | Metrics doesn't have to be built in, if current state of things are exposed, another component can scrape that to build metrics | 21:41:24 |
| etu | Then things can check the metrics component to fire events | 21:41:47 |
 dotlambda | How do I mark Home Assistant component tests as online only? Is that something upstream does at all? | 22:34:59 |
 @hexa:lossy.network | I thought about adding pytest.mark.network upstream, if they're interested | 22:36:28 |
| dotlambda | But you didn't try it yet? | 22:36:55 |
| @hexa:lossy.network | no, I did not | 22:37:10 |
| @hexa:lossy.network | this is the kind of thing we need social capital for 🤔 | 22:37:47 |
| @hexa:lossy.network | because it is working for them already | 22:38:00 |
| @hexa:lossy.network | no added value | 22:38:07 |
| dotlambda | And what about having a big patch just for us that adds these marks everywhere? | 22:40:16 |
| dotlambda | I guess we should try upstreaming them first. | 22:40:27 |
| @hexa:lossy.network | testing them downstream, making sure they're working well, then upstream | 22:40:48 |
| @hexa:lossy.network | make the process painless | 22:40:53 |
| dotlambda | I'm not too invested but if you make a PR upstream, feel free to add test_form_invalid_auth and test_form_cannot_connect from wallbox's tests. | 22:43:33 |
| @hexa:lossy.network | https://github.com/NixOS/nixpkgs/pull/126895 | 23:27:30 |
| 15 Jun 2021 |
 Zhaofeng Li | Actually, someone should DM balloob/home_assistant to let them know about the HN thread. I think it's only fair and they don't seem to be aware. | 01:11:30 |
| @hexa:lossy.network | I would be surprised if they hadn't discussed that internally yet | 01:13:06 |
| @hexa:lossy.network | pretty sure that is what turned the reaction around | 01:13:47 |
| @hexa:lossy.network | https://nixos.wiki/wiki/index.php?diff=5894&oldid=5892 | 02:11:59 |
| @hexa:lossy.network | first draft of a limited support passage | 02:12:18 |
 Mic92 (Old) | I won't do anything right now in this matter over the couple next week. I have other priorities right now. | 07:05:09 |
| @hexa:lossy.network | https://nixos.wiki/wiki/Home_Assistant#Running_a_recent_version_using_an_overlay | 13:07:54 |
| @hexa:lossy.network | untested, but should work | 13:08:13 |
| @hexa:lossy.network | Outdated dependencies
┏━━━━━━━━━━━━━━━┳━━━━━━━━━━━━┳━━━━━━━━┓
┃ Package ┃ Current ┃ Wanted ┃
┡━━━━━━━━━━━━━━━╇━━━━━━━━━━━━╇━━━━━━━━┩
│ numpy │ 1.20.2 │ 1.20.3 │
│ pydroid-ipcam │ 2021-06-01 │ 0.8 │
│ pyotgw │ 2021-03-25 │ 1.1b1 │
│ sqlalchemy │ 1.3.23 │ 1.4.13 │
└───────────────┴────────────┴────────┘
| 13:22:37 |
| @hexa:lossy.network | I tried to expose the python package set post-overrides in passthru | 13:22:57 |
| @hexa:lossy.network | and then use that as the package set checked in parse-requirements.py | 13:23:09 |
| @hexa:lossy.network | diff --git a/pkgs/servers/home-assistant/default.nix b/pkgs/servers/home-assistant/default.nix
index c2eae375658..805306a2d9c 100644
--- a/pkgs/servers/home-assistant/default.nix
+++ b/pkgs/servers/home-assistant/default.nix
@@ -799,6 +799,7 @@ in with py.pkgs; buildPythonApplication rec {
tests = {
inherit (nixosTests) home-assistant;
};
+ pythonPackages = py;
};
meta = with lib; {
diff --git a/pkgs/servers/home-assistant/parse-requirements.py b/pkgs/servers/home-assistant/parse-requirements.py
index 27de7a60f37..1eeb0f1356c 100755
--- a/pkgs/servers/home-assistant/parse-requirements.py
+++ b/pkgs/servers/home-assistant/parse-requirements.py
@@ -32,7 +32,7 @@ from rich.console import Console
from rich.table import Table
COMPONENT_PREFIX = "homeassistant.components"
-PKG_SET = "python3Packages"
+PKG_SET = "home-assistant.pythonPackages"
# If some requirements are matched by multiple Python packages,
# the following can be used to choose one of them
| 13:24:55 |
| @hexa:lossy.network | looks to be working, but parse-requirements doesn't yet find all packages again | 13:25:11 |
| @hexa:lossy.network | * diff --git a/pkgs/servers/home-assistant/default.nix b/pkgs/servers/home-assistant/default.nix
index c2eae375658..805306a2d9c 100644
--- a/pkgs/servers/home-assistant/default.nix
+++ b/pkgs/servers/home-assistant/default.nix
@@ -799,6 +799,7 @@ in with py.pkgs; buildPythonApplication rec {
tests = {
inherit (nixosTests) home-assistant;
};
+ pythonPackages = py.pkgs;
};
meta = with lib; {
diff --git a/pkgs/servers/home-assistant/parse-requirements.py b/pkgs/servers/home-assistant/parse-requirements.py
index 27de7a60f37..1eeb0f1356c 100755
--- a/pkgs/servers/home-assistant/parse-requirements.py
+++ b/pkgs/servers/home-assistant/parse-requirements.py
@@ -32,7 +32,7 @@ from rich.console import Console
from rich.table import Table
COMPONENT_PREFIX = "homeassistant.components"
-PKG_SET = "python3Packages"
+PKG_SET = "home-assistant.pythonPackages"
# If some requirements are matched by multiple Python packages,
# the following can be used to choose one of them
| 13:26:28 |
| @hexa:lossy.network | nvm, works | 13:26:45 |
