Merge pull request #15120 from opensourcerouting/fix/memory_leak_for_default_originate

bgpd: Fix memory leak for default-originate with route-map

Merge pull request #15106 from opensourcerouting/fix/rename_thread_cli

vtysh: Add `show event ...` commands

bgpd: Fix memory leak for default-originate with route-map

```
Direct leak of 40 byte(s) in 1 object(s) allocated from:
    0 0x7fc4b81eed28 in __interceptor_calloc (/usr/lib/x86_64-linux-gnu/libasan.so.4+0xded28)
    1 0x7fc4b7bd60bb in qcalloc lib/memory.c:105
    2 0x56221dc19207 in aspath_dup bgpd/bgp_aspath.c:689
    3 0x56221daacd42 in route_set_aspath_prepend bgpd/bgp_routemap.c:2283
    4 0x7fc4b7c3891a in route_map_apply_ext lib/routemap.c:2687
    5 0x56221dace552 in subgroup_default_originate bgpd/bgp_updgrp_adv.c:906
    6 0x56221dabf79c in update_group_default_originate_route_map_walkcb bgpd/bgp_updgrp.c:2105
    7 0x56221dabde4e in update_group_walkcb bgpd/bgp_updgrp.c:1721
    8 0x7fc4b7b9d398 in hash_walk lib/hash.c:270
    9 0x56221dac94cb in update_group_af_walk bgpd/bgp_updgrp.c:2062
    10 0x56221dac9b0f in update_group_walk bgpd/bgp_updgrp.c:2071
    11 0x56221dac9fd5 in update_group_refresh_default_originate_route_map bgpd/bgp_updgrp.c:2118
    12 0x7fc4b7c7fc54 in event_call lib/event.c:1974
    13 0x7fc4b7bb9276 in frr_run lib/libfrr.c:1214
    14 0x56221d9217fd in main bgpd/bgp_main.c:510
    15 0x7fc4b6bf2c86 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x21c86)
```

tmp_pi.attr should be flushed since it's already interned (new_attr) or the
origin value is used (attr).

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #15119 from donaldsharp/pytest_mgmtd

tests: Add mgmtd to the official pytest.mark list

Merge pull request #15111 from donaldsharp/json_vpn_wait_what

bgpd: show_adj_route_vpn always leaked json memory

tests: Add mgmtd to the official pytest.mark list

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Merge pull request #15110 from donaldsharp/revert_zrouter.stale_client

Revert "zebra: Cleanup zrouter.stale_client_list on shutdown"

bgpd: show_adj_route_vpn always leaked json memory

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Merge pull request #15101 from LabNConsulting/chopps/new-libyang-req

build: require libyang 2.1.128

Revert "zebra: Cleanup zrouter.stale_client_list on shutdown"

This reverts commit 71f7ecb571cd8a87c97ae952db1f1fafa7ef627a.

Merge pull request #15102 from opensourcerouting/Revert-memleak-PR14811

Revert "bgpd: On shutdown do not create a workqueue for the self peer"

Merge pull request #15104 from donaldsharp/label_pool_manager

bgpd: bgp_sync_label_manager failure case

doc: Rename `Thread` to `Event` in events example

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

vtysh: Rename `show thread ...` to `show event ...`

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Rename `show thread ...` to `show event ...`

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tools: Rename `show thread ...` to `show event ...`

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

doc: Rename `show thread ...` to `show event ...`

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #15105 from donaldsharp/event_thread_mishap

lib: Use event name instead of thread

lib: Use event name instead of thread

The thread system has been renamed, let's use the proper
terminology now in the show commands.  Also realign
output a tiny bit for stuff that was missing.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

bgpd: bgp_sync_label_manager failure case

There are several problems with the bgp_sync_label_manager
function:

a) It is possible that a request in the lp->requests
fifo will be unable to be filled at this point in time
and the lf will be leaked and not ever fullfilled.

b) The bgp_sync_label_manager runs one time a second
irrelevant if there is work to do or not.

To fix (a) just add the request back to the requests
fifo and set the timer to pop in the future.

To fix (b) just every time something is put into
the request pool start a timer to run in 1 second
and do not restart it if all the work is done.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Revert "bgpd: On shutdown do not create a workqueue for the self peer"

This reverts commit 7bf3c2fb195f34382e1bf00ed2c91310a1dc9f86.
Commit reverted as it introduces a memoery leak during the tests

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

build: require libyang 2.1.128

Update building docs to reflect this as well.

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #15054 from k0ste/help

redhat: rpm spec: added protobuf-c to deps & fix bogus changelog date

Merge pull request #15092 from donaldsharp/lib_zebra_h_cleanup

Lib zebra h cleanup

Merge pull request #15085 from LabNConsulting/chopps/mgmtd-doc-update

doc: add diagram of oper-state queries.

Merge pull request #15087 from cscarpitta/fix/fix-cid-1575079

zebra: Fix coverity issue in `dplane_srv6_encap_srcaddr_set`

Merge pull request #15089 from pguibert6WIND/asan_sharpd_fix

lib: fix memory leak when disabling nexthop_hold contexts

Merge pull request #15091 from louis-6wind/fix-ecommunity_fill_pbr_action

bgpd: fix ecommunity_fill_pbr_action heap-buffer-overflow

Merge pull request #15097 from louis-6wind/cleanup_vrf_update

lib: remove zebra_interface_vrf_update_read()

Merge pull request #15086 from LabNConsulting/chopps/remove-old-oper-iter-code

lib: remove unused/replaced oper-state iteration code

Merge pull request #15095 from LabNConsulting/chopps/fix-centos-warn

lib: fix initializer warning on centos 7.

lib: remove zebra_interface_vrf_update_read()

zebra_interface_vrf_update_read is orphan. Remove it.

Fixes: b580c52698 ("*: remove ZEBRA_INTERFACE_VRF_UPDATE")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

lib: fix initializer warning on centos 7.

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: sys/utsname.h is not needed in zebra.h

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib: zebra.h does not need to include syslog.h

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

*: sys/uio.h does not need to be in zebra.h

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib, sharpd: Move sys/resource.h to where it is used

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

*: Remove sys/ioctl.h from zebra.h

Practically no-one uses this and ioctls are pretty much
wrappered.  Further wrappering could make this even better.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib, vtysh: Remove inclusion of grp.h for everyone

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib: Remove in_systm.h it is never used

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

*: Remove cost of including netinet/icmp.h everywhere

This file inclusion is only used in 3 places move
the includes to those spots.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib: Remove inclusion of various MAXINT from zebra.h

All modern systems should have these defined.  Let's stop
paying the cost of this compilation.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib: HAVE_GLIBC_BACKTRACE does not belong in zebra.h

The backtrace functionality has been abstracted over
to zlog_backtrace().  Now that every place uses this
move the inclusion for HAVE_GLIBC_BACKTRACE into
the appropriate files instead of having everyone
pay for this costly include.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

bgpd: rfapi was not using internal zlog_backtrace()

The rfapi code was not using the zlog_backtrace()
functionality.  Let's just convert over to using
the proper functionality that we have built in now.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib: Move some priviledge headers to file they are used.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

*: Remove Crypto openSSL define from zebra.h

Only a couple of places use this.  Move these to a better
spot.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

lib: Breakout sendmmsg into it's own header

The only 2 places sendmmsg is used is in zlog_5424.c
and zlog_live.c.  Why is the rest of the entire system
paying for this compilation?

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

Merge pull request #15083 from louis-6wind/fix-isis_spftree_del

isisd: fix _isis_spftree_del heap-use-after-free

bgpd: fix ecommunity_fill_pbr_action heap-buffer-overflow

Fix the following heap-buffer-overflow:

> ==3901635==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x6020003a5940 at pc 0x56260067bb48 bp 0x7ffe8a4f3840 sp 0x7ffe8a4f3838
> READ of size 4 at 0x6020003a5940 thread T0
>     #0 0x56260067bb47 in ecommunity_fill_pbr_action bgpd/bgp_ecommunity.c:1587
>     #1 0x5626007a246e in bgp_pbr_build_and_validate_entry bgpd/bgp_pbr.c:939
>     #2 0x5626007b25e6 in bgp_pbr_update_entry bgpd/bgp_pbr.c:2933
>     #3 0x562600909d18 in bgp_zebra_announce bgpd/bgp_zebra.c:1351
>     #4 0x5626007d5efd in bgp_process_main_one bgpd/bgp_route.c:3528
>     #5 0x5626007d6b43 in bgp_process_wq bgpd/bgp_route.c:3641
>     #6 0x7f450f34c2cc in work_queue_run lib/workqueue.c:266
>     #7 0x7f450f327a27 in event_call lib/event.c:1970
>     #8 0x7f450f21a637 in frr_run lib/libfrr.c:1213
>     #9 0x56260062fc04 in main bgpd/bgp_main.c:540
>     #10 0x7f450ee2dd09 in __libc_start_main ../csu/libc-start.c:308
>     #11 0x56260062ca29 in _start (/usr/lib/frr/bgpd+0x2e3a29)
>
> 0x6020003a5940 is located 0 bytes to the right of 16-byte region [0x6020003a5930,0x6020003a5940)
> allocated by thread T0 here:
>     #0 0x7f450f6aa1f8 in __interceptor_realloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:164
>     #1 0x7f450f244f8a in qrealloc lib/memory.c:112
>     #2 0x562600673313 in ecommunity_add_val_internal bgpd/bgp_ecommunity.c:143
>     #3 0x5626006735bc in ecommunity_uniq_sort_internal bgpd/bgp_ecommunity.c:193
>     #4 0x5626006737e3 in ecommunity_parse_internal bgpd/bgp_ecommunity.c:228
>     #5 0x562600673890 in ecommunity_parse bgpd/bgp_ecommunity.c:236
>     #6 0x562600640469 in bgp_attr_ext_communities bgpd/bgp_attr.c:2674
>     #7 0x562600646eb3 in bgp_attr_parse bgpd/bgp_attr.c:3893
>     #8 0x562600791b7e in bgp_update_receive bgpd/bgp_packet.c:2141
>     #9 0x56260079ba6b in bgp_process_packet bgpd/bgp_packet.c:3406
>     #10 0x7f450f327a27 in event_call lib/event.c:1970
>     #11 0x7f450f21a637 in frr_run lib/libfrr.c:1213
>     #12 0x56260062fc04 in main bgpd/bgp_main.c:540
>     #13 0x7f450ee2dd09 in __libc_start_main ../csu/libc-start.c:308

Fixes: dacf6ec120 ("bgpd: utility routine to convert flowspec actions into pbr actions")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

lib: fix memory leak when disabling nexthop_hold contexts

A memory leak is detected when stopping the sharpd daemon
with a nexthop group configuration that includes nexthops.

The nexthop_hold structure and its attributes are not
freed. Fix it by adding the missing free function.

Fixes: 98cbbaea91f6 ("lib: Handle if up/down and vrf enable/disable events")
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

zebra: `ctx` has to be non NULL at this point

Fix the following coverity issue:

*** CID 1575079:  Null pointer dereferences  (REVERSE_INULL)
/zebra/zebra_dplane.c: 5950 in dplane_srv6_encap_srcaddr_set()
5944      if (ret == AOK)
5945      result = ZEBRA_DPLANE_REQUEST_QUEUED;
5946      else {
5947      atomic_fetch_add_explicit(&zdplane_info
5948         .dg_srv6_encap_srcaddr_set_errors,
5949        1, memory_order_relaxed);
     CID 1575079:  Null pointer dereferences  (REVERSE_INULL)
     Null-checking "ctx" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
5950      if (ctx)
5951      dplane_ctx_free(&ctx);
5952      }
5953      return result;
5954     }
5955

Remove the pointer check for `ctx`. At this point in the
function it has to be non null since we deref'ed it.
Additionally the alloc function that creates it cannot
fail.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

lib: remove unused/replaced oper-state iteration code

Signed-off-by: Christian Hopps <chopps@labn.net>

doc: add diagram of oper-state queries.

Signed-off-by: Christian Hopps <chopps@labn.net>

isisd: fix _isis_spftree_del heap-use-after-free

Fix the following heap-use-after-free

> ==82961==ERROR: AddressSanitizer: heap-use-after-free on address 0x6020001e4750 at pc 0x55a8cc7f63ac bp 0x7ffd6948e340 sp 0x7ffd6948e330
> READ of size 8 at 0x6020001e4750 thread T0
>     #0 0x55a8cc7f63ab in isis_route_node_cleanup isisd/isis_route.c:335
>     #1 0x7ff25ec617c1 in route_node_free lib/table.c:75
>     #2 0x7ff25ec619fc in route_table_free lib/table.c:111
>     #3 0x7ff25ec61661 in route_table_finish lib/table.c:46
>     #4 0x55a8cc800d83 in _isis_spftree_del isisd/isis_spf.c:397
>     #5 0x55a8cc800e45 in isis_spftree_clear isisd/isis_spf.c:414
>     #6 0x55a8cc80bd9a in isis_run_spf isisd/isis_spf.c:2020
>     #7 0x55a8cc80c370 in isis_run_spf_with_protection isisd/isis_spf.c:2076
>     #8 0x55a8cc80cf52 in isis_run_spf_cb isisd/isis_spf.c:2165
>     #9 0x7ff25ec7c4dc in event_call lib/event.c:1970
>     #10 0x7ff25eb64423 in frr_run lib/libfrr.c:1213
>     #11 0x55a8cc7799da in main isisd/isis_main.c:318
>     #12 0x7ff25e623d8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58
>     #13 0x7ff25e623e3f in __libc_start_main_impl ../csu/libc-start.c:392
>     #14 0x55a8cc778e44 in _start (/usr/lib/frr/isisd+0x109e44)
>
> 0x6020001e4750 is located 0 bytes inside of 16-byte region [0x6020001e4750,0x6020001e4760)
> freed by thread T0 here:
>     #0 0x7ff25f000537 in __interceptor_free ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:127
>     #1 0x7ff25eb9012e in qfree lib/memory.c:130
>     #2 0x55a8cc7f6485 in isis_route_table_info_free isisd/isis_route.c:351
>     #3 0x55a8cc800cf4 in _isis_spftree_del isisd/isis_spf.c:395
>     #4 0x55a8cc800e45 in isis_spftree_clear isisd/isis_spf.c:414
>     #5 0x55a8cc80bd9a in isis_run_spf isisd/isis_spf.c:2020
>     #6 0x55a8cc80c370 in isis_run_spf_with_protection isisd/isis_spf.c:2076
>     #7 0x55a8cc80cf52 in isis_run_spf_cb isisd/isis_spf.c:2165
>     #8 0x7ff25ec7c4dc in event_call lib/event.c:1970
>     #9 0x7ff25eb64423 in frr_run lib/libfrr.c:1213
>     #10 0x55a8cc7799da in main isisd/isis_main.c:318
>     #11 0x7ff25e623d8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58
>
> previously allocated by thread T0 here:
>     #0 0x7ff25f000a57 in __interceptor_calloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:154
>     #1 0x7ff25eb8ffdc in qcalloc lib/memory.c:105
>     #2 0x55a8cc7f63eb in isis_route_table_info_alloc isisd/isis_route.c:343
>     #3 0x55a8cc80052a in _isis_spftree_init isisd/isis_spf.c:334
>     #4 0x55a8cc800e51 in isis_spftree_clear isisd/isis_spf.c:415
>     #5 0x55a8cc80bd9a in isis_run_spf isisd/isis_spf.c:2020
>     #6 0x55a8cc80c370 in isis_run_spf_with_protection isisd/isis_spf.c:2076
>     #7 0x55a8cc80cf52 in isis_run_spf_cb isisd/isis_spf.c:2165
>     #8 0x7ff25ec7c4dc in event_call lib/event.c:1970
>     #9 0x7ff25eb64423 in frr_run lib/libfrr.c:1213
>     #10 0x55a8cc7799da in main isisd/isis_main.c:318
>     #11 0x7ff25e623d8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58

Fixes: 7153c3cabf ("isisd: update struct isis_route_info has multiple sr info by algorithm")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

Merge pull request #15074 from chiragshah6/fdev4

bgpd: unimport evpn routes when implicit withdraw

Merge pull request #15055 from opensourcerouting/fix/zebra_ipv6_ll_configured

zebra: Push all configured IP addresses when the interface comes up

Merge pull request #15062 from opensourcerouting/fix/mark_as_failed_if_do_not_merge_labeled

github: Mark the build as failed if 'do not merge' label is set

Merge pull request #12261 from cscarpitta/srv6-encap-src-addr

zebra: Add the support of the Source Addr param of the SRv6 Encapsulation

Merge pull request #15070 from LabNConsulting/chopps/fix-coverity-errors

chopps/fix coverity errors

Merge pull request #15071 from LabNConsulting/chopps/building-cleanup

build: protobuf is required so update building docs

Merge pull request #15073 from LabNConsulting/chopps/fix-yang-key-queries

lib: fix specific entry queries

Merge pull request #15076 from LabNConsulting/chopps/libyang-no-schema-node-key-guarantee

lib: don't count on scheme key nodes being first children

Merge pull request #15075 from pguibert6WIND/pathd_metric_computed

pathd: add dynamic candidate path metric [computed] keyword

lib: don't count on scheme key nodes being first children

Heard back from libyang folks that this is not something they consider part of
the API and/or are going to guarantee. So we cannot count on it. Expect keys at
any location on the child sibling list.

Signed-off-by: Christian Hopps <chopps@labn.net>

pathd: add dynamic candidate path metric [computed] keyword

Add the 'computed' keyword for a given metric.

> [..]
> metric te computed

When set by the PCC, the PCE must send back a computed metric
value in the PCResponse value.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

lib: fix specific entry queries

- fix key leaf queries
- fix specific list entry with non-key leaf top element

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #15068 from chiragshah6/zdev

bgpd: lttng tp add ethtag to macip zebra send

Merge pull request #15066 from chiragshah6/mdev1

bgpd: revamp evpn debugs nexthop and l3vni

bgpd: unimport evpn routes when implicit withdraw

When bgp update is received for EVPN prefix
where for an existing path's nexthop becomes unreachable,
the path is marked as not VALID but the routes
were not unimported from tenant vrfs, which lead to
stale unicast route(s) and nexthop(s).

In Multipath scenario only a specific path may have marked as
not VALID, then specific path info for the EVPN prefix required to be
unimported from tenant vrf.

Ticket: #3671288

Signed-off-by: Chirag Shah <chirag@nvidia.com>
Signed-off-by: Donald Sharp <sharpd@nvidia.com>

build: protobuf is required so update building docs

- recommend using libyang 2.1.128 but accept 2.0.0 for now.
- update ubuntu building doc to add GRPC section
- update ubuntu reference docker builds to match grpc doc change

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: mgmtd: increase soft-max msg size to 64K

- remove unused mgmt client/server constants

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: fix coverity CID 1574981

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: fix coverity CID 1574980

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: fix coverity CID 1574979

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: fix coverity CID 1574978

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: fix coverity CID 1574977

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #14492 from LabNConsulting/oper-state

oper state

tests: add mgmt_oper topotest

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: client testing

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: fixes required after rebasing

lib: fix checkpatch warnings
lib: darr: disable check
mgmtd: fix initialization

Signed-off-by: Christian Hopps <chopps@labn.net>

zebra: fix cleanup of mgmt backend state

Signed-off-by: Christian Hopps <chopps@labn.net>

zebra: fix oper-state walk bug

Signed-off-by: Christian Hopps <chopps@labn.net>

zebra: support yielding between oper state routes query

Signed-off-by: Christian Hopps <chopps@labn.net>

zebra: add zebra to mgmtd oper-state

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: add dedicated API functions for native msgs

- reorg native message header

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: northbound: add yielding and batching to oper-state queries

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: darr: add ability to set MTYPE for dynamic arrays

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: yang: add yang_get_node_keys() function

- comment out unused mem-leaking function.
- improve yang_dnode_get_path return type

Signed-off-by: Christian Hopps <chopps@labn.net>

tools: add more libyang iter macros to .clang-format

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: create and use libyang tree during oper-state walk

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: northbound: improve xpath functionality

Allow user to leave keys off of a list entry node at the end of the xpath. This
will return all list entries. Previously there was no way to just get the list
entries. One had to leave off the last list entry node which would then return
all list nodes as well as all the siblings at the same level.

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: yang: add new functions

- yang_get_key_preds
- yang_lyd_new_list

A function like new_list was added recently to libyang,
this is a compat version.

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: darr: add new access and str functions

- darr_last(), and darr_strdup_cap().
- strcat, strdup, strlen, strnul equivs.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: add debugs and count in static_simple test

Also add option to use unified config.

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: step 6: remove old unfinished get-data code

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: step 5: add get-tree txn functionality

Adds the guts of the get-tree functionality that is called by or calls
the FE and BE code for get-tree processing.

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: step 4: FE adapter get-tree functionality

Signed-off-by: Christian Hopps <chopps@labn.net>

mgmtd: step 3: BE adapter native message handling

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: step 2: mgmtd: BE client code for get-tree functionality

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: step 1: mgmtd: add FE get-tree functionality

Signed-off-by: Christian Hopps <chopps@labn.net>