Skip to content

[gitlab] Remove the forwarded tag from filestream input by default - #16915

Merged
brijesh-elastic merged 2 commits into
elastic:mainfrom
brijesh-elastic:gitlab-2.4.0
Feb 25, 2026
Merged

[gitlab] Remove the forwarded tag from filestream input by default#16915
brijesh-elastic merged 2 commits into
elastic:mainfrom
brijesh-elastic:gitlab-2.4.0

Conversation

@brijesh-elastic

Copy link
Copy Markdown
Contributor

Proposed commit message

gitlab: remove the `forwarded` tag from filestream input by default

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/gitlab directory.
  • Run the following command to run tests.

elastic-package test

Related issues

@brijesh-elastic brijesh-elastic self-assigned this Jan 9, 2026
@brijesh-elastic
brijesh-elastic requested a review from a team as a code owner January 9, 2026 12:12
@brijesh-elastic brijesh-elastic added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:gitlab GitLab Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Jan 9, 2026
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

cc @brijesh-elastic

@ShourieG

Copy link
Copy Markdown
Contributor

@brijesh-elastic, before we go ahead with this, we need to update the issue by saying, having forwarded in defaults is actually the consistent behaviour.

@ShourieG ShourieG 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.

LGTM

@brijesh-elastic
brijesh-elastic merged commit 9d4b5f6 into elastic:main Feb 25, 2026
15 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package gitlab - 2.4.0 containing this change is available at https://epr.elastic.co/package/gitlab/2.4.0/

navnit-elastic pushed a commit to navnit-elastic/integrations that referenced this pull request Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:gitlab GitLab Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

3 participants