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

[RELEASE-1.14] add startup probe support to Knative Service #758

Conversation

ReToCode
Copy link
Member

@ReToCode ReToCode commented Jul 9, 2024

What this PR does / why we need it:

Which issue(s) this PR fixes:

JIRA: https://issues.redhat.com/browse/SRVKS-1229

Does this PR needs for other branches:
No

Does this PR (patch) needs to update/drop in the future?:
No

/assign @skonto

* Add startup probe to CRDs

* Allow startup probes in Knative Services

* Increase ProgressDeadlineSeconds when startup probe is present

* Remove ProgressDeadlineSeconds increase when startup probe is present

(cherry picked from commit ac7f585)
@ReToCode
Copy link
Member Author

ReToCode commented Jul 9, 2024

/override "lint"

Copy link

openshift-ci bot commented Jul 9, 2024

@ReToCode: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • lint

Only the following failed contexts/checkruns were expected:

  • Lint
  • ci/prow/415-images
  • ci/prow/415-test-e2e-aws-415
  • ci/prow/415-test-e2e-tls-aws-415
  • pull-ci-openshift-knative-serving-release-next-415-images
  • pull-ci-openshift-knative-serving-release-next-415-test-e2e-aws-415
  • pull-ci-openshift-knative-serving-release-v1.10-415-test-e2e-tls-aws-415
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override "lint"

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.

@ReToCode
Copy link
Member Author

ReToCode commented Jul 9, 2024

/override "Lint"

Copy link

openshift-ci bot commented Jul 9, 2024

@ReToCode: Overrode contexts on behalf of ReToCode: Lint

In response to this:

/override "Lint"

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.

@skonto
Copy link

skonto commented Jul 9, 2024

/lgtm
/approve

/hold for the tests

Copy link

openshift-ci bot commented Jul 9, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ReToCode, skonto

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@skonto skonto requested review from dsimansk and removed request for mvinkler July 9, 2024 09:01
@skonto
Copy link

skonto commented Jul 9, 2024

/test ?

Copy link

openshift-ci bot commented Jul 9, 2024

@skonto: The following commands are available to trigger required jobs:

  • /test 412-images
  • /test 412-perf-tests-aws-412
  • /test 412-test-e2e-aws-412
  • /test 412-test-e2e-tls-aws-412
  • /test 415-images
  • /test 415-perf-tests-aws-415
  • /test 415-test-e2e-aws-415
  • /test 415-test-e2e-tls-aws-415
  • /test 416-images
  • /test 416-perf-tests-aws-416
  • /test 416-test-e2e-aws-416
  • /test 416-test-e2e-tls-aws-416

Use /test all to run the following jobs that were automatically triggered:

  • pull-ci-openshift-knative-serving-release-v1.14-415-images
  • pull-ci-openshift-knative-serving-release-v1.14-415-test-e2e-aws-415
  • pull-ci-openshift-knative-serving-release-v1.14-415-test-e2e-tls-aws-415

In response to this:

/test ?

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.

@ReToCode
Copy link
Member Author

ReToCode commented Jul 9, 2024

/unhold

tests passed, lint errors are not relevant for this PR (but we probably should fix those separately).

@openshift-merge-bot openshift-merge-bot bot merged commit 6120450 into openshift-knative:release-v1.14 Jul 9, 2024
7 of 8 checks passed
@ReToCode ReToCode mentioned this pull request Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants