Skip to content
This repository has been archived by the owner on Sep 30, 2021. It is now read-only.

Add setup instructions for read only plugin, refs #69 #72

Merged
merged 1 commit into from
Oct 30, 2018

Conversation

La0
Copy link
Contributor

@La0 La0 commented Sep 19, 2018

No description provided.

@La0 La0 self-assigned this Sep 19, 2018
@La0 La0 requested a review from klahnakoski September 19, 2018 14:31
@klahnakoski
Copy link
Contributor

Will the ActiveData service be blocked with this config? Can this still be behind nginx? Should this have its own node, maybe its own docker image, like as tested?

@La0
Copy link
Contributor Author

La0 commented Sep 20, 2018

With this configuration, everything is allowed on localhost, so ActiveData should be fine as its gunicorn process queries on localhost (right ?).
This plugin is only needed on the frontend (or any other instance publicly exposed).

It should work as-is behing an Nginx server, but i do not recommend to add another layer in front of this server.

It could also run on another server, on docker or bare-metal, but must be included in the cluster (of course !)

@klahnakoski
Copy link
Contributor

klahnakoski commented Sep 24, 2018

Sorry for the delay. I think we just need a time window to deploy this. The ETL pipeline uses this node, and its requests come from a particular subnet (172.31.2.X), same with TUID machine and a management machine https://github.com/mozilla/ActiveData/blob/dev/docs/Architecture.png

@klahnakoski
Copy link
Contributor

Thinking more, given the number of moving parts, I am tending toward a separate machine holding this endpoint; which can be deployed without shutting down the frontend node.

@klahnakoski
Copy link
Contributor

The TUID project has its own node, so complications and crashes are isolated to itself.

https://github.com/mozilla/TUID/blob/dev/resources/scripts/setup_server.sh

@La0 How much drivespace would you need on a standalone machine? The node will not store data, so the ES only needs space for logging.

@La0
Copy link
Contributor Author

La0 commented Sep 25, 2018

@klahnakoski No problem for using the TUID project, as long as the server has access to the codecoverage and repo indexes.

The server should not need any data except for logs, and storing the plugin during install (14 Mb)

@klahnakoski
Copy link
Contributor

@La0, when you have the machine working, please link the setup script here, or put the script in /resources/scripts/setup_es_codecoverage_t2.small.sh, or something like that

@klahnakoski klahnakoski merged commit 039980c into mozilla:dev Oct 30, 2018
@klahnakoski
Copy link
Contributor

Sorry for the delay, I must not be getting notifications from this repo.

This looks fine to me

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.

2 participants