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

pulsar-client-go test cannot pass on apachepulsar/pulsar:latest #748

Closed
shoothzj opened this issue Mar 12, 2022 · 7 comments · Fixed by #747
Closed

pulsar-client-go test cannot pass on apachepulsar/pulsar:latest #748

shoothzj opened this issue Mar 12, 2022 · 7 comments · Fixed by #747

Comments

@shoothzj
Copy link
Member

Five days ago, apachepulsar/pulsar:latest point on pulsar 2.8.1, now apachepulsar/pulsar:latest point on pulsar 2.9.1

I verified that test can't pass on neither 2.9.0 or 2.9.1 on #746

I also verified that test can paas on pulsar 2.8.2 on #747

I suggest we change to 2.8.2 for not blocking the PR, and change back to apachepulsar/pulsar:latest when 2.9.2 are released

@wolfstudy @merlimat

@freeznet
Copy link
Contributor

context: the failed CI related with apache/pulsar#13885 and apache/pulsar#14176

@ZiyaoWei
Copy link
Contributor

ZiyaoWei commented Mar 15, 2022

I'm looking into the issue too so I can unblock #741 - LMK if there's anything we can do to tackle this together!

Also, does/should the Pulsar CI also include tests for "official" clients?

@shoothzj
Copy link
Member Author

I suggest to merge #747 for unblocking pr.

@ZiyaoWei
Copy link
Contributor

I've been trying to run the CI tests using the 2.10.0 rc (lph890127/pulsar:2.10.0) but it's failing in some other way - I'll do some more local testing and meanwhile will file a ticket for creating a nightly build so we can test all the external repos against the latest code in Pulsar.

@ZiyaoWei
Copy link
Contributor

It turns out, as @freeznet pointed out in private messages, TestSendTimeout would not fail on its own. I did some digging and it seems it would only fail when ran after TestConsumerSeek and TestConsumerMetadata. Will look into it to see what's going on.

@ZiyaoWei
Copy link
Contributor

Actually the tests pass under lph890127/pulsar:2.10.0-rc4, so maybe we should temporarily point to 2.8.2 until 2.10 is out?

@merlimat
Copy link
Contributor

👍 Makes sense to me

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 a pull request may close this issue.

4 participants