Skip to content

fix(dev): fix sources-okta component features check#23673

Merged
thomasqueirozb merged 2 commits into
masterfrom
okta-component-features
Aug 29, 2025
Merged

fix(dev): fix sources-okta component features check#23673
thomasqueirozb merged 2 commits into
masterfrom
okta-component-features

Conversation

@thomasqueirozb
Copy link
Copy Markdown
Member

Summary

Fix the component features check when sources-okta is the only feature

Vector configuration

NA

How did you test this PR?

./scripts/check-one-feature sources-okta
/ci-run-component-features

Change Type

  • Bug fix
  • New feature
  • 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

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • cargo fmt --all
      • cargo clippy --workspace --all-targets -- -D warnings
      • cargo nextest run --workspace (alternatively, you can run cargo test --all)
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run cargo vdev build licenses to regenerate the license inventory and commit the changes (if any). More details here.

@thomasqueirozb thomasqueirozb requested a review from a team as a code owner August 29, 2025 19:16
@github-actions github-actions Bot added the domain: sources Anything related to the Vector's sources label Aug 29, 2025
@thomasqueirozb thomasqueirozb force-pushed the okta-component-features branch from 9cb63d6 to 93caba6 Compare August 29, 2025 19:16
Copy link
Copy Markdown
Member Author

@thomasqueirozb thomasqueirozb left a comment

Choose a reason for hiding this comment

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

/ci-run-component-features

@thomasqueirozb thomasqueirozb added no-changelog Changes in this PR do not need user-facing explanations in the release changelog source: okta labels Aug 29, 2025
Copy link
Copy Markdown
Member Author

@thomasqueirozb thomasqueirozb left a comment

Choose a reason for hiding this comment

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

/ci-run-component-features

@github-actions github-actions Bot added the domain: ci Anything related to Vector's CI environment label Aug 29, 2025
@thomasqueirozb thomasqueirozb added this pull request to the merge queue Aug 29, 2025
Merged via the queue into master with commit b4a78c8 Aug 29, 2025
115 checks passed
@thomasqueirozb thomasqueirozb deleted the okta-component-features branch August 29, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants