report: place Upload() on io.systemd.Report.Uploader rather than io.systemd.Report interface (#42584)

We really want to use io.systemd.Report for the interface provided by
systemd-report itself, not by its backend. hence, rename the interface
that uploading plugins shall implement to io.systemd.Report.Uploader.

Note that we ideally should have a varlink interface definition for that
interface. if we had, we'd have noticed that earlier.
This commit is contained in:
Luca Boccassi
2026-06-15 22:05:18 +01:00
committed by GitHub
3 changed files with 4 additions and 4 deletions
+2 -2
View File
@@ -88,8 +88,8 @@
to an external server. Two upload mechanisms are supported. If an <literal>http://</literal> or
<literal>https://</literal> URL is specified with <option>--url=</option>, an HTTP upload will be
performed to the specified location. Otherwise, any sockets under
<filename>/run/systemd/metrics-upload/</filename> will be used to call
<function>io.systemd.Report.Upload()</function>.</para>
<filename>/run/systemd/report.upload/</filename> will be used to call
<function>io.systemd.Report.Uploader.Upload()</function>.</para>
<para>Match expressions supported by <command>metrics</command> are supported here too.</para>
+1 -1
View File
@@ -249,7 +249,7 @@ static int upload_collected(Context *context, sd_json_variant *report) {
ssize_t jobs = varlink_execute_directory(
REPORT_UPLOAD_DIR,
"io.systemd.Report.Upload",
"io.systemd.Report.Uploader.Upload",
params,
/* more= */ false,
arg_network_timeout_usec,
+1 -1
View File
@@ -9,7 +9,7 @@
#define REPORT_CERT_FILE CERTIFICATE_ROOT "/certs/systemd-report.pem"
#define REPORT_TRUST_FILE CERTIFICATE_ROOT "/ca/trusted.pem"
#define REPORT_UPLOAD_DIR "/run/systemd/metrics-upload"
#define REPORT_UPLOAD_DIR "/run/systemd/report.upload"
extern sd_json_format_flags_t arg_json_format_flags;
extern char *arg_url, *arg_key, *arg_cert, *arg_trust;