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

Self contained execution, without any dependencies #17

Merged
merged 2 commits into from
Sep 23, 2016
Merged

Conversation

cristim
Copy link
Member

@cristim cristim commented Sep 23, 2016

Fixes #6, implemented as a result of some off-topic discussions which happened on #15.

The goal is to be able to run without any runtime dependencies
- everything is now self-contained, included in the Lambda zip file
- no more runtime downloads of data from external infrastructure
- more reliable, everything is available out of the box
- stable software, no longer auto-updates behind the user's back
- remove dependency on UPX, since the size no longer matters so much
- saving some execution time and a few pennies spent on bandwidth
- lots of Lambda wrapper cleanups, the wrapper code is now 3x smaller
- some build system cleanups and improvements
- introduced a command-line argument to avoid hard-coding the
  instances.json file name

Downside: the users need to update the CloudFormation stack in order to
update the software to the latest version
@cristim cristim merged commit 40ac8b5 into master Sep 23, 2016
cristim added a commit that referenced this pull request Sep 23, 2016
Self contained execution, without any dependencies
cristim added a commit that referenced this pull request Sep 23, 2016
Self contained execution, without any dependencies
@cristim cristim self-assigned this Nov 3, 2016
@cristim cristim deleted the self-contained branch December 22, 2016 20:09
Copy link
Contributor

@xlr-8 xlr-8 left a comment

Choose a reason for hiding this comment

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

Seems like many things in this PR have been modified since. Not sure of the actual need of reviewing it, but rest seems fine

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