)]}'
{"specs/puppet-modules.rst":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"0270e1bf57e750131034da1de11dbc5859d8b63d","unresolved":false,"context_lines":[{"line_number":151,"context_line":"       PuppetLint.configuration.send(\u0027disable_class_parameter_defaults\u0027)"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"   * Modulefile ::"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"       name \u0027openstack-$module\u0027"},{"line_number":157,"context_line":"       version \u00270.0.1\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_762e8ba6","line":154,"updated":"2014-11-17 21:24:49.000000000","message":"Modulefile is no longer required by the forge. Only metadata.json is required. Is there another reason to have both?","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"a13cae234a69ddbcfa426dee45998e4a71717692","unresolved":false,"context_lines":[{"line_number":151,"context_line":"       PuppetLint.configuration.send(\u0027disable_class_parameter_defaults\u0027)"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"   * Modulefile ::"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"       name \u0027openstack-$module\u0027"},{"line_number":157,"context_line":"       version \u00270.0.1\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_7c1bc980","line":154,"in_reply_to":"5a890539_39e6b344","updated":"2014-11-17 23:19:43.000000000","message":"Modulefile was only deprecated in 3.6, so I guess this will still be required. Please disregard.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"change_message_id":"564c1533f0c5aab14179e09f5fe23ff505c567dd","unresolved":false,"context_lines":[{"line_number":151,"context_line":"       PuppetLint.configuration.send(\u0027disable_class_parameter_defaults\u0027)"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"   * Modulefile ::"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"       name \u0027openstack-$module\u0027"},{"line_number":157,"context_line":"       version \u00270.0.1\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_39e6b344","line":154,"in_reply_to":"5a890539_762e8ba6","updated":"2014-11-17 22:06:55.000000000","message":"I remember we used to pin puppet at version below 3.0. What is the oldest puppet version that requires Modulefile? +1 for less duplication, if it can be dropped.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"d3189f69e73ddae9037c411f7185f85241a8c1ca","unresolved":false,"context_lines":[{"line_number":151,"context_line":"       PuppetLint.configuration.send(\u0027disable_class_parameter_defaults\u0027)"},{"line_number":152,"context_line":""},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"   * Modulefile ::"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"       name \u0027openstack-$module\u0027"},{"line_number":157,"context_line":"       version \u00270.0.1\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_62dfa0d6","line":154,"in_reply_to":"5a890539_7c1bc980","updated":"2014-11-18 00:46:27.000000000","message":"Leaving in.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"b518cfe7fd5b8f6208eb36f16629f885ea91151b","unresolved":false,"context_lines":[{"line_number":158,"context_line":"       source \u0027git://git.openstack.org/openstack-infra/puppet-$module.git\u0027"},{"line_number":159,"context_line":"       author \u0027openstackci\u0027"},{"line_number":160,"context_line":"       license \u0027Apache 2.0\u0027"},{"line_number":161,"context_line":"       summary \u0027Puppet module for the pip package manager\u0027"},{"line_number":162,"context_line":"       description \u0027This module provides providers to help pip install well. Only tested against 2.7.\u0027"},{"line_number":163,"context_line":"       project_page \u0027https://github.com/openstack-ci/puppet-$module\u0027"},{"line_number":164,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_d6881f73","line":161,"updated":"2014-11-17 20:54:56.000000000","message":"Note to self: Update this reference of \u0027pip\u0027","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"d3189f69e73ddae9037c411f7185f85241a8c1ca","unresolved":false,"context_lines":[{"line_number":158,"context_line":"       source \u0027git://git.openstack.org/openstack-infra/puppet-$module.git\u0027"},{"line_number":159,"context_line":"       author \u0027openstackci\u0027"},{"line_number":160,"context_line":"       license \u0027Apache 2.0\u0027"},{"line_number":161,"context_line":"       summary \u0027Puppet module for the pip package manager\u0027"},{"line_number":162,"context_line":"       description \u0027This module provides providers to help pip install well. Only tested against 2.7.\u0027"},{"line_number":163,"context_line":"       project_page \u0027https://github.com/openstack-ci/puppet-$module\u0027"},{"line_number":164,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_82f1ec6c","line":161,"in_reply_to":"5a890539_d6881f73","updated":"2014-11-18 00:46:27.000000000","message":"Done","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"b518cfe7fd5b8f6208eb36f16629f885ea91151b","unresolved":false,"context_lines":[{"line_number":159,"context_line":"       author \u0027openstackci\u0027"},{"line_number":160,"context_line":"       license \u0027Apache 2.0\u0027"},{"line_number":161,"context_line":"       summary \u0027Puppet module for the pip package manager\u0027"},{"line_number":162,"context_line":"       description \u0027This module provides providers to help pip install well. Only tested against 2.7.\u0027"},{"line_number":163,"context_line":"       project_page \u0027https://github.com/openstack-ci/puppet-$module\u0027"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_f67b5b78","line":162,"updated":"2014-11-17 20:54:56.000000000","message":"Note to self: Update this reference of \u0027pip\u0027","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"d3189f69e73ddae9037c411f7185f85241a8c1ca","unresolved":false,"context_lines":[{"line_number":159,"context_line":"       author \u0027openstackci\u0027"},{"line_number":160,"context_line":"       license \u0027Apache 2.0\u0027"},{"line_number":161,"context_line":"       summary \u0027Puppet module for the pip package manager\u0027"},{"line_number":162,"context_line":"       description \u0027This module provides providers to help pip install well. Only tested against 2.7.\u0027"},{"line_number":163,"context_line":"       project_page \u0027https://github.com/openstack-ci/puppet-$module\u0027"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_a2f4285b","line":162,"in_reply_to":"5a890539_f67b5b78","updated":"2014-11-18 00:46:27.000000000","message":"Done","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"0270e1bf57e750131034da1de11dbc5859d8b63d","unresolved":false,"context_lines":[{"line_number":183,"context_line":"   * Metadata.json ::"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"       {"},{"line_number":186,"context_line":"         \"name\": \"openstackci-$module\","},{"line_number":187,"context_line":"         \"version\": \"0.0.1\","},{"line_number":188,"context_line":"         \"author\": \"Openstack CI\","},{"line_number":189,"context_line":"         \"summary\": \"Puppet module for $module\","}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_313dedc4","line":186,"updated":"2014-11-17 21:24:49.000000000","message":"The namespace here is inconsistent with the namespace given in the Modulefile example (\"openstackci\" vs \"openstack\"). Assuming there is good reason to keep both files, they should be consistent with one another.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"d3189f69e73ddae9037c411f7185f85241a8c1ca","unresolved":false,"context_lines":[{"line_number":183,"context_line":"   * Metadata.json ::"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"       {"},{"line_number":186,"context_line":"         \"name\": \"openstackci-$module\","},{"line_number":187,"context_line":"         \"version\": \"0.0.1\","},{"line_number":188,"context_line":"         \"author\": \"Openstack CI\","},{"line_number":189,"context_line":"         \"summary\": \"Puppet module for $module\","}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_c29e9406","line":186,"in_reply_to":"5a890539_313dedc4","updated":"2014-11-18 00:46:27.000000000","message":"I updated both to \"openstackci-$module\".","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"change_message_id":"e88b8e3103683f7bfdae04b8e63bfe7f2fdefb5c","unresolved":false,"context_lines":[{"line_number":187,"context_line":"         \"version\": \"0.0.1\","},{"line_number":188,"context_line":"         \"author\": \"Openstack CI\","},{"line_number":189,"context_line":"         \"summary\": \"Puppet module for $module\","},{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_1e391ef3","line":190,"updated":"2014-11-14 14:59:57.000000000","message":"I think we should make it consistent with Modulefile and use Apache 2.0 instead of ASL 2.0. (\u0027puppet module generate\u0027 also uses Apache 2.0)","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"change_message_id":"666e4582add4742412f3bccf0e722325c957f58b","unresolved":false,"context_lines":[{"line_number":187,"context_line":"         \"version\": \"0.0.1\","},{"line_number":188,"context_line":"         \"author\": \"Openstack CI\","},{"line_number":189,"context_line":"         \"summary\": \"Puppet module for $module\","},{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_ac244460","line":190,"in_reply_to":"5a890539_1e391ef3","updated":"2014-11-16 11:50:46.000000000","message":"I wholeheartedly agree. I guess I thought ASL 2 stood for Apache Software License 2.0.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"d3189f69e73ddae9037c411f7185f85241a8c1ca","unresolved":false,"context_lines":[{"line_number":187,"context_line":"         \"version\": \"0.0.1\","},{"line_number":188,"context_line":"         \"author\": \"Openstack CI\","},{"line_number":189,"context_line":"         \"summary\": \"Puppet module for $module\","},{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_028c1cd9","line":190,"in_reply_to":"5a890539_2c4774bd","updated":"2014-11-18 00:46:27.000000000","message":"I made them consistent and chose \"Apache 2.0\".","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"change_message_id":"aa71e9952b2016fd1c6f66fe5d69166b136888e3","unresolved":false,"context_lines":[{"line_number":187,"context_line":"         \"version\": \"0.0.1\","},{"line_number":188,"context_line":"         \"author\": \"Openstack CI\","},{"line_number":189,"context_line":"         \"summary\": \"Puppet module for $module\","},{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_2c4774bd","line":190,"in_reply_to":"5a890539_ac244460","updated":"2014-11-16 13:18:29.000000000","message":"Correct, ASL 2.0 does stand for Apache Software License 2.0. My comment is about using either ASL 2.0 or Apache 2.0 form consistently. I see no need to have both in the same project.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"change_message_id":"e88b8e3103683f7bfdae04b8e63bfe7f2fdefb5c","unresolved":false,"context_lines":[{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":194,"context_line":"         \"dependencies\": []"},{"line_number":195,"context_line":"       }"},{"line_number":196,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_beef2ae8","line":193,"updated":"2014-11-14 14:59:57.000000000","message":"For both project_page and issues_url\ns/openstack-ci/openstack-infra/\n\nAnd add /issues for issues_url. So it should be\n  \"project_page\": \"https://github.com/openstack-infra/puppet-$module\",\n  \"issues_url\": \"https://github.com/openstack-infra/puppet-$module/issues\",\n\nUnless issues_url should point to gerrit review page, I would like to get clarification on that myself.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":9624,"name":"Ramy Asselin","email":"ramyasselin@yahoo.com","username":"asselin"},"change_message_id":"d3189f69e73ddae9037c411f7185f85241a8c1ca","unresolved":false,"context_lines":[{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":194,"context_line":"         \"dependencies\": []"},{"line_number":195,"context_line":"       }"},{"line_number":196,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_2d94a989","line":193,"in_reply_to":"5a890539_ac58c48a","updated":"2014-11-18 00:46:27.000000000","message":"Ok, I chose the \u0027search\u0027 url for now, until we have a better option.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"change_message_id":"f0529af13e5888e043cdb91b6bfac2668ee31f26","unresolved":false,"context_lines":[{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":194,"context_line":"         \"dependencies\": []"},{"line_number":195,"context_line":"       }"},{"line_number":196,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_ec21ec4d","line":193,"in_reply_to":"5a890539_beef2ae8","updated":"2014-11-16 11:58:19.000000000","message":"I disagree. I don\u0027t think we should point at github at all. Project page should be http://ci.openstack.org/ and issues should be something on storyboard? Do we have a puppet project on storyboard or a storyboard for each puppet project?\n\nI think the correct place to report bugs right now is probably: https://storyboard.openstack.org/#!/project_group/55\n\nHowever, its not obvious from there where to go. I\u0027ll make an action item to talk about this at the next infra meeting. I think a storyboard project for puppet-modules might be the best approach but I\u0027m not sure if we can do that. Maybe storyboard requires a 1:1 with git repo. We could make storyboard projects for each puppet module (we do have a puppet-storyboard project) so idunno.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"},{"author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"change_message_id":"aa71e9952b2016fd1c6f66fe5d69166b136888e3","unresolved":false,"context_lines":[{"line_number":190,"context_line":"         \"license\": \"ASL 2.0\","},{"line_number":191,"context_line":"         \"source\": \"git://git.openstack.org/openstack-infra/puppet-$module.git\","},{"line_number":192,"context_line":"         \"project_page\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":193,"context_line":"         \"issues_url\": \"https://github.com/openstack-ci/puppet-$module\","},{"line_number":194,"context_line":"         \"dependencies\": []"},{"line_number":195,"context_line":"       }"},{"line_number":196,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_ac58c48a","line":193,"in_reply_to":"5a890539_ec21ec4d","updated":"2014-11-16 13:18:29.000000000","message":"I support the direction. \n\nI just did not want to have broken links in the spec. Although they are a result of repeated copy-pasta from existing module splits. At fault for this myself.","commit_id":"0c3001cac915fd928d7e86f8d322c4576702286f"}]}