)]}'
{"doc/upstream-training/source/slides/workflow-using-sandbox.rst":[{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":36,"context_line":"- Using Sandbox repository and Launchpad, next number of slides will show"},{"line_number":37,"context_line":"  the following steps:"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  - A reports a bug on Launchpad"},{"line_number":40,"context_line":"  - B sees the bug and assign her/himself to fix the bug"},{"line_number":41,"context_line":"  - B uploads a patch and invite A as a reviewer"},{"line_number":42,"context_line":"  - A reviews B\u0027s patch and gives feedback to B with comment"},{"line_number":43,"context_line":"  - B uploads a revised patch with comment"},{"line_number":44,"context_line":"  - A reviews B\u0027s revised patch and merge the patch"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"A: Sandbox Launchpad Bug list"},{"line_number":47,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_347b947c","line":44,"range":{"start_line":39,"start_character":4,"end_line":44,"end_character":51},"updated":"2016-10-16 07:41:11.000000000","message":"Let\u0027s make it as a group exercise and introduce the exercise in steps too.\n\nLike someone reports a bug and the rest of the group does the administration on it. We can use the detailed slides as a summary when they completed the exercise to see whether they did all the necessary steps.\n\nAs the next step someone assigns the bug to him/herself and uploads a patch as a solution. The rest of the group reviews the solution and comments, if needed they can discuss the solution on IRC too and the exercise ends when the patch is agreed and merged.\n\nAlso the idea behind the training is to give an overview to the students and let them figure out how to do things as opposed to describe everything to them as lectures.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":36,"context_line":"- Using Sandbox repository and Launchpad, next number of slides will show"},{"line_number":37,"context_line":"  the following steps:"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"  - A reports a bug on Launchpad"},{"line_number":40,"context_line":"  - B sees the bug and assign her/himself to fix the bug"},{"line_number":41,"context_line":"  - B uploads a patch and invite A as a reviewer"},{"line_number":42,"context_line":"  - A reviews B\u0027s patch and gives feedback to B with comment"},{"line_number":43,"context_line":"  - B uploads a revised patch with comment"},{"line_number":44,"context_line":"  - A reviews B\u0027s revised patch and merge the patch"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"A: Sandbox Launchpad Bug list"},{"line_number":47,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_efc4b060","line":44,"range":{"start_line":39,"start_character":4,"end_line":44,"end_character":51},"in_reply_to":"fa6399be_347b947c","updated":"2016-10-18 00:24:07.000000000","message":"Good idea. Let\u0027s add this concept to \"closing exercises\" part. I slightly changed the concept of this slide as to describe \"Learning example with two Sandboxes\", rather then letting students to do the same kind exercise.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":43,"context_line":"  - B uploads a revised patch with comment"},{"line_number":44,"context_line":"  - A reviews B\u0027s revised patch and merge the patch"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"A: Sandbox Launchpad Bug list"},{"line_number":47,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"- https://bugs.launchpad.net/openstack-dev-sandbox"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_d48a5863","line":46,"range":{"start_line":46,"start_character":0,"end_line":46,"end_character":2},"updated":"2016-10-16 07:41:11.000000000","message":"Remove \u0027A\u0027 and \u0027B\u0027 from the titles please.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":43,"context_line":"  - B uploads a revised patch with comment"},{"line_number":44,"context_line":"  - A reviews B\u0027s revised patch and merge the patch"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"A: Sandbox Launchpad Bug list"},{"line_number":47,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"- https://bugs.launchpad.net/openstack-dev-sandbox"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_efed90f0","line":46,"range":{"start_line":46,"start_character":0,"end_line":46,"end_character":2},"in_reply_to":"fa6399be_d48a5863","updated":"2016-10-18 00:24:07.000000000","message":"Done","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":97,"context_line":"  :width: 80%"},{"line_number":98,"context_line":"  :align: center"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"B: Branch creation"},{"line_number":101,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"- To differentiate working results from master branch"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_547e486a","line":100,"range":{"start_line":100,"start_character":0,"end_line":100,"end_character":18},"updated":"2016-10-16 07:41:11.000000000","message":"I would add here that it is the recommended way of using the local repo.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":97,"context_line":"  :width: 80%"},{"line_number":98,"context_line":"  :align: center"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"B: Branch creation"},{"line_number":101,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"- To differentiate working results from master branch"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_afe39804","line":100,"range":{"start_line":100,"start_character":0,"end_line":100,"end_character":18},"in_reply_to":"fa6399be_547e486a","updated":"2016-10-18 00:24:07.000000000","message":"Done","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":380,"context_line":"Good manners on Sandbox"},{"line_number":381,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"- Please only create 2 or 3 different changes and submit new patchsets"},{"line_number":384,"context_line":"  to those few changes in Sandbox repository and Launchpad."},{"line_number":385,"context_line":"- Please do not create 10 or more changes."},{"line_number":386,"context_line":"  (This is not the intention of Sandboxes!)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_54f568ea","line":383,"range":{"start_line":383,"start_character":38,"end_line":383,"end_character":45},"updated":"2016-10-16 07:41:11.000000000","message":"What do you mean by changes here?\n\nWe will have one bug per group preferably, which will be more than 3 hopefully. Same for solutions.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":380,"context_line":"Good manners on Sandbox"},{"line_number":381,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"- Please only create 2 or 3 different changes and submit new patchsets"},{"line_number":384,"context_line":"  to those few changes in Sandbox repository and Launchpad."},{"line_number":385,"context_line":"- Please do not create 10 or more changes."},{"line_number":386,"context_line":"  (This is not the intention of Sandboxes!)"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_bd9bd571","line":383,"range":{"start_line":383,"start_character":38,"end_line":383,"end_character":45},"in_reply_to":"fa6399be_54f568ea","updated":"2016-10-18 00:24:07.000000000","message":"I incorporated the explanation from http://docs.openstack.org/infra/manual/sandbox.html. Since we are using Sandbox with the administration from upstream trainers, I think your saying is reasonable :)","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":384,"context_line":"  to those few changes in Sandbox repository and Launchpad."},{"line_number":385,"context_line":"- Please do not create 10 or more changes."},{"line_number":386,"context_line":"  (This is not the intention of Sandboxes!)"},{"line_number":387,"context_line":"- Cleaning up uploaded files on Sandbox repository"},{"line_number":388,"context_line":"- Changing bug status in Launchpad to \"Invalid\", \"Won\u0027t Fix\", or"},{"line_number":389,"context_line":"  \"Fix Released\" for cleaning up purposes"},{"line_number":390,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_74998cc3","line":387,"range":{"start_line":387,"start_character":2,"end_line":387,"end_character":50},"updated":"2016-10-16 07:41:11.000000000","message":"We can do that after the training I don\u0027t think it should be part of the training or even leave it there by marked every merged item as Barcelona training so it will be visible how we interacted with the students there.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":384,"context_line":"  to those few changes in Sandbox repository and Launchpad."},{"line_number":385,"context_line":"- Please do not create 10 or more changes."},{"line_number":386,"context_line":"  (This is not the intention of Sandboxes!)"},{"line_number":387,"context_line":"- Cleaning up uploaded files on Sandbox repository"},{"line_number":388,"context_line":"- Changing bug status in Launchpad to \"Invalid\", \"Won\u0027t Fix\", or"},{"line_number":389,"context_line":"  \"Fix Released\" for cleaning up purposes"},{"line_number":390,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_3d9d055a","line":387,"range":{"start_line":387,"start_character":2,"end_line":387,"end_character":50},"in_reply_to":"fa6399be_74998cc3","updated":"2016-10-18 00:24:07.000000000","message":"My original purpose is that clean up activities are needed. I added more to address your comment and to describe the original purpose.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":385,"context_line":"- Please do not create 10 or more changes."},{"line_number":386,"context_line":"  (This is not the intention of Sandboxes!)"},{"line_number":387,"context_line":"- Cleaning up uploaded files on Sandbox repository"},{"line_number":388,"context_line":"- Changing bug status in Launchpad to \"Invalid\", \"Won\u0027t Fix\", or"},{"line_number":389,"context_line":"  \"Fix Released\" for cleaning up purposes"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"Exercise"},{"line_number":392,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_9494608b","line":389,"range":{"start_line":388,"start_character":2,"end_line":389,"end_character":41},"updated":"2016-10-16 07:41:11.000000000","message":"This could/should be a last exercise.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":385,"context_line":"- Please do not create 10 or more changes."},{"line_number":386,"context_line":"  (This is not the intention of Sandboxes!)"},{"line_number":387,"context_line":"- Cleaning up uploaded files on Sandbox repository"},{"line_number":388,"context_line":"- Changing bug status in Launchpad to \"Invalid\", \"Won\u0027t Fix\", or"},{"line_number":389,"context_line":"  \"Fix Released\" for cleaning up purposes"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"Exercise"},{"line_number":392,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_6fafc094","line":389,"range":{"start_line":388,"start_character":2,"end_line":389,"end_character":41},"in_reply_to":"fa6399be_9494608b","updated":"2016-10-18 00:24:07.000000000","message":"My intention here is not related to exercise. It would be good to mention in closing exercises. I include this to the modification of my explanation on cleaning up activities.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"7a1f7a0a8fcc4c9075f77f9c0f9379d95ad3b619","unresolved":false,"context_lines":[{"line_number":391,"context_line":"Exercise"},{"line_number":392,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"- Match the example practice scenario with Gerrit Workflow"},{"line_number":395,"context_line":"- Do similar practice with your peer"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_b48fa473","line":394,"range":{"start_line":394,"start_character":2,"end_line":394,"end_character":58},"updated":"2016-10-16 07:41:11.000000000","message":"Could you please clarify what you mean by this exercise?","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":391,"context_line":"Exercise"},{"line_number":392,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"- Match the example practice scenario with Gerrit Workflow"},{"line_number":395,"context_line":"- Do similar practice with your peer"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_8f125457","line":394,"range":{"start_line":394,"start_character":2,"end_line":394,"end_character":58},"in_reply_to":"fa6399be_b48fa473","updated":"2016-10-18 00:24:07.000000000","message":"I meant to match with the steps what students learned. Instead of this exercise I originally meant, I incorporated several \"We have seen in\" things in the revised patchset.","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"1a4a8f6af8d1a6862610715d0da9a0c09c76cb80","unresolved":false,"context_lines":[{"line_number":388,"context_line":"- Changing bug status in Launchpad to \"Invalid\", \"Won\u0027t Fix\", or"},{"line_number":389,"context_line":"  \"Fix Released\" for cleaning up purposes"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"Exercise"},{"line_number":392,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"- Match the example practice scenario with Gerrit Workflow"},{"line_number":395,"context_line":"- Do similar practice with your peer"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa6399be_4f083c29","line":395,"range":{"start_line":391,"start_character":0,"end_line":395,"end_character":36},"updated":"2016-10-18 00:24:07.000000000","message":"I removed it. It is better to deal with exercises in closing exercises part, as Ildiko mentioned: https://review.openstack.org/#/c/386728/","commit_id":"de3a03ef52f7a09b7b5242070327c0de18b66730"},{"author":{"_account_id":21723,"name":"Gergely Csatari","email":"gergely.csatari@nokia.com","username":"csatari"},"change_message_id":"51dfc20f912dfe0061cec1b1c687e015908e075f","unresolved":false,"context_lines":[{"line_number":147,"context_line":".. code-block:: console"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"  $ git add [FILE_PATH]"},{"line_number":150,"context_line":"  $ git commit -a"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"- Writing \u0027Related-Bug: #[Bug number in Lanuchpad]\u0027 on message content"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa6399be_8afea903","line":150,"range":{"start_line":150,"start_character":8,"end_line":150,"end_character":14},"updated":"2016-10-18 14:07:34.000000000","message":"--signoff is also needed.","commit_id":"ba4479f903f4a35cc285547d49e1af991c19a946"},{"author":{"_account_id":21723,"name":"Gergely Csatari","email":"gergely.csatari@nokia.com","username":"csatari"},"change_message_id":"52f077fe3e6e0d4582ac13b52d33e53bd9b54e95","unresolved":false,"context_lines":[{"line_number":147,"context_line":".. code-block:: console"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"  $ git add [FILE_PATH]"},{"line_number":150,"context_line":"  $ git commit -a"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"- Writing \u0027Related-Bug: #[Bug number in Lanuchpad]\u0027 on message content"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa6399be_4a4e51c2","line":150,"range":{"start_line":150,"start_character":8,"end_line":150,"end_character":14},"in_reply_to":"fa6399be_2a791d30","updated":"2016-10-18 14:22:55.000000000","message":"Oh, okay ;)","commit_id":"ba4479f903f4a35cc285547d49e1af991c19a946"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"8037a99becb256bc5e633ccbdf7255c5d49a67f9","unresolved":false,"context_lines":[{"line_number":147,"context_line":".. code-block:: console"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"  $ git add [FILE_PATH]"},{"line_number":150,"context_line":"  $ git commit -a"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"- Writing \u0027Related-Bug: #[Bug number in Lanuchpad]\u0027 on message content"},{"line_number":153,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa6399be_2a791d30","line":150,"range":{"start_line":150,"start_character":8,"end_line":150,"end_character":14},"in_reply_to":"fa6399be_8afea903","updated":"2016-10-18 14:15:41.000000000","message":"It\u0027s not required in OpenStack.","commit_id":"ba4479f903f4a35cc285547d49e1af991c19a946"}]}