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.
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
Latest sse-gateway plugin #1767
Conversation
@kshultzCB PTAL, we need your go ahead on this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; tests are passing 👍
@kshultzCB in terms of what we want more that ATH tests would be to confirm manually whether SSE events are delivered on different interactions with blueocean. Thanks. |
There might be some problems. It's hard to say for sure, since I'm not able to hit the update center when running from
I don't know if this is relevant to the SSE problems I'm having.
I've been working on a reliable way to recreate this, and haven't found it yet. |
I take back what I said about not being able to recreate it. Looks like you can hit this by going back to the terminal window in which you started Jenkins, and hitting
One of the more relevant looking bits is:
I've never seen Jenkins do anything like this before, until running this PR in dev mode. |
@kshultzCB Thanks for looking in to it. so the exception below happens only after you hit enter, forcing jenkins to reload?
|
@vivek - it looks that way, yes. As I mentioned earlier, I'm having problems with not being able to connect to the update center correctly. I suppose it's possible those are related? That problem can easily be recreated without restarting Jenkins. And was an issue before I ran this PR build - in other words, I hit this when I run from master as well.
Might be worthwhile for someone without a wonky local environment to run through these quick steps:
That seems the fastest way to recreate. |
@kshultzCB pressing Enter on jenkins run using We should be able to reproduce it without it to call out genuine bug. Since these
I doubt. sse gateway and all dependencies are served from local build update center shouldn't cause CCE errors. |
Things look good to me from an end-user standpoint. Standard restarts via http://1.2.3.4:8080/jenkins/restart don't trigger the odd behavior I described above with I figured that certificate problem with the update center was unrelated - just was providing it as a "preexisting condition." :) It do think it would be good to understand why the |
Cool, thanks.
This feature never worked for me from the beginning, I do not use it often so never paid attention to find more about it. So to try again, I launched blueocean (from master branch) and tried pressing enter running instance and it gives strange errors. So yeah, probably there is something fixable there. Maybe @imeredith knows. |
Closed. Upgrade to sse-gateway 1.16 pr #1809. |
Description
Test latest sue-gateway plugin. See discussion on sse-gateway PR for details.
Submitter checklist
Reviewer checklist