-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: Correct comment on toFQDN API definition #27496
Conversation
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.
Thanks Alex, LGTM. This comment is for the old poller implementation which we haven't relied on for a long time. The proxy based ToFQDNs implementation doesn't have these limitations.
@joestringer are you able to trigger the CI tests? Unfortunately the CRD needed updating so these tests probably do need to be run. |
/test |
Could you squash the two commits together? Also the |
ae19bc5
to
14c0ac6
Compare
The comment is 5 years old and no longer valid, removing and updating the CRD. Signed-off-by: Alex Waring <ajmwaring@gmail.com>
14c0ac6
to
e81b757
Compare
Squashed and bowed to our bot overlords. |
/test |
@joestringer any idea what Travis is complaining about? |
It looks like Travis just timed out. The actual coverage Travis provides is also available in other jobs, Travis just runs on arm64 as well. There's no platform-specific issue here, I'm happy to merge this as-is. |
Thanks for the contribution @Alex-Waring! |
This comment was added 5 years ago, and is no longer correct. Removing as discussed in slack conversation: https://cilium.slack.com/archives/C01JALNQAR1/p1692007243176749
This is a change in a comment to a file, so should need no release notes, or testing, just validation that the removal is correct.