diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index 3ecd52c7e..e71c8cf64 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -19,11 +19,11 @@ jobs: - conda-python-build - conda-python-tests - docs-build - - telemetry-setup - wheel-build-cpp - wheel-build-python - wheel-tests - devcontainer + - telemetry-setup secrets: inherit uses: rapidsai/shared-workflows/.github/workflows/pr-builder.yaml@branch-25.02 if: always() @@ -33,9 +33,11 @@ jobs: runs-on: ubuntu-latest continue-on-error: true env: - OTEL_SERVICE_NAME: "pr-rmm" + OTEL_SERVICE_NAME: "pr-rmm" steps: - name: Telemetry setup + # This gate is here and not at the job level because we need the job to not be skipped, + # since other jobs depend on it. if: ${{ vars.TELEMETRY_ENABLED == 'true' }} uses: rapidsai/shared-actions/telemetry-dispatch-stash-base-env-vars@main changed-files: @@ -141,16 +143,11 @@ jobs: sccache -s; telemetry-summarize: - runs-on: ubuntu-latest + # This job must use a self-hosted runner to record telemetry traces. + runs-on: linux-amd64-cpu4 needs: pr-builder if: ${{ vars.TELEMETRY_ENABLED == 'true' && !cancelled() }} continue-on-error: true steps: - - name: Load stashed telemetry env vars - uses: rapidsai/shared-actions/telemetry-dispatch-load-base-env-vars@main - with: - load_service_name: true - name: Telemetry summarize - uses: rapidsai/shared-actions/telemetry-dispatch-write-summary@main - with: - cert_concat: "${{ secrets.OTEL_EXPORTER_OTLP_CA_CERTIFICATE }};${{ secrets.OTEL_EXPORTER_OTLP_CLIENT_CERTIFICATE }};${{ secrets.OTEL_EXPORTER_OTLP_CLIENT_KEY }}" + uses: rapidsai/shared-actions/telemetry-dispatch-summarize@main diff --git a/include/rmm/mr/device/limiting_resource_adaptor.hpp b/include/rmm/mr/device/limiting_resource_adaptor.hpp index d19fa3a0a..16729fd68 100644 --- a/include/rmm/mr/device/limiting_resource_adaptor.hpp +++ b/include/rmm/mr/device/limiting_resource_adaptor.hpp @@ -194,25 +194,6 @@ class limiting_resource_adaptor final : public device_memory_resource { std::size_t alignment_; }; -/** - * @brief Convenience factory to return a `limiting_resource_adaptor` around the - * upstream resource `upstream`. - * - * @tparam Upstream Type of the upstream `device_memory_resource`. - * @param upstream Pointer to the upstream resource - * @param allocation_limit Maximum amount of memory to allocate - * @return The new limiting resource adaptor - */ -template -[[deprecated( - "make_limiting_adaptor is deprecated in RMM 24.10. Use the limiting_resource_adaptor constructor " - "instead.")]] -limiting_resource_adaptor make_limiting_adaptor(Upstream* upstream, - std::size_t allocation_limit) -{ - return limiting_resource_adaptor{upstream, allocation_limit}; -} - /** @} */ // end of group } // namespace mr } // namespace RMM_NAMESPACE diff --git a/include/rmm/mr/device/logging_resource_adaptor.hpp b/include/rmm/mr/device/logging_resource_adaptor.hpp index 3e6e5babc..2d2291e3f 100644 --- a/include/rmm/mr/device/logging_resource_adaptor.hpp +++ b/include/rmm/mr/device/logging_resource_adaptor.hpp @@ -334,52 +334,6 @@ class logging_resource_adaptor final : public device_memory_resource { ///< allocation requests }; -/** - * @brief Convenience factory to return a `logging_resource_adaptor` around the - * upstream resource `upstream`. - * - * @tparam Upstream Type of the upstream `device_memory_resource`. - * @param upstream Pointer to the upstream resource - * @param filename Name of the file to write log info. If not specified, - * retrieves the log file name from the environment variable "RMM_LOG_FILE". - * @param auto_flush If true, flushes the log for every (de)allocation. Warning, this will degrade - * performance. - * @return The new logging resource adaptor - */ -template -[[deprecated( - "make_logging_adaptor is deprecated in RMM 24.10. Use the logging_resource_adaptor constructor " - "instead.")]] -logging_resource_adaptor make_logging_adaptor( - Upstream* upstream, - std::string const& filename = logging_resource_adaptor::get_default_filename(), - bool auto_flush = false) -{ - return logging_resource_adaptor{upstream, filename, auto_flush}; -} - -/** - * @brief Convenience factory to return a `logging_resource_adaptor` around the - * upstream resource `upstream`. - * - * @tparam Upstream Type of the upstream `device_memory_resource`. - * @param upstream Pointer to the upstream resource - * @param stream The ostream to write log info. - * @param auto_flush If true, flushes the log for every (de)allocation. Warning, this will degrade - * performance. - * @return The new logging resource adaptor - */ -template -[[deprecated( - "make_logging_adaptor is deprecated in RMM 24.10. Use the logging_resource_adaptor constructor " - "instead.")]] -logging_resource_adaptor make_logging_adaptor(Upstream* upstream, - std::ostream& stream, - bool auto_flush = false) -{ - return logging_resource_adaptor{upstream, stream, auto_flush}; -} - /** @} */ // end of group } // namespace mr } // namespace RMM_NAMESPACE diff --git a/include/rmm/mr/device/polymorphic_allocator.hpp b/include/rmm/mr/device/polymorphic_allocator.hpp index 442632d4f..2e6fcd70d 100644 --- a/include/rmm/mr/device/polymorphic_allocator.hpp +++ b/include/rmm/mr/device/polymorphic_allocator.hpp @@ -293,25 +293,6 @@ bool operator!=(stream_allocator_adaptor const& lhs, stream_allocator_adaptor return not(lhs == rhs); } -/** - * @brief Factory to construct a `stream_allocator_adaptor` from an existing stream-ordered - * allocator. - * - * @tparam Allocator Type of the stream-ordered allocator - * @param allocator The allocator to use as the underlying allocator of the - * `stream_allocator_adaptor` - * @param stream The stream on which the `stream_allocator_adaptor` will perform (de)allocations - * @return A `stream_allocator_adaptor` wrapping `allocator` and `s` - */ -template -[[deprecated( - "make_stream_allocator_adaptor is deprecated in RMM 24.10. Use the stream_allocator_adaptor " - "constructor " - "instead.")]] -auto make_stream_allocator_adaptor(Allocator const& allocator, cuda_stream_view stream) -{ - return stream_allocator_adaptor{allocator, stream}; -} /** @} */ // end of group } // namespace mr } // namespace RMM_NAMESPACE diff --git a/include/rmm/mr/device/statistics_resource_adaptor.hpp b/include/rmm/mr/device/statistics_resource_adaptor.hpp index 025c51aa7..8de6e1fe6 100644 --- a/include/rmm/mr/device/statistics_resource_adaptor.hpp +++ b/include/rmm/mr/device/statistics_resource_adaptor.hpp @@ -287,23 +287,6 @@ class statistics_resource_adaptor final : public device_memory_resource { device_async_resource_ref upstream_; }; -/** - * @brief Convenience factory to return a `statistics_resource_adaptor` around the - * upstream resource `upstream`. - * - * @param upstream Pointer to the upstream resource - * @return The new statistics resource adaptor - */ -template -[[deprecated( - "make_statistics_adaptor is deprecated in RMM 24.10. Use the statistics_resource_adaptor " - "constructor " - "instead.")]] -statistics_resource_adaptor make_statistics_adaptor(Upstream* upstream) -{ - return statistics_resource_adaptor{upstream}; -} - /** @} */ // end of group } // namespace mr } // namespace RMM_NAMESPACE diff --git a/include/rmm/mr/device/tracking_resource_adaptor.hpp b/include/rmm/mr/device/tracking_resource_adaptor.hpp index 09631960e..1b2c13ba9 100644 --- a/include/rmm/mr/device/tracking_resource_adaptor.hpp +++ b/include/rmm/mr/device/tracking_resource_adaptor.hpp @@ -283,23 +283,6 @@ class tracking_resource_adaptor final : public device_memory_resource { // allocation requests }; -/** - * @brief Convenience factory to return a `tracking_resource_adaptor` around the - * upstream resource `upstream`. - * - * @tparam Upstream Type of the upstream `device_memory_resource`. - * @param upstream Pointer to the upstream resource - * @return The new tracking resource adaptor - */ -template -[[deprecated( - "make_tracking_adaptor is deprecated in RMM 24.10. Use the tracking_resource_adaptor constructor " - "instead.")]] -tracking_resource_adaptor make_tracking_adaptor(Upstream* upstream) -{ - return tracking_resource_adaptor{upstream}; -} - /** @} */ // end of group } // namespace mr } // namespace RMM_NAMESPACE