Skip to main content
commvault.com commvault.com
Question

Github - protect only repositories (skip metadata)

L
Byte
Forum|alt.badge.img+7

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 [https://api.github.com/repos/ble/ble/issues/2121/comments?sort=updated&per_page=100]
[...]
79957 13866 09/23 10:42:28 749 CGitHubOperations::AddPullRequestCommentsAndReviewsToFile(605) - URL [https://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:[X-RateLimit-Reset: 1695633798]
588889 8fc75 09/25 11:15:13 5087419 CGitConnection::SleepBeforeRetry(230) - API rate limit reset time:[1695633798]
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. 

    11 replies

    P
    Vaulter
    Forum|alt.badge.img+1

    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.

    M
    Vaulter
    Forum|alt.badge.img
    • Mani GoyalVaulter
    • Vaulter
    • 2 replies
    • October 4, 2023

    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

    P
    Vaulter
    Forum|alt.badge.img+1

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

    S
    Vaulter
    Forum|alt.badge.img+4
    • Syed_SVaulter
    • Vaulter
    • 21 replies
    • October 9, 2023

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

     

    L
    Byte
    Forum|alt.badge.img+7
    • lborek
    • Author
    • Byte
    • 49 replies
    • June 3, 2024

    Hi, 

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

    Thanks. 

    No special characters
    1 person likes this
    • No special characters
      No special characters
    No special characters
    Byte
    Forum|alt.badge.img+8

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

    L
    Byte
    Forum|alt.badge.img+7
    • lborek
    • Author
    • Byte
    • 49 replies
    • June 5, 2024

    Fix is ready for 11:34

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

     

     

    Do the work. Thanks. 

    No special characters
    1 person likes this
    • No special characters
      No special characters
    No special characters
    Byte
    Forum|alt.badge.img+8

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

    L
    Byte
    Forum|alt.badge.img+7
    • lborek
    • Author
    • Byte
    • 49 replies
    • June 5, 2024
    Supported Versions 11.34 and higher
    No special characters
    1 person likes this
    • No special characters
      No special characters
    No special characters
    Byte
    Forum|alt.badge.img+8

    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.

    L
    Byte
    Forum|alt.badge.img+7
    • lborek
    • Author
    • Byte
    • 49 replies
    • December 5, 2024

    Related question.  It there an option to change concurrent backup streams?  Looks like now 4 repos is protected same time. Is it possible to increase that number? 

     

    Thanks 

    Reply


    Most helpful members this week

    Terms & ConditionsCookie settings

    Cookie policy

    We use cookies to enhance and personalize your experience. If you accept you agree to our full cookie policy. Learn more about our cookies.

     
    Cookie settings