[MESOS-8742] Agent resource provider config API calls should be idempotent.

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 1.5.1, 1.6.0
• Labels: mesosphere storage
• Target Version/s: 1.5.1
• Epic Link: Resource Provider and CSI Tech Debt
• Sprint: Mesosphere Sprint 76
• Story Points: 2
• Assignee: Chun-Hung Hsiao
• Reporter: Chun-Hung Hsiao
• Shepherd: Jie Yu
• Created: 27/Mar/18 23:26
• Updated: 25/Sep/18 23:14
• Resolved: 29/Mar/18 18:07

Description

There are some issues w.r.t. using the current agent resource provider config API calls:

1. UPDATE_RESOURCE_PROVIDER_CONFIG: If the caller fail to receive the HTTP response code, there is no way to retry the operation without triggering an RP restart.
2. REMOVE_RESOURCE_PROVIDER_CONFIG: If the caller fail to receive the HTTP response code, a retry will return a 404 Not Found. But due to MESOS-7697, there is no way for the caller to know if the 404 is due to a previous successful config removal or not.

To address these issues, we should make these calls idempotent, such that they return 200 OK when the caller retry. It would be nice if ADD_RESOURCE_PROVIDER_CONFIG is also idempotent for consistency.

Issue Links

• is duplicated by: MESOS-8693 agent: update_resource_provider w/ identical RP info should not always force-restart plugin