8000 Correctly starting Selenoid from image with custom registry host and … by vania-pooh · Pull Request #256 · aerokube/cm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Dec 17, 2024. It is now read-only.

Correctly starting Selenoid from image with custom registry host and … #256

Merged
merged 4 commits into from
Feb 29, 2020

Conversation

vania-pooh
Copy link
Member

…port (fixes #250)

@codecov
Copy link
codecov bot commented Dec 31, 2019

Codecov Report

Merging #256 into master will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #256      +/-   ##
==========================================
+ Coverage   72.88%   72.96%   +0.08%     
==========================================
  Files           4        4              
  Lines        1180     1180              
==========================================
+ Hits          860      861       +1     
  Misses        214      214              
+ Partials      106      105       -1
Impacted Files Coverage Δ
selenoid/drivers.go 73.95% <ø> (-0.07%) ⬇️
selenoid/docker.go 77.04% <100%> (+0.2%) ⬆️

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 39a8e88...91cbfdb. Read the comment docs.

@@ -261,6 +261,7 @@ func (c *DockerConfigurator) getImage(name string, version string) *types.ImageS
c.Errorf("Failed to list images: %v", err)
return nil
}
//name = c.getFullyQualifiedImageRef(name)
Copy link
Contributor
@lanwen lanwen Jan 5, 2020

Choose a reason for hiding this comment

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

what does it suppose to mean?

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, accidentally commented stuff. Need to recheck.

@vania-pooh vania-pooh merged commit b8dd6c5 into aerokube:master Feb 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed to start: Selenoid with registry if dont run docker pull
2 participants
0