-
Notifications
You must be signed in to change notification settings - Fork 82
Keep track of operations per formation assignment #3869
Conversation
This reverts commit a9c4b40.
/unhold |
/hold |
@nyordanoff: 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. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dzahariev, StanislavStefanov 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 |
/hold cancel |
Description
Currently, we are not storing any information about the current (or past) operations happening on an assignment. We use the state to calculate what was the last operation that happened whenever we need that information: this results in states like: CREATE_ERROR, CREATE_READY, DELETE_ERROR, DELETE_READY, etc...
This PR introduces the new
assignment_operation
entity and manages its lifecycle. The operation can be eitherASSIGN
orUNASSIGN
.Changes proposed in this pull request:
assignment_operation
entity along with dataloaders, service and repo layerassignment_operation
entity has some fields entirely for troubleshooting purposes for the future -triggered_by
,started/finished_at_timestamps
assignment_operation
for all existing FAs based on their statePull Request status
chart/compass/values.yaml
is updated