-
Notifications
You must be signed in to change notification settings - Fork 28
Introduce AutoMonitor for simpler enablement of Application Signals #297
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
Closed
Closed
Changes from 63 commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
7c8774d
almost all passing
the-mann 95b11c2
passing
the-mann 2a6d9df
check pod template labels instead of workload labels
the-mann b50163c
Provide Monitor functions (except Mutate) workload instead of pod tem…
the-mann 3ce94cf
big progress
the-mann 97ee02b
more progress
the-mann 24c29cd
works
the-mann b2d6c1c
blah
the-mann c70f7f8
blah
the-mann e47e85a
fix logging, add more testing
the-mann 6ab7980
remove unnecessary HasSynced
the-mann 82750e1
Handle service deletion, service update. test service deletion
the-mann a886b9a
Improve backwards compatability, make MutateAndPatchAll work with bot…
the-mann 9d6b4a7
Maintain legacy autoAnnotateAutoInstrumentation behavior.
the-mann 1216808
Clean up
the-mann b5a0e96
undo dockerfile update
the-mann e1cf759
Fix integration tests
the-mann 33df3e8
Rename auto_monitor*.go files to monitor*.go, rename AutoRestart to R…
the-mann 734966f
Enable multi instrumentation feature flag, enable skip multi instrume…
the-mann ed696e5
come up with actual commit message
the-mann 319e18a
Update tests to expect `namespace` in exclude and `customSelector` to…
the-mann 42f6e23
add transformer, warning function
the-mann d35ed59
fix integration tests to work with new exclude syntax
the-mann fbafe30
namespace-level exclude should update pod template, but namespace-lev…
the-mann d65c9f0
Use correct autoAnnotationConfigStr to create Instrumentation Annotator
the-mann eecdc14
Restart namespace for AutoMonitor based on `restartPods`, always rest…
the-mann 185ff81
omit languages, exclude, and customselector when empty
the-mann c28093a
add more integ tests for AutoMonitor
the-mann 707856b
fix permutation 9 (don't use kube-system)
the-mann a702b6a
improve comments
the-mann d9c3a00
Fix integ tests
the-mann 20b463c
add `DISABLE_AUTO_MONITOR` environment variable
the-mann 47e3b80
update github workflow to run new integration tests
the-mann d533077
Merge remote-tracking branch 'origin/main' into mpmann/appsignals-1-step
the-mann 936f0c2
Add controller manager name flag for minikube deployment
the-mann 4a86baf
Add controller manager name flag for minikube deployment
the-mann cabba78
Add manifest files
the-mann 955e643
temp: add log for cloudwatch controller manager for instrumentation t…
the-mann a2007b0
temp: add log for cloudwatch controller manager for instrumentation t…
the-mann d55e3cf
Correctly return nil when automonitor and autoannotation are disabled
the-mann aae6fd5
Remove redundant annotation config clears
the-mann e603175
Refactor annotation functions
the-mann 3bfe07c
disable autoannotation if autoAnnotationConfig is empty and AutoMonit…
the-mann 0ffe221
Add namespace MutateObject tests, fix namespace mutation.
the-mann 3b7cb97
Add default exclusion for kube-system and amazon-cloudwatch
the-mann 68101a0
Split MutateAndPatchAll into MutateAndPatchWorklads and MutateAndPatc…
the-mann efbe7c3
Add license
the-mann 88ade73
Do not modify feature flag values, instead override at podmutator lev…
the-mann 464bf37
Update generated api.md
the-mann 8007439
Fix linter issues
the-mann 5552674
Revert Dockerfile change
the-mann 9c8bc8d
Implement feedback
the-mann b68d141
Remove integ tests
the-mann 8667214
Remove integ tests
the-mann 3fb6a0f
go format
the-mann 910a042
finish rolling back integration test changes
the-mann 2f987de
SupportedTypes() -> SupportedTypes
the-mann ba8063c
update docs
the-mann 9c6ce5a
inline function for shouldOverrideMultiInstrumentation
the-mann b145244
name of variable should be same as struct name
the-mann 3ee911c
Remove todo, only check if autoAnnotationConfig is empty to use AutoM…
the-mann ddfd552
move excludedNamespaces to top
the-mann 0e27206
Refactor InstrumentationAnnotator creation out of main.go and into ut…
the-mann fe01d21
Incorporate auto/util.go feedback
the-mann File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,4 +4,5 @@ build | |
| .DS_Store | ||
| .idea | ||
| .attach_pid* | ||
| bin | ||
| bin | ||
| /amazon-cloudwatch-agent-operator | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.