Opened 4 years ago

Closed 21 months ago

#16597 closed enhancement (wontfix)

Consider merging the hourly updater and servlet into a single daemon

Reported by: karsten Owned by: iwakeh
Priority: Low Milestone:
Component: Metrics/ExoneraTor Version:
Severity: Normal Keywords: metrics-2018
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

ExoneraTor consists of two parts: an hourly updater and a servlet. Setting both up correctly is not exactly trivial, and I could imagine that merging the two parts into a single daemon would facilitate that.

Maybe we can make the setup as easy as building or downloading a .jar file and running it. That application would then connect to the database, create tables and indexes, populate it with data, and handle web requests. We would probably want to switch from Tomcat to embedded Jetty, similar to what we did with Onionoo.

Possible downsides are that the two parts could not be separated anymore for better resource usage or for security reasons. Of course, we could include command-line options for only running the hourly updater or for only handling web requests.

Child Tickets

Change History (9)

comment:1 Changed 2 years ago by iwakeh

Severity: Normal

Set to 'normal' as 'blocker' seems inappropriate.

comment:2 Changed 2 years ago by iwakeh

See #23549 for additional input.

comment:3 Changed 23 months ago by karsten

Keywords: metrics-2018 added

comment:4 Changed 23 months ago by karsten

Owner: set to metrics-team
Status: newassigned

comment:5 Changed 22 months ago by iwakeh

Owner: changed from metrics-team to iwakeh
Status: assignedaccepted

As a first step (as #21145 is finished now) exonerator will move to an embedded jetty in order to first avoid the configuration problems using an external tomcat instance.

comment:6 Changed 22 months ago by iwakeh

The ticket for moving to Jetty is #24174.

comment:7 in reply to:  description Changed 21 months ago by karsten

Status: acceptedneeds_information

Replying to karsten:

Possible downsides are that the two parts could not be separated anymore for better resource usage or for security reasons. Of course, we could include command-line options for only running the hourly updater or for only handling web requests.

Update: We changed deployment yesterday by having a separate user for the web server part with read-only permissions for the database and the data directory of the hourly importer. So, if we still want to merge the two parts, we'll have to provide those command-line options. But maybe we shouldn't do this anymore. Hmm.

comment:8 Changed 21 months ago by iwakeh

I think this ticket can be closed for now. It matches the updater-web separation we use in Onionoo and metrics-web too, which makes sense. And, the setup is not extraordinarily complicated.

comment:9 Changed 21 months ago by karsten

Resolution: wontfix
Status: needs_informationclosed

Works for me. Thanks!

Note: See TracTickets for help on using tickets.