Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

Disable service replicas with development mode flag #265

Closed

Conversation

wilkers-steve
Copy link
Contributor

@wilkers-steve wilkers-steve commented Mar 10, 2017

Disable service replicas with development mode flag in all charts

To remain consistent with other services, this adds a development
mode flag to reduce the number of replicas to 1.

What is the purpose of this pull request?: Disables replicas for a service when development flag is set to enabled

What issue does this pull request address?: Partially addresses #258

Notes for reviewers to consider:

Specific reviewers for pull request:

To remain consistent with other services, this adds a development
mode flag to reduce the number of replicas to 1.
@wilkers-steve wilkers-steve changed the title WIP: Consistent Development Mode for Services Disable service replicas with development mode flag Mar 13, 2017
compute_replicas: 1
replicas:
control: 1
compute: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Is compute used? If not can we remove it?

@v1k0d3n v1k0d3n added this to the 0.3.0 milestone Mar 22, 2017
@wilkers-steve
Copy link
Contributor Author

@v1k0d3n @intlabs related to our discussion re: rethinking development mode, do we want to close this PR out? After some thought, I'm not a fan of the idea of controlling replicas with a development flag.

@v1k0d3n
Copy link
Collaborator

v1k0d3n commented Mar 23, 2017

yup, i'll let you close it out. i think that if we just stick with a developer focused config-override on the client side, rather than with debt in the form of conditionals we'll be in a lot better shape (imo, and as we've discussed collectively).

@wilkers-steve
Copy link
Contributor Author

Absolutely agreed. 👍

@wilkers-steve wilkers-steve deleted the dev_mode_updates branch April 11, 2017 14:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants