Skip to content

Commit

Permalink
Back out "Update directly connected routes if any updates"
Browse files Browse the repository at this point in the history
Summary:
Original commit changeset: 8d2fc26d5172

Original Phabricator Diff: D54049829

Reviewed By: zechengh09

Differential Revision: D56641955

fbshipit-source-id: a27d46d517fe12c4aba978870c223a0e2ca16444
  • Loading branch information
Shrikrishna (Shri) Khare authored and facebook-github-bot committed Apr 26, 2024
1 parent 5d7c3bb commit 90cdd78
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions fboss/agent/DsfStateUpdaterUtil.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -73,12 +73,11 @@ namespace facebook::fboss {
std::shared_ptr<SwitchState> DsfStateUpdaterUtil::getUpdatedState(
const std::shared_ptr<SwitchState>& in,
const SwitchIdScopeResolver* scopeResolver,
RoutingInformationBase* rib,
RoutingInformationBase* /* rib */,
const std::map<SwitchID, std::shared_ptr<SystemPortMap>>&
switchId2SystemPorts,
const std::map<SwitchID, std::shared_ptr<InterfaceMap>>& switchId2Intfs) {
bool changed{false};
bool intfChanged{false};
auto out = in->clone();

auto skipProgramming = [&](const auto& nbrEntryIter) -> bool {
Expand Down Expand Up @@ -264,11 +263,6 @@ std::shared_ptr<SwitchState> DsfStateUpdaterUtil::getUpdatedState(
InterfaceMapDelta delta(origRifs.get(), newRifs.get());
auto remoteRifs = out->getRemoteInterfaces()->modify(&out);
processDelta(delta, remoteRifs, makeRemoteRif);
intfChanged |= (delta.begin() != delta.end());
}

if (intfChanged) {
updateRemoteConnectedRoutes(out, scopeResolver, rib);
}

if (changed) {
Expand Down

0 comments on commit 90cdd78

Please sign in to comment.