-
Notifications
You must be signed in to change notification settings - Fork 54
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
[DEVHAS-611] Update Component create metrics for better data #441
Conversation
/retest |
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.
Changes looks good to me.
I saw e2e tests did not fail under a recent PR, #442, seeing if retest does not reproduce error: the path "deploy/member-operator/appstudio-dev/" does not exist
.
@maysunfaisal Rebasing on your recent changes might fix https://github.com/redhat-appstudio/application-service/actions/runs/7791255552/job/21327116796?pr=441, but I also noticed your comment about the linter fix: #442 (comment) |
/retest Failures this time look like flaky ones, retrying. |
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
5608783
to
96339ae
Compare
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
This comment was marked as outdated.
This comment was marked as outdated.
@michael-valdron This PR can be lgtm'd. I confirmed that the tekton pipeline we are looking for a success is The outdated tekton builds are now replaced by konflux builds and the (Confirmed on older PRs and my direct conversation with John) |
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 This pull-request has been approved by: maysunfaisal, michael-valdron 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 |
What does this PR do?:
Which issue(s)/story(ies) does this PR fixes:
https://issues.redhat.com/browse/DEVHAS-611
PR acceptance criteria:
Unit/Functional tests
Documentation
Client Impact
How to test changes / Special notes to the reviewer:
tests should pass