feat(error-detection): add multiple pattern support
๐ Description of the Change
Add support for configuring multiple error detection regex patterns at both global (ConfigMap) and repository (CR) levels. This allows users to match different error formats from various linters and tools in a single pipeline.
Global Configuration (ConfigMap):
- Changed error-detection-simple-regexp to support arrays
- Supports 3 formats: single pattern (backward compatible), multi-line YAML, and JSON array
Repository CR:
- Added ErrorDetectionSettings with patterns array and max_number_of_lines
- Patterns are additive with global patterns
- Per-repository max_number_of_lines override
๐จ๐ปโ Linked Jira
Jira: https://issues.redhat.com/browse/SRVKP-7237
๐ Linked GitHub Issue
N/A
๐ Type of Change
- [ ] ๐ Bug fix (
fix:) - [x] โจ New feature (
feat:) - [ ] ๐ฅ Breaking change (
feat!:,fix!:) - [ ] ๐ Documentation update (
docs:) - [ ] โ๏ธ Chore (
chore:) - [ ] ๐
Refactor (
refactor:) - [ ] ๐ง Enhancement (
enhance:) - [ ] ๐ฆ Dependency update (
deps:)
๐งช Testing Strategy
- [x] Unit tests
- [ ] Integration tests
- [ ] End-to-end tests
- [ ] Manual testing
- [ ] Not Applicable
๐ค AI Assistance
- [ ] I have not used any AI assistance for this PR.
- [x] I have used AI assistance for this PR.
If you have used AI assistance, please provide the following details:
Which LLM was used?
- [ ] GitHub Copilot
- [ ] ChatGPT (OpenAI)
- [ ] Claude (Anthropic)
- [x] Cursor
- [ ] Gemini (Google)
- [ ] Other: ____________
Extent of AI Assistance:
- [x] Documentation and research only
- [x] Unit tests or E2E tests only
- [ ] Code generation (parts of the code)
- [ ] Full code generation (most of the PR)
- [ ] PR description and comments
- [x] Commit message(s)
[!IMPORTANT] If the majority of the code in this PR was generated by an AI, please add a
Co-authored-bytrailer to your commit message. For example:Co-authored-by: Gemini [email protected] Co-authored-by: ChatGPT [email protected] Co-authored-by: Claude [email protected] Co-authored-by: Cursor [email protected] Co-authored-by: Copilot [email protected]
**๐กYou can use the script
./hack/add-llm-coauthor.shto automatically add these co-author trailers to your commits.
โ Submitter Checklist
- [x] ๐ My commit messages are clear, informative, and follow the project's How to write a git commit message guide. The Gitlint linter ensures in CI it's properly validated
- [x] โจ I have ensured my commit message prefix (e.g.,
fix:,feat:) matches the "Type of Change" I selected above. - [x] โฝ I have run
make testandmake lintlocally to check for and fix any issues. For an efficient workflow, I have considered installing pre-commit and runningpre-commit installto automate these checks. - [x] ๐ I have added or updated documentation for any user-facing changes.
- [x] ๐งช I have added sufficient unit tests for my code changes.
- [ ] ๐ I have added end-to-end tests where feasible. See README for more details.
- [ ] ๐ I have addressed any CI test flakiness or provided a clear reason to bypass it.
- [ ] If adding a provider feature, I have filled in the following and updated the provider documentation:
- [ ] GitHub App
- [ ] GitHub Webhook
- [ ] Gitea/Forgejo
- [ ] GitLab
- [ ] Bitbucket Cloud
- [ ] Bitbucket Data Center