Skip to content

feat: quark receiver#15289

Draft
biscout42 wants to merge 8 commits into
mainfrom
d4c-poc
Draft

feat: quark receiver#15289
biscout42 wants to merge 8 commits into
mainfrom
d4c-poc

Conversation

@biscout42

Copy link
Copy Markdown
Contributor

initial mock receiver

@mergify

mergify Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label. Could you fix it @biscout42? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-./d./d is the label that automatically backports to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@biscout42 biscout42 added the backport-active-all Automated backport with mergify to all the active branches label Jul 1, 2026
@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

TL;DR

golangci-lint failed on lint (macos-latest) because the new root go.work excludes the repository root module, so CI's golangci-lint run ./... starts from a directory outside the workspace module roots. Add the repo root to go.work or remove the workspace file if it is only for local experimentation.

Remediation

  • Update go.work so the main Elastic Agent module is part of the workspace, for example add . to the use block alongside ./internal/edot and ./go-quark.
  • If the in-tree go-quark copy is intended to satisfy github.com/elastic/go-quark, also align go-quark/go.mod's module path with that import path; otherwise the workspace entry for ./go-quark will not replace the dependency imported by the receiver.
  • Re-run the lint workflow or locally run the same lint target after the workspace change to confirm golangci-lint run ./... can typecheck from the repository root.
Investigation details

Root Cause

The PR adds a root go.work containing only ./internal/edot and ./go-quark. With GOWORK pointing at that file, golangci-lint runs from the repository root with ./..., but the root directory is not one of the workspace module roots.

Evidence

GOWORK='/Users/runner/work/elastic-agent/elastic-agent/go.work'
level=error msg="[linters_context] typechecking error: pattern ./...: directory /Users/runner/work/elastic-agent/elastic-agent is outside module roots (/Users/runner/work/elastic-agent/elastic-agent/internal/edot, /Users/runner/work/elastic-agent/elastic-agent/go-quark)"
##[error]golangci-lint exit with code 7
  • Relevant PR changes:
go.work:
use (
    ./internal/edot
    ./go-quark
)

go-quark/go.mod:
module quark

Validation

  • Not run locally; this report is based on the failed GitHub Actions job log and the PR file contents.

Follow-up

  • The Windows and Ubuntu lint jobs were cancelled after the macOS lint failure, so they should be rechecked after the workspace fix.

What is this? | From workflow: PR Actions Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@mergify

mergify Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b d4c-poc upstream/d4c-poc
git merge upstream/main
git push upstream d4c-poc

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

Labels

backport-active-all Automated backport with mergify to all the active branches

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant