travisghansen / hass-opnsense

OPNsense integration with Home Assistant
Apache License 2.0
227 stars 29 forks source link

OPNsense Firmware Updates Available #58

Closed JohnyBeGood closed 2 months ago

JohnyBeGood commented 1 year ago

Hello,

Thanks for all your work on this!

Installed it today and got a notification there is an update available in HA. After I click on "Install" nothing happens. Most likely because OPNsense is already up-to-date?

image

image

alexdelprete commented 1 year ago

v23.1.r_20 does not exist. The correct version is v23.1.r2. How did you upgrade to v23.1.x?

image

image

JohnyBeGood commented 1 year ago

I'm on the development release..

image

alexdelprete commented 1 year ago

That explains it. :)

I don't think the integration supports dev release channel.

JohnyBeGood commented 1 year ago

Got it. Hopefully, it can be added in the future :)

Thanks!

travisghansen commented 1 year ago

The api data shouldn’t really change if that’s been selected. But if it’s not returning the correct data then I’m not sure what can be done :(

How is the new version anyway?

alexdelprete commented 1 year ago

The api data shouldn’t really change if that’s been selected

So it's a bug in the API. But wait, when you call the function that gives you the latest version, do you send in the req. the release channel?

How is the new version anyway?

didn't notice anything special...but I didn't test much...

JohnyBeGood commented 1 year ago

So it's a bug in the API. But wait, when you call the function that gives you the latest version, do you send in the req. the release channel? Not sure how to verify if req. is in the release channel.

didn't notice anything special...but I didn't test much... Did not notice any issues. Everything works for me. That's always good :)

travisghansen commented 1 year ago

I didn’t notice an option to send in release channel. The backend should handle that imo.

JohnyBeGood commented 1 year ago

I didn’t notice an option to send in release channel. The backend should handle that imo.

I guess if you can’t see it, then nothing can be done until OPNsense sends that in API.

alexdelprete commented 1 year ago

I didn’t notice an option to send in release channel. The backend should handle that imo.

so it's a bug in the API or the backend...

travisghansen commented 1 year ago

On the surface it seems to be but I could be missing something.

alexdelprete commented 1 year ago

On the surface it seems to be but I could be missing something.

23.1 final has been released, I'm on 23.1-rc2. HA update sensor tells me it's available, 8 packages will be installed (correct, I double-checked in OPNsense), but tells me no reboot is needed (incorrect, OPNsense UI says reboot is needed).

I will trigger the update through HA, will report if OPNsense reboots anyway.

image image

alexdelprete commented 1 year ago

Upgrade triggered by HA went smoothly, and OPNsense rebooted. Everything ok.

There are minor inconsitencies, but the process works.

JohnyBeGood commented 1 year ago

My also went from v23.1.r_20 to 23.1.r 44

Screenshot_20230126_063912_Home Assistant

Screenshot_20230126_064036_Chrome