)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"bbbd9f9f7d5b43d4fcf79f68e77498047fbc763d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This adds the option to specify the replication mode as:"},{"line_number":10,"context_line":"Synchronous, Asynchronous or SnapshotOnly."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: I920f958db1d48e52b548082d852c03f427a279ca"},{"line_number":13,"context_line":"Change-Id: I74d31e7a262f3a59f5e82c710f3ca5fd84e3ea60"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_02a82975","line":12,"updated":"2019-10-02 20:59:09.000000000","message":"Oh, just for future reference, this format is deprecated:\n\nhttp://lists.openstack.org/pipermail/openstack-dev/2018-January/126535.html","commit_id":"cf1b53643d55277a70861f1835f6895c5a2e8e16"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"dff69a2071b98f8265802dc45d1a6bce397c27c0","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This adds the option to specify the replication mode as:"},{"line_number":10,"context_line":"Synchronous, Asynchronous or SnapshotOnly."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Depends-On: I920f958db1d48e52b548082d852c03f427a279ca"},{"line_number":13,"context_line":"Change-Id: I74d31e7a262f3a59f5e82c710f3ca5fd84e3ea60"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_4803a310","line":12,"in_reply_to":"3fa7e38b_02a82975","updated":"2019-10-02 22:33:45.000000000","message":"Hi Sean, thanks for point this out. The NetApp CI is still using an older version of Zuul and the Depends-On works only with the Change-Id. We are upgrading our CI and we\u0027ll solve this issue soon. Thanks.","commit_id":"cf1b53643d55277a70861f1835f6895c5a2e8e16"}],"cinder/volume/drivers/solidfire.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"48b8439e4a4c16fbb9d4263a172e4005ba45f9ea","unresolved":false,"context_lines":[{"line_number":335,"context_line":"            \"string\","},{"line_number":336,"context_line":"            enum\u003d[\"Async\", \"Sync\", \"SnapshotsOnly\"])"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        return properties, \u0027SolidFire\u0027"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    def __getattr__(self, attr):"},{"line_number":341,"context_line":"        if hasattr(self.target_driver, attr):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_2ed98f21","line":338,"range":{"start_line":338,"start_character":28,"end_line":338,"end_character":37},"updated":"2019-10-03 01:20:00.000000000","message":"The documentation for this function in cinder/volume/driver.py says that the vendor prefix is equal to the vendor name, and BaseVD.init_capabilities() in that same file is not case insensitive.  (It will log a warning and not register the property.)  So you either need to change this to \u0027solidfire\u0027, or change the prefix at line 332 to \u0027SolidFire\u0027.  I don\u0027t think it matters which you choose as long as you are consistent.","commit_id":"cf1b53643d55277a70861f1835f6895c5a2e8e16"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"404bd4eaf672535e2fb31192d80c5ddd028d422c","unresolved":false,"context_lines":[{"line_number":335,"context_line":"            \"string\","},{"line_number":336,"context_line":"            enum\u003d[\"Async\", \"Sync\", \"SnapshotsOnly\"])"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        return properties, \u0027SolidFire\u0027"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    def __getattr__(self, attr):"},{"line_number":341,"context_line":"        if hasattr(self.target_driver, attr):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_d173bab0","line":338,"range":{"start_line":338,"start_character":28,"end_line":338,"end_character":37},"in_reply_to":"3fa7e38b_2ed98f21","updated":"2019-10-03 02:35:02.000000000","message":"Fixed. Thank you Rosmaita!","commit_id":"cf1b53643d55277a70861f1835f6895c5a2e8e16"}],"releasenotes/notes/netapp-solidfire-add-replication-mode-eb26535d0ec78cb4.yaml":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"f2c944f4a9f9a5ef2b3339bd6b4cc099e221f945","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    SolidFire supports Synchronous, Asynchronous and SnapshotsOnly replication"},{"line_number":5,"context_line":"    modes, but when the replication was being managed by Cinder it would"},{"line_number":6,"context_line":"    always set Asynchronous mode. This adds the option to use all these modes"},{"line_number":7,"context_line":"    from Cinder as well."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3fa7e38b_20ef65b1","line":7,"range":{"start_line":4,"start_character":3,"end_line":7,"end_character":24},"updated":"2019-10-02 16:40:25.000000000","message":"This doesn\u0027t need to be a bug report. Just state the name of the new config option and how it can be used to configure the replication type.","commit_id":"85f6ba5f64aa7d5cb14139c20f3b2e42aff150c7"},{"author":{"_account_id":18058,"name":"Lucio Seki","email":"lseki@redhat.com","username":"lseki"},"change_message_id":"4a4540ea1148d75156acaee10dae86b7f5954c14","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    SolidFire supports Synchronous, Asynchronous and SnapshotsOnly replication"},{"line_number":5,"context_line":"    modes, but when the replication was being managed by Cinder it would"},{"line_number":6,"context_line":"    always set Asynchronous mode. This adds the option to use all these modes"},{"line_number":7,"context_line":"    from Cinder as well."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3fa7e38b_932b112f","line":7,"range":{"start_line":4,"start_character":3,"end_line":7,"end_character":24},"in_reply_to":"3fa7e38b_20ef65b1","updated":"2019-10-02 17:10:54.000000000","message":"Done","commit_id":"85f6ba5f64aa7d5cb14139c20f3b2e42aff150c7"}]}