Update update_monitor_table() to use django models

[bhilbert4]

Update the update_monitor_table function, which is called by most (all?) monitors and records the date and success or failure of a monitor run, to use Django models.

This PR involves an update to the definition of the status column in the table, and therefore will need a migration attached to it.

Resolves #1485

[pep8speaks]

Hello @bhilbert4, Thank you for updating !

• In the file jwql/website/apps/jwql/monitor_models/common.py:

Line 188:1: E305 expected 2 blank lines after class or function definition, found 1
Line 188:1: E402 module level import not at top of file

Comment last updated at 2025-01-28 02:38:00 UTC

[bhilbert4]

• Note: wait for Add wisp finder monitor #1666 to be merged before creating the migration for this PR

[mfixstsci]

This looks good, do we need to add a migration for update_monitor_table?

[mfixstsci]

• Note: wait for Add wisp finder monitor #1666 to be merged before creating the migration for this PR

I see, this answers my question 😆

[mfixstsci]

@bhilbert4 #1666 is in, waiting on the rebase and then we can merge this too 💯

[bhilbert4]

I still need to create the migration file for this one...

[mfixstsci]

I still need to create the migration file for this one...

Of course, I am having PR merging tunnel vision 😵‍💫

[bhilbert4]

I just ran the wisp monitor (locally) using the updated code in this PR, and I can see the new entry in the Monitor database table. All looks good. I got a warning about datetime objects with no timezone, but I think that's something to be fixed in the wisp monitor in a separate PR. I think this PR is good to go.

[mfixstsci]

I just ran the wisp monitor (locally) using the updated code in this PR, and I can see the new entry in the Monitor database table. All looks good. I got a warning about datetime objects with no timezone, but I think that's something to be fixed in the wisp monitor in a separate PR. I think this PR is good to go.

Nice, that is typical something to do with a datetime field entry from my experience.