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
Add optional hostAliases to cert-manager pod #6456
Conversation
Hi @Pionerd. Thanks for your PR. I'm waiting for a cert-manager member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I would like to see this PR merged for a different reason: May I expect this to be merged in the foreseeable future? |
@Pionerd Can you update this PR? |
Issues go stale after 90d of inactivity. |
/remove-lifecycle stale |
/ok-to-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, thanks for raising this and sorry it took a while to get eyes on it - pinging us in slack for a review is usually a good way to get our attention 😁
The tests will fail for this at the time I'm writing this comment because helm docs will need to be generated. Do you think we could update the comment a little too?
Happy to merge this once tests are passing!
/retest |
I'm having some trouble getting the merge commit signed, any tips? |
Signed-off-by: Pieter van der Giessen <pieter@pionative.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
/kind feature
Looks like you got it signed correctly! Thanks for your contribution, this is a great improvement 🚀
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: SgtCoDFish The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Pull Request Motivation
As a mitigation for #1292 it may be desired to hardcode DNS entries on pod level to allow the self-check to pass, e.g. in the scenario where the network does not allow traffic to an internal IP by calling its external IP (which happens in the case of a public DNS).
Kind
feature
Release Note