Skip to content
Snippets Groups Projects

Corrected pims_split templates

Merged Daniel Nemergut requested to merge htc10_wf_template_fixes into 2.8.3-DEVELOPMENT
1 unresolved thread

Fixed split.condor for HTC10 by removing the quotes around an already quoted variable. I didn't find any other instances of this happening in other template files.

Also brought over update_stage commands into split.sh that disappeared sometime since 2.8.1.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
5 Create Date: 2024-02-29 12:37:02.423565
6
7 """
8 from alembic import op
9 import sqlalchemy as sa
10
11
12 # revision identifiers, used by Alembic.
13 revision = 'd992b379d8a7'
14 down_revision = '36591dc3f14d'
15 branch_labels = None
16 depends_on = None
17
18 # Remove quotes around string variables that have already been quoted
19 new_split_condor = """executable = split.sh
20 arguments = "$(split_dir)"{{#quicklook_url}} "$(quicklook_url)"{{/quicklook_url}}
  • Charlotte Hausman approved this merge request

    approved this merge request

  • Daniel Nemergut added 1 commit

    added 1 commit

    • 6ab3a8f6 - Moved the pims_split workflow templates into text files to make them simpler to maintain

    Compare with previous version

  • Charlotte Hausman
  • Daniel Nemergut added 1 commit

    added 1 commit

    • 699d5a66 - Update stage was also missing from the restore and finish scripts, also...

    Compare with previous version

  • Daniel Nemergut added 1 commit

    added 1 commit

    • cc48621a - Removing the quotes around the split_dir in the arguments, just in case. Seems to work

    Compare with previous version

  • Charlotte Hausman unapproved this merge request

    unapproved this merge request

  • Charlotte Hausman approved this merge request

    approved this merge request

  • Brittany Faciane approved this merge request

    approved this merge request

  • Daniel Nemergut mentioned in commit a19af00d

    mentioned in commit a19af00d

  • Please register or sign in to reply
    Loading