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

Ticket8631 #14

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Ticket8631 #14

wants to merge 10 commits into from

Conversation

Tom-Willemsen
Copy link
Contributor

@Tom-Willemsen Tom-Willemsen commented Jan 17, 2025

Closes ISISComputingGroup/IBEX#8631
Closes ISISComputingGroup/IBEX#8628

Scope expanded a bit but:

  • Migrate user-office auth to REST api as required rather than SOAP
    • As far as I can tell, BusApps are not turning off SOAP to the scheduler service, just the auth side as far as I could tell. But it will now be easier to migrate that side over too if necessary.
  • Make it all work on modern python versions on dev machines (and ensure CI checks this too)
  • Make it work with MySQL 8.4
  • Refactored to use something closer to best-practice setup
    • Main cli moves to a "project script" - i.e. something you can directly execute after installing this into a venv
  • Move CI to GHA as this lets us easily test against a wider range of python versions etc
  • Ruff
  • Didn't do pyright / type-hinting much in this PR as it's quite painful if we're still stuck on python 3.8. instead put that in a different PR, but it's not viable to merge that until we can update python versions (I really don't want to backport loads of typing stuff to old syntaxes)

Docs updated at https://github.com/ISISComputingGroup/ibex_developers_manual/wiki/Experimental-Database#deployment

Will need to do the "deployment" steps on the central server once this is merged

@Tom-Willemsen Tom-Willemsen force-pushed the Ticket8631 branch 2 times, most recently from f835e01 to 1968da7 Compare January 17, 2025 16:22
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.

Exp database populator: migrate auth scheme Exp database populator: cannot install
1 participant