Skip to content

add support for some missing API entries#960

Closed
troubadout wants to merge 2 commits intodocker-java:3.0.xfrom
troubadout:3.0.x
Closed

add support for some missing API entries#960
troubadout wants to merge 2 commits intodocker-java:3.0.xfrom
troubadout:3.0.x

Conversation

@troubadout
Copy link
Copy Markdown

@troubadout troubadout commented Dec 5, 2017

  • IPC namespace mode
  • Env support in docker exec

This change is Reviewable

@troubadout troubadout force-pushed the 3.0.x branch 2 times, most recently from 3a378cd to 300e4b1 Compare December 5, 2017 16:19
@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 5, 2017

Codecov Report

Merging #960 into 3.0.x will increase coverage by 0.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff            @@
##           3.0.x     #960      +/-   ##
=========================================
+ Coverage   71.7%   71.71%   +0.01%     
=========================================
  Files        306      306              
  Lines       6882     6888       +6     
  Branches     516      516              
=========================================
+ Hits        4935     4940       +5     
- Misses      1648     1651       +3     
+ Partials     299      297       -2
Impacted Files Coverage Δ
...va/com/github/dockerjava/api/model/HostConfig.java 59.15% <33.33%> (-0.56%) ⬇️
...hub/dockerjava/core/command/ExecCreateCmdImpl.java 74.19% <33.33%> (-4.38%) ⬇️
...va/org/apache/http/impl/io/ChunkedInputStream.java 61.32% <0%> (+2.83%) ⬆️

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 a9f9838...f45a55b. Read the comment docs.

@Override
@JsonIgnore
public String getIpcMode() {
return hostConfig.getIpcMode();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Don't proxy hostconfig settings, hostconfig should be configured separately

Copy link
Copy Markdown
Member

@KostyaSha KostyaSha left a comment

Choose a reason for hiding this comment

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

Use some collection instead array

@KostyaSha KostyaSha changed the base branch from 3.0.x to master November 26, 2018 01:37
@KostyaSha KostyaSha changed the base branch from master to 3.0.x November 26, 2018 01:37
@KostyaSha
Copy link
Copy Markdown
Member

please switch to 3.1 releases

@KostyaSha KostyaSha closed this Mar 9, 2019
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.

3 participants