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.13] Back-port multi-container probing #658

Merged

Conversation

ReToCode
Copy link
Member

Description

Do we need to patch other branches?

No, this will be in 1.14 upstream

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

/assign @skonto

* Add multi-container probing

* Add e2e test for multi container probing

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

@skonto I don't think we need a git patch for this, it will be in 1.14 and the back-port applies without changes. WDYT?

@skonto
Copy link

skonto commented Mar 13, 2024

Yes I can't think of an issue. What will happen if someone downgrades I guess we dont support it as with any other feature right?

@skonto
Copy link

skonto commented Mar 13, 2024

/lgtm
/hold for tests

Copy link

openshift-ci bot commented Mar 13, 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

@ReToCode
Copy link
Member Author

/unhold as tests passed.

@ReToCode
Copy link
Member Author

/unhold

@openshift-merge-bot openshift-merge-bot bot merged commit 6b2774d into openshift-knative:release-v1.13 Mar 13, 2024
9 checks passed
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