Skip to content

Conversation

@LoopedBard3
Copy link
Member

dotnet/performance#2622 adds the ability to pass the timestamp of the runtime commit to ci_setup instead of calling the GitHub API to retrieve it. This will keep us from hitting GitHub API rate limiting during our runs.

@ghost
Copy link

ghost commented Sep 29, 2022

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

dotnet/performance#2622 adds the ability to pass the timestamp of the runtime commit to ci_setup instead of calling the GitHub API to retrieve it. This will keep us from hitting GitHub API rate limiting during our runs.

Author: LoopedBard3
Assignees: LoopedBard3
Labels:

area-Infrastructure-coreclr

Milestone: -

@DrewScoggins
Copy link
Member

LGTM. Have you run a private run of this?

@LoopedBard3
Copy link
Member Author

Yes, the private run successfully ran using this with the performance branch.

@LoopedBard3 LoopedBard3 marked this pull request as ready for review September 30, 2022 22:47
Copy link
Member

@DrewScoggins DrewScoggins left a comment

Choose a reason for hiding this comment

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

LGTM

@LoopedBard3 LoopedBard3 merged commit 23697bb into dotnet:main Oct 3, 2022
@LoopedBard3 LoopedBard3 deleted the PerfGetCommitTimeFromLocalRuntime branch October 3, 2022 16:48
LoopedBard3 added a commit to LoopedBard3/runtime that referenced this pull request Oct 31, 2022
)

* Adds passing of locally obtained git commit time to ci_setup allowing for the bypassing of calling the github.com api.

* Added use local commit time to run-scenarios yml.

* Fixed uselocalcommittime format for linux setup.

* Use correct quote style for passing time to ci_setup.py.
@ghost ghost locked as resolved and limited conversation to collaborators Nov 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants