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

feat(RELEASE-1171): create integration test for release-service #566

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

jinqi7
Copy link
Contributor

@jinqi7 jinqi7 commented Sep 19, 2024

No description provided.

Copy link

openshift-ci bot commented Sep 19, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@jinqi7 jinqi7 changed the title feat(RELEASE-1171): create integration test pipeline for release-servi… feat(RELEASE-1171): create integration test for release-service repo Sep 19, 2024
Copy link

codecov bot commented Sep 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.97%. Comparing base (a72af8b) to head (f1d3b25).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #566      +/-   ##
==========================================
- Coverage   84.09%   83.97%   -0.13%     
==========================================
  Files          26       26              
  Lines        1635     1635              
==========================================
- Hits         1375     1373       -2     
- Misses        182      183       +1     
- Partials       78       79       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jinqi7 jinqi7 force-pushed the create_integration_release branch 3 times, most recently from fc2eb0c to bd59d88 Compare September 19, 2024 08:18
@jinqi7 jinqi7 changed the title feat(RELEASE-1171): create integration test for release-service repo feat(RELEASE-1171): create integration test for release-service Sep 19, 2024
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 24, 2024

/retest

@jinqi7 jinqi7 force-pushed the create_integration_release branch 2 times, most recently from 0167a80 to 3f21df9 Compare September 24, 2024 09:54
@flacatus
Copy link

/retest

@flacatus
Copy link

/retest

2 similar comments
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 25, 2024

/retest

@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 25, 2024

/retest

@jinqi7 jinqi7 force-pushed the create_integration_release branch 2 times, most recently from 6cedaa1 to ddbffe7 Compare September 25, 2024 11:26
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 25, 2024

/retest

1 similar comment
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 25, 2024

/retest

@kasemAlem
Copy link
Contributor

it's failing on the task quality-dashboard-upload , should we create a ticket for it, @psturc @flacatus ?

@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 25, 2024

/retest

@kasemAlem
Copy link
Contributor

it's failing on the task quality-dashboard-upload , should we create a ticket for it, @psturc @flacatus
okay, I see that Jing already fixed it according to this

@flacatus
Copy link

/retest

2 similar comments
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 26, 2024

/retest

@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 26, 2024

/retest

@jinqi7 jinqi7 marked this pull request as ready for review September 26, 2024 12:19
@jinqi7 jinqi7 requested a review from a team as a code owner September 26, 2024 12:19
@flacatus
Copy link

LGTM

@openshift-ci openshift-ci bot added the lgtm label Sep 27, 2024
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 27, 2024

/retest

1 similar comment
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 27, 2024

/retest

@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 27, 2024

/hold
Currently there is some issue with "provision rosa" in e2e-tests repo, May meet the same issue here.

@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 29, 2024

/unhold

@openshift-ci openshift-ci bot removed the lgtm label Sep 29, 2024
Copy link

openshift-ci bot commented Sep 29, 2024

New changes are detected. LGTM label has been removed.

@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 29, 2024

/retest

1 similar comment
@jinqi7
Copy link
Contributor Author

jinqi7 commented Sep 29, 2024

/retest

@flacatus
Copy link

/test release-service-e2e

@jinqi7 jinqi7 merged commit c60de20 into konflux-ci:main Sep 29, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants