Skip to content

Conversation

jmacd
Copy link
Contributor

@jmacd jmacd commented Oct 1, 2025

This replaces #7735.
It's a complete rewrite.

This is to announce our new sampling specifications.


PREVIEW: https://deploy-preview-7967--opentelemetry.netlify.app/blog/2025/sampling-milestones/

@svrnm
Copy link
Member

svrnm commented Oct 2, 2025

thanks @jmacd , take it out of draft status whenever you need us to take a look for review :-)

Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From a technical perspective, this looks great!

@jmacd jmacd marked this pull request as ready for review October 9, 2025 18:19
@jmacd jmacd requested a review from a team as a code owner October 9, 2025 18:19
@jmacd jmacd changed the title [Draft2] Sampling milestones blog post Sampling milestones blog post Oct 9, 2025
@jmacd
Copy link
Contributor Author

jmacd commented Oct 13, 2025

/fix:refcache

@jmacd
Copy link
Contributor Author

jmacd commented Oct 13, 2025

I have tried the command locally:

% npm run fix:refcache

> fix:refcache
> npm run _refcache:prune; npm run check:links


> _refcache:prune
> npx gulp prune

Error [ERR_REQUIRE_ESM]: require() of ES Module /home/jmacd/src/otel/opentelemetry.io/node_modules/markdownlint/lib/exports-async.mjs not supported.
Instead change the require of /home/jmacd/src/otel/opentelemetry.io/node_modules/markdownlint/lib/exports-async.mjs to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/home/jmacd/src/otel/opentelemetry.io/gulp-src/lint-md.js:5:22)
    at requireDir (/home/jmacd/src/otel/opentelemetry.io/node_modules/require-dir/index.js:128:33)
    at Object.<anonymous> (/home/jmacd/src/otel/opentelemetry.io/gulpfile.js:3:1) {
  code: 'ERR_REQUIRE_ESM'
}

> precheck:links
> npm run build && npm run fix:htmltest-config


> prebuild
> npm run _prebuild


> _prebuild
> npm run seq -- get:submodule cp:spec


> seq
> bash -c 'for cmd in "$@"; do npm run $cmd || exit 1; done' - get:submodule cp:spec


> get:submodule
> npm run _get:${GET:-submodule} --


> _get:submodule
> set -x && git submodule update --init ${DEPTH:- --depth 999}

+ git submodule update --init --depth 999

> postget:submodule
> git submodule

 e9411eec532047b73e68acb492ff7dd3e2b91256 content-modules/community (heads/main-2-ge9411ee)
 fb2ff487c70ac1895652ab5c4a965dfc2ae20d03 content-modules/opamp-spec (v0.10.0)
 5661ff0ded32cf1b83f1147dae96ca403c198504 content-modules/opentelemetry-go (exporters/stdout/stdoutlog/v0.3.0)
 cce0f6a84169b978bba17b3bf7cbf972a0fda230 content-modules/opentelemetry-java-examples (cce0f6a)
 c0a98a1847d3124ac5f9ecd02d0e2d2732bbb590 content-modules/opentelemetry-proto (v0.11.0-259-gc0a98a1)
 7295dc4defb9b0f53d8170dc2fb0629257d1d696 content-modules/opentelemetry-specification (v1.49.0)
 aec6e9d3e86754683dab7c707655d69d953b2768 content-modules/semantic-conventions (v1.37.0)
 82d492bbf2725009451dcb5fe32d777bddc900e3 themes/docsy (v0.12.0-25-g82d492b)

> cp:spec
> scripts/content-modules/cp-pages.sh

OTEL SPEC pages: copied and processed
OTLP SPEC pages: copied and processed
OTLP SPEC protos copied and processed
COMMUNITY pages: copied and processed
SEM CONV  pages: copied and processed
OpAMP SPEC page: copied and processed

> build
> npm run _build --


> _build
> npm run _hugo -- -e dev --buildDrafts --buildFuture --baseURL "${DEPLOY_PRIME_URL:-http://localhost}"


> _hugo
> hugo --cleanDestinationDir -e dev --buildDrafts --buildFuture --baseURL http://localhost

Total in 5 ms
Error: failed to load modules: module "github.com/FortAwesome/Font-Awesome" not found in "/home/jmacd/src/otel/opentelemetry.io/themes/github.com/FortAwesome/Font-Awesome"; either add it as a Hugo Module or store it in "/home/jmacd/src/otel/opentelemetry.io/themes".: module does not exist

@otelbot-docs
Copy link
Contributor

otelbot-docs bot commented Oct 13, 2025

fix:refcache applied successfully in run 18474612946.

@svrnm
Copy link
Member

svrnm commented Oct 15, 2025

I have tried the command locally:

This often either points to an old version of nodejs being used, or some packages not being updated for a while, but don't worry I can fix this on our end, if the blog post is otherwise good to go.

@otelbot-docs otelbot-docs bot requested a review from a team October 15, 2025 14:01
@svrnm svrnm added this pull request to the merge queue Oct 15, 2025
Merged via the queue into open-telemetry:main with commit fc5e1ef Oct 15, 2025
22 checks passed
svrnm added a commit to svrnm/opentelemetry.io that referenced this pull request Oct 16, 2025
Co-authored-by: Otmar Ertl <[email protected]>
Co-authored-by: Tiffany Hrabusa <[email protected]>
Co-authored-by: otelbot <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
Co-authored-by: Severin Neumann <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants