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

Mapr deprecation #441

Merged
merged 14 commits into from
Dec 9, 2024
Merged
2 changes: 1 addition & 1 deletion ansible/group_vars/omero-hosts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ omero_web_config_set:
omero_web_apps_packages:
- omero-mapr==0.5.2
- omero-iviewer==0.14.0
- idr-gallery==3.11.0
- idr-gallery==3.12.0
will-moore marked this conversation as resolved.
Show resolved Hide resolved
- omero-figure==6.2.0
Copy link
Member

Choose a reason for hiding this comment

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

Figure is not used but some libraries bumped happened since and it could potential lead to some dependencies issues down the line

Copy link
Member

Choose a reason for hiding this comment

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

I can see one of three options:

  • do nothing and capture as an issue
  • bump OMERO.figure - if so to which version?
  • remove OMERO.figure
    Which one would you recommend?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm happy to bump OMERO.figure to latest 7.1.0...

Copy link
Member Author

Choose a reason for hiding this comment

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

@sbesson Could you re-run the deployment just to check that Figure bump doesn't cause any unexpected issues, thx

Copy link
Member

Choose a reason for hiding this comment

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

Deployed on test125

Copy link
Member

Choose a reason for hiding this comment

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

@jburel as discussed at the weekly IDR meeting, leaving you to have a final look at the changes pushed by @will-moore . Once approved, I'll work on updating prod125

omero_web_apps_names:
- omero_iviewer
Expand Down
Loading