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

Issues with ietf-snmp modules #353

Closed
vvsosed opened this issue Aug 17, 2022 · 2 comments
Closed

Issues with ietf-snmp modules #353

vvsosed opened this issue Aug 17, 2022 · 2 comments

Comments

@vvsosed
Copy link
Contributor

vvsosed commented Aug 17, 2022

I use standard ietf-snmp yang models from https://github.com/YangModels/yang/tree/main/standard/ietf/RFC. But in runtime logs I see

Aug 16 10:00:48: Warning: Augment failed in module ietf-snmp-notification: node anydata 2 cannot be added to target node /snmp:snmp
Aug 16 10:00:48: Warning: Augment failed in module ietf-snmp-notification: node anydata 2 cannot be added to target node /snmp:snmp/snmp:target-params

Looks like the yang parser can't parse ietf-snmp-notification submodule properly.

@olofhagsand
Copy link
Member

Actually, the warning may be misleading but it is not an error.
The yang of one of the augment nodes is:

    list notify-filter-profile {
      if-feature snmp:notification-filter;
      key name;

and the snmp:notification-filter feature is disabled. Disabled nodes are transformed to ANYDATA nodes internally and since you cannot augment with ANYDATA, (see RFC 7950 Sec 7.17) it causes this warning message.
I will change the warning message to be more descriptive and also keep track of disabled + anydata nodes and ensure that no warning is printed in this case.

@olofhagsand
Copy link
Member

Error message fixed and not printed in this case. Please verify.

@vvsosed vvsosed closed this as completed Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants