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

util/log: tests fail on non-amd64 platforms #72227

Open
amitsadaphule opened this issue Oct 29, 2021 · 9 comments
Open

util/log: tests fail on non-amd64 platforms #72227

amitsadaphule opened this issue Oct 29, 2021 · 9 comments
Labels
B-unsupported-arch Non-x86_64 architectures: PPC, MIPS, etc C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) O-community Originated from the community X-blathers-triaged blathers was able to find an owner

Comments

@amitsadaphule
Copy link
Contributor

amitsadaphule commented Oct 29, 2021

(edited 2021-11-01 by knz)

Describe the problem

The tests under util/log and util/log/logcrash assert the rendering of stack traces, which inclues the presence of a file asm_amd64.s inside the stack trace.

On non-amd64 platforms this file is not present in stack traces and the tests fail.

To Reproduce

run tests on non-amd64 platform, eg ARM or PPC.

Jira issue: CRDB-11049

@amitsadaphule amitsadaphule added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Oct 29, 2021
@blathers-crl
Copy link

blathers-crl bot commented Oct 29, 2021

Hello, I am Blathers. I am here to help you get the issue triaged.

Hoot - a bug! Though bugs are the bane of my existence, rest assured the wretched thing will get the best of care here.

I have CC'd a few people who may be able to assist you:

  • @cockroachdb/bulk-io (found keywords: export)

If we have not gotten back to your issue within a few business days, you can try the following:

  • Join our community slack channel and ask on #cockroachdb.
  • Try find someone from here if you know they worked closely on the area and CC them.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is otan.

@blathers-crl blathers-crl bot added A-disaster-recovery O-community Originated from the community X-blathers-triaged blathers was able to find an owner labels Oct 29, 2021
@blathers-crl
Copy link

blathers-crl bot commented Oct 29, 2021

cc @cockroachdb/bulk-io

@knz
Copy link
Contributor

knz commented Nov 1, 2021

There are two separate (unrelated) issues here:

  • a misdesign in the sql test
  • platform compatibility in the log package

We need to split them into two different issues.

@knz
Copy link
Contributor

knz commented Nov 1, 2021

@amitsadaphule why do you need these particular tests to pass?

Generally, CockroachDB's team does not support alternative build platforms, so we're likely not going to work actively to resolve this particular problem. However, on the other hand we have many users that report that cockroachDB works on their platform, even if not officially supported.

Does this block your work in any way?

@knz knz added B-unsupported-arch Non-x86_64 architectures: PPC, MIPS, etc C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) and removed C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. labels Nov 1, 2021
@knz
Copy link
Contributor

knz commented Nov 1, 2021

moved the SQL issue to #72284

@knz knz changed the title sql, util/log tests fail to consider arch specific asm files on ppc64le util/log: tests fail on non-amd64 platforms Nov 1, 2021
@seth-priya
Copy link

Hi @knz thanks for your response, responding on behalf of @amitsadaphule as he is on vacation. To answer your query

@amitsadaphule why do you need these particular tests to pass?

Generally, CockroachDB's team does not support alternative build platforms, so we're likely not going to work actively to resolve this particular problem. However, on the other hand we have many users that report that cockroachDB works on their platform, even if not officially supported.

Does this block your work in any way?

I understand that alternative build platforms are not officially supported but the one way for us to ensure that cockroachdb won't run into any functional issues on ppc64le which is where we are currently trying to get it to work is to rely on its test suite and that is the reason why we are trying to get all the tests to work correctly including those mentioned in #72225 and #72226. It won't block us if it does not for example impact any functionality/feature but I am not sure we know enough at this point to be sure about that atleast on our side.

Any thoughts?

In the meantime, we are trying to debug and root cause the failures but any pointers from you / anyone else who is familiar with these areas of the code would be of great help, thank you!

@jtsiros
Copy link

jtsiros commented Feb 17, 2022

cc: @cockroachdb/dev-inf for triage

@jtsiros jtsiros removed the T-server-and-security DB Server & Security label Feb 17, 2022
@dt
Copy link
Member

dt commented Feb 17, 2022

I fixed a similar case of these platform-specific stack traces in the SQL codebase with a quick trivial string replacement. Can we we just do the same thing in logcrash tests? d7efe41

@github-actions
Copy link

We have marked this issue as stale because it has been inactive for
18 months. If this issue is still relevant, removing the stale label
or adding a comment will keep it active. Otherwise, we'll close it in
10 days to keep the issue queue tidy. Thank you for your contribution
to CockroachDB!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-unsupported-arch Non-x86_64 architectures: PPC, MIPS, etc C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) O-community Originated from the community X-blathers-triaged blathers was able to find an owner
Projects
None yet
Development

No branches or pull requests

6 participants