Skip to content

Commit

Permalink
Fix kelvin calls to MDS
Browse files Browse the repository at this point in the history
Summary: this broke when i made my namespace changes. the kelvin's mds_addr flag is now only a part of the actual mds address. this should have been updated to use the fully constructed version above.

Test Plan: ran skaffold

Reviewers: vihang

Reviewed By: vihang

Differential Revision: https://phab.corp.pixielabs.ai/D8471

GitOrigin-RevId: f240bb9
  • Loading branch information
aimichelle authored and copybaranaut committed Apr 29, 2021
1 parent ac0282d commit a5ae0ae
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/vizier/services/agent/kelvin/kelvin_main.cc
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ int main(int argc, char** argv) {
absl::Substitute("$0.$1.svc:$2", FLAGS_mds_addr, FLAGS_namespace, FLAGS_mds_port);

auto manager = KelvinManager::Create(agent_id, FLAGS_pod_name, FLAGS_host_ip, addr,
FLAGS_rpc_port, FLAGS_nats_url, FLAGS_mds_addr)
FLAGS_rpc_port, FLAGS_nats_url, mds_addr)
.ConsumeValueOrDie();

AgentTerminationHandler::set_manager(manager.get());
Expand Down

0 comments on commit a5ae0ae

Please sign in to comment.