airship/airshipctl
Code Issues Proposed changes

Fix describe-kubernetes-objects role

Browse Source 
According to [0] kubectl get all -A does not include all types (e.g.
configmaps) and is going to be deprecated. Instead of getting all
objects with one kubectl request, doing the same for each object type
separately.

[0] https://github.com/kubernetes/kubectl/issues/151

Change-Id: I624e785b95e0c5697a567251b42855545d771179
Signed-off-by: Andrii Ostapenko <andrii.ostapenko@att.com>
This commit is contained in:
Andrii Ostapenko 2020-10-24 13:15:06 -05:00
parent 81efbceea6
commit cd1ca0b787
No known key found for this signature in database
GPG Key ID: F3E83668DBB223B3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
roles/describe-kubernetes-objects/tasks/save-context-objects.yaml
View File

@ -67,9 +67,9 @@
export CONTEXT={{ kctl_context | default("dummy_cluster") }}
kubectl config use-context ${CONTEXT}
export OBJECT_TYPE="{{ collect_namespaced_objects | join(',') }}"
export ALL_OBJECTS_YAML_FILE=$(tempfile)
kubectl get all -A -oyaml > ${ALL_OBJECTS_YAML_FILE}
export TMP_DIR=$(mktemp -d)
export PARALLELISM_FACTOR=2
printf ${OBJECT_TYPE} | xargs -d ',' -I {} -P ${PARALLELISM_FACTOR} -n1 bash -c 'kubectl get -A -oyaml "$@" > ${TMP_DIR}/"$@".yaml' _ {}
function get_namespaces () {
kubectl get namespaces -o name | awk -F '/' '{ print $NF }'
@ -97,7 +97,7 @@
echo "${NAMESPACE}/${OBJECT}/${NAME}"
DIR="{{ logs_dir }}/objects/${CONTEXT}/namespaced/${NAMESPACE}/${OBJECT}"
mkdir -p ${DIR}
cat "${ALL_OBJECTS_YAML_FILE}" | kustomize cfg grep "metadata.namespace=${NAMESPACE}" | kustomize cfg grep "metadata.name=${OBJECT}" > "${DIR}/${NAME}.yaml"
cat "${TMP_DIR}/${OBJECT}.yaml" | kustomize cfg grep "metadata.namespace=${NAMESPACE}" | kustomize cfg grep "metadata.name=${NAME}" > "${DIR}/${NAME}.yaml"
kubectl describe -n ${NAMESPACE} ${OBJECT} ${NAME} > "${DIR}/${NAME}.txt"
}
export -f get_objects