<p><b>@cbeams</b> approved this pull request.</p>

<p>utACK, but with a few things that would still be good to fix up.</p>
<p>I've added a link from the main README to the Docker README so it's a bit more discoverable.</p>
<p>I've added a couple nits inline, fix them if you have a chance. The naming stuff would be good to get right in any case. The looping / restarting stuff I'll leave to you.</p>
<p>I'll merge this soon in any case. Thanks.</p><hr>

<p>In <a href="https://github.com/bisq-network/bisq-pricenode/pull/12#discussion_r195043737">docker/Dockerfile</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,27 @@
+###
+# The directory of the Dockerfile should contain your 'hostname' and 'private_key' files.
+# In the docker-compose.yml file you can pass the ONION_ADDRESS referenced below.
+###
+
+# pull base image
+FROM openjdk:8-jdk
+
+RUN apt-get update && apt-get install -y --no-install-recommends \
+    vim \
+    tor \
+    fakeroot \
+    sudo \
+    gradle \
</pre>
<p>Nit: It's not necessary to install <code>gradle</code> given that everything is done with the <code>./gradlew</code> wrapper script below. This is basically just a waste of time.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq-pricenode/pull/12#discussion_r195043953">docker/README.md</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,43 @@
+Needed information to start a priceprovider
</pre>
<p>Nit: It would be good to use the name "pricenode" consistently now. "provider" language still exists on the core / desktop side, but everywhere here in bisq-pricenode land it's "pricenode" now.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq-pricenode/pull/12#discussion_r195044785">docker/torrc</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,192 @@
+## Configuration file for a typical Tor user
+## Last updated 9 October 2013 for Tor 0.2.5.2-alpha.
+## (may or may not work for much older or much newer versions of Tor.)
+##
+## Lines that begin with "## " try to explain what's going on. Lines
+## that begin with just "#" are disabled commands: you can enable them
+## by removing the "#" symbol.
+##
+## See 'man tor', or https://www.torproject.org/docs/tor-manual.html,
+## for more options you can use in this file.
+##
+## Tor will look for this file in various places based on your platform:
+## https://www.torproject.org/docs/faq#torrc
</pre>
<p>I'd prefer a minimal <code>torrc</code> with all the boilerplate comments, etc stripped out. See the top-level <code>torrc</code> in this repo for an example.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq-pricenode/pull/12#discussion_r195045466">docker/loop.sh</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,8 @@
+#!/bin/bash
+while true
+do
+echo `date`  "(Re)-starting node"
+BITCOIN_AVG_PUBKEY=$BTCAVERAGE_PUBKEY BITCOIN_AVG_PRIVKEY=$BTCAVERAGE_PRIVKEY java -jar ./build/libs/bisq-pricenode.jar 1 2 2
+echo `date` "node terminated unexpectedly!!"
+sleep 3
+done
</pre>
<p>I'm not sure what led to the need to restart price providers in the past, but this is a completely re-written app, where instances have been quite stable in production. I'm not sure we need this kind of loop script at all, and if we do, I'd like to know about it and actually root out the problem rather than just reboot stuff every so often. I won't push too hard on this, but I'd rather keep things as simple as possible for a start. Feels like we're just copying and pasting what we did prior to the rewrite here, and it's not obvious to me that we need to.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq-pricenode/pull/12#discussion_r195045589">docker/docker-compose.yml</a>:</p>
<pre style='color:#555'>> @@ -0,0 +1,21 @@
+version: '3'
+
+# Fill in your own BTCAVERAGE public and private keys
+
+services:
+    priceprovider:
+        restart: always
+        build:
+            context: .
+        image: bisq:priceprovider
</pre>
<p>Same thing here with naming, not sure about conventions in Docker image naming. Can this be <code>bisq:pricenode</code> instead?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq-pricenode/pull/12#pullrequestreview-128329787">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtmg249xcfhwBQlfY87js_eaP19Nnks5t8PZ5gaJpZM4UlIeQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtp1L0DZosgIudFpcaj0roXtjEOinks5t8PZ5gaJpZM4UlIeQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/bisq-network/bisq-pricenode/pull/12#pullrequestreview-128329787","url":"https://github.com/bisq-network/bisq-pricenode/pull/12#pullrequestreview-128329787","name":"View Pull Request"},"description":"View this Pull Request on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}</script>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq-pricenode","title":"bisq-network/bisq-pricenode","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq-pricenode"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cbeams approved #12"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-pricenode/pull/12#pullrequestreview-128329787"}}}</script>
<script type="application/ld+json">{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "@cbeams approved 12",
"sections": [
{
"text": "utACK, but with a few things that would still be good to fix up.\r\n\r\nI've added a link from the main README to the Docker README so it's a bit more discoverable.\r\n\r\nI've added a couple nits inline, fix them if you have a chance. The naming stuff would be good to get right in any case. The looping / restarting stuff I'll leave to you.\r\n\r\nI'll merge this soon in any case. Thanks.",
"activityTitle": "**Chris Beams**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@cbeams",
"facts": [

]
}
],
"potentialAction": [
{
"targets": [
{
"os": "default",
"uri": "https://github.com/bisq-network/bisq-pricenode/pull/12#pullrequestreview-128329787"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 345278352\n}"
}
],
"themeColor": "26292E"
}</script>