Skip to content

Manage Cockpit incident annotations#85

Open
hauptmedia wants to merge 1 commit into
mainfrom
codex/cockpit-incident-annotations
Open

Manage Cockpit incident annotations#85
hauptmedia wants to merge 1 commit into
mainfrom
codex/cockpit-incident-annotations

Conversation

@hauptmedia

Copy link
Copy Markdown

Summary

  • add an Incident REST resource for runtime incident lists and annotation updates
  • use runtime incidents in live Cockpit incident tabs while keeping history mode read-only
  • let operators save and clear incident annotations from definition and instance incident tables

Reference

Verification

  • node -e 'const fs=require("fs"); for (const locale of fs.readdirSync("public/locales")) JSON.parse(fs.readFileSync("public/locales/"+locale+"/translation.json", "utf8")); console.log("locale JSON ok")'\n- npx eslint src/api/resources/incident.js src/api/resources/incident.test.js src/api/engine_rest.jsx src/state.js src/pages/Processes.jsx src/pages/Processes.test.jsx\n- npx vitest run src/api/resources/incident.test.js src/pages/Processes.test.jsx\n- TZ=UTC npx vitest run --maxWorkers=4\n- npm run build\n- git diff --check\n- browser smoke at http://127.0.0.1:5175/processes/proc:1/incidents: reached login boundary with title Operaton and no console errors\n\nNote: a plain npx vitest run --maxWorkers=4 at local 00:00 CEST hit the existing time-sensitive date_formatter.test 12-hour subtraction flake; the stable TZ=UTC full run above passed.

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant