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

Incomplete pool runner doc #26

Open
denismakogon opened this issue Nov 13, 2018 · 2 comments
Open

Incomplete pool runner doc #26

denismakogon opened this issue Nov 13, 2018 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@denismakogon
Copy link
Member

Pool runner doc misses FN_PUBLIC_LB_URL configuration which makes the deployment broken.

@michael-w-williams michael-w-williams added the bug Something isn't working label Nov 13, 2018
@michael-w-williams michael-w-williams self-assigned this Nov 13, 2018
@michael-w-williams
Copy link
Member

Denis, What changes do I need to make to https://github.com/fnproject/docs/blob/master/fn/operate/runner_pools.md? Does the Env var need to be set as a first step? What would a sample value look like?

@denismakogon
Copy link
Member Author

Well, my answer wouldn't be easy. So, the following env var needs to be set in docker run for API node. By itself, this env var represents a public (reachable through the internet) IP address of the LB container.
So, missing steps here are:

  1. Deploy API container at the end.
  2. When LB container deployed - check its IP address (that could be docker inspect -f '{{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}' <lb-container>).
  3. Add -e FN_PUBLIC_LB_URL=http://<lb-container-ip>:<lb-container-http-port>

The most crucial part here is Fn CLI simple test. Since this type of the deployment is not usual to anyone expect us, you need to show the following things:

  1. Create a new context with API container IP address.
  2. Deploy hello-world.
  3. Inspect a function, highlight the annotation because it would be different compared to regular annotation you'd get from FN running throughfn start.
  4. Call a function with both an HTTP trigger and invoke.

denismakogon added a commit that referenced this issue Jan 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants