Skip to content

fix: cloud_controller_ng#4971#5026

Draft
b1tamara wants to merge 1 commit intocloudfoundry:mainfrom
sap-contributions:fix-hash
Draft

fix: cloud_controller_ng#4971#5026
b1tamara wants to merge 1 commit intocloudfoundry:mainfrom
sap-contributions:fix-hash

Conversation

@b1tamara
Copy link
Copy Markdown
Contributor

@b1tamara b1tamara commented Apr 14, 2026

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

Output during cf push with my change:


    buildpacks:
      nodejs_buildpack
+   routes:
+   - options:
+       loadbalancing: hash
+     route: hash-invalid.cfapps.aws-cfn07.aws.cfn.sapcloud.io
+   - options:
+       hash_balance: 1.25
+       loadbalancing: hash
+     route: hash-no-header.cfapps.aws-cfn07.aws.cfn.sapcloud.io
+   - options:
+       hash_header: hash-header
+       loadbalancing: hash
+     route: hash-no-balance.cfapps.aws-cfn07.aws.cfn.sapcloud.io
  version: 1
For application 'retry-test': For route 'hash-invalid.cfapps.aws-cfn07.aws.cfn.sapcloud.io': Hash header must be present when loadbalancing is set to hash.
  • Links to any other associated PRs: related to hash-based routing implementation in PR#4746

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

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.

1 participant