scripts: Make the -p arg optional

Signed-off-by: Sayan Chowdhury <[email protected]>

README.md

@@ -21,7 +21,7 @@ a job manually. If Fedimg is installed properly, the `bin/trigger_upload.py`
 script can be used for this purpose:
 
 ```
-usage: trigger_upload.py [-h] -u URL -c COMPOSE_ID [-p PUSH_NOTIFICATIONS]
+usage: trigger_upload.py [-h] -u URL -c COMPOSE_ID [-p]
 
 Trigger a manual upload process with the specified raw.xz URL
 
@@ -30,7 +30,7 @@ optional arguments:
   -u URL, --url URL     .raw.xz URL
   -c COMPOSE_ID, --compose-id COMPOSE_ID
                         compose id of the .raw.xz file
-  -p PUSH_NOTIFICATIONS, --push-notifications PUSH_NOTIFICATIONS
+  -p, --push-notifications
                         Bool to check if we need to push fedmsg notifications
 ```
 

bin/trigger_upload.py

@@ -30,9 +30,9 @@ def get_args():
         "-c", "--compose-id", type=str, help="compose id of the .raw.xz file",
         required=True)
     parser.add_argument(
-        "-p", "--push-notifications", type=bool,
+        "-p", "--push-notifications",
         help="Bool to check if we need to push fedmsg notifications",
-        required=False, default=False)
+        action="store_true", required=False)
     args = parser.parse_args()
 
     return args.url, args.compose_id, args.push_notifications