Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

added external GCP DB support for CredHub #378

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

poligraph
Copy link

This PR adds proper CredHub configuration for external DB on GCP
Without it deploy-ert job failed and you should manually configure PAS tile configuration.
It also adds logic to configure jobs according to version, some jobs exist in 2.1 and 2.2, but not in 2.3 and 2.4.

  • Expected result after the change:
    pcf-pipelines on GCP completes without failure

  • Current result before the change:
    deploy-ert job on GCP fails

  • I have viewed signed and have submitted the Contributor License Agreement

  • I have made this pull request to the master branch

  • I have run all the unit tests

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

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

@poligraph
Copy link
Author

Hi @kcboyle @jtarchie @nbconklin, is it any luck to merge PR this year? Or maybe it requires some improvements? Thanks, Konstantin.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants