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

docs: guides/cert-manager #2804

Merged
merged 2 commits into from
Aug 18, 2020
Merged

docs: guides/cert-manager #2804

merged 2 commits into from
Aug 18, 2020

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 404rajat@gmail.com

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

Fixes #2800

Signed-off-by: Rajat Goyal <404rajat@gmail.com>
@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