Add log-driver and options to service inspect "pretty" format

[RahulZoldyck]

What I did

Fixes issue #1943 docker inspect not having log information

How I did it

Added template for Log Driver information in serviceInspectPrettyTemplate

How to verify it

docker service create --log-driver=json-file --log-opt max-file=5 --name loggie nginx:alpine
docker service inspect --format=pretty loggie

Description for the changelog

Added Log Driver information in the prettify template.

[GordonTheTurtle]

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "1943-bug-fix" git@github.com:RahulZoldyck/cli.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

[codecov-io]

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1950   +/-   ##
=========================================
  Coverage          ?   56.74%           
=========================================
  Files             ?      310           
  Lines             ?    21808           
  Branches          ?        0           
=========================================
  Hits              ?    12375           
  Misses            ?     8518           
  Partials          ?      915

[RahulZoldyck]

@GordonTheTurtle requesting review for the PR

[thaJeztah]

Whoops, this one dropped off my list.

Tested this, and looks good:

ID:		ix149209ecm256o1ty9b5zw06
Name:		loggie
Service Mode:	Replicated
 Replicas:	1
Placement:
UpdateConfig:
 Parallelism:	1
 On failure:	pause
 Monitoring Period: 5s
 Max failure ratio: 0
 Update order:      stop-first
RollbackConfig:
 Parallelism:	1
 On failure:	pause
 Monitoring Period: 5s
 Max failure ratio: 0
 Rollback order:    stop-first
ContainerSpec:
 Image:		nginx:alpine@sha256:08a230429c2d27b8a5668163f3c20e73e6bba6aad4796aaea90372fbaebca122
 Init:		false
Log Driver:
 Name:		json-file
 LogOpts:
  max-file:       5

Resources:
Endpoint Mode:	vip

[thaJeztah]

LGTM

[thaJeztah]

ping @silvin-lubecki PTAL

[silvin-lubecki]

I wonder if we need/want those empty lines in the template? 🤔

[thaJeztah]

Ah, yes, it's a bit redundant as we have the indentation already (at least inconsistent). Let me clone this one and update that

[thaJeztah]

rebased, squashed, and removed the newlines

[silvin-lubecki]

LGTM

[silvin-lubecki]

Thank you @RahulZoldyck for your contribution!! 👍