Skip to content

Conversation

@alireza-pazooki
Copy link
Contributor

@alireza-pazooki alireza-pazooki commented Oct 20, 2025

Context

  • patch api upgrade
  • better naming for script vars
  • remove census from script as not needed anymore

Checklist

@alireza-pazooki alireza-pazooki requested a review from a team as a code owner October 20, 2025 11:09
Copilot AI review requested due to automatic review settings October 20, 2025 11:09
@platon-github-app-production
Copy link

Comment /request-review to automatically request reviews from the following teams:

You can also request review from a specific team by commenting /request-review team-name, or you can add a description with --notes "<message>"

💡 If you see something that doesn't look right, check the configuration guide.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades Envoy to version 1.34.10, improving script variable naming and updating API dependencies. The upgrade involves updating version hashes, renaming variables for consistency, and removing deprecated OpenCensus dependencies.

  • Updated Envoy SHA from 6fe1905459ff267a43a8a26d042ae03a8aa7bc98 to 40ab4828a48f05103573ddee3c807dfa0e3e23f7
  • Renamed variables from *_SHA to *_VERSION for better naming consistency
  • Removed OpenCensus proto files and dependencies as they are no longer needed

Reviewed Changes

Copilot reviewed 36 out of 36 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tools/update-sha.sh Updated variable names from SHA to VERSION for better consistency
tools/update-api.sh Updated variable references and removed OpenCensus dependency
tools/API_SHAS Updated Envoy version hash and dependency versions, removed date comments
src/main/proto/**/*.proto Updated proto files with new API versions and removed deprecated files
gradle.properties Bumped version from 1.34.4-1 to 1.34.10-1
Comments suppressed due to low confidence (1)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@niting3c niting3c merged commit e36c39e into main Oct 20, 2025
7 checks passed
@niting3c niting3c deleted the envoy-patch-upgrade-1-34-10 branch October 20, 2025 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants