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

Shorten ES cluster name in TestEnterpriseSearchTLSDisabledVersionUpgr… #6256

Conversation

thbkrkr
Copy link
Contributor

@thbkrkr thbkrkr commented Dec 21, 2022

Fixes:

Invalid value: "test-ent-notls-version-upgrade-8x-rg6h": Elasticsearch configuration would
generate resources with invalid names: name exceeds maximum allowed length of 36

I missed this because when I triggered a manual run of the e2e test, the current version of the stack is used, which caused us to skip the test as we are already on the latest version 8x.

Follow-up of #6224.

@thbkrkr thbkrkr added >bug Something isn't working :ci Things related to Continuous Integration, automation and releases >test Related to unit/integration/e2e tests labels Dec 21, 2022
@thbkrkr thbkrkr marked this pull request as ready for review December 21, 2022 09:05
@thbkrkr thbkrkr added the v2.7.0 label Dec 21, 2022
@thbkrkr thbkrkr merged commit 8890533 into elastic:main Dec 21, 2022
@thbkrkr thbkrkr deleted the fix-TestEnterpriseSearchTLSDisabledVersionUpgradeToLatest8x-name-max-length branch January 11, 2023 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug Something isn't working :ci Things related to Continuous Integration, automation and releases >test Related to unit/integration/e2e tests v2.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants