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

pipeline-service update #3728

Merged
merged 6 commits into from
May 21, 2024
Merged

pipeline-service update #3728

merged 6 commits into from
May 21, 2024

Conversation

@@ -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
Copy link
Contributor

Choose a reason for hiding this comment

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

@enarha
Copy link
Member

enarha commented May 18, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member

enarha commented May 18, 2024

/test appstudio-hac-e2e-tests

@enarha
Copy link
Member

enarha commented May 18, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member

enarha commented May 19, 2024

The e2e tests fail on deploying the monitoring-workload-grafana-in-cluster-local GitOps Application. The error I see in the logs is Application CR was missing 'databaseID' label. I couldn't find anything more specific. I tested local deployment from the pipeline-service and it worked. I see other PRs passing, so I'll rebase and try again.

@enarha
Copy link
Member

enarha commented May 19, 2024

/test appstudio-e2e-tests

1 similar comment
@enarha
Copy link
Member

enarha commented May 19, 2024

/test appstudio-e2e-tests

@enarha
Copy link
Member

enarha commented May 19, 2024

/test appstudio-hac-e2e-tests

@enarha
Copy link
Member

enarha commented May 20, 2024

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.

@enarha
Copy link
Member

enarha commented May 20, 2024

/test appstudio-e2e-tests

@gabemontero
Copy link
Contributor

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 ?

@enarha
Copy link
Member

enarha commented May 20, 2024

The hac test was green earlier and is also green now. Let wait for the running tests to finish, hopefully succeed.

Copy link
Member

@enarha enarha left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link

openshift-ci bot commented May 20, 2024

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gabemontero
Copy link
Contributor

hmmm ... the test has been running for over 7 hours now ...something happened in prow. Will try a

/refresh

then close/reopen.

@gabemontero
Copy link
Contributor

/retest

@gabemontero
Copy link
Contributor

ok new tests got restarted ... let's see if they finish.

Copy link

openshift-ci bot commented May 21, 2024

@rh-tap-build-team[bot]: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/appstudio-upgrade-tests 9b9c7c2 link false /test appstudio-upgrade-tests

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.

@openshift-merge-bot openshift-merge-bot bot merged commit 2c57b0e into main May 21, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants