New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reconciler: kubeconfig update #73
Conversation
Define the reconciler as a system critical resource. Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
With this change the reconciler can run in two different ways: - when ran in a k8s pod, it does not require to be told how to connect to the cluster. - when ran as a binary, it does require to know how to connect to the cluster, via the -kubeconfig config option. The reconciler cron spec is updated to use the correct service account name, and also is updated to run in the `kube-system` namespace. Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
All (non error) logs have the same verbosity; it would be interesting to see the reconciled IPs by default, as such I'm bumping this particular logging call to `verbose`. That enables users to `kubectl logs <reconciler-pod-name` and see which IP addresses got reconciled. The below calls would be bumped to `verbose`. ``` 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.100] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.111] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.113] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.118] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.13] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.170] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.18] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.3] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.35] 2021-10-20T15:50:00Z [debug] removed stale overlappingIP allocation [10.10.0.4] 2021-10-20T15:50:01Z [debug] removed stale overlappingIP allocation [10.10.0.83] ``` Signed-off-by: Miguel Duarte Barroso <mdbarroso@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks!
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dougbtv, s1061123 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Includes fix for excluding IPv6 ranges as well as unit test
This also brings in the change for the upstream github namespace change, this was the first set of downstream changes that relied on those.