Skip to content

Add per route load balancing test app to sample apps workflow#378

Merged
Dariquest merged 10 commits into
mainfrom
sample-app-per-route-lb
Jun 22, 2026
Merged

Add per route load balancing test app to sample apps workflow#378
Dariquest merged 10 commits into
mainfrom
sample-app-per-route-lb

Conversation

@Dariquest

@Dariquest Dariquest commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@a18e a18e self-assigned this Jun 22, 2026
@Dariquest

Dariquest commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Workflow run success https://github.com/SAP-samples/cf-routing-samples/actions/runs/27941827135

@Dariquest
Dariquest enabled auto-merge (squash) June 22, 2026 09:33
a18e
a18e approved these changes Jun 22, 2026
View reviewed changes

@a18e a18e left a comment

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.

LGTM, but please test these changes by opening another PR after merge and ensuring the new tests still succeed.
(For security reasons, the CI validation always runs with the workflow on the master branch, so the new tests are not evaluated in this PR)

@Dariquest
Dariquest merged commit 1e2dd1e into main Jun 22, 2026
11 checks passed
@Dariquest
Dariquest deleted the sample-app-per-route-lb branch June 22, 2026 12:31
@Dariquest

Dariquest commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

LGTM, but please test these changes by opening another PR after merge and ensuring the new tests still succeed. (For security reasons, the CI validation always runs with the workflow on the master branch, so the new tests are not evaluated in this PR)

@a18e The tests were run on the branch to commit, you can see it here https://github.com/SAP-samples/cf-routing-samples/actions/runs/27941827135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@a18e a18e a18e approved these changes

Assignees

@a18e a18e

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Dariquest @a18e