search

openshift-pipelines / pipeline-service

SaaS for Tekton Pipelines
Apache License 2.0
23 stars 44 forks source link

Fix container argument syntax #893

Closed sayan-biswas closed 8 months ago

sayan-biswas commented 8 months ago

Not sure how command line binary arguments are interpreted, but this seems to work.

Roming22 commented 8 months ago

/test test-pipeline-service-upgrade-ocp-414

Roming22 commented 8 months ago

/test test-pipeline-service-upgrade-ocp-414

Roming22 commented 8 months ago

/test test-pipeline-service-upgrade-ocp-414

sayan-biswas commented 8 months ago

@xinredhat Are there any issues with the upgrade test currently?

Roming22 commented 8 months ago

@sayan-biswas The test-upgrade failed when trying to access logs:

[run-tests] + echo -n '    - Test records: '
[run-tests] ++ kubectl get route tekton-results -n tekton-results '--template={{.spec.host}}'
[run-tests] + RESULT_ROUTE=tekton-results-tekton-results.apps.rosa.ci-02d381e53c.qur3.p3.openshiftapps.com
[run-tests] + QUERY_URL=https://tekton-results-tekton-results.apps.rosa.ci-02d381e53c.qur3.p3.openshiftapps.com/apis/results.tekton.dev/v1alpha2/parents/plnsvc-tests/results/22b7b160-2500-4a78-8ce9-f4b0edc87a93/records
[run-tests] + QUERY_CMD=("curl" "--insecure" "-H" "Authorization: Bearer $token" "-H" "Accept: application/json" "$QUERY_URL")
[run-tests] ++ curl --insecure -H 'Authorization: Bearer eyJhbGciOiJSUzI1NiIsImtpZCI6Ijc3cGdUaC1KYmJERFJ2bnlWOGpCTWRBMjFFZ1dodUtkaWlQMW1kT3BpNEkifQ.eyJhdWQiOlsiaHR0cHM6Ly9yaC1vaWRjLnMzLnVzLWVhc3QtMS5hbWF6b25hd3MuY29tLzI3M3Riajcxc2txa3NncWFmb2U1YW90c3VjNDRibHA0Il0sImV4cCI6MTcwNDMwNzY5MCwiaWF0IjoxNzA0MzA0MDkwLCJpc3MiOiJodHRwczovL3JoLW9pZGMuczMudXMtZWFzdC0xLmFtYXpvbmF3cy5jb20vMjczdGJqNzFza3Frc2dxYWZvZTVhb3RzdWM0NGJscDQiLCJrdWJlcm5ldGVzLmlvIjp7Im5hbWVzcGFjZSI6InBsbnN2Yy10ZXN0cyIsInNlcnZpY2VhY2NvdW50Ijp7Im5hbWUiOiJ0ZWt0b24tcmVzdWx0cy10ZXN0cyIsInVpZCI6Ijg3YWY1ZTZkLTM3M2ItNGJhNS05ZjNjLWIxNmQ5M2Y1YTYyMCJ9fSwibmJmIjoxNzA0MzA0MDkwLCJzdWIiOiJzeXN0ZW06c2VydmljZWFjY291bnQ6cGxuc3ZjLXRlc3RzOnRla3Rvbi1yZXN1bHRzLXRlc3RzIn0.W6Z3yc2t0q5Hq_00iQfhQBJFU5SLS536OR-l63YfvUN_qsOSJzr6NQMRDFi4kQQA2dAKKOB7Y4dBCp61ZBlArhno6dYln9-WHfJs_-i2NmkCNol1RZZshHLjSPio1j5U_B8jzpE6CZ2w5wS3mZq0KNHsLYZ_KdQ0WngEkNPGSi8MVHMc6PcR5MR03daTLeO8dLhF81ozVkRJU81eIjEPZoIbPrvLMDlxNxsmD6JmsTFeh4oj8-gyj6zh3aynOO62J2UjwgUrXtqoyk-Gde3I9XH2CPNCQHJUeS8lrRTlAAPUM4eIUwjH7cbmDxR_0KDk3hx-B-ut2MoWdOJaAnlmMHWDdAd3CfOBQ7K62Oui2xW6dkWruZXk-6XDM7h3hex-VK7kdjWqqFeJGodjpiRybY8kduvaS132ncS2wRYVy3Op-Zs1Rt0XutODuX12XGm0Z6a2q6WI5lrnAPKM-xCTGuGpzBLsNG-HEgwbVbxx7NOQJsXRPl5b35ezzMbUctgom_UbiOD6LDvJMPoskDU8MAqVebQUInbMxp7OVOyeklV9xnj5sv6RBEJzmgaJylQ1O6_gvbwM5c3_lEfK0bF45X9taER6nSsNMkqb4cVIAyctJVNy6Tx4zeRj4e_qkkNkk0Bvmx6uHQwTNIoA-EkpntH0euLOxZtAi4wF08B5R_w' -H 'Accept: application/json' https://tekton-results-tekton-results.apps.rosa.ci-02d381e53c.qur3.p3.openshiftapps.com/apis/results.tekton.dev/v1alpha2/parents/plnsvc-tests/results/22b7b160-2500-4a78-8ce9-f4b0edc87a93/records
[run-tests] Failed
[run-tests] + QUERY_RESULT='{"records":[], "nextPageToken":""}'
[run-tests] + wait
[run-tests] + [[ {"records":[], "nextPageToken":""} == *\2\2\b\7\b\1\6\0\-\2\5\0\0\-\4\a\7\8\-\8\c\e\9\-\f\4\b\0\e\d\c\8\7\a\9\3\/\r\e\c\o\r\d\s* ]]
[run-tests] + echo Failed
[run-tests] + echo '[ERROR] Unable to retrieve records for 22b7b160-2500-4a78-8ce9-f4b0edc87a93 from pipeline run pipelinerun.tekton.dev/test-pipelinerun-slvcm'
[run-tests] [ERROR] Unable to retrieve records for 22b7b160-2500-4a78-8ce9-f4b0edc87a93 from pipeline run pipelinerun.tekton.dev/test-pipelinerun-slvcm

Logs are from kubectl tekton log tr -n tekton-ci test-pipeline-service-upgrade-ocp-414-d5c5w-test-upgrade. I can see a similar behavior on an older run, but at a later step (c.f. kubectl tekton log tr -n tekton-ci test-pipeline-service-upgrade-ocp-414-sb4vl-test-downgrade).

sayan-biswas commented 8 months ago

/test test-pipeline-service-upgrade-ocp-414

Roming22 commented 8 months ago

Forcing merge, as we believe the failure is because of flakiness.