)]}'
{"nodepool/elements/puppet/element-deps":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"c06048fa376449cfc5940104f935b04137fe0a72","unresolved":false,"context_lines":[{"line_number":1,"context_line":"cache-url"},{"line_number":2,"context_line":"openstack-repos"},{"line_number":3,"context_line":"package-installs"},{"line_number":4,"context_line":"source-repositories"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"5a5ae5dd_c95f150d","line":3,"updated":"2016-02-03 13:32:01.000000000","message":"We need this to curl down the puppet package correct?","commit_id":"0c50fd334bfc1d3175328759afabd4e6d51842ff"},{"author":{"_account_id":13505,"name":"Igor Belikov","email":"mail@igorbelikov.com","username":"ibelikov"},"change_message_id":"d5d7ad4cc77337f24002ee2f3754c2f46f4eeb93","unresolved":false,"context_lines":[{"line_number":1,"context_line":"cache-url"},{"line_number":2,"context_line":"openstack-repos"},{"line_number":3,"context_line":"package-installs"},{"line_number":4,"context_line":"source-repositories"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"5a5ae5dd_5d5f5f72","line":3,"in_reply_to":"5a5ae5dd_c95f150d","updated":"2016-02-03 14:52:33.000000000","message":"to curl down get-pip.py actually, otherwise it fails here https://github.com/openstack-infra/system-config/blob/d39b05bd97a75dd89cb7af2492789130514d4a0d/install_puppet.sh#L201-L219","commit_id":"0c50fd334bfc1d3175328759afabd4e6d51842ff"}],"nodepool/elements/puppet/package-installs.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"4db4c280472149203c881541ea874e8b3a49bb94","unresolved":false,"context_lines":[{"line_number":1,"context_line":"curl:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5a5ae5dd_cf77fa00","line":1,"updated":"2016-02-03 23:15:26.000000000","message":"we can clearly argue about what constitutes part of the core system in a \"minimal\" image ... but so far it seems that ubuntu-minimal, fedora-minimal and centos-minimal are all including this as part of their base image (because they all work with this element unmodified)\n\nSo I think that weighs heavily for the argument that \"curl\" should just be installed as part of the debian-minimal default build\n\njust when it\u0027s here, it becomes a maintenance point for everyone (different names for gentoo, etc etc).","commit_id":"0c50fd334bfc1d3175328759afabd4e6d51842ff"},{"author":{"_account_id":13505,"name":"Igor Belikov","email":"mail@igorbelikov.com","username":"ibelikov"},"change_message_id":"387eff2fb0b26d9be7e9de305ffbce8393021a9d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"curl:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5a5ae5dd_d6478d1b","line":1,"in_reply_to":"5a5ae5dd_cf77fa00","updated":"2016-02-04 04:36:06.000000000","message":"ok, this sounds reasonable","commit_id":"0c50fd334bfc1d3175328759afabd4e6d51842ff"}],"nodepool/nodepool.yaml":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"35310608032eba79122baa9c734887e9df0a9814","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    ready-script: configure_mirror.sh"},{"line_number":34,"context_line":"    min-ready: 1"},{"line_number":35,"context_line":"    providers:"},{"line_number":36,"context_line":"      - name: hpcloud-b1"},{"line_number":37,"context_line":"      - name: hpcloud-b2"},{"line_number":38,"context_line":"      - name: hpcloud-b3"},{"line_number":39,"context_line":"      - name: hpcloud-b4"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a5de9d1_a00bcd2a","line":36,"updated":"2016-02-02 17:37:20.000000000","message":"hpcloud is no longer needed. It has been retired.","commit_id":"1616baa7771a28363e68408b82d78e5f7adb3e83"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"35310608032eba79122baa9c734887e9df0a9814","unresolved":false,"context_lines":[{"line_number":558,"context_line":"  - name: jenkins07"},{"line_number":559,"context_line":""},{"line_number":560,"context_line":"diskimages:"},{"line_number":561,"context_line":"  - name: debian-jessie"},{"line_number":562,"context_line":"    elements:"},{"line_number":563,"context_line":"      - debian"},{"line_number":564,"context_line":"      - vm"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a5de9d1_2a081b28","line":561,"updated":"2016-02-02 17:37:20.000000000","message":"Nice! Do you mind posting the outbound of diskimage-builder online, will help with the review.","commit_id":"1616baa7771a28363e68408b82d78e5f7adb3e83"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"35310608032eba79122baa9c734887e9df0a9814","unresolved":false,"context_lines":[{"line_number":560,"context_line":"diskimages:"},{"line_number":561,"context_line":"  - name: debian-jessie"},{"line_number":562,"context_line":"    elements:"},{"line_number":563,"context_line":"      - debian"},{"line_number":564,"context_line":"      - vm"},{"line_number":565,"context_line":"      - openstack-repos"},{"line_number":566,"context_line":"      - nodepool-base"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7a5de9d1_6a25b3ac","line":563,"updated":"2016-02-02 17:37:20.000000000","message":"you\u0027ll want debian-minimal here.","commit_id":"1616baa7771a28363e68408b82d78e5f7adb3e83"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d51716112b51e87d9852446bab070d237289de81","unresolved":false,"context_lines":[{"line_number":458,"context_line":"diskimages:"},{"line_number":459,"context_line":"  - name: debian-jessie"},{"line_number":460,"context_line":"    elements:"},{"line_number":461,"context_line":"      - debian-packageinstall"},{"line_number":462,"context_line":"      - vm"},{"line_number":463,"context_line":"      - simple-init"},{"line_number":464,"context_line":"      - openstack-repos"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"1af94dfe_42dda241","line":461,"updated":"2016-03-15 17:01:31.000000000","message":"I\u0027m just jumping in here, sorry if I missed something, but can\u0027t you just go back to using debian-minimal here until the blocking dependency is commited in diskimage-builder?","commit_id":"a9824522ccd07cc792954d0d5327deff8144ebcb"}],"nodepool/scripts/configure_mirror.sh":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"c06048fa376449cfc5940104f935b04137fe0a72","unresolved":false,"context_lines":[{"line_number":83,"context_line":"EOF"},{"line_number":84,"context_line":"elif [ \"$LSBDISTID\" \u003d\u003d \"Debian\" ] ; then"},{"line_number":85,"context_line":"sudo dd of\u003d/etc/apt/sources.list \u003c\u003cEOF"},{"line_number":86,"context_line":"deb http://httpredir.debian.org/debian $LSBDISTCODENAME main"},{"line_number":87,"context_line":"deb-src http://httpredir.debian.org/debian $LSBDISTCODENAME main"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"deb http://httpredir.debian.org/debian $LSBDISTCODENAME-updates main"}],"source_content_type":"text/x-sh","patch_set":5,"id":"5a5ae5dd_e94bf129","line":86,"updated":"2016-02-03 13:32:01.000000000","message":"We might want to pin to a us mirror, if we start having repo timeouts.","commit_id":"0c50fd334bfc1d3175328759afabd4e6d51842ff"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"d87a3c80aaf70debf789b3c1ffd1f71b6a5378ff","unresolved":false,"context_lines":[{"line_number":101,"context_line":"deb http://httpredir.debian.org/debian $LSBDISTCODENAME-updates main"},{"line_number":102,"context_line":"deb-src http://httpredir.debian.org/debian $LSBDISTCODENAME-updates main"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"deb http://security.debian.org/ $LSBDISTCODENAME/updates main contrib non-free"},{"line_number":105,"context_line":"deb-src http://security.debian.org/ $LSBDISTCODENAME/updates main contrib non-free"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"deb http://httpredir.debian.org/debian $LSBDISTCODENAME-backports main"}],"source_content_type":"text/x-sh","patch_set":6,"id":"5a5ae5dd_4d1e0871","line":104,"updated":"2016-02-08 15:33:19.000000000","message":"do we really need contrib and non-free? non-free should be unneeded, let\u0027s remove it. Also, you only add main everywhere else, so why add contrib here as update?","commit_id":"7bc66d3fa492fa6e770ddcc565fb776587e0bead"},{"author":{"_account_id":13505,"name":"Igor Belikov","email":"mail@igorbelikov.com","username":"ibelikov"},"change_message_id":"8861316598cc65efcd2417ba934e03568ce2e299","unresolved":false,"context_lines":[{"line_number":101,"context_line":"deb http://httpredir.debian.org/debian $LSBDISTCODENAME-updates main"},{"line_number":102,"context_line":"deb-src http://httpredir.debian.org/debian $LSBDISTCODENAME-updates main"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"deb http://security.debian.org/ $LSBDISTCODENAME/updates main contrib non-free"},{"line_number":105,"context_line":"deb-src http://security.debian.org/ $LSBDISTCODENAME/updates main contrib non-free"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"deb http://httpredir.debian.org/debian $LSBDISTCODENAME-backports main"}],"source_content_type":"text/x-sh","patch_set":6,"id":"5a5ae5dd_c0f71fd1","line":104,"in_reply_to":"5a5ae5dd_4d1e0871","updated":"2016-02-08 15:40:33.000000000","message":"Thanks, it\u0027s not needed indeed, fixed.","commit_id":"7bc66d3fa492fa6e770ddcc565fb776587e0bead"}]}