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

chore(tests): add http2 smoke test #10452

Closed
wants to merge 4 commits into from

Conversation

windmgc
Copy link
Member

@windmgc windmgc commented Mar 8, 2023

Summary

This PR adds a simple smoke test to check if the HTTP/2 listening port can respond with valid response data.

Checklist

Full changelog

  • [Implement ...]

Issue reference

KAG-854

@windmgc windmgc requested a review from fffonion March 8, 2023 08:59
@github-actions github-actions bot added build/bazel chore Not part of the core functionality of kong, but still needed labels Mar 8, 2023
@fffonion
Copy link
Contributor

fffonion commented Mar 8, 2023

let's verify this once by copy pasting the alpine piece from matrix-full.yml to matrix-commitly.yml, and then revert back before merge. so we don't need to wait for another day to test it, or by manually click the button to release.

@fffonion fffonion modified the milestones: 3.2.1, 3.2.2 Mar 8, 2023
@windmgc windmgc force-pushed the chore-add-http2-smoke-test branch from 0d474b3 to f921d21 Compare March 8, 2023 12:08
@windmgc
Copy link
Member Author

windmgc commented Mar 8, 2023

Superseded by #10454 as smoke test cannot be triggered from personal fork due to lack of credentials so we cannot test in this PR. cc @fffonion

@windmgc windmgc closed this Mar 8, 2023
@windmgc windmgc deleted the chore-add-http2-smoke-test branch March 8, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build/bazel chore Not part of the core functionality of kong, but still needed size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants