Skip to content

dispatcher: Remove obsolete runtime feature envoy.reloadable_features.activate_fds_next_event_loop#14055

Merged
ggreenway merged 2 commits intoenvoyproxy:masterfrom
antoniovicente:remove_file_feature_tag
Nov 18, 2020
Merged

dispatcher: Remove obsolete runtime feature envoy.reloadable_features.activate_fds_next_event_loop#14055
ggreenway merged 2 commits intoenvoyproxy:masterfrom
antoniovicente:remove_file_feature_tag

Conversation

@antoniovicente
Copy link
Copy Markdown
Contributor

Commit Message:
dispatcher: Remove obsolete runtime feature envoy.reloadable_features.activate_fds_next_event_loop
Additional Description:
Feature flag was introduced in PR #11750 on 2020-06-27. Would be good to obsolete in preparation to changes to activate/setEnable behavior which should make activate behavior more intuitive and simplify edge trigger emulation and userspace file event implementations.
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

….activate_fds_next_event_loop

Signed-off-by: Antonio Vicente <avd@google.com>
Copy link
Copy Markdown
Member

@ggreenway ggreenway left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@ggreenway ggreenway left a comment

Choose a reason for hiding this comment

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

release note? In case someone has it in their config, so they know to remove it to de-clutter

Signed-off-by: Antonio Vicente <avd@google.com>
@ggreenway
Copy link
Copy Markdown
Member

/retest

@repokitteh-read-only
Copy link
Copy Markdown

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

🐱

Caused by: a #14055 (comment) was created by @ggreenway.

see: more, trace.

@ggreenway ggreenway merged commit 7c0e0b9 into envoyproxy:master Nov 18, 2020
andreyprezotto pushed a commit to andreyprezotto/envoy that referenced this pull request Nov 24, 2020
….activate_fds_next_event_loop (envoyproxy#14055)

Feature flag was introduced in PR envoyproxy#11750 on 2020-06-27. Would be good to obsolete in preparation to changes to activate/setEnable behavior which should make activate behavior more intuitive and simplify edge trigger emulation and userspace file event implementations.

Signed-off-by: Antonio Vicente <avd@google.com>
qqustc pushed a commit to qqustc/envoy that referenced this pull request Nov 24, 2020
….activate_fds_next_event_loop (envoyproxy#14055)

Feature flag was introduced in PR envoyproxy#11750 on 2020-06-27. Would be good to obsolete in preparation to changes to activate/setEnable behavior which should make activate behavior more intuitive and simplify edge trigger emulation and userspace file event implementations.

Signed-off-by: Antonio Vicente <avd@google.com>
Signed-off-by: Qin Qin <qqin@google.com>
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