Skip to content

Add Nginx receiver for the metrics side and related tests and docs. #173

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
Merged
merged 7 commits into from
Aug 25, 2021
Merged

Add Nginx receiver for the metrics side and related tests and docs. #173

merged 7 commits into from
Aug 25, 2021

Conversation

@qingling128
Copy link
Contributor

@qingling128 qingling128 commented Aug 25, 2021 •
edited
Loading

djaglowski and others added 7 commits August 25, 2021 18:08
@djaglowski @qingling128
Add nginxreceiver and normalizesumsprocessor (#153)
1d99cca 
* Add nginxreceiver and normalizesumsprocessor

* Update opentelemetry collector submodule, to bring in nginx receiver and normalizesums processor.

* Remove custom url validator

* Make nginx url optional, and adjust test cases accordingly.

* Remove port from golden nginx test

* Switch nginx to workload.googleapis.com
@djaglowski @qingling128
Update golden files that were missed in recent merge. (#167)
79aba6e
@djaglowski @qingling128
Add nginx docs (#159)
4a98044
@qingling128
Update golden files in the nginx branch after rebasing from the maste…
f2ca5e6 
…r branch. (#171)
@djaglowski
Add nginx integration test (#161)
7e61c78
@parkjam4
Update nginx.md (#172)
c434b3f
@djaglowski
Change nginx 'endpoint' to 'stub_status_url' (#166)
8131666
@google-cla
Copy link

google-cla bot commented Aug 25, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@djaglowski
Copy link
Contributor

djaglowski commented Aug 25, 2021

@googlebot I consent.

@djaglowski djaglowski added the kokoro:force-run Forces kokoro to run integration tests on a CL label Aug 25, 2021
@qingling128 qingling128 added kokoro:force-run Forces kokoro to run integration tests on a CL and removed kokoro:force-run Forces kokoro to run integration tests on a CL labels Aug 25, 2021
@kokoro-team kokoro-team removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Aug 25, 2021
@qingling128 qingling128 requested a review from jsuereth August 25, 2021 22:45
@jsuereth jsuereth merged commit 8357686 into master Aug 25, 2021
@jsuereth jsuereth deleted the nginx branch August 25, 2021 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsuereth jsuereth jsuereth approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@qingling128 @djaglowski @jsuereth @kokoro-team @parkjam4