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

Conversation

keithmattix
Copy link
Contributor

Description:
This PR fixes k8s service port protocol detection and a headless service example to the demo. Fixes #3477

Testing done:

Affected area:

Functional Area
Networking [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? no
  2. Is this a breaking change? no

  3. Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?

@keithmattix keithmattix changed the title Feature/statefulsets Feature/statefulsets: fix protocol detection for ports May 19, 2022
@codecov-commenter
Copy link

codecov-commenter commented May 20, 2022

Codecov Report

Merging #4752 (7efbb2d) into main (dd42d04) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #4752      +/-   ##
==========================================
+ Coverage   68.87%   68.88%   +0.01%     
==========================================
  Files         227      227              
  Lines       16433    16434       +1     
==========================================
+ Hits        11318    11321       +3     
+ Misses       5063     5061       -2     
  Partials       52       52              
Flag Coverage Δ
unittests 68.88% <100.00%> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
pkg/catalog/outbound_traffic_policies.go 94.67% <ø> (ø)
pkg/k8s/client.go 93.33% <100.00%> (+0.27%) ⬆️
pkg/workerpool/workerpool.go 100.00% <0.00%> (ø)
pkg/certificate/manager.go 82.05% <0.00%> (+1.70%) ⬆️

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 dd42d04...7efbb2d. Read the comment docs.

Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>

	modified:   .env.example
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
@keithmattix keithmattix force-pushed the feature/statefulsets branch from c22a230 to fda448a Compare May 20, 2022 19:20
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
@keithmattix keithmattix requested a review from shashankram May 20, 2022 20:36
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
Signed-off-by: Keith Mattix II <keithmattix2@gmail.com>
@keithmattix keithmattix requested a review from nojnhuh May 23, 2022 18:08
@shashankram shashankram merged commit 9b11d76 into openservicemesh:main May 23, 2022
@keithmattix keithmattix deleted the feature/statefulsets branch May 23, 2022 18:51
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.

Support for Statefulsets
4 participants