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

Intermittent CI test failure #413

Open
esabol opened this issue Sep 24, 2024 · 3 comments
Open

Intermittent CI test failure #413

esabol opened this issue Sep 24, 2024 · 3 comments

Comments

@esabol
Copy link
Member

esabol commented Sep 24, 2024

This test in t/client suddenly has started to fail frequently but not always:

tests/libgearman-1.0/client_test.cc:821: in hostname_resolution() pid(37552) Assertion 'GEARMAN_COULD_NOT_CONNECT' != 'gearman_client_echo(&client, test_literal_param("foo"))' client.gearman_client_st.gearman_client_add_servers(GEARMAN_GETADDRINFO) [ failed ]

That corresponds to this code:

ASSERT_EQ(GEARMAN_COULD_NOT_CONNECT,
gearman_client_echo(&client, test_literal_param("foo")));

In what scenario would that ASSERT_EQ() fail?

@esabol
Copy link
Member Author

esabol commented Sep 24, 2024

Actually, more often than not the CI workflow will just hang for a long time – 30+ minutes is not uncommon – before it finally fails that test.

@esabol
Copy link
Member Author

esabol commented Sep 24, 2024

A couple lines earlier before the test that intermittently fails is this skipped test:

test_skip(GEARMAN_SUCCESS, gearman_client_add_servers(&client, "exist.gearman.info:12345"));

Does a skipped test still execute the code? I think so? test_skip is a #define which does a do { ... } while (0);

So that means the failing test is trying to send foo to exist.gearman.info:12345?

@esabol
Copy link
Member Author

esabol commented Sep 24, 2024

It took 12 attempts, but I finally got all of the CI workflow jobs to pass.

I'm tempted to just disable all of these hostname_resolution() tests in CI.

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

No branches or pull requests

1 participant