Skip to content

fix(ci): remove redundant image tag determination logic from multiple…#647

Merged
Wikid82 merged 2 commits intomainfrom
hotfix/ci
Feb 4, 2026
Merged

fix(ci): remove redundant image tag determination logic from multiple…#647
Wikid82 merged 2 commits intomainfrom
hotfix/ci

Conversation

@Wikid82
Copy link
Owner

@Wikid82 Wikid82 commented Feb 4, 2026

… workflows

Copilot AI review requested due to automatic review settings February 4, 2026 14:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes duplicate "Determine image tag" step definitions from multiple GitHub Actions workflow files. Each affected workflow had the same image tag determination logic duplicated twice in sequence, and this PR eliminates the redundant first occurrence while preserving the functional implementation.

Changes:

  • Removed duplicate image tag determination logic from 4 integration test workflows (WAF, Rate Limit, CrowdSec, Cerberus)
  • Removed unused image_tag input parameter from e2e-tests workflow's manual trigger configuration
  • Eliminated 67-104 lines of redundant bash scripting per affected workflow file

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/waf-integration.yml Removed duplicate "Determine image tag" step (lines 35-95), keeping the functional version with step ID determine-tag
.github/workflows/rate-limit-integration.yml Removed duplicate "Determine image tag" step (lines 35-95), keeping the functional version with step ID determine-tag
.github/workflows/crowdsec-integration.yml Removed duplicate "Determine image tag" step and associated artifact fallback logic (lines 35-137), keeping the functional version
.github/workflows/cerberus-integration.yml Removed duplicate "Determine image tag" step (lines 35-95), keeping the functional version with step ID determine-tag
.github/workflows/e2e-tests.yml Removed unused image_tag input parameter from workflow_dispatch configuration (lines 43-46)

@Wikid82 Wikid82 merged commit 138fd2a into main Feb 4, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants