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

Build Heartbeat for Agent from the OSS directory #20561

Closed

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Aug 11, 2020

What does this PR do?

Heartbeat packages both for OSS and X-Pack are built using the magefile in the OSS directory. Use this magefile also when building heartbeat for the Elastic Agent.

It also reverts #20549 as produces some issues when building packages.

Why is it important?

To use the same magefile when building Heartbeat for Agent, and for the rest of distributions.

Related issues

@jsoriano jsoriano self-assigned this Aug 11, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Aug 11, 2020
@jsoriano jsoriano added Agent Heartbeat Team:Integrations Label for the Integrations team Team:obs-ds-hosted-services Label for the Observability Hosted Services team labels Aug 11, 2020
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Aug 11, 2020
@jsoriano jsoriano marked this pull request as ready for review August 11, 2020 15:38
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/uptime (Team:Uptime)

@jsoriano
Copy link
Member Author

/packaging

1 similar comment
@jsoriano
Copy link
Member Author

/packaging

@elasticmachine
Copy link
Collaborator

elasticmachine commented Aug 11, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20561 updated]

  • Start Time: 2020-08-11T18:20:16.534+0000

  • Duration: 51 min 28 sec

Test stats 🧪

Test Results
Failed 0
Passed 1200
Skipped 28
Total 1228

@jsoriano jsoriano force-pushed the heartbeat-agent-build-directory branch from 8068033 to cd7c245 Compare August 11, 2020 18:19
@jsoriano jsoriano requested a review from a team as a code owner August 11, 2020 18:19
@elasticmachine
Copy link
Collaborator

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

@jsoriano jsoriano requested a review from a team August 11, 2020 18:19
@jsoriano
Copy link
Member Author

/packaging

@ruflin
Copy link
Contributor

ruflin commented Aug 12, 2020

@jsoriano There is an open PR from blake which should fix some tests: #20498 But would this solve the packaging issue?

There is also a PR from @andrewvc to fix things: #20563 Also adding @andrewkroh here.

I think we should have a quick chat today on how to get CI back to green quickly.

@jsoriano
Copy link
Member Author

CI should be fixed after #20566, closing this.

@jsoriano jsoriano closed this Aug 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agent Heartbeat Team:Integrations Label for the Integrations team Team:obs-ds-hosted-services Label for the Observability Hosted Services team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants