Closed Bug 1016199 Opened 10 years ago Closed 10 years ago

move skins/assets to data/assets

Categories

(Bugzilla :: Bugzilla-General, defect)

4.5.4
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: glob, Assigned: glob)

References

Details

Attachments

(1 file)

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.
Attached patch 1016199_1.patchSplinter Review
- move skins/assets to data/assets
Attachment #8429069 - Flags: review?(dkl)
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+
Flags: approval?
+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: 10 years ago
Resolution: --- → FIXED
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.

Attachment

General

Creator:
Created:
Updated:
Size: