Skip to content

[activemq] Fix activemq test error#15921

Merged
shmsr merged 49 commits intoelastic:mainfrom
giorgi-imerlishvili-elastic:fix-activemq-test-error
Nov 13, 2025
Merged

[activemq] Fix activemq test error#15921
shmsr merged 49 commits intoelastic:mainfrom
giorgi-imerlishvili-elastic:fix-activemq-test-error

Conversation

@giorgi-imerlishvili-elastic
Copy link
Contributor

@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic commented Nov 10, 2025

Proposed commit message

Replace deprecated docker image by alternative one for activemq tests

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

  • [ ]

How to test this PR locally

Related issues

Screenshots

…tions

# Conflicts:
#	packages/statsd_input/changelog.yml
#	packages/statsd_input/manifest.yml
@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic requested a review from a team as a code owner November 10, 2025 13:56
@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic added the bugfix Pull request that fixes a bug issue label Nov 10, 2025
@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic changed the title Fix activemq test error [activemq] Fix activemq test error Nov 10, 2025
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added Integration:activemq ActiveMQ Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Nov 10, 2025
Copy link
Contributor

@mykola-elastic mykola-elastic left a comment

Choose a reason for hiding this comment

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

Looks good.
I think you can skip adding changelog entry/increasing version as the change doesn't actually change the package. Am I right?

@giorgi-imerlishvili-elastic
Copy link
Contributor Author

Looks good. I think you can skip adding changelog entry/increasing version as the change doesn't actually change the package. Am I right?

Sounds reasonable, @muthu-mps , @ishleenk17 what do you think?

@mykola-elastic
Copy link
Contributor

@giorgi-imerlishvili-elastic rechecked with team, confirmed, we don't need the version bump here

@shmsr shmsr added enhancement New feature or request and removed bugfix Pull request that fixes a bug issue labels Nov 12, 2025
Co-authored-by: Mykola Kmet <mykola.kmet@elastic.co>
Co-authored-by: Mykola Kmet <mykola.kmet@elastic.co>
@elasticmachine
Copy link

💚 Build Succeeded

History

cc @giorgi-imerlishvili-elastic

@shmsr shmsr merged commit 87bcadd into elastic:main Nov 13, 2025
7 checks passed
tehbooom pushed a commit to tehbooom/integrations that referenced this pull request Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Integration:activemq ActiveMQ Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Subscription basic] [activemq] Failing test daily: system test: metric (variant: v5.17.1) in activemq.topic

5 participants