-
Notifications
You must be signed in to change notification settings - Fork 158
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
[Release] Update version to 8.14.1 #4594
Conversation
|
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.
Approved but not to be merged before 8.14.0
is released.
buildkite test this |
@@ -11,7 +11,7 @@ MAGE_SUBTARGET="${3:-""}" | |||
# Override the agent package version using a string with format <major>.<minor>.<patch> | |||
# NOTE: use only after version bump when the new version is not yet available, for example: | |||
# OVERRIDE_AGENT_PACKAGE_VERSION="8.10.3" otherwise OVERRIDE_AGENT_PACKAGE_VERSION="". | |||
OVERRIDE_AGENT_PACKAGE_VERSION="" | |||
OVERRIDE_AGENT_PACKAGE_VERSION="8.14.0" |
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.
#4844 should remove the need to do this.
@@ -32,6 +32,8 @@ func TestUpgradeBrokenPackageVersion(t *testing.T) { | |||
Sudo: true, // requires Agent installation | |||
}) | |||
|
|||
t.Skip("Skip until we have 8.14.1 build available") |
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.
We need to figure out why we keep having to skip only this test. It is doing something incorrectly.
This pull request is now in conflicts. Could you fix it? 🙏
|
|
Updates references to the new release 8.14.1.
Merge after the release 8.14.0.