Using local notifications and nc-notifications if not found on NETCONF Server #497
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
esmasth
force-pushed
the
esmasth/issue492
branch
2 times, most recently
from
f27ad3f
to
eae0c05
Compare
esmasth
force-pushed
the
esmasth/issue492
branch
2 times, most recently
from
6d52b45
to
c5e88eb
Compare
michalvasko
approved these changes
Jul 25, 2024
michalvasko
requested changes
Jul 25, 2024
esmasth
force-pushed
the
esmasth/issue492
branch
from
c5e88eb
to
ee9c437
Compare
michalvasko
requested changes
Jul 26, 2024
If NETCONF server supports RFC5277 notification capability and libnetconf2 required notifications and nc-notifications are not present on the NETCONF server (which it is not obligated to support), then the libyang context needs to be initialized using client side local YANG schema files. This change assumes that notifications and nc-notifications models are present in searchpath directory set via nc_client_set_schema_searchpath or in the default YANG_MODULE_DIR set via build time configuration. Signed-off-by: Siddharth Sharma <[email protected]>
esmasth
force-pushed
the
esmasth/issue492
branch
from
ee9c437
to
bcc859f
Compare
michalvasko
approved these changes
Jul 26, 2024
|
Thanks @michalvasko! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If NETCONF server supports RFC5277 notification capability and
libnetconf2 required notifications and nc-notifications are not present
on the NETCONF server (which it is not obligated to support), then the
libyang context needs to be initialized using client side local YANG
schema files.
This change assumes that notifications and nc-notifications models are
present in searchpath directory set via nc_client_set_schema_searchpath
or in the default YANG_MODULE_DIR set via build time configuration.
This addresses #492