Skip to content

Fix 'bke.yaml' discovery file#14524

Merged
Jellyfrog merged 1 commit intolibrenms:masterfrom
PipoCanaja:bke
Oct 26, 2022
Merged

Fix 'bke.yaml' discovery file#14524
Jellyfrog merged 1 commit intolibrenms:masterfrom
PipoCanaja:bke

Conversation

@PipoCanaja
Copy link
Copy Markdown
Contributor

@PipoCanaja PipoCanaja commented Oct 26, 2022

Missing spaces and $ in sensor_index.

DO NOT DELETE THE UNDERLYING TEXT

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

  • Have you followed our code guidelines?
  • If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.
  • If my Pull Request makes discovery/polling/yaml changes, I have added/updated test data.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

@PipoCanaja PipoCanaja added Bug 🕷️ Device 🖥️ New or added device support labels Oct 26, 2022
@PipoCanaja PipoCanaja self-assigned this Oct 26, 2022
@PipoCanaja PipoCanaja changed the title Fix 'bke.yaml' file Fix 'bke.yaml' discovery file Oct 26, 2022
@Jellyfrog Jellyfrog merged commit 0470b03 into librenms:master Oct 26, 2022
@PipoCanaja PipoCanaja deleted the bke branch October 26, 2022 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug 🕷️ Device 🖥️ New or added device support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants