Skip to content
This repository was archived by the owner on Nov 28, 2022. It is now read-only.

[release-v0.17.2] Hotfix: Deploying receiver as ksvc if serving is available#938

Merged
openshift-merge-robot merged 1 commit into
openshift:release-v0.17.2from
cardil:hotfix/wathola-use-ksvc
Nov 1, 2020
Merged

[release-v0.17.2] Hotfix: Deploying receiver as ksvc if serving is available#938
openshift-merge-robot merged 1 commit into
openshift:release-v0.17.2from
cardil:hotfix/wathola-use-ksvc

Conversation

@cardil
Copy link
Copy Markdown

@cardil cardil commented Oct 30, 2020

This is a possible backport of upstream PR knative#4430. That depends on that upstream PR being merged. I don't give a huge confidence that happening.

If upstream PR will be merged, then nothing more isn't required for 0.18+.

Otherwise, this PR must be also converted to a patch on master branch. Then it must be applied on each release-next, until a proper fix for knative#3175 is provided.

This is required for openshift-knative/serverless-operator#530 (comment)

@openshift-ci-robot openshift-ci-robot added the area/test-and-release Issues or PRs related to test and release label Oct 30, 2020
@cardil cardil changed the title Hotfix: Deploying receiver as ksvc if serving is available [release-v0.17.2] Hotfix: Deploying receiver as ksvc if serving is available Oct 30, 2020
@matzew
Copy link
Copy Markdown
Member

matzew commented Oct 31, 2020

/test all

@matzew
Copy link
Copy Markdown
Member

matzew commented Oct 31, 2020

/lgtm

@matzew
Copy link
Copy Markdown
Member

matzew commented Oct 31, 2020

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Oct 31, 2020
@cardil cardil force-pushed the hotfix/wathola-use-ksvc branch from 188c208 to c95b942 Compare October 31, 2020 12:05
@openshift-ci-robot openshift-ci-robot removed lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Oct 31, 2020
@cardil
Copy link
Copy Markdown
Author

cardil commented Oct 31, 2020

/retest

@matzew
Copy link
Copy Markdown
Member

matzew commented Nov 1, 2020

/test all

@matzew
Copy link
Copy Markdown
Member

matzew commented Nov 1, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 1, 2020
@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cardil, matzew

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-merge-robot openshift-merge-robot merged commit 56c41d2 into openshift:release-v0.17.2 Nov 1, 2020
@cardil cardil deleted the hotfix/wathola-use-ksvc branch November 1, 2020 11:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/test-and-release Issues or PRs related to test and release lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants