Skip to content

Remove unused "containerd-addr" and "containerd-namespace" flags - #2704

Merged
dperny merged 1 commit into
moby:masterfrom
thaJeztah:remove_dead_code
Jul 11, 2018
Merged

dperny merged 1 commit into
moby:masterfrom
thaJeztah:remove_dead_code

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

commit fe9b063 (#2568) removed the containerd executor, as it was not currently maintained, but these two flags were left behind and are no longer functional.

see #2595 (comment)

commit fe9b063 removed the containerd executor,
as it was not currently maintained, but these two flags were left behind and are
no longer functional.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
Member Author

ping @ijc @stevvooe ptal

@codecov

codecov Bot commented Jul 11, 2018

Copy link
Copy Markdown

Codecov Report

Merging #2704 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2704      +/-   ##
==========================================
+ Coverage   61.79%   61.79%   +<.01%     
==========================================
  Files         134      134              
  Lines       21764    21764              
==========================================
+ Hits        13448    13449       +1     
+ Misses       6873     6862      -11     
- Partials     1443     1453      +10

@dperny

dperny commented Jul 11, 2018

Copy link
Copy Markdown
Collaborator

PURGE

@dperny
dperny merged commit 231d181 into moby:master Jul 11, 2018
@thaJeztah
thaJeztah deleted the remove_dead_code branch July 11, 2018 17:16
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.

4 participants