Skip to content

[7.15] [ML] Job updates in tests should not update blocked on deleting jobs (#78169)#78173

Merged
elasticsearchmachine merged 1 commit intoelastic:7.15from
dimitris-athanasiou:backport/7.15/pr-78169
Sep 22, 2021
Merged

[7.15] [ML] Job updates in tests should not update blocked on deleting jobs (#78169)#78173
elasticsearchmachine merged 1 commit intoelastic:7.15from
dimitris-athanasiou:backport/7.15/pr-78169

Conversation

@dimitris-athanasiou
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.15:

…lastic#78169)

Deleting jobs do not allow for their `blocked` field to be changed
from `delete`. Tests were creating job updates that ignored that
leading to a test failure when a non-noop update made no change to
a job. This commit ensures random job updates do not have a change
to the `blocked` field for deleting jobs.

Closes elastic#78160
@dimitris-athanasiou dimitris-athanasiou added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Sep 22, 2021
@elasticsearchmachine elasticsearchmachine merged commit be589b5 into elastic:7.15 Sep 22, 2021
@dimitris-athanasiou dimitris-athanasiou deleted the backport/7.15/pr-78169 branch September 22, 2021 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v7.15.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants