Skip to content
This repository has been archived by the owner on Mar 18, 2021. It is now read-only.

[WIP] Issue 25 #49

Closed
wants to merge 7 commits into from
Closed

[WIP] Issue 25 #49

wants to merge 7 commits into from

Conversation

cpitkin
Copy link
Contributor

@cpitkin cpitkin commented Sep 23, 2018

@nicholasjackson @acornies Can I get a check of my implementation. The implementation details are in Issue #25. I am having an issue setting up the tests to work with the new format. If either of you could make some suggestions on where I am going wrong that would be great.

nicholasjackson and others added 7 commits February 22, 2018 14:47
Added a create constraints to parse the constraints slice
Updated create datacenters to just accept a string from the constraints

Signed-off-by: Charlie Pitkin <charlie.pitkin@gmail.com>
@cpitkin
Copy link
Contributor Author

cpitkin commented Sep 23, 2018

I also just noticed I need to squash and rebase. I'll do that here in the next day or so.

@acornies
Copy link
Collaborator

Looks like this is still unfinished? Should probably not open a PR unless make test runs successfully on your machine (and tests aren't commented out). I suggest if you want a review of your implementation, provide a link to your fork/branch etc. before PR.

@acornies
Copy link
Collaborator

Hi @cpitkin , I've opened #56 which should supersede this one. I needed to tackle this for my company's dev environment as well as use the agent's configured DC instead of static dc1. Thanks for the understanding.

@cpitkin
Copy link
Contributor Author

cpitkin commented Oct 19, 2018

@acornies No worries. Thanks for making the update.

@cpitkin cpitkin closed this Oct 19, 2018
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.

None yet

3 participants