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

Disable HTTP/2 #190

Merged
merged 1 commit into from
Apr 6, 2022
Merged

Conversation

MikeEdgar
Copy link
Contributor

MGDSTRM-8107

Signed-off-by: Michael Edgar [email protected]

MGDSTRM-8107

Signed-off-by: Michael Edgar <[email protected]>
@MikeEdgar MikeEdgar added this to the 0.9.1 milestone Apr 5, 2022
@rareddy
Copy link
Contributor

rareddy commented Apr 5, 2022

I believe @shawkins added a way from Strimzi configmap to add the ENV properties if you want to take that route too.

@shawkins
Copy link

shawkins commented Apr 5, 2022

I believe @shawkins added a way from Strimzi configmap to add the ENV properties if you want to take that route too.

It's not wired in automatically, it's just that the fleetshard_operands.yaml can accommodate that if needed.

@MikeEdgar
Copy link
Contributor Author

As I was committing this change I was wishing we had it ready :-)

I believe @shawkins added a way from Strimzi configmap to add the ENV properties if you want to take that route too.

It's not wired in automatically, it's just that the fleetshard_operands.yaml can accommodate that if needed.

@MikeEdgar MikeEdgar marked this pull request as ready for review April 6, 2022 09:54
Copy link
Contributor

@rareddy rareddy left a comment

Choose a reason for hiding this comment

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

lgtm

@MikeEdgar MikeEdgar merged commit a4dc66c into bf2fc6cc711aee1a0c2a:main Apr 6, 2022
@MikeEdgar MikeEdgar deleted the disable-http2 branch April 6, 2022 12:15
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.

3 participants