Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • L LHCb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 193
    • Issues 193
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 44
    • Merge requests 44
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • LHCb
  • Merge requests
  • !3774

Draft: auto flush `key_registry` to git repo

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Gerhard Raven requested to merge auto-flush-to-git into master Sep 15, 2022
  • Overview 20
  • Commits 2
  • Pipelines 5
  • Changes 2
  • make sure new decodings are guaranteed to be flushed to git (unless explicitly opted out):

    previously, an explicit flush_to_git had to be called to make sure newly generated decoding keys were written to git -- with a warning in the __del__ of the key registry if that was not done. Note that only a warning was produced, as writing to git from __del__ was not possible, as was called when python was shutting down. So instead, this functionality has been moved to an 'exit handler' which is registered with the atexit module.

    note: there is a flag in the options to explicitly opt-out of writing to git which is still (of course) respected -- but if doing would lead to a total loss of keys (i.e. when they are also not written to the output manifest) a warning is printed.

Edited Sep 16, 2022 by Gerhard Raven
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: auto-flush-to-git