<!-- 
RSS generated by JIRA (8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d) at Wed Feb 07 19:55:21 UTC 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>OpenDaylight JIRA</title>
    <link>https://jira.opendaylight.org</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>8.20.10</version>
        <build-number>820010</build-number>
        <build-date>22-06-2022</build-date>
    </build-info>


<item>
            <title>[CONTROLLER-1361] Restconf attempt to change TCPMD5 password for PCEP client fails silently</title>
                <link>https://jira.opendaylight.org/browse/CONTROLLER-1361</link>
                <project id="10113" key="CONTROLLER">controller</project>
                    <description>&lt;p&gt;This bug is critical for TCPMD5 usability.&lt;br/&gt;
This bug affects both RC0 and Snapshot Lithium builds.&lt;/p&gt;

&lt;p&gt;There may be semantic errors in test data, or internal errors in TCPMD5 od PCEP. But in any case, NETCONF/RESTCONF should detect the change was unsuccessful and it should return different HTTP status.&lt;/p&gt;

&lt;p&gt;Steps to reproduce, first alternative:&lt;br/&gt;
0. Download and unpack recent Lithium build.&lt;br/&gt;
1. Start ODL.&lt;br/&gt;
2. feature:install odl-restconf odl-bgpcep-pcep-all odl-netconf-connector-all&lt;br/&gt;
3. Kill ODL.&lt;br/&gt;
Steps 1-3 are to create config files in etc/opendaylight/karaf.&lt;br/&gt;
4. Edit config files accordingly to &lt;a href=&quot;https://wiki.opendaylight.org/view/BGP_LS_PCEP:TCP_MD5_Guide#PCEP&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://wiki.opendaylight.org/view/BGP_LS_PCEP:TCP_MD5_Guide#PCEP&lt;/a&gt;&lt;br/&gt;
5. rm -rf data/ snapshots/ journal/ lock idmlight.db.mv.db&lt;br/&gt;
6. Start ODL again.&lt;br/&gt;
7. feature:install odl-restconf odl-bgpcep-pcep-all odl-netconf-connector-all&lt;br/&gt;
8. Check the current configuration (address=192.0.2.2, password=changeme):&lt;br/&gt;
curl -u &apos;admin:admin&apos; 127.0.0.1:8181/restconf/config/network-topology:network-topology/topology/topology-netconf/node/controller-config/yang-ext:mount/config:modules/module/odl-pcep-topology-provider-cfg:pcep-topology-provider/pcep-topology&lt;br/&gt;
9. Attempt to use RESTCONF PUT to change the configuration:&lt;br/&gt;
 curl -v -X PUT -u &apos;admin:admin&apos; -H &quot;Content-Type:application/xml&quot; -d &apos;&amp;lt;client xmlns=&quot;urn:opendaylight:params:xml:ns:yang:controller:pcep:topology:provider&quot;&amp;gt;&lt;br/&gt;
 &amp;lt;address xmlns=&quot;urn:opendaylight:params:xml:ns:yang:controller:pcep:topology:provider&quot;&amp;gt;192.0.2.2&amp;lt;/address&amp;gt;&lt;br/&gt;
 &amp;lt;password xmlns=&quot;urn:opendaylight:params:xml:ns:yang:controller:pcep:topology:provider&quot;&amp;gt;topsecret&amp;lt;/password&amp;gt;&lt;br/&gt;
&amp;lt;/client&amp;gt;&apos; 127.0.0.1:8181/restconf/config/network-topology:network-topology/topology/topology-netconf/node/controller-config/yang-ext:mount/config:modules/module/odl-pcep-topology-provider-cfg:pcep-topology-provider/pcep-topology/odl-pcep-topology-provider-cfg:client/192.0.2.2&lt;br/&gt;
10. Observe HTTP status to be 200.&lt;br/&gt;
11. Repeat step 9. See that the password was not changed.&lt;/p&gt;

&lt;p&gt;Second alternative: In step 4, edit only 20-tcpmd5.xml and 32-pcep.xml, step 8 shows no client configuration, step 10 is still 200, step 11 shows still no client configuration.&lt;/p&gt;

&lt;p&gt;Third alternative: As in second, but use POST in step 9 (URI two levels shorter, XML data the same). Step 10 even reports new location, but no client configuration is created.&lt;/p&gt;

&lt;p&gt;Fourth (and fifth) alternative: Do not edit files before step 6, use restconf to add the changes between steps 7 and 8.&lt;/p&gt;

&lt;p&gt;In either case, there is nothing logged at INFO level, and it is not clear to me what is wrong when looking at the long DEBUG level log.&lt;/p&gt;</description>
                <environment>&lt;p&gt;Operating System: All&lt;br/&gt;
Platform: All&lt;/p&gt;</environment>
        <key id="25915">CONTROLLER-1361</key>
            <summary>Restconf attempt to change TCPMD5 password for PCEP client fails silently</summary>
                <type id="10104" iconUrl="https://jira.opendaylight.org/secure/viewavatar?size=xsmall&amp;avatarId=10303&amp;avatarType=issuetype">Bug</type>
                                                <status id="5" iconUrl="https://jira.opendaylight.org/images/icons/statuses/resolved.png" description="A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.">Resolved</status>
                    <statusCategory id="3" key="done" colorName="green"/>
                                    <resolution id="10000">Done</resolution>
                                        <assignee username="mmarsale@cisco.com">Maros Marsalek</assignee>
                                    <reporter username="vrpolak">Vratko Polak</reporter>
                        <labels>
                    </labels>
                <created>Mon, 8 Jun 2015 13:19:54 +0000</created>
                <updated>Wed, 17 Jun 2015 14:57:16 +0000</updated>
                            <resolved>Wed, 17 Jun 2015 14:57:16 +0000</resolved>
                                    <version>Lithium</version>
                                                    <component>netconf</component>
                        <due></due>
                            <votes>0</votes>
                                    <watches>4</watches>
                                                                                                                <comments>
                            <comment id="50806" author="vrpolak" created="Mon, 8 Jun 2015 13:49:52 +0000"  >&lt;p&gt;This time I made sure to delete etc/opendaylight/current/controller.currentconfig.xml&lt;/p&gt;</comment>
                            <comment id="50814" author="vrpolak" created="Mon, 8 Jun 2015 13:49:52 +0000"  >&lt;p&gt;Attachment log_20150608.tar.xz has been added with description: archive with TRACE logs for PUT operation&lt;/p&gt;</comment>
                            <comment id="50807" author="mmarsale@cisco.com" created="Wed, 10 Jun 2015 08:47:16 +0000"  >&lt;p&gt;The problem is in the config-netconf-connector component. It does not handle netconf operations for nested configuration DTOs properly.&lt;/p&gt;

&lt;p&gt;Needs to be fixed, but in the meantime you can use a workaround: Putting the entire configuration for module, with changed client container. Instead of putting just the client container.&lt;/p&gt;</comment>
                            <comment id="50808" author="mmarsale@cisco.com" created="Mon, 15 Jun 2015 08:05:31 +0000"  >&lt;p&gt;So far, there was no support for netconf operations apart from merge on nested containers and lists etc. in the netconf-&amp;gt;config subsystem pipeline. So updating configuration relied on using merge operation (POST from RESTCONF used to perform merge) but since the RESTCONF change POST behavior, PUT is the only option. &lt;/p&gt;

&lt;p&gt;Need to add support for nested netconf operations (at least replace to make PUT work from RESTCONF).&lt;/p&gt;</comment>
                            <comment id="50809" author="vrpolak" created="Mon, 15 Jun 2015 15:43:09 +0000"  >&lt;p&gt;&amp;gt; Putting the entire configuration for module, with changed client container.&lt;/p&gt;

&lt;p&gt;I have verified this workaround works indeed.&lt;br/&gt;
Still, PUT on sub-element would more user friendly than GET, local merge and PUT on parent container.&lt;/p&gt;</comment>
                            <comment id="50810" author="mmarsale@cisco.com" created="Tue, 16 Jun 2015 11:57:56 +0000"  >&lt;p&gt;Agree and heres the fix:&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://git.opendaylight.org/gerrit/#/c/22687/&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://git.opendaylight.org/gerrit/#/c/22687/&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;But theres a catch, this works fine for nested containers, but the client attribute in pcep-topology-provider is a list and there is no support for replacing/merging specific elements in a list. So the replace operation(PUT from RESTCONF) replaces the entire list, which works fine for when theres only one client element. For other cases, the workaround has to be used.&lt;/p&gt;

&lt;p&gt;I think we can add this support (full support for nested attributes, especially lists), but it would have to be targeted for Berylium.&lt;/p&gt;</comment>
                            <comment id="50811" author="mmarsale@cisco.com" created="Tue, 16 Jun 2015 12:01:10 +0000"  >&lt;p&gt;One note: The replace of entire list when changing just one entry is also present when using merge operation. So the POST from RESTCONF on a list attribute (that used to work a while back) would also replace the entire list (list of dependencies was a special case).&lt;/p&gt;</comment>
                            <comment id="50812" author="vrpolak" created="Tue, 16 Jun 2015 14:25:00 +0000"  >&lt;p&gt;&amp;gt; So the replace operation(PUT from RESTCONF) replaces the entire list, which works fine for when theres only one client element. For other cases, the workaround has to be used.&lt;/p&gt;

&lt;p&gt;Good enough for Lithium.&lt;/p&gt;

&lt;p&gt;&amp;gt; I think we can add this support (full support for nested attributes, especially lists), but it would have to be targeted for Berylium.&lt;/p&gt;

&lt;p&gt;Is this for config subsystem, or restconf in general?&lt;br/&gt;
Is such an enhancement request already created in Bugzilla?&lt;/p&gt;</comment>
                            <comment id="50813" author="mmarsale@cisco.com" created="Tue, 16 Jun 2015 14:34:54 +0000"  >&lt;p&gt;I have already reported an enhancement bug for this:&lt;br/&gt;
&lt;a href=&quot;https://bugs.opendaylight.org/show_bug.cgi?id=3753&quot; class=&quot;external-link&quot; target=&quot;_blank&quot; rel=&quot;nofollow noopener&quot;&gt;https://bugs.opendaylight.org/show_bug.cgi?id=3753&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;I have set config component for it, but it is an issue of the mapping component between netconf &amp;lt;-&amp;gt; config subsystem&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                            <attachment id="13521" name="log_20150608.tar.xz" size="41428" author="vrpolak" created="Mon, 8 Jun 2015 13:49:52 +0000"/>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                                            <customfield id="customfield_11400" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                        <customfield id="customfield_10208" key="com.atlassian.jira.plugin.system.customfieldtypes:textfield">
                        <customfieldname>External issue ID</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>3625</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                <customfield id="customfield_10201" key="com.atlassian.jira.plugin.system.customfieldtypes:url">
                        <customfieldname>External issue URL</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue><![CDATA[https://bugs.opendaylight.org/show_bug.cgi?id=3625]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                            <customfield id="customfield_10206" key="com.atlassian.jira.plugin.system.customfieldtypes:select">
                        <customfieldname>Issue Type</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue key="10300"><![CDATA[Bug]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                                                                                                                                                                        <customfield id="customfield_10204" key="com.atlassian.jira.plugin.system.customfieldtypes:select">
                        <customfieldname>ODL SR Target Milestone</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue key="10377"><![CDATA[Lithium-RC2]]></customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                    <customfield id="customfield_10000" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0|i02q3j:</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                </customfields>
    </item>
</channel>
</rss>