Skip to content

service: support --mount type=bind,bind-nonrecursive#38788

Merged
cpuguy83 merged 1 commit intomoby:masterfrom
AkihiroSuda:bind-nonrecursive-swarm
Mar 13, 2020
Merged

service: support --mount type=bind,bind-nonrecursive#38788
cpuguy83 merged 1 commit intomoby:masterfrom
AkihiroSuda:bind-nonrecursive-swarm

Conversation

@AkihiroSuda
Copy link
Member

@AkihiroSuda AkihiroSuda commented Feb 23, 2019

follow-up to #38003 for services

Signed-off-by: Akihiro Suda [email protected]

- What I did

Support docker service create --mount type=bind,bind-nonrecursive

- How I did it

Updated daemon/cluster

- How to verify it

$ docker service create --mount type=bind,src=/,dst=/foo,ro,bind-nonrecursive nginx:alpine
$ docker exec -it frosty_boyd.1.5ci8zh74r3uns8nzp7h6jsloa sh
/ # ls /foo/mnt/other-mounts

- Description for the changelog
service: support --mount type=bind,bind-nonrecursive

- A picture of a cute animal (not mandatory but encouraged)
🐧

@codecov
Copy link

codecov bot commented Feb 24, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@d959c20). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master   #38788   +/-   ##
=========================================
  Coverage          ?   36.43%           
=========================================
  Files             ?      613           
  Lines             ?    45854           
  Branches          ?        0           
=========================================
  Hits              ?    16707           
  Misses            ?    26854           
  Partials          ?     2293

@AkihiroSuda
Copy link
Member Author

@cpuguy83 PTAL?

@thaJeztah
Copy link
Member

ping @kolyshkin @cpuguy83 PTAL 🙏

Copy link
Member

@tonistiigi tonistiigi left a comment

Choose a reason for hiding this comment

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

ping @cpuguy83

@AkihiroSuda You have to repush, apparently there is no way to trigger ci rebuilds in this new ci system.

@AkihiroSuda AkihiroSuda force-pushed the bind-nonrecursive-swarm branch from 9e81db6 to 0599390 Compare October 12, 2019 17:05
@AkihiroSuda
Copy link
Member Author

ping @tonistiigi @cpuguy83

@thaJeztah thaJeztah assigned cpuguy83 and unassigned tonistiigi Dec 13, 2019
@AkihiroSuda AkihiroSuda changed the title service: support --mount type=bind,bind-nonrecursive [WIP] service: support --mount type=bind,bind-nonrecursive Feb 10, 2020
@AkihiroSuda AkihiroSuda force-pushed the bind-nonrecursive-swarm branch 3 times, most recently from b652923 to 19f897b Compare March 13, 2020 09:48
@AkihiroSuda AkihiroSuda force-pushed the bind-nonrecursive-swarm branch from 19f897b to 591dd3e Compare March 13, 2020 09:59
@AkihiroSuda AkihiroSuda changed the title [WIP] service: support --mount type=bind,bind-nonrecursive service: support --mount type=bind,bind-nonrecursive Mar 13, 2020
@AkihiroSuda
Copy link
Member Author

addressed comments

@AkihiroSuda AkihiroSuda force-pushed the bind-nonrecursive-swarm branch from 591dd3e to 745fa04 Compare March 13, 2020 10:01
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@cpuguy83 PTAL

Copy link
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

@cpuguy83 cpuguy83 merged commit 714cba6 into moby:master Mar 13, 2020
@thaJeztah thaJeztah added this to the 20.03.0 milestone Apr 2, 2020
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.

5 participants