Skip to content

hack: Make tests pass by increasing http2-max-requests in the ingress controller#3787

Closed
akshaymankar wants to merge 1 commit intodevelopfrom
fed-nginx-keepalive
Closed

hack: Make tests pass by increasing http2-max-requests in the ingress controller#3787
akshaymankar wants to merge 1 commit intodevelopfrom
fed-nginx-keepalive

Conversation

@akshaymankar
Copy link
Member

@akshaymankar akshaymankar commented Dec 18, 2023

https://wearezeta.atlassian.net/browse/WPB-5934

Workaround for: kazu-yamamoto/http2#102

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d No changelog.
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Dec 18, 2023
@akshaymankar akshaymankar changed the title hack: Make tests pass by increasing keepalive_requests for federator ingress hack: Make tests pass by increasing http2-max-requests in ingress-nginx Dec 19, 2023
@akshaymankar akshaymankar changed the title hack: Make tests pass by increasing http2-max-requests in ingress-nginx hack: Make tests pass by increasing http2-max-requests in the ingress controller Dec 19, 2023
Copy link
Contributor

@mdimjasevic mdimjasevic left a comment

Choose a reason for hiding this comment

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

I know nothing about YAML templating, but why is the limit in one place in quotes and not in the other?

@akshaymankar
Copy link
Member Author

superseded by #3789

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

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants