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 database measurements Endpoint #80

Merged
merged 5 commits into from Apr 13, 2020

Conversation

gssbzn
Copy link
Collaborator

@gssbzn gssbzn commented Apr 9, 2020

No description provided.

Copy link
Collaborator

@andreaangiolillo andreaangiolillo left a comment

Choose a reason for hiding this comment

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

LGTM

@gssbzn gssbzn requested a review from themantissa April 9, 2020 16:36
@gssbzn
Copy link
Collaborator Author

gssbzn commented Apr 9, 2020

@themantissa ready for your review

themantissa
themantissa previously approved these changes Apr 9, 2020
Copy link
Collaborator

@themantissa themantissa left a comment

Choose a reason for hiding this comment

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

LGTM!

@themantissa
Copy link
Collaborator

@PacoDw @marinsalinas @coderGo93 this one is just a sanity check PR, shouldn't impact any downstream work for Terraform/Vault/CFN.

Comment on lines 24 to 25
mediaType = "application/json"
gzipMediaType = "application/gzip"
Copy link
Contributor

@PacoDw PacoDw Apr 13, 2020

Choose a reason for hiding this comment

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

Nice! maybe we should rename mediaType tojsonMediaType, what do you think?

PacoDw
PacoDw previously approved these changes Apr 13, 2020
gssbzn and others added 4 commits April 13, 2020 17:06
Co-Authored-By: Andrea Angiolillo <andrea.angiolillo@mongodb.com>
Co-Authored-By: Andrea Angiolillo <andrea.angiolillo@mongodb.com>
Copy link
Contributor

@PacoDw PacoDw left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you so much! 👍

@gssbzn gssbzn merged commit 5574d7c into mongodb:master Apr 13, 2020
@gssbzn gssbzn deleted the database-measurements branch April 13, 2020 16:37
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.

None yet

4 participants