Skip to content

Conversation

rajmishra1997
Copy link
Contributor

@rajmishra1997 rajmishra1997 commented Aug 26, 2025

Context

Explain the context or motivation behind this PR. Include links to any related Azure DevOps Work Items or GitHub issues.
📌 How to link to ADO Work Items
Upgrades Git for windows version, as version 2.49.0 contains a regression.
Fixes #5293
Git Issue: git-for-windows/git#5427


Description

Provide a concise summary of the changes introduced in this PR.
Git issue is fixed in Git version 2.50.0, hence upgraded version to latest version 2.51.0


Risk Assessment (Low / Medium / High)

Assess the risk level and justify your assessment. For example: code path sensitivity, usage scope, or backward compatibility concerns.
Medium

Unit Tests Added or Updated (Yes / No)

Indicate whether unit tests were added or modified to reflect the changes.
No

Additional Testing Performed

_List manual or automated tests performed beyond unit tests (e.g., integration, scenario, regression)._Tested locally by building locally, git 51.0 is fetched successfully
buildImage


Change Behind Feature Flag (Yes / No)

Can this change be behind feature flag, if not why?
No, changes are limited to an .sh file

Tech Design / Approach

  • Design has been written and reviewed.
  • Any architectural decisions, trade-offs, and alternatives are captured.
    No, not required

Documentation Changes Required (Yes/No)

Indicate whether related documentation needs to be updated.

  • User guides, API specs, system diagrams, or runbooks are updated.
    No

Logging Added/Updated (Yes/No)

  • Appropriate log statements are added with meaningful messages.
  • Logging does not expose sensitive data.
  • Log levels are used correctly (e.g., info, warn, error).
    No

Telemetry Added/Updated (Yes/No)

  • Custom telemetry (e.g., counters, timers, error tracking) is added as needed.
  • Events are tagged with proper metadata for filtering and analysis.
  • Telemetry is validated in staging or test environments.
    No

Rollback Scenario and Process (Yes/No)

  • Rollback plan is documented.
    No

Dependency Impact Assessed and Regression Tested (Yes/No)

  • All impacted internal modules, APIs, services, and third-party libraries are analyzed.
  • Results are reviewed and confirmed to not break existing functionality.

@rajmishra1997 rajmishra1997 requested review from a team as code owners August 26, 2025 08:59
@rajmishra1997
Copy link
Contributor Author

rajmishra1997 commented Aug 26, 2025

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azat-galiev
Copy link

Hi @rajmishra1997,

JFYI: we in Azure Repos team would also appreciate upgrading the version to the latest. There was a fix in 2.50 that is blocking compression of certain repos in ADO.

"git index-pack --fix-thin" used to abort to prevent a cycle in delta chains from forming in a corner case even when there is no such cycle.

@rajmishra1997
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rajmishra1997
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rajmishra1997
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rishabhmalikMS
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tarunramsinghani
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rajmishra1997
Copy link
Contributor Author

PR uses fork repository branch, which causes PR checks to fail (Token cannot be accessed by forked repository) hence closing this PR. Another PR: #5311

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Question]: Getting git error after updating agents from 4.258.1 to 4.259.0
4 participants