Ori.livneh has uploaded a new change for review. https://gerrit.wikimedia.org/r/201421
Change subject: webservice2: EAFP, not LBYL ...................................................................... webservice2: EAFP, not LBYL https://docs.python.org/2/glossary.html#term-lbyl https://docs.python.org/2/glossary.html#term-eafp Change-Id: I6a2d0549326129aa048bcc6620eacb231a4c47ee --- M modules/toollabs/files/webservice2 1 file changed, 6 insertions(+), 2 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/21/201421/1 diff --git a/modules/toollabs/files/webservice2 b/modules/toollabs/files/webservice2 index a705739..810a22e 100644 --- a/modules/toollabs/files/webservice2 +++ b/modules/toollabs/files/webservice2 @@ -6,6 +6,7 @@ import time import subprocess import argparse +import errno import xml.etree.ElementTree as ET @@ -29,10 +30,13 @@ :param default: Value to return if the file does not exist :return: String containing either contents of the file, or default value """ - if os.path.exists(path): + try: with open(path) as f: return f.read() - return default + except IOError as e: + if e.errno == errno.ENOENT: + return default + raise def start_web_job(server, release): -- To view, visit https://gerrit.wikimedia.org/r/201421 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6a2d0549326129aa048bcc6620eacb231a4c47ee Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Ori.livneh <o...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
