Dzahn has submitted this change and it was merged.
Change subject: osm: fix top-scope var without namespace, rm cruft
......................................................................
osm: fix top-scope var without namespace, rm cruft
./modules/osm/manifests/planet_import.pp
WARNING: top-scope variable being used without an explicit namespace on line 25
removing the entire line that is not used anymore
Change-Id: Ib941673a231f1de31a2f39346c3a15c6287f2f90
---
M modules/osm/manifests/planet_import.pp
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/modules/osm/manifests/planet_import.pp
b/modules/osm/manifests/planet_import.pp
index 1f87513..56a9cf7 100644
--- a/modules/osm/manifests/planet_import.pp
+++ b/modules/osm/manifests/planet_import.pp
@@ -22,7 +22,6 @@
# Check if our db tables exist
$tables_exist = "/usr/bin/psql -d ${name} --tuples-only -c \'SELECT
table_name FROM information_schema.tables;\' | /bin/grep \'planet_osm\'"
- $shapelines_exist = "/usr/bin/psql -d ${name} --tuples-only -c \'SELECT
table_name FROM information_schema.tables;\' | /bin/grep \'${shape_table}\'"
# Note. This is not needed anymore with osm2pgsql 0.81
exec { "load_900913-${name}":
--
To view, visit https://gerrit.wikimedia.org/r/266971
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib941673a231f1de31a2f39346c3a15c6287f2f90
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: ArielGlenn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
