Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upmerge changes from 1.49.x #31068

Merged
merged 16 commits into from
Sep 20, 2022
Merged

Upmerge changes from 1.49.x #31068

merged 16 commits into from
Sep 20, 2022

Conversation

veblush
Copy link
Contributor

@veblush veblush commented Sep 20, 2022

No description provided.

apolcyn and others added 16 commits August 17, 2022 18:21
* bump version to 1.49.0-pre1

* regenerate projects
* Fix forkable globals (grpc#30608)

* Fix forkable repeated registration (grpc#30642)

This fixes a bug that could occur on repeated grpc initialization (after a complete shutdown)

Fixes grpc#30640
* bump version to 1.49.0-pre2

* regenerate projects
* bump version to 1.49.0-pre3

* regenerate projects
… (grpc#30856)

- Enables pod log collection in all PSM interop jobs implemented in grpc#30594.
- Associate test suite runs with their own log file, so it's displayed on "Target Log" tab
…rpc#30875)

To capture the return status of the test in run_test the last command must be the call to the test itself.
This removes `set +x`, which makes the run_test always return success, and not propagate the test status.

I can't find it, but this exact error bit us before. Looks like it leaked to other scripts.
The good thing is if the test was executed, it's failure would still be picked up from the result xml.

However, if the test framework didn't start in the first place, the result will be false positive.
Example: https://source.cloud.google.com/results/invocations/98d3e679-ec8a-40bd-9f36-88179747b0d6/targets

```
/home/kbuilder/.pyenv/versions/k8s_xds_test_runner/bin/python3: Error while finding module specification for 'tests.authz_test' (ModuleNotFoundError: No module named 'tests')
+ set +x
Failed test suites: 0

[ID: 3548168] Command finished after 625 secs, exit value: 0
```
* Support Python 3.11

* Update build images for 3.11

* Whoopsie

* The architecture of this thing is garbage

* Silence ownership warning

* Account for change in git behavior

* Fix directory

* I am in great pain

* Update Windows and arm linux

* Agh

* Clean up
* bump version to 1.49.0

* regenerate projects
* Update third_party/protobuf

* run tools/distrib/python/make_grpcio_tools.py

* update build_handwritten.yaml

* regenerate projects
* bump version to 1.49.1

* regenerate projects

* Automated change: Fix sanity tests

Co-authored-by: gnossen <gnossen@users.noreply.github.com>
@veblush veblush added the release notes: no Indicates if PR should not be in release notes label Sep 20, 2022
@veblush veblush enabled auto-merge (squash) September 20, 2022 18:09
@veblush veblush merged commit fa65716 into grpc:master Sep 20, 2022
@veblush veblush deleted the upmerge branch September 20, 2022 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build release notes: no Indicates if PR should not be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants