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.

Sign up for GitHub

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

Jump to bottom

feat: new blog post "Observing Lambdas" #5990

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
Open

feat: new blog post "Observing Lambdas" #5990

wants to merge 24 commits into from
+688 −0

Conversation

theSuess
Copy link

@theSuess theSuess commented Jan 20, 2025 •
edited by tiffany76
Loading

@opentelemetrybot opentelemetrybot requested a review from a team January 20, 2025 12:05
@github-actions github-actions bot added the blog label Jan 20, 2025
@theSuess theSuess mentioned this pull request Jan 20, 2025
Open
@theSuess theSuess force-pushed the blog/observing-lambdas branch from ed6c07d to 7c5394a Compare January 23, 2025 09:40
tiffany76
tiffany76 reviewed Jan 23, 2025
View reviewed changes
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.

A few copy edits. Thanks!

content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
@tiffany76
Copy link
Contributor

Sorry, I got ahead of you @theSuess! I just realized this was still in draft.

@opentelemetrybot opentelemetrybot requested a review from a team January 24, 2025 09:52
@theSuess
Copy link
Author

All good, I've set this to draft because of concerns whether a video is suitable, but it seems like there is no objection, so I'm promoting this PR as ready to review :)

@theSuess theSuess marked this pull request as ready for review January 24, 2025 09:54
@theSuess theSuess requested a review from a team as a code owner January 24, 2025 09:54
tiffany76
tiffany76 approved these changes Jan 24, 2025
View reviewed changes
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.

LGTM from a copy edit perspective.

content/en/blog/2025/observing-lambdas/index.md Outdated Show resolved Hide resolved
@chalin chalin force-pushed the blog/observing-lambdas branch from 4992595 to 7f6a19a Compare January 24, 2025 22:33
@opentelemetrybot opentelemetrybot requested a review from a team January 24, 2025 22:33
@opentelemetrybot opentelemetrybot requested a review from a team January 24, 2025 22:38
@opentelemetrybot opentelemetrybot requested a review from a team January 24, 2025 22:38
@chalin
Copy link
Contributor

chalin commented Jan 24, 2025

/fix:all

@opentelemetrybot
Copy link
Collaborator

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

chalin
chalin reviewed Jan 24, 2025
View reviewed changes
content/en/blog/2025/observing-lambdas/index.md
<video controls style="width: 100%">
<source src="./video-lambda-real-time.webm" />
</video>
</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add something at the end here to "wrap up" / offer some closing words?

Copy link
Author

Choose a reason for hiding this comment

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

I've added a short explanation & reference to language specific layers - let me know what you think!

@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 27, 2025 07:25
@svrnm svrnm requested review from a team and evan-bradley and removed request for a team January 27, 2025 07:54
@svrnm
Copy link
Member

svrnm commented Jan 27, 2025

@open-telemetry/lambda-extension-approvers @open-telemetry/collector-approvers PTAL

@theSuess theSuess force-pushed the blog/observing-lambdas branch from 711fe20 to 4ca5f23 Compare January 27, 2025 11:24
theSuess and others added 22 commits February 1, 2025 05:28
@theSuess @chalin
feat: new blog post "Observing Lambdas"
53edc9a
@theSuess @chalin
chore: replace video
b5b16e2
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
c5c3f62 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
f3bc6cf 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
e127047 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
b5a8cd5 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
3a81ae1 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
43b008a 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
4ee1290 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
e9d1a89 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
e3c96ca 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
3a91752 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
d2bf9c8 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@theSuess @tiffany76 @chalin
Update content/en/blog/2025/observing-lambdas/index.md
75871ac 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@chalin @tiffany76
Update content/en/blog/2025/observing-lambdas/index.md
a653e09 
Co-authored-by: Tiffany Hrabusa <[email protected]>
@chalin
Update content/en/blog/2025/observing-lambdas/index.md
a92e677
@chalin
Update content/en/blog/2025/observing-lambdas/index.md
71590a5
@chalin
Update content/en/blog/2025/observing-lambdas/index.md
78ef8bb
@opentelemetrybot @chalin
Results from /fix:all
09714e3
@theSuess @chalin
blog: add ending section
567d786
@theSuess @chalin
style: better legibility for timing diagram
280fc4c
@theSuess @chalin
docs: add reference to auto configuration
dceebea
@chalin chalin force-pushed the blog/observing-lambdas branch from 6f9d2ed to dceebea Compare February 1, 2025 10:28
@opentelemetrybot opentelemetrybot requested a review from a team February 1, 2025 10:29
@theSuess theSuess requested a review from chalin February 3, 2025 10:32
@opentelemetrybot opentelemetrybot requested a review from a team February 3, 2025 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@svrnm svrnm svrnm left review comments

@tylerbenson tylerbenson tylerbenson approved these changes

@tiffany76 tiffany76 tiffany76 approved these changes

@evan-bradley evan-bradley Awaiting requested review from evan-bradley evan-bradley was automatically assigned from open-telemetry/collector-approvers

@chalin chalin Awaiting requested review from chalin

Assignees
No one assigned
Labels
blog
Projects
SIG Comms: PRs & Issues
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@theSuess @tiffany76 @chalin @opentelemetrybot @svrnm @tylerbenson