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.

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

task_name_prefix doesnt work with folders #4066

Closed
RayJin2000 opened this issue Mar 12, 2024 · 1 comment · Fixed by #4143
Closed

task_name_prefix doesnt work with folders #4066

RayJin2000 opened this issue Mar 12, 2024 · 1 comment · Fixed by #4143
Assignees

Comments

@RayJin2000
Copy link

RayJin2000 commented Mar 12, 2024

Summary

name[prefix] doesnt work with tasks in folders:

We have a Role with the following structure:

roles/haproxy/
├── defaults
├── files
├── handlers
├── tasks
│   ├── copy_certificates.yml
│   ├── haproxy1
│   │   ├── config.yml
│   │   └── main.yml
│   ├── haproxy2
│   │   ├── confd.yml
│   │   ├── config.yml
│   │   └── main.yml
│   ├── main.yml
│   └── set_backend_state.yml
└── templates

The tasks are named like: Folder | File | Taskname

Issue Type
name[prefix]: Task name should start with 'config | '.
roles/haproxy/tasks/haproxy2/config.yml:2 Task/Handler: HAproxy2 | config | Create directories
```

##### OS / ENVIRONMENT

<!--- Paste verbatim output between triple backticks -->

```console (paste below)
ansible-lint 24.2.0 using ansible-core:2.16.2 ansible-compat:4.1.11 ruamel-yaml:0.18.6 ruamel-yaml-clib:0.2.7
```

<!--- Provide all relevant information below, e.g. target OS versions, network
 device firmware, etc. -->

- ansible installation method: OS package
- ansible-lint installation method:  OS package

#### Desired Behavior
task_name_prefix with optional folder name?

@RayJin2000 RayJin2000 added bug new Triage required labels Mar 12, 2024
@ssbarnea ssbarnea added enhancement and removed new Triage required bug labels Mar 13, 2024
@cavcrosby
Copy link
Contributor

I've started looking into this request. If possible, could a maintainer assign me to this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants