ceph - ceph-devel - 2024-07-09

Timestamp (UTC)Message
2024-07-09T06:44:38.217Z
<Vikram Kumar Vaidyam> bump^
2024-07-09T07:44:42.138Z
<Ilya Dryomov> The external network that the lab is connected to had a failure period yesterday/earlier today
It manifested mostly as DNS issues
2024-07-09T12:31:07.965Z
<Rost Khudov> Hello team
@Casey Bodley @Matt Benjamin could you pls take a look at the following [comment](https://github.com/ceph/ceph/pull/55076#issuecomment-2217546486) related to recent changes in [following PR](https://github.com/ceph/ceph/pull/55076)?
2024-07-09T12:32:08.527Z
<Matt Benjamin> so building it
2024-07-09T12:32:23.440Z
<Matt Benjamin> on...a platform
2024-07-09T12:33:49.538Z
<Casey Bodley> have you updated your xxHash submodule?
2024-07-09T12:33:55.224Z
<Matt Benjamin> you did update your xxhash submodule?
2024-07-09T12:36:16.701Z
<Igor Gomon> Hello everyone,

I am trying to build Ceph with ENABLE_SHARED set to OFF. While building an individual target (such as radosgw) compiled without issues doing the complete build gives me the errors below, which seems to me like incompatible build flags.

A separate question about ENABLE_SHARED: what is the scope of this variable? Does it only affect the build from the immediate source tree or does it affect submodules too?

Thanks!

[372/1453] Linking CXX shared library lib/libcls_refcount.so.1.0.0
FAILED: lib/libcls_refcount.so.1.0.0
: && /opt/rh/gcc-toolset-11/root/usr/bin/g++ -fPIC -Og -g  -shared -Wl,-soname,libcls_refcount.so.1 -o lib/libcls_refcount.so.1.0.0 src/cls/CMakeFiles/cls_refcount.dir/refcount/cls_refcount.cc.o src/cls/CMakeFiles/cls_refcount.dir/refcount/cls_refcount_ops.cc.o src/cls/CMakeFiles/cls_refcount.dir/__/common/ceph_json.cc.o lib/libjson_spirit.a lib/libcommon_utf8.a  -Wl,--as-needed -latomic && :
/opt/rh/gcc-toolset-11/root/usr/bin/ld: lib/libjson_spirit.a(json_spirit_reader.cpp.o): relocation R_X86_64_32 against `.bss’ can not be used when making a shared object; recompile with -fPIC
/opt/rh/gcc-toolset-11/root/usr/bin/ld: lib/libjson_spirit.a(json_spirit_writer.cpp.o): relocation R_X86_64_32 against `.bss’ can not be used when making a shared object; recompile with -fPIC
collect2: error: ld returned 1 exit status
2024-07-09T12:36:28.842Z
<Rost Khudov> Hmm, the last update for [xxHash submodule](https://github.com/ceph/xxHash/commits/master/) is 8 years ago
2024-07-09T12:38:47.960Z
<Rost Khudov> or did you use <https://github.com/Cyan4973/xxHash> instead of <https://github.com/ceph/xxHash/tree/master>?
2024-07-09T12:41:51.071Z
<Casey Bodley> <https://github.com/ceph/ceph/pull/55076/commits/8cf0e09a34c6a00ae291c53c8bcc8d8b9d81c484> from that pr changed the xxhash submodule. running `git submodule update --init --recursive` should fix your build issues
2024-07-09T12:43:30.120Z
<Rost Khudov> okay, then I think it makes to update <https://github.com/ceph/xxHash> with the latest changes from it's upstream: <https://github.com/Cyan4973/xxHash>
2024-07-09T12:44:17.024Z
<Rost Khudov> because it is not clear why inside ceph v0.8.2 is used and repo under ceph organisation has v0.5.1
2024-07-09T12:44:18.673Z
<Casey Bodley> i think it has been. you might be looking at the wrong branch
2024-07-09T12:44:27.216Z
<Matt Benjamin> well, that was done as above;
2024-07-09T12:44:37.348Z
<Casey Bodley> <https://github.com/ceph/xxHash/tree/update-to-v082>
2024-07-09T12:45:06.899Z
<Rost Khudov> ohhh, you created separate branch...
2024-07-09T12:45:19.112Z
<Rost Khudov> thank you for heads up
2024-07-09T15:12:12.586Z
<Zhipeng Li> how to change ceph proetheus  password?
2024-07-09T19:18:23.823Z
<Igor Gomon> I see that the error above happens when building a shared object (libcls_refcount.so.1.0.0) and trying to link it with static libraries which in turn were built without -fPIC option. Furthermore while looking at [this CMakeLists.txt file](https://github.com/ceph/ceph/blob/main/src/cls/CMakeLists.txt) I see that many libraries are always built as SHARED without taking into account ENABLE_SHARED option. Is this by design or is this an oversight?

Any issue? please create an issue here and use the infra label.