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.

Sign up for GitHub

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

Jump to bottom

Add environment variable support to flocker.node.Deployer.calculate_necessary_state_changes #586

Closed
wallrj opened this issue Aug 19, 2014 · 0 comments · Fixed by #591
Closed

Add environment variable support to flocker.node.Deployer.calculate_necessary_state_changes #586

wallrj opened this issue Aug 19, 2014 · 0 comments · Fixed by #591
Assignees
@wallrj
Milestone
Release 0.2

Comments

@wallrj
Copy link
Contributor

wallrj commented Aug 19, 2014

A sub-ticket of #95

The application YAML file now allows environment variables and the GearClient API allows environment variables to be supplied to a unit.

So now we need to hook the two up by modifying Application, calculate_necessary_state_changes and StartApplication.
@robhaswell robhaswell mentioned this issue Aug 19, 2014
Closed
4 tasks
@wallrj wallrj added this to the Release 0.1.1 milestone Aug 19, 2014
@wallrj wallrj self-assigned this Aug 19, 2014
@wallrj wallrj mentioned this issue Aug 19, 2014
Merged
itamarst added a commit that referenced this issue Aug 19, 2014
@itamarst
Merge pull request #591 from ClusterHQ/application-environment-586-2
f897d9e 
Pass the supplied Application environment during StartApplication state changes.

Fixes #586.
@adamtheturtle adamtheturtle mentioned this issue Sep 24, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wallrj wallrj

Labels
None yet
Projects
None yet
Milestone
Release 0.2
1 participant
@wallrj