Closed
Bug 1016199
Opened 11 years ago
Closed 11 years ago
move skins/assets to data/assets
Categories
(Bugzilla :: Bugzilla-General, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 5.0
People
(Reporter: glob, Assigned: glob)
References
Details
Attachments
(1 file)
|
5.90 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
when deploying bug 977969 to a cluster with multiple webheads but without a sticky load balancer, it's possible for requests for concatenated css files to return 404.
what happens is the html request performs the check if the assets file exists, and creates it if it's missing.
however the request for the asset itself may land on a different webhead, where that file doesn't exist.
given data/ is designed to be shared across webheads, it makes sense to move it to there.
- move skins/assets to data/assets
Attachment #8429069 -
Flags: review?(dkl)
|
||
Comment 2•11 years ago
|
||
Comment on attachment 8429069 [details] [diff] [review]
1016199_1.patch
Review of attachment 8429069 [details] [diff] [review]:
-----------------------------------------------------------------
r=dkl
Attachment #8429069 -
Flags: review?(dkl) → review+
|
|
||
Updated•11 years ago
|
Flags: approval?
|
||
Comment 3•11 years ago
|
||
+1
any webserver content that's generated should be in data/
Flags: approval? → approval+
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
60965ea..ccff248 master -> master
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•11 years ago
|
Target Milestone: --- → Bugzilla 5.0
Version: unspecified → 4.5.4
You need to log in
before you can comment on or make changes to this bug.
Description
•