Placeholder for 3.2.0.0 minor fixes and improvements to control node

[Nischal Sheth]

Placeholder for 3.2.0.0 minor fixes and improvements to control node.

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/21831
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/21831
Committed: http://github.org/Juniper/contrail-controller/commit/25182dadbd5e94005d8ee26a77455cb070b697a7
Submitter: Zuul
Branch: master

commit 25182dadbd5e94005d8ee26a77455cb070b697a7
Author: Nischal Sheth <email address hidden>
Date: Mon Jul 11 22:04:40 2016 -0700

Make sm_policy symmetric

Change-Id: I2287a0178ae339bde026d297b3dd6692bc1f23c9
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/22784
Submitter: Ananth Suryanarayana (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/22784
Committed: http://github.org/Juniper/contrail-controller/commit/335b84a80d119e095e246298894be6013d4575be
Submitter: Zuul
Branch: master

commit 335b84a80d119e095e246298894be6013d4575be
Author: Ananth Suryanarayana <email address hidden>
Date: Tue Aug 2 17:40:33 2016 -0700

Fix cpplint errors in src/io

All production C++ code under src/io pass cleanly with these changes

Change-Id: I7fb24437dd502d7ef36f82d5b5a12dfdb4ab901e
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/23270
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/23270
Committed: http://github.org/Juniper/contrail-controller/commit/3312684eac3e8afdc5cc15ec1cda92cc19ace5f1
Submitter: Zuul
Branch: master

commit 3312684eac3e8afdc5cc15ec1cda92cc19ace5f1
Author: Nischal Sheth <email address hidden>
Date: Fri Aug 12 17:16:20 2016 -0700

Use const iterator to traverse vectors

Change-Id: If0024876d023775d0a0462dfa2fc6fc88fb0dc19
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24185
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/24185
Committed: http://github.org/Juniper/contrail-controller/commit/d7e11785e0a11fb5f4c59b589c29886748383673
Submitter: Zuul
Branch: master

commit d7e11785e0a11fb5f4c59b589c29886748383673
Author: Nischal Sheth <email address hidden>
Date: Thu Sep 15 11:11:33 2016 -0700

Reuse ifmap xmpp message string for tracing

Pass a reference to std:string to SendUpdate so that it can be passed
on to XmppChannel::Send and then on to XmppConnection::Send, which can
now use the provided string (changed via bug number 1607132) instead
of constructing a new one from the data pointer.

Change-Id: If94a1cf63ab07a9ba9703bc5e54a6e177835217b
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24213
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/24213
Committed: http://github.org/Juniper/contrail-controller/commit/ffd723e4b991064f406d7e2644d67f8f76a193f2
Submitter: Zuul
Branch: master

commit ffd723e4b991064f406d7e2644d67f8f76a193f2
Author: Nischal Sheth <email address hidden>
Date: Fri Sep 16 09:09:36 2016 -0700

Reduce memory allocation overhead in update generation code

Note that pugixml library allocates memory for a document in increments
of 32KB pages and then manages smaller allocations (nodes or attributes)
using these pages. Calling the reset method on a document frees all the
pages. Hence a 32B page allocation/free happens for each config message
sent to a client. Instead, removing the only child node of the document
ensures that pugixml does only a single 32KB allocation per document and
recycles the same memory when building the tree for each message.

Change-Id: Id56d5ad6c45fc2b6bedfe402d4758b454efc183e
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24471
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/24471
Committed: http://github.org/Juniper/contrail-controller/commit/f491a90939dc050d78b66bce2ca35db222bd4b02
Submitter: Zuul
Branch: master

commit f491a90939dc050d78b66bce2ca35db222bd4b02
Author: Nischal Sheth <email address hidden>
Date: Mon Sep 26 21:20:16 2016 -0700

Include bgp_ribout.h as the first file in bgp_ribout.cc

Change-Id: I3da6feeb254607d835e308497a2f093d9a284f42
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24566
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24798
Submitter: Ananth Suryanarayana (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/24798
Committed: http://github.org/Juniper/contrail-controller/commit/88369ba6ebd585f5e4b3c61163adbc4a42f24ecb
Submitter: Zuul
Branch: master

commit 88369ba6ebd585f5e4b3c61163adbc4a42f24ecb
Author: Ananth Suryanarayana <email address hidden>
Date: Tue Oct 11 11:56:20 2016 -0700

Convert pointer to bool correctly

Also remove default in the switch and cover all cases explicitly

Change-Id: I0fdb4d440bb4067a35519ea2af1cd75cc52ec242
Partial-Bug: #1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/24908
Submitter: Ananth Suryanarayana (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/25173
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/25189
Submitter: Nischal Sheth (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/25189
Committed: http://github.org/Juniper/contrail-controller/commit/905eeeef0ff249e6390c69726f20fc98433e9e62
Submitter: Zuul
Branch: R3.2

commit 905eeeef0ff249e6390c69726f20fc98433e9e62
Author: Nischal Sheth <email address hidden>
Date: Wed Oct 19 15:06:12 2016 -0700

Make bgp_peer_test more stable

This test used to fail occasionally when the StateMachine tried to
delete the mock TcpSession which would have an invalid refcount 0.

Fix by using a mock StateMachine which overrides the DeleteSession
method.

Change-Id: Ibf2654477b4f6d995d8be757ac636dee4a4901b1
Partial-Bug: 1602347

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/25241
Submitter: Ananth Suryanarayana (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/24908
Committed: http://github.org/Juniper/contrail-controller/commit/f13c92e65ddf5b13fac79bcc12b9e6e3cb2142eb
Submitter: Zuul
Branch: master

commit f13c92e65ddf5b13fac79bcc12b9e6e3cb2142eb
Author: Ananth Suryanarayana <email address hidden>
Date: Fri Oct 14 12:26:20 2016 -0700

Remove unsupported families from Address::Family enumeration

Also remove them from table stats sandesh structures

Change-Id: Iba6234518f4606ca04abdaa4227950c3f07aca42
Partial-Bug: #1602347

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/25241
Committed: http://github.org/Juniper/contrail-controller/commit/1f4fbf8045f9bd7e251f8d61b2a288e06d691ae9
Submitter: Zuul
Branch: R3.2

commit 1f4fbf8045f9bd7e251f8d61b2a288e06d691ae9
Author: Ananth Suryanarayana <email address hidden>
Date: Fri Oct 14 12:26:20 2016 -0700

Remove unsupported families from Address::Family enumeration

Also remove them from table stats sandesh structures

Change-Id: Iba6234518f4606ca04abdaa4227950c3f07aca42
Partial-Bug: #1602347