Skip to content

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

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

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

Conversation

@dimitris-athanasiou
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.14:

…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 cedf17d into elastic:7.14 Sep 22, 2021
@dimitris-athanasiou dimitris-athanasiou deleted the backport/7.14/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.14.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants