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

Add test job for Kafka versiom 2.7.1 #742

Merged
merged 1 commit into from
Sep 20, 2021
Merged

Conversation

rhansen2
Copy link
Collaborator

No description provided.

@rhansen2 rhansen2 force-pushed the 270-tests branch 2 times, most recently from a085d64 to f58f999 Compare September 18, 2021 23:42
@rhansen2 rhansen2 changed the title Add test job for Kafka versiom 2.7.0 Add test job for Kafka versiom 2.7.1 Sep 18, 2021
Comment on lines +234 to +243
# Need to skip nettest to avoid these kinds of errors:
# --- FAIL: TestConn/nettest (17.56s)
# --- FAIL: TestConn/nettest/PingPong (7.40s)
# conntest.go:112: unexpected Read error: [7] Request Timed Out: the request exceeded the user-specified time limit in the request
# conntest.go:118: mismatching value: got 77, want 78
# conntest.go:118: mismatching value: got 78, want 79
# ...
#
# TODO: Figure out why these are happening and fix them (they don't appear to be new).
KAFKA_SKIP_NETTEST: "1"
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you check whether this was still failing with 2.7.1?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah they're still failing for me in 2.7.1

@rhansen2 rhansen2 merged commit e6f0ae9 into segmentio:main Sep 20, 2021
@rhansen2 rhansen2 deleted the 270-tests branch September 20, 2021 17:48
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.

2 participants