)]}'
{"deployment/Puppetfile":[{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"6c3bf065c8dd9fb0f9f0fed7af2cf75fba7945e3","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# The location to pull forge modules from. This could be a local"},{"line_number":7,"context_line":"# repository, see https://tickets.puppetlabs.com/browse/FORGE-27"},{"line_number":8,"context_line":"# and https://github.com/drrb/puppet-library"},{"line_number":9,"context_line":"forge \"https://forgeapi.puppetlabs.com\""}],"source_content_type":"application/octet-stream","patch_set":20,"id":"3a50d1a3_4308b891","line":9,"updated":"2015-07-21 00:41:33.000000000","message":"Is there a way to override this value from CLI or would CI have to replace this whole file to use a local repository?","commit_id":"3ea994c602682990545cb6e2dbc4806caf8e6b99"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"274723d0f2611a3d0e640016989c01044a3a23be","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# The location to pull forge modules from. This could be a local"},{"line_number":7,"context_line":"# repository, see https://tickets.puppetlabs.com/browse/FORGE-27"},{"line_number":8,"context_line":"# and https://github.com/drrb/puppet-library"},{"line_number":9,"context_line":"forge \"https://forgeapi.puppetlabs.com\""}],"source_content_type":"application/octet-stream","patch_set":20,"id":"3a50d1a3_ce3dcfbc","line":9,"in_reply_to":"3a50d1a3_4308b891","updated":"2015-07-21 00:57:53.000000000","message":"Depends how we use this. If we go with git reps this is not used. It is only used when pulling modules via the forge api but could provide our own endpoint instead.","commit_id":"3ea994c602682990545cb6e2dbc4806caf8e6b99"}],"deployment/update_modules.sh":[{"author":{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},"change_message_id":"f5c77f3663926728590af018aa299eebf18c9ce8","unresolved":false,"context_lines":[{"line_number":42,"context_line":"PUPPET_GEM_VERSION\u003d${PUPPET_GEM_VERSION:-\u00273.4.3\u0027}"},{"line_number":43,"context_line":"LIBRARIAN_TMP\u003d${LIBRARIAN_TMP:-\u0027/var/tmp/.librarian_tmp\u0027}"},{"line_number":44,"context_line":"BUNDLE_DIR\u003d${BUNDLE_DIR:-\u0027/var/tmp/.bundle_home\u0027}"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# We need to be in the deployment directory to run librarian-puppet"},{"line_number":47,"context_line":"cd $DEPLOYMENT_DIR"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a50d1a3_afc8de55","line":45,"updated":"2015-07-23 13:36:38.000000000","message":"We also need export GEM_HOME variable. Otherwise it will require root privileges and will cause result is like this http://paste.openstack.org/show/403991/\n\n export GEM_HOME\u003d${GEM_HOME:-\u0027/var/tmp/.gem_home\u0027}","commit_id":"1eb186e2e9ef4c9ab78eba710594e3c23c0a5849"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"a27d6ebf5e0d0648627d5ec3374382e94b2b2c0b","unresolved":false,"context_lines":[{"line_number":42,"context_line":"PUPPET_GEM_VERSION\u003d${PUPPET_GEM_VERSION:-\u00273.4.3\u0027}"},{"line_number":43,"context_line":"LIBRARIAN_TMP\u003d${LIBRARIAN_TMP:-\u0027/var/tmp/.librarian_tmp\u0027}"},{"line_number":44,"context_line":"BUNDLE_DIR\u003d${BUNDLE_DIR:-\u0027/var/tmp/.bundle_home\u0027}"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# We need to be in the deployment directory to run librarian-puppet"},{"line_number":47,"context_line":"cd $DEPLOYMENT_DIR"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-sh","patch_set":28,"id":"3a50d1a3_688acc65","line":45,"in_reply_to":"3a50d1a3_afc8de55","updated":"2015-07-23 13:49:50.000000000","message":"This was actually an older file that got pushed, i\u0027ve added this to the new formatted file where we don\u0027t require bundler anymore but bundler is used for the tests","commit_id":"1eb186e2e9ef4c9ab78eba710594e3c23c0a5849"},{"author":{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},"change_message_id":"d0a45e345aa84d29c84cb735a93ccadaa43fc983","unresolved":false,"context_lines":[{"line_number":103,"context_line":"PUPPET_GEM_VERSION\u003d${PUPPET_GEM_VERSION:-\u00273.4.3\u0027}"},{"line_number":104,"context_line":"LIBRARIAN_TMP\u003d${LIBRARIAN_TMP:-\u0027/var/tmp/.librarian_tmp\u0027}"},{"line_number":105,"context_line":"BUNDLE_DIR\u003d${BUNDLE_DIR:-\u0027/var/tmp/.bundle_home\u0027}"},{"line_number":106,"context_line":"GEM_HOME\u003d${GEM_HOME:-\u0027/var/tmp/.gem_home\u0027}"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"# We need to be in the deployment directory to run librarian-puppet"},{"line_number":109,"context_line":"cd $DEPLOYMENT_DIR"}],"source_content_type":"text/x-sh","patch_set":30,"id":"3a50d1a3_b43829d4","line":106,"updated":"2015-07-23 14:43:03.000000000","message":"It still does not work and bundler tries to install in system path and requests sudo password. We need to export this variable or add \"--path $GEM_HOME\" to bundle commands.","commit_id":"ff67b942ddc107bf0bb0a0a715023a4390fd333b"},{"author":{"_account_id":11827,"name":"Stanislaw Bogatkin","email":"regs+ubuntu.com@sbog.ru","username":"sbogatkin"},"change_message_id":"eff28ecf42ca5d49e8a694a6c23ad529b6d99bb3","unresolved":false,"context_lines":[{"line_number":21,"context_line":"#"},{"line_number":22,"context_line":"#  This script uses librarian-puppet-simple to populate the puppet folder with"},{"line_number":23,"context_line":"#  upstream puppet modules.  By default, it assumes librarian-puppet-simple is"},{"line_number":24,"context_line":"#  alreadyavailable to the environment or it will fail. You can provide command"},{"line_number":25,"context_line":"#  line options to have the script use bundler to install librarian-puppet-simple"},{"line_number":26,"context_line":"#  if neccessary."},{"line_number":27,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":34,"id":"3a50d1a3_7e6c669c","line":24,"updated":"2015-07-25 17:04:01.000000000","message":"missed space","commit_id":"c713b6c76ecb348108d77f4c9f6e8728d906b07e"}],"specs/fuel-library7.0.spec":[{"author":{"_account_id":8787,"name":"Dmitry Borodaenko","email":"angdraug@gmail.com","username":"angdraug"},"change_message_id":"aaa191fee5bf7d6bc2dd7967783d509a378f3cae","unresolved":false,"context_lines":[{"line_number":33,"context_line":"%build"},{"line_number":34,"context_line":"if test -x %{_builddir}/%{name}-%{version}/deployment/update_modules.sh; then"},{"line_number":35,"context_line":"  # need bundler installed to run update_modules.sh"},{"line_number":36,"context_line":"  gem install bundler --no-ri --no-rdoc --user-install"},{"line_number":37,"context_line":"  PATH\u003d\"$(ruby -rubygems -e \u0027puts Gem.user_dir\u0027)/bin:$PATH\""},{"line_number":38,"context_line":"  %{_builddir}/%{name}-%{version}/deployment/update_modules.sh"},{"line_number":39,"context_line":"fi"}],"source_content_type":"text/x-rpm-spec","patch_set":16,"id":"3a50d1a3_d12ca7ad","line":36,"updated":"2015-07-20 15:59:09.000000000","message":"This introduces a build dependency on Internet access to external third-party repositories. All Fuel dependencies and build dependencies must be satisfied by packages that can be installed from a local mirror. And no, creating a local mirror of the whole rubygems and pypi repositories is not a good way to adress that requirement.\n\nWhy not just take librarian-puppet from Debian, there\u0027s librarian-puppet_1.0.3-1_all.deb (more recent than the version in trusty)?","commit_id":"4911ae1dca4086b3390211f5062ff698f9322ef0"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"373f3819422c87d0638063daf3e8b94eee3329f9","unresolved":false,"context_lines":[{"line_number":33,"context_line":"%build"},{"line_number":34,"context_line":"if test -x %{_builddir}/%{name}-%{version}/deployment/update_modules.sh; then"},{"line_number":35,"context_line":"  # need bundler installed to run update_modules.sh"},{"line_number":36,"context_line":"  gem install bundler --no-ri --no-rdoc --user-install"},{"line_number":37,"context_line":"  PATH\u003d\"$(ruby -rubygems -e \u0027puts Gem.user_dir\u0027)/bin:$PATH\""},{"line_number":38,"context_line":"  %{_builddir}/%{name}-%{version}/deployment/update_modules.sh"},{"line_number":39,"context_line":"fi"}],"source_content_type":"text/x-rpm-spec","patch_set":16,"id":"3a50d1a3_0c3f58b5","line":36,"in_reply_to":"3a50d1a3_ac1d0468","updated":"2015-07-20 16:02:01.000000000","message":"Or at least the CI version uses centos.","commit_id":"4911ae1dca4086b3390211f5062ff698f9322ef0"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"9ee8bd633332a3f98f8d230a8a3764c049697b6e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"%build"},{"line_number":34,"context_line":"if test -x %{_builddir}/%{name}-%{version}/deployment/update_modules.sh; then"},{"line_number":35,"context_line":"  # need bundler installed to run update_modules.sh"},{"line_number":36,"context_line":"  gem install bundler --no-ri --no-rdoc --user-install"},{"line_number":37,"context_line":"  PATH\u003d\"$(ruby -rubygems -e \u0027puts Gem.user_dir\u0027)/bin:$PATH\""},{"line_number":38,"context_line":"  %{_builddir}/%{name}-%{version}/deployment/update_modules.sh"},{"line_number":39,"context_line":"fi"}],"source_content_type":"text/x-rpm-spec","patch_set":16,"id":"3a50d1a3_ac1d0468","line":36,"in_reply_to":"3a50d1a3_d12ca7ad","updated":"2015-07-20 16:00:54.000000000","message":"Our build system uses centos.","commit_id":"4911ae1dca4086b3390211f5062ff698f9322ef0"}],"utils/jenkins/fuel_noop_tests.rb":[{"author":{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},"change_message_id":"f5c77f3663926728590af018aa299eebf18c9ce8","unresolved":false,"context_lines":[{"line_number":467,"context_line":""},{"line_number":468,"context_line":"    # run librarian-puppet update to prepare the library"},{"line_number":469,"context_line":"    if options[:update_librarian_puppet]"},{"line_number":470,"context_line":"        prepare_library"},{"line_number":471,"context_line":"    end"},{"line_number":472,"context_line":""},{"line_number":473,"context_line":"    if options[:missing_specs]"}],"source_content_type":"text/x-ruby","patch_set":28,"id":"3a50d1a3_6f4896b3","line":470,"updated":"2015-07-23 13:36:38.000000000","message":"I suggest to not do this here, since it will cause bundle execs on every fuel_noop_tests.sh run, like:\nhttp://paste.openstack.org/show/403991/\nIt would be better to call update only before real tests run","commit_id":"1eb186e2e9ef4c9ab78eba710594e3c23c0a5849"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"a27d6ebf5e0d0648627d5ec3374382e94b2b2c0b","unresolved":false,"context_lines":[{"line_number":467,"context_line":""},{"line_number":468,"context_line":"    # run librarian-puppet update to prepare the library"},{"line_number":469,"context_line":"    if options[:update_librarian_puppet]"},{"line_number":470,"context_line":"        prepare_library"},{"line_number":471,"context_line":"    end"},{"line_number":472,"context_line":""},{"line_number":473,"context_line":"    if options[:missing_specs]"}],"source_content_type":"text/x-ruby","patch_set":28,"id":"3a50d1a3_e8f49cf1","line":470,"in_reply_to":"3a50d1a3_6f4896b3","updated":"2015-07-23 13:49:50.000000000","message":"Done","commit_id":"1eb186e2e9ef4c9ab78eba710594e3c23c0a5849"},{"author":{"_account_id":9387,"name":"Aleksandr Didenko","email":"adidenko@mirantis.com","username":"AD"},"change_message_id":"f5c77f3663926728590af018aa299eebf18c9ce8","unresolved":false,"context_lines":[{"line_number":495,"context_line":"    if options[:purge_globals]"},{"line_number":496,"context_line":"      purge_globals"},{"line_number":497,"context_line":"      exit 0"},{"line_number":498,"context_line":"    end"},{"line_number":499,"context_line":""},{"line_number":500,"context_line":"    success, result \u003d for_every_astute_yaml do"},{"line_number":501,"context_line":"      if options[:run_individually]"}],"source_content_type":"text/x-ruby","patch_set":28,"id":"3a50d1a3_25d62d4f","line":498,"updated":"2015-07-23 13:36:38.000000000","message":"I\u0027d put it here, right before rspec runs:\n\n    if options[:update_librarian_puppet]\n      prepare_library\n    end","commit_id":"1eb186e2e9ef4c9ab78eba710594e3c23c0a5849"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"a27d6ebf5e0d0648627d5ec3374382e94b2b2c0b","unresolved":false,"context_lines":[{"line_number":495,"context_line":"    if options[:purge_globals]"},{"line_number":496,"context_line":"      purge_globals"},{"line_number":497,"context_line":"      exit 0"},{"line_number":498,"context_line":"    end"},{"line_number":499,"context_line":""},{"line_number":500,"context_line":"    success, result \u003d for_every_astute_yaml do"},{"line_number":501,"context_line":"      if options[:run_individually]"}],"source_content_type":"text/x-ruby","patch_set":28,"id":"3a50d1a3_88e0782a","line":498,"in_reply_to":"3a50d1a3_25d62d4f","updated":"2015-07-23 13:49:50.000000000","message":"Done","commit_id":"1eb186e2e9ef4c9ab78eba710594e3c23c0a5849"}]}