Skip to content

Change udpa renaming workaround to not compile the same archive twice#32647

Merged
ravenblackx merged 2 commits intoenvoyproxy:mainfrom
ravenblackx:udpa
Mar 1, 2024
Merged

Change udpa renaming workaround to not compile the same archive twice#32647
ravenblackx merged 2 commits intoenvoyproxy:mainfrom
ravenblackx:udpa

Conversation

@ravenblackx
Copy link
Copy Markdown
Contributor

Commit Message: Change udpa renaming workaround to not compile the same archive twice
Additional Description: I copied the prior workaround into a different build environment and it caused linker errors in one subset of the space, which made it clear that the repo was downloaded and compiled twice in the bazel cache under different paths. This change fixes that.
Risk Level: None - if it builds it's better.
Testing: If it builds it ships.
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Signed-off-by: Raven Black <ravenblack@dropbox.com>
@repokitteh-read-only
Copy link
Copy Markdown

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #32647 was opened by ravenblackx.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Feb 29, 2024
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @RyanTheOptimist

🐱

Caused by: #32647 was opened by ravenblackx.

see: more, trace.

@ravenblackx
Copy link
Copy Markdown
Contributor Author

/retest

Signed-off-by: Raven Black <ravenblack@dropbox.com>
phlax
phlax previously approved these changes Mar 1, 2024
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @ravenblackx

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Mar 1, 2024
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Mar 1, 2024
@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Mar 1, 2024
@ravenblackx ravenblackx marked this pull request as ready for review March 1, 2024 16:13
@ravenblackx ravenblackx enabled auto-merge (squash) March 1, 2024 16:14
@ravenblackx ravenblackx merged commit 0d3bdfe into envoyproxy:main Mar 1, 2024
@ravenblackx ravenblackx deleted the udpa branch March 1, 2024 21:58
mattjo added a commit to mattjo/envoy that referenced this pull request Mar 5, 2024
* main: (151 commits)
  http3: fixing an upstream threading issue and bumping http3 upstream code back to alpha (envoyproxy#32640)
  rlqs: reset quota usage (envoyproxy#32569)
  proxy status: add more mapping to proxystatus (envoyproxy#32606)
  rlqs: add logging around token bucket (envoyproxy#32612)
  Logging: ENVOY_BUG include filter name. (envoyproxy#32663)
  mobile: Reenable the FilterIntegrationTest.AltSvcCachedH2Slow test (envoyproxy#32675)
  c++20 by default (envoyproxy#32585)
  Docs: Add diagram for histogram stat sink. (envoyproxy#32665)
  Fix null node for list of struct in payload_to_metadaata filter (envoyproxy#32309)
  metrics_service: populate histogram summary sample sum (envoyproxy#32666)
  build(deps): bump postgres from `0e564da` to `f58300a` in /examples/shared/postgres (envoyproxy#32632)
  build(deps): bump the examples-ext-authz group in /examples/ext_authz with 1 update (envoyproxy#32654)
  build(deps): bump distroless/base-nossl-debian12 from `28dc895` to `0e777c6` in /ci (envoyproxy#32652)
  Update QUICHE from 02047e04d to 3373df94b (envoyproxy#32650)
  ci/logging: Add failure detection (envoyproxy#32662)
  Change udpa renaming workaround to not compile the same archive twice (envoyproxy#32647)
  sockets: flipping graceful client socket creation failure (envoyproxy#32602)
  TcpAsyncClient: enhance reconnect robustness (envoyproxy#32578)
  owners: adding Fredy as an Enovy Mobile maintainer (envoyproxy#32638)
  ci: Add scheduled garbage collection (and fix retest) (envoyproxy#32639)
  ...
Alice-Lilith pushed a commit to gravitee-io/envoy that referenced this pull request May 9, 2024
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
Alice-Lilith pushed a commit to gravitee-io/envoy that referenced this pull request May 10, 2024
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
tenshinhigashi pushed a commit to gravitee-io/envoy that referenced this pull request Jun 10, 2024
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
Alice-Lilith pushed a commit to gravitee-io/envoy that referenced this pull request Jun 26, 2024
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
Alice-Lilith pushed a commit to gravitee-io/envoy that referenced this pull request Sep 19, 2024
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
tenshinhigashi pushed a commit to gravitee-io/envoy that referenced this pull request Nov 11, 2024
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
tenshinhigashi pushed a commit to gravitee-io/envoy that referenced this pull request Apr 10, 2025
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
tenshinhigashi pushed a commit to gravitee-io/envoy that referenced this pull request Apr 23, 2025
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
tenshinhigashi pushed a commit to gravitee-io/envoy that referenced this pull request Oct 1, 2025
see: envoyproxy#32647 for details

Signed-off-by: Alice Wasko <alicewasko@datawire.io>
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