Catch 404 when deleting user in arborist

[k-burt-uch]

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

• Usersync no longer fails when deleting a user that does not exist in arborist.

Improvements

Dependency updates

Deployment changes

[k-burt-uch]

Relevant log in BDC
https://monitoring.planx-pla.net/goto/bfhx0ay95mc5cb?orgId=1

I have no idea where this N/A user comes from. Maybe dbGaP??? But its not in our useryaml: https://raw.githubusercontent.com/uc-cdis/datastage-users/refs/heads/master/users/stageprod/user.yaml?token=GHSAT0AAAAAADSQPZWBYNQLRDPJQZM7M3RO2OO7JSA

[coveralls]

Pull Request Test Coverage Report for Build 23924966144

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 49 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.01%) to 75.055%

---

Files with Coverage Reduction	New Missed Lines	%
sync/sync_users.py	49	81.9%

Totals
Change from base Build 23756457076:	-0.01%
Covered Lines:	8458
Relevant Lines:	11269

---

💛 - Coveralls

[paulineribeyre]

I think we should only ignore 404s, and raise any other type of error 😬

[github-actions]

filepath	passed	skipped	SUBTOTAL
tests/test_oauth2.py	15	0	15
tests/test_centralized_auth.py	16	0	16
tests/test_audit_service.py	3	3	6
tests/test_data_upload.py	8	1	9
tests/test_presigned_url.py	7	0	7
tests/test_dbgap.py	4	1	5
tests/test_drs_endpoint.py	4	0	4
tests/test_register_user.py	2	0	2
tests/test_user_token.py	5	0	5
tests/test_google_data_access.py	1	0	1
tests/test_client_credentials.py	1	0	1
tests/test_oidc_client.py	2	0	2
tests/test_ras_authn.py	0	3	3
TOTAL	68	8	76

Please find the detailed integration test report here

Please find the Github Action logs here