ISSUE TYPE

Bug Report

ANSIBLE VERSION

ansible --version
ansible 2.6.1

ansible-galaxy list | grep ansible.network
- ansible-network.network-engine, v2.6.0

Network OS

n/a for this report.

SUMMARY

STEPS TO REPRODUCE

% find ./roles/ansible-network.network-engine -type f -name \*.yml -print0 | while read -d $'\0' -r YML
  do
    yamllint "$YML"
    ansible-lint "$YML"
  done
./roles/ansible-network.network-engine/tasks/main.yml
  2:32      error    no new line character at the end of file  (new-line-at-end-of-file)

./roles/ansible-network.network-engine/tests/json_template/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/netcfg_diff/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/text_parser/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/vlan_compress/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/textfsm_parser/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/interface_range/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/command_parser/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/textfsm/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/interface_split/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/tests/vlan_expand/test.yml
  1:1       warning  missing document start "---"  (document-start)

./roles/ansible-network.network-engine/meta/main.yml
  4:81      error    line too long (154 > 80 characters)  (line-length)
  13:81     error    line too long (84 > 80 characters)  (line-length)
  19:81     error    line too long (81 > 80 characters)  (line-length)
  22:4      warning  missing starting space in comment  (comments)

./roles/ansible-network.network-engine/.travis.yml
  12:5      error    wrong indentation: expected 6 but found 4  (indentation)
  29:61     error    no new line character at the end of file  (new-line-at-end-of-file)

[ANSIBLE0008] deprecated sudo feature
./roles/ansible-network.network-engine/.travis.yml:2
{'sudo': False}

./roles/ansible-network.network-engine/handlers/main.yml
  2:35      error    no new line character at the end of file  (new-line-at-end-of-file)

EXPECTED RESULTS

Lint checks should not report any results. Almost all of the items above are very minor and trivial to fix.

The one deprecated feature may take longer to resolve.

ACTUAL RESULTS

https://docs.ansible.com/ansible/devel/dev_guide/testing/sanity/

% ansible-test sanity
zsh: command not found: ansible-test

I was not able to test with your published sanity checker. The above commands were available, and did produce reasonable items, so they were used instead.

gundalow commented 6 years ago

@fenris02 (DJ) Thank you for this.

1) I'll need to check, though I'm fairly sure ansible-test runs yamllint. I think we ignore requiring --- at the start 2) Linelength is the same as ansible/ansible 160chrs 3) deprecated sudo feature looks seems valid. Though I think we can just delete .travis.yml 4) Wonder if we should run Will Thames' ansible-lint