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

TP2000 1407 Sqlite dump process failing #1254

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Luisella21
Copy link
Contributor

TP2000 1407 daily sqlite dump process failing

Add logging to pin point the problem.

TP2000-1407-daily-sqlite-dump-process-failing

Why

What

@Luisella21 Luisella21 requested a review from a team as a code owner June 25, 2024 14:14

logger.debug(f"Check that filename {filename} exists")
logger.debug(
f" ****** SQLITE_S3_ENDPOINT_URL {settings.SQLITE_S3_ENDPOINT_URL}",
Copy link
Collaborator

@paulpepper-trade paulpepper-trade Jun 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For debugging this particular problem, we should look at the env vars on the Celery instance to validate correct config - otherwise I think this adds noise to the logs.

Comment on lines +61 to +65
# except Exception as e:
# logger.error(f"Error encountered while checking {filename}")
# logger.error(f"Exception: {e}")
# return False

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hangover comment requiring deletion.

Comment on lines +49 to +52
logger.debug(
f" In export_and_upload_sqlite "
f"SQLITE_S3_ENDPOINT_URL {settings.SQLITE_S3_ENDPOINT_URL}",
)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For debugging this particular problem, we should look at the env vars on the Celery instance to validate correct config - otherwise I think this adds noise to the logs.

@@ -713,7 +713,7 @@
},
"exporter": {
"handlers": ["console"],
"level": os.environ.get("LOG_LEVEL", "DEBUG"),
"level": os.environ.get("LOG_LEVEL_EXPORTER", "DEBUG"),
Copy link
Collaborator

@paulpepper-trade paulpepper-trade Jun 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checking our config store and env vars, I don't think we actually need to adapt the log level to get the debugging info that we'd like to gather.

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