Skip to content

Conversation

@Arvindthiru
Copy link
Contributor

@Arvindthiru Arvindthiru commented Apr 4, 2025

What this PR does / why we need it:

Fixing CVEs,

Requirements:

How has this code been tested

Special notes for your reviewer

@kaito-pr-agent
Copy link

kaito-pr-agent bot commented Apr 4, 2025

Title

fix: bump go version to 1.23.6 for CVE


User description

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Requirements:

How has this code been tested

Special notes for your reviewer


PR Type

bug_fix


Description

  • Bumps Go version to 1.23.6 across multiple workflows and Dockerfiles for CVE.

  • Updates Go version in go.mod to match the new workflow configuration.


Changes walkthrough 📝

Relevant files
Bug_fix
3 files
unit-integration-tests.yml
Update Go version to 1.23.6                                                           
member-net-controller-manager.Dockerfile
Update Go version to 1.23.6                                                           
go.mod
Update Go version to 1.23.6                                                           
+1/-1     
Additional files
9 files
build-publish-mcr.yml +1/-1     
e2e-tests.yml +1/-1     
go.yml +1/-1     
publish-image.yml +1/-1     
trivy.yml +1/-1     
unit-integration-tests.yml +1/-1     
hub-net-controller-manager.Dockerfile +1/-1     
mcs-controller-manager.Dockerfile +1/-1     
member-net-controller-manager.Dockerfile +1/-1     

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @kaito-pr-agent
    Copy link

    kaito-pr-agent bot commented Apr 4, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 PR contains tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @kaito-pr-agent
    Copy link

    kaito-pr-agent bot commented Apr 4, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @codecov
    Copy link

    codecov bot commented Apr 4, 2025

    Codecov Report

    All modified and coverable lines are covered by tests ✅

    Project coverage is 79.99%. Comparing base (5c38041) to head (22c80ce).
    Report is 1 commits behind head on main.

    Additional details and impacted files
    @@            Coverage Diff             @@
    ##             main     #286      +/-   ##
    ==========================================
    - Coverage   80.49%   79.99%   -0.50%     
    ==========================================
      Files          29       29              
      Lines        4024     4024              
    ==========================================
    - Hits         3239     3219      -20     
    - Misses        625      640      +15     
    - Partials      160      165       +5     

    ☔ View full report in Codecov by Sentry.
    📢 Have feedback on the report? Share it here.

    🚀 New features to boost your workflow:
    • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

    Copy link
    Contributor

    @zhiying-lin zhiying-lin left a comment

    Choose a reason for hiding this comment

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

    Please update your PR description to include the CVE info

    @Arvindthiru Arvindthiru merged commit f361fc6 into Azure:main Apr 7, 2025
    10 of 11 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants