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

Alcatel-Lucent Aos7 sensors nobulk #12935

Merged
merged 17 commits into from
Jun 8, 2021
Merged

Conversation

paulierco
Copy link
Contributor

Continues from here.

Thanks @murrant for nobulk change.

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.

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.

@paulierco
Copy link
Contributor Author

Tests are failing. Is it something by design ?

1) LibreNMS\Tests\YamlSchemaTest::testDiscoveryDefinitionSchema aos7.yaml does not validate. Violations: [modules.sensors.state.data[0]] The property nobulk is not defined and the definition does not allow additional properties [modules.sensors.state.data[1]] The property nobulk is not defined and the definition does not allow additional properties

@PipoCanaja
Copy link
Contributor

Tests are failing. Is it something by design ?

Hello,
A small change is missing to allow the use of "nubulk" currently.
I'll add the fix.

Copy link
Contributor

@PipoCanaja PipoCanaja left a comment

Choose a reason for hiding this comment

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

LGTM

@Jellyfrog Jellyfrog added the Device 🖥️ New or added device support label Jun 8, 2021
@Jellyfrog Jellyfrog merged commit 4ee1abb into librenms:master Jun 8, 2021
@paulierco paulierco deleted the aos7-nobulk2 branch June 9, 2021 04:09
@librenms-bot
Copy link

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/21-6-0-changelog/16173/1

trympet pushed a commit to trympet/librenms that referenced this pull request Oct 28, 2021
* first commit

* fix style

* fix style

* fix style

* fix style

* fix style

* fix style

* fix style

* change local_port_id

* fix style

* fix style

* fix style

* first commit

* fid disc

* tests?

* fix "nobulk" test-check

Co-authored-by: PipoCanaja <38363551+PipoCanaja@users.noreply.github.com>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants