Skip to content

Fixing Aruba build CI #15461

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
qcorporation merged 3 commits into from
Sep 25, 2025
Merged

Fixing Aruba build CI #15461

qcorporation merged 3 commits into from
Sep 25, 2025

Conversation

@qcorporation
Copy link
Contributor

@qcorporation qcorporation commented Sep 24, 2025

Proposed commit message

Fixing CI errors with no definition of log.file.fingerprint which fails filestream system tests

  • added log.file.fingerprint definition
  • updated docs with the new field

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

  • Validated that system test are passing specifically for filestream
@qcorporation
fixing aruba missing fields
758bf78 
- added log.file.fingerprint definition - updated docs with the new field
@qcorporation qcorporation self-assigned this Sep 24, 2025
@qcorporation qcorporation added bug Something isn't working, use only for issues Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] labels Sep 24, 2025
@qcorporation qcorporation marked this pull request as ready for review September 24, 2025 19:41
@qcorporation qcorporation requested a review from a team as a code owner September 24, 2025 19:41
@elasticmachine
Copy link

elasticmachine commented Sep 24, 2025

Pinging @elastic/integration-experience (Team:Integration-Experience)
@qcorporation qcorporation enabled auto-merge (squash) September 24, 2025 19:44
robester0403
robester0403 approved these changes Sep 24, 2025
View reviewed changes
Copy link
Contributor

@robester0403 robester0403 left a comment

Choose a reason for hiding this comment

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

lgtm
@andrewkroh andrewkroh added Integration:hpe_aruba_cx HPE Aruba CX documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. labels Sep 24, 2025
@elasticmachine
Copy link

elasticmachine commented Sep 25, 2025

💚 Build Succeeded

History

cc @qcorporation
@qcorporation qcorporation merged commit d12eead into main Sep 25, 2025
7 checks passed
@qcorporation qcorporation deleted the fix-build-errors branch September 25, 2025 13:44
@elastic-sonarqube
Copy link

elastic-sonarqube bot commented Sep 25, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube
@elastic-vault-github-plugin-prod
Copy link

elastic-vault-github-plugin-prod bot commented Sep 25, 2025

Package hpe_aruba_cx - 0.1.3 containing this change is available at https://epr.elastic.co/package/hpe_aruba_cx/0.1.3/
tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
@qcorporation
Fixing Aruba build CI (elastic#15461)
60a5f10 
* fixing aruba missing fields - added log.file.fingerprint definition - updated docs with the new field
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:hpe_aruba_cx HPE Aruba CX Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience]

6 participants

@qcorporation @elasticmachine @robester0403 @taylor-swanson @andrewkroh