Skip to content

2030 fixed link when bridge stated#2046

Open
pcellix wants to merge 1 commit into
docker:masterfrom
pcellix:2030-fix
Open

2030 fixed link when bridge stated#2046
pcellix wants to merge 1 commit into
docker:masterfrom
pcellix:2030-fix

Conversation

@pcellix

@pcellix pcellix commented Aug 14, 2019

Copy link
Copy Markdown

added additional check if it is not default network named 'bridge'
If it is then links can be passed

Can be checked by running:
docker run --network=bridge --link containera:aliasa busybox

Signed-off-by: Wojciech Kosowicz pcellix@gmail.com

Signed-off-by: Wojciech Kosowicz <pcellix@gmail.com>
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #2046 into master will increase coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #2046      +/-   ##
==========================================
+ Coverage   56.79%   56.79%   +<.01%     
==========================================
  Files         311      311              
  Lines       21836    21837       +1     
==========================================
+ Hits        12402    12403       +1     
  Misses       8519     8519              
  Partials      915      915

1 similar comment
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #2046 into master will increase coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #2046      +/-   ##
==========================================
+ Coverage   56.79%   56.79%   +<.01%     
==========================================
  Files         311      311              
  Lines       21836    21837       +1     
==========================================
+ Hits        12402    12403       +1     
  Misses       8519     8519              
  Partials      915      915

@pcellix

pcellix commented Aug 15, 2019

Copy link
Copy Markdown
Author

Hi can anybody can tell me what's the problem with codecov and how to have it passed?
This is my 1st PR here :)
Thank you!

@pcellix

pcellix commented Oct 10, 2019

Copy link
Copy Markdown
Author

Hello
Any feedback so this one can be closed?

@pcellix

pcellix commented Dec 16, 2019

Copy link
Copy Markdown
Author

Hi

Please let know how to continue with it

@thaJeztah thaJeztah added this to the 29.5.1 milestone May 14, 2026
@vvoland vvoland modified the milestones: 29.5.1, 29.5.2 May 18, 2026
@thaJeztah thaJeztah modified the milestones: 29.5.2, 29.5.3 May 20, 2026
@vvoland vvoland modified the milestones: 29.5.3, 29.6.0 Jun 3, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/container/opts.go 33.33% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

links are only supported for user-defined networks - Inconsistency

8 participants