)]}'
{
  "log": [
    {
      "commit": "bfce1f219b292f27b902d3b34388710c0fe44a3c",
      "tree": "6ba41fecbfac8b2d6c3a831131b7d54f1add0b9d",
      "parents": [
        "ccf56b5ca278fd083946137a5c36cdd8ba2f230d"
      ],
      "author": {
        "name": "Ghanshyam Mann",
        "email": "gmann@ghanshyammann.com",
        "time": "Wed Feb 03 16:50:34 2021 -0600"
      },
      "committer": {
        "name": "Ghanshyam Mann",
        "email": "gmann@ghanshyammann.com",
        "time": "Wed Feb 03 16:50:34 2021 -0600"
      },
      "message": "Change Review merge policy from two +2 to single +2 requirement\n\nNow a days, Tempest does not have much active core reviewers due\nto that merging the incoming code change is taking too much time.\n\nAs you can see we have lot of backlogs in open review ~400\n\nIf situation improve in future and we have more Core review\nthen we can change the policy back to two +2 requirement.\n\nDiscussion in QA office hour:\n-  http://eavesdrop.openstack.org/meetings/qa/2021/qa.2021-01-26-14.00.log.html#l-147\n\nChange-Id: Ic005de3e56cb4fde9cfce183ee598c7e9f77f464\n"
    },
    {
      "commit": "1d11c5d696940d448337df72337000ed076dfbb1",
      "tree": "51e034a4029f69a2e072992f9b2aaa7b76614465",
      "parents": [
        "e227022df9728e7b04cb5b3482a9b3360706962d",
        "2096631932384a02a6ceb91a9fdf274862af8d87"
      ],
      "author": {
        "name": "Zuul",
        "email": "zuul@review.opendev.org",
        "time": "Wed Sep 25 03:55:51 2019 +0000"
      },
      "committer": {
        "name": "Gerrit Code Review",
        "email": "review@openstack.org",
        "time": "Wed Sep 25 03:55:51 2019 +0000"
      },
      "message": "Merge \"Add \u0027single core approve\u0027 policy for plugins sanity jobs failure\""
    },
    {
      "commit": "2096631932384a02a6ceb91a9fdf274862af8d87",
      "tree": "577d8f6ed8bb51e39bbe6e41ca6ad63ab160142e",
      "parents": [
        "4006211b59babda45fa20b1938f9dad40a584bf2"
      ],
      "author": {
        "name": "Ghanshyam Mann",
        "email": "gmann@ghanshyammann.com",
        "time": "Tue Jul 16 05:19:41 2019 +0000"
      },
      "committer": {
        "name": "Ghanshyam Mann",
        "email": "gmann@ghanshyammann.com",
        "time": "Tue Jul 16 05:19:41 2019 +0000"
      },
      "message": "Add \u0027single core approve\u0027 policy for plugins sanity jobs failure\n\nWe discussed in Denver PTG[1] to make plugins\nsanity jobs as voting which will help to mantain\nthe active plugins and blacklist the faulty plugins.\n\nBut when we get faulty plugins this job which will\nbe voting after https://review.opendev.org/#/c/641188/\ncan block the Tempest gate. So adding the process of\nsingle core approve policy for mergin the patches which\nwill fix or remove the faulty plugins to unblock the gate.\n\n[1] L196 https://etherpad.openstack.org/p/qa-train-ptg\n\nChange-Id: Id38e6270ac1bae8ba63761f9bab6f1e5d0a2252b\n"
    },
    {
      "commit": "349ba7541070376614421c33fc8394cce3103452",
      "tree": "6a4f911ca29bea7763ada040eeb5299d064461a0",
      "parents": [
        "7b1b398dbc1b7b80f5dc0d2485511069ca16e553"
      ],
      "author": {
        "name": "caoyuan",
        "email": "cao.yuan@99cloud.net",
        "time": "Tue Apr 23 19:40:06 2019 +0800"
      },
      "committer": {
        "name": "jacky06",
        "email": "zhang.min@99cloud.net",
        "time": "Thu May 23 10:19:31 2019 +0000"
      },
      "message": "Replace git.openstack.org URLs with opendev.org URLs\n\nThorough replacement of git.openstack.org and review.openstack.org URLs\nwith their opendev.org counterparts.\n\nChange-Id: I88e894db7b854d32593c770f5aa9b8a91fad7866\n"
    },
    {
      "commit": "6fe405dc616add887733082a7d89e47e2ed93bd0",
      "tree": "598a53b532dde83d81351b74fbb49a98366f3301",
      "parents": [
        "5d96f65d981c0272664160907b94548cfe6ec2a6"
      ],
      "author": {
        "name": "Felipe Monteiro",
        "email": "felipe.monteiro@att.com",
        "time": "Sun Sep 16 11:13:39 2018 -0600"
      },
      "committer": {
        "name": "Felipe Monteiro",
        "email": "felipe.monteiro@att.com",
        "time": "Tue Oct 30 19:32:56 2018 -0400"
      },
      "message": "Update REVIEWING docs with fast-track approval process\n\nThis patch set follows through on discussion related to fast\ntracking changes in Tempest that are required to unblock other\nproject gates. So, the \"When to approve\" section of the REVIEWING\ndocumentation has been expanded to include a fast track\nprovision: that 1 core reviwer can +2 a change and approve it,\nprovided that the core reviwer belonging to the project with\nthe failing gate +1\u0027s the change, too.\n\nChange-Id: Iafb9de650cd79b3c72b03e20690dd80a95308b5b\n"
    },
    {
      "commit": "a7365ae219d03bec5cc2477298fba6356b51180d",
      "tree": "8fec0b4abc29359177d35d3e595346c699f5dc10",
      "parents": [
        "7e3c936898d05eb9f52e5ce37f463f903ee16ed5"
      ],
      "author": {
        "name": "Felipe Monteiro",
        "email": "felipe.monteiro@att.com",
        "time": "Fri Sep 21 20:32:56 2018 +0100"
      },
      "committer": {
        "name": "Felipe Monteiro",
        "email": "felipe.monteiro@att.com",
        "time": "Fri Sep 21 20:32:56 2018 +0100"
      },
      "message": "fix: Re-clarify unit test REVIEWING documentation\n\nThis patch set corrects some misleading documentation under \"Unit\nTests\" section in REVIEWING.rst.\n\nIt currently claims that service clients do not require unit test\ncoverage -- but this is false. This is because Tempest now places\nall of its service clients in tempest.lib. And as per\nhttps://docs.openstack.org/tempest/latest/library.html#testing it\nis required to add unit tests for all service client interfaces.\n\nThus this makes the documentation language clear that service clients\nrequire unit tests.\n\nChange-Id: Iac6ba71496e20b6724a6a2ca4ec2beb42f2a58c8\n"
    },
    {
      "commit": "f89ab81c38a7ffc87303b10e156759e25c09abdb",
      "tree": "e635bcd47560e231d6dd44590ca84a8364b6fdef",
      "parents": [
        "de5f0da10e44a52e5bc5f577e4ec33231add3831"
      ],
      "author": {
        "name": "Felipe Monteiro",
        "email": "felipe.monteiro@att.com",
        "time": "Tue Jul 10 20:34:02 2018 -0400"
      },
      "committer": {
        "name": "Felipe Monteiro",
        "email": "felipe.monteiro@att.com",
        "time": "Sat Jul 14 19:30:06 2018 -0400"
      },
      "message": "docs: Add information on test removal/interop to REVIEWING\n\nThis patch set adds information on test removal, relocation,\nrenaming to REVIEWING because it is important that such\nactions do not break interop. Interop is closely tied to\nTempest because it directly references Tempest tests that\nare not only expected to exist, but to also work.\n\nThe same is true of breaking blacklist/whitelist\nreferences to Tempest tests, this is also included\nin the new documentation section.\n\nIt is important that there be REVIEWING guidelines\nin place to assist reviewers understand this importance.\nAlso references are included for defcore/interop to help\nusers find more information on these topics.\n\nCurrently interop is only mentioned in 1 place in Tempest [0]\nand yet there is little information about it. This patchset\naims to make it easier to find more information about it\nfor reviewers and users alike.\n\n[0] http://git.openstack.org/cgit/openstack/tempest/tree/doc/source/test_removal.rst\n\nChange-Id: Ifbde674b42355077fcd8daa07be8be1248b77b0f\n"
    },
    {
      "commit": "18610f9c54b616aa85ba80447d943d7d73aa8b4f",
      "tree": "88c4580237af1916299bcc2caf94ec333f80a554",
      "parents": [
        "98cf467c58e4521376569e44bfdb8c99f6826344"
      ],
      "author": {
        "name": "Chang Liu",
        "email": "liuchang77@chinaunicom.cn",
        "time": "Mon Apr 02 09:57:46 2018 +0800"
      },
      "committer": {
        "name": "Chang Liu",
        "email": "liuchang77@chinaunicom.cn",
        "time": "Mon Apr 02 08:44:29 2018 +0000"
      },
      "message": "Fix a plural mistake REVIEWING.rst\n\nReviewing the REVIEWING.rst of tempest,\nI noticed a plural mistake.\nThe word \"APIs\" is plural and it shouldn\u0027t add \u0027a\u0027 in the front.\nAnd there is an extra \"that\" before \"which\".\nSo I fix it.\n\nChange-Id: I84d81f44314feadaebfcd4a834f854364328593b\n"
    },
    {
      "commit": "b78b923e5a9c8f8a21cad2c9aed4bd12c28d1a9b",
      "tree": "efa5149ef27017f08712450ef524aaf3a0b2f293",
      "parents": [
        "10d9e733494a9a898a97dfe8abf8e4ba39d88db8"
      ],
      "author": {
        "name": "Masayuki Igawa",
        "email": "masayuki.igawa@gmail.com",
        "time": "Fri Nov 17 16:12:37 2017 +0900"
      },
      "committer": {
        "name": "Masayuki Igawa",
        "email": "masayuki.igawa@gmail.com",
        "time": "Mon Nov 20 07:39:09 2017 +0000"
      },
      "message": "Fix indentation in docs\n\nThis commit fixes indentation in tempest docs. These indentations are\nnot necessary and it causes a weird html outputs.\n\nChange-Id: I9c8714558a3327b7ad0b0ab0d3fdc7e770c3c75b\n"
    },
    {
      "commit": "ac670dce6d2f99176306815dabaa65fe9e0de575",
      "tree": "2c945c5203147d7fec733941fcdf079614c4ec1c",
      "parents": [
        "7c8dd48f75aeefb2ca0601c54b4f2a23eaad3e19"
      ],
      "author": {
        "name": "davyyy",
        "email": "si.dawei@zte.com.cn",
        "time": "Thu Nov 16 21:27:03 2017 +0800"
      },
      "committer": {
        "name": "davyyy",
        "email": "si.dawei@zte.com.cn",
        "time": "Thu Nov 16 21:55:52 2017 +0800"
      },
      "message": "Update URL from \"http\" to \"https\"\n\nMost of them can still be visited through http, but the following\none is necessary to update, so I change them all by this chance.\n\n $ git clone http://git.openstack.org/openstack/tempest\n\nChange-Id: I2b4da0670b599666e7151bddfff0fa6655640bc4\n"
    },
    {
      "commit": "c7a403debdb270412836cca30103512b9a7d497f",
      "tree": "50b63ac3d8f3d4a5e77fc6c9709d0edfc14f5a22",
      "parents": [
        "878845412eee5f0b2c5b86835372096723fa8596",
        "74a56aba03aaed32669ea7f0b011a5a7c2c6a730"
      ],
      "author": {
        "name": "Jenkins",
        "email": "jenkins@review.openstack.org",
        "time": "Sat Jul 22 01:47:46 2017 +0000"
      },
      "committer": {
        "name": "Gerrit Code Review",
        "email": "review@openstack.org",
        "time": "Sat Jul 22 01:47:46 2017 +0000"
      },
      "message": "Merge \"Doc: fix markups, capitalization and add 2 REVIEWING advices\""
    },
    {
      "commit": "e98720a22f70079a2b3bc55dce2c7ff214dd1ff5",
      "tree": "610306ada41094253287c936d124854e875a3bb0",
      "parents": [
        "55e301689c3e2bb0bf7189dafc824b53bdd7ea3c"
      ],
      "author": {
        "name": "chenxing",
        "email": "chen.xing@99cloud.net",
        "time": "Wed Jul 19 03:42:23 2017 +0000"
      },
      "committer": {
        "name": "chenxing",
        "email": "chen.xing@99cloud.net",
        "time": "Wed Jul 19 03:42:23 2017 +0000"
      },
      "message": "Update the documention for doc migration\n\nChange-Id: Ie6d322d1b9c93118af7c74986993010b9f4b76b2\n"
    },
    {
      "commit": "74a56aba03aaed32669ea7f0b011a5a7c2c6a730",
      "tree": "2914b1dad1904088d49509355877a83a2c446b23",
      "parents": [
        "21dd8a5ee2ab5a068cbb20d0468bd5f444fef59a"
      ],
      "author": {
        "name": "Jordan Pittier",
        "email": "jordan.pittier@scality.com",
        "time": "Wed Apr 26 16:46:20 2017 +0200"
      },
      "committer": {
        "name": "Jordan Pittier",
        "email": "jordan.pittier@scality.com",
        "time": "Tue Jul 11 20:26:32 2017 +0200"
      },
      "message": "Doc: fix markups, capitalization and add 2 REVIEWING advices\n\nI\u0027ve been reading our entire documentation to see where it could\nbe improved. It\u0027s guide good actually. While reading I\u0027ve fixed\nsome typos, added some capitalization to project names, mostly\ntrivial stuff.\n\nWhat\u0027s worth reviewing is the 2 paragraphs I added to the REVIEWING\nguidelines.\n\nChange-Id: I977de335119f4ff7b9aedcfbed31c264ed531ea8\n"
    },
    {
      "commit": "5d4f9cd5d562752a73c4d494e78011a3fb5f7811",
      "tree": "ad1fe15d8652212aaa9ae3fde741de0330351c76",
      "parents": [
        "c285b5dd600eee2d3b6d8bfc47f00746cd8c172f",
        "4661706f683afaeb9e0eb5d2109bb6c937d3fcfc"
      ],
      "author": {
        "name": "Jenkins",
        "email": "jenkins@review.openstack.org",
        "time": "Tue Nov 08 08:19:28 2016 +0000"
      },
      "committer": {
        "name": "Gerrit Code Review",
        "email": "review@openstack.org",
        "time": "Tue Nov 08 08:19:28 2016 +0000"
      },
      "message": "Merge \"Add deprecated code review guideline\""
    },
    {
      "commit": "f6a0b5e3725ec795e51902ff280a9c91b3631522",
      "tree": "7cdc3258030dd10770c9b3cb386ff37cfd6d1ccb",
      "parents": [
        "4db514cc0178662163e337bc0cddbdc7357c2220"
      ],
      "author": {
        "name": "Jordan Pittier",
        "email": "jordan.pittier@scality.com",
        "time": "Fri Sep 30 16:20:30 2016 +0200"
      },
      "committer": {
        "name": "Jordan Pittier",
        "email": "jordan.pittier@scality.com",
        "time": "Fri Sep 30 16:20:30 2016 +0200"
      },
      "message": "Reviewing guideline: ask for CRD when new test+feature flag\n\nTo make sure that we don\u0027t accept new test that never gets executed.\n\nChange-Id: I24435e575c9488c04a38e26719d5d4cfa29d7c27\n"
    },
    {
      "commit": "4661706f683afaeb9e0eb5d2109bb6c937d3fcfc",
      "tree": "7ea83b750f16c8a1c712a87df87d3523c18549b7",
      "parents": [
        "29741bc7c278e48925d58b27743540ac384701d8"
      ],
      "author": {
        "name": "Masayuki Igawa",
        "email": "masayuki.igawa@gmail.com",
        "time": "Fri Sep 02 16:38:56 2016 +0900"
      },
      "committer": {
        "name": "Masayuki Igawa",
        "email": "masayuki.igawa@gmail.com",
        "time": "Sat Sep 03 15:13:28 2016 +0000"
      },
      "message": "Add deprecated code review guideline\n\nThis commit adds a deprecated code review guideline. We have some\ndeprecated code (e.g. stress test) now. And we sometimes wonder whether\nwe should review a patch for deprecated code or not. So this patch tries\nto clear up the situation.\n\nChange-Id: I188cd9fccae9396b143fa85071387e8a5bb5f3d1\n"
    },
    {
      "commit": "b786dcae6c5e8e6e17838adebb7a98dc256c1464",
      "tree": "653366e64e67413787c57456d36c2d25ed1ed840",
      "parents": [
        "26300f9bd3fc929817a3784bfb550e4ec928dc12"
      ],
      "author": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Wed Jun 22 10:32:45 2016 -0400"
      },
      "committer": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Wed Jun 22 10:32:45 2016 -0400"
      },
      "message": "Add section on release notes to reviewing doc\n\nThis commit adds a brief section about ensuring release notes are\npresent to the reviewing doc. This should hopefully make it more\nclear to reviewers when we need to include release notes.\n\nChange-Id: I0815b9c24f6ec9e384b1657c94ed387d0d748392\n"
    },
    {
      "commit": "1608cc0950e24697039672e3d809e0dab8a931f8",
      "tree": "ffc2451fe6fe5b8e8f679bcfb5969e2321ef0626",
      "parents": [
        "2595e6584100c704f5f736d2f6247347a6935b5a"
      ],
      "author": {
        "name": "Masayuki Igawa",
        "email": "masayuki.igawa@gmail.com",
        "time": "Mon Apr 11 13:54:38 2016 +0900"
      },
      "committer": {
        "name": "Masayuki Igawa",
        "email": "masayuki.igawa@gmail.com",
        "time": "Mon Apr 11 13:54:38 2016 +0900"
      },
      "message": "Fix Review Checklist URL\n\nThis commit fixes the Review Checklist URL. The page has been move to\nthe OpenStack Infra Manual.\n\nChange-Id: Ieca7a9aca8fd13187533c2a6c7c8b0ff32ff80a3\n"
    },
    {
      "commit": "13a1ed668a438a9c96ba013e1237ddf2aac241c7",
      "tree": "2119a6f4dea44b0144fd04fdd518cb21d16db1e8",
      "parents": [
        "daca419af5f75443a09c6897e968c4af158412c1"
      ],
      "author": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Tue Sep 08 10:39:58 2015 -0400"
      },
      "committer": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Tue Sep 08 10:39:58 2015 -0400"
      },
      "message": "Add docs section on config options to reviewers guide\n\nThis commit adds an additional section to the reviewers guide to\nelaborate on the policy around configuration options. We already\ndocumented part of it in the plugin interface documentation, but this\nshould make the policy around it a bit more clear.\n\nChange-Id: Ied2d7b3298d613b7e986b31fd0a08ad7c73afa3b\n"
    },
    {
      "commit": "f45ba2e604755e6ef92da690e80ad695913230a5",
      "tree": "c3981396fcc1e73683b86ad21d1beb3fd06b7f7d",
      "parents": [
        "f4c63c4e79c99ca159578ee6c91fa47c27230cb1"
      ],
      "author": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Mon Aug 24 15:05:01 2015 -0400"
      },
      "committer": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Wed Sep 02 11:06:44 2015 -0400"
      },
      "message": "Cleanup tempest docs a bit\n\nThis commit fixes a few things in the tempest docs. First it fixes all\nof the sphinx warnings and enables fail on warn to ensure we\u0027re using\nvalid sphinx everywhere. It also adds a link from the configuration\nguide to the sample config file.\n\nChange-Id: I3ad645a6bbfa46b4498e1732410743e46d6eb0cb\n"
    },
    {
      "commit": "550788819a10219a022587ad505832c6407b35df",
      "tree": "7b095b90e4e5f7fd2e32b434ef341936bc6a783b",
      "parents": [
        "0e068fb726e7511f196dc82aa59a7fc70a1cfb94"
      ],
      "author": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Tue Aug 12 19:01:34 2014 -0400"
      },
      "committer": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Tue Sep 23 10:04:40 2014 -0400"
      },
      "message": "Add a test documentation section to the docs\n\nThis commit adds a new section the HACKING and REVIEWING docs. It\nexplains the guidelines around inline documentation when\ncontributing new tests.\n\nChange-Id: Ib56723d0c748baf41439975102d1363f81085f50\n"
    },
    {
      "commit": "16dd51b50f9c44d757f9ea6a64cc6746d33c30a2",
      "tree": "e273cb981082ed1daf859534395a1819ad0ae84e",
      "parents": [
        "9cfa75e3af8098fe676d38880ab0f345df424472"
      ],
      "author": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Wed Jun 11 12:04:51 2014 -0400"
      },
      "committer": {
        "name": "Matthew Treinish",
        "email": "mtreinish@kortar.org",
        "time": "Thu Aug 21 20:54:10 2014 -0400"
      },
      "message": "Add REVIEWING.rst\n\nThis commit adds a file to outline what to look for as a tempest code\nreviewer.\n\nChange-Id: I945469842183d0e5022cd41f823d4892adff333f\n"
    }
  ]
}
