Skip to content
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

Update Target Allocator Documentation - Add Diagrams #2229

Closed
avillela opened this issue Oct 13, 2023 · 1 comment
Closed

Update Target Allocator Documentation - Add Diagrams #2229

avillela opened this issue Oct 13, 2023 · 1 comment
Labels
area:target-allocator Issues for target-allocator documentation Improvements or additions to documentation

Comments

@avillela
Copy link
Contributor

Component(s)

No response

Describe the issue you're reporting

It would be helpful to include diagrams in the Target Allocator Readme to further illustrate how it works.

avillela added a commit to avillela/opentelemetry-operator that referenced this issue Oct 13, 2023
@jaronoff97 jaronoff97 added documentation Improvements or additions to documentation area:target-allocator Issues for target-allocator and removed needs triage labels Oct 13, 2023
jaronoff97 pushed a commit that referenced this issue Oct 17, 2023
#2230)

* Add diagrams to further enhance Target Allocator documentation - #2229

* Update cmd/otel-allocator/README.md

Co-authored-by: Sebastian Poxhofer <[email protected]>

* Update cmd/otel-allocator/README.md

Co-authored-by: Sebastian Poxhofer <[email protected]>

* Update TA diagram

* Update TA diagram and description

* Add mermaid diagrams

* Add mermaid diagrams

* Remove diagram images (using mermaid instead)

---------

Co-authored-by: Sebastian Poxhofer <[email protected]>
@jaronoff97
Copy link
Contributor

closed by #2230

ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this issue May 1, 2024
…-telemetry#2229 (open-telemetry#2230)

* Add diagrams to further enhance Target Allocator documentation - open-telemetry#2229

* Update cmd/otel-allocator/README.md

Co-authored-by: Sebastian Poxhofer <[email protected]>

* Update cmd/otel-allocator/README.md

Co-authored-by: Sebastian Poxhofer <[email protected]>

* Update TA diagram

* Update TA diagram and description

* Add mermaid diagrams

* Add mermaid diagrams

* Remove diagram images (using mermaid instead)

---------

Co-authored-by: Sebastian Poxhofer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:target-allocator Issues for target-allocator documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants