Adding vipul-21 as a fqdn reviewer#361
Merged
Merged
Conversation
vipul-21
force-pushed
the
singhvipul/fqdn
branch
from
fc09f4a to
376e78a
Compare
Member
|
@cilium/fqdn PTAL |
Contributor
Author
|
@squeed Friendly ping for the review. Thanks ! |
squeed
approved these changes
Feb 2, 2026
Contributor
squeed
left a comment
squeed
left a comment
There was a problem hiding this comment.
I'd be glad to have Vipul help review!
joestringer
approved these changes
Feb 4, 2026
Member
joestringer
left a comment
joestringer
left a comment
There was a problem hiding this comment.
Glad to have you contributing 🎉
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moving up the Contributor Ladder
My Relevant Contributions to the Cilium Ecosystem:
Link to issues you have opened
Duplicate identities with large scale cluster cilium#25251
Dualstack cilium config fatal error on single stack cluster cilium#28909
Cilium connectivity test failure for
allow-all-with-metrics-checkcilium#29213High Memory Usage during scale testing of fqdn filtering cilium#35789
Inconsistent Policy Map State for Endpoints with Same Identity cilium#37334
CI: Conformance Multi Pool IPAM (ci-multi-pool) - seq-client-egress-l7-extra-tls-headers/pod-to-world-with-extra-tls-intercept: command terminated with exit code 28 cilium#40854
CI: Conformance Runtime - Runtime Test (agent) cilium#41640
CI: Conformance Ginkgo (ci-ginkgo) - f18-datapath-lrp cilium#41708
Link to pull requests you have reviewed.
policy: ensure matched policy order is deterministic cilium#42416
Fix nodeSelector validation in CiliumNetworkPolicy Specs array cilium#40702
Revert "fix: Adding ipv6 check on the node init function" cilium#40143
dnsproxy: don't verify names of answers cilium#39926
fix: allow single-stack IPv6 without address set on the node cilium#37928
policy: make
--enable-well-known-identitiesa cell flag cilium#37174proxy: Check if acknowledged proxy port is listening cilium#36955
proxy: Only set/update L7 rules for DNS proxy cilium#36809
endpoint: store state in ep_config.json cilium#31559
Fix Cilium Datapath Prometheus metric names cilium#29226
Link to pull requests you have authored.
feat(sdp): Documentation and templates for standalone DNS Proxy cilium#43249
fix(sdp): Make []prefix as non unique index PrefixToIdentityTable cilium#43235
feat(sdp): Adding the hive shell to standalone dns proxy cilium#43204
feat(sdp): Dockerfile for standalone dns proxy cilium#42647
feat(sdp): Updating DNS rules in Standalone DNS proxy cilium#41609
feat(sdp): Updating the ip to endpoint table for sdp cilium#41548
feat(sdp): sdp connection to cilium agent cilium#41507
feat(sdp): interaction flow between cells for standalone dns proxy cilium#40982
refactor: Add proxy lookup handler cell for DNS policy enforcement cilium#40882
fix: create policy snapshot only for sdp cilium#40785
feat(sdp): Cilium agent sending and recv data for SDP cilium#40689
cli: Add the check for l7 proxy for
to-fqdnstest cilium#40549feat(sdp): Hive cmd for standalone dns proxy cilium#39906
fix: Fix implicit conversion warnings in bpf_sec_to_mono macro cilium#39845
feat(sdp): Cilium agent server handling SDP conn cilium#39220
feat(sdp): Adding the cilium grpc server cilium#38669
Fixing typos in fqdn and policy pkg cilium#37060
Update isFqdn fn based on the source github.com/miekg/dns cilium#37037
refactor: config options combined into DNS proxy config struct cilium#32777
.github: adding daemon/cmd/fqdn to sig/policy PRs cilium#32442
fqdn: updating the variable name in fqdn cilium#32298
feat: Add the http return code to metric api_processed_total cilium#31227
fix: remove help message in build config failure cilium#28974
fix: Adding the fatal error for ipv6 cilium config on a single stack node cilium#28953
docs: Updating Azure CNI chaining as Legacy approach cilium#28571
fix: Remove the latest image tag from docs as latest tag is not published cilium#28241
docs: rephrasing the hubble intro doc cilium#27712
CI: Link checker script cilium#27116
Documentation: fix the broken links/dead links cilium#26880
Parses the IP addr passed as CIDR from the delegated IPAM and then use the IP addr from the parsed prefix. cilium#22918
CI: Using the same function for Native CIDR for GKE and AKS cilium#21701
CI: Add AKS helm overrides for E2E test cilium#21277
Links to discussions or issues you actively participated in
CFP-30984 : Add CFP for DNS proxy HA design-cfps#32
Other relevant community involvement
Regularly join the sig-policy meeting