Spectator2.0

.gitignore

@@ -1,7 +1,8 @@
 .DS_Store
 .idea/
+.vscode/
 CMakeUserPresets.json
 cmake-build/
 conan_provider.cmake
 spectator/valid_chars.inc
-venv/
+venv/

CMakeLists.txt

@@ -1,70 +1,26 @@
-cmake_minimum_required(VERSION 3.23)
-
-project(spectator-cpp)
+# filepath: /home/ebadeaux/Saturday/spec2-cpp/CMakeLists.txt
+cmake_minimum_required(VERSION 3.15)
+project(spectator-cpp VERSION 2.0 LANGUAGES CXX)
 
+# Set C++ standard
 set(CMAKE_CXX_STANDARD 20)
-set(CMAKE_CXX_STANDARD_REQUIRED True)
+set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
-set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib)
-set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
 
-add_compile_options(-pedantic -Werror -Wall -Wno-missing-braces -fno-omit-frame-pointer "$<$<CONFIG:Debug>:-fsanitize=address>")
+# Options
+option(BUILD_TESTS "Build the test suite" ON)
 
-find_package(absl REQUIRED)
-find_package(asio REQUIRED)
-find_package(Backward REQUIRED)
-find_package(fmt REQUIRED)
-find_package(GTest REQUIRED)
+# Find dependencies (handled by Conan)
 find_package(spdlog REQUIRED)
+find_package(GTest REQUIRED)
+find_package(Boost REQUIRED COMPONENTS system)
 
-include(CTest)
-
-#-- spectator_test test executable
-file(GLOB spectator_test_source_files
-    "spectator/*_test.cc"
-    "spectator/test_*.cc"
-    "spectator/test_*.h"
-)
-add_executable(spectator_test ${spectator_test_source_files})
-target_link_libraries(spectator_test
-    spectator
-    gtest::gtest
-)
-add_test(
-    NAME spectator_test
-    COMMAND spectator_test
-    WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
-)
-
-#-- spectator library
-add_library(spectator SHARED
-    "spectator/logger.cc"
-    "spectator/publisher.cc"
-    "spectator/config.h"
-    "spectator/id.h"
-    "spectator/logger.h"
-    "spectator/measurement.h"
-    "spectator/meter_type.h"
-    "spectator/publisher.h"
-    "spectator/registry.h"
-    "spectator/stateful_meters.h"
-    "spectator/stateless_meters.h"
-    "spectator/valid_chars.inc"
-)
-target_link_libraries(spectator
-    abseil::abseil
-    asio::asio
-    Backward::Backward
-    fmt::fmt
-    spdlog::spdlog
-)
+# Build tests if enabled
+if(BUILD_TESTS)
+    enable_testing()
+endif()
 
-#-- generator tools
-add_executable(gen_valid_chars "tools/gen_valid_chars.cc")
+# Add subdirectories
+add_subdirectory(libs)
+add_subdirectory(spectator)
 
-#-- file generators, must exist where the outputs are referenced
-add_custom_command(
-    OUTPUT "spectator/valid_chars.inc"
-    COMMAND "${CMAKE_BINARY_DIR}/bin/gen_valid_chars" > "${CMAKE_SOURCE_DIR}/spectator/valid_chars.inc"
-    DEPENDS gen_valid_chars
-)

Dockerfiles/README.md

@@ -0,0 +1,32 @@
+# Docker Build :hammer_and_wrench:
+
+The `spectator-cpp` project also supports a platform-agnostic build. The only prerequisite is the 
+installation of `Docker`. Once `Docker` is installed, you can build the project by running the
+following commands from the root directory of the project.
+
+## Linux & Mac :penguin:
+
+##### Warning:
+
+- Do not prepend the command with `sudo` on Mac
+- Start `Docker` before opening terminal on Mac 
+
+```shell
+sudo docker build -t spectator-cpp-image -f Dockerfiles/Ubuntu.Dockerfile .
+sudo docker run -it spectatord-cpp-image
+./build.sh
+```
+
+## Windows
+
+##### Warning:
+
+- Start `Docker` before opening `Powershell`
+
+```shell
+docker build -t spectatord-cpp-image -f Dockerfiles/Ubuntu.Dockerfile .
+docker run -it spectatord-cpp-image
+apt-get install dos2unix
+dos2unix build.sh
+./build.sh
+```

Dockerfiles/Ubuntu.Dockerfile

@@ -0,0 +1,25 @@
+# Use the official Ubuntu base image from Docker Hub
+FROM ubuntu:latest
+
+# Add a few required packages for building and developer tools
+RUN apt-get update && apt-get install -y \
+    vim \
+    git \
+    python3 \
+    python3-venv \
+    gcc-13\
+    g++-13 \
+    cmake 
+
+# Create a default working directory
+WORKDIR /home/ubuntu/spectator-cpp
+
+# Copy all files & folders in the projects root directory 
+# Exclude files listed in the dockerignore file
+COPY ../ /home/ubuntu/spectator-cpp
+
+# Setup Python virtual environment using the existing script
+RUN chmod +x setup-venv.sh && ./setup-venv.sh
+
+# When container starts, activate the virtual environment
+ENTRYPOINT ["/bin/bash", "-c", "source venv/bin/activate && exec /bin/bash"]

Dockerfiles/Ubuntu.Dockerfile.dockerignore

@@ -0,0 +1,2 @@
+# Ignore copying the default build folder if it exists
+cmake-build/

LICENSE

@@ -199,4 +199,4 @@
    distributed under the License is distributed on an "AS IS" BASIS,
    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    See the License for the specific language governing permissions and
-   limitations under the License.
+   limitations under the License.

OSSMETADATA

@@ -1 +1 @@
-osslifecycle=active
+osslifecycle=active

README.md

@@ -8,72 +8,7 @@ consists of a thin client designed to send metrics through [spectatord](https://
 ## Instrumenting Code
 
 ```C++
-#include <spectator/registry.h>
 
-// use default values
-static constexpr auto kDefault = 0;
-
-struct Request {
-  std::string country;
-};
-
-struct Response {
-  int status;
-  int size;
-};
-
-class Server {
- public:
-  explicit Server(spectator::Registry* registry)
-      : registry_{registry},
-        request_count_id_{registry->CreateId("server.requestCount", spectator::Tags{})},
-        request_latency_{registry->GetTimer("server.requestLatency")},
-        response_size_{registry->GetDistributionSummary("server.responseSizes")} {}
-
-  Response Handle(const Request& request) {
-    auto start = std::chrono::steady_clock::now();
-
-    // do some work and obtain a response...
-    Response res{200, 64};
-
-    // Update the Counter id with dimensions, based on information in the request. The Counter
-    // will be looked up in the Registry, which is a fairly cheap operation, about the same as
-    // the lookup of an id object in a map. However, it is more expensive than having a local
-    // variable set to the Counter.
-    auto cnt_id = request_count_id_
-        ->WithTag("country", request.country)
-        ->WithTag("status", std::to_string(res.status));
-    registry_->GetCounter(std::move(cnt_id))->Increment();
-    request_latency_->Record(std::chrono::steady_clock::now() - start);
-    response_size_->Record(res.size);
-    return res;
-  }
-
- private:
-  spectator::Registry* registry_;
-  std::shared_ptr<spectator::Id> request_count_id_;
-  std::shared_ptr<spectator::Timer> request_latency_;
-  std::shared_ptr<spectator::DistributionSummary> response_size_;
-};
-
-Request get_next_request() {
-  return Request{"US"};
-}
-
-int main() {
-  auto logger = spdlog::stdout_color_mt("console"); 
-  std::unordered_map<std::string, std::string> common_tags{{"xatlas.process", "some-sidecar"}};
-  spectator::Config cfg{"unix:/run/spectatord/spectatord.unix", common_tags};
-  spectator::Registry registry{std::move(cfg), logger);
-
-  Server server{&registry};
-
-  for (auto i = 1; i <= 3; ++i) {
-    // get a request
-    auto req = get_next_request();
-    server.Handle(req);
-  }
-}
 ```
 
 ## High-Volume Publishing
@@ -106,4 +41,4 @@ source venv/bin/activate
     * Open the project. The wizard will show three CMake profiles.
         * Disable the default Cmake `Debug` profile.
         * Enable the CMake `conan-debug` profile.
-    * CLion > View > Tool Windows > Conan > (gear) > Conan Executable: `$PROJECT_HOME/venv/bin/conan`
+    * CLion > View > Tool Windows > Conan > (gear) > Conan Executable: `$PROJECT_HOME/venv/bin/conan`