Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #2660] Logon-Screen painfully slow since upgrade from 1.6.2 to 1.7.0 #802

Closed
icinga-migration opened this issue Jun 4, 2012 · 22 comments
Labels
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Jun 4, 2012

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2660

Created by nerve on 2012-06-04 13:20:01 +00:00

Assignee: mhein
Status: Resolved (closed on 2012-06-21 13:15:28 +00:00)
Target Version: 1.7.1
Last Update: 2012-06-21 13:31:10 +00:00 (in Redmine)


Since I have upgraded icinga-web from 1.6.2 to 1.7.0 (software and database), it takes about 18 seconds to open the logon screen in the browser (tested Firefox and Chrome).
If I enter the user credentials and press the login-button, it takes another 18 seconds until I see the web interface.
The same happens if I switch between the monitoring and the admin part within the web interface.

Please take a look at the attached screenshot.

With Internet Explorer, the web interface doesn't work at all anymore.
Internet Explorer 7 shows a java script error, Internet Explorer 9 shows just a blank page without any error message.

Please fix this bugs as soon as possible, because in my company many people are using this internet almost hourly.

Attachments

Changesets

2012-06-18 14:21:08 +00:00 by mhein 3cea518

Applied patch for Date.now function refs #2660

2012-06-18 14:21:43 +00:00 by mhein e480152

Applied patch for Date.now function refs #2660

2012-06-21 11:40:58 +00:00 by mhein f335f8e

Removed gz compression (optional configuration possible)

refs #2660

2012-06-21 12:27:17 +00:00 by mhein e36f237

Removed gz compression (optional configuration possible)

refs #2660
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 4, 2012

Updated by sa.mueller on 2012-06-04 14:26:51 +00:00

Hi,

I must confirm this! I'am searching since more than one week (after I Install the 1.7 Update) for an Answer why it is so slow.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 6, 2012

Updated by sgronewold on 2012-06-06 15:50:45 +00:00

Which browser versions did you use to test? I also had a blank page on clients with Chrome version 11, but after upgrading to version 19 everything worked fine for me.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 6, 2012

Updated by mopp on 2012-06-06 17:10:45 +00:00

The login bug is fixed in this revision https://dev.icinga.org/projects/icinga-web/repository/revisions/8f8382b2bcff82b84c6c1dce7cb9cca9a5154c95

Remove the , in line 102 from InfoIconColumnRenderer.js

But I've still problems loading the cronk. Shows "Loading..." with IE8
Message: Object doesn't support this property or method
Line: 40038
Char: 1
Code: 0

URI: https://ICINGA-Server/icinga-web/modules/appkit/squishloader/javascript

According to the IE debugger the problem is on line 249 in app/modules/AppKit/lib/js/AppKitUtil.js "timestamp = (Date.now()-timestamp)/1000;"

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 6, 2012

Updated by mopp on 2012-06-06 17:51:55 +00:00

  • File added AppKitUtil.patch

I think I found the solution. According to http://blog.yjl.im/2012/01/datenow-in-ie8-or-before-ie9.html Date.now() is not supported before IE9. So I applied the mentioned fix.
It seems to work in my environment. Did a short test with FireFox, IE8,9.

The patch is attached.
You still need to apply the patch for InfoIconColumnRenderer.js
After patching do a
# /usr/sbin/icinga-web-clearcache

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 7, 2012

Updated by nerve on 2012-06-07 08:28:08 +00:00

I can confirm that icinga-web works again with Internet Explorer if I apply this two patches.

But the load time of the logon-screen is still extremely high and this doesn't depend on the browser.
It's exactly the same situation with IE, Firefox and Chrome.

I startet the developer-tools in Chrome and saw that the load time of "/modules/appkit/squishloader/javascript" is 15 seconds, so I think the problem should be arround there.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 14, 2012

Updated by darkarchon on 2012-06-14 07:27:41 +00:00

nerve wrote:

I startet the developer-tools in Chrome and saw that the load time of "/modules/appkit/squishloader/javascript" is 15 seconds, so I think the problem should be arround there.

I can confirm this; also, once the JS is loaded, the rest works just fine and without that high loading times. But initially loading the JavaScript file (or maybe the generation of it?) takes a long while.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 14, 2012

Updated by pdeneu on 2012-06-14 08:00:02 +00:00

I can confirm this too.
Its not only at the logon-screen, its there after login and if you change to another module (admin, lconf, heatmap etc.)

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 14, 2012

Updated by mfriedrich on 2012-06-14 08:05:04 +00:00

some sort of caching for the webclient. once your client (session) has got it, it works fine again. but i did not yet find the root cause for it. some might step into the chrome dev konsole and measure timings on javascript loadings?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 20, 2012

Updated by sa.mueller on 2012-06-20 09:25:38 +00:00

I seen that a Update to Icinga 1.7.1 was deployed.[[https://www.icinga.org/2012/06/18/icinga-and-icinga-web-1-7-1-bugfix-release-is-out/?utm\_source=feedburner&utm\_medium=feed&utm\_campaign=Feed%3A+icinga-org+%28Icinga%29\]\] Does this fix this error?
The 15 seconds to login nerves very much.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 20, 2012

Updated by mfriedrich on 2012-06-20 09:30:31 +00:00

rather than expressing your nerve state, any useful traces on your browsers dev console?

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 20, 2012

Updated by sa.mueller on 2012-06-20 10:40:19 +00:00

  • File added chrome.png
  • File added firefox.png

So, now i have traces for you:

One from Google Chrome, but if the Site was loaded one time, it is so fast how it should be :) I Think the Squishloader (javascript) keeps in the Cache and so it is fast after load the login Page until you restart your Browser.

In Firefox it is slow everytime (about 16 Seconds).

i hope this will help you.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 20, 2012

Updated by mfriedrich on 2012-06-20 10:42:52 +00:00

once you've uploaded picture, wrap two '!' around the pasted filename. then pictures are visible in redmine.

chrome
chrome.png

firefox
firefox.png

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 20, 2012

Updated by mfriedrich on 2012-06-20 10:43:41 +00:00

  • Status changed from New to Assigned
  • Assigned to set to mhein

@mxhash please have a look, thx.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by sa.mueller on 2012-06-21 08:40:56 +00:00

  • File added bugreport.png

Here i Have a Bugreport, seen in google chrome:
Its in the squishloader too.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by sa.mueller on 2012-06-21 08:42:40 +00:00

bugreport.png

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by mhein on 2012-06-21 11:47:16 +00:00

  • Status changed from Assigned to Feedback
  • Done % changed from 0 to 100

Hi.

Made a commit and removed gz compression per default. Can anyone test this please?

Kind regards,
Marius.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by mfriedrich on 2012-06-21 11:58:56 +00:00

  • File added icinga_web_1.7.1_master_no_compression_javascript.png

better, much better, thanks.

~/coding/icinga/icinga-web $ git log -1
commit f335f8ed2e5f17b8259f3cc41cd5023d1c74f4ed
Author: Marius Hein 
Date:   Thu Jun 21 13:40:58 2012 +0200

    Removed gz compression (optional configuration possible)

    refs #2660

icinga_web_1.7.1_master_no_compression_javascript.png

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by mhein on 2012-06-21 12:30:49 +00:00

Thanks. I picked that back to r1.7. I'll close that issue therefore.

Kind regards,
Marius.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by mfriedrich on 2012-06-21 12:35:35 +00:00

please wait for the other reporters to test their issues, and when they report back ok, finally close the issue.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by fmbiete on 2012-06-21 12:55:06 +00:00

For me, that patch also fix the problem.

Thank you very much.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by mhein on 2012-06-21 13:15:28 +00:00

  • Status changed from Feedback to Resolved
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented Jun 21, 2012

Updated by sa.mueller on 2012-06-21 13:31:10 +00:00

Great! :)

@icinga-migration icinga-migration added this to the 1.7.1 milestone Jan 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.