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

adding dss deployment on EC2 #302

Merged
merged 6 commits into from
Dec 5, 2024
Merged

adding dss deployment on EC2 #302

merged 6 commits into from
Dec 5, 2024

Conversation

carlosbravoa
Copy link
Contributor

No description provided.

Copy link
Collaborator

@k-dimple k-dimple left a comment

Choose a reason for hiding this comment

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

Thanks for this Carlos! There are just some minor typos that I have corrected. If the suggestions seem acceptable, just use the 'Add suggestion to batch' button on each of them to create a single commit for all the changes.

Also, to avoid the build errors, add the following list of words to the ".custom_wordlist.txt" file: DSS, Jupyter, MLflow, PuTTy, WSL

aws/aws-how-to/index.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/data-science-stack-on-ec2.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/index.rst Outdated Show resolved Hide resolved
aws/aws-how-to/instances/index.rst Outdated Show resolved Hide resolved
Comment on lines +86 to +103
Install MicroK8s and DSS
------------------------

Install MicroK8s:


.. code::

sudo snap install microk8s --channel 1.28/stable --classic
sudo microk8s enable storage dns rbac
sudo microk8s enable gpu

Install DSS:

.. code::

sudo snap install data-science-stack --channel latest/stable
dss initialize --kubeconfig="$(sudo microk8s config)"
Copy link
Contributor

Choose a reason for hiding this comment

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

i would prefer to reference the official DSS docs for this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

TLDR; I'll leave that to @k-dimple
In a conversation with another team (Anbox), we were discussing how referencing other guides sometimes impacted the experience, as a 1/the official guides tend to be more generic and sometimes confuse the customer with other things, and 2/ the How-to stops to be a straightforward recipe.

The documentation person there told me that they are trying now to have self-sufficient guides for that reason.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah, I agree with the argument. If the whole how-to guide was just about installing DSS, we could have directly pointed to the DSS docs. Here since we are talking about specific EC2 instances and including specific drivers, it is better to give a complete set of instructions and to point them to the DSS docs for further reference. (Carlos has already done this by adding a link in the introduction section.) So I think we can keep it in its current state.

@carlosbravoa
Copy link
Contributor Author

updated

Copy link
Contributor

@rpocase rpocase left a comment

Choose a reason for hiding this comment

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

+1 pending dimple nacking the suggestion for redirecting to other docs

@k-dimple k-dimple merged commit 837bf4f into canonical:main Dec 5, 2024
1 of 9 checks passed
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.

3 participants