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.

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

Add ability to get DSN from multiple Cloud Foundry service bindings #106

Closed
wants to merge 4 commits into from

Conversation

ihrigb
Copy link

@ihrigb ihrigb commented Sep 19, 2017

This enables the service (if DATA_SOURCE_NAME is not specified) to try to parse VCAP_SERVICES in order to get all bound postgres services.
If it finds some, it will retrieve the URIs and enable multiple exporters. This is while userQueriesPath-file will be applied to each of them and it will search for a file with name <cf-service-instance-name>.yml, which will be individually added.

As a first step, the first bound Postgres instance will be used.
This could be extended to support multiple bindings/postgesses.
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.08%) to 49.265% when pulling 996124b on ihrigb:dev/cf into e701b80 on wrouesnel:master.

This will prefix the service names to the metric descriptor names.
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.1%) to 49.237% when pulling fd3b357 on ihrigb:dev/cf into e701b80 on wrouesnel:master.

@ihrigb ihrigb changed the title Add ability to get DSN from CF Binding credentials. Add ability to get DSN from multiple Cloud Foundry service bindings Sep 20, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.1%) to 49.239% when pulling 9c471ca on ihrigb:dev/cf into e701b80 on wrouesnel:master.

@ihrigb
Copy link
Author

ihrigb commented Sep 20, 2017

Need to do some rework on this.

@ihrigb ihrigb closed this Sep 20, 2017
@ihrigb ihrigb deleted the dev/cf branch September 25, 2017 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants