Skip to content

Do not truncate ID on docker service ps --quiet#579

Merged
dnephin merged 1 commit intodocker:masterfrom
thaJeztah:dont-trunc-on-quiet
Oct 2, 2017
Merged

Do not truncate ID on docker service ps --quiet#579
dnephin merged 1 commit intodocker:masterfrom
thaJeztah:dont-trunc-on-quiet

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Sep 30, 2017

fixes #543

Running docker service ps --quiet should print the
full, non-truncated ID, even if the --no-trunc option
is not set.

This patch disables truncation if the --quiet flag
is set.

- What I did

Updated the code to override options.noTrunc if options.quiet is set

- How to verify it

Run docker service ps <servicename> and verify that the full, non-truncated ID is printed.

- Description for the changelog

Print non-truncated task ID when using `docker service ps --quiet` [docker/cli#579](https://github.com/docker/cli/pull/579)

ping @stevvooe PTAL

Running `docker service ps --quiet` should print the
full, non-truncated ID, even if the `--no-trunc` option
is not set.

This patch disables truncation if the `--quiet` flag
is set.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 30, 2017

Codecov Report

Merging #579 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #579      +/-   ##
==========================================
+ Coverage   49.38%   49.41%   +0.03%     
==========================================
  Files         208      208              
  Lines       17178    17181       +3     
==========================================
+ Hits         8483     8490       +7     
+ Misses       8262     8259       -3     
+ Partials      433      432       -1

Copy link
Copy Markdown
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐸

Copy link
Copy Markdown
Contributor

@dnephin dnephin left a comment

Choose a reason for hiding this comment

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

LGTM

@dnephin dnephin merged commit adfc2d1 into docker:master Oct 2, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.11.0 milestone Oct 2, 2017
@thaJeztah thaJeztah deleted the dont-trunc-on-quiet branch October 2, 2017 15:00
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.

task ids should not be shortened

5 participants