You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/12/05 10:31:12 UTC
[GitHub] [incubator-kyuubi] pan3793 commented on a diff in pull request #3869: [K8S][IT][BUG] Fix the issue that connect conf is not used in the jdbc connection string
pan3793 commented on code in PR #3869:
URL: https://github.com/apache/incubator-kyuubi/pull/3869#discussion_r1039417919
##########
.github/workflows/master.yml:
##########
@@ -355,17 +355,28 @@ jobs:
run: |
kubectl get serviceaccount
kubectl create serviceaccount kyuubi
+ kubectl create clusterrolebinding kyuubi-role --clusterrole=edit --serviceaccount=default:kyuubi
kubectl get serviceaccount
- name: start kyuubi
run: kubectl apply -f integration-tests/kyuubi-kubernetes-it/src/test/resources/kyuubi-server.yaml
- name: kyuubi pod check
- run: kubectl get pods
+ run: |
+ kubectl get pods
+ kubectl describe pods kyuubi-test
- name: integration tests
run: >-
./build/mvn ${MVN_OPT} clean install
-pl integration-tests/kyuubi-kubernetes-it -am
-Pkubernetes-it
-Dtest=none -DwildcardSuites=org.apache.kyuubi.kubernetes.test.deployment,org.apache.kyuubi.kubernetes.test.KubernetesUtilsTest
+ - name: Cat kyuubi server log
+ if: failure()
+ run: kubectl logs kyuubi-test
Review Comment:
can we redirect logs to file and upload?
##########
.github/workflows/master.yml:
##########
@@ -355,17 +355,28 @@ jobs:
run: |
kubectl get serviceaccount
kubectl create serviceaccount kyuubi
+ kubectl create clusterrolebinding kyuubi-role --clusterrole=edit --serviceaccount=default:kyuubi
kubectl get serviceaccount
- name: start kyuubi
run: kubectl apply -f integration-tests/kyuubi-kubernetes-it/src/test/resources/kyuubi-server.yaml
- name: kyuubi pod check
- run: kubectl get pods
+ run: |
+ kubectl get pods
+ kubectl describe pods kyuubi-test
- name: integration tests
run: >-
./build/mvn ${MVN_OPT} clean install
-pl integration-tests/kyuubi-kubernetes-it -am
-Pkubernetes-it
-Dtest=none -DwildcardSuites=org.apache.kyuubi.kubernetes.test.deployment,org.apache.kyuubi.kubernetes.test.KubernetesUtilsTest
+ - name: Cat kyuubi server log
+ if: failure()
+ run: kubectl logs kyuubi-test
+ - name: Cat spark driver log
+ if: failure()
+ run: |
+ kubectl get pods
+ kubectl get pods | grep driver | awk -F " " '{print$1}' | xargs -I {} kubectl logs {}
Review Comment:
ditto
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org