)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"060a5bdbede14a1b257f9d5c15ecbc57971a9442","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     lkuchlan \u003clkuchlan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-12-03 09:11:06 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create a volume snapshot based on volume with non-ascii description"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" This test was uploaded since there was a bug [1] while creating a volume"},{"line_number":10,"context_line":" snapshot if the description of the parent volume contains non-ascii code."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_07c9ff76","line":7,"updated":"2017-12-03 17:17:24.000000000","message":"Line too long. Maybe change to \"Add test for volume create with non-ascii characters\"?","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"d043db7d459a0fa0161a086009d29629400c46f6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     lkuchlan \u003clkuchlan@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-12-03 09:11:06 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create a volume snapshot based on volume with non-ascii description"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" This test was uploaded since there was a bug [1] while creating a volume"},{"line_number":10,"context_line":" snapshot if the description of the parent volume contains non-ascii code."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_a67370e7","line":7,"in_reply_to":"df87a7cf_07c9ff76","updated":"2017-12-04 05:59:21.000000000","message":"Thanks for your comments,\nDone","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"060a5bdbede14a1b257f9d5c15ecbc57971a9442","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create a volume snapshot based on volume with non-ascii description"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" This test was uploaded since there was a bug [1] while creating a volume"},{"line_number":10,"context_line":" snapshot if the description of the parent volume contains non-ascii code."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" [1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1383899"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_27c68369","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":1},"updated":"2017-12-03 17:17:24.000000000","message":"Leading spaces next few lines.","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"d043db7d459a0fa0161a086009d29629400c46f6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Create a volume snapshot based on volume with non-ascii description"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":" This test was uploaded since there was a bug [1] while creating a volume"},{"line_number":10,"context_line":" snapshot if the description of the parent volume contains non-ascii code."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" [1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1383899"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_667d68b0","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":1},"in_reply_to":"df87a7cf_27c68369","updated":"2017-12-04 05:59:21.000000000","message":"Done","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"060a5bdbede14a1b257f9d5c15ecbc57971a9442","unresolved":false,"context_lines":[{"line_number":9,"context_line":" This test was uploaded since there was a bug [1] while creating a volume"},{"line_number":10,"context_line":" snapshot if the description of the parent volume contains non-ascii code."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" [1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1383899"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9fc41a426b29fad4f4d42a584325f0465a20effe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_e7a5fbb6","line":12,"updated":"2017-12-03 17:17:24.000000000","message":"Is there a Cinder bug you can link to?","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"d043db7d459a0fa0161a086009d29629400c46f6","unresolved":false,"context_lines":[{"line_number":9,"context_line":" This test was uploaded since there was a bug [1] while creating a volume"},{"line_number":10,"context_line":" snapshot if the description of the parent volume contains non-ascii code."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" [1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1383899"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I9fc41a426b29fad4f4d42a584325f0465a20effe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df87a7cf_2651a037","line":12,"in_reply_to":"df87a7cf_e7a5fbb6","updated":"2017-12-04 05:59:21.000000000","message":"This is the only bug i know about","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"}],"cinder/tests/tempest/api/volume/test_volume_unicode.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"060a5bdbede14a1b257f9d5c15ecbc57971a9442","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        volume \u003d self.create_volume(description\u003ddescription)"},{"line_number":65,"context_line":"        vol_info \u003d self.volumes_client.show_volume(volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":66,"context_line":"        self.assertEqual(description, vol_info[\u0027description\u0027])"},{"line_number":67,"context_line":"        self.create_snapshot(volume[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_87aaef82","line":67,"updated":"2017-12-03 17:17:24.000000000","message":"Should we check anything on the snapshot creation?","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"d043db7d459a0fa0161a086009d29629400c46f6","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        volume \u003d self.create_volume(description\u003ddescription)"},{"line_number":65,"context_line":"        vol_info \u003d self.volumes_client.show_volume(volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":66,"context_line":"        self.assertEqual(description, vol_info[\u0027description\u0027])"},{"line_number":67,"context_line":"        self.create_snapshot(volume[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_e67178e2","line":67,"in_reply_to":"df87a7cf_87aaef82","updated":"2017-12-04 05:59:21.000000000","message":"We only need to check that the snapshot was created successfully and create_snapshot wrapper do it for us.","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c42e9de6730e2febc5365a3941fa28abbc671e59","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        volume \u003d self.create_volume(description\u003ddescription)"},{"line_number":65,"context_line":"        vol_info \u003d self.volumes_client.show_volume(volume[\u0027id\u0027])[\u0027volume\u0027]"},{"line_number":66,"context_line":"        self.assertEqual(description, vol_info[\u0027description\u0027])"},{"line_number":67,"context_line":"        self.create_snapshot(volume[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_51910099","line":67,"in_reply_to":"df87a7cf_e67178e2","updated":"2017-12-04 06:12:49.000000000","message":"Maybe we can add check to verify the snapshot description was updated","commit_id":"e6aa64a78edac33a1e77e3700697d2ad5a984de1"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d1c960655fc8fe30bd69e5e7198853163af66641","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        self.assertEqual(fetched_volume[\u0027name\u0027],"},{"line_number":59,"context_line":"                         self.volume_name)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    def test_snapshot_create_volume_description_non_ascii_code(self):"},{"line_number":62,"context_line":"        # Create a volume with non-ascii description"},{"line_number":63,"context_line":"        description \u003d u\u0027\\u05e7\\u05d9\\u05d9\\u05e4\\u05e9\u0027"},{"line_number":64,"context_line":"        volume \u003d self.create_volume(description\u003ddescription)"}],"source_content_type":"text/x-python","patch_set":3,"id":"df87a7cf_28cfb8b0","line":61,"updated":"2017-12-07 18:47:17.000000000","message":"This needs to use a tempest config option to see if snapshots are supported, because it fails the ceph job.\n\nhttps://github.com/openstack/tempest/blob/master/tempest/config.py#L819","commit_id":"a323becd5c1c07283203b8cf97cb53dea9dd64ca"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b095c27cbb04d13b02711f0c873ce6abfab51bc6","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        self.assertEqual(fetched_volume[\u0027name\u0027],"},{"line_number":59,"context_line":"                         self.volume_name)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    def test_snapshot_create_volume_description_non_ascii_code(self):"},{"line_number":62,"context_line":"        # Create a volume with non-ascii description"},{"line_number":63,"context_line":"        description \u003d u\u0027\\u05e7\\u05d9\\u05d9\\u05e4\\u05e9\u0027"},{"line_number":64,"context_line":"        volume \u003d self.create_volume(description\u003ddescription)"}],"source_content_type":"text/x-python","patch_set":3,"id":"df87a7cf_e8ceb080","line":61,"in_reply_to":"df87a7cf_083f5487","updated":"2017-12-07 18:57:28.000000000","message":"Opened bug 1737015.","commit_id":"a323becd5c1c07283203b8cf97cb53dea9dd64ca"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"03526954a9b6943e675e915dd27251f36b2f68a4","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        self.assertEqual(fetched_volume[\u0027name\u0027],"},{"line_number":59,"context_line":"                         self.volume_name)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    def test_snapshot_create_volume_description_non_ascii_code(self):"},{"line_number":62,"context_line":"        # Create a volume with non-ascii description"},{"line_number":63,"context_line":"        description \u003d u\u0027\\u05e7\\u05d9\\u05d9\\u05e4\\u05e9\u0027"},{"line_number":64,"context_line":"        volume \u003d self.create_volume(description\u003ddescription)"}],"source_content_type":"text/x-python","patch_set":3,"id":"df87a7cf_083f5487","line":61,"in_reply_to":"df87a7cf_28cfb8b0","updated":"2017-12-07 18:48:35.000000000","message":"The failure in the ceph job is likely due to something else:\n\nhttp://logs.openstack.org/85/330285/176/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/da266b0/job-output.txt.gz#_2017-12-07_16_43_05_561043\n\nSince the ceph backend can handle snapshots.","commit_id":"a323becd5c1c07283203b8cf97cb53dea9dd64ca"}]}