openvibers/openvibe.actor Commits
Actor: request telemetry in its own table, 30-day retention (#86)
Changes
nnew file mode 100644@@@ -0,0 +1,28 @@++-- phase: expand++-- Request telemetry (openvibe-sdk/telemetry telemetrySchema()): one row per aggregated sample, pruned after 30 days (server/index.js).++CREATE TABLE IF NOT EXISTS telemetry_samples (++ id bigint GENERATED ALWAYS AS IDENTITY PRIMARY KEY,++ at timestamptz NOT NULL,++ service text NOT NULL,++ project text,++ subject text,++ resource text,++ provider text,++ node text,++ cell text,++ region text,++ operation text NOT NULL,++ latency_ms double precision,++ queue_delay_ms double precision,++ ttfb_ms double precision,++ throughput_per_second double precision,++ bytes bigint,++ status text,++ cache_status text,++ cost_estimate double precision,++ route_epoch bigint,++ trace_id text,++ extra jsonb NOT NULL DEFAULT '{}'::jsonb++);++CREATE INDEX IF NOT EXISTS telemetry_samples_at_idx ON telemetry_samples (at);++CREATE INDEX IF NOT EXISTS telemetry_samples_service_idx ON telemetry_samples (service, operation, at);@@@ -58,6 +58,8 @@ const { html } = require('./render/html');const PUBLIC_DIR = path.join(__dirname, '..', 'public');const VERSION = require('../package.json').version;++const { createHttpTelemetry } = require('openvibe-sdk/service');++const { createPgTelemetrySink } = require('openvibe-sdk/telemetry');/*** opts: config, store, now (clock), fetchImpl (Network), providerFetch (model providers and OpenVibe services), log,@@@ -112,6 +114,11 @@ async function createApp(opts = {}) {require('./render/layout').setRelease(release.release);const metrics = require('openvibe-shared/metrics').instrument(app, { service: 'actor', release: release.release });app.locals.metrics = metrics.registry;++ // Request telemetry (openvibe-sdk/service, plan T1): one aggregated sample per route template, method and status++ // class each flush, stored in this service's telemetry_samples table; /shared, health, readiness and /metrics make++ // none. server/index.js flushes it on stop and keeps the table to 30 days.++ ctx.telemetry = createHttpTelemetry({ service: 'actor', sink: opts.telemetrySink || createPgTelemetrySink({ db: s.db, log }), skipPrefixes: ['/shared'] });++ app.use(ctx.telemetry.middleware());app.locals.ctx = ctx;const valkey = opts.valkey !== undefined ? opts.valkey : (config.valkey.url ? require('openvibe-sdk/valkey').createValkey({ url: config.valkey.url, prefix: config.valkey.prefix, log }) : null);ctx.valkey = valkey;@@@ -6,6 +6,7 @@*/const { createApp } = require('./app');const { gracefulStop } = require('openvibe-sdk/service');++const { startTelemetryRetention } = require('openvibe-sdk/telemetry');const store = require('./tasks/store');const PRUNE_EVERY_MS = 6 * 60 * 60_000;@@@ -19,7 +20,7 @@ const PRUNE_EVERY_MS = 6 * 60 * 60_000;function createLifecycle({ server, ctx, exit, signals, timers = [], extra = [] }) {return gracefulStop({name: 'Actor', server, deadlineExitCode: 0, exit, signals, deadlineMs: 10_000,-- close: [() => { for (const t of timers) clearInterval(t); }, () => ctx.engine.stop(), () => ctx.webhooks.stop(), () => ctx.chatAsks.stop(), () => ctx.keys.client.stop(), () => ctx.s.close(), ...extra],++ close: [() => { for (const t of timers) clearInterval(t); }, () => ctx.engine.stop(), () => ctx.webhooks.stop(), () => ctx.chatAsks.stop(), () => ctx.keys.client.stop(), async () => { if (ctx.retention) await ctx.retention.stop(); if (ctx.telemetry) await ctx.telemetry.stop(); }, () => ctx.s.close(), ...extra],});}@@@ -41,6 +42,8 @@ async function start() {console.log(`[Actor] ${config.nodeEnv} on http://${config.host}:${config.port} → ${config.baseUrl} (db ${ctx.s.db.store}); agents available: ${available.join(', ') || 'none'}`);});server.keepAliveTimeout = 65_000;++ // telemetry_samples rows older than 30 days go: a minute after start, then daily (openvibe-sdk/telemetry).++ ctx.retention = startTelemetryRetention({ db: ctx.s.db, log: ctx.log || console });// The Network's signing keys, for its tokens: a self-hosted Actor takes none.if (!config.selfHosted.on) ctx.keys.client.start();// Tasks older than the retention window (tasks/store.js RETENTION_DAYS) go, with their events.@@@ -42,7 +42,7 @@ async function boot(opts = {}) {const { createStore } = require('../../server/db');const testdb = await require('./db').testDb();const store = createStore(testdb.db, { now: opts.now });-- const built = await createApp({ config, store, log, limitsNow: opts.limitsNow, callerLimits: opts.callerLimits === true, adapters: opts.adapters, accountSend: opts.accountSend, webhookPost: opts.webhookPost, chatClient: opts.chatClient, secretSigner: opts.secretSigner, bot: opts.bot, projects: opts.projects });++ const built = await createApp({ config, store, log, telemetrySink: opts.telemetrySink, limitsNow: opts.limitsNow, callerLimits: opts.callerLimits === true, adapters: opts.adapters, accountSend: opts.accountSend, webhookPost: opts.webhookPost, chatClient: opts.chatClient, secretSigner: opts.secretSigner, bot: opts.bot, projects: opts.projects });await built.ctx.keys.ensure();const server = await new Promise((resolve) => { const s = http.createServer(built.app); s.listen(0, '127.0.0.1', () => resolve(s)); });const base = `http://127.0.0.1:${server.address().port}`;nnew file mode 100644@@@ -0,0 +1,44 @@++'use strict';++/**++ * Request telemetry (openvibe-sdk/service, plan T1 step 10): app.js builds one collector for service 'actor' with a++ * PostgreSQL sink, a page hit five times flushes as ONE aggregated http.request sample (route template, 2xx, count 5),++ * /metrics and /shared files make none, and the 30-day retention deletes only older rows.++ */++const assert = require('assert');++const { pruneTelemetry } = require('openvibe-sdk/telemetry');++const { boot, check, done } = require('./helpers/boot');++++(async () => {++ const samples = [];++ const t = await boot({ telemetrySink: async (batch) => { samples.push(...batch); } });++ const requests = () => samples.filter((s) => s.operation === 'http.request');++ try {++ await check('five hits on the home page flush as one http.request sample: template, 2xx, count 5', async () => {++ for (let i = 0; i < 5; i++) assert.strictEqual((await t.get('/')).status, 200);++ await t.ctx.telemetry.flush();++ const home = requests().filter((s) => s.resource === '/' && s.status === '2xx');++ assert.strictEqual(home.length, 1, JSON.stringify(requests()));++ assert.strictEqual(home[0].service, 'actor');++ assert.strictEqual(home[0].extra.count, 5);++ });++++ await check('/metrics and a /shared file make no sample', async () => {++ samples.length = 0;++ await t.get('/metrics');++ await t.get('/shared/navbar.js');++ await t.ctx.telemetry.flush();++ assert.deepStrictEqual(requests(), []);++ });++++ await check('the table exists and the retention deletes rows older than 30 days, keeping newer ones', async () => {++ const db = t.ctx.s.db;++ await db.query("DELETE FROM telemetry_samples");++ const insert = (days) => db.query("INSERT INTO telemetry_samples (at, service, operation) VALUES (now() - make_interval(days => $1), 'actor', 'http.request')", [days]);++ await insert(45);++ await insert(1);++ assert.strictEqual(await pruneTelemetry({ db }), 1);++ assert.strictEqual((await db.query('SELECT count(*)::int AS n FROM telemetry_samples')).rows[0].n, 1);++ });++ } finally { await t.close(); }++ done();++})().catch((e) => { console.error(e); process.exit(1); });
new file mode 100644
new file mode 100644
@@ -0,0 +1,28 @@
@@ -0,0 +1,28 @@
+-- phase: expand
+-- Request telemetry (openvibe-sdk/telemetry telemetrySchema()): one row per aggregated sample, pruned after 30 days (server/index.js).
+CREATE TABLE IF NOT EXISTS telemetry_samples (
+ id bigint GENERATED ALWAYS AS IDENTITY PRIMARY KEY,
+ at timestamptz NOT NULL,
+ service text NOT NULL,
+ project text,
+ subject text,
+ resource text,
+ provider text,
+ node text,
+ cell text,
+ region text,
+ operation text NOT NULL,
+ latency_ms double precision,
+ queue_delay_ms double precision,
+ ttfb_ms double precision,
+ throughput_per_second double precision,
+ bytes bigint,
+ status text,
+ cache_status text,
+ cost_estimate double precision,
+ route_epoch bigint,
+ trace_id text,
+ extra jsonb NOT NULL DEFAULT '{}'::jsonb
+);
+CREATE INDEX IF NOT EXISTS telemetry_samples_at_idx ON telemetry_samples (at);
+CREATE INDEX IF NOT EXISTS telemetry_samples_service_idx ON telemetry_samples (service, operation, at);
@@ -58,6 +58,8 @@ const { html } = require('./render/html');
@@ -58,6 +58,8 @@ const { html } = require('./render/html');
const PUBLIC_DIR = path.join(__dirname, '..', 'public');
const PUBLIC_DIR = path.join(__dirname, '..', 'public');
const VERSION = require('../package.json').version;
const VERSION = require('../package.json').version;
+const { createHttpTelemetry } = require('openvibe-sdk/service');
+const { createPgTelemetrySink } = require('openvibe-sdk/telemetry');
/**
/**
* opts: config, store, now (clock), fetchImpl (Network), providerFetch (model providers and OpenVibe services), log,
* opts: config, store, now (clock), fetchImpl (Network), providerFetch (model providers and OpenVibe services), log,
@@ -112,6 +114,11 @@ async function createApp(opts = {}) {
@@ -112,6 +114,11 @@ async function createApp(opts = {}) {
require('./render/layout').setRelease(release.release);
require('./render/layout').setRelease(release.release);
const metrics = require('openvibe-shared/metrics').instrument(app, { service: 'actor', release: release.release });
const metrics = require('openvibe-shared/metrics').instrument(app, { service: 'actor', release: release.release });
app.locals.metrics = metrics.registry;
app.locals.metrics = metrics.registry;
+ // Request telemetry (openvibe-sdk/service, plan T1): one aggregated sample per route template, method and status
+ // class each flush, stored in this service's telemetry_samples table; /shared, health, readiness and /metrics make
+ // none. server/index.js flushes it on stop and keeps the table to 30 days.
+ ctx.telemetry = createHttpTelemetry({ service: 'actor', sink: opts.telemetrySink || createPgTelemetrySink({ db: s.db, log }), skipPrefixes: ['/shared'] });
+ app.use(ctx.telemetry.middleware());
app.locals.ctx = ctx;
app.locals.ctx = ctx;
const valkey = opts.valkey !== undefined ? opts.valkey : (config.valkey.url ? require('openvibe-sdk/valkey').createValkey({ url: config.valkey.url, prefix: config.valkey.prefix, log }) : null);
const valkey = opts.valkey !== undefined ? opts.valkey : (config.valkey.url ? require('openvibe-sdk/valkey').createValkey({ url: config.valkey.url, prefix: config.valkey.prefix, log }) : null);
ctx.valkey = valkey;
ctx.valkey = valkey;
@@ -6,6 +6,7 @@
@@ -6,6 +6,7 @@
*/
*/
const { createApp } = require('./app');
const { createApp } = require('./app');
const { gracefulStop } = require('openvibe-sdk/service');
const { gracefulStop } = require('openvibe-sdk/service');
+const { startTelemetryRetention } = require('openvibe-sdk/telemetry');
const store = require('./tasks/store');
const store = require('./tasks/store');
const PRUNE_EVERY_MS = 6 * 60 * 60_000;
const PRUNE_EVERY_MS = 6 * 60 * 60_000;
@@ -19,7 +20,7 @@ const PRUNE_EVERY_MS = 6 * 60 * 60_000;
@@ -19,7 +20,7 @@ const PRUNE_EVERY_MS = 6 * 60 * 60_000;
function createLifecycle({ server, ctx, exit, signals, timers = [], extra = [] }) {
function createLifecycle({ server, ctx, exit, signals, timers = [], extra = [] }) {
return gracefulStop({
return gracefulStop({
name: 'Actor', server, deadlineExitCode: 0, exit, signals, deadlineMs: 10_000,
name: 'Actor', server, deadlineExitCode: 0, exit, signals, deadlineMs: 10_000,
- close: [() => { for (const t of timers) clearInterval(t); }, () => ctx.engine.stop(), () => ctx.webhooks.stop(), () => ctx.chatAsks.stop(), () => ctx.keys.client.stop(), () => ctx.s.close(), ...extra],
+ close: [() => { for (const t of timers) clearInterval(t); }, () => ctx.engine.stop(), () => ctx.webhooks.stop(), () => ctx.chatAsks.stop(), () => ctx.keys.client.stop(), async () => { if (ctx.retention) await ctx.retention.stop(); if (ctx.telemetry) await ctx.telemetry.stop(); }, () => ctx.s.close(), ...extra],
});
});
}
}
@@ -41,6 +42,8 @@ async function start() {
@@ -41,6 +42,8 @@ async function start() {
console.log(`[Actor] ${config.nodeEnv} on http://${config.host}:${config.port} → ${config.baseUrl} (db ${ctx.s.db.store}); agents available: ${available.join(', ') || 'none'}`);
console.log(`[Actor] ${config.nodeEnv} on http://${config.host}:${config.port} → ${config.baseUrl} (db ${ctx.s.db.store}); agents available: ${available.join(', ') || 'none'}`);
});
});
server.keepAliveTimeout = 65_000;
server.keepAliveTimeout = 65_000;
+ // telemetry_samples rows older than 30 days go: a minute after start, then daily (openvibe-sdk/telemetry).
+ ctx.retention = startTelemetryRetention({ db: ctx.s.db, log: ctx.log || console });
// The Network's signing keys, for its tokens: a self-hosted Actor takes none.
// The Network's signing keys, for its tokens: a self-hosted Actor takes none.
if (!config.selfHosted.on) ctx.keys.client.start();
if (!config.selfHosted.on) ctx.keys.client.start();
// Tasks older than the retention window (tasks/store.js RETENTION_DAYS) go, with their events.
// Tasks older than the retention window (tasks/store.js RETENTION_DAYS) go, with their events.
@@ -42,7 +42,7 @@ async function boot(opts = {}) {
@@ -42,7 +42,7 @@ async function boot(opts = {}) {
const { createStore } = require('../../server/db');
const { createStore } = require('../../server/db');
const testdb = await require('./db').testDb();
const testdb = await require('./db').testDb();
const store = createStore(testdb.db, { now: opts.now });
const store = createStore(testdb.db, { now: opts.now });
- const built = await createApp({ config, store, log, limitsNow: opts.limitsNow, callerLimits: opts.callerLimits === true, adapters: opts.adapters, accountSend: opts.accountSend, webhookPost: opts.webhookPost, chatClient: opts.chatClient, secretSigner: opts.secretSigner, bot: opts.bot, projects: opts.projects });
+ const built = await createApp({ config, store, log, telemetrySink: opts.telemetrySink, limitsNow: opts.limitsNow, callerLimits: opts.callerLimits === true, adapters: opts.adapters, accountSend: opts.accountSend, webhookPost: opts.webhookPost, chatClient: opts.chatClient, secretSigner: opts.secretSigner, bot: opts.bot, projects: opts.projects });
await built.ctx.keys.ensure();
await built.ctx.keys.ensure();
const server = await new Promise((resolve) => { const s = http.createServer(built.app); s.listen(0, '127.0.0.1', () => resolve(s)); });
const server = await new Promise((resolve) => { const s = http.createServer(built.app); s.listen(0, '127.0.0.1', () => resolve(s)); });
const base = `http://127.0.0.1:${server.address().port}`;
const base = `http://127.0.0.1:${server.address().port}`;
new file mode 100644
new file mode 100644
@@ -0,0 +1,44 @@
@@ -0,0 +1,44 @@
+'use strict';
+/**
+ * Request telemetry (openvibe-sdk/service, plan T1 step 10): app.js builds one collector for service 'actor' with a
+ * PostgreSQL sink, a page hit five times flushes as ONE aggregated http.request sample (route template, 2xx, count 5),
+ * /metrics and /shared files make none, and the 30-day retention deletes only older rows.
+ */
+const assert = require('assert');
+const { pruneTelemetry } = require('openvibe-sdk/telemetry');
+const { boot, check, done } = require('./helpers/boot');
+
+(async () => {
+ const samples = [];
+ const t = await boot({ telemetrySink: async (batch) => { samples.push(...batch); } });
+ const requests = () => samples.filter((s) => s.operation === 'http.request');
+ try {
+ await check('five hits on the home page flush as one http.request sample: template, 2xx, count 5', async () => {
+ for (let i = 0; i < 5; i++) assert.strictEqual((await t.get('/')).status, 200);
+ await t.ctx.telemetry.flush();
+ const home = requests().filter((s) => s.resource === '/' && s.status === '2xx');
+ assert.strictEqual(home.length, 1, JSON.stringify(requests()));
+ assert.strictEqual(home[0].service, 'actor');
+ assert.strictEqual(home[0].extra.count, 5);
+ });
+
+ await check('/metrics and a /shared file make no sample', async () => {
+ samples.length = 0;
+ await t.get('/metrics');
+ await t.get('/shared/navbar.js');
+ await t.ctx.telemetry.flush();
+ assert.deepStrictEqual(requests(), []);
+ });
+
+ await check('the table exists and the retention deletes rows older than 30 days, keeping newer ones', async () => {
+ const db = t.ctx.s.db;
+ await db.query("DELETE FROM telemetry_samples");
+ const insert = (days) => db.query("INSERT INTO telemetry_samples (at, service, operation) VALUES (now() - make_interval(days => $1), 'actor', 'http.request')", [days]);
+ await insert(45);
+ await insert(1);
+ assert.strictEqual(await pruneTelemetry({ db }), 1);
+ assert.strictEqual((await db.query('SELECT count(*)::int AS n FROM telemetry_samples')).rows[0].n, 1);
+ });
+ } finally { await t.close(); }
+ done();
+})().catch((e) => { console.error(e); process.exit(1); });