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

fix: protect when deployment not in gathering endpoints dictionary #5440

Merged
merged 1 commit into from Nov 24, 2022

Conversation

JoanFM
Copy link
Member

@JoanFM JoanFM commented Nov 24, 2022

Goals:
Protect for the case where the deployment is not in the endpoint dictionary

@github-actions github-actions bot added size/S area/core This issue/PR affects the core codebase labels Nov 24, 2022
@codecov
Copy link

codecov bot commented Nov 24, 2022

Codecov Report

Merging #5440 (50fbd91) into master (8344d16) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5440   +/-   ##
=======================================
  Coverage   87.26%   87.27%           
=======================================
  Files          99       99           
  Lines        6566     6571    +5     
=======================================
+ Hits         5730     5735    +5     
  Misses        836      836           
Flag Coverage Δ
jina 87.27% <100.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
...ina/serve/runtimes/gateway/graph/topology_graph.py 99.48% <100.00%> (ø)
jina/serve/runtimes/gateway/request_handling.py 98.13% <100.00%> (+0.09%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@JoanFM JoanFM requested review from samsja, alaeddine-13 and JohannesMessner and removed request for samsja and alaeddine-13 November 24, 2022 08:53
@JoanFM JoanFM changed the title fix: fix gathering endpoints failed fix: protect when deployment not in gathering endpoints dictionary Nov 24, 2022
@JoanFM JoanFM merged commit bb27bc2 into master Nov 24, 2022
@JoanFM JoanFM deleted the fix-name-not-in-endpoint branch November 24, 2022 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core This issue/PR affects the core codebase size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant