)]}'
{"id":"openstack%2Fopenstack-manuals~663478","triplet_id":"openstack%2Fopenstack-manuals~master~Ia91d2abe303ce18f9eb392ca4db2be64df79e1d6","project":"openstack/openstack-manuals","branch":"master","topic":"bug/1831822","hashtags":[],"change_id":"Ia91d2abe303ce18f9eb392ca4db2be64df79e1d6","subject":"Add Placement service to Minimal deployment for Stein","status":"MERGED","created":"2019-06-06 02:26:14.000000000","updated":"2019-06-17 16:28:43.000000000","submitted":"2019-06-17 16:28:43.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"663478-1560788923638-91b8215b","meta_rev_id":"7dd73a9d9e24ab479a918f316ea0a073c3b82011","_number":663478,"virtual_id_number":663478,"owner":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},{"value":2,"date":"2019-06-17 16:28:43.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"all":[{"value":2,"date":"2019-06-17 16:17:57.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":2,"date":"2019-06-17 10:52:19.000000000","_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"all":[{"value":1,"date":"2019-06-17 16:17:57.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-07 11:59:33.000000000","updated_by":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"reviewer":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"state":"REVIEWER"},{"updated":"2019-06-17 10:52:19.000000000","updated_by":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"reviewer":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"state":"REVIEWER"},{"updated":"2019-06-17 16:17:57.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"},{"updated":"2019-06-17 16:28:43.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"f2d13abffdfdf8712f8ffe8667700e8c0f1a1746","author":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"date":"2019-06-06 02:26:14.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e92691be945789424621e248b8e8fc6d4b3acbe8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-06 02:37:44.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-linters http://logs.openstack.org/78/663478/1/check/openstack-tox-linters/1fc9eba/ : SUCCESS in 3m 41s\n- build-tox-manuals-checkbuild http://logs.openstack.org/78/663478/1/check/build-tox-manuals-checkbuild/09fe0ba/publish-docs/ : SUCCESS in 5m 23s","accounts_in_message":[],"_revision_number":1},{"id":"d55710b638ad596cad4c6fbf00dc86d98ba69022","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-06-07 11:59:33.000000000","message":"Patch Set 1: Code-Review+1\n\nyup, nova requires placement","accounts_in_message":[],"_revision_number":1},{"id":"9ec52b48820ba6b7fb5ed145e4950cb69c4e4316","author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"date":"2019-06-10 12:42:04.000000000","message":"Patch Set 1:\n\nWhile we have a placement install guide for Stein, I would also like to see these docs updated before this is merged:\n1. https://docs.openstack.org/install-guide/openstack-services.html#minimal-deployment-for-stein\n2. https://docs.openstack.org/install-guide/environment-networking-compute.html\n3. https://docs.openstack.org/install-guide/overview.html#example-architecture\n\nI\u0027m not entirely sure we should be adding this to the minimal deployment before we have verified documentation: https://docs.openstack.org/placement/stein/install/ it\u0027s something to say that it NEEDS to work together, but if a user can\u0027t get the config right, then it\u0027s alllll broken.\n\nI think we also need to be updating: https://www.openstack.org/software/project-navigator/openstack-components#openstack-services","accounts_in_message":[],"_revision_number":1},{"id":"ad7168fe879e614f92fc84efd9f9c17b0fddee2d","author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"date":"2019-06-10 12:42:11.000000000","message":"Patch Set 1: Code-Review-1","accounts_in_message":[],"_revision_number":1},{"id":"bf143e1703cfeff9de7c5a7ca58287ed2af2ca80","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-06-10 12:45:17.000000000","message":"Patch Set 1:\n\n\u003e I\u0027m not entirely sure we should be adding this to the minimal\n \u003e deployment before we have verified documentation: https://docs.openstack.org/placement/stein/install/\n \u003e it\u0027s something to say that it NEEDS to work together, but if a user\n \u003e can\u0027t get the config right, then it\u0027s alllll broken.\n\nI agree that the docs need work across several dimensions, but the simple fact is that nova can\u0027t work without placement installed (since something like ocata or pike) so we need to highlight that, here. It\u0027s only since stein that it is separate, however.","accounts_in_message":[],"_revision_number":1},{"id":"c3e67b6eeaabc186cee8d714bfe9500110732987","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-06-10 12:46:30.000000000","message":"Patch Set 1:\n\nOh, I should also mention: the warning on the top of https://docs.openstack.org/placement/stein/install/ was put there because none of the distros produced any verification of that information despite multiple promptings.\n\nhttps://docs.openstack.org/placement/stein/install/from-pypi.html is known to work well. There others are best guesses.","accounts_in_message":[],"_revision_number":1},{"id":"a91a5fa40b1f44dae7548b57b2bbb8a0f3fa6b66","author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"date":"2019-06-10 14:41:24.000000000","message":"Patch Set 1:\n\n\u003e \u003e I\u0027m not entirely sure we should be adding this to the minimal\n \u003e \u003e deployment before we have verified documentation:\n \u003e https://docs.openstack.org/placement/stein/install/\n \u003e \u003e it\u0027s something to say that it NEEDS to work together, but if a\n \u003e user\n \u003e \u003e can\u0027t get the config right, then it\u0027s alllll broken.\n \u003e \n \u003e I agree that the docs need work across several dimensions, but the\n \u003e simple fact is that nova can\u0027t work without placement installed\n \u003e (since something like ocata or pike) so we need to highlight that,\n \u003e here. It\u0027s only since stein that it is separate, however.\n\nMakes sense. This as a doc update, and if you\u0027re going to update that placement is required I think we really need to ensure that 1,2,3 (obviously this is 1) is completed on the list I provided.\nPlus I think it\u0027s worth updating the project navigator as a larger part of this change. The docs and what\u0027s on the official website should reflect each other.","accounts_in_message":[],"_revision_number":1},{"id":"f26773dfdcd3b66f65303fe53b3b9d597da54ecd","author":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"date":"2019-06-11 04:31:40.000000000","message":"Patch Set 1:\n\n\u003e 1. https://docs.openstack.org/install-guide/openstack-services.html#minimal-deployment-for-stein\n \u003e 2. https://docs.openstack.org/install-guide/environment-networking-compute.html\n \u003e 3. https://docs.openstack.org/install-guide/overview.html#example-architecture\n\nThis patch updates 1. I understand we should update 3, but can not understand 2. Are you assuming \"Placement node\"? Could you point out in detail?","accounts_in_message":[],"_revision_number":1},{"id":"e49d38620c1350001444c5954323cc9623a9e1f2","author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"date":"2019-06-11 15:34:41.000000000","message":"Patch Set 1:\n\n\u003e \u003e 1. https://docs.openstack.org/install-guide/openstack-services.html#minimal-deployment-for-stein\n \u003e \u003e 2. https://docs.openstack.org/install-guide/environment-networking-compute.html\n \u003e \u003e 3. https://docs.openstack.org/install-guide/overview.html#example-architecture\n \u003e \n \u003e This patch updates 1. I understand we should update 3, but can not\n \u003e understand 2. Are you assuming \"Placement node\"? Could you point\n \u003e out in detail?\n\nI admit I just went through the docs and found all the locations I felt it would be best updated other than just 1. \n2 was a concern for any configuration details that are required. Although I admit re-reading I am not particularly sure why I thought this was relevant... Any architecture updates would be appreciated, in summary :)","accounts_in_message":[],"_revision_number":1},{"id":"b7b68b2af761c17a6992e6f8fb71055f5e4286e9","author":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"date":"2019-06-15 00:40:55.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2d5e6c1fdd512aa0382450a15e8813e22602b280","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-15 01:03:07.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-linters http://logs.openstack.org/78/663478/2/check/openstack-tox-linters/6fba1b2/ : SUCCESS in 4m 23s\n- build-tox-manuals-checkbuild http://logs.openstack.org/78/663478/2/check/build-tox-manuals-checkbuild/a937981/publish-docs/ : SUCCESS in 6m 35s","accounts_in_message":[],"_revision_number":2},{"id":"97e73181e8164372d482db50ec7ee777afe3ab6c","author":{"_account_id":10607,"name":"Alexandra Settle","email":"a.settle@outlook.com","username":"asettle"},"date":"2019-06-17 10:52:19.000000000","message":"Patch Set 2: Code-Review+2\n\nLooking good!","accounts_in_message":[],"_revision_number":2},{"id":"012a831d33b64f00734d14b83c2908dfff0ef74f","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2019-06-17 16:17:57.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nThese can be seen at [1] and look good\n\n[1] http://logs.openstack.org/78/663478/2/check/build-tox-manuals-checkbuild/a937981/publish-docs/install-guide/overview.html","accounts_in_message":[],"_revision_number":2},{"id":"a3413314a53c2c4b6a38f0a42133db658b773bbd","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 16:18:10.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"57d1503aa10233a4d21d04d924910c8bf75e815c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 16:28:43.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-linters http://logs.openstack.org/78/663478/2/gate/openstack-tox-linters/1e18c86/ : SUCCESS in 4m 16s\n- build-tox-manuals-checkbuild http://logs.openstack.org/78/663478/2/gate/build-tox-manuals-checkbuild/af591d2/publish-docs/ : SUCCESS in 6m 05s","accounts_in_message":[],"_revision_number":2},{"id":"f821a82953aa8461c6e35cb9c81096f0efe50033","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-17 16:28:43.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"4b01c6cba00dab980cd031e3978cc15218bddebd","revisions":{"f829f506c78a748d81801841528926279d3d6229":{"kind":"REWORK","_number":1,"created":"2019-06-06 02:26:14.000000000","uploader":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"ref":"refs/changes/78/663478/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-manuals","ref":"refs/changes/78/663478/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/1"}}},"commit":{"parents":[{"commit":"1e0e60eec36e07eff7773598ecec685470f56e7f","subject":"Merge \"Update tools/www-generator.py for repo renames\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-manuals/commit/1e0e60eec36e07eff7773598ecec685470f56e7f"}]}],"author":{"name":"Hidekazu Nakamura","email":"hid-nakamura@vf.jp.nec.com","date":"2019-06-06 02:24:28.000000000","tz":0},"committer":{"name":"Hidekazu Nakamura","email":"hid-nakamura@vf.jp.nec.com","date":"2019-06-06 02:24:28.000000000","tz":0},"subject":"Add Placement service to Minimal deployment for Stein","message":"Add Placement service to Minimal deployment for Stein\n\nPlacement service installation is removed in nova installation since Stein.\nThis patch adds Placement service to Minimal deployment for Stein.\n\nChange-Id: Ia91d2abe303ce18f9eb392ca4db2be64df79e1d6\nCloses-Bug: #1831822\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-manuals/commit/f829f506c78a748d81801841528926279d3d6229"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-manuals/commit/f829f506c78a748d81801841528926279d3d6229"}]},"branch":"refs/heads/master"},"4b01c6cba00dab980cd031e3978cc15218bddebd":{"kind":"REWORK","_number":2,"created":"2019-06-15 00:40:55.000000000","uploader":{"_account_id":13111,"name":"Hidekazu Nakamura","email":"hidekazuna@gmail.com","username":"hid-nakamura"},"ref":"refs/changes/78/663478/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-manuals","ref":"refs/changes/78/663478/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-manuals refs/changes/78/663478/2"}}},"commit":{"parents":[{"commit":"1e0e60eec36e07eff7773598ecec685470f56e7f","subject":"Merge \"Update tools/www-generator.py for repo renames\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-manuals/commit/1e0e60eec36e07eff7773598ecec685470f56e7f"}]}],"author":{"name":"Hidekazu Nakamura","email":"hid-nakamura@vf.jp.nec.com","date":"2019-06-06 02:24:28.000000000","tz":0},"committer":{"name":"Hidekazu Nakamura","email":"hid-nakamura@vf.jp.nec.com","date":"2019-06-15 00:40:07.000000000","tz":0},"subject":"Add Placement service to Minimal deployment for Stein","message":"Add Placement service to Minimal deployment for Stein\n\nPlacement service installation is removed in nova installation since Stein.\nThis patch adds Placement service to Minimal deployment for Stein.\n\nChange-Id: Ia91d2abe303ce18f9eb392ca4db2be64df79e1d6\nCloses-Bug: #1831822\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-manuals/commit/4b01c6cba00dab980cd031e3978cc15218bddebd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-manuals/commit/4b01c6cba00dab980cd031e3978cc15218bddebd"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
