Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Unusable 'prepare -h ansible' in tmt minimal install #3425

Open
lukaszachy opened this issue Dec 13, 2024 · 4 comments · May be fixed by #3465
Open

Unusable 'prepare -h ansible' in tmt minimal install #3425

lukaszachy opened this issue Dec 13, 2024 · 4 comments · May be fixed by #3465
Assignees
@bajertom
Labels
packaging Changes related to the rpm packaging priority | must high priority, must be included in the next release
Milestone
1.42

Comments

@lukaszachy
Copy link
Collaborator

tmt doesn't require 'ansible' in minimal install, making this prepare step (and all features?) unusable.

# dnf install -y tmt
# tmt --feeling-safe  run prepare -h ansible --playbook playbook.yaml provision -h local
...
        prepare task #2: default-0 on default-0
        how: ansible
        playbook: playbook.yaml
        fail: Command 'sudo -E ansible-playbook -c local -i localhost, /playbook.yaml' returned 1.
...
        # stderr (1/1 lines)
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        sudo: ansible-playbook: command not found
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@lukaszachy lukaszachy added the packaging Changes related to the rpm packaging label Dec 13, 2024
@martinhoyer
Copy link
Collaborator

I think this is about catching the error and printing "install tmt+provision-ansible" message, as is being done for other provisioning plugins.

@lukaszachy
Copy link
Collaborator Author

@martinhoyer except we don't have such subpackage, nor 'tmt+prepare-ansible'.
The /usr/lib/python3.13/site-packages/tmt/steps/prepare/ansible.py is part of tmt package itself.

@martinhoyer
Copy link
Collaborator

@martinhoyer except we don't have such subpackage, nor 'tmt+prepare-ansible'. The /usr/lib/python3.13/site-packages/tmt/steps/prepare/ansible.py is part of tmt package itself.

Aha, I see it's being pulled by provision-container and/or provision-virtual.

@martinhoyer martinhoyer added this to the 1.41 milestone Dec 17, 2024
@martinhoyer martinhoyer added the priority | must high priority, must be included in the next release label Dec 17, 2024
@martinhoyer martinhoyer self-assigned this Dec 17, 2024
@bajertom bajertom linked a pull request Jan 13, 2025 that will close this issue
Open
3 tasks
@bajertom bajertom modified the milestones: 1.41, 1.42 Jan 13, 2025
@thrix thrix assigned bajertom and unassigned martinhoyer Jan 14, 2025
@bajertom
Copy link
Collaborator

This issue will be solved by introducing a new subpackage tmt+ansible that will pull ansible-core. See #3465

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bajertom bajertom

Labels
packaging Changes related to the rpm packaging priority | must high priority, must be included in the next release
Projects
None yet
Milestone
1.42
Development

Successfully merging a pull request may close this issue.

Introduce the tmt+ansible subpackage teemtee/tmt
3 participants
@lukaszachy @martinhoyer @bajertom