Skip to content

fix list crashed issue#424

Merged
metrue merged 1 commit intomasterfrom
fix-list-exception
Dec 19, 2019
Merged

fix list crashed issue#424
metrue merged 1 commit intomasterfrom
fix-list-exception

Conversation

@metrue
Copy link
Owner

@metrue metrue commented Dec 19, 2019

with kubernetes single node cluster, "fx list" will crash since some
service has no names, but the code trying to access names at index 0

Signed-off-by: Minghe Huang h.minghe@gmail.com

Issue:
Summary:

The checklist before PR is ready for review:

  • has unit testing for new added codes
  • has functional testing for new added features
  • has checked the lint or style issues
  • README updated if need

with kubernetes single node cluster, "fx list" will crash since some
service has no names, but the code trying to access names at index 0

Signed-off-by: Minghe Huang <h.minghe@gmail.com>
@codecov
Copy link

codecov bot commented Dec 19, 2019

Codecov Report

Merging #424 into master will decrease coverage by 0.1%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #424      +/-   ##
==========================================
- Coverage   32.79%   32.68%   -0.11%     
==========================================
  Files          48       48              
  Lines        2162     2169       +7     
==========================================
  Hits          709      709              
- Misses       1331     1338       +7     
  Partials      122      122
Impacted Files Coverage Δ
container_runtimes/docker/http/api.go 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3b6438...e0d5bf3. Read the comment docs.

@metrue metrue merged commit a84e7da into master Dec 19, 2019
@metrue metrue deleted the fix-list-exception branch December 19, 2019 08:05
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.

1 participant