Generate shared drive usage report CSV
Based of !30 (merged) - so that needs merging first
Was going to update https://gitlab.developers.cam.ac.uk/uis/devops/gsuite/operational-documentation/-/snippets/255 but thought it was about time to add this functionality to the gdrivemanager
tool, especially as it already had google drive and lookup api client code.
First two commits is the splitting the manage.py
(and its tests) as it was just too big (so reviewing each commit may be easier).
Example output CSV: https://drive.google.com/file/d/1LW0W8UIm9AIAAoLo1ePiFzNZ4BYGor2Q/view?usp=drive_link
Added to Google Sheet: https://docs.google.com/spreadsheets/d/1kqIiOfKoCmLkdS5VHvxUIREKMAVtW-OHd9I_k9qHmjE/edit
Merge request reports
Activity
assigned to @rjg21
added 8 commits
-
4382758e...d32af9a9 - 2 commits from branch
main
- 6086c0b8 - feat: split manage.py
- 7b9c8ec8 - feat: split tests for previous manage.py
- 9e057289 - feat: produce all shared drive usage report csv
- 50c205a3 - feat: test for shared drive usage report generation
- 23717395 - chore: bump version to 1.1.5
- 4d2a2cb2 - fix: copy changes to shared drive scanning from 1.1.4
Toggle commit list-
4382758e...d32af9a9 - 2 commits from branch
requested review from @jb2595
- Resolved by Robin Goodall
I don't have permissions to view the example files, however the codes changes look good to me
requested review from @mk2155
- Resolved by Mike Knee
- Resolved by Mike Knee
Could you please move the
test_manage_*
tests togdrivemanager/manage/tests/
(and then drop themanage
from the test file name). Sorry, I know this is a little fussy, but that matches our pattern with the Django apps and is better for consistency if we're keeping our tests in the same structure as our regular source code.
added 1 commit
- b4a2774f - review: move manage tests to appropriate subfolder
reset approvals from @jb2595 by pushing to the branch
mentioned in issue #33 (closed)
removed review request for @jb2595
mentioned in commit 7b288607