Skip to content

Conversation

@JulienPeloton
Copy link

@JulienPeloton JulienPeloton commented Jun 25, 2025

This PR is an attempt to update the code and its dependencies to latest versions.

Python version

I upgraded from python 3.11 to 3.12

External dependencies

Major change concerns:

  • tomtoolkit: 2.18.15 to 2.24.5
  • tom-registration: 0.1.0 to 1.0.1
  • add setuptools, as distutils has been dropped in py3.12

DB migration

22 migrations have been performed:

(env) peloton@peloton:~/codes/sso-alert/sso_tom$ ./manage.py migrate
...
registering new views: args: ('groups', <class 'tom_common.api_views.GroupViewSet'>, 'groups'), kwargs: {}
registering new views: args: ('targets', <class 'tom_targets.api_views.TargetViewSet'>, 'targets'), kwargs: {}
registering new views: args: ('targetextra', <class 'tom_targets.api_views.TargetExtraViewSet'>, 'targetextra'), kwargs: {}
registering new views: args: ('targetname', <class 'tom_targets.api_views.TargetNameViewSet'>, 'targetname'), kwargs: {}
registering new views: args: ('targetlist', <class 'tom_targets.api_views.TargetListViewSet'>, 'targetlist'), kwargs: {}
registering new views: args: ('observations', <class 'tom_observations.api_views.ObservationRecordViewSet'>, 'observations'), kwargs: {}
registering new views: args: ('dataproducts', <class 'tom_dataproducts.api_views.DataProductViewSet'>, 'dataproducts'), kwargs: {}
registering new views: args: ('reduceddatums', <class 'tom_dataproducts.api_views.ReducedDatumViewSet'>, 'reduceddatums'), kwargs: {}
...
Operations to perform:
  Apply all migrations: admin, auth, authtoken, chained, contenttypes, django_comments, guardian, sessions, sites, sso_alerts, tom_alerts, tom_common, tom_dataproducts, tom_observations, tom_targets
Running migrations:
  Applying authtoken.0004_alter_tokenproxy_options... OK
  Applying chained.0001_initial... OK
  Applying chained.0002_templatedchain_chainedtemplate... OK
  Applying chained.0003_alter_templatedchain_description... OK
  Applying chained.0004_templatedchain_status... OK
  Applying chained.0005_chainedtemplate_name... OK
  Applying chained.0006_alter_chainedtemplate_templated_chain... OK
  Applying chained.0007_alter_chainedobservation_chain_and_more... OK
  Applying chained.0008_alter_chainedobservation_observation... OK
  Applying sso_alerts.0001_initial... OK
  Applying sso_alerts.0002_alter_alertstreams_description... OK
  Applying sso_alerts.0003_alertstreams_active... OK
  Applying sso_alerts.0004_targetstream... OK
  Applying tom_common.0001_initial... OK
  Applying tom_dataproducts.0013_alter_reduceddatum_message... OK
  Applying tom_dataproducts.0014_alter_reduceddatum_timestamp... OK
  Applying tom_targets.0021_rename_target_basetarget_alter_basetarget_options... OK
  Applying tom_targets.0022_persistentshare... OK
  Applying tom_targets.0023_alter_basetarget_created... OK
  Applying tom_targets.0024_basetarget_permissions... OK
  Applying tom_targets.0025_auto_20250206_2017... OK
  Applying tom_targets.0026_alter_basetarget_permissions... OK

Checks

python manage.py server

June 25, 2025 - 12:43:20
Django version 4.2.9, using settings 'sso_tom.settings'
Starting development server at http://127.0.0.1:8000/
Quit the server with CONTROL-C.

Misc

  1. In order to run the UI, I had to manually set a lot of env variable. They should eventually be defined somewhere (this is somewhat discussed in README_rocky9.md)
  2. there is no documentation on the Docker part, and running ./run_docker_build_and_launch.sh fails because of missing definition of env variables mentioned above.

@JulienPeloton JulienPeloton changed the title WIP: Update dependencies Update dependencies Jun 25, 2025
@anaismoller
Copy link

@shiblisaleheen can you review and approve this pull request? thanks!

Copy link
Contributor

@shiblisaleheen shiblisaleheen left a comment

Choose a reason for hiding this comment

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

Approved

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

Successfully merging this pull request may close these issues.

3 participants