-
Notifications
You must be signed in to change notification settings - Fork 5
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
remove news-search-api from story-indexer so we can independently manage deployment #268
Comments
|
Draft PR here, pending clarifications below:
|
since we require VPN access to ssh into the angwin cluster, can we inquire if we can make a provision for deployments via GH Actions. There is no direct mention of VPN support for Github hosted runners, so that may mean we need some kind of customizations around this.
Considering that the major secret we are trying to protect is the
sentry DSN, (at worst, an annoyance if stolen and spammed?) that doesn't
seem like a great trade-off against credentials that could compromise
the campus network and our cluster!
It's possible the CS support folks might have a suggestion on if/how
this has been dealt with by other projects...
|
Short term deployment plan for now is to use a private repo with a Docker(?) config file, like we do for story-indexer. |
Background:
story-indexer uses a shell script (deploy.sh) that generates a JSON
file with parameters (in the script and from private config files)
based on the currently checked out branch (production, staging,
other), and generates a docker stack name and tag based on the branch.
The JSON file is passed to (command line) jinja2 to process
docker-compose.yml.j2 template, to create docker-compose.yml which is
then used to build, tag and push an image, and then "compose" the
stack, and apply a newly generated tag to the image, the source repo
and the config repo for BOTH staging and production deployments. The
template file means that there is only one compose file to maintain.
I don't _THINK_ there is any way to substitute values into a
docker-compose.yml file. There IS an idea of secrets in the docker
compose universe, but I have no knowledge of the pain/benefit
equation. My preference is to have configuration under revision
control, so we can examine past changes, and revert to known good
image/configuration combinations.
The news-search-api case is different in that the image is being built
on github (whenever a tag is applied?) and available from an image
registry. For something like the indexer deploy script, I might have
the new script take a previosly applied image/source tag name as
input. If the tag ends in "bNNN", generate a staging stack (stack
name and config). If the tag name already exists in the config, check
it out and use it, if not, apply the tag at the head of config repo???
|
News-search API deployemnt Resolved in mediacloud/news-search-api#66 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The goal is to pull our news-search-api so that we can manage deployment of it independently from the overall story-index.
Related mediacloud/news-search-api#27, but tracking on here so we can see it in the same place as other tasks.
The text was updated successfully, but these errors were encountered: