-
Notifications
You must be signed in to change notification settings - Fork 1
feat(telemetry): integrate the new FOE telemetry sidecar #566
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
Merged
Merged
Changes from 65 commits
Commits
Show all changes
103 commits
Select commit
Hold shift + click to select a range
0ca266b
feat: enable extraContainers
kaixi-wang 2151e2e
fix: share process namespace
kaixi-wang a46b8f0
fix: add logs access
kaixi-wang 93359dc
chore: update tests
kaixi-wang 4c2865e
fix: rm unread rbac file
kaixi-wang adb3b9e
feat: add nativeSidecarContainers
kaixi-wang 9578958
fix: add missing files
kaixi-wang e9e710a
chore: update compose files
kaixi-wang ee0ba72
chore: fix tests
kaixi-wang 1ca27a4
chore: fix tests
kaixi-wang 30282b1
chore: trigger ci
kaixi-wang ea4c386
fix: auto-inject sidecar settings
kaixi-wang d58c01c
fix: fix auto inject sidecar vars and update test
kaixi-wang d532ec4
chore: attempt to fix test
kaixi-wang fe89df4
feat: default true for telemetry and add templates
kaixi-wang 8348c4b
feat: enable sidecar by default in docker deployments
kaixi-wang b290329
chore: temp fix for tests
kaixi-wang 229c9eb
feat: default enable telemetric with all default settings generated
kaixi-wang 5d410c4
test: temporarily hardcode the sidecar
kaixi-wang a8c0357
fix: use latest sidecar version 0.1.62
kaixi-wang 047041c
fix: runAsNonRoot override in sidecar
kaixi-wang c4013e9
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 20655e2
feat(helm): support external Redis via `telemetry.redis.external.url`
mo-getter bc87707
feat(helm): configurable telemetry Redis persistence
mo-getter 0581d39
fix(helm): route DO env vars through `telemetry.redis.url` helper
mo-getter bf7d297
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 57f1cc6
feat(compose): bring telemetry config to parity with helm chart
claude 71a9456
refactor(compose): consolidate telemetry services into common-service…
claude 1161abf
fix(compose): default sidecar to GAR image; fix verify-section Redis …
mo-getter 703d4c7
fix(helm): use Recreate strategy for telemetry-redis Deployment
mo-getter ebd2edb
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot b4afaaf
refactor: align telemetry-sidecar image versioning with other voxel51…
mo-getter 079c65a
refactor(helm): drop unused extraContainers/nativeSidecarContainers k…
mo-getter 97649d2
fix: telemetry-sidecar repo
mo-getter 5f413e8
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 660f544
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 5c8a53d
docs(telemetry): fix sidecar examples; flag UI log-viewer dependency
mo-getter 04fb4f6
fix(compose): bump GPU DO image tag to v2.19.0
mo-getter 0862f09
fix(helm): use FQDN for in-cluster telemetry Redis URL
mo-getter 1a567a2
docs(helm): clarify telemetry sidecar tag default
mo-getter 5065ebf
feat(helm): add securityContext to telemetry-redis
mo-getter a05f11c
test(helm): cover shareProcessNamespace + sidecar securityContext
mo-getter 488da4b
docs: add v2.19+ telemetry upgrade callout
mo-getter 2d17007
refactor(helm): drop redundant teams-api SA from telemetry RoleBinding
mo-getter e6850fb
feat(helm): support pre-provisioned PVC for telemetry Redis
mo-getter 43d140c
docs: clarify upgrading docs
mo-getter 63bc17a
fix: rm sys_ptrace from services, explicit user rather than root, set…
kaixi-wang 23793fa
chore: trigger build
kaixi-wang 60ec930
chore: trigger build
kaixi-wang 046fd10
fix: split into one resource per template file to match existing pattern
mo-getter 9cbff10
fix: revert validation logic
mo-getter a11ef54
fix(test): update to support split redis into multiple files
mo-getter 6684bc2
chore: remove unused code
mo-getter 9b807cf
style: swap key order for consistency
mo-getter 95a28b8
chore: remove redis max memory policy config option
mo-getter b3a9524
style: use helper since it's available
mo-getter 1ce213d
refactor(helm): drop sidecar targetUid; inherit UID from pod
mo-getter 4de5de9
style(helm): normalize telemetry-socket injection across DO templates
mo-getter 34627d5
chore(compose): remove redis max memory policy config option
mo-getter 355dacd
docs: tighten telemetry comments and prose
mo-getter 96cdd13
fix(compose): set telemetry env on teams-do-gpu main container
mo-getter e197ff4
docs(telemetry): fix mutually-exclusive base-file layering example
mo-getter 22d1e8c
docs: apply suggestions from code review
mo-getter 1802954
style: make var name match rest of file
mo-getter b9f9ee1
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot e6d87c6
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 4cc0da0
fix: enable do logs
kaixi-wang 7128d15
Merge pull request #570 from voxel51/kacey/add-dac-read-search
kaixi-wang be81e4a
feat(compose): profile-gated DO worker slots for telemetry observability
mo-getter 2b2f3ea
test(compose): expect new telemetry-socket-{2,3} volumes in DO overlay
mo-getter 67e8049
fix: do not suggest helm as an alternative in the docker docs
mo-getter 3165f88
chore: less flaky do integration tests
kaixi-wang 4fc2b70
fix: skip the emptyDir warning when existingClaim is configured
mo-getter 0b143c7
test(compose): tolerate single- or double-space Compose CLI output
mo-getter e88f372
fix: guard redis-url-env since it assumed `.ctx`
mo-getter f4fbc3f
Merge pull request #571 from voxel51/kacey/update-do-intg-test
kaixi-wang 709044e
fix: add missing `DAC_READ_SEARCH` to DO telemetry to enable logging
mo-getter 7b52308
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 047e9c4
docs: add justification for added capabilities
mo-getter 952027c
fix: set user so kernel matches workload
kaixi-wang 2f0c6ba
fix: api log expectation
kaixi-wang f009fa7
chore: rm comment
kaixi-wang a8c5439
fix(telemetry): drop DAC_READ_SEARCH, run inline DO sidecars as 1000:…
mo-getter 16a662a
fix(tests): use a more consistently used string for log verification
mo-getter 03ac3d2
fix(tests): fix DO log expect
mo-getter 79402bb
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 54a13c5
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot 49ffb73
fix: use fiftyone debug log msg instead of sanic motd
kaixi-wang 95dd6a7
fix: use connecting str
kaixi-wang 8abcc14
fix: add driver-ops and docs warning
kaixi-wang 94b5263
ci(fixtures): automated bump for integration test fixtures teams-app:…
voxelbuildbot b56fe9f
chore: rm comments about upgrade issues and rm pythonbuffered
kaixi-wang b9462ad
Merge remote-tracking branch 'refs/remotes/origin/release/v2.19.0-vol…
kaixi-wang 82453e8
Merge pull request #576 from voxel51/kacey/v2.19.0-volume-ownership
kaixi-wang a0e7ece
fix(tests): flaking on log entries again
mo-getter 79d57ab
Update docker/docs/configuring-delegated-operators.md
kaixi-wang a27178f
Update docker/docs/configuring-gpu-workloads.md
kaixi-wang f435db5
Apply suggestions from code review
mo-getter 170cddd
Apply suggestions from code review
mo-getter d2f7f32
fix: apply suggestions from code review
mo-getter ed9cd75
fix: apply suggestion from code review
mo-getter e08cff5
Apply suggestions from code review
mo-getter d25ee44
style: fix pre-commit lints
mo-getter 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
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
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.