Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@rajat404
Copy link
Contributor

Set correct secretName for resources related to httpbinproxy.
Also set correct namespace for the dummy ingress object.

Fixes #2800

Signed-off-by: Rajat Goyal [email protected]

Set correct `secretName` for resources related to `httpbinproxy`.
Also set correct namespace for the dummy ingress object.

Fixes #2800

Signed-off-by: Rajat Goyal <[email protected]>
@codecov
Copy link

codecov bot commented Aug 18, 2020

Codecov Report

Merging #2804 into main will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2804      +/-   ##
==========================================
- Coverage   76.82%   76.78%   -0.04%     
==========================================
  Files          73       73              
  Lines        5669     5669              
==========================================
- Hits         4355     4353       -2     
- Misses       1227     1228       +1     
- Partials       87       88       +1     
Impacted Files Coverage Ξ”
internal/dag/cache.go 97.36% <0.00%> (-0.88%) ⬇️

@rajat404
Copy link
Contributor Author

One of the integration tests is failing.
Not able to figure out why (given that i didn't change anything in code) πŸ€”

Can someone please help me out here πŸ™

@youngnick youngnick self-requested a review August 18, 2020 11:23
@rajat404
Copy link
Contributor Author

Ah! It's passing now πŸŽ‰
Can you please share if the build failure happened due to a flaky integration test, or something else? (just curious)

@youngnick
Copy link
Member

I suspect it was a flaky test - it was the SNI check. I triggered a rerun and it seems to have passed now.

@stevesloka stevesloka merged commit 06fa5a5 into projectcontour:main Aug 18, 2020
@rajat404 rajat404 deleted the fix-cert-guide branch August 18, 2020 15:08
@rajat404
Copy link
Contributor Author

Thanks a lot for the quick review & merge πŸ™
This was a really nice experience πŸ˜„

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[site][guides/cert-manager] Incorrect secret name value for httpbinproxy resources

3 participants