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

Add Licensing information to operations guide #405

Merged
merged 5 commits into from
Jul 31, 2017

Conversation

seglo
Copy link
Contributor

@seglo seglo commented Jul 27, 2017

Information on how to license the ConductR clusters. Tips for different types of cluster environments. Caveats of the licensing process.

I have not tested to see how this markdown looks rendered with play-doc. Can someone point me to directions on how to use this documentation tool?

`commercial.credentials` it’s important to use the email associated with your organization's license, and not a personal
account. If enterprise environments this is usually your corporate email.
2. Download your `license` file manually. You can find your license by visiting
https://www.lightbend.com/product/conductr/license
Copy link
Contributor

Choose a reason for hiding this comment

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

We can use https://www.lightbend.com/account/license instead now. It provides the copy button, like access-token.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@edwardcallahan Cool. Why does it say "Once you've copied the above license to your clipboard then paste it into the prompt that caused you to come here." What prompt asks you for the license?

Copy link
Contributor

Choose a reason for hiding this comment

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

That particular page was created for the HPE Fortify Scala plugin feature. The verbiage on that page can certainly be improved. Particularly now that we've added another use-case.

Copy link
Contributor

@edwardcallahan edwardcallahan left a comment

Choose a reason for hiding this comment

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

Thanks so much for writing this up, @seglo ! Appreciate the contribution.
Two change requests, please.
WRT the rendering. The MD preview is sufficient, thanks. Also note that we are moving to tech hub for doc this fall.

when starting your cluster in the same manner you plan to license it. You should load all required support bundles
(`visualizer`, `haproxy`, `continuous-delivery`, etc.) as well as your latest running app bundles.
- While there is at least one node of the cluster up, the license will be intact.
- Future versions of ConductR will persist the license to disk in the cluster.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's note the backup and restore feature, typesafehub/conductr-cli#517, instead. For the immediate future, users preparing for full data center shutdown recovery should have their license and bundle backups in the plan. When/If CRDT state is persisted, the two are a shared concern. Or, a licensed but empty post-disaster cluster isn't very useful.

Copy link
Contributor Author

@seglo seglo Jul 28, 2017

Choose a reason for hiding this comment

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

@edwardcallahan I can remove the notes about reloading bundles and cluster state. Maybe the cluster backup stuff should be mentioned in another section and not Licensing. WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed

@seglo
Copy link
Contributor Author

seglo commented Jul 28, 2017

Thanks @edwardcallahan . I made the changes we agreed to as well as misc. wording fixes. Please review.

@seglo seglo changed the title Add Licensing information to operations guide -WIP Add Licensing information to operations guide Jul 28, 2017
@seglo
Copy link
Contributor Author

seglo commented Jul 31, 2017

Removed single line breaks after previewing page with play-doc. play-doc includes single line breaks in the rendered page.


If you need to update the license on an already licensed cluster you can do so using the `--force` command switch. This
is useful when you want to change the license to a different user or when your license is close to expiring. Enter
`conduct load-license --offline` and the command line and then follow the prompts to load a new license.

Choose a reason for hiding this comment

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

Did you mean to write conduct load-license --force here ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Thanks @patrick-premont .

it's first started up.

Remember that licenses expire, so set a reminder to occassionally retrieve an updated `./.conductr/license` file for
your deployment script.

Choose a reason for hiding this comment

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

It might be good to add a sentence here like "License expiry does not affect running bundles but would prevent the execution of further ConductR operations, such as new bundle deployments."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's interesting. I've never tested this scenario. Have you confirmed this with the Enterprise Suite team?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@edwardcallahan edwardcallahan Jul 31, 2017

Choose a reason for hiding this comment

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

Agents need a license to have resource offers be usable/accepted. There's nothing that will stop bundles from running the moment the currently installed license expires. Once the license expires, no new agents can join the cluster as licensed members.

If, post expiry, an agent restarts for example, the new agent for the node will not validate and no bundle instances will be started on that agent's node. If the node host had reset, existing bundle load was also lost and now is no longer reschedulable on that agent node as that agent is not licensed.

Patrick is correct, if nothing happens to your cluster: nothing starts, stops, restarts, nothing; then nothing will happen to your bundles either.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@edwardcallahan thanks. To confirm, if you were to deploy a bundle into an expired cluster, would the agents that joined the cluster before expiry still be able to deploy and run the bundle?

Copy link
Contributor

Choose a reason for hiding this comment

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

@seglo I don't see/know of anything that clearly would prevent the new scale request from being fulfilled, provided said agents have sufficient resources in their offers. We want you to be successful and we have no motivation to stop existing bundles in the future, for example. Beyond that it is difficult to provide warranty for future behavior.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok. I feel like this is getting pretty deep into the weeds. Given we're not 100% about what happens to newly deployed bundles would this statement be satisfactory?

An existing ConductR cluster and its running bundles will continue to run when the license expires. New agents, or agents that try to rejoin after a failure, will not be able to join the cluster.

@edwardcallahan @patrick-premont

Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM
Thanks @seglo

@edwardcallahan edwardcallahan merged commit 2050af9 into typesafehub:master Jul 31, 2017
@seglo seglo deleted the licensing branch July 31, 2017 20:54
edwardcallahan pushed a commit that referenced this pull request Jul 31, 2017
* Add Licensing information to operations guide

* Remove caveats section an incorporate info into rest of doc. Misc typo's.

* Remove single line breaks

* Fixed typo with load-license --force command

* Notes on cluster behaviour after license expires
huntc added a commit that referenced this pull request Aug 1, 2017
Add Licensing information to operations guide (CherryPick of #405)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants