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

Prepare Derek for use as an SDK/package #123

Open
alexellis opened this issue Apr 7, 2019 · 1 comment
Open

Prepare Derek for use as an SDK/package #123

alexellis opened this issue Apr 7, 2019 · 1 comment

Comments

@alexellis
Copy link
Owner

alexellis commented Apr 7, 2019

Expected Behaviour

We should prepare Derek for use as an SDK/package so that he can be run on any platform - a Go microservice, AWS Lambda, Heroku etc.

This is the first step that makes Derek a package that someone can vendor in Go and then make use of in their own stand-alone way. #13

Current Behaviour

Derek may "os.Exit" at several points in the code.

Derek's configuration may depend on certain characteristics of OpenFaaS.

Possible Solution

  • Try to vendor Derek as he is right now, see what happens.
  • Compile a list of suggestions on this issue and ask for input
  • Move all code out of main and into packages

Acceptance criteria

  • Test Derek spun up in a single Go HTTP server outside of a container
  • Make sure there is no regression
@alexellis
Copy link
Owner Author

Related work: #96

zoidyzoidzoid pushed a commit to zoidyzoidzoid/derek that referenced this issue Oct 18, 2021
zoidyzoidzoid pushed a commit to zoidyzoidzoid/derek that referenced this issue Oct 18, 2021
Closes alexellis#96

Might also close alexellis#123

Signed-off-by: zoidbergwill <[email protected]>
zoidyzoidzoid pushed a commit to zoidyzoidzoid/derek that referenced this issue Oct 18, 2021
Closes alexellis#96

Might also close alexellis#123

Signed-off-by: zoidbergwill <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant