Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support for managing individual plugin versions #1188

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

UiP9AV6Y
Copy link
Contributor

currently on the remote_execution/cockpit plugin class has support for managing the plugin version. all other plugins are tied to the foreman::plugin_version. this change adds the same ensure parameter to all plugin classes allowing for granular state management.

@UiP9AV6Y
Copy link
Contributor Author

UiP9AV6Y commented Oct 14, 2024

i kept the parameter documentation in the same style for each class, as opposed to using @param everywhere, as it will make it easier in the future to assimilate the documentation

edit: it seems this decision causes some tests to fail as the deprecated documentation style is no longer recognized as valid. how do we proceed here?

@UiP9AV6Y UiP9AV6Y force-pushed the feature_plugin_ensure_version branch from 04a836c to 1d04c0e Compare October 14, 2024 09:41
currently on the  remote_execution/cockpit plugin class has support
for managing the plugin version. all other plugins are tied to the
`foreman::plugin_version`. this change adds the same `ensure` parameter
to all plugin classes allowing for granular state management.
@UiP9AV6Y UiP9AV6Y force-pushed the feature_plugin_ensure_version branch from 1d04c0e to 5527093 Compare October 14, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant