Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

[backport] ingress: don't error if candidate group version is not available (#4007) #4008

Merged
merged 1 commit into from
Aug 19, 2021

Conversation

shashankram
Copy link
Member

Description:
Backports f96e93d to allow v0.9.2 to work with v1.22
of k8s.

Affected area:

Functional Area
Ingress [X]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project? no

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change? no

…nservicemesh#4007)

If a candidate API GroupVersion (ex. networking.k8s.io/v1beta1) is not
available, don't error. Only error if none of the candidates are
available (already the case).
This is required for OSM to work in k8s v1.22, where the
networking.k8s.io/v1beta1 is deprecated.

Signed-off-by: Shashank Ram <[email protected]>
@shashankram shashankram requested a review from a team as a code owner August 18, 2021 22:32
@codecov-commenter
Copy link

Codecov Report

Merging #4008 (eeae094) into release-v0.9 (227ad23) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff                @@
##           release-v0.9    #4008      +/-   ##
================================================
+ Coverage         66.62%   66.64%   +0.02%     
================================================
  Files               171      171              
  Lines              9249     9246       -3     
================================================
  Hits               6162     6162              
+ Misses             3053     3050       -3     
  Partials             34       34              
Flag Coverage Δ
unittests 66.64% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/ingress/client.go 81.48% <100.00%> (+1.30%) ⬆️
pkg/endpoint/providers/kube/client.go 81.96% <0.00%> (-0.82%) ⬇️
pkg/kubernetes/announcement_handlers.go 78.37% <0.00%> (+5.40%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 227ad23...eeae094. Read the comment docs.

@shashankram shashankram merged commit da4bfda into openservicemesh:release-v0.9 Aug 19, 2021
@shashankram shashankram deleted the rel-v0.9.2-bk branch August 19, 2021 00:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants