Skip to content

ci: fix security-model.adoc Antora build + relative-link errors#2836

Merged
oscerd merged 1 commit into
apache:mainfrom
oscerd:ci-issue/kamelets-security-model-asciidoc-attr-refs
May 15, 2026
Merged

ci: fix security-model.adoc Antora build + relative-link errors#2836
oscerd merged 1 commit into
apache:mainfrom
oscerd:ci-issue/kamelets-security-model-asciidoc-attr-refs

Conversation

@oscerd
Copy link
Copy Markdown
Contributor

@oscerd oscerd commented May 15, 2026

CI Fix

Two independent camel-website CI failures, both rooted in this single page
docs/modules/ROOT/pages/security-model.adoc (added in #2835 / issue #2834).
Neither is caused by the camel-website PR that surfaced them — they break
every camel-website pull request.

Failed runs (camel-website): https://github.com/apache/camel-website/actions/runs/25926572539

1. build:antora-perf — Asciidoctor missing-attribute warnings

{{property}} / ${body} placeholder text is parsed by Asciidoctor as
attribute references (skipping reference to missing attribute: property ×22,
body ×1). The strict production Antora build fails on these.
Fix: backslash-escape the inner reference ({\{property}}, $\{body}) —
canonical Asciidoctor idiom, renders unchanged, no warning.

2. check:htmlcamel/relative-links violations

The page used absolute https://camel.apache.org/... URLs (4 errors). The
camel-website linter requires in-site links to be relative.
Fix: convert to root-relative link:/security/ and
link:/manual/security-model.html, matching apache/camel core security-model.adoc
(fixed identically in apache/camel#23224). private-security@camel.apache.org
email references intentionally left unchanged.

Documentation-only; no rendered-output change. Verified statically (no
Asciidoctor/Antora CLI in the environment); the camel-website Antora build is
the definitive check.

Deferred Issues

None — both root causes fixed inline. Sibling per-subproject security models
have the same relative-link issue and are fixed in companion PRs
(camel-kafka-connector, camel-quarkus, camel-karaf).


Claude Code on behalf of Andrea Cosentino

The Kamelet Catalog security model page (added in apache#2835, issue apache#2834) uses
Kamelet property-placeholder syntax {{property}} and Camel simple syntax
${body} as literal text in inline prose and tables. Asciidoctor parses the
inner {property} / {body} as attribute references, cannot resolve them, and
emits 'skipping reference to missing attribute' warnings.

camel-website's strict production Antora build (build:antora-perf) aggregates
this page from apache/camel-kamelets main and fails on those warnings, which
turns every camel-website pull request red regardless of its content.

Escape the inner attribute reference with a backslash ({\{property}},
$\{body}); Asciidoctor consumes the backslash and renders the literal
{{property}} / ${body} unchanged, with no attribute resolution and no
warning. Documentation-only change; no rendered-output difference.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
@oscerd oscerd requested review from davsclaus and squakez May 15, 2026 16:48
@oscerd oscerd merged commit df9a0aa into apache:main May 15, 2026
1 check passed
@oscerd oscerd changed the title ci: escape Kamelet placeholder syntax in security-model.adoc to fix Antora build ci: fix security-model.adoc Antora build + relative-link errors May 15, 2026
oscerd added a commit that referenced this pull request May 16, 2026
Follow-up to #2836. That PR was squash-merged with only its first commit
(placeholder escaping); the relative-link commit was pushed afterwards and
never merged, so camel-kamelets main still carries 4 absolute
https://camel.apache.org/... links. They are now the sole remaining
camel/relative-links failure in the camel-website check:html step (4 errors
in camel-kamelets/next/security-model.html), keeping every camel-website PR
red.

Convert the 4 in-site links to root-relative link: macros (link:/security/,
link:/manual/security-model.html), matching apache/camel core
security-model.adoc (apache/camel#23224) and the sibling subproject fixes
(apache/camel-kafka-connector#1774, apache/camel-quarkus#8661). The merged
placeholder escaping is left intact; private-security@camel.apache.org email
references are unchanged. Documentation-only; same link targets, rendered
relative.

Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant