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

WIP: Launch 3-component deployment using docker-compose.yml #1300

Closed
wants to merge 1 commit into from

Conversation

riconnon
Copy link
Contributor

@riconnon riconnon commented Nov 6, 2018

I thought I'd experiment with launching a 3-component deployment as a local environment using docker-compose

Interested to know what people think.

@riconnon riconnon force-pushed the experiment/rconnon/components branch from edb6275 to 45e35bf Compare November 6, 2018 11:26
@rikgibson
Copy link
Contributor

I think it would be good to maintain the single-container model for fnserver if possible, even if internally the container was running three processes - it keeps the black box simple and consistent for the user.

@rdallman
Copy link
Contributor

rdallman commented Nov 7, 2018

it should be possible to do this all in a single process, too, there's some... factoring issues... to solve.

- fnserver
- api
- lb
- runner
volumes:
- ./examples/grafana/prometheus.yml:/etc/prometheus/prometheus.yml
fnserver-ui:
Copy link
Member

Choose a reason for hiding this comment

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

let's exclude the UI component, because it doesn't work =(

lb:
restart: always
depends_on:
- db
Copy link
Member

Choose a reason for hiding this comment

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

should also depend on api node.

build: .
networks:
- fn-network
build:
Copy link
Member

Choose a reason for hiding this comment

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

Network attribute was necessary to let functions be within the Fn network, and the idea was to have a flat network where each component has a binding.

@denismakogon
Copy link
Member

As for testing, i'm getting Failed during call placement while calling hello-world samples.

@denismakogon
Copy link
Member

Let's do the following, I'd rather get rid of compose file and use OSS helm charts. @riconnon thoughts?

@riconnon riconnon closed this Jan 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants