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 only cloud image variant when it's requested #6408

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rdner
Copy link
Member

@rdner rdner commented Dec 19, 2024

Before this change we were building all possible Docker images to produce the cloud image. Which is unnecessary and takes a long time.

Now only the cloud variant is built.

On my machine this reduces this build time from 27 to 20 minutes.

For testing:

mage cloud:image

Or follow this guide https://github.com/elastic/elastic-agent?tab=readme-ov-file#testing-on-elastic-cloud

@rdner rdner added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team skip-changelog backport-8.x Automated backport to the 8.x branch with mergify backport-8.16 Automated backport with mergify backport-8.17 Automated backport with mergify labels Dec 19, 2024
@rdner rdner self-assigned this Dec 19, 2024
@rdner rdner marked this pull request as ready for review December 19, 2024 17:36
@rdner rdner requested a review from a team as a code owner December 19, 2024 17:36
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@rdner rdner marked this pull request as draft December 22, 2024 12:55
Before this change we were building all possible Docker images to
produce the cloud image. Which is unnecessary and takes a long time.

Now only the `cloud` variant is built.
@rdner rdner force-pushed the faster-cloud-image-build branch from 1b352dd to e06c01d Compare December 22, 2024 12:59
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@rdner rdner marked this pull request as ready for review December 22, 2024 20:30
@rdner rdner requested review from ycombinator, jlind23 and pchila and removed request for swiatekm December 22, 2024 20:31
@jlind23
Copy link
Contributor

jlind23 commented Dec 30, 2024

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.x Automated backport to the 8.x branch with mergify backport-8.16 Automated backport with mergify backport-8.17 Automated backport with mergify skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants