Skip to content

Clarify modifying workload resource mappings#199

Merged
nebhale merged 1 commit intoservicebinding:mainfrom
sadlerap:altering-workload-resource-mappings
Mar 3, 2022
Merged

Clarify modifying workload resource mappings#199
nebhale merged 1 commit intoservicebinding:mainfrom
sadlerap:altering-workload-resource-mappings

Conversation

@sadlerap
Copy link
Contributor

If a user needs to modify a ClusterWorkloadResourceMapping resource, the specification didn't specify what should happen. This specifies what should happen.

  • Leave current resource mappings alone. If a cluster administrator wants to propogate this change in resource mappings, they can recreate the relevant binding; this should not be the responsibility of the operator.
  • Future modifications to a mapped resource must use the new mapping, not the old.

Fixes #194.

Signed-off-by: Andy Sadler ansadler@redhat.com

@nebhale nebhale requested review from arthurdm and nebhale January 20, 2022 18:41
@nebhale nebhale requested review from baijum and scothis March 1, 2022 00:11
@nebhale nebhale added this to the core/1.0.0 milestone Mar 3, 2022
@sadlerap sadlerap force-pushed the altering-workload-resource-mappings branch from 8df917c to 0831f09 Compare March 3, 2022 16:02
@sadlerap sadlerap force-pushed the altering-workload-resource-mappings branch from 0831f09 to dbebe5d Compare March 3, 2022 16:18
@sadlerap sadlerap force-pushed the altering-workload-resource-mappings branch from dbebe5d to 8eb0f51 Compare March 3, 2022 16:30
If a user needs to modify a ClusterWorkloadResourceMapping resource, the
specification didn't specify what should happen.  This specifies what
should happen.

Signed-off-by: Andy Sadler <ansadler@redhat.com>
@sadlerap sadlerap force-pushed the altering-workload-resource-mappings branch from 8eb0f51 to adcf788 Compare March 3, 2022 16:32
@nebhale nebhale merged commit b8f444d into servicebinding:main Mar 3, 2022
@sadlerap sadlerap deleted the altering-workload-resource-mappings branch March 3, 2022 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Altering workload resource mappings

5 participants