Skip to main content

Hi! 

I’m testing github protection. Repository part works well (it generates simple git clone script). Later it starts API call to get “issues” and “pull request” via API : 

79957 13866 09/23 10:32:01 749 CGitHubOperations::AddPullRequestCommentsAndReviewsToFile(605) - URL ehttps://api.github.com/repos/ble/ble/issues/2121/comments?sort=updated&per_page=100]
g...]
79957 13866 09/23 10:42:28 749 CGitHubOperations::AddPullRequestCommentsAndReviewsToFile(605) - URL ehttps://api.github.com/repos/ble/ble/pulls/743/reviews?per_page=100]

Random production repo (~2500 commits) saturates hourly API limit for github enterprise : 

588889 8fc75 09/25 11:15:13 5087419 CGitHubConnection::ExecuteRequestWithRetry(58) - The GitHub API Interface has reached the rate limit
588889 8fc75 09/25 11:15:13 5087419 CGitConnection::SleepBeforeRetry(221) - API rate limit reset header:iX-RateLimit-Reset: 1695633798]
588889 8fc75 09/25 11:15:13 5087419 CGitConnection::SleepBeforeRetry(230) - API rate limit reset time:i1695633798]
588889 8fc75 09/25 11:15:13 5087419 CGitConnection::SleepBeforeRetry(241) - Will retry after 485 seconds

Two questions : 

  • is high API call usage expected? No option to use some kind of bulk requests? Now there is a loop for each commit. 
  • is there an option to disable API part (no issues and pull requests detail) and protect only raw repository data? 

 

Thanks. 

Good day Iborek,

I apologize for the delay in responding to your query. We are currently in the process of arranging an engineering review for your question. Please rest assured that we are working diligently to provide you with a detailed response as soon as possible.

Thank you for your patience and understanding.

Hi lborek,
 

Yes, it’s expected when there are high number of pull requests attached to the repository. For every pull request we have to send a unique API request to get the comments and reviews associated with it. We can’t club multiple pull requests into one API call as it’s not supported by Github.

We don’t have the option to disable API part. Pull requests are backed up as part of the repository itself.

Thanks,
Mani

Hi @lborek, just to keep you aware we are working with your colleague Lukkasz on the same query in support incident # 230928-327. Thanks.

Hi @lborek , Created a CMR on this, CMR: 411340 

 

Hi, 

Can i ask you a favor to check where we are with this CMR? When it will be available? 

Thanks. 

CMR 411340 was created october 9th 2023.. so don’t hold your breath =)

Fix is ready for 11:34

https://documentation.commvault.com/additionalsetting/details?name=bSkipGitPullRequests

 

 

Do the work. Thanks. 

Anyone confirm this is only for 11.34?
We are currently on 11.32.54

Supported Versions 11.34 and higher

yes, I agree it says so in the link, but maybe there is something in the works or it’s not the true truth.
It happens.

Reply


Terms & ConditionsCookie settings