-
Notifications
You must be signed in to change notification settings - Fork 23.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docker_swarm_service: Documentation fixes #53479
docker_swarm_service: Documentation fixes #53479
Conversation
This PR is pretty hard to read. Can you split it up into two PRs:
That would make reviewing the PRs much easier. |
This reverts commit 51dabcc.
@felixfontein good idea. I'll create a PR with the reordering once this is merged. |
@@ -461,7 +465,6 @@ | |||
mode: | |||
description: | |||
- What publish mode to use. | |||
- Service will be removed and recreated when changed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this no longer true for publish
mode?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was never true. In #52634 I added that string to the wrong mode
-option.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's supposed to be documented on the actual mode
-option and not publish.mode
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
@felixfontein thank you for reviewing. @sivel thank you for merging. |
@hannseman and thanks for the PR! :) Looks like #53482 needs a rebase now... |
SUMMARY
Fixes a number of issues:
mode
choices is now validated and documented. Documentation about recreation of service is now correctly under this option and notpublish.mode
secrets
andusers
to module return valueBTW @felixfontein I love these new fast tests!
ISSUE TYPE
COMPONENT NAME
docker_swarm_service