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

Customise Windows service name: omero.windows.servicename #3122

Merged
merged 1 commit into from
Nov 3, 2014

Conversation

manics
Copy link
Member

@manics manics commented Oct 20, 2014

This is needed to allow multiple server deployments on the same Windows system.

C:\OMERO.server>bin\omero config set omero.windows.servicename toast

C:\OMERO.server>bin\omero config get
omero.data.dir=\OMERO
omero.windows.servicename=toast

C:\OMERO.server>bin\omero admin start
Found default value: c:\omero_dist\var\master
Attempting to correct...
Converting from c:\omero_dist to C:\OMERO.server
Changes made: 6
Creating C:\OMERO.server\var\master
Creating C:\OMERO.server\var\registry
No descriptor given. Using etc\grid\windefault.xml
Installing toast.master Windows service.
Successfully installed toast.master Windows service.
Starting toast.master Windows service.
Waiting on startup. Use CTRL-C to exit
..................
C:\OMERO.server>

This is currently deployed as a merge-deploy on Hake (OMERO-5.1-merge-deploy-win, port 4064, servicename: OMEROmerge) alongside latest-deploy (OMERO-5.1-latest-deploy-win, port 14064, default servicename since PR isn't merged). Note multiple web/IIS isn't working for other reasons so ignore web for now.

Test locally on your own VM if you're feeling brave.

@sbesson sbesson closed this Oct 21, 2014
@sbesson sbesson reopened this Oct 21, 2014
@jburel jburel added the develop label Oct 21, 2014
@kennethgillen
Copy link
Member

OMEROmerge.master on Hake, from trout/merge:
Created session a932a26a-2032-4545-8bd0-016f32d27bbd (root@hake.openmicroscopy.org:14064). Idle timeout: 10.0 min. Current group: system

@kennethgillen
Copy link
Member

OMERO.master on Hake, from trout/merge:

Created session 51bf90ce-892e-4ed0-9855-29a81beaf8cb (root@hake.openmicroscopy.org:4064).

@kennethgillen
Copy link
Member

Interestingly, the CLI told me:

Previously logged in to hake.openmicroscopy.org:4064 as root

Which is wrong, since you can clearly see in my previous comment it definitely connected to 14064.. :/

Other than that, 👍 🐟

@sbesson
Copy link
Member

sbesson commented Oct 27, 2014

Do you remember if you did login/logout before connecting to port 4064?

@sbesson
Copy link
Member

sbesson commented Oct 27, 2014

In general, the problem is that the port of the previous session is saved in the session file itself (and not aliased list last host and last user), so if the session file is removed, there is no way to remember which port was used. A workaround may be to detect this case and not specify the port for the last connection if it cannot be read.

@joshmoore
Copy link
Member

Alternatively, to add the port to the host name on disk?

@sbesson
Copy link
Member

sbesson commented Oct 27, 2014

As in an additional subfolder?

@joshmoore
Copy link
Member

that or, say, everything after a comma: localhost,14064

@sbesson
Copy link
Member

sbesson commented Oct 27, 2014

Yes that would likely minimize the impact on the sessions logic (default to hostname if the port is the default). I wonder if there is a better separator than ',' for this kind of things. Want to open a ticket?

@joshmoore
Copy link
Member

@joshmoore
Copy link
Member

Merging this to leave @sbesson fun with 12609. Thanks, @manics.

joshmoore added a commit that referenced this pull request Nov 3, 2014
Customise Windows service name: omero.windows.servicename
@joshmoore joshmoore merged commit 6b2b63a into ome:develop Nov 3, 2014
@manics
Copy link
Member Author

manics commented Nov 5, 2014

--no-rebase

@manics manics deleted the windows_service_name branch November 5, 2014 10:56
@sbesson sbesson added this to the 5.1.0-m2 milestone Nov 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants