-
Notifications
You must be signed in to change notification settings - Fork 137
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
Add option for selfregistering activegate #423
Conversation
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
2ebfe56
to
be7d0ef
Compare
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.
Just a few nitpicks about unnecessary checks, otherwise looks great, lots of tests ❤️
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
Please rebase on master so you have the Github Actions run. |
You should update the tests, now that you removed the checks I asked some of your tests are failing. |
- Added api client for monitored entities and settings - Added reconciler to trigger the automatic kubernetes api monitoring to create kubernetes settings for the given kube-system uuid - Added conditions (annotation and capabilities) to check whether automatic api monitoring should be executed - Added appropriate tests
- Adapted PR according to the requested changes
- Removes unnecessary checks
- Removes unnecessary tests
900f96e
to
4dea336
Compare
Unfortunate misclick 😅 |
- Move method to appropriate test file
# Conflicts: # src/controllers/dynakube/dynakube_controller_test.go
- Update test to use new test setup methods
- Refactors method to better reflect the usage
e576ae7
to
226d754
Compare
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
src/controllers/activegate/reconciler/automaticapimonitoring/reconciler.go
Outdated
Show resolved
Hide resolved
- Improves logging - Refactors method to better reflect the functionality
This PR adds the Kubernetes endpoint configuration creation logic to the Dynatrace operator to be available in the release 0.4.0 as an opt-in feature.