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

[CI][C++][Python] Verification jobs fail on ubuntu for Python due to missing libzstd #35205

Closed
raulcd opened this issue Apr 18, 2023 · 3 comments · Fixed by #35220
Closed

[CI][C++][Python] Verification jobs fail on ubuntu for Python due to missing libzstd #35205

raulcd opened this issue Apr 18, 2023 · 3 comments · Fixed by #35220

Comments

@raulcd
Copy link
Member

raulcd commented Apr 18, 2023

Describe the bug, including details regarding any error messages, version, and platform.

Verification jobs for Python on Ubuntu 20.04 and 22.04 due to zstd not found:

verify-rc-source-python-linux-ubuntu-20.04-amd64
verify-rc-source-python-linux-ubuntu-22.04-amd64

CMake Warning at /tmp/arrow-HEAD.xCuHo/install/lib/cmake/Gandiva/FindzstdAlt.cmake:29 (find_package):
  By not providing "Findzstd.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "zstd", but
  CMake did not find one.

  Could not find a package configuration file provided by "zstd" with any of
  the following names:

    zstdConfig.cmake
    zstd-config.cmake

  Add the installation prefix of "zstd" to CMAKE_PREFIX_PATH or set
  "zstd_DIR" to a directory containing one of the above files.  If "zstd"
  provides a separate development package or SDK, be sure it has been
  installed.
Call Stack (most recent call first):
  /usr/share/cmake-3.16/Modules/CMakeFindDependencyMacro.cmake:47 (find_package)
  /tmp/arrow-HEAD.xCuHo/install/lib/cmake/Gandiva/GandivaConfig.cmake:64 (find_dependency)
  CMakeLists.txt:679 (find_package)


-- Checking for module 'libzstd'
--   No package 'libzstd' found
CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
  Could NOT find zstdAlt (missing: ZSTD_LIB ZSTD_INCLUDE_DIR)
Call Stack (most recent call first):
  /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:393 (_FPHSA_FAILURE_MESSAGE)
  /tmp/arrow-HEAD.xCuHo/install/lib/cmake/Gandiva/FindzstdAlt.cmake:124 (find_package_handle_standard_args)
  /usr/share/cmake-3.16/Modules/CMakeFindDependencyMacro.cmake:47 (find_package)
  /tmp/arrow-HEAD.xCuHo/install/lib/cmake/Gandiva/GandivaConfig.cmake:64 (find_dependency)
  CMakeLists.txt:679 (find_package)


-- Configuring incomplete, errors occurred!
See also "/arrow/python/build/temp.linux-x86_64-cpython-38/CMakeFiles/CMakeOutput.log".
See also "/arrow/python/build/temp.linux-x86_64-cpython-38/CMakeFiles/CMakeError.log".
error: command '/usr/bin/cmake' failed with exit code 1
Failed to verify release candidate. See /tmp/arrow-HEAD.xCuHo for details.

It started failing since this PR was merged which seems could be related:

#35023

Those are the commits since the first failure:
6120345...6432a23

This should be fixed for 12.0.0 as this commit was the last one introduced before the feature freeze.

Component(s)

C++, Continuous Integration

@raulcd
Copy link
Member Author

raulcd commented Apr 18, 2023

I have no idea what is happening but could be something to do with GandivaConfig.cmake.in temporary modifying CMAKE_MODULE_PATH?

if(DEFINED CMAKE_MODULE_PATH)
  set(GANDIVA_CMAKE_MODULE_PATH_OLD ${CMAKE_MODULE_PATH})
else()
  unset(GANDIVA_CMAKE_MODULE_PATH_OLD)
endif()
set(CMAKE_MODULE_PATH "${CMAKE_CURRENT_LIST_DIR}")

@raulcd
Copy link
Member Author

raulcd commented Apr 18, 2023

@kou any idea?

@kou
Copy link
Member

kou commented Apr 18, 2023

Oh, sorry...
I'll take a look at this.

kou added a commit to kou/arrow that referenced this issue Apr 18, 2023
kou added a commit that referenced this issue Apr 19, 2023
…led one (#35220)

### Rationale for this change

If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.

### What changes are included in this PR?

Add a missing check whether we're using system Zstandard or bundled Zstandard.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: #35205

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
raulcd pushed a commit that referenced this issue Apr 19, 2023
…led one (#35220)

### Rationale for this change

If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.

### What changes are included in this PR?

Add a missing check whether we're using system Zstandard or bundled Zstandard.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: #35205

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
liujiacheng777 pushed a commit to LoongArch-Python/arrow that referenced this issue May 11, 2023
…e bundled one (apache#35220)

### Rationale for this change

If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.

### What changes are included in this PR?

Add a missing check whether we're using system Zstandard or bundled Zstandard.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#35205

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
…e bundled one (apache#35220)

### Rationale for this change

If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.

### What changes are included in this PR?

Add a missing check whether we're using system Zstandard or bundled Zstandard.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#35205

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
…e bundled one (apache#35220)

### Rationale for this change

If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.

### What changes are included in this PR?

Add a missing check whether we're using system Zstandard or bundled Zstandard.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#35205

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants