)]}'
{
  "commit": "23923f0eba8f79b2a7015dd97b97323c1ab1db02",
  "tree": "33be6a62b9dbc19a35bb49673ae97ab484120498",
  "parents": [
    "e83a257b06a79ff3fc8f8d857f3a7e24728ca118"
  ],
  "author": {
    "name": "Felipe Monteiro",
    "email": "felipe.monteiro@att.com",
    "time": "Fri Mar 17 02:15:07 2017 +0000"
  },
  "committer": {
    "name": "Felipe Monteiro",
    "email": "felipe.monteiro@att.com",
    "time": "Fri Mar 17 02:15:07 2017 +0000"
  },
  "message": "Partially revert removal of time.sleep if v3 auth enabled in conf.\n\nIn a previous patch [0], an attempt was made to speed up Patrole\ntests by skipping over time.sleep(1) if v3 auth was enabled\nin tempest.conf.\n\nHowever, this resulted in more tests failing than otherwise should\nbe for non-admin roles (i.e. Member role). This is because:\n\n\"Fernet tokens are not subsecond aware and Keystone should only be\n precise to the second. Sleep to ensure we are passing the second\n boundary before attempting to authenticate.\" [1][2]\n\nThis patch guarantees that time.sleep(1) is always executed,\nregardless of identity auth version used by Tempest. While this\nprolongs the test runs in the gates, this change is necessary\nto guarantee correct test execution for non-admin roles.\n\n[0] https://review.openstack.org/#/c/444967/\n[1] https://github.com/openstack/tempest/blob/master/tempest/api/identity/v2/test_users.py\n[2] https://github.com/openstack/tempest/blob/master/tempest/api/identity/v3/test_users.py\n\nChange-Id: I007190906e710f540ed0e44e76bd17047fcddea4\nCloses-Bug: #1673497\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "abbb435bde60316de018276aee62c0dead57806f",
      "old_mode": 33188,
      "old_path": "patrole_tempest_plugin/rbac_utils.py",
      "new_id": "18f132e78255a7749504ef364e5cd6ccd0c8e4c5",
      "new_mode": 33188,
      "new_path": "patrole_tempest_plugin/rbac_utils.py"
    }
  ]
}