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

Zurich lambda polling code #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Zurich lambda polling code #8

wants to merge 2 commits into from

Conversation

matthewberryman
Copy link
Contributor

My thoughts as code (and comments) on bringing in Zurich data as per urbanriskmap/riskmap#32
Note that this includes a more recent and substantive example of data under zurich/test/zurich_example.json
This also starts to address #7 although to fully address that for all Lambdas here I need to refactor things a bit—I know what to do exactly just ran out of time as that also requires some work in Jenkins build scripts and Jenkins configuration. One nice side benefit of using serverless for these is that you can define your rates as code.

@matthewberryman
Copy link
Contributor Author

@talltom I've done some further work on Zurich (around environment variables and documentation) in the new serverless branch for this repo, along with the refactoring towards #7 - a little bit messy but made more sense to keep those changes together.
Suggested workflow:

  1. Review and merge Zurich schema and fns (fns need work) cognicity-schema#16
  2. Review and merge feeds API for Zurich cognicity-server#17 after Merge dev to form v3.0.0 release cognicity-server#18
  3. Review this code and push to this branch if necessary.
  4. I will then merge any changes into my serverless branch and test the Zurich code from here along with Qlue changes, and if the Zurich code works I will close & merge this pull request and then merge serverless branch into master after testing both (unless you want to review that in which case I will open a pull request). These step will also require some work in jenkins.

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

Successfully merging this pull request may close these issues.

2 participants