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

Fix failing ansible-lint CI job #4520

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Conversation

aphedges
Copy link
Contributor

The "Packit-as-a-Service" CI jobs have been failing since commit 34eb1791 from 2024-04-04. From the log2 of a recent failed build3, I saw the following output:

2025-01-23 17:15:27.261 commands.py       DEBUG  Command: python3 -m build --sdist --outdir dist
...
2025-01-23 17:15:30.617 logging.py        INFO   Successfully built ansible_lint-25.1.1.dev1.tar.gz
2025-01-23 17:15:30.634 commands.py       DEBUG  Command: sh -c ls dist/ansible-lint-*.tar.gz
2025-01-23 17:15:30.636 logging.py        INFO   ls: cannot access 'dist/ansible-lint-*.tar.gz': No such file or directory
2025-01-23 17:15:30.636 commands.py       ERROR  Command 'sh -c ls dist/ansible-lint-*.tar.gz' failed.

The problem is therefore that line 8 of .packit.yaml4 expects a file of the form ansible-lint-* when the file is actually named ansible_lint-*. Changing the line should fix the build, or at the very least, this single error.


I'm unfortunately not sure how to test this change without making a PR and running CI. I hope it works!

Footnotes

  1. https://github.com/ansible/ansible-lint/commit/34eb1794a1e2e69b51bde4d953e463f756fe70f5

  2. https://download.copr.fedorainfracloud.org/results/packit/ansible-ansible-lint-main/srpm-builds/08563847/builder-live.log.gz

  3. https://copr.fedorainfracloud.org/coprs/packit/ansible-ansible-lint-main/build/8563847/

  4. https://github.com/ansible/ansible-lint/blob/395b674bbc78bc4d9da641c38a7e959b4993d9a4/.packit.yaml#L8

The "Packit-as-a-Service" CI jobs have been failing since commit
34eb179[^1] from 2024-04-04. From the log[^2] of a recent failed
build[^3], I saw the following output:

```log
2025-01-23 17:15:27.261 commands.py       DEBUG  Command: python3 -m build --sdist --outdir dist
...
2025-01-23 17:15:30.617 logging.py        INFO   Successfully built ansible_lint-25.1.1.dev1.tar.gz
2025-01-23 17:15:30.634 commands.py       DEBUG  Command: sh -c ls dist/ansible-lint-*.tar.gz
2025-01-23 17:15:30.636 logging.py        INFO   ls: cannot access 'dist/ansible-lint-*.tar.gz': No such file or directory
2025-01-23 17:15:30.636 commands.py       ERROR  Command 'sh -c ls dist/ansible-lint-*.tar.gz' failed.
```

The problem is therefore that line 8 of `.packit.yaml`[^4] expects a
file of the form `ansible-lint-*` when the file is _actually_ named
`ansible_lint-*`. Changing the line should fix the build, or at the very
least, this single error.

[^1]: ansible@34eb179
[^2]: https://download.copr.fedorainfracloud.org/results/packit/ansible-ansible-lint-main/srpm-builds/08563847/builder-live.log.gz
[^3]: https://copr.fedorainfracloud.org/coprs/packit/ansible-ansible-lint-main/build/8563847/
[^4]: https://github.com/ansible/ansible-lint/blob/395b674bbc78bc4d9da641c38a7e959b4993d9a4/.packit.yaml#L8
@audgirka audgirka enabled auto-merge (squash) February 12, 2025 10:47
@audgirka audgirka merged commit 42bf4d2 into ansible:main Feb 12, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants