-
Notifications
You must be signed in to change notification settings - Fork 184
chore: remove agent version pinning in the CI #9105
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
chore: remove agent version pinning in the CI #9105
Conversation
This pull request is now in conflicts. Could you fix it? 🙏
|
dae34f9
to
5864ac8
Compare
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
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
What does this PR do?
This PR removes the manual version pinning for
AGENT_VERSION
,AGENT_PACKAGE_VERSION
, andBEAT_VERSION
from our CI configuration files, and bumps the.package-version
file to8.18.5
.It also re-enables a test that was previously skipped due to the version pinning, as the condition blocking its execution has now been resolved.
PS: as discussed here a review from observablt-ci and observablt-ci-contractors is not required (cc @ebeahan @pierrehilbert )
Why is it important?
The version pinning was introduced as a temporary workaround to support the 8.18.4 release. Now that 8.18.5 is released, we are reverting to an unpinned state
Checklist
./changelog/fragments
using the changelog toolDisruptive User Impact
None expected. This change only affects CI configuration and test gating logic.
How to test this PR locally
Open a draft PR 🙂
Related issues