Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

(docs): add available docker versions #3631

Merged
merged 1 commit into from
Aug 8, 2018
Merged

Conversation

seanknox
Copy link
Contributor

@seanknox seanknox commented Aug 8, 2018

What this PR does / why we need it: adds available docker versions to the cluster definition doc.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

If applicable:

  • documentation
  • unit tests
  • tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

@ghost ghost assigned seanknox Aug 8, 2018
@ghost ghost added the in progress label Aug 8, 2018
| imageReference.name | no | The name of a a Linux OS image. Needs to be used in conjunction with resourceGroup, below |
| imageReference.resourceGroup | no | Resource group that contains the Linux OS image. Needs to be used in conjunction with name, above |
| osType | no | Specifies the agent pool's Operating System. Supported values are `Windows` and `Linux`. Defaults to `Linux` |
| distro | no | Specifies the agent pool's Linux distribution. Supported values are `ubuntu` and `coreos` (CoreOS support is
Copy link
Contributor

Choose a reason for hiding this comment

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

the * put stuff in italic, I think you need to escape them...

@jackfrancis
Copy link
Member

/lgtm thanks @seanknox!

@jackfrancis
Copy link
Member

(pending @CecileRobertMichon's note :) )

@codecov
Copy link

codecov bot commented Aug 8, 2018

Codecov Report

Merging #3631 into master will decrease coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3631      +/-   ##
==========================================
- Coverage   55.77%   55.49%   -0.28%     
==========================================
  Files         107      107              
  Lines       16238    16195      -43     
==========================================
- Hits         9057     8988      -69     
- Misses       6407     6433      +26     
  Partials      774      774

@seanknox
Copy link
Contributor Author

seanknox commented Aug 8, 2018

@CecileRobertMichon made formatting changes, thanks

Copy link
Contributor

@CecileRobertMichon CecileRobertMichon left a comment

Choose a reason for hiding this comment

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

/lgtm

@acs-bot
Copy link

acs-bot commented Aug 8, 2018

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: CecileRobertMichon, jackfrancis, seanknox

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [CecileRobertMichon,jackfrancis]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@CecileRobertMichon CecileRobertMichon merged commit 7eda9ed into master Aug 8, 2018
@ghost ghost removed the in progress label Aug 8, 2018
@seanknox seanknox deleted the docker-engine-docs branch August 10, 2018 23:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants