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

Investigate removing YAML config #284

Closed
Tracked by #274
strawgate opened this issue Apr 22, 2024 · 1 comment · Fixed by #288
Closed
Tracked by #274

Investigate removing YAML config #284

strawgate opened this issue Apr 22, 2024 · 1 comment · Fixed by #288
Milestone

Comments

@strawgate
Copy link
Collaborator

We should identify if we can remove the YAML configuration without disrupting existing users.

Ideally we'd just stop pulling new settings from the YAML without breaking existing users and present a "repair" that tells them to move to UI-based configuration

@strawgate
Copy link
Collaborator Author

So I performed a test where I added an elasticsearch configuration to configuration.yaml:

elasticsearch:
  url: https://host.docker.internal:9200
  verify_ssl: false
  username: hass_writer
  password: changeme

I then removed the configuration from the yaml and saw the elasticsearch integration was still configured in hass.

I then readded the configuration to the yaml, but removed all yaml config handling from the integration. I then started the integration again.

In the logs I see an exception thrown: homeassistant.data_entry_flow.UnknownStep: Handler ConfigEntriesFlowManager doesn't support step import but otherwise the integration persists and I am now able to modify options via the UI.

So it would appear as though we can simply remove the yaml configuration and anyone who was using it is effectively migrated to UI-based management on upgrade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants