search

cloudfoundry-attic / etcd-release

Apache License 2.0
3 stars 17 forks source link

add optional links; manifests/bosh-lite now promote links/cloud-config #38

Closed drnic closed 7 years ago

cfdreddbot commented 7 years ago

Hey drnic!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

cf-gitbot commented 7 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/142046757

The labels on this github issue will be updated when the story is started.

drnic commented 7 years ago

Think I still have some work to do to support etcd.cluster via links.

And locally having inter-node connectivity issues; not resolved yet.

drnic commented 7 years ago

And this issue:

$ curl https://etcd-0.etcd.service.cf.internal:4001/v2/keys -k
curl: (35) error:14094412:SSL routines:SSL3_READ_BYTES:sslv3 alert bad certificate
genevieve commented 7 years ago

Hey @drnic. We are working on bosh 2.0 manifests now. Are you planning on working more on the issues you listed above in this PR or should we merge this and make those fixes?

genevieve commented 7 years ago

@drnic We will merge this and resolve any issues as they come up.

drnic commented 7 years ago

My networking issues were local and fixed. The only changes I've made since this PR were to expose properties via provides. Then I wanted to the remove to the etcd. namespace prefix but you can make the timing call on when to roll that out. Thanks if this PR is helpful.