[adding ceph-devel] On Wed, 22 Jun 2016, Igor Fedotov wrote: > Well, having ceph.conf helps a bit. But PluginRegistry::load still appends > plugin type (i.e. "/compressor") to the path. Hence it wouldn't work on a > clean build IMHO... > Another point - I'm not sure if that's a good practice to require config file > to run the tests. IMO it's preferable to have they working properly as-is. I think the solution to this shoudl focus on the cmake developer workflow. Build is in ceph.git/build/, stuff goes in lib/, and the default build should set the default ceph lib path to ./lib. Or, have do_cmake.sh create a ceph.conf that sets these paths in a build/ceph.conf file. Either way, we should make sure the recommended and documented developer workflow (building and running via vstart, running unit tests, etc.) works well out of the box... Ali, does that sound like the right way to go? sage > > > On 22.06.2016 18:57, Sage Weil wrote: > > It works for me because my ./ceph.conf specifies the right dir. If you > > run vstart.sh this will get set up. > > > > > > On Wed, 22 Jun 2016, Igor Fedotov wrote: > > > > > Hi Sage, > > > > > > as I mentioned during the syncup I faced some issues when running > > > ceph_test_objectstore. Namely BluestoreStatFSTest fails. > > > > > > The root cause is that it's unable to load snappy compressor plugin after > > > your > > > recent cleanup. Now store_test attempts to do the load from > > > /usr/local/lib... > > > folder only while previously it tried .libs/compressor one as well. > > > > > > Additionally there are some other test cases that use the same mechanics > > > to > > > locate plugin as the one you removed from store_test. Here they are: > > > > > > test/common/test_async_compressor.cc > > > > > > test/compressor/test_compression_plugin.cc > > > > > > test/compressor/test_compression_plugin_snappy.cc > > > > > > test/compressor/test_compression_plugin_zlib.cc > > > > > > > > > Hence the question is what's your suggestion about resolving that? As for > > > me > > > I'd prefer to be able to use .libs for testing somehow... > > > > > > > > > Thanks, > > > > > > Igor. > > > > > > > > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
