)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"417570806f4a0483cc6711514a7de8088b1b13a6","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Related to blueprint image-precache-support"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Depends-On: https://review.opendev.org/#/c/687141"},{"line_number":14,"context_line":"Change-Id: I121294a0064f64fa0c041b77b13b0c8399ecb062"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_8ec43161","line":13,"updated":"2019-10-21 23:49:51.000000000","message":"This will eventually change to whatever requirements change that bumps upper-constraints so we can bump required version of novaclient in this osc patch.","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"}],"openstackclient/compute/v2/aggregate.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"417570806f4a0483cc6711514a7de8088b1b13a6","unresolved":false,"context_lines":[{"line_number":409,"context_line":""},{"line_number":410,"context_line":""},{"line_number":411,"context_line":"class CacheImagesOnAggregate(command.Command):"},{"line_number":412,"context_line":"    _description \u003d _(\"Request images be cached on hosts within an aggregate\")"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"    def get_parser(self, prog_name):"},{"line_number":415,"context_line":"        parser \u003d super(CacheImagesOnAggregate, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_ae924d57","line":412,"updated":"2019-10-21 23:49:51.000000000","message":"Need to mention that ``--os-compute-api-version`` 2.81 or greater is required.","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"417570806f4a0483cc6711514a7de8088b1b13a6","unresolved":false,"context_lines":[{"line_number":419,"context_line":"            help\u003d_(\"Aggregate on which to cache images\")"},{"line_number":420,"context_line":"        )"},{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \"--image\","},{"line_number":423,"context_line":"            metavar\u003d\"\u003cimage\u003e\","},{"line_number":424,"context_line":"            action\u003d\"append\","},{"line_number":425,"context_line":"            help\u003d_(\"Image to be cached on hosts within the aggregate\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_4ea4f9b9","line":422,"range":{"start_line":422,"start_character":13,"end_line":422,"end_character":15},"updated":"2019-10-21 23:49:51.000000000","message":"I\u0027m not sure if this is correct since we require at least one image right?\n\nI\u0027m thinking like the \u0027host\u0027 arg in AddAggregateHost.","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"a540f03c677eaf4b781d112c60786091bec00951","unresolved":false,"context_lines":[{"line_number":419,"context_line":"            help\u003d_(\"Aggregate on which to cache images\")"},{"line_number":420,"context_line":"        )"},{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \"--image\","},{"line_number":423,"context_line":"            metavar\u003d\"\u003cimage\u003e\","},{"line_number":424,"context_line":"            action\u003d\"append\","},{"line_number":425,"context_line":"            help\u003d_(\"Image to be cached on hosts within the aggregate\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_abf47488","line":422,"range":{"start_line":422,"start_character":13,"end_line":422,"end_character":15},"in_reply_to":"3fa7e38b_4ea4f9b9","updated":"2019-10-22 13:55:28.000000000","message":"It seemed like this was the only option, but I really just threw this together waiting for the rest of the stack to get closer so I haven\u0027t done a bunch of digging.","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ccaeb8b2bdd3fd96e7c2e1f500c08631de6a71f9","unresolved":false,"context_lines":[{"line_number":419,"context_line":"            help\u003d_(\"Aggregate on which to cache images\")"},{"line_number":420,"context_line":"        )"},{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \"--image\","},{"line_number":423,"context_line":"            metavar\u003d\"\u003cimage\u003e\","},{"line_number":424,"context_line":"            action\u003d\"append\","},{"line_number":425,"context_line":"            help\u003d_(\"Image to be cached on hosts within the aggregate\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_cceea7ec","line":422,"range":{"start_line":422,"start_character":13,"end_line":422,"end_character":15},"in_reply_to":"3fa7e38b_abf47488","updated":"2019-10-22 15:59:42.000000000","message":"If AddAggregateHost takes multiple host args, I\u0027m not seeing how that is done...is there magic?","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"beb5d2dd5e0927886b42422df1841aa77e9c875d","unresolved":false,"context_lines":[{"line_number":419,"context_line":"            help\u003d_(\"Aggregate on which to cache images\")"},{"line_number":420,"context_line":"        )"},{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \"--image\","},{"line_number":423,"context_line":"            metavar\u003d\"\u003cimage\u003e\","},{"line_number":424,"context_line":"            action\u003d\"append\","},{"line_number":425,"context_line":"            help\u003d_(\"Image to be cached on hosts within the aggregate\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_f39b2861","line":422,"range":{"start_line":422,"start_character":13,"end_line":422,"end_character":15},"in_reply_to":"3fa7e38b_cceea7ec","updated":"2019-10-23 16:42:28.000000000","message":"No it doesn\u0027t, it was just an example of an aggregate command that took an arg rather than an option. The server delete command is a better example since it takes one or more server args:\n\nhttps://github.com/openstack/python-openstackclient/blob/4.0.0/openstackclient/compute/v2/server.py#L1071","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"8430195c39a6f2ba2bbb5fc86ae3143727a88264","unresolved":false,"context_lines":[{"line_number":419,"context_line":"            help\u003d_(\"Aggregate on which to cache images\")"},{"line_number":420,"context_line":"        )"},{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \"--image\","},{"line_number":423,"context_line":"            metavar\u003d\"\u003cimage\u003e\","},{"line_number":424,"context_line":"            action\u003d\"append\","},{"line_number":425,"context_line":"            help\u003d_(\"Image to be cached on hosts within the aggregate\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_133924f1","line":422,"range":{"start_line":422,"start_character":13,"end_line":422,"end_character":15},"in_reply_to":"3fa7e38b_f39b2861","updated":"2019-10-23 17:03:51.000000000","message":"Ah okay I didn\u0027t realize I could nargs a non-flag option.","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"417570806f4a0483cc6711514a7de8088b1b13a6","unresolved":false,"context_lines":[{"line_number":422,"context_line":"            \"--image\","},{"line_number":423,"context_line":"            metavar\u003d\"\u003cimage\u003e\","},{"line_number":424,"context_line":"            action\u003d\"append\","},{"line_number":425,"context_line":"            help\u003d_(\"Image to be cached on hosts within the aggregate\")"},{"line_number":426,"context_line":"        )"},{"line_number":427,"context_line":"        return parser"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_ce8f892c","line":425,"range":{"start_line":425,"start_character":68,"end_line":425,"end_character":69},"updated":"2019-10-21 23:49:51.000000000","message":". May be specified multiple times.","commit_id":"6a76f2b7795aa4362769790255f70c42d2a784f2"}]}