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

Add single writer principle note to deployment documentation #5166

Conversation

michael2893
Copy link
Contributor

@michael2893 michael2893 commented Sep 8, 2024

Summary

This change addresses the request for documentation on the Single-Writer principle. #4368

Description

  • add section on multiple collector deployments in deployment/gateway
  • define single writer principle
  • provide examples and context

@michael2893 michael2893 requested a review from a team September 8, 2024 15:04
@opentelemetrybot opentelemetrybot requested a review from a team September 8, 2024 15:04
@opentelemetrybot opentelemetrybot requested review from atoulme and removed request for a team September 8, 2024 15:04
@opentelemetrybot opentelemetrybot requested a review from a team September 8, 2024 15:05
Copy link
Contributor

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First pass copy edits. Thanks!

content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
@tiffany76
Copy link
Contributor

tiffany76 commented Sep 9, 2024

Hi @michael2893. Thanks for sticking with this. A couple of the Collector maintainers reviewed the earlier PR (#4433), so I'm going to ask them to take a look here and make sure their comments have been addressed. @jpkrohling, @mx-psi, PTAL. Thanks!

@michael2893 michael2893 requested a review from a team as a code owner September 27, 2024 16:24
@opentelemetrybot opentelemetrybot requested a review from a team September 27, 2024 16:24
@svrnm
Copy link
Member

svrnm commented Nov 22, 2024

@open-telemetry/collector-approvers PTAL!

@svrnm svrnm added the sig-approval-missing Co-owning SIG didn't provide an approval label Nov 22, 2024
@svrnm
Copy link
Member

svrnm commented Dec 11, 2024

@open-telemetry/collector-approvers PTAL!

Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a couple of minor adjustments.

content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
content/en/docs/collector/deployment/gateway.md Outdated Show resolved Hide resolved
michael2893 and others added 2 commits January 2, 2025 09:35
Co-authored-by: Juraci Paixão Kröhling <juraci.github@kroehling.de>
Co-authored-by: Juraci Paixão Kröhling <juraci.github@kroehling.de>
@opentelemetrybot opentelemetrybot requested a review from a team January 2, 2025 14:35
michael2893 and others added 2 commits January 2, 2025 09:36
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
Copy link
Contributor

@tiffany76 tiffany76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm approving so we can get this merged. @michael2893, it looks like there's a conflict. Let us know if you need help resolving it.

@opentelemetrybot opentelemetrybot requested review from a team January 6, 2025 23:22
@windsonsea windsonsea removed the request for review from a team January 7, 2025 00:56
@cartermp cartermp removed request for atoulme and a team January 7, 2025 03:40
@cartermp
Copy link
Contributor

cartermp commented Jan 7, 2025

/fix:all

@opentelemetrybot
Copy link
Collaborator

You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/12644777977

@opentelemetrybot
Copy link
Collaborator

fix:all was successful.

IMPORTANT: (RE-)RUN /fix:all to ensure that there are no remaining check issues.

@opentelemetrybot opentelemetrybot requested a review from a team January 7, 2025 14:14
@cartermp
Copy link
Contributor

cartermp commented Jan 7, 2025

Merging! Thanks for the patience @michael2893

@cartermp cartermp added this pull request to the merge queue Jan 7, 2025
Merged via the queue into open-telemetry:main with commit 790dd33 Jan 7, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants