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

Get commiter email dynamically from github application for renovate nudging PRs #396

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

rcerven
Copy link
Contributor

@rcerven rcerven commented Jan 29, 2025

Checklist:

  • PR has reference to the issue(s) it resolves
  • Write / update unit tests
  • Write / update integration (envtest) tests
  • Ensure there is an issue for e2e tests if needed
  • Ensure make test passes
  • Ensure test coverage hasn't decreased
  • Test code changes manually
  • Update readme and documentation
  • Write PR description that highlights overall changes and add usage examples if applicable

@rcerven rcerven requested a review from a team as a code owner January 29, 2025 20:12
@rcerven rcerven force-pushed the dynamic_commiter_nudging branch 3 times, most recently from 18f98d6 to 9263e22 Compare February 4, 2025 20:17
@rcerven rcerven changed the title [WIP] fix [WIP] Get commiter email dynamically for renovate nudging PRs Feb 4, 2025
@rcerven rcerven force-pushed the dynamic_commiter_nudging branch 3 times, most recently from 066d1b3 to 6e954b5 Compare February 5, 2025 20:58
@rcerven rcerven changed the title [WIP] Get commiter email dynamically for renovate nudging PRs [WIP] Get commiter email dynamically from github application for renovate nudging PRs Feb 5, 2025
@rcerven rcerven force-pushed the dynamic_commiter_nudging branch from 6e954b5 to 8695ac3 Compare February 5, 2025 21:05
@rcerven rcerven changed the title [WIP] Get commiter email dynamically from github application for renovate nudging PRs Get commiter email dynamically from github application for renovate nudging PRs Feb 5, 2025
@rcerven rcerven force-pushed the dynamic_commiter_nudging branch 2 times, most recently from cd1f6d7 to 72de6c2 Compare February 17, 2025 17:19
@rcerven
Copy link
Contributor Author

rcerven commented Feb 17, 2025

/retest

@rcerven rcerven force-pushed the dynamic_commiter_nudging branch from 72de6c2 to 0c89c90 Compare February 18, 2025 13:36
…udging PRs

STONEBLD-3194

Signed-off-by: Robert Cerven <[email protected]>
@rcerven rcerven force-pushed the dynamic_commiter_nudging branch from 0c89c90 to fa238fc Compare February 18, 2025 13:37
@rcerven
Copy link
Contributor Author

rcerven commented Feb 18, 2025

/retest

@konflux-ci-qe-bot
Copy link

@rcerven: The following test has Failed, say /retest to rerun failed tests.

PipelineRun Name Status Rerun command Build Log Test Log
konflux-e2e-build-service-fk2pf Failed /retest View Pipeline Log View Test Logs

Inspecting Test Artifacts

To inspect your test artifacts, follow these steps:

  1. Install ORAS (see the ORAS installation guide).
  2. Download artifacts with the following commands:
mkdir -p oras-artifacts
cd oras-artifacts
oras pull quay.io/konflux-test-storage/konflux-team/build-service:konflux-e2e-build-service-fk2pf

Test results analysis

🚨 Error occurred while running the E2E tests, list of failed Spec(s):

➡️ [failed] [It] [build-service-suite Build service E2E tests] test build secret lookup when two secrets are created triggered PipelineRun is for component [build-service, pac-build, secret-lookup]

Click to view logs

Timed out after 300.001s.
timed out when waiting for the PipelineRun to start for the component component-two-hahi/build-e2e-wkds-tenant
Expected success, but got an error:
    <*errors.errorString | 0xc0007bb0e0>: 
    no pipelinerun found for component component-two-hahi
    {
        s: "no pipelinerun found for component component-two-hahi",
    }

@rcerven rcerven merged commit 006c610 into konflux-ci:main Feb 18, 2025
6 of 7 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.

4 participants