Skip to content
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.

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

[Ingest Manager] Allow using drop path for agent build #20019

Merged
merged 4 commits into from
Jul 20, 2020

Conversation

michalpristas
Copy link
Contributor

@michalpristas michalpristas commented Jul 17, 2020

What does this PR do?

This PR allows specifying drop path for dependencies for agent packaging.
When AGENT_DROP_PATH is specified as ENV packaging skips preparation of dependencies and uses entire directory.
It requires directory to contain tar.gz or zip archives, sha512 files.

If env is omitted agent packaging prepares filebeat and metrcibeat and includes them in a resulting package.

Why is it important?

This allows release manager to prepare beats, endpoint and asc files which are needed for release ready packages.
Discussed in #19623

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

@michalpristas michalpristas added enhancement review needs_backport PR is waiting to be backported to other branches. Team:Ingest Management labels Jul 17, 2020
@michalpristas michalpristas self-assigned this Jul 17, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jul 17, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ingest-management (Team:Ingest Management)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jul 17, 2020
Copy link
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

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

Looks good!

@ph
Copy link
Contributor

ph commented Jul 17, 2020

Super exciting, I am testing this.

@elasticmachine
Copy link
Collaborator

💔 Build Failed

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Started by user Michal Pristas, Replayed #3]

  • Start Time: 2020-07-17T18:35:54.074+0000

  • Duration: 74 min 53 sec

Test stats 🧪

Test Results
Failed 0
Passed 17671
Skipped 1814
Total 19485

Steps errors

Expand to view the steps failures

  • Name: Report to Codecov

    • Description: curl -sSLo codecov https://codecov.io/bash for i in auditbeat filebeat heartbeat libbeat metricbeat packetbeat winlogbeat journalbeat do FILE="${i}/build/coverage/full.cov" if [ -f "${FILE}" ]; then bash codecov -f "${FILE}" fi done

    • Duration: 1 min 26 sec

    • Start Time: 2020-07-17T19:03:54.998+0000

    • log

  • Name: Install docker-compose 1.21.0

    • Description: .ci/scripts/install-docker-compose.sh

    • Duration: 1 min 57 sec

    • Start Time: 2020-07-17T18:59:25.283+0000

    • log

  • Name: Make test

    • Description: make -C generator/_templates/metricbeat test

    • Duration: 7 min 4 sec

    • Start Time: 2020-07-17T19:40:34.990+0000

    • log

  • Name: Recursively delete the current directory from the workspace

    • Description: script returned exit code 2

    • Duration: 1 min 56 sec

    • Start Time: 2020-07-17T19:47:55.156+0000

    • log

Log output

Expand to view the last 100 lines of log output

[2020-07-17T19:49:18.645Z] + [ -f winlogbeat/build/coverage/full.cov ]
[2020-07-17T19:49:18.645Z] + FILE=journalbeat/build/coverage/full.cov
[2020-07-17T19:49:18.645Z] + [ -f journalbeat/build/coverage/full.cov ]
[2020-07-17T19:49:19.180Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats
[2020-07-17T19:49:19.485Z] + find . -type f -name TEST*.xml -path */build/* -delete
[2020-07-17T19:49:19.496Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Lint
[2020-07-17T19:49:19.581Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Elastic-Agent-x-pack
[2020-07-17T19:49:19.659Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-oss-Mac-OS-X
[2020-07-17T19:49:19.738Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Winlogbeat-oss
[2020-07-17T19:49:19.815Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Filebeat-x-pack-Mac-OS-X
[2020-07-17T19:49:19.895Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Dockerlogbeat
[2020-07-17T19:49:19.972Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Elastic-Agent-Mac-OS-X
[2020-07-17T19:49:20.054Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-crosscompile
[2020-07-17T19:49:20.139Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Journalbeat-oss
[2020-07-17T19:49:20.217Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-Mac-OS-X
[2020-07-17T19:49:20.297Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Functionbeat-x-pack
[2020-07-17T19:49:20.374Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Filebeat-Mac-OS-X
[2020-07-17T19:49:20.457Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack-Mac-OS-X
[2020-07-17T19:49:20.539Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Packetbeat-Linux
[2020-07-17T19:49:20.614Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Elastic-Agent-x-pack-Windows
[2020-07-17T19:49:20.689Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-OSS-Unit-tests
[2020-07-17T19:49:20.764Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Generators-Metricbeat-Linux
[2020-07-17T19:49:20.840Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-x-pack
[2020-07-17T19:49:20.917Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-oss-Windows
[2020-07-17T19:49:20.991Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Heartbeat-oss
[2020-07-17T19:49:21.067Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-x-pack-Windows
[2020-07-17T19:49:21.148Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Libbeat-x-pack
[2020-07-17T19:49:21.225Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Packetbeat-Mac-OS-X
[2020-07-17T19:49:21.304Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-crosscompile
[2020-07-17T19:49:21.380Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Functionbeat-Mac-OS-X-x-pack
[2020-07-17T19:49:21.457Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-oss-Linux
[2020-07-17T19:49:21.533Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Heartbeat-Mac-OS-X
[2020-07-17T19:49:21.608Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Filebeat-x-pack-Windows
[2020-07-17T19:49:21.684Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Filebeat-Windows
[2020-07-17T19:49:21.761Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Winlogbeat-Windows-x-pack
[2020-07-17T19:49:21.837Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Winlogbeat-Windows
[2020-07-17T19:49:21.918Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Auditbeat-x-pack-Mac-OS-X
[2020-07-17T19:49:21.994Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack-Windows
[2020-07-17T19:49:22.070Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-Windows
[2020-07-17T19:49:22.147Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Generators-Beat-Linux
[2020-07-17T19:49:22.225Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Packetbeat-Windows
[2020-07-17T19:49:22.302Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Functionbeat-Windows
[2020-07-17T19:49:22.379Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Heartbeat-Windows
[2020-07-17T19:49:22.457Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Filebeat-x-pack
[2020-07-17T19:49:22.534Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Filebeat-oss
[2020-07-17T19:49:22.611Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Libbeat-oss
[2020-07-17T19:49:22.688Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-OSS-Go-Integration-tests
[2020-07-17T19:49:22.764Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-OSS-Python-Integration-tests
[2020-07-17T19:49:22.845Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Libbeat-crosscompile
[2020-07-17T19:49:22.924Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Libbeat-stress-tests
[2020-07-17T19:49:23.000Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Generators-Metricbeat-Mac-OS-X
[2020-07-17T19:49:23.077Z] Running in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack
[2020-07-17T19:49:23.436Z] + cat
[2020-07-17T19:49:23.436Z] + /usr/local/bin/runbld ./runbld-script
[2020-07-17T19:49:23.436Z] Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF8
[2020-07-17T19:49:30.011Z] runbld>>> runbld started
[2020-07-17T19:49:30.011Z] runbld>>> 1.6.12/f45d832f2ba0aa2722ab4ec1fda8ad140f027f8b
[2020-07-17T19:49:31.391Z] runbld>>> The following profiles matched the job 'Beats/beats/PR-20019' in order of occurrence in the config (last value wins).
[2020-07-17T19:49:32.769Z] runbld>>> Debug logging enabled.
[2020-07-17T19:49:32.769Z] runbld>>> Storing result
[2020-07-17T19:49:32.769Z] runbld>>> Store result: created {:total 2, :successful 2, :failed 0} 1
[2020-07-17T19:49:32.769Z] runbld>>> BUILD: https://c150076387b5421f9154dfbf536e5c60.us-west1.gcp.cloud.es.io:9243/build-1587637540455/t/20200717194932-A8B3FD29
[2020-07-17T19:49:32.769Z] runbld>>> Adding system facts.
[2020-07-17T19:49:33.709Z] runbld>>> Adding vcs info for the latest commit:  2d4c180fc43b65f952c291c0312ae26dfe5ddc13
[2020-07-17T19:49:33.709Z] runbld>>> >>>>>>>>>>>> SCRIPT EXECUTION BEGIN >>>>>>>>>>>>
[2020-07-17T19:49:33.709Z] runbld>>> Adding /usr/lib/jvm/java-8-openjdk-amd64/bin to the path.
[2020-07-17T19:49:33.709Z] Processing JUnit reports with runbld...
[2020-07-17T19:49:33.709Z] + echo 'Processing JUnit reports with runbld...'
[2020-07-17T19:49:33.969Z] runbld>>> <<<<<<<<<<<< SCRIPT EXECUTION END <<<<<<<<<<<<
[2020-07-17T19:49:33.969Z] runbld>>> DURATION: 20ms
[2020-07-17T19:49:33.969Z] runbld>>> STDOUT: 40 bytes
[2020-07-17T19:49:33.969Z] runbld>>> STDERR: 49 bytes
[2020-07-17T19:49:33.969Z] runbld>>> WRAPPED PROCESS: SUCCESS (0)
[2020-07-17T19:49:33.969Z] runbld>>> Searching for build metadata in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats
[2020-07-17T19:49:34.907Z] runbld>>> Storing build metadata: 
[2020-07-17T19:49:34.907Z] runbld>>> Adding test report.
[2020-07-17T19:49:34.907Z] runbld>>> Searching for junit test output files with the pattern: TEST-.*\.xml$ in: /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats
[2020-07-17T19:49:35.845Z] runbld>>> Found 136 test output files
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack/x-pack/metricbeat/build/TEST-go-integration-istio.xml
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack/x-pack/metricbeat/build/TEST-go-integration-openmetrics.xml
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack/x-pack/metricbeat/build/TEST-go-integration-tomcat.xml
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack/x-pack/metricbeat/build/TEST-go-integration-activemq.xml
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-x-pack/x-pack/metricbeat/build/TEST-go-integration-iis.xml
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-OSS-Go-Integration-tests/metricbeat/build/TEST-go-integration-graphite.xml
[2020-07-17T19:49:37.753Z] runbld>>> No testsuite node found in /var/lib/jenkins/workspace/Beats_beats_PR-20019/src/github.com/elastic/beats/Metricbeat-OSS-Go-Integration-tests/metricbeat/build/TEST-go-integration-windows.xml
[2020-07-17T19:49:39.131Z] runbld>>> Test output logs contained: Errors: 0 Failures: 0 Tests: 19339 Skipped: 1548
[2020-07-17T19:49:39.131Z] runbld>>> Storing result
[2020-07-17T19:49:39.131Z] runbld>>> FAILURES: 0
[2020-07-17T19:49:39.391Z] runbld>>> Store result: updated {:total 2, :successful 2, :failed 0} 2
[2020-07-17T19:49:39.391Z] runbld>>> BUILD: https://c150076387b5421f9154dfbf536e5c60.us-west1.gcp.cloud.es.io:9243/build-1587637540455/t/20200717194932-A8B3FD29
[2020-07-17T19:49:39.649Z] runbld>>> Email notification disabled by environment variable.
[2020-07-17T19:49:39.649Z] runbld>>> Slack notification disabled by environment variable.
[2020-07-17T19:49:45.687Z] Running on Jenkins in /var/lib/jenkins/workspace/Beats_beats_PR-20019
[2020-07-17T19:49:45.904Z] [INFO] getVaultSecret: Getting secrets
[2020-07-17T19:49:45.985Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2020-07-17T19:49:47.117Z] + chmod 755 generate-build-data.sh
[2020-07-17T19:49:47.118Z] + ./generate-build-data.sh https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-20019/ https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-20019/runs/4 FAILURE 4432782
[2020-07-17T19:49:47.118Z] INFO: curl https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-20019/runs/4/steps/?limit=10000 -o steps-info.json
[2020-07-17T19:49:49.601Z] INFO: curl https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-20019/runs/4/tests/?status=FAILED -o tests-errors.json
[2020-07-17T19:49:50.151Z] INFO: curl https://beats-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/Beats/beats/PR-20019/runs/4/log/ -o pipeline-log.txt

@michalpristas michalpristas merged commit fe30916 into elastic:master Jul 20, 2020
v1v added a commit to v1v/beats that referenced this pull request Jul 20, 2020
…ngs-archive

* upstream/master: (274 commits)
  Test export commands in all Beats (elastic#20016)
  [Ingest Manager] Allow using drop path for agent build (elastic#20019)
  [DOCS] Remove include for deleted file in monitoring docs (elastic#20038)
  Removing xpack.monitoring.* settings (elastic#18608)
  [Heartbeat] Add service_name option for APM integration (elastic#19932)
  [Elastic Agent] Fix merging of fleet.yml. Add --staging to enroll cmd. (elastic#20026)
  Stricter stalebot configuration for PRs (elastic#20004)
  [ci] Favor direct mage invocation on CI (elastic#19960)
  Add cloudwatch input into Filebeat configure inputs documentation (elastic#19973)
  [Filebeat] remove delimiter \n from log line in s3 input (elastic#19972)
  [Metricbeat] Update MySQL dashboard (elastic#19913)
  Packetbeat process monitor: Ignore missing /proc/net/tcp6 (elastic#19945)
  [CI] fix MODULE variable cornercases (elastic#19985)
  Ignore timestamp in fortinet/clientendpoint and netscout/sightline (elastic#19998)
  add 7.9 to .backportrc.json (elastic#19952)
  Update internal links to external (elastic#19947)
  Remove Dynamic Script Compilations warning in Cisco module (elastic#19840)
  [Elastic Agent] Fix RPM and DEB packaging for Elastic Agent (elastic#19959)
  [Ingest Manager] Do not compare err with custom type (elastic#19980)
  Fix nanocore sum for non default intervals on Kubernetes Overview Dashboard (elastic#19675)
  ...
v1v added a commit to v1v/beats that referenced this pull request Jul 20, 2020
…nly-mod

* upstream/master: (75 commits)
  Test export commands in all Beats (elastic#20016)
  [Ingest Manager] Allow using drop path for agent build (elastic#20019)
  [DOCS] Remove include for deleted file in monitoring docs (elastic#20038)
  Removing xpack.monitoring.* settings (elastic#18608)
  [Heartbeat] Add service_name option for APM integration (elastic#19932)
  [Elastic Agent] Fix merging of fleet.yml. Add --staging to enroll cmd. (elastic#20026)
  Stricter stalebot configuration for PRs (elastic#20004)
  [ci] Favor direct mage invocation on CI (elastic#19960)
  Add cloudwatch input into Filebeat configure inputs documentation (elastic#19973)
  [Filebeat] remove delimiter \n from log line in s3 input (elastic#19972)
  [Metricbeat] Update MySQL dashboard (elastic#19913)
  Packetbeat process monitor: Ignore missing /proc/net/tcp6 (elastic#19945)
  [CI] fix MODULE variable cornercases (elastic#19985)
  Ignore timestamp in fortinet/clientendpoint and netscout/sightline (elastic#19998)
  add 7.9 to .backportrc.json (elastic#19952)
  Update internal links to external (elastic#19947)
  Remove Dynamic Script Compilations warning in Cisco module (elastic#19840)
  [Elastic Agent] Fix RPM and DEB packaging for Elastic Agent (elastic#19959)
  [Ingest Manager] Do not compare err with custom type (elastic#19980)
  Fix nanocore sum for non default intervals on Kubernetes Overview Dashboard (elastic#19675)
  ...
@ph
Copy link
Contributor

ph commented Jul 20, 2020

@michalpristas You are looking with @Conky5 to test this on master before backporting it?

@michalpristas
Copy link
Contributor Author

yes so far no backports

@blakerouse
Copy link
Contributor

@michalpristas Have you tried mage package locally with this change. I keep getting an error saying that build/distribution/agent-drop-path filepath does not exist.

@michalpristas
Copy link
Contributor Author

yeah i tried, you do this on windows?

@michalpristas
Copy link
Contributor Author

can you also get me an error with a stacktrace?

@ph
Copy link
Contributor

ph commented Jul 21, 2020

@michalpristas I've done a bit more testing locally on this, but I haven't been able to make it work like I was expecting it.

If I run mage package It generates the artifacts correctly, I don't have the issues that @blakerouse is reporting.
If I create an /tmp/hello directory and I copied over the metricbeat and filebeat files and I run AGENT_DROP_PATH=/tmp/hello mage package it seems to generate the right thing.
But if I add a ph-8.0.0-linux-x86_64.tar.gz and ph-8.0.0-linux-x86_64.tar.gz.sha512 to the drop path and run the same command as above the ph* files aren't included in the artifact.

@michalpristas
Copy link
Contributor Author

@ph oh yeah, if you look at packaging file (packages.yml) it stayed the same and we're still whitelisting things we want to include.
if we want to blindly copy everything there i can make it so together with including endpoint change. but i'm afraid that this might result in agent bringing too many things which will make our binary too big.

@ph
Copy link
Contributor

ph commented Jul 22, 2020

I've miss the "allowlist" let me try it again. @blakerouse do you still have issues?

@ph
Copy link
Contributor

ph commented Jul 22, 2020

@michalpristas Oh I see, you wanted to make the endpoint support another PR?

@michalpristas
Copy link
Contributor Author

yes this just lays down the base for that

@ph
Copy link
Contributor

ph commented Jul 22, 2020

ok, lets make a followup to add optional bundling of endpoint?

@blakerouse
Copy link
Contributor

My repo must have been in a bad state, working good for me as well.

@Conky5
Copy link

Conky5 commented Jul 23, 2020

@michalpristas You are looking with @Conky5 to test this on master before backporting it?

This looks good to me and is working in testing an initial release-manager integration.

michalpristas added a commit to michalpristas/beats that referenced this pull request Jul 27, 2020
melchiormoulin pushed a commit to melchiormoulin/beats that referenced this pull request Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement needs_backport PR is waiting to be backported to other branches. review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants