Tags: facebook/openr
Tags
Updating submodules Summary: GitHub commits: facebook/fbthrift@a346734 facebook/proxygen@d7535e2 facebook/rocksdb@3a6d943 facebook/wangle@610fa46 facebookincubator/fizz@547bdd2 facebookincubator/katran@b122356 facebook/mvfst@a7e933e Reviewed By: zpao fbshipit-source-id: 7c3bc5d3e3e555c0545ce20f9df733c67526a331
Updating submodules Summary: GitHub commits: facebook/fbthrift@c10f5bf facebook/wangle@b6d8b72 facebookincubator/fizz@35f57f4 facebookincubator/katran@286e860 facebook/mvfst@8996c53 Reviewed By: yns88 fbshipit-source-id: b8cea63518aa662ae640c6d5e4a7c8c14cf0690f
Add support in OpenrCtrl server for longPollAdj API (25/N) Summary: As per discussion, Ebb will use longPoll API to mimick the server side "PUSH" behavior for any "adj:" key change. This diff contains: 1). thrift API definition; 2). server side handler logic to handle client requests and return when kvStore updates received; 3). lazy cleanup when there is KvStore publication happened. Cleanup will be based on timestamp. 4). UT to mimick client side receive/timeout cases; Reviewed By: saifhhasan Differential Revision: D17817739 fbshipit-source-id: 3d19134ed97ed0f8d648098a95902150455fd2b6
Remove the peer from latestSentPeerSync_ when the peer is Summary: Seeing the following in logs: Oct 04 16:28:59 e00.0001.node1 openr[113]: I1004 16:28:59.347575 141 KvStore.cpp:1323] Sync response received from e00.0033.node33::7::TCP::CMD::LOCAL with 0 key value pairs which incured 0 key-value updates Oct 04 16:29:12 e00.0001.node1 openr[113]: V1004 16:29:12.720877 121 NetlinkMessage.cpp:366] Asynchronous Neighbor Event: neighbor fe80::f8a6:3ff:fe90:b32f reachable Yes, intf-index 4, mac-addr fa:a6:03:90:b3:2f, state stale Oct 04 16:29:47 e00.0001.node1 openr[113]: I1004 16:29:47.337414 141 KvStore.cpp:1380] Requesting full-sync from e00.0033.node33 Oct 04 16:29:47 e00.0001.node1 openr[113]: I1004 16:29:47.349166 141 KvStore.cpp:836] 2 full dump sync in progress Oct 04 16:29:47 e00.0001.node1 openr[113]: I1004 16:29:47.352319 141 KvStore.cpp:849] Scheduling full sync after 10000ms. Oct 04 16:29:47 e00.0001.node1 openr[113]: I1004 16:29:47.362323 141 KvStore.cpp:1323] Sync response received from e00.0033.node33::7::TCP::CMD::LOCAL with 7 key value pairs which incured 2 key-value updates Oct 04 16:30:04 e00.0001.node1 openr[113]: V1004 16:30:04.432837 121 NetlinkMessage.cpp:366] Asynchronous Neighbor Event: neighbor fe80::f8a6:3ff:fe90:b32f reachable Yes, intf-index 4, mac-addr fa:a6:03:90:b3:2f, state probe Oct 04 16:30:04 e00.0001.node1 openr[113]: V1004 16:30:04.432895 121 NetlinkMessage.cpp:366] Asynchronous Neighbor Event: neighbor fe80::f8a6:3ff:fe90:b32f reachable Yes, intf-index 4, mac-addr fa:a6:03:90:b3:2f, state reachable Oct 04 16:30:35 e00.0001.node1 openr[113]: I1004 16:30:35.341146 141 KvStore.cpp:1380] Requesting full-sync from e00.0033.node33 Oct 04 16:30:35 e00.0001.node1 openr[113]: I1004 16:30:35.344977 141 KvStore.cpp:836] 2 full dump sync in progress Oct 04 16:30:35 e00.0001.node1 openr[113]: I1004 16:30:35.345535 141 KvStore.cpp:849] Scheduling full sync after 10000ms. This can happen when the peers for which the request is pending have gone. When deleting peers in delPeers() API also delete from latestSentPeerSync_. Reviewed By: saifhhasan Differential Revision: D17793566 fbshipit-source-id: 450f01bdf8c8e4d4d5e0768a237e10f69a81b8aa
Updating submodules Reviewed By: cdelahousse fbshipit-source-id: db358a6d032aeab5fc544133a729f74401061539
Attr to distinguish between LFA vs 2nd-shortest path Summary: Fib module forwards only smallest metric nexthops to FibAgent. This is inteded to not program LFA routes and keep them in backup. However in case of KSP2 we want both shortest and non-shortest paths to be programmed. Adding `useNonShortestRoute` attribute to nextHop to achieve the same. Nexthops that have this attribute set will always be programmed Reviewed By: jstrizich Differential Revision: D15728459 fbshipit-source-id: 46170e55ce8906fc3d73a44d3a646535f6ae319a
open source BaseService.h Summary: Move the default implementation of BaseService into the open source fb303 repository. Reviewed By: simpkins Differential Revision: D15565027 fbshipit-source-id: 3fd9bc06d747858ae0b0f093cb1f29947f4cf493
Updating submodules Reviewed By: yns88 fbshipit-source-id: 4f2cb905e806c20a3319ef4470efeda1d9f13549
remove integration_tests Summary: These tests don't really work, deleting Reviewed By: mariof Differential Revision: D14972292 fbshipit-source-id: 179e7706b629c686e004dbbe958a7e7ca50d2b69
Ensure withdraw prefix is client or type specific Summary: IpPrefix + type/client uniquely identifies a instance to withdraw. Even though thrift interface takes PrefixEntry, PrefixManagerClient, is only taking IpPrefix as input, this causes conflict as to which specific prefix to be deleted. Going forward as we enable multi client support etc we need to uniquely delete a specific instance of prefix mgr entry. Ensured that PrefixEntry is deleted only if type matches. Ensured that all the prefixes passed for withdraw must exist, i.e. we only act on the message, if full message can be acted upon without any errors. Another reason for doing this change is, we can use prefix type in input parameters to figure out if we need to update persistent store or not (for next diff) without checking values in prefixMap_. Reviewed By: saifhhasan Differential Revision: D14453541 fbshipit-source-id: da3e3d45a56c29f7025a6fcd1396e75d91300aa2
PreviousNext