)]}'
{"manifests/rgw.pp":[{"author":{"_account_id":8797,"name":"Andrew Woodward","email":"xarses@gmail.com","username":"xarses"},"change_message_id":"48fe0b4ecf7c64cd8b0c6ec0797cf5d0a2efe059","unresolved":false,"context_lines":[{"line_number":46,"context_line":"#   Optional. Default is \u0027/tmp/radosgw.sock\u0027."},{"line_number":47,"context_line":"#"},{"line_number":48,"context_line":"# [*rgw_print_continue*] True to send 100 codes to the client."},{"line_number":49,"context_line":"#   Optional. Default is false."},{"line_number":50,"context_line":"#"},{"line_number":51,"context_line":"# [*rgw_port*] Port the rados gateway listens."},{"line_number":52,"context_line":"#   Optional. Default is undef."}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_5302ad0c","line":49,"updated":"2015-08-05 00:15:16.000000000","message":"why switch to false? My understanding is that it\u0027s basically required for larger files from the client side (swift)","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"c62c8b67b66cdf6df0af50f1baaae68398a2dfae","unresolved":false,"context_lines":[{"line_number":46,"context_line":"#   Optional. Default is \u0027/tmp/radosgw.sock\u0027."},{"line_number":47,"context_line":"#"},{"line_number":48,"context_line":"# [*rgw_print_continue*] True to send 100 codes to the client."},{"line_number":49,"context_line":"#   Optional. Default is false."},{"line_number":50,"context_line":"#"},{"line_number":51,"context_line":"# [*rgw_port*] Port the rados gateway listens."},{"line_number":52,"context_line":"#   Optional. Default is undef."}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_54e4c90b","line":49,"in_reply_to":"1a4dcd0f_5302ad0c","updated":"2015-08-05 09:08:38.000000000","message":"Currently we\u0027re still configuring mod_fastcgi (firefly compatible) and don\u0027t use mod_proxy_fcgi (the hammer/current default).\nI.e. following http://docs.ceph.com/docs/v0.80.5/install/install-ceph-gateway/ instead of http://docs.ceph.com/docs/master/install/install-ceph-gateway/\nIn the firefly compatible version we do not always have support for 100-continue. Specifically we need the ceph provided fastcgi packages, available via the ceph::repo::fastcgi switch. But those are off by default, so this is choosing a compatible default.","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":8797,"name":"Andrew Woodward","email":"xarses@gmail.com","username":"xarses"},"change_message_id":"48fe0b4ecf7c64cd8b0c6ec0797cf5d0a2efe059","unresolved":false,"context_lines":[{"line_number":49,"context_line":"#   Optional. Default is false."},{"line_number":50,"context_line":"#"},{"line_number":51,"context_line":"# [*rgw_port*] Port the rados gateway listens."},{"line_number":52,"context_line":"#   Optional. Default is undef."},{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*syslog*] Whether or not to log to syslog."},{"line_number":55,"context_line":"#   Optional. Default is true."}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_770a3423","line":52,"updated":"2015-08-05 00:15:16.000000000","message":"What is the effect of this being undef from 443? Does this mean that only the socket is open now?","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"c62c8b67b66cdf6df0af50f1baaae68398a2dfae","unresolved":false,"context_lines":[{"line_number":49,"context_line":"#   Optional. Default is false."},{"line_number":50,"context_line":"#"},{"line_number":51,"context_line":"# [*rgw_port*] Port the rados gateway listens."},{"line_number":52,"context_line":"#   Optional. Default is undef."},{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*syslog*] Whether or not to log to syslog."},{"line_number":55,"context_line":"#   Optional. Default is true."}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_4f3754e3","line":52,"in_reply_to":"1a4dcd0f_770a3423","updated":"2015-08-05 09:08:38.000000000","message":"From my understanding, the two switches rgw_port and rgw_socket_path are mutually exclusive. If using external fastcgi then rgw_socket_path should be specified:\nhttp://docs.ceph.com/docs/v0.80.5/radosgw/config-ref/\nAlso, the default never was set to 443, it was actually set to 80 in params.pp, and this overlaps with the default port 80 in apache and consequently wasn\u0027t a valid config.","commit_id":"cde508695b45732facb68d389cbc8be825779387"}],"manifests/rgw.pp.ensure":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d0c504b2e0c5b07a7a803fafe5c1ee8ca4d542af","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Copyright (C) 2014 Catalyst IT Limited."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\");"}],"source_content_type":"application/octet-stream","patch_set":14,"id":"1a4dcd0f_2e4c656f","line":1,"updated":"2015-08-04 18:39:42.000000000","message":"What is this file about ?","commit_id":"95c19d07bf29d7eb01ab6af444575b8c7e6705dd"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"92e51b1e4b98f94767f11f487ca4c0aa5a85ab9c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Copyright (C) 2014 Catalyst IT Limited."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\");"}],"source_content_type":"application/octet-stream","patch_set":14,"id":"1a4dcd0f_bfd41dbd","line":1,"in_reply_to":"1a4dcd0f_2e4c656f","updated":"2015-08-04 22:01:56.000000000","message":"This is about bash-globbing gone wrong (meaning it wasn\u0027t supposed to be added to the commit). Going to remove it.\nBut for the curious - this was a try to add an \"ensure\" parameter which would also support an \"ensure \u003d\u003e absent\"... but never managed to get it to work fully - mostly because you can\u0027t do an \"absent\" on the socket files...","commit_id":"95c19d07bf29d7eb01ab6af444575b8c7e6705dd"}],"manifests/rgw/keystone.pp":[{"author":{"_account_id":8797,"name":"Andrew Woodward","email":"xarses@gmail.com","username":"xarses"},"change_message_id":"48fe0b4ecf7c64cd8b0c6ec0797cf5d0a2efe059","unresolved":false,"context_lines":[{"line_number":86,"context_line":"set -ex"},{"line_number":87,"context_line":"wget --no-check-certificate ${rgw_keystone_url}/${rgw_keystone_version}/certificates/ca -O /tmp/ca"},{"line_number":88,"context_line":"openssl x509 -in /tmp/ca -pubkey | certutil -A -d ${nss_db_path} -n ca -t \\\"TCu,Cu,Tuw\\\""},{"line_number":89,"context_line":"chown ${user} -R ${nss_db_path}\","},{"line_number":90,"context_line":"    unless  \u003d\u003e \"/bin/true  # comment to satisfy puppet syntax requirements"},{"line_number":91,"context_line":"set -ex"},{"line_number":92,"context_line":"certutil -d ${nss_db_path} -L | grep ^ca"}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_7758f432","line":89,"updated":"2015-08-05 00:15:16.000000000","message":"should we use a recursive file resource instead to set owner? We could also avoid this entirely if the exec is ran as that user instead.","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"c62c8b67b66cdf6df0af50f1baaae68398a2dfae","unresolved":false,"context_lines":[{"line_number":86,"context_line":"set -ex"},{"line_number":87,"context_line":"wget --no-check-certificate ${rgw_keystone_url}/${rgw_keystone_version}/certificates/ca -O /tmp/ca"},{"line_number":88,"context_line":"openssl x509 -in /tmp/ca -pubkey | certutil -A -d ${nss_db_path} -n ca -t \\\"TCu,Cu,Tuw\\\""},{"line_number":89,"context_line":"chown ${user} -R ${nss_db_path}\","},{"line_number":90,"context_line":"    unless  \u003d\u003e \"/bin/true  # comment to satisfy puppet syntax requirements"},{"line_number":91,"context_line":"set -ex"},{"line_number":92,"context_line":"certutil -d ${nss_db_path} -L | grep ^ca"}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_af75b898","line":89,"in_reply_to":"1a4dcd0f_7758f432","updated":"2015-08-05 09:08:38.000000000","message":"Done","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":8797,"name":"Andrew Woodward","email":"xarses@gmail.com","username":"xarses"},"change_message_id":"48fe0b4ecf7c64cd8b0c6ec0797cf5d0a2efe059","unresolved":false,"context_lines":[{"line_number":98,"context_line":"set -ex"},{"line_number":99,"context_line":"wget --no-check-certificate ${rgw_keystone_url}/${rgw_keystone_version}/certificates/signing -O /tmp/signing"},{"line_number":100,"context_line":"openssl x509 -in /tmp/signing -pubkey | certutil -A -d ${nss_db_path} -n signing_cert -t \\\"P,P,P\\\""},{"line_number":101,"context_line":"chown ${user} -R ${nss_db_path}\","},{"line_number":102,"context_line":"    unless  \u003d\u003e \"/bin/true  # comment to satisfy puppet syntax requirements"},{"line_number":103,"context_line":"set -ex"},{"line_number":104,"context_line":"certutil -d ${nss_db_path} -L | grep ^signing_cert"}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_975b402e","line":101,"updated":"2015-08-05 00:15:16.000000000","message":"same","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"c62c8b67b66cdf6df0af50f1baaae68398a2dfae","unresolved":false,"context_lines":[{"line_number":98,"context_line":"set -ex"},{"line_number":99,"context_line":"wget --no-check-certificate ${rgw_keystone_url}/${rgw_keystone_version}/certificates/signing -O /tmp/signing"},{"line_number":100,"context_line":"openssl x509 -in /tmp/signing -pubkey | certutil -A -d ${nss_db_path} -n signing_cert -t \\\"P,P,P\\\""},{"line_number":101,"context_line":"chown ${user} -R ${nss_db_path}\","},{"line_number":102,"context_line":"    unless  \u003d\u003e \"/bin/true  # comment to satisfy puppet syntax requirements"},{"line_number":103,"context_line":"set -ex"},{"line_number":104,"context_line":"certutil -d ${nss_db_path} -L | grep ^signing_cert"}],"source_content_type":"text/x-puppet","patch_set":18,"id":"1a4dcd0f_6f6fb0c5","line":101,"in_reply_to":"1a4dcd0f_975b402e","updated":"2015-08-05 09:08:38.000000000","message":"Done","commit_id":"cde508695b45732facb68d389cbc8be825779387"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":48,"context_line":"#   Optional. Default is undef."},{"line_number":49,"context_line":"#"},{"line_number":50,"context_line":"# [*user*] User running the web frontend."},{"line_number":51,"context_line":"#   Optional. Default is \u0027www-data\u0027."},{"line_number":52,"context_line":"#"},{"line_number":53,"context_line":"define ceph::rgw::keystone ("},{"line_number":54,"context_line":"  $rgw_keystone_admin_token,"}],"source_content_type":"text/x-puppet","patch_set":19,"id":"1a4dcd0f_d160d403","line":51,"updated":"2015-08-05 14:24:47.000000000","message":"This is mostly a nitpick but the default is not www-data on a RHEL distro.","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":48,"context_line":"#   Optional. Default is undef."},{"line_number":49,"context_line":"#"},{"line_number":50,"context_line":"# [*user*] User running the web frontend."},{"line_number":51,"context_line":"#   Optional. Default is \u0027www-data\u0027."},{"line_number":52,"context_line":"#"},{"line_number":53,"context_line":"define ceph::rgw::keystone ("},{"line_number":54,"context_line":"  $rgw_keystone_admin_token,"}],"source_content_type":"text/x-puppet","patch_set":19,"id":"1a4dcd0f_e07e5818","line":51,"in_reply_to":"1a4dcd0f_d160d403","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}],"spec/acceptance/ceph_rgw_spec.rb":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            include ::apt"},{"line_number":50,"context_line":"            apt::source { \u0027cloudarchive-juno\u0027:"},{"line_number":51,"context_line":"              location          \u003d\u003e \u0027http://ubuntu-cloud.archive.canonical.com/ubuntu\u0027,"},{"line_number":52,"context_line":"              release           \u003d\u003e \u0027trusty-updates/juno\u0027,"},{"line_number":53,"context_line":"              repos             \u003d\u003e \u0027main\u0027,"},{"line_number":54,"context_line":"              include_src       \u003d\u003e false,"},{"line_number":55,"context_line":"              required_packages \u003d\u003e \u0027ubuntu-cloud-keyring\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_94b79a39","line":52,"updated":"2015-08-05 14:24:47.000000000","message":"We could probably test against kilo instead ?","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            include ::apt"},{"line_number":50,"context_line":"            apt::source { \u0027cloudarchive-juno\u0027:"},{"line_number":51,"context_line":"              location          \u003d\u003e \u0027http://ubuntu-cloud.archive.canonical.com/ubuntu\u0027,"},{"line_number":52,"context_line":"              release           \u003d\u003e \u0027trusty-updates/juno\u0027,"},{"line_number":53,"context_line":"              repos             \u003d\u003e \u0027main\u0027,"},{"line_number":54,"context_line":"              include_src       \u003d\u003e false,"},{"line_number":55,"context_line":"              required_packages \u003d\u003e \u0027ubuntu-cloud-keyring\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_0a7cbe1d","line":52,"in_reply_to":"1a4dcd0f_94b79a39","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            }"},{"line_number":65,"context_line":"            yumrepo { \u0027openstack-juno\u0027:"},{"line_number":66,"context_line":"              descr    \u003d\u003e \u0027OpenStack Juno Repository\u0027,"},{"line_number":67,"context_line":"              baseurl  \u003d\u003e \u0027http://repos.fedorapeople.org/repos/openstack/openstack-juno/epel-7/\u0027,"},{"line_number":68,"context_line":"              enabled  \u003d\u003e \u00271\u0027,"},{"line_number":69,"context_line":"              gpgcheck \u003d\u003e \u00271\u0027,"},{"line_number":70,"context_line":"              gpgkey   \u003d\u003e \u0027https://raw.githubusercontent.com/redhat-openstack/rdo-release/juno/RPM-GPG-KEY-RDO-Juno\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_d44d223f","line":67,"updated":"2015-08-05 14:24:47.000000000","message":"Same as above","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            }"},{"line_number":65,"context_line":"            yumrepo { \u0027openstack-juno\u0027:"},{"line_number":66,"context_line":"              descr    \u003d\u003e \u0027OpenStack Juno Repository\u0027,"},{"line_number":67,"context_line":"              baseurl  \u003d\u003e \u0027http://repos.fedorapeople.org/repos/openstack/openstack-juno/epel-7/\u0027,"},{"line_number":68,"context_line":"              enabled  \u003d\u003e \u00271\u0027,"},{"line_number":69,"context_line":"              gpgcheck \u003d\u003e \u00271\u0027,"},{"line_number":70,"context_line":"              gpgkey   \u003d\u003e \u0027https://raw.githubusercontent.com/redhat-openstack/rdo-release/juno/RPM-GPG-KEY-RDO-Juno\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_aa74aa35","line":67,"in_reply_to":"1a4dcd0f_d44d223f","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}],"spec/defines/ceph_rgw_keystone_spec.rb":[{"author":{"_account_id":8797,"name":"Andrew Woodward","email":"xarses@gmail.com","username":"xarses"},"change_message_id":"03efad4b6f52eee62a420db8e18f175a4e7f2d3b","unresolved":false,"context_lines":[{"line_number":252,"context_line":"         \u0027user\u0027    \u003d\u003e \u0027apache\u0027,"},{"line_number":253,"context_line":"      ) }"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    end  end"},{"line_number":256,"context_line":"end"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"# Local Variables:"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_99890678","line":255,"updated":"2015-08-05 17:16:15.000000000","message":"nit: if you touch this any way, can you separate these and indent","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}],"spec/spec_helper_acceptance.rb":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":45,"context_line":"      on host, puppet(\u0027module install puppetlabs/apt --version \"\u003e\u003d1.4.0 \u003c2.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":46,"context_line":"      on host, puppet(\u0027module install puppetlabs/concat --version \"\u003e\u003d1.2.1 \u003c2.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":47,"context_line":"      on host, puppet(\u0027module install puppetlabs/apache --version \"\u003e\u003d1.4.1 \u003c2.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":48,"context_line":"      on host, puppet(\u0027module install stackforge/keystone --version \"\u003e\u003d5.1.0 \u003c6.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":49,"context_line":"      puppet_module_install(:source \u003d\u003e proj_root, :module_name \u003d\u003e \u0027ceph\u0027)"},{"line_number":50,"context_line":"      # Flush the firewall"},{"line_number":51,"context_line":"      flushfw \u003d \u003c\u003c-EOS"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_df43fb40","line":48,"updated":"2015-08-05 14:24:47.000000000","message":"If we test against Kilo, we might want to bump this","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":45,"context_line":"      on host, puppet(\u0027module install puppetlabs/apt --version \"\u003e\u003d1.4.0 \u003c2.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":46,"context_line":"      on host, puppet(\u0027module install puppetlabs/concat --version \"\u003e\u003d1.2.1 \u003c2.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":47,"context_line":"      on host, puppet(\u0027module install puppetlabs/apache --version \"\u003e\u003d1.4.1 \u003c2.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":48,"context_line":"      on host, puppet(\u0027module install stackforge/keystone --version \"\u003e\u003d5.1.0 \u003c6.0.0\"\u0027), { :acceptable_exit_codes \u003d\u003e [0,1] }"},{"line_number":49,"context_line":"      puppet_module_install(:source \u003d\u003e proj_root, :module_name \u003d\u003e \u0027ceph\u0027)"},{"line_number":50,"context_line":"      # Flush the firewall"},{"line_number":51,"context_line":"      flushfw \u003d \u003c\u003c-EOS"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_ea7e3218","line":48,"in_reply_to":"1a4dcd0f_df43fb40","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}],"spec/spec_helper_system.rb":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            :node \u003d\u003e vm)"},{"line_number":55,"context_line":"      shell(:command \u003d\u003e \u0027puppet module install --version \"\u003e\u003d1.4.1 \u003c2.0.0\" puppetlabs/apache\u0027,"},{"line_number":56,"context_line":"            :node \u003d\u003e vm)"},{"line_number":57,"context_line":"      shell(:command \u003d\u003e \u0027puppet module install --version \"\u003e\u003d5.1.0 \u003c6.0.0\" stackforge/keystone\u0027,"},{"line_number":58,"context_line":"            :node \u003d\u003e vm)"},{"line_number":59,"context_line":"      rcp(:sp \u003d\u003e File.join(proj_root, \u0027spec/fixtures/hieradata/hiera.yaml\u0027),"},{"line_number":60,"context_line":"          :dp \u003d\u003e \u0027/etc/puppet/hiera.yaml\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_5f760ba5","line":57,"updated":"2015-08-05 14:24:47.000000000","message":"If we test against kilo, we might want to bump this","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":54,"context_line":"            :node \u003d\u003e vm)"},{"line_number":55,"context_line":"      shell(:command \u003d\u003e \u0027puppet module install --version \"\u003e\u003d1.4.1 \u003c2.0.0\" puppetlabs/apache\u0027,"},{"line_number":56,"context_line":"            :node \u003d\u003e vm)"},{"line_number":57,"context_line":"      shell(:command \u003d\u003e \u0027puppet module install --version \"\u003e\u003d5.1.0 \u003c6.0.0\" stackforge/keystone\u0027,"},{"line_number":58,"context_line":"            :node \u003d\u003e vm)"},{"line_number":59,"context_line":"      rcp(:sp \u003d\u003e File.join(proj_root, \u0027spec/fixtures/hieradata/hiera.yaml\u0027),"},{"line_number":60,"context_line":"          :dp \u003d\u003e \u0027/etc/puppet/hiera.yaml\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_ca9a566e","line":57,"in_reply_to":"1a4dcd0f_5f760ba5","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}],"spec/system/ceph_rgw_apache_spec.rb":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"describe \u0027ceph::rgw::apache\u0027 do"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"  releases \u003d ENV[\u0027RELEASES\u0027] ? ENV[\u0027RELEASES\u0027].split : [ \u0027hammer\u0027 ]"},{"line_number":24,"context_line":"  fsid \u003d \u0027a4807c9a-e76f-4666-a297-6d6cbc922e3a\u0027"},{"line_number":25,"context_line":"  mon_key \u003d\u0027AQCztJdSyNb0NBAASA2yPZPuwXeIQnDJ9O8gVw\u003d\u003d\u0027"},{"line_number":26,"context_line":"  admin_key \u003d \u0027AQA0TVRTsP/aHxAAFBvntu1dSEJHxtJeFFrRsg\u003d\u003d\u0027"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_9f8ed377","line":23,"updated":"2015-08-05 14:24:47.000000000","message":"Is there a reason we only test against hammer ?","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"describe \u0027ceph::rgw::apache\u0027 do"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"  releases \u003d ENV[\u0027RELEASES\u0027] ? ENV[\u0027RELEASES\u0027].split : [ \u0027hammer\u0027 ]"},{"line_number":24,"context_line":"  fsid \u003d \u0027a4807c9a-e76f-4666-a297-6d6cbc922e3a\u0027"},{"line_number":25,"context_line":"  mon_key \u003d\u0027AQCztJdSyNb0NBAASA2yPZPuwXeIQnDJ9O8gVw\u003d\u003d\u0027"},{"line_number":26,"context_line":"  admin_key \u003d \u0027AQA0TVRTsP/aHxAAFBvntu1dSEJHxtJeFFrRsg\u003d\u003d\u0027"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_c8601ed2","line":23,"in_reply_to":"1a4dcd0f_9f8ed377","updated":"2015-08-05 16:20:44.000000000","message":"I think testing against firefly and hammer makes sense, as those are the currently supported releases.\nWill add firefly.","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}],"spec/system/ceph_rgw_keystone_spec.rb":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":50,"context_line":"              include ::apt"},{"line_number":51,"context_line":"              apt::source { \u0027cloudarchive-juno\u0027:"},{"line_number":52,"context_line":"                location          \u003d\u003e \u0027http://ubuntu-cloud.archive.canonical.com/ubuntu\u0027,"},{"line_number":53,"context_line":"                release           \u003d\u003e \u0027trusty-updates/juno\u0027,"},{"line_number":54,"context_line":"                repos             \u003d\u003e \u0027main\u0027,"},{"line_number":55,"context_line":"                include_src       \u003d\u003e false,"},{"line_number":56,"context_line":"                required_packages \u003d\u003e \u0027ubuntu-cloud-keyring\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_ffdcb774","line":53,"updated":"2015-08-05 14:24:47.000000000","message":"Did a comment about kilo elsewhere, same thing here.","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":50,"context_line":"              include ::apt"},{"line_number":51,"context_line":"              apt::source { \u0027cloudarchive-juno\u0027:"},{"line_number":52,"context_line":"                location          \u003d\u003e \u0027http://ubuntu-cloud.archive.canonical.com/ubuntu\u0027,"},{"line_number":53,"context_line":"                release           \u003d\u003e \u0027trusty-updates/juno\u0027,"},{"line_number":54,"context_line":"                repos             \u003d\u003e \u0027main\u0027,"},{"line_number":55,"context_line":"                include_src       \u003d\u003e false,"},{"line_number":56,"context_line":"                required_packages \u003d\u003e \u0027ubuntu-cloud-keyring\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_aaefcae7","line":53,"in_reply_to":"1a4dcd0f_ffdcb774","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6634cfed9b98872619d03940890d1b7d9b53916","unresolved":false,"context_lines":[{"line_number":65,"context_line":"              }"},{"line_number":66,"context_line":"              yumrepo { \u0027openstack-juno\u0027:"},{"line_number":67,"context_line":"                descr    \u003d\u003e \u0027OpenStack Juno Repository\u0027,"},{"line_number":68,"context_line":"                baseurl  \u003d\u003e \u0027http://repos.fedorapeople.org/repos/openstack/openstack-juno/epel-7/\u0027,"},{"line_number":69,"context_line":"                enabled  \u003d\u003e \u00271\u0027,"},{"line_number":70,"context_line":"                gpgcheck \u003d\u003e \u00271\u0027,"},{"line_number":71,"context_line":"                gpgkey   \u003d\u003e \u0027https://raw.githubusercontent.com/redhat-openstack/rdo-release/juno/RPM-GPG-KEY-RDO-Juno\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_5fd28b5d","line":68,"updated":"2015-08-05 14:24:47.000000000","message":"See above","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"},{"author":{"_account_id":11479,"name":"David Gurtner","email":"aldavud@crimson.ch","username":"aldavud"},"change_message_id":"0ff690d059c065b6ca83b0112cc53e9f7056fd8e","unresolved":false,"context_lines":[{"line_number":65,"context_line":"              }"},{"line_number":66,"context_line":"              yumrepo { \u0027openstack-juno\u0027:"},{"line_number":67,"context_line":"                descr    \u003d\u003e \u0027OpenStack Juno Repository\u0027,"},{"line_number":68,"context_line":"                baseurl  \u003d\u003e \u0027http://repos.fedorapeople.org/repos/openstack/openstack-juno/epel-7/\u0027,"},{"line_number":69,"context_line":"                enabled  \u003d\u003e \u00271\u0027,"},{"line_number":70,"context_line":"                gpgcheck \u003d\u003e \u00271\u0027,"},{"line_number":71,"context_line":"                gpgkey   \u003d\u003e \u0027https://raw.githubusercontent.com/redhat-openstack/rdo-release/juno/RPM-GPG-KEY-RDO-Juno\u0027,"}],"source_content_type":"text/x-ruby","patch_set":19,"id":"1a4dcd0f_8ac1ae56","line":68,"in_reply_to":"1a4dcd0f_5fd28b5d","updated":"2015-08-05 16:20:44.000000000","message":"Done","commit_id":"2eb9458fdac9ac03a3474d104450c8634ab285ae"}]}