Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • autosubmit autosubmit
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 338
    • Issues 338
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 21
    • Merge requests 21
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Earth SciencesEarth Sciences
  • autosubmitautosubmit
  • Merge requests
  • !456

Draft: Fix critical error in Monitor due to Profiler

  • Review changes

  • Download
  • Patches
  • Plain diff
Closed Pablo Goitia requested to merge profiler-fix into master Jul 10, 2024
  • Overview 0
  • Commits 1
  • Pipelines 0
  • Changes 3

This merge request aims to fix the error in issue #1123 (closed). The solution was to split the error handling in the stop() function in order to support the multiple calls in the monitor() function, and to move the profiler.stop() calls in Monitor from finally to except, in order to avoid finishing the profiling process early.

It also introduces a more comprehensive way to manage the status of the Profiler, using the status class as @bdepaula proposed and deleting the old flags.

Edited Jul 10, 2024 by Pablo Goitia
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: profiler-fix