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

OTP27: case_clause when generating CT results #8594

Closed
mkuratczyk opened this issue Jun 19, 2024 · 5 comments
Closed

OTP27: case_clause when generating CT results #8594

mkuratczyk opened this issue Jun 19, 2024 · 5 comments
Assignees
Labels
bug Issue is reported as a bug team:PS Assigned to OTP team PS

Comments

@mkuratczyk
Copy link
Contributor

Describe the bug
I just had a crash when running tests with OTP27.

CRASHED while updating /Users/mkuratczyk/rabbitmq/server/otp27/deps/rabbit/logs/index.html!
{{case_clause,{176,0,0,0,510171915}},
 [{ct_logs,make_one_index_entry,6,[{file,"ct_logs.erl"},{line,1398}]},
  {ct_logs,make_all_suites_index3,9,[{file,"ct_logs.erl"},{line,2829}]},
  {ct_logs,make_all_suites_index2,3,[{file,"ct_logs.erl"},{line,2763}]},
  {ct_logs,make_all_suites_index1,4,[{file,"ct_logs.erl"},{line,2735}]},
  {ct_logs,make_all_suites_index_from_cache,5,
           [{file,"ct_logs.erl"},{line,2543}]},
  {ct_logs,make_all_suites_index,2,[{file,"ct_logs.erl"},{line,2461}]},
  {ct_logs,logger,4,[{file,"ct_logs.erl"},{line,757}]}]}
Test run crashed! This could be an internal error - please report!

{could_not_start_process,ct_logs,
    {badarg,
        [{erlang,'++',
             [{error,
                  {{case_clause,{176,0,0,0,510171915}},
                   [{ct_logs,make_one_index_entry,6,
                        [{file,"ct_logs.erl"},{line,1398}]},
                    {ct_logs,make_all_suites_index3,9,
                        [{file,"ct_logs.erl"},{line,2829}]},
                    {ct_logs,make_all_suites_index2,3,
                        [{file,"ct_logs.erl"},{line,2763}]},
                    {ct_logs,make_all_suites_index1,4,
                        [{file,"ct_logs.erl"},{line,2735}]},
                    {ct_logs,make_all_suites_index_from_cache,5,
                        [{file,"ct_logs.erl"},{line,2543}]},
                    {ct_logs,make_all_suites_index,2,
                        [{file,"ct_logs.erl"},{line,2461}]},
                    {ct_logs,logger,4,[{file,"ct_logs.erl"},{line,757}]}]}},
              []],
             [{error_info,#{module => erl_erts_errors}}]},
         {ct_logs,update_tests_in_cache,2,[{file,"ct_logs.erl"},{line,2716}]},
         {ct_logs,make_all_suites_index_from_cache,5,
             [{file,"ct_logs.erl"},{line,2554}]},
         {ct_logs,make_all_suites_index,2,[{file,"ct_logs.erl"},{line,2461}]},
         {ct_logs,logger,4,[{file,"ct_logs.erl"},{line,757}]}]}}

To Reproduce
Unknown. It happened after I added a ct:pal() call to a test but it seems to crash updating the index.html so I'm not sure if that's even related. It also fails now after removing the ct:pal().

Affected versions
27.0

@mkuratczyk mkuratczyk added the bug Issue is reported as a bug label Jun 19, 2024
@IngelaAndin IngelaAndin added the team:PS Assigned to OTP team PS label Jun 22, 2024
@u3s
Copy link
Contributor

u3s commented Jun 24, 2024

@Mikaka27 wondering could this some case missed in #8112?

@Mikaka27
Copy link
Contributor

I will take a look

@Mikaka27 Mikaka27 self-assigned this Jun 24, 2024
@Mikaka27
Copy link
Contributor

@mkuratczyk I think you're running this with otp-26 instead of otp-27.
This could happen if for example you executed tests with otp-27, they generated their logs, and then you switched to otp-26 and tried to run the tests again while still having the ct_logs directory from otp-27 run.

Unfortunately we're not forwards compatible (so otp-26 will not know how to handle ct_logs from otp-27).
Could you check if it's the case?

@mkuratczyk
Copy link
Contributor Author

I'm fairly sure that wasn't the case but I can't really check at this point. Feel free to close it, I will reopen/open a new one if I run into this again.

Thanks for looking into this and sorry if that was my mistake.

@Mikaka27
Copy link
Contributor

Ok, I will close it for now, feel free to reach out again if needed.
I think it must be the case because the line numbers in stacktraces match otp-26 source code not otp-27 :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is reported as a bug team:PS Assigned to OTP team PS
Projects
None yet
Development

No branches or pull requests

4 participants