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

ecds: deprecate top-level stats guard #23611

Merged
merged 5 commits into from
Nov 3, 2022

Conversation

kyessenov
Copy link
Contributor

Signed-off-by: Kuat Yessenov kuat@google.com

Commit Message: Deprecate and remove stats prefix guard for ECDS.
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes: yes
Fixes: #23604

Signed-off-by: Kuat Yessenov <kuat@google.com>
@kyessenov
Copy link
Contributor Author

Flake #23618
/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #23611 (comment) was created by @kyessenov.

see: more, trace.

@wbpcode
Copy link
Member

wbpcode commented Oct 21, 2022

/assign @jmarantz

jmarantz
jmarantz previously approved these changes Oct 24, 2022
Copy link
Contributor

@jmarantz jmarantz left a comment

Choose a reason for hiding this comment

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

/assign-from @envoyproxy/senior-maintainers

@repokitteh-read-only
Copy link

@envoyproxy/senior-maintainers assignee is @lizan

🐱

Caused by: a #23611 (review) was submitted by @jmarantz.

see: more, trace.

@jmarantz
Copy link
Contributor

needs main merge

/wait

jmarantz
jmarantz previously approved these changes Oct 25, 2022
@jmarantz
Copy link
Contributor

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit didn't fail.

🐱

Caused by: a #23611 (comment) was created by @jmarantz.

see: more, trace.

@jmarantz
Copy link
Contributor

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit didn't fail.

🐱

Caused by: a #23611 (comment) was created by @jmarantz.

see: more, trace.

@kyessenov kyessenov closed this Oct 27, 2022
@kyessenov kyessenov reopened this Oct 27, 2022
@kyessenov kyessenov enabled auto-merge (squash) October 27, 2022 16:40
@kyessenov
Copy link
Contributor Author

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit didn't fail.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #23611 (comment) was created by @kyessenov.

see: more, trace.

lizan
lizan previously approved these changes Oct 28, 2022
@jmarantz
Copy link
Contributor

needs main merge; then we should be good to go.

@jmarantz
Copy link
Contributor

/wait

jmarantz
jmarantz previously approved these changes Oct 31, 2022
@jmarantz
Copy link
Contributor

jmarantz commented Nov 1, 2022

sorry need to merge main again. I'll try to check back after you do it and merge before we get another conflict.

@jmarantz
Copy link
Contributor

jmarantz commented Nov 1, 2022

/wait

jmarantz
jmarantz previously approved these changes Nov 2, 2022
@jmarantz
Copy link
Contributor

jmarantz commented Nov 2, 2022

docs failure looks real. snippet:

yaml.parser.ParserError: while parsing a block mapping
  in "<unicode string>", line 1, column 1:
    date: Pending
    ^
expected <block end>, but found '-'
  in "<unicode string>", line 67, column 1:
    - area: eds
    ^
"""

Signed-off-by: Kuat Yessenov <kuat@google.com>
@kyessenov
Copy link
Contributor Author

@jmarantz Thanks, git merge did a bad merge, manually fixed it.

@kyessenov
Copy link
Contributor Author

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #23611 (comment) was created by @kyessenov.

see: more, trace.

@jmarantz
Copy link
Contributor

jmarantz commented Nov 3, 2022

please ping me if you see this passes so I can merge before it grows another conflict on the release notes.

@kyessenov
Copy link
Contributor Author

@jmarantz I should be able to merge it with your approval.

@kyessenov
Copy link
Contributor Author

"abandoned" coverage job, restarting

@kyessenov kyessenov merged commit d93afd7 into envoyproxy:main Nov 3, 2022
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.

envoy_reloadable_features_top_level_ecds_stats deprecation
4 participants