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

[Feature] Support for OMV 6 (Shaitan) #50

Closed
mohyat opened this issue Aug 4, 2021 · 5 comments
Closed

[Feature] Support for OMV 6 (Shaitan) #50

mohyat opened this issue Aug 4, 2021 · 5 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@mohyat
Copy link

mohyat commented Aug 4, 2021

Describe the issue

Not working after upgrade to 6.0-20 (Shaitan)

How to reproduce the issue

Just clean install of latest omv 6.0-20 (Shaitan)

Expected behavior

Screenshots

Here is error in HA:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 293, in async_setup
    result = await component.async_setup_entry(hass, self)  # type: ignore
  File "/config/custom_components/openmediavault/__init__.py", line 23, in async_setup_entry
    await controller.async_hwinfo_update()
  File "/config/custom_components/openmediavault/omv_controller.py", line 112, in async_hwinfo_update
    await self.hass.async_add_executor_job(self.get_hwinfo)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/openmediavault/omv_controller.py", line 171, in get_hwinfo
    tmp = self.data["hwinfo"]["uptime"].split(" ")
AttributeError: 'float' object has no attribute 'split'

Software versions

  • Home Assistant version:
  • openmediavault integration version:
  • openmediavault software version:

Home Assistant version: 2021.7.4
openmediavault integration version: 1.1.7
openmediavault software version: 6.0-20 (Shaitan)

Traceback/Error logs

Additional context

@mohyat mohyat added the bug Something isn't working label Aug 4, 2021
@tomaae tomaae changed the title [Bug] Not working after upgrade to 6.0-20 (Shaitan) [Feature] Support for OMV 6 (Shaitan) Aug 4, 2021
@tomaae tomaae added enhancement New feature or request and removed bug Something isn't working labels Aug 4, 2021
@tomaae
Copy link
Owner

tomaae commented Aug 4, 2021

Yea, I didnt expect OMV to keep consistency with new version as usual, that was one of reasons why I decided to step away from OMV.
I have no plan add OMV 6 support at the moment. If someone else wants to, I will help as much as I can.
I may add it myself if I get a hold of machine where I can install OMV temporarily, since it does not want to boot in my VM.

@tomaae tomaae added the help wanted Extra attention is needed label Aug 4, 2021
@Yannik25
Copy link
Contributor

Hi, I just created a Pull Request: #55.
With this PR OMV 5.6 as well as 6.22 is working again.

@tomaae
Copy link
Owner

tomaae commented Sep 15, 2021

thanks, merged

@LanceMcCarthy
Copy link

LanceMcCarthy commented Sep 21, 2021

Although I do not see any errors in the core logs, the openmediavault Integration doesn't appear in the HA Integrations list:

image

Note: I did restart Core a few times just to make sure it wasn't user-error :)

Details on Home Assistant OS (running on Home Assistant Blue / Odroid N2+):

image

Is there is a way to set this up manually through configuration.yaml as a temporary workaround?

@LanceMcCarthy
Copy link

Sorry, disregard. I needed to reboot the host. Just restarting Home Assistant didn't work.

@tomaae tomaae closed this as completed Sep 24, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Mar 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants