Skip to content

281 user friendly decommutation of received reports#332

Open
d-linko wants to merge 53 commits intomainfrom
281-user-friendly-decommutation-of-received-reports
Open

281 user friendly decommutation of received reports#332
d-linko wants to merge 53 commits intomainfrom
281-user-friendly-decommutation-of-received-reports

Conversation

@d-linko
Copy link
Copy Markdown
Contributor

@d-linko d-linko commented Apr 3, 2026

Overhauled querying and structures used for pulling reports from the database and responding to REST calls
The rpt items are stored individual as CBOR in database and Transcoder is used to convert to string representation.
Organized reports based off report source and will send back a list of reports that are organized based off the report definition.

DavidEdell and others added 30 commits November 29, 2025 16:42
…ostgresSQL, consolidated dashboard+widget definitions and moved to a text provisioning file, and updated anms-ui monitor to use the new dashboard. Address #282 and #283
…nts the formal release process which also triggers publication.
…ols to bring in fixes for SQL concurrency causing reliability issues.
@d-linko d-linko linked an issue Apr 3, 2026 that may be closed by this pull request
@github-project-automation github-project-automation bot moved this to Todo in ANMS Apr 3, 2026
@d-linko d-linko requested a review from a team as a code owner April 3, 2026 19:27
@d-linko d-linko marked this pull request as draft April 3, 2026 19:30
@d-linko d-linko marked this pull request as ready for review April 7, 2026 16:48
@d-linko d-linko self-assigned this Apr 7, 2026
Copy link
Copy Markdown
Collaborator

@BrianSipos BrianSipos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not dig through changes in detail, is there expected to be an update to the grafana query style to use the item table and item position index more explicitly?

Comment thread anms-core/anms/components/schemas/ARIs/rpt_entry.py Outdated
@d-linko
Copy link
Copy Markdown
Contributor Author

d-linko commented Apr 10, 2026

I did not dig through changes in detail, is there expected to be an update to the grafana query style to use the item table and item position index more explicitly?

not explicitly it is pulling the data from view vw_ari_rpt_set(either using the rest endpoint or straight from the database) which is ordering the items by the index

@d-linko d-linko requested a review from BrianSipos April 10, 2026 18:01
Copy link
Copy Markdown
Collaborator

@BrianSipos BrianSipos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One suggested search-and-replace to make a field meaning more clear from its name.

@@ -39,7 +39,6 @@ class Config:
time_offset: Optional[datetime] = None
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This applies to other cases of this term elsewhere also: If this is the computed timestamp of the individual report, it would be better to call this "agent_time" because it is the time point at which the report was created on the Agent. Separate from the "mgr_time" which is the time the message was received on the Manager.

@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

Status: Todo

Development

Successfully merging this pull request may close these issues.

User friendly decommutation of received reports

3 participants