Fix identityref namespace binding in config-path-info XML

[yamontv]

Bind YANG on CLI edit-config XML before sending RPC and add missing xmlns for identityref values. Extend identityref namespace helper to cover leaf-lists and allow yspec fallback.

[olofhagsand]

I am unsure, did you do this patch after the new series of commits that move yang-dependent functionality from the cli client to the backend I did friday, see eg 046886a ?

With these commits, the YANG is not present in the client and the identityref adds should be done by the backend.
With those I am not sure this bind + apply is needed in cli_dbxml(), instead the call is in the previous rpc in from_client_config_path_info().

However, I see the patch in identityref_add_ns() is needed to run the new openconfig test case.

If I am right then with my new commits on friday, it should be enough with the identityref_add_ns patch and not the cli_dbxml patch to run your openconfig testcase.

Does that make sense?

[yamontv]

I just retested on the latest master my test case fails without my changes

[yamontv]

The patch you referenced doesn’t touch the edit‑config payload, so the prefix is still missing when validation runs. That’s why the CLI‑side bind+apply is effective (and also why a backend‑side fix must be placed in the edit‑config processing path, not in config‑path‑info). Do you strongly prefer backend-only fix?

[yamontv]

Binding YANG on CLI indeed is not needed.

[olofhagsand]

Can you please make the test executable?
Also, I think a validate would make the test more complete (before) commit, so it exercises the identityref validation code
Further, it would be nice to have a RESTCONF test as well

[yamontv]

Sorry, my bad again with +x. Added validate and RESTCONF tests