Skip to content

fix(vector): Use relative path to patchable checked out source#1471

Open
NickLarsenNZ wants to merge 2 commits intomainfrom
fix/vector-sbom-copy
Open

fix(vector): Use relative path to patchable checked out source#1471
NickLarsenNZ wants to merge 2 commits intomainfrom
fix/vector-sbom-copy

Conversation

@NickLarsenNZ
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ commented Apr 17, 2026

Fixes this error:

find: '/src/': No such file or directory

Because for this (vector) build, WORKDIR is /stackable (so patchable checked out src/ is below that)

Fixes this error:

```
find: '/src/': No such file or directory
```

Because fot this build, WORKDIR is /stackable (so patchable checked out src/ is below that)
@NickLarsenNZ NickLarsenNZ self-assigned this Apr 17, 2026
@NickLarsenNZ NickLarsenNZ moved this to Development: Waiting for Review in Stackable Engineering Apr 17, 2026
@dervoeti dervoeti moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Apr 17, 2026
@dervoeti dervoeti self-requested a review April 17, 2026 09:45
Copy link
Copy Markdown
Member

@dervoeti dervoeti left a comment

Choose a reason for hiding this comment

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

Build worked but the SBOM was not there (it's at the workspace level, not in ./src). With the fix it is located here in the final image: /usr/local/bin/vector_bin.cdx.xml

Comment thread vector/copy_artifacts.sh
# assuming WORKDIR was `/stackable`.
base=$(basename "$1")
find /src/ -type f -name "${base}_bin.cdx.xml" -exec cp {} /app \;
find ./src/ -type f -name "${base}_bin.cdx.xml" -exec cp {} /app \;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
find ./src/ -type f -name "${base}_bin.cdx.xml" -exec cp {} /app \;
find . -maxdepth 1 -type f -name "${base}_bin.cdx.xml" -exec cp {} /app \;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

2 participants