Merge pull request #94 from PierreRochard/script-logging

Script logging
PierreRochard · Sep 23, 2020 · fd217ba · fd217ba
2 parents 34f7ad9 + cf235e4
commit fd217ba
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/bitcoin_acks/github_data/pull_request_events.py b/src/bitcoin_acks/github_data/pull_request_events.py
@@ -9,6 +9,7 @@
 from bitcoin_acks.github_data.polling_data import PollingData
 from bitcoin_acks.github_data.pull_requests_data import PullRequestsData
 from bitcoin_acks.github_data.repositories_data import RepositoriesData
+from bitcoin_acks.logging import log
 
 
 class PullRequestEvents(RepositoriesData):
@@ -52,8 +53,15 @@ def get(self):
 
 
 if __name__ == '__main__':
+    import os
+
     repository_path = 'bitcoin'
     repository_name = 'bitcoin'
+    log.debug('Running pull request events update script',
+              path=os.path.realpath(__file__),
+              repository_name=repository_name,
+              repository_path=repository_path
+              )
     create_or_update_database()
     pr_events = PullRequestEvents(repository_path=repository_path,
                                   repository_name=repository_name)
@@ -68,6 +76,7 @@ def get(self):
         time.sleep(math.ceil(sleep_time)+5)
 
         now = datetime.utcnow()
+        log.debug('In while True loop', sleep_time=sleep_time, now=now, last_update=pr_events.last_update)
         if pr_events.last_update.day != now.day:
             pr_data.update_all(state=PullRequestState.OPEN)
             # polling_data.update(last_open_update=True)

diff --git a/src/bitcoin_acks/scripts/send_tweet.py b/src/bitcoin_acks/scripts/send_tweet.py
@@ -5,6 +5,7 @@
 from twython import Twython
 
 from bitcoin_acks.database.session import session_scope
+from bitcoin_acks.logging import log
 from bitcoin_acks.models import PullRequests
 from bitcoin_acks.models.tweets import Tweets
 
@@ -36,10 +37,12 @@ def send_tweet(pull_request_number=None):
                     .first()
             )
         if next_pull_request is None:
+            log.debug('No pull requests found.')
             return
         commits_url = 'https://api.github.com/repos/bitcoin/bitcoin/commits'
         params = {'author': next_pull_request.author.login}
         response = requests.get(commits_url, params=params)
+        log.debug('github response', response=response)
         response_json = response.json()
         author_name = next_pull_request.author.name or next_pull_request.author.login
         if len(response_json) > 1 and next_pull_request.number != 14802:
@@ -53,7 +56,9 @@ def send_tweet(pull_request_number=None):
             Congratulations!  🎉🍾🎆
             '''.format(author_name,
                        next_pull_request.html_url)
+        log.debug('tweet status', status=status)
         tweet = twitter.update_status(status=status)
+        log.debug('tweet', tweet=tweet)
         new_tweet = Tweets()
         new_tweet.id = tweet['id']
         new_tweet.pull_request_id = next_pull_request.number
@@ -69,4 +74,5 @@ def send_tweet(pull_request_number=None):
                         type=int,
                         default=None)
     args = parser.parse_args()
+    log.debug(f'sending tweet for {args.pr_number}')
     send_tweet(pull_request_number=args.pr_number)