Skip to content

[Gitlab] Fix Gitlab Pages duration_ms field - to accept float - #20747

Merged
efd6 merged 1 commit into
elastic:mainfrom
flux-ricky:fix/#20675-gitlab-pages
Aug 17, 2026
Merged

[Gitlab] Fix Gitlab Pages duration_ms field - to accept float#20747
efd6 merged 1 commit into
elastic:mainfrom
flux-ricky:fix/#20675-gitlab-pages

Conversation

@flux-ricky

@flux-ricky flux-ricky commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

See Title

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

Author's Checklist

  • Impact of the field type change

How to test this PR locally

Related issues

Screenshots

@flux-ricky
flux-ricky requested review from a team as code owners August 17, 2026 00:14
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Reviewers

Buildkite won't run for external contributors automatically; you need to add a comment:

  • /test : will kick off a build in Buildkite.

NOTE: https://github.com/elastic/integrations/blob/main/.buildkite/pull-requests.json contains all those details.

@flux-ricky
flux-ricky force-pushed the fix/#20675-gitlab-pages branch from 72b46b4 to a20c76d Compare August 17, 2026 00:15
@efd6

efd6 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

/test

Comment thread packages/gitlab/changelog.yml Outdated
Comment thread packages/gitlab/data_stream/pages/fields/fields.yml
Comment thread packages/gitlab/manifest.yml Outdated
@flux-ricky
flux-ricky force-pushed the fix/#20675-gitlab-pages branch from 4b71997 to 7ce4e59 Compare August 17, 2026 03:09
@flux-ricky

Copy link
Copy Markdown
Contributor Author

Thanks for the quick review, I believe I have addressed the feedback.

@efd6

efd6 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

/test

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

Package gitlab 👍(1) 💚(0) 💔(6)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
api 3300.33 2074.69 -1225.64 (-37.14%) 💔
audit 6289.31 4484.3 -1805.01 (-28.7%) 💔
auth 9433.96 4484.3 -4949.66 (-52.47%) 💔
pages 12048.19 6535.95 -5512.24 (-45.75%) 💔
production 3816.79 3236.25 -580.54 (-15.21%) 💔
sidekiq 3367 2857.14 -509.86 (-15.14%) 💔

To see the full report comment with /test benchmark fullreport

@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

💚 Build Succeeded

History

@efd6
efd6 merged commit e4f635a into elastic:main Aug 17, 2026
4 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants