add interpreter node attribute and info - #81
Merged
Conversation
Contributor
Author
# Conflicts: # plugin.yaml
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Rundeck plugin metadata to expose the Python interpreter setting as a node-level override, making it possible to select a different interpreter per target node without changing the plugin instance configuration.
Changes:
- Expands the
interpreterconfig description to mention the node attribute override key (winrm-interpreter). - Adds
renderingOptions.instance-scope-node-attribute: winrm-interpreterand groups the setting under an “Interpreter” UI group for both the NodeExecutor and FileCopier providers.
fdevans
approved these changes
Aug 3, 2026
This file contains hidden or 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
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.
Adds a
winrm-interpreternode attribute so theinterpreterconfig (which Python binary/path to invoke) can be overridden per node, matching the pattern already used forauthtype,winrmtransport,certpath, etc. Fixes rundeckpro#1452.Maintainer notes:
maininto this branch and resolved a conflict inplugin.yaml(the interpreter description text collided with feat: allow defining arbitrary python interpreter for newer distros #108's change of the default topython3).WinRMCheck) takeshostnameas a plain argument rather than running against a Rundeck-managed node, so a per-node override doesn't apply there. Corrected that provider's description text, which had been updated to claim node-level override support it doesn't actually have../gradlew buildpasses and the full unit test suite (26 tests) passes.