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

docs: Update cloud-init tutorial for 24.04.1 #500

Merged
merged 5 commits into from
Oct 2, 2024

Conversation

CarlosNihelton
Copy link
Collaborator

With that release comes changes to the distro launcher making it no longer necessary for users to run cloud-init commands. The distro launcher does it, so the tutorial can be made shorter and more direct.

Also there was a caveat about wsl --install that's no longer applicable, as that installation command also goes through the launcher, both workflows are now consistent.

With that release comes changes to the distro launcher making no longer
necessary for users to run cloud-init commands. The distro launcher does
it, so the tutorial can be made shorter.

Also there was a caveat about `wsl --install` that's no longer
applicable, as that installation command also goes through the launcher.
@CarlosNihelton
Copy link
Collaborator Author

CI Link checks are failing spuriously 🤷🏽‍♂️

@CarlosNihelton CarlosNihelton marked this pull request as ready for review October 1, 2024 18:00
@CarlosNihelton CarlosNihelton requested a review from a team as a code owner October 1, 2024 18:00
@CarlosNihelton
Copy link
Collaborator Author

Question for a follow-up PR maybe: shouldn't we refer to this tutorial in the page screenshotted below?

image

@edibotopic
Copy link
Collaborator

Question for a follow-up PR maybe: shouldn't we refer to this tutorial in the page screenshotted below?

The auto-installation should probably just be removed (discussed previously with JB).

As this PR relates to updating the doc it should point to, I think we could remove it here.

Copy link
Collaborator

@edibotopic edibotopic left a comment

Choose a reason for hiding this comment

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

Thanks a lot Carlos. The changes with the new release and the revisions in the tutorial make this story a lot more clear.

I have added some minor comments for you to consider.

The CI seems to be failing because two links are timing out:

Comment on lines 76 to 77
It will register a new Ubuntu-24.04 instance and wait for cloud-init to perform its initialisation tasks.
That process can take several minutes, depending on the computer and network speeds.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would rewrite slightly to:

This command will register a new Ubuntu-24.04 instance that will be configured automatically by cloud-init.
The process can take several minutes, depending on your computer and network speeds.

...................
status: done
```

## Verify that it worked
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would consider a more descriptive title here:

Verify automatic configuration by cloud-init

/home/cn/.hushlogin file.
jdoe@mib:~$
/home/jdoe/.hushlogin file.
jdoe@mib01:~$
```

Once logged into the new distro instance's shell, verify that:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not something that needs to be addressed in this PR, and maybe a matter of taste 😄 but we could include a more compelling configuration setup in future. The functionality is demonstrated clearly but it is not obvious to me what the user is setting up with Octave and ginac-tools. For someone less familiar with auto-configuration or who is seeking a concrete use-case we could help them along with one or more examples for specific scenarios (e.g., data science, asset pipeline, programming in Go...).

@edibotopic
Copy link
Collaborator

Next cycle we need to consider if this tutorial and a few others might be a better fit in the how-to guides section. This, for example, includes concise, step-by-step instructions on how to achieve one specific task and could work better as a how-to.

@CarlosNihelton CarlosNihelton merged commit 17a5564 into main Oct 2, 2024
2 checks passed
@CarlosNihelton CarlosNihelton deleted the update-cloudinit-tutorial branch October 2, 2024 14:02
edibotopic added a commit that referenced this pull request Oct 3, 2024
This removes documentation of the autoinstall feature (deprecated) and
redirects the old url for autoinstall to the cloud-init doc. Follow up
to discussion in #500 .
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.

2 participants