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

Build conda #332

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

Build conda #332

wants to merge 4 commits into from

Conversation

jburel
Copy link
Member

@jburel jburel commented Aug 31, 2022

This PR introduces a job testing the installation of omero-py in a conda env and connect to IDR
The installation is done on MAc, Windows and Ubuntu. This follows a post on image.sc
Open against this repo but https://github.com/conda-forge/omero-py-feedstock could be considered

cc @dominikl

@joshmoore
Copy link
Member

  • Big 👍 for testing conda.
  • Pretty strong preference for the feedstock, since changes there wouldn't be tested here (but vice versa works)
  • A bit uncertain about testing against IDR.

@jburel
Copy link
Member Author

jburel commented Sep 6, 2022

We could use another server but I did not want to introduce a secret param

@joshmoore
Copy link
Member

Sorry for having dropped this. Should we discuss maybe having a read-only user for demo? The two positives I can think of:

  • It would keep us regularly testing demo (and alert us if it has fallen over).
  • The SLA specifies that if the tests cause it to fall over, well so be it.

@jburel
Copy link
Member Author

jburel commented Sep 30, 2022

I forgot about it too.
Having a read-only user will work

@jburel jburel closed this Feb 13, 2024
@jburel jburel reopened this Feb 13, 2024
@@ -0,0 +1,11 @@
import sys
from omero.gateway import BlitzGateway
HOST = 'ws://idr.openmicroscopy.org/omero-ws'
Copy link
Member

Choose a reason for hiding this comment

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

My concerns here persist.

Copy link
Member Author

Choose a reason for hiding this comment

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

not suggesting to merge as it is. I will discuss post migration to RHEL 9 the demo option with @pwalczysko
I was just using it to validate.

Copy link
Member

Choose a reason for hiding this comment

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

👍

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