Updating docs with information on using VMs. Fixes T680
ClosedPublic

Authored by tflink on Jan 14 2016, 10:11 PM.

Details

Summary

Adding simple information on using VMs with runtask

Test Plan

look at docs to make sure information is accurate and they render properly

Diff Detail

Repository
rLTRN libtaskotron
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
tflink retitled this revision from to Updating docs with information on using VMs. Fixes T680.Jan 14 2016, 10:11 PM
tflink updated this object.
tflink edited the test plan for this revision. (Show Details)
tflink added reviewers: mkrizek, kparal, jskladan.

Bah, I managed to use the wrong ticket in the commit message. Will fix when squashing and pushing to develop

mkrizek accepted this revision.Jan 15 2016, 9:49 AM

LGTM, one minor nit though

docs/source/index.rst
120

I think we should mention the --local option somewhere which might be confusing for some.

This revision is now accepted and ready to land.Jan 15 2016, 9:49 AM
tflink added inline comments.Jan 18 2016, 6:48 PM
docs/source/index.rst
120

Do you think that it needs to be mentioned on the main page if the default execution mode is local if not in production?

kparal accepted this revision.Jan 19 2016, 11:12 AM
kparal added inline comments.
docs/source/index.rst
107

Maybe "using virtual machines or remote machines"?

111

"virtual/remote"?

112

"single use (disposable)"? Because we use that terminology elsewhere.

114

Here I'd swap "virtual" for "remote".

118

And here I'd just use "disposable virtual machine".

mkrizek added inline comments.Jan 19 2016, 8:24 PM
docs/source/index.rst
120

OK, I take that back, for some reason I thought there is some hidden black magic behind --local, there isn't.

tflink updated this revision to Diff 1869.Jan 20 2016, 6:04 PM
  • slight changes in terminology

Looks good.

This revision was automatically updated to reflect the committed changes.