Skip to content

Add zone-sync with no TLS to ConfigMap #7347

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.

Sign up for GitHub

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

Merged
merged 73 commits into from
Feb 28, 2025
Merged

Add zone-sync with no TLS to ConfigMap #7347

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
73 commits
Select commit Hold shift + click to select a range
0a96ecf
add zone-sync using configmap values to nic
AlexFenlon Feb 11, 2025
15b8c1a
remove helm snaps
AlexFenlon Feb 11, 2025
003dad4
Update OIDC example with zone-sync
jjngx Feb 11, 2025
9c4bea6
fix imports
AlexFenlon Feb 11, 2025
6d6652d
most comments addressed
AlexFenlon Feb 12, 2025
05d83f7
Merge remote-tracking branch 'origin/main' into feat/zone-sync-implem…
AlexFenlon Feb 12, 2025
f3b438f
revert comment and spaces
AlexFenlon Feb 12, 2025
4e79ef7
allow unique control of headless service, enforce port to be required…
AlexFenlon Feb 13, 2025
53be6cd
Use minimum zone-sync config required
jjngx Feb 14, 2025
11a81c9
Update zone-sync example
jjngx Feb 17, 2025
5a59f1e
Update zone-sync validation tests
jjngx Feb 17, 2025
d2d3e35
Add comments
jjngx Feb 17, 2025
2c40003
WIP - zone-sync validation
jjngx Feb 17, 2025
09c60c8
seperate zone-sync configmap parsing into seperate function, fix tests
AlexFenlon Feb 17, 2025
fb1db2d
fix lint
AlexFenlon Feb 18, 2025
4374479
Merge remote-tracking branch 'origin/main' into feat/zone-sync-implem…
AlexFenlon Feb 18, 2025
263c83c
add default zone-sync-port and tests
AlexFenlon Feb 18, 2025
973d131
Merge branch 'main' into feat/zone-sync-implementation-no-tls
AlexFenlon Feb 18, 2025
fae67d6
Add zone-sync examples
jjngx Feb 18, 2025
f1df991
Add tests for zone-sync
jjngx Feb 18, 2025
716752f
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 18, 2025
25e7f9c
Update zone-sync tests
jjngx Feb 19, 2025
89057f6
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 19, 2025
fdb754b
change owner to ConfigMap to allow removing orphaned headless service…
AlexFenlon Feb 19, 2025
4caafcd
Update zone-sync tests
jjngx Feb 20, 2025
93ada49
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 20, 2025
0c730ad
revert values file back to default
AlexFenlon Feb 20, 2025
1187b86
revert values file back to default
AlexFenlon Feb 20, 2025
ab06930
Merge remote-tracking branch 'origin/feat/zone-sync-implementation-no…
AlexFenlon Feb 20, 2025
94acad0
fix runtime
AlexFenlon Feb 20, 2025
66cec1e
fix nil pointer and reduce the calls to syncZoneSyncHeadlessService
AlexFenlon Feb 20, 2025
65b9530
Merge remote-tracking branch 'origin/main' into feat/zone-sync-implem…
AlexFenlon Feb 20, 2025
abe447f
update logs on startup
AlexFenlon Feb 20, 2025
c44ec48
Lookup zone-sync in nginx.conf
jjngx Feb 21, 2025
a97cb39
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 21, 2025
3cc6cc0
Merge branch 'main' into feat/zone-sync-implementation-no-tls
AlexFenlon Feb 21, 2025
d600c56
Update examples/custom-resources/zone-sync/README.md
AlexFenlon Feb 21, 2025
abf1eb4
make boolToPointerBool global
AlexFenlon Feb 21, 2025
8842a49
Merge remote-tracking branch 'origin/feat/zone-sync-implementation-no…
AlexFenlon Feb 21, 2025
257e502
change headless service name to use deployment name and kind instead …
AlexFenlon Feb 21, 2025
c33cd2a
symlink oidc secrets to central secrets
AlexFenlon Feb 21, 2025
8b4ef5f
Make test pass
jjngx Feb 21, 2025
7ad3eb6
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 21, 2025
c540014
Add placeholders for headless service lookups - assertion funcs
jjngx Feb 21, 2025
93a4ee5
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 21, 2025
f002e89
Merge branch 'main' into feat/zone-sync-implementation-no-tls
AlexFenlon Feb 24, 2025
a4e1e35
Update cmd/nginx-ingress/main.go
AlexFenlon Feb 24, 2025
034f3f2
adds events to failed creation of headless service
AlexFenlon Feb 24, 2025
8e428a9
Merge branch 'main' into feat/zone-sync-implementation-no-tls
AlexFenlon Feb 24, 2025
4d9da64
WIP - add service lookup to zone sync tests
jjngx Feb 24, 2025
5c8a754
WIP - headless service lookup
jjngx Feb 24, 2025
b4dd5d3
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 24, 2025
8e75b58
headless svc lookup
jjngx Feb 24, 2025
04ff61e
assert headless service
jjngx Feb 24, 2025
036b1db
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 24, 2025
173b2e1
fix ports in data for tests
jjngx Feb 24, 2025
71757ac
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 24, 2025
ca29e45
add assertions for headless service
jjngx Feb 24, 2025
c67286a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Feb 24, 2025
6179a10
Update assertions
jjngx Feb 24, 2025
58a86d0
fix domain from ingressclass, make default port if zone-sync-port has…
AlexFenlon Feb 25, 2025
236f4c0
Merge remote-tracking branch 'origin/main' into feat/zone-sync-implem…
AlexFenlon Feb 25, 2025
b91509e
Update assertion func
jjngx Feb 25, 2025
5873b30
Update test port
jjngx Feb 25, 2025
d8ce99e
clean up template tests and reduce calls to createCombinedDeploymentH…
AlexFenlon Feb 25, 2025
8c8c358
revert mistake deployment commit
AlexFenlon Feb 25, 2025
32828c4
fix selector and domain for hl service
AlexFenlon Feb 26, 2025
48b21e7
remove unused permissions for ingressclass and move resolver outside …
AlexFenlon Feb 26, 2025
2ae156a
Merge remote-tracking branch 'origin/main' into feat/zone-sync-implem…
AlexFenlon Feb 26, 2025
e3dbda8
make resolver only for zone-sync
AlexFenlon Feb 27, 2025
51a5d2a
update oidc example
AlexFenlon Feb 27, 2025
da1514e
Merge remote-tracking branch 'origin/main' into feat/zone-sync-implem…
AlexFenlon Feb 28, 2025
0e1e4d3
fix snaps
AlexFenlon Feb 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion charts/nginx-ingress/templates/clusterrole.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -26,6 +25,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down
143 changes: 132 additions & 11 deletions charts/tests/__snapshots__/helmunit_test.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -90,6 +89,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -538,7 +549,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -551,6 +561,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -994,7 +1016,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -1007,6 +1028,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -1481,7 +1514,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -1494,6 +1526,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -1886,7 +1930,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -1899,6 +1942,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -2318,7 +2373,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -2331,6 +2385,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -2750,7 +2816,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -2763,6 +2828,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -3204,7 +3281,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -3217,6 +3293,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -3638,7 +3726,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -3651,6 +3738,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -4086,7 +4185,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -4099,6 +4197,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down Expand Up @@ -4532,7 +4642,6 @@ rules:
- namespaces
- pods
- secrets
- services
verbs:
- get
- list
Expand All @@ -4545,6 +4654,18 @@ rules:
- create
- patch
- list
- apiGroups:
- ""
resources:
- services
verbs:
- get
- list
- watch
- create
- update
- patch
- delete
- apiGroups:
- coordination.k8s.io
resources:
Expand Down
Loading
Loading