-
Notifications
You must be signed in to change notification settings - Fork 82
Fix request body validation when assignment status is reported #3814
Conversation
/test pre-main-compass-integration-no-dump |
e9e83ec
to
ba0c709
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ivantenevvasilev The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@PetarTodorovv: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Description
In the "status API" when someone reports status for assignment we incorrectly interpret the configuration property when it's set to
null
. And it seems the config is provided whereas it should be empty/null, resulting in a request body validation error.Changes proposed in this pull request:
Related issue(s)
Pull Request status
chart/compass/values.yaml
is updated