Skip to content

chore(ci): centralize CARGO_INCREMENTAL=0 in setup action#25401

Merged
thomasqueirozb merged 3 commits intomasterfrom
ci/cargo-incremental-0
May 9, 2026
Merged

chore(ci): centralize CARGO_INCREMENTAL=0 in setup action#25401
thomasqueirozb merged 3 commits intomasterfrom
ci/cargo-incremental-0

Conversation

@thomasqueirozb
Copy link
Copy Markdown
Contributor

@thomasqueirozb thomasqueirozb commented May 8, 2026

Summary

Incremental compilation is only useful when intermediate artifacts are cached between builds. In CI, each run starts from a clean state, so incremental compilation adds overhead (extra bookkeeping, larger artifacts) without any benefit. It can also cause subtle cache poisoning issues where stale incremental state from a partially-warmed cache leads to build failures.

Previously, CARGO_INCREMENTAL=0 was set independently in several workflow files, and some workflows didn't set it at all. This consolidates it into the shared setup action so it's applied uniformly across all CI jobs without requiring each workflow to remember to set it.

Vector configuration

NA

How did you test this PR?

NA

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

NA

@github-actions github-actions Bot added the domain: ci Anything related to Vector's CI environment label May 8, 2026
@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label May 8, 2026
@thomasqueirozb thomasqueirozb marked this pull request as ready for review May 8, 2026 20:44
@thomasqueirozb thomasqueirozb requested a review from a team as a code owner May 8, 2026 20:44
@thomasqueirozb thomasqueirozb enabled auto-merge May 8, 2026 20:45
Comment thread .github/actions/setup/action.yml Outdated
Co-authored-by: Pavlos Rontidis <pavlos.rontidis@gmail.com>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0293b1a0cd

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/workflows/k8s_e2e.yml
@thomasqueirozb thomasqueirozb added this pull request to the merge queue May 8, 2026
Merged via the queue into master with commit c3676b9 May 9, 2026
106 checks passed
@thomasqueirozb thomasqueirozb deleted the ci/cargo-incremental-0 branch May 9, 2026 00:15
@github-actions github-actions Bot locked and limited conversation to collaborators May 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

domain: ci Anything related to Vector's CI environment no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants