Skip to content
This repository was archived by the owner on May 24, 2023. It is now read-only.

Conversation

@Dean-Coakley
Copy link
Contributor

@Dean-Coakley Dean-Coakley commented Mar 4, 2021

Fixes: #69

Proposed changes

  • Make service and replicas optional.
  • Add default of 1 for replicas when not set.
  • Make service nullable to allow a replicas be set to 0.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto master
  • I will ensure my PR is targeting the master branch and pulling from my branch from my own fork

* Replicas now sets a default of 1 when not set.
@Dean-Coakley Dean-Coakley added the bug An issue reporting a potential bug label Mar 4, 2021
@Dean-Coakley Dean-Coakley self-assigned this Mar 4, 2021
@Dean-Coakley Dean-Coakley merged commit 8e21f19 into master Mar 5, 2021
@Dean-Coakley Dean-Coakley deleted the fix/optional-replicas branch March 5, 2021 11:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug An issue reporting a potential bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Number of replicas is needed

4 participants