Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Doc updates for Create VCH Wizard #1259

Merged
merged 40 commits into from
Jan 17, 2018
Merged

WIP: Doc updates for Create VCH Wizard #1259

merged 40 commits into from
Jan 17, 2018

Conversation

stuclem
Copy link
Contributor

@stuclem stuclem commented Dec 13, 2017

Updating and reorganizing the docs for VCH creation to include the new Create VCH wizard.

Introducing the Create VCH Wizard has led me to completely reorganize and redo how we document VCH creation, in such as way as both the wizard and the CLI methods are documented, but avoiding duplication of information that is applicable to both methods.

This is a big reorg, so I think that the best bet is to review the generated HTML content on github.io rather than in this PR. And also probably in chunks.

The whole of the Deploy VCHs chapter has been reorg'd.

https://stuclem.github.io/vic-product/assets/files/html/1.3/vic_vsphere_admin/deploy_vch.html

TBD: Add table of topics/reviewers, and add the list of issues fixed in this PR.

Copy link
Member

@zjs zjs left a comment

Choose a reason for hiding this comment

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

Feedback on Compute Capacity (it's basically just the same comment, applied in a few places).


#### Create VCH Wizard

In the **CPU** text box, leave the default value of `Unlimited`, or optionally enter a limit of between 1 and 9779 MHz.
Copy link
Member

Choose a reason for hiding this comment

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

9779 MHz

I don't think this is a global maximum; I suspect this was just the maximum on the system you were looking at, for the cluster/resource pool you had selected.

Perhaps "between 1 and 9779 MHz" could be re-written as "between the minimum and maximum shown" or something along those lines.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed as per your suggestion.


`--cpu`, no short name

Specify a CPU limit value of between 1 and 9779 MHz. If not specified, `vic-machine create` sets the limit to 0 (unlimited).
Copy link
Member

Choose a reason for hiding this comment

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

between 1 and 9779 MHz

I don't think this applies to the CLI.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed


#### Create VCH Wizard

In the **Memory** text box, leave the default value of `Unlimited`, or optionally enter a limit of between 1 and 21765 MB.
Copy link
Member

Choose a reason for hiding this comment

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

As above; the minimum/maximum values shown in the UI are based on choices the user made earlier in the wizard.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed


`--memory`, `--mem`

Specify a limit of between 1 and 21765 MB. If not specified, `vic-machine create` sets the limit to 0 (unlimited).
Copy link
Member

Choose a reason for hiding this comment

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

As above; I don't think we provide minimum/maximum guidance for the CLI.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

#### Create VCH Wizard

1. Expand **Advanced**.
2. In the **CPU reservation** text box, leave the default value of 1, or optionally enter a limit of between 1 and 9779 MHz.
Copy link
Member

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed


`--cpu-reservation`, `--cpur`

Specify a limit of between 1 and 9779 MHz. If not specified, `vic-machine create` sets the reservation to 1.
Copy link
Member

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

#### Create VCH Wizard

1. Expand **Advanced**.
2. In the **Memory reservation** text box, leave the default value of 1, or optionally enter a limit of between 1 and 21153 MB.
Copy link
Member

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed


`--memory-reservation`, `--memr`

Specify a limit of between 1 and 21153 MB. If not specified, `vic-machine create` sets the reservation to 1.
Copy link
Member

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

#### Create VCH Wizard

1. Expand **Advanced**.
2. In the **Memory** text box, leave the default value of 2048 MB, or enter a value of between 1 and 21765 MB.
Copy link
Member

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed


`--endpoint-memory `, no short name

Specify a value of between 1 and 21765 MB. If not specified, `vic-machine create` sets memory to 2048 MB.
Copy link
Member

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Member

@zjs zjs left a comment

Choose a reason for hiding this comment

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

4. Click **Finish** to deploy the VCH.
5. In the **Virtual Container Hosts** tab, click the **>** icon next to the new VCH to follow its deployment progress.

At the end of a successful deployment, the deployment log shows connection details for this VCH, and a success message.
Copy link
Member

Choose a reason for hiding this comment

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

Wizard-based deployments don't show connection details as a part of the deployment log (but that information is available in the list view/table).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed to "At the end of a successful deployment, you can see the connection details for this VCH in the Virtual Container Hosts view."

Copy link
Contributor

@anchal-agrawal anchal-agrawal left a comment

Choose a reason for hiding this comment

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


## Example `vic-machine` Commmand <a id="example"></a>
Specify a value in GB or MB. If not specified, `vic-machine create` sets the image size to 8 MB.
Copy link
Contributor

Choose a reason for hiding this comment

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

... sets the image size to 8 MB.

This should be 8 GB.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

Copy link
Contributor

@anchal-agrawal anchal-agrawal left a comment

Choose a reason for hiding this comment

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

Virtual Container Host Storage Capacity - the main page LGTM. The Specify Volume Datastores sub-topic is pending review from @matthewavery.

Copy link
Member

@zjs zjs left a comment

Choose a reason for hiding this comment

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


If you specify `--tls-cname`, you can optionally specify `--organization`. If not specified, `vic-machine create` uses the name of the VCH as the `organization` value.

**NOTE**: If you specify a static IP address on the client network, the `client-ip-address` is used for `CommonName` but not for `Organisation`.
Copy link
Member

Choose a reason for hiding this comment

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

Organisation

nit-pick: should be Organization

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed. Old British English habits die hard....

@stuclem
Copy link
Contributor Author

stuclem commented Jan 15, 2018

Thanks @zjs and @anchal-agrawal. I integrated your comments. All good now?

@stuclem stuclem removed the request for review from jak-atx January 15, 2018 11:58
@matthewavery
Copy link

Spent some time reading it, Looks good! I did not see anything that needs changing.

@stuclem
Copy link
Contributor Author

stuclem commented Jan 17, 2018

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.

10 participants