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

Added a PSC GCLB example #1179

Merged
merged 15 commits into from
Feb 24, 2023
Merged

Added a PSC GCLB example #1179

merged 15 commits into from
Feb 24, 2023

Conversation

cgrotz
Copy link
Collaborator

@cgrotz cgrotz commented Feb 24, 2023

I created a simple blueprint where a L7XLB is connecting to an application in another project via a PSC Neg. The application in turn is exposed internally via a L7ILB, that is exposed via PSC. The application is a Cloud Run service, but could be anything really.

@ludoo
Copy link
Collaborator

ludoo commented Feb 24, 2023

Can you add a basic test like other blueprints? Otherwise it will stop working as soon as we change one of its modules...

@cgrotz
Copy link
Collaborator Author

cgrotz commented Feb 24, 2023

Just discovered the tests, will add one thank you. Can you give me some guidance what is missing for the linting check?

@ludoo
Copy link
Collaborator

ludoo commented Feb 24, 2023

Just discovered the tests, will add one thank you. Can you give me some guidance what is missing for the linting check?

wanna have a quick vc?

@ludoo
Copy link
Collaborator

ludoo commented Feb 24, 2023

All you need is to run tfdoc on the blueprint, as explained in the contributing guide. Ping me if you want to do it together.

@cgrotz
Copy link
Collaborator Author

cgrotz commented Feb 24, 2023

I'll try to figure it out, I will ping you if I can't make it work.

@cgrotz cgrotz enabled auto-merge February 24, 2023 19:55
@cgrotz cgrotz merged commit df9e332 into GoogleCloudPlatform:master Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants