Opened 6 months ago

Closed 5 months ago

#29597 closed task (implemented)

Cleanup bridgedb-admin git repository

Reported by: dgoulet Owned by: dgoulet
Priority: Medium Milestone:
Component: Circumvention/BridgeDB Version:
Severity: Normal Keywords:
Cc: cohosh Actual Points:
Parent ID: Points:
Reviewer: sysrqb Sponsor:


It is full of either out of date scripts or thing that aren't used.

The branch will probably have many commits touching many things ;). Spring cleanup!

Child Tickets

Change History (3)

comment:1 Changed 6 months ago by dgoulet

Status: assignedneeds_review

Branch: ticket29597_01

comment:2 Changed 6 months ago by sysrqb

Status: needs_reviewneeds_revision


  • At the top of the file: "This is bring run by the bridge authority", "being run"? We can probably simply say "This is run by".


  • Does this script use any bash-isms? I'm not sure if requiring bash is necessary.
  • The new lines use the wrong indentation.
    -       --exclude from-tonga-latest.tar.gz \
    -       --exclude from-bifroest-latest.tar.gz \
    +       --exclude from-${AUTHORITY_NAME}-latest.tar.gz \
            --remove-source-files \
            . \
    +  perl -lane 'print time() . ":descriptors:" . $_' >> $LOGDIR/sync-to-colchicifolium.log
    @@ -25,5 +38,6 @@ if test "$SYNC_ASSIGNMENTS" = "true" ; then
         rsync \
            -a \
            assignments.log* \
    +  perl -lane 'print time() . ":descriptors:" . $_' >> $LOGDIR/sync-to-colchicifolium.log

The other changes lgtm.

lgtm. I don't know if we'll want something like this in the future. We probably do, but this script probably doesn't work now due to other bridgedb changes over the last 8 years, and we'll probably want something more maintainable and scalable than a single python script, too.

5eb94dd19eaaa7b1cc470c1641332e2cf44b80f6: lgtm
c00e188bdd8d130ed981741095ae0ef6e6c22b9d: This wasn't previously publicly available, but I don't see the harm in publishing it. lgtm.
c8541173da02717ac9e788c20c3ba4c1a9f4e508: heh. lgtm.
970dc485f20e710592cf12ca177ba36058264851: lgtm.

comment:3 Changed 5 months ago by sysrqb

Resolution: implemented
Status: needs_revisionclosed

merged with mentioned fixups.

Note: See TracTickets for help on using tickets.