Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

OddmarDam
Copy link
Contributor

Please give a short description what your pull request is for

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.

@Jellyfrog Jellyfrog added the Device 🖥️ New or added device support label Sep 25, 2025
@@ -0,0 +1,190 @@
1.3.6.1.2.1.1.1.0|4|
1.3.6.1.2.1.1.2.0|6|..
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do these two OIDs genuinely return nothing or ..?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the manufacturor of the appliance says they didn not implement the generic snmp objects.
They say they will look in to a fesability study and then maybe implement them in a newer firmware, but i have not heard back from them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is also why the save-snmp-data.php is failing to generate the test data.

group: Channel Out
descr: 'Slot {{ $index }}'
-
oid: muxMeasurementTable
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you update oid, value and anywhere else an oid is called with the MIB-NAME::OID format please.

@laf
Copy link
Member

laf commented Sep 28, 2025

I've made some small tweaks to this PR @OddmarDam. However your test data doesn't actually contain the OID that you're using to do discovery: .1.3.6.1.4.1.23180.2.1.1.1.1.8.0 that's needed for this to be merged and so we can generate the remaining test data.

@laf
Copy link
Member

laf commented Oct 12, 2025

@OddmarDam Still doesn't contain test data for 1.3.6.1.4.1.23180.2.1.1.1.1.8.0

@laf laf added the User-Pending Currently waiting for user response label Oct 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Device 🖥️ New or added device support User-Pending Currently waiting for user response

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.