Do not commit. Bug issue in the form of a CL.

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld and Gerrit."""
 
 from __future__ import print_function
@@ skipping 336 matching lines @@
       print_text.append('  %s: %s' % (builder, tests))
       parameters = {
           'builder_name': builder,
           'changes': [{
               'author': {'email': issue_props['owner_email']},
               'revision': options.revision,
           }],
           'properties': {
               'category': category,
               'issue': issue,
+              # Nodir, do even need this thing? Can it be removed?
               'master': master,

[tandrii(chromium), 2016/10/10 15:15:35]

i mean this line.

[nodir, 2016/10/10 17:18:11]

IDK. There may be recipes that rely on master properties.

[tandrii(chromium), 2016/10/10 17:25:45]

I see. Well, these recipes will be broken with LUCI then, so there is hope of
removing this some day.

               'patch_project': issue_props['project'],
               'patch_storage': 'rietveld',
               'patchset': patchset,
               'reason': options.name,
               'rietveld': rietveld_url,
           },
       }
       if 'presubmit' in builder.lower():
         parameters['properties']['dry_run'] = 'true'
       if tests:
         parameters['properties']['testfilter'] = tests
       if properties:
         parameters['properties'].update(properties)
       if options.clobber:
         parameters['properties']['clobber'] = True
       batch_req_body['builds'].append(
           {
               'bucket': bucket,
               'parameters_json': json.dumps(parameters),
               'client_operation_id': str(uuid.uuid4()),
               'tags': ['builder:%s' % builder,
                        'buildset:%s' % buildset,
+                       # Nodir, do even need this thing? Can it be removed?
                        'master:%s' % master,

[tandrii(chromium), 2016/10/10 15:15:35]

and this line.

[nodir, 2016/10/10 17:18:11]

same

                        'user_agent:git_cl_try']
           }
       )
 
   _buildbucket_retry(
       'triggering try jobs',
       http,
       buildbucket_put_url,
       'PUT',
       body=json.dumps(batch_req_body),
@@ skipping 4908 matching lines @@
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   setup_color.init()
   try:
     sys.exit(main(sys.argv[1:]))
   except KeyboardInterrupt:
     sys.stderr.write('interrupted\n')
     sys.exit(1)