Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #12281 (closed)

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

Bug: bin\omero admin start should optionally create a Windows service

Reported by: bpindelski Owned by: bpindelski
Priority: major Milestone: 5.0.3
Component: Deployment Version: 5.0.1
Keywords: n.a. Cc: jamoore, spli
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description (last modified by bpindelski)

Many times now it has been discussed that being able to start up the server on Windows without creating a Windows service would be desirable (e.g. for JMX debugging, where the presence of the service interferes with user/process permissions; or for quick start-up/shutdown of the server).

Update admin.py with the new code path, which might add a new CLI parameter. Follow law of least surprise.

Change History (9)

comment:1 Changed 5 years ago by bpindelski

  • Description modified (diff)

comment:2 Changed 5 years ago by bpindelski

Also along the way, check if it's feasible to reduce the OMERO service privileges to Local Service.

comment:3 Changed 5 years ago by bpindelski

Final major point: icegridnode doesn't detach from the Command Prompt when used without --service. This parameter has been put into place by ZeroC to allow backgrounding of the Ice servers. If this ticket gets fixed, then the debug mode (service-less start) will need one prompt window to be open during the lifetime of the server.

A more general solution has been proposed by Josh: merge the service-less option with the debug target into --debug. This would require removing of suspend=y from templates.xml.

comment:5 Changed 5 years ago by bpindelski

Using --debug as a CLI parameter for bin\omero admin start is not possible. cli.py already defines such a parameter and it's not trivial to modify argparse's parameter list (it's immutable).

comment:6 Changed 5 years ago by spli

  • Cc spli added

comment:7 Changed 5 years ago by bpindelski

  • Owner set to bpindelski
  • Status changed from new to accepted

comment:8 Changed 5 years ago by bpindelski

  • Resolution set to fixed
  • Status changed from accepted to closed

comment:9 Changed 5 years ago by Josh Moore <josh@…>

(In [c0363a7fbef0fd1fe2e343d5ec1a3270eaf2e918/ome.git] on branch develop) Merge pull request #2811 from bpindelski/rebased/develop/12281_debug

Allow foregrounding of the server process (see #12281). (rebased onto develop)

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.113016 sec.)

We're Hiring!