Skip to content

Development env fixes#418

Merged
evgeni merged 3 commits intotheforeman:masterfrom
adamruzicka:dev-fixes
Mar 27, 2026
Merged

Development env fixes#418
evgeni merged 3 commits intotheforeman:masterfrom
adamruzicka:dev-fixes

Conversation

@adamruzicka
Copy link
Copy Markdown
Contributor

No description provided.

Apparently ansible_facts['user_dir'] is a home directory of the user who
performed the setup task, not the home directory of the user executing
the current task.
ansible.builtin.include_tasks: "{{ item }}"
with_fileglob:
- "smart-proxy/plugin/{{ foreman_development_plugin_name }}.yml"
# Apparently, this is relative to the role's root
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

or at least the role's root is on the lookup path

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I shouldn't have read that in the morning... Well, good to know. Can I keep it as is or should I change it somehow?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't mind either way as long it works

@adamruzicka
Copy link
Copy Markdown
Contributor Author

Hold on, I'll add a couple more things while I'm at it

@adamruzicka
Copy link
Copy Markdown
Contributor Author

There

@evgeni evgeni enabled auto-merge (rebase) March 27, 2026 10:16
@evgeni evgeni merged commit 65ed4e1 into theforeman:master Mar 27, 2026
11 checks passed
@adamruzicka adamruzicka deleted the dev-fixes branch March 27, 2026 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants