sebr / bhyve-home-assistant

Orbit BHyve custom component for Home Assistant
MIT License
252 stars 42 forks source link

3.0.0 beta should carry over config from `configuration.yaml` #134

Closed fireheadman closed 2 years ago

fireheadman commented 2 years ago

redownloaded and selected Beta version (latest). installed it, then restarted.

everything pertaining to orbit was unavailable.

If there is something I missed, please let me know and will retry

sebr commented 2 years ago

Hi @fireheadman - any more details you can provide about what you've looked at would be helpful. For example, in the Integrations & Services settings page is BHyve listed? If not, can you select "+ Add integration" and add a BHyve config? It's possible the config wasn't carried over correctly.

Separately, were there any logs you could provide?

fireheadman commented 2 years ago

I did not see it in the integrations page, was expecting to see that. I will go back through the process once more this evening/tomorrow.

On Mon, May 23, 2022 at 4:49 PM Seb Ruiz @.***> wrote:

Hi @fireheadman https://github.com/fireheadman - any more details you can provide about what you've looked at would be helpful. For example, in the Integrations & Services settings page is BHyve listed? If not, can you select "+ Add integration" and add a BHyve config? It's possible the config wasn't carried over correctly.

Separately, were there any logs you could provide?

— Reply to this email directly, view it on GitHub https://github.com/sebr/bhyve-home-assistant/issues/134#issuecomment-1135210187, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEXNQG2PHWPWODCC5GKA3A3VLQDN3ANCNFSM5WXHHHIQ . You are receiving this because you were mentioned.Message ID: @.***>

disforw commented 2 years ago

I just upgraded to your beta, rebooted and manually added via config flow. You must remove all elements of this integration from the config.yaml file before adding it config flow (the transfer over is broke). I can confirm that this is working perfectly and no errors in the log file!!! Great job! I have a 6 zone controller, a 2 zone XD hose controller, and 5 flood sensors (see attached image). ALL DEVICES and entities are working perfectly! 542A8BB6-160F-4773-B0C0-905F2898EA55

AnthonyBe commented 2 years ago

I'm a little confused here.

I was using v2.2.3 of the BHyve integration just fine and then it stopped working entirely when I updated HACS to v1.25.1

I've tried updating the BHyve integration to 3.0.0 beta 1, and removed the manual YAML config from my configuration.yaml, but the integration isn't appearing anywhere for me to add. I've also tried removing the BHyve repository from HACS, restarting, and re-adding, but there was no change.

I'm not sure what is meant by 'manually added via config flow' in the post above?

disforw commented 2 years ago

Sorry, here are the steps I took when upgrading from 2.2.3 to 3.0.0B1.

  1. download upgraded code via HACS
  2. reboot
  3. +Add Integration Orbit BHyve (Settings -> Integrations)
  4. Worked, looked good!
  5. Rebooted and it broke (errors indicating could not load platform from sensor.yaml...)
  6. removed all related code from configuration.yaml including sensor and switch
  7. deleted all related unavailable entities (to get a fresh start)
  8. rebooted, working great ever since
fireheadman commented 2 years ago

I uninstall the manual (non Beta) version from HACs Renamed all my yaml files to *.DISABLED rebooted downloaded the latest Beta version in HACS rebooted

I can see bhyve in /config/custom_components/bhyve (even watched it download while doing a tail -f home-assistant.log) rebooted once more

I do not see the integration in HA... searched for Orbit and Bhyve, even went through the entire list. How do we get it to pop in there?

fireheadman commented 2 years ago

nevermind... cached browser was the issue SHIFT + RELOAD did the trick

image

fireheadman commented 2 years ago

I like it!.... really no change from previous other than having devices and the integration inside HA Still waiting to bring water online at house, then my graphs will populate image

image

image

fireheadman commented 2 years ago

oh and the sensors I created for the graph are:

- sensor:
  - name: "Orbit: Garden Hose Timer (Water Used)"
    unit_of_measurement: "gal"
    state_class: measurement
    icon: mdi:water-pump
    state: "{{ state_attr('sensor.zone_history', 'consumption_gallons') }}"
- sensor:
  - name: "Orbit: Tiered Garden Drip (Water Used)"
    unit_of_measurement: "gal"
    state_class: measurement
    icon: mdi:water-pump
    state: "{{ state_attr('sensor.drip_zone_history', 'consumption_gallons') }}"
- sensor:
  - name: "Orbit: Tiered Garden Spray (Water Used)"
    unit_of_measurement: "gal"
    state_class: measurement
    icon: mdi:water-pump
    state: "{{ state_attr('sensor.spray_zone_history', 'consumption_gallons') }}"
disforw commented 2 years ago

Ha! I was literally zooming in to your images and going to ask you to post your graph code... And then I scrolled down.

fireheadman commented 2 years ago

not sure if this is the correct place to submit this info... Seeing this in logs since going to the beta version...

2022-05-25 20:42:44 WARNING (MainThread) [custom_components.bhyve.sensor] Unable to retreive data for Garden Hose 1 zone history: Error requesting data from https://api.orbitbhyve.com/v1/watering_events/6279da724f0c66c76bccccccccc 500, message='Internal Server Error', url=URL('https://api.orbitbhyve.com/v1/watering_events/6279da724f0c66c76b3e1ebd?t=1653532cccccccccc07&page=1&per-page=10')

2022-05-25 20:57:44 WARNING (MainThread) [custom_components.bhyve.sensor] Unable to retreive data for Spray zone history: Error requesting data from https://api.orbitbhyve.com/v1/watering_events/626b4f3c4f0c040bxxxxxxxxx: 500, message='Internal Server Error', url=URL('https://api.orbitbhyve.com/v1/watering_events/626b4f3c4f0c040b8bxxxxxxxxxx?t=1653533863.9044642&page=1&per-page=10')
sebr commented 2 years ago

No, this is just the b-hyve API being rubbish. It has nothing to do with the beta.

AnthonyBe commented 2 years ago

nevermind... cached browser was the issue SHIFT + RELOAD did the trick

Yep, that was my issue as well!

Happened to try it on a different device and boom, there was Orbit BHyve when I went to add a new integration. All worked perfectly from then on!

thirtytwobits commented 2 years ago

The documentation for this readme is inadequate. It simply states "This integration should be configured from the Home Assistant devices & services UI page" but it doesn't mention how. I'm inferring from this thread that the beta is supposed to contribute an integration available by clicking the +integration button under Devices & Services? I do not see any such integration after I install the beta. I'm either experiencing a bug or making an improper assumption about how the new plugin should be configured.

Please clarify your documentation to say what under "Devices & Services" the user need to, specifically, do to configure the integration.

sebr commented 2 years ago

@thirtytwobits you happened to stumble upon it in between some work I've been doing on the integration to support UI configuration.

Yes you are correct in everything you have written, but since this is something I do in my spare time, some kindness in your tone would be appreciated.

For now please read the documentation for the 2.x tag

disforw commented 2 years ago

@sebr, I have been following this project from the beginning, I use a lot of Bhyve products. Your dedication and work here is appreciated tremendously. Perhaps @thirtytwobits would like to create a PR and submit his own revised documentation?

thirtytwobits commented 2 years ago

@thirtytwobits you happened to stumble upon it in between some work I've been doing on the integration to support UI configuration.

Yes you are correct in everything you have written, but since this is something I do in my spare time, some kindness in your tone would be appreciated.

For now please read the documentation for the 2.x tag

Oh, sorry @sebr, I didn't mean for my tone to sound harsh. In my head, my tone was neutral but re-reading it I see that "inadequate" is a bit of a terse term. Sorry again and thank you for your efforts.

I did get your updates and now everything makes sense and is working great. Sorry thanks.