Skip to content

Netconf service commit missing "persist-id" tag #796#797

Merged
1 commit merged intoCiscoDevNet:masterfrom
AntonAbik:796
Jun 6, 2018
Merged

Netconf service commit missing "persist-id" tag #796#797
1 commit merged intoCiscoDevNet:masterfrom
AntonAbik:796

Conversation

@AntonAbik
Copy link
Copy Markdown

@AntonAbik AntonAbik commented Jun 6, 2018

Tested with the simple fix and seems it fixed the issue and now persist-id tag is added to the confirmed commit:
2018-06-06 13:52:47,723 - ydk - DEBUG - Trace: Missing message-id in rpc.
2018-06-06 13:52:47,723 - ydk - DEBUG - Netconf SSH Client: sending rpc
2018-06-06 13:52:47,724 - ydk - DEBUG - Trace: Writing message (session 2711111191):

<?xml version="1.0"?>
<rpc xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" message-id="5">
  <commit>
    <persist-id>1234</persist-id>
  </commit>
</rpc>

2018-06-06 13:52:47,724 - ydk - DEBUG - Netconf SSH Client: receiving rpc
2018-06-06 13:52:48,170 - ydk - DEBUG - Trace: Received message (session 2711

<rpc-reply message-id="5" xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
 <ok/>
</rpc-reply>

2018-06-06 13:52:48,170 - ydk - DEBUG - Netconf SSH Client: processing reply
2018-06-06 13:52:48,171 - ydk - INFO - =============Reply payload received fr
2018-06-06 13:52:48,171 - ydk - INFO -

<?xml version="1.0"?>
<rpc-reply xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" message-id="5">
  <ok/>
</rpc-reply>

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @AntonAbik for this fix!

@ghost ghost merged commit dd0c84c into CiscoDevNet:master Jun 6, 2018
@AntonAbik AntonAbik deleted the 796 branch June 9, 2018 21:01
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant