Skip to content

OCPBUGS-42536: Trying to reduce the flakiness of the samples smoke tests#29143

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
metlos:increase-build-timeout-in-samples-smoke-tests
Oct 26, 2024
Merged

OCPBUGS-42536: Trying to reduce the flakiness of the samples smoke tests#29143
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
metlos:increase-build-timeout-in-samples-smoke-tests

Conversation

@metlos
Copy link
Copy Markdown
Contributor

@metlos metlos commented Sep 27, 2024

Trying to reduce the flakiness of the samples smoke tests by increasing the timeout for the build of the app.

Especially the rails example tends to take forever to build.

@metlos
Trying to reduce the flakiness of the samples smoke tests
8d8691f 
by increasing the timeout for the build of the app.
@metlos metlos mentioned this pull request Sep 27, 2024
Merged
@metlos metlos changed the title Trying to reduce the flakiness of the samples smoke tests OCPBUGS-42536: Trying to reduce the flakiness of the samples smoke tests Sep 27, 2024
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Sep 27, 2024
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Sep 27, 2024

@metlos: This pull request references Jira Issue OCPBUGS-42536, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.18.0) matches configured target version for branch (4.18.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @jitendar-singh

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Trying to reduce the flakiness of the samples smoke tests by increasing the timeout for the build of the app.

Especially the rails example tends to take forever to build.

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 openshift-eng/jira-lifecycle-plugin repository.

@metlos
Copy link
Copy Markdown
Contributor Author

metlos commented Sep 27, 2024

/retest-required

@metlos
Copy link
Copy Markdown
Contributor Author

metlos commented Oct 4, 2024

/retest

@openshift-trt-bot
Copy link
Copy Markdown

openshift-trt-bot commented Oct 4, 2024

Job Failure Risk Analysis for sha: 8d8691f

Job Name Failure Risk
pull-ci-openshift-origin-master-e2e-aws-ovn-single-node-upgrade High
[sig-arch] events should not repeat pathologically for ns/openshift-machine-api
This test has passed 98.74% of 239 runs on release 4.18 [Architecture:amd64 FeatureSet:default Installer:ipi Network:ovn NetworkStack:ipv4 Platform:aws SecurityMode:default Topology:single Upgrade:micro] in the last week.

@metlos
Copy link
Copy Markdown
Contributor Author

metlos commented Oct 7, 2024

/retest-required

mfrancisc
mfrancisc approved these changes Oct 7, 2024
View reviewed changes
Copy link
Copy Markdown

@mfrancisc mfrancisc left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 7, 2024
@metlos
Copy link
Copy Markdown
Contributor Author

metlos commented Oct 9, 2024

@coreydaley, @dperaza4dustbit, are you guys able to approve this PR as the owners of the image-ecosystem?

@metlos
Copy link
Copy Markdown
Contributor Author

metlos commented Oct 23, 2024

/retest

dperaza4dustbit
dperaza4dustbit approved these changes Oct 24, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@dperaza4dustbit dperaza4dustbit left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 24, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dperaza4dustbit, metlos, mfrancisc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 24, 2024
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 24, 2024

/retest-required

Remaining retests: 0 against base HEAD 35b2af7 and 2 for PR HEAD 8d8691f in total

2 similar comments
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 24, 2024

/retest-required

Remaining retests: 0 against base HEAD 35b2af7 and 2 for PR HEAD 8d8691f in total

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 24, 2024

/retest-required

Remaining retests: 0 against base HEAD 35b2af7 and 2 for PR HEAD 8d8691f in total

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 24, 2024

/retest-required

Remaining retests: 0 against base HEAD c59b061 and 2 for PR HEAD 8d8691f in total

2 similar comments
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 24, 2024

/retest-required

Remaining retests: 0 against base HEAD c59b061 and 2 for PR HEAD 8d8691f in total

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 25, 2024

/retest-required

Remaining retests: 0 against base HEAD c59b061 and 2 for PR HEAD 8d8691f in total

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 25, 2024

/retest-required

Remaining retests: 0 against base HEAD 4560b2b and 2 for PR HEAD 8d8691f in total

1 similar comment
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 25, 2024

/retest-required

Remaining retests: 0 against base HEAD 4560b2b and 2 for PR HEAD 8d8691f in total

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 25, 2024 •
edited
Loading

@metlos: The following tests 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/e2e-aws-ovn-ipsec-serial 8d8691f link false /test e2e-aws-ovn-ipsec-serial
ci/prow/e2e-agnostic-ovn-cmd 8d8691f link false /test e2e-agnostic-ovn-cmd

Full PR test history. Your PR dashboard.

Details

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-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 25, 2024

/retest-required

Remaining retests: 0 against base HEAD 4560b2b and 2 for PR HEAD 8d8691f in total

@openshift-merge-bot openshift-merge-bot Bot merged commit 04db7da into openshift:master Oct 26, 2024
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Oct 26, 2024

@metlos: Jira Issue OCPBUGS-42536: All pull requests linked via external trackers have merged:

Jira Issue OCPBUGS-42536 has been moved to the MODIFIED state.

Details

In response to this:

Trying to reduce the flakiness of the samples smoke tests by increasing the timeout for the build of the app.

Especially the rails example tends to take forever to build.

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Oct 26, 2024

[ART PR BUILD NOTIFIER]

Distgit: openshift-enterprise-tests
This PR has been included in build openshift-enterprise-tests-container-v4.18.0-202410260239.p0.g04db7da.assembly.stream.el9.
All builds following this will include this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coreydaley coreydaley Awaiting requested review from coreydaley

@p0lyn0mial p0lyn0mial Awaiting requested review from p0lyn0mial

@jitendar-singh jitendar-singh Awaiting requested review from jitendar-singh

2 more reviewers

@dperaza4dustbit dperaza4dustbit dperaza4dustbit approved these changes

@mfrancisc mfrancisc mfrancisc approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@dperaza4dustbit dperaza4dustbit

@mfrancisc mfrancisc

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@metlos @openshift-ci-robot @openshift-trt-bot @openshift-bot @dperaza4dustbit @mfrancisc