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

Control git push command output #861

Merged
merged 1 commit into from
Mar 14, 2024
Merged

Conversation

mmorhun
Copy link
Collaborator

@mmorhun mmorhun commented Mar 6, 2024

In case of push failure the task produces the following output:

step-patch-gitops
Cloning into 'testgo-gitops'...
[main 747a7b0] Update 'testgo' component image to: quay.io/user-org/rhdh-test:d719c735bdb7573482a9a24cee8442856885e36d@sha256:37607c61eec84d0ffefc8ec55dd859e688b10bb7391ec64a6f324e5e869eb7a9
 1 file changed, 1 insertion(+), 1 deletion(-)
Failed to push update to gitops repository: https://github.com/user-org/testgo-gitops
Do you have correct git credentials configured?

@mmorhun mmorhun self-assigned this Mar 6, 2024
@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 6, 2024

/test build-definitions-pull-request

1 similar comment
@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 6, 2024

/test build-definitions-pull-request

@mmorhun mmorhun force-pushed the RHTAPBUGS-1153 branch 5 times, most recently from d108073 to a1b5666 Compare March 14, 2024 08:53
@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

/test build-definitions-pull-request

3 similar comments
@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

/test build-definitions-pull-request

@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

/test build-definitions-pull-request

@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

/test build-definitions-pull-request

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

/test build-definitions-pull-request

1 similar comment
@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

/test build-definitions-pull-request

@mmorhun
Copy link
Collaborator Author

mmorhun commented Mar 14, 2024

Tests were rerun about 15 times and all except 1 (but then rebase was needed) failed due to failed to get cosign result for image or quay instability. Since it's a known issue and not related to the changes, merging this PR anyway.

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