https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/OPNFLWPLUG-482 OpenFlowPlugin <p>In the package:</p> <p>org.opendaylight.openflowplugin.extension.vendor.nicira.convertor.match</p> <p>There's a TunIPv4SrcConvertor.java, but no corresponding TunIPv4DstConverter.java. I think this means that matches against the tunnel destination won't work.</p> <p>Operating System: All<br/> Platform: All</p> OPNFLWPLUG-482

Missing TunIPv4DstConverter

Bug Resolved Done Jozef Gloncak Thomas Bachman Wed, 3 Jun 2015 18:10:12 +0000 Mon, 27 Sep 2021 09:01:34 +0000 Wed, 10 Jun 2015 08:19:49 +0000 General 0 3 <p>Attachment screenshot.jpg has been added with description: screenshot of structure of package org.opendaylight.openflowplugin.extension.vendor.nicira.convertor.match</p> <p>I can see TunIpv4DstConverter see enclosed attachment.</p> <p>I'm a bit confused by the screen shot, as it differs from mine by quite a bit. I pulled the latest source, did a git checkout stable/lithium, and took a screenshot of what I have:</p> <p><a href="https://www.dropbox.com/s/ikoznbnuyt78l60/Screenshot%202015-06-05%2018.24.56.png?dl=0" class="external-link" target="_blank" rel="nofollow noopener">https://www.dropbox.com/s/ikoznbnuyt78l60/Screenshot%202015-06-05%2018.24.56.png?dl=0</a></p> <p>There are quite a few differences &#8211; note the NSH extensions present in mine, as well as the absence of the one you're referencing. Can you provide the git hash and branch of the source tree you're using?</p> <p>I was referencing stable/helium.<br/> E.g. hash f509d36bb7dfff637fe74f8bf820f48b8de015cc</p> <p>That's odd &#8211; is there a reason this was removed after Helium? Or was it just moved? If it was removed, I think it should be added back in.</p> <p>Thomas noted that this is not a blocker for GBP. This may be pushed to Beryllium if we run out of time.</p> <p>I prepared this patch<br/> <a href="https://git.opendaylight.org/gerrit/22149" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22149</a>.<br/> It adds missing class to package.</p> <p>Merged,<br/> please retest.</p> <p>Thomas can you, please, retest.</p> <p>I'm not going to be able to retest this for a while. If the UTs pass, then that's good enough for me.</p> <p>ok, so as Michal has written it had been merged. Once you will have time to test and you will see any problem with it please open new or reopen this one. Thanks.</p> Development External issue ID 3569 External issue URL Issue Type Priority Rank 0|i031fb: