Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Butler operator health status #912

Merged
merged 8 commits into from
Feb 22, 2021
Merged

Conversation

cpgo
Copy link
Contributor

@cpgo cpgo commented Feb 12, 2021

Update routed column on v2deployments table on each reconcile loop.
If all desired components+circles are found on the observed state we set the column to true.

To make it easier to perform de "diff" of the observed stated and desired state I introduced a new annotation to the VirtualService and DestinationRule manifests.

annotations: {
  circles: '["some-circle-id", "another-circle-id"]'
 }

Since the annotation fields can only be a key/value pair of strings we have to json encode the circle id list to a string, and decode it when reading the field. This will also be relevant when discussing a future feature to add user defined metadata to the deployments.

Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
…ules

Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
@boring-cyborg boring-cyborg bot added the butler Improvements/additions/fixes for butler module. label Feb 12, 2021
Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
Signed-off-by: Cassio Godinho <cassiopgodinho@gmail.com>
@cpgo cpgo merged commit 1fd4e7c into butler-operator Feb 22, 2021
@cpgo cpgo deleted the butler-operator-health-status branch February 22, 2021 12:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
butler Improvements/additions/fixes for butler module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant