-
Notifications
You must be signed in to change notification settings - Fork 155
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
feat: add helm package mage target #7232
base: main
Are you sure you want to change the base?
feat: add helm package mage target #7232
Conversation
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
* upstream/main: feat: bump go to 1.24.0 (elastic#6932) [CI] Fix Core DRA crossbuild (elastic#7203) fix(ci): Use correct `READ_ONLY` access value instead of `READ` (elastic#7239) ci(buildkite): add initial elastic-agent-helm-charts pipeline (elastic#7176) chore: update elastic-agent-libs to v0.18.8 (elastic#6938) [flakiness] Correct group for TestEndpointAgentServiceMonitoring (elastic#7165)
plugins: | ||
- elastic/oblt-google-auth#v1.0.0: | ||
lifetime: 1800 # seconds |
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.
I use https://github.com/elastic/oblt-google-auth-buildkite-plugin
Still need to create the relevant copy file from local to the specific bucket
|
||
set -euo pipefail | ||
|
||
echo "--- mage package tests" |
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.
typo
then we can infer the helmchart file name
speed the build
@@ -0,0 +1,23 @@ | |||
#!/usr/bin/env bash |
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.
|
||
agentVersion := bversion.GetParsedAgentPackageVersion() | ||
agentCoreVersion := agentVersion.CoreVersion() | ||
agentImageTag := agentCoreVersion + "-SNAPSHOT" |
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.
for some reason SNAPSHOT=true
is not honoured
CHARTS_URL: "https://storage.googleapis.com/elastic-agent-helm-chart/elastic-agent-9.1.0-beta.tgz"
when using SNAPSHOT=true mage helm:package
is that the expected behaviour?
|
💛 Build succeeded, but was flaky
Failed CI StepsHistory
|
What does this PR do?
This PR introduces a new Mage target,
helm:package
, to package the Elastic-Agent Helm chart.This is a necessary step toward automating the Helm chart release process.
Why is it important?
This change is critical for supporting the release of Elastic-Agent Helm charts in a structured and automated manner:
DRA
) and production (release
) packages (controlled by explicitly setting SNAPSHOT="false" for production).Checklist
./changelog/fragments
using the changelog tool.Disruptive User Impact
This change does not introduce disruptive user impact but improves automation for Helm chart releases.
How to test this PR locally
CI
Run https://buildkite.com/elastic/elastic-agent-helm-charts
Related issues
N/A