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

Write install guide #272

Merged
merged 10 commits into from
Sep 14, 2023
Merged

Write install guide #272

merged 10 commits into from
Sep 14, 2023

Conversation

EduardGomezEscandell
Copy link
Contributor

@EduardGomezEscandell EduardGomezEscandell commented Sep 12, 2023

An install guide to set up Ubuntu Pro for Windows on your local machine or VM.

Tested side-to-side with an Azure VM to verify.


UDENG-1262

@EduardGomezEscandell EduardGomezEscandell self-assigned this Sep 12, 2023
INSTALL.md Outdated
@@ -0,0 +1,108 @@
# Installing Ubuntu Pro For Windows
Copy link
Contributor

@CarlosNihelton CarlosNihelton Sep 12, 2023

Choose a reason for hiding this comment

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

HO: Should this file be in the wiki instead of in the repo?

INSTALL.md Outdated
3. Go to `HKEY_CURRENT_USER\Software\Canonical\UbuntuPro`
4. There are two relevant fields:
- LandscapeAgentURL should contain the URL where the Landscape Hostagent server is hosted.
- LandscapeClientConfig should contain the contents of the yaml file with the settings, such as the one in the example.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we point to the example Landscape config 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.

Ah, forgot to add the link!

Copy link
Collaborator

@didrocks didrocks left a comment

Choose a reason for hiding this comment

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

Some suggestions for changes, it’s a good starting point to have now this documentation for manually playing with it, thanks! :)

INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
INSTALL.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@didrocks didrocks left a comment

Choose a reason for hiding this comment

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

Just a TODO to add IMHO so that we spot what we have to update later on. Otherwise, approved! Nice to have this documentation now :)

INSTALL.md Outdated Show resolved Hide resolved
@EduardGomezEscandell EduardGomezEscandell merged commit 30e9cbf into main Sep 14, 2023
29 of 30 checks passed
@EduardGomezEscandell EduardGomezEscandell deleted the install branch September 14, 2023 06:51
EduardGomezEscandell added a commit that referenced this pull request Sep 14, 2023
Seems like #272 broke the auto updates due to the name changes to the
docs. Also the update-workspace-dependencies was not executable.

I also did a `go work sync` because vscode autoupdates `go.work`,
constantly polluting my working dir. Let's call it gardening 🌱.

---
UDENG-1262
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