Last modified: 2012-06-28 15:39:05 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T40027, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 38027 - test geoip puppet class on labs
test geoip puppet class on labs
Status: VERIFIED FIXED
Product: Wikimedia Labs
Classification: Unclassified
deployment-prep (beta) (Other open bugs)
unspecified
All All
: Low enhancement
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-06-28 14:28 UTC by Antoine "hashar" Musso (WMF)
Modified: 2012-06-28 15:39 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Antoine "hashar" Musso (WMF) 2012-06-28 14:28:54 UTC

    
Comment 1 Antoine "hashar" Musso (WMF) 2012-06-28 14:31:12 UTC
applicationserver::labs used to have generic::geoip::files, I have removed that class since it got rewritten to `geoip`.

We need to test that class on some of the apache to see if that works well on labs. Once validated, we could read `include geoip` to applicationserver::labs
Comment 2 Andrew Otto 2012-06-28 14:34:06 UTC
This should work as long as labs has access to the puppet:///volatile fileserver module.  By default geoip copies the database files over via puppet.  


This could also work if labs had access to the puppet://private module, but that would put our paid Maxmind credentials on labs hosts, which is probably not what we want.
Comment 3 Antoine "hashar" Musso (WMF) 2012-06-28 15:25:00 UTC
I have created the geoip-on-labs instance under the 'integration' project on labs. puppet does not report any error.   Need to find out how to test the result though.
Comment 4 Andrew Otto 2012-06-28 15:33:50 UTC
Does it just include the geoip class?  If so, then /usr/share/GeoIP should contain a few different .dat files.
Comment 5 Antoine "hashar" Musso (WMF) 2012-06-28 15:36:32 UTC
ottomata instructed to verify presence of 3 dat files in /usr/share/GeoIP

hashar@geoip-on-labs:/usr/share/GeoIP$ ls 
GeoIPCity.dat  GeoIP.dat  GeoIPv6.dat

So that is a success!
Comment 6 Antoine "hashar" Musso (WMF) 2012-06-28 15:39:05 UTC
Enabling geoip on applicationserver::labs is https://gerrit.wikimedia.org/r/#/c/13339/

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links