Skip to content
This repository has been archived by the owner on Feb 25, 2020. It is now read-only.

quick start instruction #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

quick start instruction #3

wants to merge 1 commit into from

Conversation

zimbatm
Copy link
Contributor

@zimbatm zimbatm commented Aug 15, 2018

Use docker volumes to keep the data across restarts. Fixes out of sync
issues between RocksDB and Postgres.

Tell the webapp about the importer address.

@Profpatsch
Copy link
Contributor

Profpatsch commented Aug 16, 2018

> http --verify=no https://172.18.0.1:8080/api/healthcheck
HTTP/1.1 200 OK
Connection: keep-alive
Content-Length: 19
Content-Type: application/json
Date: Thu, 16 Aug 2018 14:52:29 GMT
Server: restify

{
    "version": "0.4.0"
}

They don’t output anything for / or /api apparently.

Now the only question is how to handle TLS; Right now you get an error and have to ignore it.

To build and start the backend run these commands:

```sh
docker load -i "$(nix-build importer/ -A dockerImages.testnetBlockchainImporter --argstr gitrev "undefined" --no-out-link)"
Copy link
Contributor

Choose a reason for hiding this comment

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

the --argstr gitrev "undefined" is unnecessary once input-output-hk/project-icarus-importer#2 is merged (and the submodules here are updated).

Copy link
Collaborator

Choose a reason for hiding this comment

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

As input-output-hk/project-icarus-importer#2 got merged, should we update the submodule and delete the argstr?

README.md Outdated
docker-compose up -d
```

It will start the database, importer and webapp. The API is now reachable at <https://localhost:8080/>.
Copy link
Contributor

Choose a reason for hiding this comment

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

Mention that it will throw certificate warnings right now (because it’s self-signed), maybe have the user test with curl --insecure.

- >-
NODE_CONFIG={
"server": {
Copy link
Contributor

Choose a reason for hiding this comment

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

I’d add "port": 8080 inside the "server" object to leave nothing to chance 🙂

@zimbatm zimbatm changed the title WIP: quick start instruction quick start instruction Aug 16, 2018
Use docker volumes to keep the data across restarts. Fixes out of sync
issues between RocksDB and Postgres.

Tell the webapp about the importer address.
@roccomuso
Copy link

I'd add that a git clone --recursive is also required.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants