Skip to content

fix: rm failing pelican - #280

Closed
wdconinc wants to merge 1 commit into
masterfrom
rm-pelican
Closed

fix: rm failing pelican#280
wdconinc wants to merge 1 commit into
masterfrom
rm-pelican

Conversation

@wdconinc

Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

Pelican fails. Remove pelican.

What is the urgency of this PR?

  • High (please describe reason below)
  • Medium
  • Low

What kind of change does this PR introduce?

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

@wdconinc
wdconinc requested review from a team and Copilot April 27, 2026 15:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitLab CI pipeline to stop using the Pelican binary in the deploy_results job, addressing a failing deployment step referenced in the linked job.

Changes:

  • Removed Pelican download/verification and pelican object copy upload steps from deploy_results.
  • Left deploy_results to only generate results/metadata.json and a summary.txt listing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .gitlab-ci.yml
@veprbl

veprbl commented Apr 27, 2026

Copy link
Copy Markdown
Member

This is not because of pelican, but because tokens are not renewed right now.

@wdconinc

Copy link
Copy Markdown
Contributor Author

This is not because of pelican, but because tokens are not renewed right now.

Nevertheless it hasn't worked in about half a year.

@veprbl

veprbl commented Apr 27, 2026

Copy link
Copy Markdown
Member

Not fair, but okay. We have another use of pelican for low-Q2 tagger by @simonge, for that we have not yet agreed on transition to rucio, and I don't have a token for rucio renewed for benchmarks yet. This one is useless for validation (cc @torrijeske), but at least it's a canary for me to kick tires on token renewal.

@torrijeske

Copy link
Copy Markdown

Sorry - do we want to keep this benchmark if it’s “useless for validation?”

@wdconinc

Copy link
Copy Markdown
Contributor Author

I don't feel strongly about this. I was just doing cleanup, and this has been failing for a long time so I figured it was defunct. If you want to keep it to fix it, then that's fine with me.

@wdconinc wdconinc closed this Apr 30, 2026
@wdconinc
wdconinc deleted the rm-pelican branch April 30, 2026 17:40
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.

4 participants