Skip to content

Remove some more test-related instances of UpdateForV9#114148

Closed
ldematte wants to merge 3 commits intoelastic:mainfrom
ldematte:remove-trivial-updatev9-2
Closed

Remove some more test-related instances of UpdateForV9#114148
ldematte wants to merge 3 commits intoelastic:mainfrom
ldematte:remove-trivial-updatev9-2

Conversation

@ldematte
Copy link
Copy Markdown
Contributor

@ldematte ldematte commented Oct 4, 2024

More things that we don't need going forward - e.g. because we do not support features (or missing features) from v7 and we do not need to support them in BwC tests.

@ldematte ldematte added >non-issue >test Issues or PRs that are addressing/adding tests :Core/Infra/Core Core issues without another label v9.0.0 labels Oct 4, 2024
@ldematte ldematte requested a review from a team October 4, 2024 15:16
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Oct 4, 2024
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@thecoop
Copy link
Copy Markdown
Member

thecoop commented Oct 4, 2024

Do we want to be a bit cautious in removing all of these, until 8.17 is released, to minimise the chance of conflicts (and devs not correctly scoping checks) in backports to 8.x?

@ldematte
Copy link
Copy Markdown
Contributor Author

ldematte commented Oct 4, 2024

We can definitely do that, and defer this PR till 8.17 FF
I think the chance of conflict is low, given that the code that is removed have been there for a long time, but there are classes like ESRestTest that might be a "hot spot" for changes

@ldematte
Copy link
Copy Markdown
Contributor Author

ldematte commented Dec 9, 2024

Closing as it seems all of these have been addressed meanwhile

@ldematte ldematte closed this Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants