Skip to content

Refactor gh action worklfows#4631

Merged
liamfallon merged 5 commits into
kptdev:mainfrom
Nordix:refactor_ghaction_workflows
Jul 9, 2026
Merged

Refactor gh action worklfows#4631
liamfallon merged 5 commits into
kptdev:mainfrom
Nordix:refactor_ghaction_workflows

Conversation

@efiacor

@efiacor efiacor commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

CI Workflow Refactor

Refactored GitHub Actions workflows for ruleset compatibility. Removed workflow-level paths-ignore and added gate jobs for proper status reporting.

Changes

  • go.yml: Path filtering + gate job. Matrix for docker/podman.
  • docs.yml: New documentation verification workflow with gate.
  • live-e2e.yml: Path filtering + gate job + K8s matrix.
  • e2eEnvironment.yml: Path filtering + gate job + K8s matrix.
  • release-api.yml: Fixed missing GOPATH env variable.
  • Deleted: verifyContent.yml, verifyDocumentation.yml (redundant/broken).

Behavior

  • Docs-only PRs: skip Go/Live tests, gates pass (not applicable)
  • Code-only PRs: run Go tests, skip Live tests
  • Live changes: run all Go, E2E, Apply tests

GitHub Ruleset Setup

Require these 4 checks:

  • go
  • docs
  • live-e2e
  • live-apply

Validation

  • Parsed by act
  • All job references verified ✓
  • Matrix aggregation confirmed ✓

AI disclosure: Fully verified all code and workflows.

@efiacor efiacor self-assigned this Jul 9, 2026
@efiacor efiacor requested a review from a team July 9, 2026 11:35
@efiacor efiacor added the cleanup label Jul 9, 2026
@efiacor efiacor requested a review from a team July 9, 2026 11:35
@netlify

netlify Bot commented Jul 9, 2026

Copy link
Copy Markdown

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit c0d0f61
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6a4fad2f6d2d1e00082c2b02
😎 Deploy Preview https://deploy-preview-4631--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. area/devops labels Jul 9, 2026
@efiacor efiacor force-pushed the refactor_ghaction_workflows branch from 3a8cd0b to 6e2d69f Compare July 9, 2026 11:52
liamfallon
liamfallon previously approved these changes Jul 9, 2026
Comment thread .github/workflows/go.yml Outdated
Comment thread .github/workflows/e2eEnvironment.yml
Comment thread .github/workflows/go.yml
Comment thread .github/workflows/live-e2e.yml
mozesl-nokia
mozesl-nokia previously approved these changes Jul 9, 2026
efiacor added 4 commits July 9, 2026 14:07
Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 refactors the repository’s GitHub Actions workflows to be compatible with GitHub rulesets that require specific checks, replacing workflow-level paths-ignore with per-job conditional execution plus “gate” jobs that always report a final required status.

Changes:

  • Added path-based change detection jobs (dorny/paths-filter) and “gate” jobs to ensure required checks always complete with a status.
  • Introduced a new docs.yml workflow and removed older redundant/broken verification workflows.
  • Updated Live E2E / Live Apply workflows to use K8s matrices and conditional execution; added GOPATH env to release-api.yml.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
.github/workflows/go.yml Adds change detection and a gate job for required “go” status; refactors Go CI to run conditionally.
.github/workflows/docs.yml New docs verification workflow with change detection and a required “docs” gate.
.github/workflows/e2eEnvironment.yml Adds change detection plus gate job for required “live-e2e” status; keeps K8s matrix.
.github/workflows/live-e2e.yml Adds change detection plus gate job for required “live-apply” status; adds K8s matrix with pinned kind node images.
.github/workflows/release-api.yml Adds workflow-level GOPATH env for API release builds.
.github/workflows/verifyDocumentation.yml Deleted (replaced by docs.yml).
.github/workflows/verifyContent.yml Deleted (stated redundant/broken).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/go.yml Outdated
Comment thread .github/workflows/go.yml Outdated
Comment thread .github/workflows/go.yml Outdated
Comment thread .github/workflows/docs.yml Outdated
Comment thread .github/workflows/live-e2e.yml Outdated
Comment thread .github/workflows/e2eEnvironment.yml Outdated
Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
@liamfallon liamfallon merged commit 8863dda into kptdev:main Jul 9, 2026
21 of 24 checks passed
@efiacor efiacor deleted the refactor_ghaction_workflows branch July 9, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/devops cleanup lgtm size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants