Skip to content

chore(ci): use dd-sts instead of DD_API_KEY#25235

Merged
thomasqueirozb merged 8 commits intomasterfrom
dd-octo-sts
Apr 24, 2026
Merged

chore(ci): use dd-sts instead of DD_API_KEY#25235
thomasqueirozb merged 8 commits intomasterfrom
dd-octo-sts

Conversation

@thomasqueirozb
Copy link
Copy Markdown
Contributor

Summary

Replace the DD_API_KEY secret with dd-octo-sts short-lived tokens across CI workflows.

Vector configuration

NA

How did you test this PR?

Verified workflow syntax locally; runtime validation will happen via CI.

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

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Apr 21, 2026
@github-actions github-actions Bot added the domain: ci Anything related to Vector's CI environment label Apr 21, 2026
@thomasqueirozb thomasqueirozb marked this pull request as ready for review April 21, 2026 19:12
@thomasqueirozb thomasqueirozb requested a review from a team as a code owner April 21, 2026 19:12
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: 8a5cba1fbc

ℹ️ 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/static-analysis.yml Outdated
Comment thread .github/workflows/ci-integration-review.yml Outdated
@thomasqueirozb thomasqueirozb requested a review from pront April 22, 2026 15:01
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: eb68c7e12f

ℹ️ 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/test.yml Outdated
@thomasqueirozb thomasqueirozb changed the title chore(ci): use dd-octo-sts instead of DD_API_KEY chore(ci): use dd-sts instead of DD_API_KEY Apr 22, 2026
# Conflicts:
#	.github/workflows/test.yml
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: eeb2e76c38

ℹ️ 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/test-make-command.yml
Comment thread .github/workflows/integration.yml
Copy link
Copy Markdown
Member

@pront pront left a comment

Choose a reason for hiding this comment

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

We can ignore the remaining codex comments. The same if: steps.check.outputs.should_run == 'true' condition is used for running tests.

@thomasqueirozb thomasqueirozb added this pull request to the merge queue Apr 24, 2026
Merged via the queue into master with commit 7428dc3 Apr 24, 2026
98 checks passed
@thomasqueirozb thomasqueirozb deleted the dd-octo-sts branch April 24, 2026 19:10
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 24, 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