Add deprecation warning to various resources #1591
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Various resources have been documented as deprecated. Additionally add a deprecation warning for the user.
|
@ozerovandrei @mnaser Do you object in "deprecating" these resources? We don't have a deprecation policy and I do not intend to remove them, but there is no test coverage in the ci for them for years and they are also removed on openstack upstream for many many releases now. This adds a warning message to the user when they use them which I consider to be helpful so that users do not use very old versions. on #1590 we moved them on the deprecated subcategory of the docs. |
ozerovandrei
approved these changes
Jul 4, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Various resources have been documented as deprecated. Additionally add a deprecation warning for the user.
The related resources have been deprecated on openstack side for many releases now. We do not have any deprecation policy in place and there is no big harm keeping the code (if it doesn't block newer features etc which typically it doesn't). Nonetheless we do not have any ci coverage for these resources for years and it is good to warn the user that these are deprecated.