)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"d0cd17fdc1ba86c97a595ffbe18097db8bb5963b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ebd21a88_cc82a3ae","updated":"2021-11-29 12:52:57.000000000","message":"Let me put WIP until I fix missing dependencies to ensure backend package is installed before services are started.","commit_id":"1c930d42107ca5b2b2daa87eda0c8e8007fb149a"}],"manifests/deps.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"3ba8b35cc69a7db4a366ed6be9269d5fd81d9217","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"  # all cache settings should be applied and all packages should be installed"},{"line_number":33,"context_line":"  # before service startup"},{"line_number":34,"context_line":"  Oslo::Cache\u003c||\u003e -\u003e Anchor[\u0027ec2api::service::begin\u0027]"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"  # all db settings should be applied and all packages should be installed"},{"line_number":37,"context_line":"  # before dbsync starts"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"b7fd2b64_748a305c","line":34,"updated":"2022-01-05 19:23:29.000000000","message":"Shouldn\u0027t this be in the config phase instead of doing it this way?\n\nAnchor[\u0027ec2api::config::begin\u0027] -\u003e Oslo::Cache\u003c||\u003e -\u003e Anchor[\u0027ec2api::config::end\u0027] ?","commit_id":"c683b00904764d2500020511f742d7949cc355e1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3f467b78fce2b05c493bca53cb67e85e62e90431","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"  # all cache settings should be applied and all packages should be installed"},{"line_number":33,"context_line":"  # before service startup"},{"line_number":34,"context_line":"  Oslo::Cache\u003c||\u003e -\u003e Anchor[\u0027ec2api::service::begin\u0027]"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"  # all db settings should be applied and all packages should be installed"},{"line_number":37,"context_line":"  # before dbsync starts"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"d6940fe5_b757bcff","line":34,"in_reply_to":"b7fd2b64_748a305c","updated":"2022-01-06 01:05:49.000000000","message":"This follows the existing implementation like [1].\n\n[1] https://github.com/openstack/puppet-nova/blob/master/manifests/deps.pp#L61-L63\n\nThe oslo::cache resource manages not only configurations but also packages to it doesn\u0027t fully fit to the config phase, so usage of service anchor would make sense here.","commit_id":"c683b00904764d2500020511f742d7949cc355e1"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"be1d985efcbe53de8affb59831194462ef1276a0","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"  # all cache settings should be applied and all packages should be installed"},{"line_number":33,"context_line":"  # before service startup"},{"line_number":34,"context_line":"  Oslo::Cache\u003c||\u003e -\u003e Anchor[\u0027ec2api::service::begin\u0027]"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"  # all db settings should be applied and all packages should be installed"},{"line_number":37,"context_line":"  # before dbsync starts"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"6b98358f_60823ca9","line":34,"in_reply_to":"d6940fe5_b757bcff","updated":"2022-01-06 14:32:35.000000000","message":"It works though I think moving it to the config phase is probably better. but since this is already somewhere we can use it.","commit_id":"c683b00904764d2500020511f742d7949cc355e1"}]}