Skip to content

[jolokia_input] Fix System Test Failing#15924

Merged
shmsr merged 51 commits intoelastic:mainfrom
giorgi-imerlishvili-elastic:jolokia_input-fix-sys-test
Nov 13, 2025
Merged

[jolokia_input] Fix System Test Failing#15924
shmsr merged 51 commits intoelastic:mainfrom
giorgi-imerlishvili-elastic:jolokia_input-fix-sys-test

Conversation

@giorgi-imerlishvili-elastic
Copy link
Contributor

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

Proposed commit message

Replace deprecated docker image.

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 added the bugfix Pull request that fixes a bug issue label Nov 10, 2025
@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic requested a review from a team as a code owner November 10, 2025 23:39
@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic added Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] Integration:jolokia_input Jolokia Input 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. Please remove changelog/manifest modifications as it will cause to rerelease the package without actual modifications to its contents

@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic added enhancement New feature or request and removed bugfix Pull request that fixes a bug issue labels Nov 12, 2025
@giorgi-imerlishvili-elastic
Copy link
Contributor Author

In last commit I updated docker image, I noticed that image which I chose initially as replacement were Java version 25, but I wanted and expecting version 8 (tag name confused me) now I set the correct one which uses java version 8

@shmsr
Copy link
Member

shmsr commented Nov 13, 2025

@giorgi-imerlishvili-elastic It seems you have checked out an earlier revision and hence you have so many old commits which ideally should not be there. Although this is getting handled for now, but you may face issues. Please let me know if you are facing this, will fix this in your system so that we always use the latest upstream's main branch as base and make changes on top of it so that we do not have such commits. Currently in this PR, there are 4 month old changes also if you see the commit history.

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @giorgi-imerlishvili-elastic

@shmsr shmsr merged commit fb0a410 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:jolokia_input Jolokia Input 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] [jolokia_input] Failing test daily: system test: default in jolokia_input.

4 participants