Skip to content

RUN-4228: allow to set ansible base dir project wide#422

Merged
fdevans merged 1 commit intorundeck-plugins:RUN-4228from
hatch01:ansible-base-dir-per-project
Mar 24, 2026
Merged

RUN-4228: allow to set ansible base dir project wide#422
fdevans merged 1 commit intorundeck-plugins:RUN-4228from
hatch01:ansible-base-dir-per-project

Conversation

@hatch01
Copy link
Copy Markdown

@hatch01 hatch01 commented Jan 6, 2026

Add the possibility to set the Ansible base directory at the project level.
This is very useful in this case, for example, where.
For each project, I have a repo with playbooks and the rundeck jobs.
So the base dir is always the path to the scmImport folder.

Closes #271

Tests :
I created a basic project
Set the value project-wide and see it used in the jobs.
Set the value also at the job level and confirm the value is used instead of the project one.

Copilot AI review requested due to automatic review settings January 6, 2026 16:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds the ability to configure the Ansible base directory at the project or framework level, addressing issue #271. Previously, the base directory could only be set at the job level, requiring users to configure it individually for each job or use workarounds like symbolic links to access required files such as group_vars.

  • Extends all Ansible plugin configurations to support project-wide and framework-wide base directory settings
  • Updates the property resolution logic to check project and framework configurations
  • Adds documentation for the new configuration option

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
AnsiblePlaybookWorkflowStep.java Adds project and framework mappings for ansible-base-dir-path to enable project-wide configuration
AnsiblePlaybookWorflowNodeStep.java Adds project and framework mappings for ansible-base-dir-path to enable project-wide configuration
AnsiblePlaybookInlineWorkflowStep.java Adds project and framework mappings for ansible-base-dir-path to enable project-wide configuration
AnsiblePlaybookInlineWorkflowNodeStep.java Adds project and framework mappings for ansible-base-dir-path to enable project-wide configuration
AnsibleNodeExecutor.java Adds project and framework mappings for ansible-base-dir-path to enable project-wide configuration
AnsibleModuleWorkflowStep.java Adds project and framework mappings for ansible-base-dir-path to enable project-wide configuration
AnsibleRunnerContextBuilder.java Updates getBaseDir() to use PropertyResolver for hierarchical property resolution (job → node → project → framework)
README.md Documents the new ansible-base-dir-path configuration option and its project-wide usage

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hatch01 hatch01 force-pushed the ansible-base-dir-per-project branch from 5d6ded9 to 54678fe Compare February 18, 2026 14:21
@hatch01
Copy link
Copy Markdown
Author

hatch01 commented Feb 18, 2026

@ltamaster @edbaltra Could you please take a look at this PR.
This is a pretty straightforward one, which can help in some workflows.

@fdevans fdevans changed the base branch from main to RUN-4228 March 24, 2026 17:41
@fdevans fdevans changed the title feat: allow to set ansible base dir project wide RUN-4228: allow to set ansible base dir project wide Mar 24, 2026
@fdevans fdevans merged commit 4b51375 into rundeck-plugins:RUN-4228 Mar 24, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to set default directory for ansible source?

3 participants