Skip to content
This repository has been archived by the owner on Jun 24, 2020. It is now read-only.

docs: add minimal usage info #5

Merged
merged 2 commits into from
Mar 10, 2017
Merged

docs: add minimal usage info #5

merged 2 commits into from
Mar 10, 2017

Conversation

euank
Copy link
Contributor

@euank euank commented Mar 9, 2017

No description provided.

euank
euank commented Mar 9, 2017
View reviewed changes
Documentation/using-docker-skim.md Outdated
Type=simple

ExecStart=/usr/bin/rkt run --dns=host --interactive \
--insecure-options=image \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This line should be removed before merging; the signatures aren't available yet due to complications.

crawford
crawford approved these changes Mar 9, 2017
View reviewed changes
Copy link

@crawford crawford left a comment

Choose a reason for hiding this comment

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

Overall, looks good.

Documentation/using-docker-skim.md Outdated
Override the existing unit by placing it at the path
`/etc/systemd/system/docker.service`.

If this is done on a running machine, rather than during provisioning via ignition, first stop the existing docker and containerd units.
Copy link

Choose a reason for hiding this comment

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

Ignition (capitalized)

@euank euank merged commit abfca74 into coreos:master Mar 10, 2017
@euank euank deleted the basic-usage-docs branch March 10, 2017 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@crawford crawford crawford approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@euank @crawford