Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#19540 closed enhancement (implemented)

Update libraries and instructions to current Debian stable

Reported by: karsten Owned by:
Priority: Medium Milestone: Metrics 1.0.0
Component: Metrics/Website Version:
Severity: Normal Keywords:
Cc: iwakeh Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Last week or so we upgraded the Metrics host from wheezy to jessie and resolved a few issues. I'll attach a branch with some fixes in a minute.

Child Tickets

Attachments (1)

0001-Additions-to-task-19540.patch (11.3 KB) - added by iwakeh 3 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 3 years ago by karsten

Cc: iwakeh added
Status: newneeds_review

Please review the last two commits in my branch task-19540.

comment:2 Changed 3 years ago by iwakeh

There were quite a few things not working due to the README not being up-to-date before the jessie related changes.

I attached a patch for the things I could find w/o setting up an entire metrics-web.
I could not identify the source for the standard-1.1.2.jar
The commands needing root privileges now all are run via sudo.
Please check carefully.

All java projects should be on the same java version (7 or 8?), and need some modernization.
New issue(s)?

Changed 3 years ago by iwakeh

comment:3 Changed 3 years ago by iwakeh

Status: needs_reviewneeds_revision

comment:4 in reply to:  2 ; Changed 3 years ago by karsten

Status: needs_revisionneeds_review

Replying to iwakeh:

There were quite a few things not working due to the README not being up-to-date before the jessie related changes.

Agreed. I didn't attempt to fix all remaining issues, which is a small project in itself, I just wanted to fix the most obvious shortcomings.

I attached a patch for the things I could find w/o setting up an entire metrics-web.

Cool! I added that patch to my branch and made a few tweaks on top of it. Mind taking another look?

I could not identify the source for the standard-1.1.2.jar

I wrote Debian stable package names next to .jar files, also as a note to myself when I need to find out in a year from now. Please shout if any of these packages look like strange choices to you.

The commands needing root privileges now all are run via sudo.
Please check carefully.

Looks good!

All java projects should be on the same java version (7 or 8?), and need some modernization.
New issue(s)?

Yes, new issue, I'd say. And should we start the Java 8 update with metrics-lib, or does that not really matter?

Thanks!

comment:5 in reply to:  4 Changed 3 years ago by iwakeh

Replying to karsten:

Replying to iwakeh:

There were quite a few things not working due to the README not being up-to-date before the jessie related changes.

Agreed. I didn't attempt to fix all remaining issues, which is a small project in itself, I just wanted to fix the most obvious shortcomings.

Looking at it this way the current branch is fine now.

...
I wrote Debian stable package names next to .jar files, also as a note to myself when I need to find out in a year from now. Please shout if any of these packages look like strange choices to you.

Good idea and the versions look ok.

...
Yes, new issue, I'd say. And should we start the Java 8 update with metrics-lib, or does that not really matter?

Whatever depends on metrics-lib needs to upgrade before using a metrics-lib java 8 version.
Thus, if there are other important changes we're forced to upgrade or back-port changes.
First upgrading the depending projects is less pressure as those could safely use metrics-lib from java 7.

comment:6 Changed 3 years ago by iwakeh

Status: needs_reviewmerge_ready

comment:7 Changed 3 years ago by karsten

Thanks for the review! Squashed my latest commit into yours and pushed to master.

Regarding updating metrics-lib and depending projects, sure, let's start with one of the depending projects then. Can you create a ticket and close this one when you're done? Thanks!

comment:8 Changed 3 years ago by iwakeh

Resolution: implemented
Status: merge_readyclosed

java 8 update ticket: #19620
closing.

comment:9 Changed 3 years ago by iwakeh

Milestone: Metrics 1.0.0
Note: See TracTickets for help on using tickets.