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

Run PostgreSQL connector samples against GCP in cibuild #302

Draft
wants to merge 4 commits into
base: latest
Choose a base branch
from

Conversation

bart-vmware
Copy link
Member

No description provided.

@bart-vmware
Copy link
Member Author

@TimHess In our current testing environment, the following GCP plans are available:

> cf marketplace -e csb-google-postgres
Getting service plan information for service offering csb-google-postgres in org engineering / space bkoelman as bkoelman...

broker: cloud-service-broker-gcp
   plan           description             free or paid   costs
   example-plan   default: example plan   free

Could you tell me if you created this plan, or does it come out of the box? The previous environment contained a plan named "default".

If this comes out of the box, I think we should update our code; otherwise, we will need to have the "default" plan be available.

@TimHess
Copy link
Member

TimHess commented Jul 20, 2023

@TimHess In our current testing environment, the following GCP plans are available:

> cf marketplace -e csb-google-postgres
Getting service plan information for service offering csb-google-postgres in org engineering / space bkoelman as bkoelman...

broker: cloud-service-broker-gcp
   plan           description             free or paid   costs
   example-plan   default: example plan   free

Could you tell me if you created this plan, or does it come out of the box? The previous environment contained a plan named "default".

If this comes out of the box, I think we should update our code; otherwise, we will need to have the "default" plan be available.

I believe this plan came out of the box with the broker. It is possible the team maintaining the broker changed the name of the plan to double-down on the point they raise in their documentation:

A default plan becomes available when installing the tile. This plan is for reference only, and is not intended for production use. Remove this default plan and create plans that fit your requirements.

I think renaming the plan in our environment would be an appropriate course of action

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