Skip to content

Symlink foreman-test script to /usr/local/bin#683

Merged
ehelms merged 1 commit into
theforeman:masterfrom
ehelms:symlink-foreman-test-to-path
Jul 20, 2026
Merged

Symlink foreman-test script to /usr/local/bin#683
ehelms merged 1 commit into
theforeman:masterfrom
ehelms:symlink-foreman-test-to-path

Conversation

@ehelms

@ehelms ehelms commented Jul 20, 2026

Copy link
Copy Markdown
Member

Why are you introducing these changes? (Problem description, related links)

Foreman merged a generic plugin test runner script (script/foreman-test) in theforeman/foreman#10963. This script replaces ad-hoc per-plugin test wrappers like ktest with a unified tool that works across all plugins. In the development VM, the script lives inside the Foreman checkout and isn't on PATH, so developers have to invoke it with a full path.

What are the changes introduced in this pull request?

  • Add a symlink from /usr/local/bin/foreman-test to the script/foreman-test inside the Foreman checkout, placed right after the git repository clone step in the foreman_development role.

How to test this pull request

Steps to reproduce:

  • Deploy a development environment: ./foremanctl deploy --foreman-initial-admin-password=changeme --tuning development
  • SSH into the VM and verify the symlink exists: ls -la /usr/local/bin/foreman-test
  • Run foreman-test --help from any directory and confirm it prints usage information

Checklist

  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)

Co-Authored-By: Claude <noreply@anthropic.com>

@jeremylenz jeremylenz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works as expected!

ACK 👍

@ehelms
ehelms merged commit 4767211 into theforeman:master Jul 20, 2026
16 checks passed
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.

2 participants