-
Notifications
You must be signed in to change notification settings - Fork 205
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
pipeline-service update #3728
pipeline-service update #3728
Conversation
867c6cb
to
dff4923
Compare
@@ -1242,7 +1242,7 @@ spec: | |||
containers: | |||
- args: | |||
- -zap-log-level=6 | |||
image: quay.io/redhat-appstudio/pipeline-service-exporter:d0109787b251d347f08f5d5e473af7ef5dec54b2 | |||
image: quay.io/redhat-appstudio/pipeline-service-exporter:baf868c848633117ac2b5ea923ae0e3b646f2330 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this sha is available at https://quay.io/repository/redhat-appstudio/pipeline-service-exporter?tab=tags
/test appstudio-e2e-tests |
/test appstudio-hac-e2e-tests |
/test appstudio-e2e-tests |
The e2e tests fail on deploying the monitoring-workload-grafana-in-cluster-local GitOps Application. The error I see in the logs is |
/test appstudio-e2e-tests |
1 similar comment
/test appstudio-e2e-tests |
/test appstudio-hac-e2e-tests |
ba0bcbd
to
c63c5a6
Compare
Even though the error is re-occurring, it looks like an intermittent one. There are some failures while pulling some git repos needed to build the kustomizations. I've checked and the referenced SHAs appear to be there. |
/test appstudio-e2e-tests |
a green appstudio-e2e-tests @enarha and the hac failure was the test getting interrupted is this sufficient to tag for merge, or do you want to see a green hac test ? |
…/kustomization.yaml
1108d4a
to
87fcb75
Compare
The hac test was green earlier and is also green now. Let wait for the running tests to finish, hopefully succeed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: enarha, rh-tap-build-team[bot] The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
hmmm ... the test has been running for over 7 hours now ...something happened in prow. Will try a /refresh then close/reopen. |
/retest |
ok new tests got restarted ... let's see if they finish. |
@rh-tap-build-team[bot]: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Included PRs: