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

no snmptrapd installed on the CI testbeds. #14523

Closed
fdumontet6WIND opened this issue Oct 2, 2023 · 6 comments
Closed

no snmptrapd installed on the CI testbeds. #14523

fdumontet6WIND opened this issue Oct 2, 2023 · 6 comments
Labels
ci_system tests Topotests, make check, etc

Comments

@fdumontet6WIND
Copy link
Contributor

snmptrapd is requested for testing bgp4v2 notification trap implementation: #14333

related documentation for SNMP support and SNMP traps handling requires : "Make sure snmptrapd is started."

currently snmptrapd binary is not installed on FRR test CI system

@fdumontet6WIND fdumontet6WIND added the triage Needs further investigation label Oct 2, 2023
@ton31337 ton31337 added tests Topotests, make check, etc ci_system and removed triage Needs further investigation labels Oct 3, 2023
@mwinter-osr
Copy link
Member

Not sure what you mean with "Make sure snmptrapd is started." - Would this be a single server on the test system? Or should this be a snmptrapd for each network namespace?
Or maybe check for availability of a installed snmptrapd and start/stop it during the tests (similar like we do with exabgp and other tools) ?

@fdumontet6WIND
Copy link
Contributor Author

"Make sure snmptrapd is started." is a citation from
SNMP Support

currently we only need a single server on the test system

it would be a shame to have tests that are not supported by CIP

@donaldsharp
Copy link
Member

IMO each individual test that needs a daemon running should start it as part of the test startup. I really don't understand what you are asking here though. It's reasonable to ask Martin to provide a snmptrapd so that a test that is interested can start the daemon

@fdumontet6WIND
Copy link
Contributor Author

I ask only the snmptrapd daemon installed on test plateform.

@mwinter-osr
Copy link
Member

snmptrapd is now installed.

@fdumontet6WIND
Copy link
Contributor Author

fdumontet6WIND commented Oct 9, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci_system tests Topotests, make check, etc
Projects
None yet
Development

No branches or pull requests

4 participants