One failed module load silently discards all previously loaded modules

[catalinvlad192]

Hi!

Version: 4.4.10

nc_ctx_check_and_fill() loads each module in a single pass (session_client.c:1212).
When one parse fails, all modules are removed, not just the one that failed. It never revisits earlier entries, so modules stay missing.
A server may advertise a module unknown to the client, which would prevent any other known module from loading. Shouldn't the client ignore unknown modules in this case and load the known ones regardless?

Thank you!

[michalvasko]

When one parse fails, all modules are removed

Please refer to a specific code or lines. I found that only fatal errors cause the context to not be properly compiled and nc_ctx_fill() prints only warnings when a YANG module fails to load.