Skip to content

fix(xtest): Use embedded platform otdfctl in auto #445

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
dmihalcik-virtru merged 2 commits into from
May 5, 2026
Merged

fix(xtest): Use embedded platform otdfctl in auto #445

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b0371b4
fix(setup-cli-tool): use platform-embedded otdfctl in auto-detect mode
dmihalcik-virtru Apr 30, 2026
6d26147
Merge branch 'main' into DSPX-3086-enable-pqc-suite
dmihalcik-virtru May 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 6 additions & 8 deletions xtest/setup-cli-tool/action.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,14 +175,12 @@ runs:
checkout_path="${INPUT_PATH}/${INPUT_SDK}/platform-src/${tag}"
fi
elif [[ "$INPUT_SDK" == "go" && -n "$PLATFORM_OTDFCTL_DIR" && "$needs_source" == "true" ]]; then
# Auto-detect fallback: resolver used standalone repo but the
# test job detected otdfctl in the platform checkout.
# NOTE: SHA comparison across repos is not meaningful (the standalone
# repo and platform repo have different commit histories), so we
# cannot safely reuse the platform checkout here. Fall through to
# a standalone checkout. To use the platform source, set
# otdfctl-source=platform explicitly.
echo "::notice::Go version ${tag} resolved from standalone repo; platform checkout available but cannot auto-reuse (different repo). Set otdfctl-source=platform to use the platform source."
# Auto-detect: resolver used standalone repo but detect-otdfctl found
# an embedded otdfctl/ in the platform checkout. PLATFORM_OTDFCTL_DIR
# is only set after confirming the directory exists, so reuse it directly.
echo "::notice::Go version ${tag} resolved from standalone repo; reusing platform-embedded otdfctl at $PLATFORM_OTDFCTL_DIR."
use_existing=true
needs_source=false
fi

echo "needs-checkout-${slot}=${needs_source}" >> "$GITHUB_OUTPUT"
Expand Down
Loading