Skip to content

Commit

Permalink
Merge pull request #1634 from edx/iamsobanjaved/fix-repo-health-scrip…
Browse files Browse the repository at this point in the history
…t-append

fix: include both orgs data while compiling dashboards
  • Loading branch information
iamsobanjaved authored Dec 22, 2022
2 parents 9865bf9 + 5d5e4f6 commit c5c0ef2
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion devops/resources/run-repo-health-on-org.sh
Original file line number Diff line number Diff line change
Expand Up @@ -142,12 +142,14 @@ done < "$input"
# Go into data repo, recalculate aggregate data, and push a PR
IFS=,
failed_repo_names=`echo "${failed_repos[*]}"`
# Delete existing dashboards to re compile
find "${WORKSPACE}/repo-health-data/dashboards" -type f -iname "\dashboard*.csv" -delete

for ORG_NAME in ${ORG_NAMES[@]}; do
echo "Pushing data for org $ORG_NAME"
cd "${WORKSPACE}/repo-health-data/individual_repo_data/${ORG_NAME}"
ls
repo_health_dashboard --data-dir . --configuration "${WORKSPACE}/edx-repo-health/repo_health_dashboard/configuration.yaml" --output-csv "${WORKSPACE}/repo-health-data/dashboards/dashboard"
repo_health_dashboard --data-dir . --configuration "${WORKSPACE}/edx-repo-health/repo_health_dashboard/configuration.yaml" --output-csv "${WORKSPACE}/repo-health-data/dashboards/dashboard" --append
done

deactivate
Expand Down

0 comments on commit c5c0ef2

Please sign in to comment.