-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
Keep-alive during Jenkins restart (offline capability) #21
Comments
Wow Jens, this sounds really complicated! :) I'm not using the standard Stapler JavaScript proxy directly; instead I hid it behind an "anti-corruption layer", which allowed me to introduce AJAX error handling - that's why you can see the "Sorry to bother you" error screen when you restart Jenkins and Build Monitor loses its connectivity with the mothership. This capability can of course be used in a much smarter way than just displaying an error message, but that's a place to start and a "Minimum Viable Product®" ;) I decided to display an error message and a button that allows the user to manually reload the page, rather than doing this automatically, because of several reasons:
I think that the ideal implementation would be the following:
What do you think? |
Wow, I appreciate your engagement :) First of all, agree, "offline capability" is misleading. I didn't know about Stapler (my Jenkins/Java background is not very strong), but I can imagine what the token is about. It's great to hear that you have done some "Sorry to bother you" error page - I didn't know this, maybe there is an issue in our environment that prevents that page from being rendered. I totally agree with your "ideal implementation". |
Engagement is a necessary ingredient of any good software ;) Basically, when a network connection drops Build Monitor should display an error page; it surprises me that you can't see it... What version are you using? I introduced that feature pretty early on. |
Currently, after restarting Jenkins someone needs to reload the monitor manually.
To fix this we could do the monitor reloading in the background via ajax, wrap an iframe around it or apply some other tricks..
The text was updated successfully, but these errors were encountered: