Skip to content

Enable validation cf-deployment#424

Open
c0d1ngm0nk3y wants to merge 2 commits into
mainfrom
validate-cf-deployment
Open

Enable validation cf-deployment#424
c0d1ngm0nk3y wants to merge 2 commits into
mainfrom
validate-cf-deployment

Conversation

@c0d1ngm0nk3y

Copy link
Copy Markdown
Contributor

prerequisite: #423

It adds a parameter pre-validation to the smoke tests. When set to true, it will test the versions currently pre-validated by cf-deployment.

@c0d1ngm0nk3y c0d1ngm0nk3y force-pushed the validate-cf-deployment branch 4 times, most recently from 3238679 to 20ee5f8 Compare June 5, 2026 12:32
@c0d1ngm0nk3y c0d1ngm0nk3y marked this pull request as ready for review June 9, 2026 12:01
@c0d1ngm0nk3y c0d1ngm0nk3y requested a review from a team as a code owner June 9, 2026 12:01
- cron: '0 6 * * *'
workflow_dispatch:
inputs:
fresh-validation:

@modulo11 modulo11 Jun 10, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: use-pre-validated-versions or use-pre-validated-releases or (to stay in the current logic) use-latest-versions/releases?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The proposal came from cf-deployment team. It seems to be the term that would be used for bosh. If it less confusing in cf-deployment, it might be OK. WDYT?

modulo11
modulo11 previously approved these changes Jun 10, 2026
Add parameter pre-validation to validate cf-deployment

Co-authored-by: Ralf Pannemans <ralf.pannemans@sap.com>
Allow fresh-validation to be triggered

Co-authored-by: Ralf Pannemans <ralf.pannemans@sap.com>
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.

2 participants