Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@yonch
Copy link
Collaborator

@yonch yonch commented Sep 18, 2025

No description provided.

@yonch yonch changed the title ci: Consolidate Build Environment, Split Fast vs Heavy Checks, and Reusable Workflows ci: consolidate build Environment, split fast vs heavy checks, and reusable workflows Sep 18, 2025
- Unified Containerfile with cargo-chef and targets
      - Containerfile:1
      - Targets: builder (devtools), collector, nri-init
      - Includes rustfmt, clippy, cargo-chef, clang/libelf, and Helm for fast-tier checks.
  - Fast-tier workflow (fmt, clippy, unit tests, helm lint/template)
      - .github/workflows/ci-fast.yaml:1
      - Builds the builder image once and runs cargo/Helm inside it via docker run.
  - Reusable workflows (workflow_call) per integration area
      - Collector (GH-hosted assumptions + optional heavy S3): .github/workflows/reusable/test-collector.yaml:1
      - Resctrl (GH-hosted unit + optional e2e smoke on metal): .github/workflows/reusable/test-resctrl.yaml:1
      - Helm e2e (k3s + helm install, S3 smoke): .github/workflows/reusable/test-helm.yaml:1
      - NRI integration (GH-hosted k3s): .github/workflows/reusable/test-nri.yaml:1
  - Orchestrator workflow to wire everything and gate heavy tests by “level”
      - .github/workflows/ci-orchestrator.yaml:1
      - Builds collector and nri-init once (reusing Containerfile targets); then runs:
          - Fast checks (uses ci-fast)
          - NRI GH-hosted integration
          - Conditionally runs heavy suites (collector/resctrl/helm) based on level:
              - PRs: level cheap-short (heavy disabled)
              - Push to main: level full-long
              - Manual dispatch: selectable level
  - Updated existing build workflows to the new Containerfile
      - .github/workflows/build-component-artifacts.yaml:1
          - Uses Containerfile with target: collector|nri-init
      - .github/workflows/build-collector.yaml:1
          - Triggers on Containerfile instead of Dockerfile.* and keeps multi-arch manifest

complete migration and retire old flows
@yonch yonch force-pushed the ci/refactor-workflows branch from bf0d535 to 6f54df3 Compare September 18, 2025 19:38
@yonch yonch marked this pull request as ready for review September 19, 2025 18:42
@yonch yonch merged commit f1541ec into unvariance:main Sep 19, 2025
17 checks passed
@yonch yonch deleted the ci/refactor-workflows branch September 19, 2025 18:42
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.

1 participant