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

Upgrade server, web and web apps #373

Merged
merged 4 commits into from
Nov 3, 2023
Merged

Conversation

pwalczysko
Copy link
Member

@pwalczysko pwalczysko commented Aug 7, 2023

cc @jburel

As discussed, upgrading 7 items.

  • nightshade
  • learning
  • training servers
  • demo

@pwalczysko pwalczysko requested review from jburel and khaledk2 August 7, 2023 21:31
@pwalczysko pwalczysko mentioned this pull request Sep 6, 2023
@pwalczysko
Copy link
Member Author

@sbesson Would you please have an idea why the test_omero_metrics is failing here ?

@sbesson
Copy link
Member

sbesson commented Sep 14, 2023

I have a suspicion this is exactly the issue I was mentioning early today with the ome.omero_prometheus_exporter role which requires a newer OMERO.py to work with server deployed with omero-certificates 0.3.0+ - see ome/ansible-role-omero-prometheus-exporter#9 for the proposed fix

@pwalczysko
Copy link
Member Author

This is green as I implemented the change suggested yesterday by @sbesson - happy for me to merge to prevent accidental upgrades please, cc @jburel ?

@pwalczysko pwalczysko requested a review from sbesson November 1, 2023 10:56
@pwalczysko pwalczysko merged commit 3f203ef into ome:master Nov 3, 2023
10 checks passed
@pwalczysko pwalczysko deleted the upgrade branch November 3, 2023 08:46
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.

2 participants