-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Heartbeat] Fix packaging #20566
[Heartbeat] Fix packaging #20566
Conversation
There were a few issues with the package target for x-pack/heartbeat.
/package |
/package |
The packaging build passed, but I forgot a Makefile in x-pack/heartbeat so lint failed. I pushed an update so I think it should be good now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM WFG
Looks better now. In a follow up I will add x-pack/heartbeat to the Jenkinfile so that it built and tested along with everything else. |
…ne-2.0 * upstream/master: (39 commits) [ITs] Revert healthcheck for elasticsearchssl service to the previous behaviour (elastic#20558) [Heartbeat] Fix packaging (elastic#20566) [Heartbeat] Add Magefile to X-Pack (elastic#20549) [Packetbeat] Add "network" to event.category (elastic#20392) fix typo in docs (elastic#20541) Add service resource in k8s cluster role (elastic#20546) Update Golang version to 1.14.7 (elastic#20508) Add missing inputs to filebeat spec (elastic#20388) add warning log in aws and googlecloud module for API cost (elastic#20523) Fix fortinet.firewall.mem value to be interpreted as integer (elastic#19335) [CI] add more resilience (elastic#20505) [JJBB] fix credentials with a service account for golang-crossbuild (elastic#20537) [ITs] change healthcheck for elasticsearch (elastic#20514) [JJBB] fix credentials with a service account (elastic#20535) chore(ci): use build step for checking if is PR (elastic#20536) [CI] runbld project name (elastic#20466) Add panw.panos.endreason field (elastic#18705) [Filebeat] Fix PANW field spelling "veredict" to "verdict" (elastic#18808) Fix typo in netflow module docs (elastic#18992) Modified auditd ingest pipeline to handle node=hostname (elastic#19659) ...
* Heartbeat packaging fixes There were a few issues with the package target for x-pack/heartbeat. * There are no integTests setup in x-pack/heartbeat * Add make files
backport here #20638 |
… to X-Pack (#20638) Cherry-pick of PR #20549, #20566, and #20242 to 7.x branch. Original message: There were a few issues with the package target for x-pack/heartbeat. Update root Makefile to build packages for x-pack/heartbeat. Update packaging CI job. Magefile fixes (mostly with where the packaging spec was looking for files). This also restructures the main magefile to share code with the x-pack. This fixes the broken agent build which is trying to run mage build in the x-pack heartbeat folder.
* Heartbeat packaging fixes There were a few issues with the package target for x-pack/heartbeat. * There are no integTests setup in x-pack/heartbeat * Add make files
What does this PR do?
There were a few issues with the package target for x-pack/heartbeat.
Why is it important?
This should allow builds and packaging from x-pack/heartbeat to work as expected.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
This was a fast fix so please try it out yourself and examine the package contents.