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

Doc/lepidopter services #68

Merged
merged 8 commits into from
Nov 11, 2016
Merged

Doc/lepidopter services #68

merged 8 commits into from
Nov 11, 2016

Conversation

anadahz
Copy link
Contributor

@anadahz anadahz commented Sep 8, 2016

This PR is based in #60 that contains important documentation and fixes.

that supports automatic discovery of hosts in the local network.

The applications that are bundled with the Bonjour software in Windows:

Copy link
Member

Choose a reason for hiding this comment

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

Include a download link to install the software and steps on how a windows user can then use bonjour once it's installed.

Copy link
Contributor Author

@anadahz anadahz Sep 8, 2016

Choose a reason for hiding this comment

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

Do we really need to add links for these applications that will most probably change in the future?

Copy link
Member

Choose a reason for hiding this comment

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

I think we should say:

If you have any of the following applications installed, then you don't need to do anything. However if you don't then install XXX, where XXX probably should be the Bonjour Print services.

If the links to them change in the future we should update them, but it's important that we give some instructions that a newbie can follow to access their pi.

@anadahz
Copy link
Contributor Author

anadahz commented Sep 13, 2016

I have updated the docs based on feedback from @hellais .

@anadahz
Copy link
Contributor Author

anadahz commented Sep 19, 2016

@hellais I made the changes in aaabcee for 012ed0b and also updated the documentation to the reflect the changes of newer lepidopter image build.

(https://get.ooni.torproject.org/lepidopter/lepidopter-v0.3.5-beta-armel.img.xz)
for a significantly reduced file size compared to the zip archive.
Note that this will require an extra program to be installed depending on your
OS.
Copy link
Contributor

Choose a reason for hiding this comment

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

What do you think about [extra program](http://tukaani.org/xz/) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are right it makes sense to add this.

```

You should look for the message `Good signature from "Lepidopter Team (signing
key)"`:

```
gpg: assuming signed data in `lepidopter-alpha-armel.img.zip'
gpg: assuming signed data in `lepidopter-v0.3.5-beta-armel.img.zip'
gpg: Signature made Sun 22 May 2016
gpg: using RSA key 0xBA56AC5A53E9C7A4
gpg: using PGP trust model
Copy link
Contributor

Choose a reason for hiding this comment

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

  • I see trailing newline in the paste after gpg: binary signature, digest algorithm SHA512
  • what about using full fingerprint like --recv-keys 625511968E240F24F6CFD0B6BA56AC5A53E9C7A4 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ack!

@@ -266,7 +266,7 @@ the name of the whole SD card as described above, not just a partition of it
(for example, sdd, not sdds1 or sddp1, or mmcblk0 not mmcblk0p1).

```
dd bs=4M if=~/lepidopter-alpha-armel.img of=/dev/sdd
dd bs=4M if=~/lepidopter-v0.3.5-beta-armel.img of=/dev/sdd
Copy link
Contributor

@darkk darkk Oct 19, 2016

Choose a reason for hiding this comment

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

Doesn't one need sudo to run dd against SD card?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The user will be prompted with a Permission denied error in case of access rights.
As someone can destroy a hard disk with dd is better to understand what the command does.


`xz --decompress --verbose --no-sparse lepidopter-alpha-armel.img.zip`
`xz --decompress --verbose --no-sparse lepidopter-v0.3.5-beta-armel.img.zip`
Copy link
Contributor

Choose a reason for hiding this comment

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

  • xz can't decompress zip
  • Why --no-sparse ? Saving 3Gb of zeros looks like a good idea to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I recall some issues with different version of dd not parsing correctly sparse files.

archive. Note that this will require an extra program to be installed depending
on your OS.
(https://get.ooni.torproject.org/lepidopter/lepidopter-v0.3.5-beta-armel.img.xz)
for a significantly reduced file size compared to the zip archive.
Copy link
Contributor

Choose a reason for hiding this comment

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

IMHO exact sizes are better.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was hoping to have a link pointing to the latest image file and having inaccurate exact file sizes numbers is not nice.

@anadahz
Copy link
Contributor Author

anadahz commented Oct 20, 2016

@darkk thanks for your comments, add fixes in 317a8bb.

@hellais
Copy link
Member

hellais commented Oct 25, 2016

So I have thought a bit about this and I have the following feedback.

I think the installation instructions for lepidopter should be restructured to take into account the fact that there are two types of users of lepidopter (or maybe even 3):

  1. A user of lepidopter that has gotten it via our partnership program. These from what I can tell are actually the majority of lepidopter users and are people that don't care how to burn the image to an SD card. This user is not technically savvy.
  2. A user of lepidopter that wants to download the image and burn it onto their SD card. These are a minority of users that are quite technically savvy.
  3. (if we include this as well) Power users or developers that want to contribute in some way to the project. Very technically savvy.

In light of this I think the guide should be divided into at least 2 sections.

  • User guide
  • How to burn an image

Inside the user guide there should be clear and simple explanations as to how to connect the raspberry pi to the power and internet (maybe even including screenshot). @andresazp showed me the document he created for VI and it looks great.
It should also cover information on how to initialise the pi (answering the informed consent) and check that measurements are being run, by visiting the web UI.

An outline of the User Guide could be:

  • Setting up the device (covering how to connect it to internet and power, putting in the SD card etc.)
  • First start (covering how to reach the pi over the network, using lepidopter or lepidopter.local) and answering the informed consent
  • Checking that everything works (checking that measurements are being run)

The How to burn an image can remain as the current section titled: "Downloading and verifying the Lepidopter Raspberry Pi image"

I would also try to make this guide overall much shorted and where possible point to external resources instead of having so much content in here (for example explaining the tests).

@hellais
Copy link
Member

hellais commented Oct 25, 2016

I am also thinking that maybe the whole part on burning the image could be tucked inside of another page entirely as it makes the whole guide much less user friendly.

Another comment is that the whole section "Lepidopter services" is far too technical and contains a bunch of information that will not be useful to most cases (why would the normal user care to know that there is a mDNS service for SSH?).

I would tuck this stuff as well inside of some advanced section and try to have a minimal guide that is easy for most users to follow.

The experience we had with real deployment of these is that even just explaining how to plug in a pi into a router and connecting to power can be challenging for some non technically savvy users and since this guide is addressed to them we should take that into account.

@anadahz
Copy link
Contributor Author

anadahz commented Oct 31, 2016

@hellais your comments:#68 (comment) and #68 (comment) make sense but they will be better addressed in another PR as we need to make the release announcement of ooniprobe v2.0.0 and WUI (#76) ASAP since the users are still using older versions of lepidopter and ooniprobe.

@hellais
Copy link
Member

hellais commented Nov 11, 2016

Filed an issue with my feedback.

Merging this into the website.

@hellais hellais merged commit 0cbccd8 into master Nov 11, 2016
@hellais hellais deleted the doc/lepidopter-services branch November 11, 2016 16:54
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