Skip to content

RUST-1501 Collect FaaS platform metadata #844

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 11 commits into from
Mar 20, 2023

Conversation

abr-egn
Copy link
Contributor

@abr-egn abr-egn commented Mar 16, 2023

RUST-1501

This implements the new data collection from environment variables and the associated fallback logic to keep the handshake within the size limit.

@abr-egn abr-egn requested a review from isabelatkinson March 16, 2023 21:57
Copy link
Contributor

@isabelatkinson isabelatkinson left a comment

Choose a reason for hiding this comment

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

Code changes LGTM. Is it expected that users will be setting these environment variable values? If so, should we document somewhere how they should do so?

@abr-egn
Copy link
Contributor Author

abr-egn commented Mar 20, 2023

Code changes LGTM. Is it expected that users will be setting these environment variable values? If so, should we document somewhere how they should do so?

Nope, they're set by the FaaS runtime, no user action needed :)

@abr-egn abr-egn merged commit ca7bf1d into mongodb:main Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants