Skip to content

Conversation

@daues
Copy link
Contributor

@daues daues commented Jan 15, 2026

No description provided.

Copy link

@MichelleGower MichelleGower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple minor requests. Merge approved.

else
echo "RHEL8 standard"
export RELEASE_DIR=/sdf/group/rubin/sw/htcondor
fi

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if RELEASE_DIR is printed elsewhere. If not, please print it too.

export _condor_CONDOR_HOST=${RUBIN_COLLECTOR_HOST}
export _condor_COLLECTOR_HOST=${RUBIN_COLLECTOR_HOST}
export _condor_CCB_ADDRESS=${RUBIN_COLLECTOR_HOST}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to have a comment about this section. Something like getting os major version to chose matching RELEASE_DIR.

@daues daues merged commit 4bc2415 into main Jan 16, 2026
6 checks passed
@daues daues deleted the tickets/DM-53801 branch January 16, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants