Thanks for the patch.  I made some minor tweaks to the commit message and merged it.<div><br></div><div>Ethan<br><br><div class="gmail_quote">On Wed, Jan 2, 2013 at 9:23 AM, Anoob Soman <span dir="ltr">&lt;<a href="mailto:anoob.soman@citrix.com" target="_blank">anoob.soman@citrix.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">While dumping lacp information using ovs-appctl, &quot;aggregation key&quot; field<br>
displays port_id even though aggregation-key is set using<br>
&quot;other-config:lacp-aggregation-key&quot;.<br>
<br>
Signed-off-by: Anoob Soman &lt;<a href="mailto:anoob.soman@citrix.com">anoob.soman@citrix.com</a>&gt;<br>
---<br>
 lib/lacp.c |    4 +++-<br>
 1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/lib/lacp.c b/lib/lacp.c<br>
index 374d915..96857ea 100644<br>
--- a/lib/lacp.c<br>
+++ b/lib/lacp.c<br>
@@ -750,7 +750,9 @@ lacp_print_details(struct ds *ds, struct lacp *lacp)<br>
     ds_put_format(ds, &quot;\tsys_priority: %u\n&quot;, lacp-&gt;sys_priority);<br>
     ds_put_cstr(ds, &quot;\taggregation key: &quot;);<br>
     if (lacp-&gt;key_slave) {<br>
-        ds_put_format(ds, &quot;%u&quot;, lacp-&gt;key_slave-&gt;port_id);<br>
+        ds_put_format(ds, &quot;%u&quot;, lacp-&gt;key_slave-&gt;key<br>
+                                ? lacp-&gt;key_slave-&gt;key<br>
+                                : lacp-&gt;key_slave-&gt;port_id);<br>
     } else {<br>
         ds_put_cstr(ds, &quot;none&quot;);<br>
     }<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.9.5<br>
<br>
_______________________________________________<br>
dev mailing list<br>
<a href="mailto:dev@openvswitch.org">dev@openvswitch.org</a><br>
<a href="http://openvswitch.org/mailman/listinfo/dev" target="_blank">http://openvswitch.org/mailman/listinfo/dev</a><br>
</font></span></blockquote></div><br></div>