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

fix(cloudFoundryDeploy): handle large amount of mta extension credentials #5211

Closed

Conversation

niki4
Copy link
Member

@niki4 niki4 commented Dec 18, 2024

Description

This is PoC PR that about to bypass limitation on large amount of mta extension credentials set in .pipeline/config.yaml for cloudFoundryDeploy step.

We introduce step parameter "checkMissingCredentials". If this option is set to false, then Piper will not check for missing mta extension credentials in env vars. Use this option if you have large amount of mtaExtensionCredentials values.

References

INC10351618 | Blocked service deployment due to failing cloudFoundryDeploy

Checklist

  • Tests
  • Documentation
  • Inner source library needs updating

@niki4 niki4 requested a review from a team as a code owner December 18, 2024 11:15
@ashlymat
Copy link
Member

@niki4 Can you add a description on the PR, just to understand better

Ivan Nikiforov added 2 commits December 18, 2024 14:37
@niki4
Copy link
Member Author

niki4 commented Dec 18, 2024

/it-go

@niki4
Copy link
Member Author

niki4 commented Dec 27, 2024

This PoC seems not fixes user issue, so I'm going to abandon it.

We found the root cause and it has been fixed in another PR: #5216

@niki4 niki4 closed this Dec 27, 2024
@niki4 niki4 deleted the ivan/fix-cloudFoundryDeploy-mtaExtensionCredentials branch December 27, 2024 09:54
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.

2 participants