Skip to content

Add steps to run xDS in a non-exclusive project#26204

Merged
lidizheng merged 5 commits intogrpc:masterfrom
lidizheng:shared-xds-k8s
May 7, 2021
Merged

Add steps to run xDS in a non-exclusive project#26204
lidizheng merged 5 commits intogrpc:masterfrom
lidizheng:shared-xds-k8s

Conversation

@lidizheng
Copy link
Copy Markdown
Contributor

@lidizheng lidizheng commented May 7, 2021 •
edited
Loading

As title.

Though, I did encounter a problem with my manual run. Still investigating why the backend status didn't turn green.

======================================================================
ERROR: test_traffic_director_grpc_setup (__main__.BaselineTest) [6_add_server_backends_to_backend_service]
BaselineTest.test_traffic_director_grpc_setup
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lidiz/grpc/tools/run_tests/xds_k8s_test_driver/tests/baseline_test.py", line 51, in test_traffic_director_grpc_setup
    self.setupServerBackends()
  File "/home/lidiz/grpc/tools/run_tests/xds_k8s_test_driver/framework/xds_k8s_testcase.py", line 145, in setupServerBackends
    self.td.wait_for_backends_healthy_status()
  File "/home/lidiz/grpc/tools/run_tests/xds_k8s_test_driver/framework/infrastructure/traffic_director.py", line 200, in wait_for_backends_healthy_status
    self.backends)
  File "/home/lidiz/grpc/tools/run_tests/xds_k8s_test_driver/framework/infrastructure/gcp/compute.py", line 284, in wait_for_backends_healthy_status
    _retry_backends_health()
  File "/home/lidiz/grpc/py37_native/lib/python3.7/site-packages/retrying.py", line 49, in wrapped_f
    return Retrying(*dargs, **dkw).call(f, *args, **kw)
  File "/home/lidiz/grpc/py37_native/lib/python3.7/site-packages/retrying.py", line 214, in call
    raise RetryError(attempt)
retrying.RetryError: RetryError[Attempts: 145, Value: False]

----------------------------------------------------------------------
Ran 1 test in 956.225s

The UHC failed to reach K8s pod because of firewall rules, it requires a special metadata tag that I wasn't aware of.

@lidizheng lidizheng added lang/Python release notes: no Indicates if PR should not be in release notes labels May 7, 2021
@lidizheng lidizheng requested a review from sergiitk May 7, 2021 21:54
sergiitk
sergiitk reviewed May 7, 2021
View reviewed changes
tools/run_tests/xds_k8s_test_driver/README.md Outdated
Comment on lines +82 to +83
--server_image="gcr.io/grpc-testing/xds-k8s-test-server-java:latest" \
--client_image="gcr.io/grpc-testing/xds-k8s-test-client-java:latest" \
--client_image="gcr.io/grpc-testing/xds-k8s-test-client-java:latest"
Copy link
Copy Markdown
Member

@sergiitk sergiitk May 7, 2021

Choose a reason for hiding this comment

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

Same here

tools/run_tests/xds_k8s_test_driver/README.md Outdated
--client_image="gcr.io/grpc-testing/xds-k8s-test-client-java:latest"
```

### Workload namespace
Copy link
Copy Markdown
Member

@sergiitk sergiitk May 7, 2021

Choose a reason for hiding this comment

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

Suggested change
### Workload namespace
### Test namespace

This is used to generate GCP resource names too

tools/run_tests/xds_k8s_test_driver/README.md Outdated
Comment on lines 112 to 113
--server_image="gcr.io/grpc-testing/xds-k8s-test-server-java:latest" \
--client_image="gcr.io/grpc-testing/xds-k8s-test-client-java:latest" \
Copy link
Copy Markdown
Member

@sergiitk sergiitk May 7, 2021

Choose a reason for hiding this comment

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

Could you update these too?

@lidizheng
Copy link
Copy Markdown
Contributor Author

lidizheng commented May 7, 2021

@sergiitk All requested changes done. PTALA.

sergiitk
sergiitk approved these changes May 7, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@sergiitk sergiitk left a comment

Choose a reason for hiding this comment

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

Thank you!

@lidizheng lidizheng enabled auto-merge (squash) May 7, 2021 23:41
@lidizheng lidizheng merged commit 0aecb90 into grpc:master May 7, 2021
sergiitk added a commit to sergiitk/grpc that referenced this pull request Dec 8, 2023
@lidizheng @sergiitk
Add steps to run xDS in a non-exclusive project (grpc#26204)
f7029d1 
* Add steps to run xDS in a non-exclusive project

* Update tools/run_tests/xds_k8s_test_driver/README.md

Co-authored-by: Sergii Tkachenko <[email protected]>

* Update tools/run_tests/xds_k8s_test_driver/README.md

Co-authored-by: Sergii Tkachenko <[email protected]>

* Add .cfg and clean-up sections

* Use a newer commit hash

Co-authored-by: Sergii Tkachenko <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sergiitk sergiitk sergiitk approved these changes

Assignees

@sergiitk sergiitk

Labels

lang/Python release notes: no Indicates if PR should not be in release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lidizheng @sergiitk