)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"af1fb26bc5ae2e53794ebfc8d2741519cc3003df","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* create calico.yaml using"},{"line_number":10,"context_line":"  `kustomize build ./manifests/function/cni/calico/v3.15/`"},{"line_number":11,"context_line":"* This will be used to apply calico part of the cloud-init steps"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Iac17111ae4115732bd0618ab57d9fd5c7406bbc3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9f560f44_a91e0d09","line":11,"updated":"2020-08-21 21:09:05.000000000","message":"Why do we need to apply calico as part of cloud-init steps - can we not just apply it as part of a airshipctl phase bundle?","commit_id":"21bf03666a955a366e4423ff8f9d8ef931e6b2d4"}],"tools/deployment/21_systemwide_executable.sh":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"af1fb26bc5ae2e53794ebfc8d2741519cc3003df","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"if [[ -f \"/usr/local/bin/kustomize\" ]]; then"},{"line_number":44,"context_line":"  echo \"Check if calico.yaml need to be re-generated\""},{"line_number":45,"context_line":"  /usr/local/bin/kustomize build ./manifests/function/cni/calico/v3.15/ \u003e ${TMP_DIR}/calico.yaml"},{"line_number":46,"context_line":"  diff -q ./manifests/function/cni/calico/v3.15/calico.yaml ${TMP_DIR}/calico.yaml"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"  if [ $? -ne 0 ]"}],"source_content_type":"text/x-sh","patch_set":7,"id":"9f560f44_89a7c9bc","line":45,"updated":"2020-08-21 21:09:05.000000000","message":"I\u0027m missing it, where is the kustomization.yaml for calico defined?  I didn\u0027t see it in the change but it may be defined elsewhere.","commit_id":"21bf03666a955a366e4423ff8f9d8ef931e6b2d4"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"af1fb26bc5ae2e53794ebfc8d2741519cc3003df","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"echo \"Install airshipctl as kustomize plugins\""},{"line_number":41,"context_line":"AIRSHIPCTL\u003d\"/usr/local/bin/airshipctl\" ./tools/document/build_kustomize_plugin.sh"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"if [[ -f \"/usr/local/bin/kustomize\" ]]; then"},{"line_number":44,"context_line":"  echo \"Check if calico.yaml need to be re-generated\""},{"line_number":45,"context_line":"  /usr/local/bin/kustomize build ./manifests/function/cni/calico/v3.15/ \u003e ${TMP_DIR}/calico.yaml"},{"line_number":46,"context_line":"  diff -q ./manifests/function/cni/calico/v3.15/calico.yaml ${TMP_DIR}/calico.yaml"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"  if [ $? -ne 0 ]"},{"line_number":49,"context_line":"  then"},{"line_number":50,"context_line":"    echo \"calico manifest is modified, please re-generate calico.yaml\""},{"line_number":51,"context_line":"    echo -e \"kustomize build ./manifests/function/cni/calico/v3.15/ \u003e ./manifests/function/cni/calico/v3.15/calico.yaml\""},{"line_number":52,"context_line":"    exit 1"},{"line_number":53,"context_line":"  fi"},{"line_number":54,"context_line":" fi"}],"source_content_type":"text/x-sh","patch_set":7,"id":"9f560f44_a9f56dcd","line":54,"range":{"start_line":42,"start_character":0,"end_line":54,"end_character":3},"updated":"2020-08-21 21:09:05.000000000","message":"Why are we doing these steps?  Can\u0027t we just perform all kustomization as part of `airshipctl phase apply`?  I may not be understanding why we\u0027re doing the diff.","commit_id":"21bf03666a955a366e4423ff8f9d8ef931e6b2d4"}]}