Skip to content

Conversation

igor-karpukhin
Copy link
Collaborator

All Submissions:

This PR marks the following providerSettings fields as deprecated:

  • DiskIOPS

  • DiskTypeName

  • EncryptEBSVolume

  • InstanceSizeName

  • VolumeType

  • AutoScaling

  • Have you signed our CLA?

  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).

  • Update docs/release-notes/release-notes.md if your changes should be included in the release notes for the next release.

Copy link
Contributor

github-actions bot commented Apr 4, 2024

@josvazg
Copy link
Collaborator

josvazg commented Apr 5, 2024

My two cents on how we should deprecate this kind of things:

  • Step 1: Mark fields deprecated, as this PR does.
  • Step 2: Introduce flag to fail validation when deprecated fields are used BUT disabled by default.
  • Step 3: Turn on flag by default. Backward compatibility is still possible, but we have given users more than enough time to migrate away from those fields.

Each step is done in a subsequent release and is properly highlighted in the release notes.

WDYT?

@igor-karpukhin
Copy link
Collaborator Author

My two cents on how we should deprecate this kind of things:

  • Step 1: Mark fields deprecated, as this PR does.
  • Step 2: Introduce flag to fail validation when deprecated fields are used BUT disabled by default.
  • Step 3: Turn on flag by default. Backward compatibility is still possible, but we have given users more than enough time to migrate away from those fields.

Each step is done in a subsequent release and is properly highlighted in the release notes.

WDYT?

I'm not sure if we have a deprecation policy written somewhere. I'd say we need to talk to @s-urbaniak when he is back.

@igor-karpukhin igor-karpukhin merged commit ade020f into main Apr 5, 2024
josvazg pushed a commit that referenced this pull request Apr 5, 2024
Signed-off-by: jose.vazquez <jose.vazquez@mongodb.com>
josvazg pushed a commit that referenced this pull request Apr 5, 2024
Signed-off-by: jose.vazquez <jose.vazquez@mongodb.com>
josvazg added a commit that referenced this pull request Apr 5, 2024
Signed-off-by: jose.vazquez <jose.vazquez@mongodb.com>
Co-authored-by: Igor Karpukhin <karpukhinigor@gmail.com>
@s-urbaniak s-urbaniak deleted the CLOUDP-235981/serverless-specs-fix branch April 17, 2024 13:29
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.

3 participants