)]}'
{
  "commit": "9cf0703f20f47143385260a6b63189f1c780f73e",
  "tree": "65bfbae7aabe4c6dca38f05c799b8d9cac2b7cbf",
  "parents": [
    "bd9798d0ec6326f6fef353461d5f7b9c8824354f"
  ],
  "author": {
    "name": "Matthew Treinish",
    "email": "mtreinish@kortar.org",
    "time": "Thu Oct 26 17:46:55 2017 -0400"
  },
  "committer": {
    "name": "Matthew Treinish",
    "email": "mtreinish@kortar.org",
    "time": "Thu Oct 26 18:08:51 2017 -0400"
  },
  "message": "Add unit tests to check for CONF getattr during import\n\nSince the early days in tempest we\u0027ve been fighting getattrs on CONF\nduring imports. We\u0027re able to get around this during test runs by lazy\nloading the conf file. However, in things like the tempest commands this\ndoesn\u0027t work because we rely on the config file not being parsed to set\nthe config file path. This commit adds unit tests to check the import\nof the command files for getattrs on CONF. This should prevent future\nregressions.\n\nWhile not strictly necessary because of the lazy loading this also gives\na framework to potentially address the CONF getatrr on discovery. The\nfirst revision of this patch includes the discovery test, for reference.\nBut we have 212 cases of getattr during import (which includes lots of\nskip decorators) so it\u0027s unlikely to change any time soon.\n\nChange-Id: Ib2c15dbd06ca810cc899258758cc8a297055fdf8\nCloses-Bug: #1726357\n",
  "tree_diff": [
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "6f1cfca7094928a9f75d39ead3d4392783d975f4",
      "new_mode": 33188,
      "new_path": "tempest/tests/test_imports.py"
    }
  ]
}