Conversation
|
This pull request does not have a backport label. Could you fix it @eyalkraft? 🙏
|
Cloudbeat CI 🤖Allure Report: http://csp-allure-reports.s3.amazonaws.com/allure_reports/cloudbeat/prs/364/index.html |
|
I'm just getting setup to try to reproduce the problem you ran into. My first attempt to build with One thing I noticed that is missing from this PR is handling the configuration transformations that are in the V1 Cloudbeat spec file. Specifically the In https://github.com/elastic/beats/pull/32673/files you can see an example of how Filebeat does this:
|
|
@cmacknz Thanks for trying this! I knew there are the configuration transformations to handle (and we'll definitely have to add them later) but since the agent just crashed before even attempting to start cloudbeat I figured there's some problem in areas not related to cloudbeat. |
|
So, currently trying to build/reproduce this, but the error makes me think the problem is part of a set of issues that are fixed here: elastic/elastic-agent#1061 This is before the beater interface gets started, so it's probably not an issue with the V2 client? |
|
@eyalkraft two of us have tried build this on an M1 Mac and a Linux machine and the build commands seem to fail consistently building the docker containers with an error like: That path is part of the agent Dockerfile template. We are wondering if there is some additional setup to build Cloudbeat we might be missing, otherwise I have been trying to trace this into the agent build. I managed to get a successful build by editing the |
|
It does seems like an issue with the agent itself and not the libbeat changes. APM was tested against the libbeat V2 branch without issue today. Perhaps this is something specific to running as a container that we haven't hit yet. Hopefully we can get past the build issue here. |
|
@fearful-symmetry I confirmed I can build this on an M1 Mac for linux/arm64 using the main branch of agent, but not on the feature-arch-v2 branch. We must have broken the agent build somehow, possibly between now when this was first tested. @eyalkraft if you get a chance can you try running I have some suspicion that the Docker base image versions are stale on the agent V2 branch. |
|
@cmacknz @fearful-symmetry Big thanks for trying to validate the problem on your machines. I've tried building using elastic/elastic-agent@5f1e54f for the agent and I got the same error you did: The previous version of the feature-v2-arch branch I used was elastic/elastic-agent@43ad01d so it seems the build broke in one of the following commits: |
|
Yah, it would make sense that this is an issue in Elastic-Agent, since there's a lot of build changes that have gone on in V2. This is the only PR that seems vaguely related? elastic/elastic-agent@5f1e54f Also, the beats V2 PR has been merged, you should be safe to work against the feature-arch-v2 branch on elastic/beats |
|
Build is now fixed after merging elastic/elastic-agent#1105. Will try to reproduce the problem / test against the bug fix branch we have already. |
|
Alright I can reproduce the issue, it fails at
Now we just need to figure out what is wrong. Edit: I think just running |
|
It appears the downloads directory was removed in v2, and we just need to update this part of the agent. Fix coming shortly. |
Hey @cmacknz 👋 , is there an ETA for this ? We've bumped into this issue while trying to integrate v2 in the APM Server. We're using the the agent image for system tests (see https://github.com/elastic/apm-server/blob/fbeb582b18f755da5bbc0f75c1eb4e383d4f66da/docker-compose.yml#L53) |
|
We've had to pause on v2 development for a short while to deal with some urgent customer escalations unfortunately. The tracking issue for the bug is here elastic/elastic-agent#1159. This will be one of the first things we fix once we start up again. |
Thank you for the update! 🙇 |
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
@elastic/cloudbeat team will handle https://github.com/elastic/security-team/issues/4792 as part of Sprint 17 |
In order to check if v2 works quickly I did a short POC.
My changes only include
reload.RegisterV2.MustRegisterInputin cloudbeatAfter having a local cloudbeat & elastic-agent on the required branched,
to build & load the image to kind run
to set up everything:
and from
integrations/packages/cloud_security_posturerunCurrent status:
The built agent daemonset is in a
CrashLoopBackOff: