Azure / Azure/azure-service-operator

Reconcile should perform a diff with Azure

Open
#2,811 20 comments 1 reaction 0 assignees View on GitHub
new-feature
Dominant language
Go
Stars
911
Forks
261
Avg merge
2d 23h
Merged PRs (30d)
51

Description

At the moment we are doing a unilateral PUT of each resource when we reconcile; this works but has some drawbacks.

* Users with many resources have run up against throttling issues (e.g. #2591, #2643)
* Users have raised bugs because they thought the operator should only PUT when there's a change required (e.g. #2590)

We should diff the current state of the resource with the spec and only do a PUT where required (see #2600 for potential design).

Follow-up to #1491 as much of that has been implemented already.

Contributor guide

Open the contributing guide

Research direction

No files or tests are named. Start by reading the existing work referenced in #1491 and the potential design in #2600, then trace the reconciliation path that currently PUTs each resource; done means unchanged resources are not PUT and changed resources still reconcile correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
azure, go, kubernetes
Domain
cloud, infrastructure
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.