Project

General

Profile

Actions

Bug #62734

closed

src/test/objectstore/test_bluefs.cc: cannot initialize a parameter of type 'bluefs_shared_alloc_context_t *' with an rvalue of type 'int'

Added by Laura Flores 8 months ago. Updated 8 months ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://jenkins.ceph.com/job/ceph-pull-requests/121471/

FAILED: src/test/objectstore/CMakeFiles/unittest_bluefs.dir/test_bluefs.cc.o 
/usr/bin/ccache /usr/bin/clang++-14 -DBOOST_ASIO_DISABLE_THREAD_KEYWORD_EXTENSION -DBOOST_ASIO_USE_TS_EXECUTOR_AS_DEFAULT -DBOOST_DATE_TIME_DYN_LINK -DBOOST_DATE_TIME_NO_LIB -DBOOST_IOSTREAMS_DYN_LINK -DBOOST_IOSTREAMS_NO_LIB -DBOOST_PROGRAM_OPTIONS_DYN_LINK -DBOOST_PROGRAM_OPTIONS_NO_LIB -DBOOST_RANDOM_DYN_LINK -DBOOST_RANDOM_NO_LIB -DBOOST_SYSTEM_DYN_LINK -DBOOST_SYSTEM_NO_LIB -DBOOST_THREAD_DYN_LINK -DBOOST_THREAD_NO_LIB -DHAVE_CONFIG_H -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE -D_REENTRANT -D_THREAD_SAFE -D__CEPH__ -D__STDC_FORMAT_MACROS -D__linux__ -I/home/jenkins-build/build/workspace/ceph-pull-requests/build/src/include -I/home/jenkins-build/build/workspace/ceph-pull-requests/src -isystem /opt/ceph/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/build/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/xxHash -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/fmt/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/googletest/googlemock/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/googletest/googlemock -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/googletest/googletest/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/googletest/googletest -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/rocksdb/include -isystem /usr/include/fuse -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/jaegertracing/opentelemetry-cpp/api/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/jaegertracing/opentelemetry-cpp/exporters/jaeger/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/jaegertracing/opentelemetry-cpp/ext/include -isystem /home/jenkins-build/build/workspace/ceph-pull-requests/src/jaegertracing/opentelemetry-cpp/sdk/include -g -Werror -fPIE -fno-builtin-malloc -fno-builtin-calloc -fno-builtin-realloc -fno-builtin-free -DBOOST_PHOENIX_STL_TUPLE_H_ -Wall -fno-strict-aliasing -fsigned-char -Wtype-limits -Wignored-qualifiers -Wpointer-arith -Werror=format-security -Winit-self -Wno-unknown-pragmas -Wnon-virtual-dtor -Wno-ignored-qualifiers -ftemplate-depth-1024 -Wpessimizing-move -Wredundant-move -Wno-inconsistent-missing-override -Wno-mismatched-tags -Wno-unused-private-field -Wno-address-of-packed-member -Wno-unused-function -Wno-unused-local-typedef -Wno-varargs -Wno-gnu-designator -Wno-missing-braces -Wno-parentheses -Wno-deprecated-register -DCEPH_DEBUG_MUTEX -D_GLIBCXX_ASSERTIONS -fdiagnostics-color=auto -std=c++20 -MD -MT src/test/objectstore/CMakeFiles/unittest_bluefs.dir/test_bluefs.cc.o -MF src/test/objectstore/CMakeFiles/unittest_bluefs.dir/test_bluefs.cc.o.d -o src/test/objectstore/CMakeFiles/unittest_bluefs.dir/test_bluefs.cc.o -c /home/jenkins-build/build/workspace/ceph-pull-requests/src/test/objectstore/test_bluefs.cc
/home/jenkins-build/build/workspace/ceph-pull-requests/src/test/objectstore/test_bluefs.cc:1415:71: error: cannot initialize a parameter of type 'bluefs_shared_alloc_context_t *' with an rvalue of type 'int'
  ASSERT_EQ(0, fs.add_block_device(BlueFS::BDEV_DB, bdev.path, false, 1048576));

Actions #1

Updated by Laura Flores 8 months ago

Possibly from this PR: https://github.com/ceph/ceph/pull/50325

It was just merged. The Shaman build initially went fine, but something may have happened afterwards to cause incompatibility.

Actions #2

Updated by Jos Collin 8 months ago

Laura Flores wrote:

Possibly from this PR: https://github.com/ceph/ceph/pull/50325

It was just merged. The Shaman build initially went fine, but something may have happened afterwards to cause incompatibility.

https://github.com/ceph/ceph/pull/50325/files#diff-5eb5704e3a1c0c9d042492c4e87d8c864ad9cb96afa1249f9ec294acc2fba977L437
That caused it.

Actions #3

Updated by Rishabh Dave 8 months ago

  • Priority changed from Normal to Urgent
  • Pull request ID set to 53322
Actions #4

Updated by Rishabh Dave 8 months ago

Jos Collin wrote:

Laura Flores wrote:

Possibly from this PR: https://github.com/ceph/ceph/pull/50325

It was just merged. The Shaman build initially went fine, but something may have happened afterwards to cause incompatibility.

https://github.com/ceph/ceph/pull/50325/files#diff-5eb5704e3a1c0c9d042492c4e87d8c864ad9cb96afa1249f9ec294acc2fba977L437
That caused it.

Right, my fix is related to this line.

Actions #5

Updated by Rishabh Dave 8 months ago

  • Pull request ID changed from 53322 to 53321

The fix for this issue was merged through PR 53321 rather than 53322 because make check CI job won't pass until both the commits were present on the same branch

Actions #6

Updated by Rishabh Dave 8 months ago

  • Status changed from New to Resolved
  • Assignee set to Rishabh Dave
Actions

Also available in: Atom PDF