bug: ansible-lint fix warning count violations

[TommyE123]

Fixes #3446

Currently the count for the warning violations regex errors and doesn’t display the correct count.

Proposed Changes

Changed the regex to count the violations that ansible-lint outputs.

Testing

I’ve checked this on both Github and Azure pipelines using 2 different repositories and the results are as expected.

Before:

After:

If there are any issues or concerns, please let me know, and I will address them promptly.
Thank you for reviewing this PR and I look forward to your feedback.
Tom

[echoix]

@TommyE123 Do you think the ansible lint test failure here is transient?

[nvuillam]

Not sure the new regex matches such output :(

[echoix]

Does it have to do with some env vars that change the logging format?

[TommyE123]

Well I'm angry now with myself!

I must have missed a scenario. However I think we need to use both mine and the existing regex but will check. Please don't merge and I will come back with a better more robust solution!

I've also been looking at a better way to detect if ansible should be active in general.

Thank you both for all your hard work today getting everything merged. 🙂

Tom

[nvuillam]

@TommyE123 no problem haha ,thanks for your hard work :)

[TommyE123]

After some testing, I've discovered that the current regex only works when quiet: false is set or not specified in the .ansible-lint config file.

Setting quiet: true doesn't significantly reduce the log information across Azure, GitHub, and GitLab environments. In fact, it results in missing out on a valuable breakdown table of issues, as shown below as an example.

Given these findings, I'm closing this PR as the change doesn't work as initially intended. However, this does explain the behavior @alexanderbazhenoff encountered in #3446 (comment).

I believe we can also close #3680 now, as this was the only outstanding issue there.

Tom

[alexanderbazhenoff]

No prob, I haven't had any time for sure for the last few months either. May be I'll take a look into ansible-lint output also once I get back to refactoring my four roles. Thanks for trying anyway.