Skip to content

fixing cross builds#112

Merged
vMaroon merged 1 commit intollm-d:mainfrom
Gregory-Pereira:fix-cross-builds
May 19, 2025
Merged

fixing cross builds#112
vMaroon merged 1 commit intollm-d:mainfrom
Gregory-Pereira:fix-cross-builds

Conversation

@Gregory-Pereira
Copy link
Member

Cross builds were failing in the process of rebuilding from my ENV when adjusting blocksize to test prefix aware routing demo.

Issue: platform was not specified on the container, so it was pulling aarch64 base images. Then all the go build stuff would target linux/amd64 on top of an aarch64 image causing cross build issues. When specifying same platform to the image builds we get success.

Tested with:

GIT_NM_USER=*** KV_CACHE_MANAGER_TOKEN=ghp_*** TARGETOS=linux TARGETARCH=amd64 make image-build

Builds on mac.

Signed-off-by: greg pereira <grpereir@redhat.com>
@Gregory-Pereira
Copy link
Member Author

cc @oglok, he can verify this works

@Gregory-Pereira
Copy link
Member Author

CI fails due to a CI related issue:

/home/runner/go/pkg/mod/cache/vcs/157c81e33862038d00edf2ec6a37e908ca4c01584ce757016e3dbd45bbb3844e: exit status 128:
	fatal: could not read Password for 'https://github.com/': terminal prompts disabled

gotta be an org / auth thing. cc @clubanderson Let me know how I can fix this, will try to cleanup my own messes 🙃

@vMaroon
Copy link
Member

vMaroon commented May 19, 2025

@Gregory-Pereira we currently have a "hack" where a token for importing the private llm-d-kv-cache-manager repo is passed as a secret. By GitHub design, secrets are only passed to PRs with bases that are within this repository.

We didn't disclose this as this token will be dropped tomorrow.

@vMaroon vMaroon merged commit 63a175f into llm-d:main May 19, 2025
1 check failed
pierDipi pushed a commit to pierDipi/llm-d-inference-scheduler that referenced this pull request Nov 28, 2025
…gest to 84286c7 (llm-d#112)

Signed-off-by: konflux-internal-p02 <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
Co-authored-by: konflux-internal-p02[bot] <170854209+konflux-internal-p02[bot]@users.noreply.github.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.

3 participants