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 v3 API metrics #9468

Merged
merged 19 commits into from
Jun 16, 2021
Merged

Add v3 API metrics #9468

merged 19 commits into from
Jun 16, 2021

Conversation

coignetp
Copy link
Contributor

@coignetp coignetp commented Jun 3, 2021

What does this PR do?

Envoy v3 API is available, and there are a few new metrics.
This PR adds a new envoy environment, with a v3 API available so we can test against it.

Motivation

Close #8877

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached

@ghost ghost added the integration/envoy label Jun 3, 2021
@codecov
Copy link

codecov bot commented Jun 3, 2021

Codecov Report

Merging #9468 (1d132a7) into master (14bf087) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

Flag Coverage Δ
envoy 93.77% <100.00%> (+0.25%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@coignetp coignetp marked this pull request as ready for review June 15, 2021 10:36
@coignetp coignetp requested a review from a team as a code owner June 15, 2021 10:36
@coignetp coignetp merged commit 8920f58 into master Jun 16, 2021
@coignetp coignetp deleted the paul/envoy-118 branch June 16, 2021 08:07
github-actions bot pushed a commit that referenced this pull request Jun 16, 2021
* Enable envoy discovery service

* Attempt to Move version to 1.18.2

* Bump version to 1.18.3

* Update server version info

* remove pip3 and use pip

* add pip to envoy image

* fix envoy yaml for api v3

* Add api v3 env

* Add missing metrics

* Update e2e tests

* Fix metadata columns

* Remove assert all

* Assert metric v3 at least 0

* Fix metadata

* Fix bench

* Rename cluster_name tag

* Add v3 metrics description

Co-authored-by: HantingZHANG <[email protected]>
Co-authored-by: Luis Gonzalez <[email protected]> 8920f58
alexandre-normand pushed a commit that referenced this pull request Jun 24, 2021
* Enable envoy discovery service

* Attempt to Move version to 1.18.2

* Bump version to 1.18.3

* Update server version info

* remove pip3 and use pip

* add pip to envoy image

* fix envoy yaml for api v3

* Add api v3 env

* Add missing metrics

* Update e2e tests

* Fix metadata columns

* Remove assert all

* Assert metric v3 at least 0

* Fix metadata

* Fix bench

* Rename cluster_name tag

* Add v3 metrics description

Co-authored-by: HantingZHANG <[email protected]>
Co-authored-by: Luis Gonzalez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[envoy] Missing Envoy metrics
4 participants