Adguard home instructions

[BrokenOnedroid]

To ease and accelerate the PR submission, please use the template below to provide all required information.
Please see our "Contributing to Rockstor documentation" in our Rockstor's documentation

Fixes #
If this is okay to use. The description in the PR can be linked to this Page. And then added to the rockon-registry. (See: rockstor/rockon-registry#397)

This pull request's proposal

Added write up for AdGuard Home Rock-on installation.

Checklist

• With the proposed changes no Sphinx errors or warnings are generated.
• I have added my name to the AUTHORS file, if required (descending alphabetical order).

[phillxnet]

@BrokenOnedroid Thanks for seeing to this doc entry to complement your on-going Rock-on contributions ; much appreciated.

I've just run the Sphinx test build via this repos GitHub Action and we look to have an issue via the following Warning:

https://github.com/rockstor/rockstor-doc/actions/runs/12518381636/job/34940209577?pr=517#step:5:97

/home/runner/work/rockstor-doc/rockstor-doc/interface/docker-based-rock-ons/adguard-home.rst: WARNING: document isn't included in any toctree

Which suggests this pull requests branch is missing a toc (Table Of Content) entry for the page your have added. I.e. there is no menu entry generated for folks to find this new page.

• Take a looks at the other "Rock-ons with write-ups" toc section in: https://github.com/rockstor/rockstor-doc/blob/master/interface/overview.rst

Which in-turn ends-up being html rendered as:

• https://rockstor.com/docs/interface/overview.html#rock-ons-with-write-ups

Also note:

• you are authoring these commits under: https://github.com/abandonedforce
• but the pull request is authored under: https://github.com/BrokenOnedroid

It would likely be better to use only one GitHub account here as it makes attribution easier. I suspect your local development environment is setup with the former: but you likely intend to use the later. We have a docs section with some details on configuring this:

• Local git repo setup: https://rockstor.com/docs/contribute/contribute.html#local-git-repo-setup

Also keep in mind the second tick item: i.e. we like folks to get attribution for their submissions. But your choice of course.

Hope that helps.

[phillxnet]

@BrokenOnedroid My apologies if I've jumped in too early here on this Draft status PR.

[BrokenOnedroid]

@phillxnet thanks for pointing these things out.
I still need to check grammar and so on, its a wip.

I did not realize that my old username is used. I rename the github user sometime ago.
But i don't seem able to get the username in the commits changed. Any idea?

[phillxnet]

But i don't seem able to get the username in the commits changed. Any idea?

I think what is happening is that the username against old commits is not changed by altering the current config. But if you changed your local config and then did this latest commit then you likely also have to modify the associated email. GitHub I think uses the email to track commits. Note the other command in our docs e.g.:

git config user.email your_email_address

Plus upon this Pull Request reaching non-draft status, you can squashing all commits down to one under the new GitHub user and email. A submission will be required but that's not bother. I've taken to doing this myself quite a lot as it makes asking questions / opinions on draft work easier. And once all is ready one can then just squash and remove the remote (GitHub) branch and push again in the new squashed (single commit) status. That will then hopefully have the new user against the commit also.

Check your two GitHub accounts and compare the emails there-in. The PR is down OK as having been submitted by @BrokenOnedroid but the commits are all under that old account. So I'm guessing there is an email association that has precedence.

Feel free to experiment here in this PR as it will likely be squashed into a single commit and resubmitted later anyway given this is an example of a shared in-development branch: with the appropriate Draft status so we can all appreciate the in-development nature: as you say WIP. Looking nice though.

Hope that helps.

[BrokenOnedroid]

From my pov i added everything necessary into the instructions.

make html does not produce errors.
make linkcheck shows errors, but these are HTTPSConnectionPool unrelated to my changes.