Skip to content

Allow setting record options externally#3789

Merged
jacomago merged 1 commit intoControlSystemStudio:masterfrom
jacomago:dbemask-public
Apr 20, 2026
Merged

Allow setting record options externally#3789
jacomago merged 1 commit intoControlSystemStudio:masterfrom
jacomago:dbemask-public

Conversation

@jacomago
Copy link
Copy Markdown
Contributor

Checklist

  • Testing:

    • The feature has automated tests
    • Tests were run
    • If not, explain how you tested your changes
  • Documentation:

    • The feature is documented
    • The documentation is up to date
    • Release notes:
      • Added an entry if the change is breaking or significant
      • Added an entry when adding a new feature

@jacomago jacomago requested a review from kasemir April 20, 2026 14:54
@jacomago jacomago self-assigned this Apr 20, 2026
@shroffk
Copy link
Copy Markdown
Member

shroffk commented Apr 20, 2026

Is this a requirement from archiver appliance

@sonarqubecloud
Copy link
Copy Markdown

@jacomago jacomago merged commit 7e3fbdf into ControlSystemStudio:master Apr 20, 2026
4 checks passed
@jacomago
Copy link
Copy Markdown
Contributor Author

Is this a requirement from archiver appliance

Yes! But also maybe alarms...

@kasemir
Copy link
Copy Markdown
Collaborator

kasemir commented Apr 21, 2026

...also maybe alarms...

We started out running the alarm server with

org.phoebus.pv.ca/monitor_mask=ALARM

Seemed logical, after all the alarm server only cares about changes in the alarm state.
But that means you won't see the current value of PVs when you list them in the alarm server console, since only changes that come with different alarm states will get through.
Also means that expressions for alarm filters will likely not get the expected behavior since they miss most value updates. OK, alarm filters were a bad idea anyway, should create appropriate alarm trigger PVs on the IOC in the first place. Still, switching monitors from VALUE to ALARM tends to create emails from surprised users.

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.

3 participants