Skip to content

Commit 216133d

Browse files
committed
Remove OLM patches for Cilium 1.15 and older
1 parent 3becc71 commit 216133d

File tree

5 files changed

+8
-141
lines changed

5 files changed

+8
-141
lines changed

component/olm.jsonnet

Lines changed: 1 addition & 66 deletions
Original file line numberDiff line numberDiff line change
@@ -350,39 +350,7 @@ local patchManifests = function(file, has_csv)
350350
resources: [ 'leases' ],
351351
verbs: [ 'create', 'get', 'update', 'list', 'delete' ],
352352
},
353-
] + if util.version.minor <= 15 then [
354-
// cilium <= 1.15 uses a clusterrole and clusterrolebinding for the
355-
// hubble certgen cronjob. This is changed to a role and rolebinding
356-
// for 1.16.
357-
// The OLM operator doesn't have permissions to create the
358-
// clusterrole and clusterrolebinding out of the box, so we patch
359-
// the OLM operator clusterrole to have the rules that need to be
360-
// created for the certgen cronjob.
361-
{
362-
apiGroups: [ '' ],
363-
resources: [ 'secrets' ],
364-
verbs: [ 'create' ],
365-
},
366-
{
367-
apiGroups: [ '' ],
368-
resources: [ 'secrets' ],
369-
resourceNames: [
370-
'hubble-server-certs',
371-
'hubble-relay-client-certs',
372-
'hubble-relay-server-certs',
373-
],
374-
verbs: [ 'update' ],
375-
},
376-
{
377-
apiGroups: [ '' ],
378-
resources: [ 'secrets' ],
379-
resourceNames: [ 'cilium-ca' ],
380-
verbs: [
381-
'get',
382-
'update',
383-
],
384-
},
385-
] else [],
353+
],
386354
},
387355
}
388356
else if (
@@ -402,38 +370,6 @@ local patchManifests = function(file, has_csv)
402370
else
403371
file;
404372

405-
local kubeSystemSecretRO = [
406-
kube.Role(metadata_name_map[release].OlmRole) {
407-
metadata+: {
408-
namespace: 'kube-system',
409-
},
410-
rules: [
411-
{
412-
apiGroups: [ '' ],
413-
resources: [ 'secrets' ],
414-
verbs: [ 'get', 'list', 'watch' ],
415-
},
416-
],
417-
},
418-
kube.RoleBinding(metadata_name_map[release].OlmRole) {
419-
metadata+: {
420-
namespace: 'kube-system',
421-
},
422-
roleRef: {
423-
apiGroup: 'rbac.authorization.k8s.io',
424-
kind: 'Role',
425-
name: metadata_name_map[release].OlmRole,
426-
},
427-
subjects: [
428-
{
429-
kind: 'ServiceAccount',
430-
namespace: 'cilium',
431-
name: metadata_name_map[release].OlmRole,
432-
},
433-
],
434-
},
435-
];
436-
437373
local migrate_to_clife = params.olm.migrate_to_clife;
438374

439375
std.foldl(
@@ -445,7 +381,6 @@ std.foldl(
445381
{
446382
[if util.version.minor >= 17 && migrate_to_clife then '97_migrate_to_clife']:
447383
import 'olm-migrate-operator.libsonnet',
448-
[if util.version.minor <= 14 then '98_fixup_bgp_controlpane_rbac']: kubeSystemSecretRO,
449384
'99_cleanup': (import 'cleanup.libsonnet'),
450385
}
451386
)

docs/modules/ROOT/pages/references/parameters.adoc

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,13 @@
22

33
= Parameters
44

5+
[IMPORTANT]
6+
====
7+
Component version v3.x supports installing and managing Cilium v1.16 and newer.
8+
9+
See component version v2.x for older Cilium versions.
10+
====
11+
512
The parent key for all of the following parameters is `cilium`.
613

714
[NOTE]

tests/golden/clustermesh/cilium/cilium/olm/cluster-network-06-cilium-00008-cilium-cilium-olm-clusterrole.yaml

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -43,28 +43,3 @@ rules:
4343
- update
4444
- list
4545
- delete
46-
- apiGroups:
47-
- ''
48-
resources:
49-
- secrets
50-
verbs:
51-
- create
52-
- apiGroups:
53-
- ''
54-
resourceNames:
55-
- hubble-server-certs
56-
- hubble-relay-client-certs
57-
- hubble-relay-server-certs
58-
resources:
59-
- secrets
60-
verbs:
61-
- update
62-
- apiGroups:
63-
- ''
64-
resourceNames:
65-
- cilium-ca
66-
resources:
67-
- secrets
68-
verbs:
69-
- get
70-
- update

tests/golden/enterprise-bgp/cilium/cilium/olm/cluster-network-06-cilium-00008-cilium-cilium-olm-clusterrole.yaml

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -43,28 +43,3 @@ rules:
4343
- update
4444
- list
4545
- delete
46-
- apiGroups:
47-
- ''
48-
resources:
49-
- secrets
50-
verbs:
51-
- create
52-
- apiGroups:
53-
- ''
54-
resourceNames:
55-
- hubble-server-certs
56-
- hubble-relay-client-certs
57-
- hubble-relay-server-certs
58-
resources:
59-
- secrets
60-
verbs:
61-
- update
62-
- apiGroups:
63-
- ''
64-
resourceNames:
65-
- cilium-ca
66-
resources:
67-
- secrets
68-
verbs:
69-
- get
70-
- update

tests/golden/olm-opensource/cilium/cilium/olm/cluster-network-06-cilium-00008-cilium-cilium-olm-clusterrole.yaml

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -43,28 +43,3 @@ rules:
4343
- update
4444
- list
4545
- delete
46-
- apiGroups:
47-
- ''
48-
resources:
49-
- secrets
50-
verbs:
51-
- create
52-
- apiGroups:
53-
- ''
54-
resourceNames:
55-
- hubble-server-certs
56-
- hubble-relay-client-certs
57-
- hubble-relay-server-certs
58-
resources:
59-
- secrets
60-
verbs:
61-
- update
62-
- apiGroups:
63-
- ''
64-
resourceNames:
65-
- cilium-ca
66-
resources:
67-
- secrets
68-
verbs:
69-
- get
70-
- update

0 commit comments

Comments
 (0)