Closed Bug 403107 Opened 17 years ago Closed 17 years ago

Blacklist Mozilla .nl IPs from download counts

Categories

(addons.mozilla.org Graveyard :: Maintenance Scripts, defect)

Product:
addons.mozilla.org Graveyard
Component:
Maintenance Scripts
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: fligtar, Assigned: fligtar)

References

Details

Attachments

(1 file, 1 obsolete file)

patch v2
4.12 KB, patch
morgamic
: review+
Details | Diff | Splinter Review 
.nl colo IPs should not count downloads as we are parsing those logs separately now.

63.245.213.*
Attached patch patch (obsolete) — — Splinter Review 
Instant results!

Downloads counted: 3044
Downloads skipped (30s blacklist): 3536
Downloads skipped (Mozilla .nl): 1602
Attachment #287951 - Flags: review?(morgamic)
Attachment #287951 - Flags: review?(morgamic) → review+
Attachment #287951 - Flags: review+
Attached patch patch v2 — — Splinter Review 
I realized that we need to do this for update pings too.

Update pings counted: 265409
Update pings skipped: 49453
Downloads skipped (Mozilla .nl): 31448
Attachment #287951 - Attachment is obsolete: true
Attachment #287956 - Flags: review?(morgamic)
Comment on attachment 287956 [details] [diff] [review]
patch v2

Works for me.  Sorry I didn't see the missing update code in the first revision.
Attachment #287956 - Flags: review?(morgamic) → review+
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: push-needed
Resolution: --- → FIXED
These bugs were pushed last Thursday.
Keywords: push-needed
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: