Skip to content

Update LogstashWriter.java to fix "Jenkins.instance is missing" error#147

Closed
tranceporter wants to merge 6 commits into
jenkinsci:masterfrom
tranceporter:patch-2
Closed

Update LogstashWriter.java to fix "Jenkins.instance is missing" error#147
tranceporter wants to merge 6 commits into
jenkinsci:masterfrom
tranceporter:patch-2

Conversation

@tranceporter

@tranceporter tranceporter commented Sep 6, 2024

Copy link
Copy Markdown
image

Image above explains the solution for the issue https://issues.jenkins.io/browse/JENKINS-71693?jql=project%20%3D%20JENKINS%20AND%20component%20%3D%20logstash-plugin

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@jakub-bochenski

Copy link
Copy Markdown

I think we should include the changes in !146

This change looks useful: https://github.com/jenkinsci/logstash-plugin/pull/138/files#diff-13917fce5a3d9038026493f94c39a0ad41afc4084922449b8d4fb06ffc107959

And we sure can use the additional tests that are added

@Seros

Seros commented Jan 7, 2026

Copy link
Copy Markdown

Hi, how is it going? Can we support with anything to have this issue fixed soon?

@jakub-bochenski

Copy link
Copy Markdown

Combining this PR with !138 should do. You are free to contribute :)

@Seros Seros mentioned this pull request Jan 9, 2026
6 tasks
@jakub-bochenski

jakub-bochenski commented Jan 27, 2026

Copy link
Copy Markdown

Superseded by #149 149

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants