Skip to content

Commit

Permalink
revert leader election change (#253)
Browse files Browse the repository at this point in the history
Co-authored-by: Ryan Zhang <[email protected]>
  • Loading branch information
ryanzhang-oss and Ryan Zhang authored Aug 29, 2022
1 parent ec2a74e commit 5394ff5
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
6 changes: 4 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,8 @@ install-hub-agent-helm:
--set image.pullPolicy=Never \
--set image.repository=$(REGISTRY)/$(HUB_AGENT_IMAGE_NAME) \
--set image.tag=$(HUB_AGENT_IMAGE_VERSION) \
--set logVerbosity=5
--set logVerbosity=5 \
--set namespace=fleet-system

.PHONY: e2e-hub-kubeconfig-secret
e2e-hub-kubeconfig-secret:
Expand All @@ -162,7 +163,8 @@ install-member-agent-helm: install-hub-agent-helm e2e-hub-kubeconfig-secret
--set refreshtoken.tag=$(REFRESH_TOKEN_IMAGE_VERSION) \
--set image.pullPolicy=Never --set refreshtoken.pullPolicy=Never \
--set config.memberClusterName="kind-$(MEMBER_KIND_CLUSTER_NAME)" \
--set logVerbosity=5
--set logVerbosity=5 \
--set namespace=fleet-system
# to make sure member-agent reads the token file.
kubectl delete pod --all -n fleet-system

Expand Down
1 change: 0 additions & 1 deletion cmd/memberagent/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,6 @@ func main() {
LeaderElection: *enableLeaderElection,
LeaderElectionNamespace: *leaderElectionNamespace,
LeaderElectionID: "3111024923.hub.fleet.azure.com",
LeaderElectionConfig: memberConfig,
Namespace: mcNamespace,
}

Expand Down
5 changes: 5 additions & 0 deletions test/e2e/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,9 @@ kubectl --context=kind-hub-testing -n fleet-system get pod
4. Check the logs of the member cluster controller
```shell
kubectl --context=kind-member-testing -n fleet-system get pod
```

5. uninstall the resources
```shell
make uninstall-helm
```

0 comments on commit 5394ff5

Please sign in to comment.