Workflow failed: open-telemetry/opentelemetry.io Build Semantic Conventions (daily) (#6)

[opentelemetrybot]

See open-telemetry/opentelemetry.io Build Semantic Conventions (daily) #6.

[trask]

I think the failure is due to the temporary patch put in place due to spec and semconv releases being staggered:

https://github.com/open-telemetry/opentelemetry.io/blob/f45870446397e6d05347ff61cff9d7b6f219f05a/scripts/content-modules/adjust-pages.pl#L138-L142

@chalin can you confirm if there's something for us to do / fix here before we make the semconv release this week?

[chalin]

Right, it's because of the patch. I have been trying to think of a simple way for the patch code to know that it's using semconv at HEAD, and not apply the patch. Haven't found something I'm satisfied with yet.

[opentelemetrybot]

See open-telemetry/opentelemetry.io Build Semantic Conventions (daily) #7.

[chalin]

I've fixed the patch script, see:

• [CI] Don't apply spec patch if submodule is at HEAD opentelemetry.io#6047

[opentelemetrybot]

See open-telemetry/opentelemetry.io Build Semantic Conventions (daily) #8.

[chalin]

Much better now that patches aren't being unnecessarily applied.

From the logs I see that most invalid links are fixes required in OTel.io pages, except for this one:

  hash does not exist --- docs/specs/semconv/exceptions/exceptions-logs/index.html --> /docs/specs/semconv/exceptions/exceptions-spans/#stacktrace-representation

@trask - can I let you dig into that?

[trask]

@trask - can I let you dig into that?

yes, definitely

[trask]

hash does not exist --- docs/specs/semconv/exceptions/exceptions-logs/index.html --> /docs/specs/semconv/exceptions/exceptions-spans/#stacktrace-representation

@chalin sorry, I think I don't understand something about the website link translations, this link looks correct, and works when I test it via its markdown

[chalin]

Here's the cause:

• [docs] incomplete <-- toc --> directive #1799

[opentelemetrybot]

See open-telemetry/opentelemetry.io Build Semantic Conventions (daily) #9.

[chalin]

@trask - can we get the following merged before the next release?

• [editorial] Fix alias in general/trace.md #1805

[chalin]

Semconv spec @ HEAD is now green via:

• [WIP] Update semantic-conventions to 1.30.0-dev opentelemetry.io#6060

[opentelemetrybot]

See open-telemetry/opentelemetry.io Build Semantic Conventions (daily) #10.