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

Add rPDU2BankStatusTable to generator example. #382

Merged
merged 3 commits into from
Feb 15, 2019
Merged

Conversation

SuperQ
Copy link
Member

@SuperQ SuperQ commented Feb 14, 2019

Also updates with latest keepalived MIB.

Closes: #375
Closes: #383

Signed-off-by: Ben Kochie [email protected]

Also updates with latest keepalived MIB.
Signed-off-by: Ben Kochie <[email protected]>
@brian-brazil
Copy link
Contributor

The generator test is failing.

@SuperQ
Copy link
Member Author

SuperQ commented Feb 15, 2019

I get different number of parse errors on my machine. Very strange.

@brian-brazil
Copy link
Contributor

Did you do MIBDIRS=mibs?

@SuperQ
Copy link
Member Author

SuperQ commented Feb 15, 2019

Yes, I did, but the whole mibs dir was a mess. I've re-done the way we fetch Cisco's broken set of MIBs by only copying what we need out of it.

* Only use select MIBs from the cisco set.
* Fixes all parse errors.
* Add generator build to Makefile.
* Update snmp.yml.

Signed-off-by: Ben Kochie <[email protected]>
@SuperQ
Copy link
Member Author

SuperQ commented Feb 15, 2019

Ok, now the generator parse_errors runs without any errors.

generator/Makefile Outdated Show resolved Hide resolved
@SuperQ SuperQ force-pushed the superq/apc_banks branch 3 times, most recently from 3332801 to 36e3caa Compare February 15, 2019 12:32
generator/main.go Outdated Show resolved Hide resolved
generator/main.go Outdated Show resolved Hide resolved
generator/main.go Outdated Show resolved Hide resolved
@brian-brazil brian-brazil merged commit fe49443 into master Feb 15, 2019
@brian-brazil brian-brazil deleted the superq/apc_banks branch February 15, 2019 16:07
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 this pull request may close these issues.

2 participants