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
  • !515

Autosubmit GUI issue 113: Vertical DB fixes, and add regression test

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Bruno de Paula Kinoshita requested to merge 475-vertical-regression-test-2 into master Nov 18, 2024
  • Overview 20
  • Commits 1
  • Pipelines 0
  • Changes 14

Related to AS GUI issue 113: autosubmit-gui#113

Supersedes part of !475 which was added in !490 (merged). @dbeltran I hand-picked the DB changes and the local wrappers changes. There were some tests added for log retrieval which I also did not include, as they depended on changes in !475. However, if this gets merged, then we can just rebase that branch and the test should stay there (Git magic 🎉 🤞 at least in theory, we can test locally before merging).

Can you review if these changes include the DB fixes? I started by picking the test/unit/test_database_regression.py, which did not pass. Then I slowly added files, and only had issues with job_list.py, had to ignore test/unit/test_log_recovery.py as mentioned above, and ignored part of the changes in autosubmit/autosubmit.py (the part about process recovery).

If there are other ways to verify if the AS GUI issue with DB values and wrappers/chunks, I can give it a try too to better understand the problem and update this branch.

Note, that I created a kind of Frankenstein commit with ~95% of the changes in that branch, but I edited the authorship to keep it as Dani's commits (I didn't write any code here 🙂).

If we can merge this one first, and rebase the zombies, then it'll be a lot easier to review the rest of that PR.

Thanks!

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 475-vertical-regression-test-2