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
Fixes #18999: when we install a plugin, if the dependency (package manager) is not met, it still tries to install it and fails #3551
Conversation
Commit modified |
38db85b
to
cc975b3
Compare
@@ -307,7 +318,36 @@ def install_dependencies(metadata): | |||
logger.warning("The binary " + executable + " was not found on the system, you must install it before installing " + metadata['name']) | |||
return False | |||
else: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should add a case in the same way that the "binary" type was defined. I am pretty sure we want to have typed dependencies and so, an "apt" and a "rpm" type of dependencies..
PR updated with a new commit |
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
…nager) is not met, it still tries to install it and fails
f2801a8
to
c99cd86
Compare
OK, merging this PR |
4 similar comments
OK, merging this PR |
OK, merging this PR |
OK, merging this PR |
OK, merging this PR |
https://issues.rudder.io/issues/18999