Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

db_check does not ERROR when cannot connect to storage #119

Open
danstoner opened this issue Oct 21, 2020 · 3 comments
Open

db_check does not ERROR when cannot connect to storage #119

danstoner opened this issue Oct 21, 2020 · 3 comments

Comments

@danstoner
Copy link
Contributor

The "error" currently only appears in DEBUG lines. In this case, the recordset could not be downloaded (could not connect to storage) which probably should be a fatal error since the program couldn't do any work. At the very least, an ERROR logline should be written.

2020-10-21 19:42:16.526 INFO  idb.db-check.7232e59b-b0e1-49cd-9632-b8ae1f49f828჻ Starting db_check ingest: False
2020-10-21 19:42:16.529 DEBUG idb.storage჻ Initialized IDigBioStorage connection (boto.connect_s3) to host 's.idigbio.org'
2020-10-21 19:42:16.673 DEBUG idb.db-check.7232e59b-b0e1-49cd-9632-b8ae1f49f828჻ Exception in get_file
2020-10-21 19:42:16.675 INFO  idb.db-check჻ 7232e59b-b0e1-49cd-9632-b8ae1f49f828 writing summary json
2020-10-21 19:42:16.680 INFO  idb.db-check.7232e59b-b0e1-49cd-9632-b8ae1f49f828჻ Finished db_check in 0.152s
2020-10-21 19:42:16.681 DEBUG idb჻ db_check(...) took 1.8s
@danstoner
Copy link
Contributor Author

The accompanying summary file is just a dummy / stub.

{
  "datafile_ok": false, 
  "records_count": 0, 
  "records_update": 0, 
  "paused": false, 
  "harvest_date": "", 
  "mediarecords_create": 0, 
  "mediarecords_update": 0, 
  "recordset_id": "7232e59b-b0e1-49cd-9632-b8ae1f49f828", 
  "commited": false, 
  "mediarecords_delete": 0, 
  "filemd5": null, 
  "filename": "7232e59b-b0e1-49cd-9632-b8ae1f49f828", 
  "records_delete": 0, 
  "records_create": 0, 
  "mediarecords_count": 0
}

@danstoner
Copy link
Contributor Author

Are we catching these in suspects file?

records_count and mediarecords_count are both 0.

@danstoner
Copy link
Contributor Author

It is potentially a transient error... seems like both error log AND suspects handling (if we don't already have that) would be good. 🤔

To test... maybe db-check a non-existent uuid?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant