Skip to content

meter/sonnenbatterie: cache repeated http status requests - #32797

Merged
andig merged 2 commits into
masterfrom
fix/issue-32796
Aug 13, 2026
Merged

meter/sonnenbatterie: cache repeated http status requests#32797
andig merged 2 commits into
masterfrom
fix/issue-32796

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

fixes #32796

The power and soc readings in templates/definition/meter/sonnenbatterie.yaml both call http://{{ .host }}:{{ .port }}/api/v1/status without a cache duration, so within a single poll cycle the same URI is requested twice, triggering the "uncached request repeated within 1s" warning. Added cache: 2s to both HTTP blocks, matching the convention used elsewhere (e.g. amsleser.yaml).

🤖 Generated with Claude Code

The power and soc readings both call /api/v1/status without a cache
duration, causing 'uncached request repeated within 1s' warnings.
@mfuchs1984

Copy link
Copy Markdown
Collaborator

@andig
andig marked this pull request as ready for review August 13, 2026 09:44

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@andig
andig enabled auto-merge (squash) August 13, 2026 09:47
@andig andig added the devices Specific device support label Aug 13, 2026
@andig
andig merged commit d3d8950 into master Aug 13, 2026
10 of 11 checks passed
@andig
andig deleted the fix/issue-32796 branch August 13, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devices Specific device support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sonnenbatterie template is missing cache

2 participants