Skip to content

Conversation

@vicentebolea
Copy link
Collaborator

No description provided.

@vicentebolea
Copy link
Collaborator Author

@lindstro the status check works!

@vicentebolea
Copy link
Collaborator Author

It appears that it is posting the status as comming from your account. Make sure that it is using the robot token.

@lindstro
Copy link
Member

It appears that it is posting the status as comming from your account. Make sure that it is using the robot token.

Hmm, that's odd. I created another token from the robot account and updated. Of course, the robot is not admin, so I had to update the token through my personal account. Not sure if that has anything to do with it.

@vicentebolea
Copy link
Collaborator Author

Of course, the robot is not admin, so I had to update the token through my personal account. Not sure if that has anything to do with it

It should not be

@vicentebolea
Copy link
Collaborator Author

@lindstro oh the new token is not associated to your account! I think that we completed this task! :)

@vicentebolea
Copy link
Collaborator Author

Actually Ill go ahead and change the cdash job to another name so that it is not confusing.

@vicentebolea vicentebolea changed the title test cdash status check ci,cdash: rename cdash workflow to external Mar 12, 2025
@vicentebolea
Copy link
Collaborator Author

@lindstro I reused this PR to rename the cdash workflow to external and the cdash job to generate_statuses. This will avoid the confusion of having two status checks named cdash.

@vicentebolea vicentebolea requested a review from lindstro March 12, 2025 23:30
@vicentebolea vicentebolea self-assigned this Mar 12, 2025
Copy link
Collaborator

@zfp-gh-robot zfp-gh-robot left a comment

Choose a reason for hiding this comment

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

LGTM

@vicentebolea vicentebolea merged commit 9f065eb into llnl:staging Mar 12, 2025
6 of 7 checks passed
@vicentebolea vicentebolea deleted the test-cdash-status branch March 12, 2025 23:41
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