Skip to content

ci: disable running the Objective-C integration app#24478

Merged
jpsim merged 1 commit intomainfrom
ci-disable-running-the-objective-c-integration-app
Dec 9, 2022
Merged

ci: disable running the Objective-C integration app#24478
jpsim merged 1 commit intomainfrom
ci-disable-running-the-objective-c-integration-app

Conversation

@jpsim
Copy link
Copy Markdown
Contributor

@jpsim jpsim commented Dec 9, 2022

Failing due to a missing extension: https://envoyproxy.slack.com/archives/C02F93EEJCE/p1670613863641819

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@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: #24478 was opened by jpsim.

see: more, trace.

@jpsim jpsim marked this pull request as ready for review December 9, 2022 21:33
@jpsim
Copy link
Copy Markdown
Contributor Author

jpsim commented Dec 9, 2022

/assign @Augustyniak

@jpsim jpsim enabled auto-merge (squash) December 9, 2022 22:12
@jpsim jpsim merged commit 0a8043c into main Dec 9, 2022
Augustyniak added a commit that referenced this pull request Dec 12, 2022
alyssawilk added a commit to alyssawilk/envoy that referenced this pull request Dec 12, 2022
Augustyniak added a commit that referenced this pull request Dec 12, 2022
This reverts commit 0a8043c.

Signed-off-by: Rafal Augustyniak <raugustyniak@lyft.com>
Augustyniak added a commit that referenced this pull request Dec 12, 2022
…24478)""

This reverts commit 8a01a75ff6888fa120a97087fd0d5309dca170ae.

Signed-off-by: Rafal Augustyniak <raugustyniak@lyft.com>
Augustyniak added a commit that referenced this pull request Dec 12, 2022
This reverts commit 0a8043c.

Signed-off-by: Rafal Augustyniak <raugustyniak@lyft.com>
Augustyniak added a commit that referenced this pull request Dec 13, 2022
…24496)

Commit Message:
Additional Description: Re-enables CI job as it should be green again as #24490 and after registering quick socket related protos.
Risk Level: Low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional [API Considerations](https://github.com/envoyproxy/envoy/blob/main/api/review_checklist.md):]
jpsim added a commit that referenced this pull request Dec 14, 2022
…-tools

* origin/main: (59 commits)
  Make IsOkAndHolds matcher work with submatchers (#24498)
  ios: fix platform key value store (#24532)
  make ClusterInfo::traffic_stats_ a unique_ptr, so that later we can lazy-init it later. (#24406)
  quic: splitting into client and server (#24513)
  fixing coverage (#24529)
  ci: Add `examplesOnly` condition (#24465)
  ci: sonatype_nexus_upload.py: remove unused format argument (#24471)
  deps: Bump `build_bazel_rules_apple` -> 1.1.3 (#24527)
  deps: Bump `com_github_nghttp2_nghttp2` -> 1.51.0 (#24525)
  deps: Bump `rules_license` -> 0.0.4 (#24523)
  build(deps): bump sphinxcontrib-httpdomain from 1.8.0 to 1.8.1 in /mobile/docs (#24126)
  build(deps): bump github/codeql-action from 2.1.35 to 2.1.36 (#24473)
  build(deps): bump openpolicyagent/opa from 0.47.2-istio to 0.47.3-istio in /examples/ext_authz (#24514)
  build(deps): bump node from `80844b6` to `2770c78` in /examples/ext_authz/auth/http-service (#24515)
  build(deps): bump abseil-cpp to latest version (#24386)
  xDS: add xDS config tracker extension point (#23485)
  kafka: add shared consumer manager (#24494)
  coverage: Improve test coverage (#24355)
  deps: Bump `rules_python` -> 0.16.1 (#24344)
  ci: revert disable running the Objective-C integration app (#24478)" (#24496)
  ...

Signed-off-by: JP Simard <jp@jpsim.com>
@mattklein123 mattklein123 deleted the ci-disable-running-the-objective-c-integration-app branch January 20, 2023 16:32
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.

2 participants