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

minor: remove unused endpointRegistry field from healthyEndpoints and fadeIn #3248

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

RomanZavodskikh
Copy link
Member

Just a small removal of unsued field.

Signed-off-by: Roman Zavodskikh <roman.zavodskikh@zalando.de>
@RomanZavodskikh RomanZavodskikh added the minor no risk changes, for example new filters label Sep 24, 2024
@AlexanderYastrebov
Copy link
Member

I think this related is also unused

endpointRegistry *routing.EndpointRegistry

Signed-off-by: Roman Zavodskikh <roman.zavodskikh@zalando.de>
@RomanZavodskikh RomanZavodskikh changed the title minor: remove unused endpointRegistry field from healthyEndpoints minor: remove unused endpointRegistry field from healthyEndpoints and fadeIn Sep 24, 2024
@RomanZavodskikh
Copy link
Member Author

@AlexanderYastrebov thanks for a good catch, I have removed.

@AlexanderYastrebov
Copy link
Member

👍

@MustafaSaber
Copy link
Member

restarted pipeline

@MustafaSaber
Copy link
Member

👍

1 similar comment
@RomanZavodskikh
Copy link
Member Author

👍

@RomanZavodskikh RomanZavodskikh merged commit 2e9e3cf into master Sep 25, 2024
14 checks passed
@RomanZavodskikh RomanZavodskikh deleted the removeRedundantEndpointregistryLink branch September 25, 2024 08:01
Pushpalanka pushed a commit that referenced this pull request Oct 11, 2024
… fadeIn (#3248)

Signed-off-by: Roman Zavodskikh <roman.zavodskikh@zalando.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor no risk changes, for example new filters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants