@intentius/chant-lexicon-k8s 0.49.0 → 0.50.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/composites/index.d.ts +2 -0
- package/dist/composites/index.d.ts.map +1 -1
- package/dist/composites/operator-stack.d.ts +183 -0
- package/dist/composites/operator-stack.d.ts.map +1 -0
- package/dist/index.d.ts +2 -2
- package/dist/index.d.ts.map +1 -1
- package/dist/integrity.json +35 -35
- package/dist/lint/post-synth/argo-helpers.d.ts.map +1 -1
- package/dist/lint/post-synth/crd-schema-helpers.d.ts.map +1 -1
- package/dist/lint/post-synth/k8s-helpers.d.ts +28 -5
- package/dist/lint/post-synth/k8s-helpers.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8005.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8006.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8041.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8042.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8101.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8102.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8103.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8104.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8105.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8201.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8202.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8203.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8204.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8205.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8207.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8208.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8209.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8301.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8302.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8303.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8304.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8305.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8306.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8401.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8402.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8403.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8404.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8405.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8406.d.ts.map +1 -1
- package/dist/lint/post-synth/wk8407.d.ts.map +1 -1
- package/dist/manifest.json +1 -1
- package/dist/op/builders.d.ts +69 -0
- package/dist/op/builders.d.ts.map +1 -0
- package/dist/rules/argo-helpers.ts +2 -6
- package/dist/rules/crd-schema-helpers.ts +4 -6
- package/dist/rules/k8s-helpers.ts +32 -23
- package/dist/rules/wk8005.ts +27 -32
- package/dist/rules/wk8006.ts +33 -38
- package/dist/rules/wk8041.ts +19 -24
- package/dist/rules/wk8042.ts +17 -22
- package/dist/rules/wk8101.ts +29 -34
- package/dist/rules/wk8102.ts +13 -18
- package/dist/rules/wk8103.ts +17 -22
- package/dist/rules/wk8104.ts +30 -35
- package/dist/rules/wk8105.ts +16 -21
- package/dist/rules/wk8201.ts +26 -31
- package/dist/rules/wk8202.ts +17 -22
- package/dist/rules/wk8203.ts +17 -22
- package/dist/rules/wk8204.ts +46 -51
- package/dist/rules/wk8205.ts +27 -32
- package/dist/rules/wk8207.ts +14 -19
- package/dist/rules/wk8208.ts +14 -19
- package/dist/rules/wk8209.ts +14 -19
- package/dist/rules/wk8301.ts +25 -30
- package/dist/rules/wk8302.ts +19 -24
- package/dist/rules/wk8303.ts +23 -26
- package/dist/rules/wk8304.ts +35 -40
- package/dist/rules/wk8305.ts +39 -42
- package/dist/rules/wk8306.ts +20 -25
- package/dist/rules/wk8401.ts +59 -64
- package/dist/rules/wk8402.ts +13 -18
- package/dist/rules/wk8403.ts +32 -37
- package/dist/rules/wk8404.ts +15 -21
- package/dist/rules/wk8405.ts +15 -18
- package/dist/rules/wk8406.ts +29 -34
- package/dist/rules/wk8407.ts +17 -22
- package/dist/serializer.d.ts.map +1 -1
- package/package.json +3 -3
- package/src/composites/composites.test.ts +283 -0
- package/src/composites/index.ts +4 -0
- package/src/composites/operator-stack.ts +385 -0
- package/src/index.ts +11 -0
- package/src/lint/post-synth/argo-helpers.ts +2 -6
- package/src/lint/post-synth/crd-schema-helpers.ts +4 -6
- package/src/lint/post-synth/k8s-helpers.test.ts +26 -9
- package/src/lint/post-synth/k8s-helpers.ts +32 -23
- package/src/lint/post-synth/wk8005.ts +27 -32
- package/src/lint/post-synth/wk8006.ts +33 -38
- package/src/lint/post-synth/wk8041.ts +19 -24
- package/src/lint/post-synth/wk8042.ts +17 -22
- package/src/lint/post-synth/wk8101.ts +29 -34
- package/src/lint/post-synth/wk8102.ts +13 -18
- package/src/lint/post-synth/wk8103.ts +17 -22
- package/src/lint/post-synth/wk8104.ts +30 -35
- package/src/lint/post-synth/wk8105.ts +16 -21
- package/src/lint/post-synth/wk8201.ts +26 -31
- package/src/lint/post-synth/wk8202.ts +17 -22
- package/src/lint/post-synth/wk8203.ts +17 -22
- package/src/lint/post-synth/wk8204.ts +46 -51
- package/src/lint/post-synth/wk8205.ts +27 -32
- package/src/lint/post-synth/wk8207.ts +14 -19
- package/src/lint/post-synth/wk8208.ts +14 -19
- package/src/lint/post-synth/wk8209.ts +14 -19
- package/src/lint/post-synth/wk8301.ts +25 -30
- package/src/lint/post-synth/wk8302.ts +19 -24
- package/src/lint/post-synth/wk8303.ts +23 -26
- package/src/lint/post-synth/wk8304.ts +35 -40
- package/src/lint/post-synth/wk8305.ts +39 -42
- package/src/lint/post-synth/wk8306.ts +20 -25
- package/src/lint/post-synth/wk8401.ts +59 -64
- package/src/lint/post-synth/wk8402.ts +13 -18
- package/src/lint/post-synth/wk8403.ts +32 -37
- package/src/lint/post-synth/wk8404.ts +15 -21
- package/src/lint/post-synth/wk8405.ts +15 -18
- package/src/lint/post-synth/wk8406.ts +29 -34
- package/src/lint/post-synth/wk8407.ts +17 -22
- package/src/op/builders.test.ts +82 -0
- package/src/op/builders.ts +95 -0
- package/src/serializer.ts +9 -1
package/dist/rules/wk8102.ts
CHANGED
|
@@ -6,7 +6,7 @@
|
|
|
6
6
|
*/
|
|
7
7
|
|
|
8
8
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
9
|
-
import {
|
|
9
|
+
import { docsToManifests } from "./k8s-helpers";
|
|
10
10
|
|
|
11
11
|
export const wk8102: PostSynthCheck = {
|
|
12
12
|
id: "WK8102",
|
|
@@ -15,25 +15,20 @@ export const wk8102: PostSynthCheck = {
|
|
|
15
15
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
16
16
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
17
17
|
|
|
18
|
-
for (const
|
|
19
|
-
|
|
20
|
-
const manifests = parseK8sManifests(yaml);
|
|
18
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
19
|
+
if (!manifest.kind || !manifest.metadata) continue;
|
|
21
20
|
|
|
22
|
-
|
|
23
|
-
|
|
21
|
+
const resourceName = manifest.metadata.name ?? manifest.kind;
|
|
22
|
+
const labels = manifest.metadata.labels;
|
|
24
23
|
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
entity: resourceName,
|
|
34
|
-
lexicon: "k8s",
|
|
35
|
-
});
|
|
36
|
-
}
|
|
24
|
+
if (!labels || typeof labels !== "object" || Object.keys(labels).length === 0) {
|
|
25
|
+
diagnostics.push({
|
|
26
|
+
checkId: "WK8102",
|
|
27
|
+
severity: "warning",
|
|
28
|
+
message: `${manifest.kind} "${resourceName}" has no metadata.labels — add labels for organizational purposes`,
|
|
29
|
+
entity: resourceName,
|
|
30
|
+
lexicon: "k8s",
|
|
31
|
+
});
|
|
37
32
|
}
|
|
38
33
|
}
|
|
39
34
|
|
package/dist/rules/wk8103.ts
CHANGED
|
@@ -6,7 +6,7 @@
|
|
|
6
6
|
*/
|
|
7
7
|
|
|
8
8
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
9
|
-
import {
|
|
9
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
10
10
|
|
|
11
11
|
export const wk8103: PostSynthCheck = {
|
|
12
12
|
id: "WK8103",
|
|
@@ -15,27 +15,22 @@ export const wk8103: PostSynthCheck = {
|
|
|
15
15
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
16
16
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
17
17
|
|
|
18
|
-
for (const
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
const
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
message: `${manifest.kind} "${resourceName}" has a container at index ${i} without a name — the name field is required`,
|
|
35
|
-
entity: resourceName,
|
|
36
|
-
lexicon: "k8s",
|
|
37
|
-
});
|
|
38
|
-
}
|
|
18
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
19
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
20
|
+
|
|
21
|
+
const containers = extractContainers(manifest);
|
|
22
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
23
|
+
|
|
24
|
+
for (let i = 0; i < containers.length; i++) {
|
|
25
|
+
const container = containers[i];
|
|
26
|
+
if (!container.name || typeof container.name !== "string" || container.name.trim() === "") {
|
|
27
|
+
diagnostics.push({
|
|
28
|
+
checkId: "WK8103",
|
|
29
|
+
severity: "error",
|
|
30
|
+
message: `${manifest.kind} "${resourceName}" has a container at index ${i} without a name — the name field is required`,
|
|
31
|
+
entity: resourceName,
|
|
32
|
+
lexicon: "k8s",
|
|
33
|
+
});
|
|
39
34
|
}
|
|
40
35
|
}
|
|
41
36
|
}
|
package/dist/rules/wk8104.ts
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
9
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
10
|
-
import {
|
|
10
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
11
11
|
|
|
12
12
|
export const wk8104: PostSynthCheck = {
|
|
13
13
|
id: "WK8104",
|
|
@@ -16,45 +16,20 @@ export const wk8104: PostSynthCheck = {
|
|
|
16
16
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
17
17
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
18
18
|
|
|
19
|
-
for (const
|
|
20
|
-
const
|
|
21
|
-
const manifests = parseK8sManifests(yaml);
|
|
19
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
20
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind ?? "(unknown)";
|
|
22
21
|
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
const
|
|
29
|
-
for (const container of containers) {
|
|
30
|
-
if (!Array.isArray(container.ports)) continue;
|
|
31
|
-
for (const port of container.ports) {
|
|
32
|
-
if (typeof port === "object" && port !== null && !port.name) {
|
|
33
|
-
diagnostics.push({
|
|
34
|
-
checkId: "WK8104",
|
|
35
|
-
severity: "warning",
|
|
36
|
-
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has an unnamed port (containerPort: ${port.containerPort ?? "?"}) — add a name for clarity`,
|
|
37
|
-
entity: resourceName,
|
|
38
|
-
lexicon: "k8s",
|
|
39
|
-
});
|
|
40
|
-
}
|
|
41
|
-
}
|
|
42
|
-
}
|
|
43
|
-
}
|
|
44
|
-
|
|
45
|
-
// Check Service ports
|
|
46
|
-
if (manifest.kind === "Service") {
|
|
47
|
-
const spec = manifest.spec;
|
|
48
|
-
if (!spec) continue;
|
|
49
|
-
const ports = spec.ports as Array<Record<string, unknown>> | undefined;
|
|
50
|
-
if (!Array.isArray(ports)) continue;
|
|
51
|
-
|
|
52
|
-
for (const port of ports) {
|
|
22
|
+
// Check container ports in workloads
|
|
23
|
+
if (manifest.kind && WORKLOAD_KINDS.has(manifest.kind)) {
|
|
24
|
+
const containers = extractContainers(manifest);
|
|
25
|
+
for (const container of containers) {
|
|
26
|
+
if (!Array.isArray(container.ports)) continue;
|
|
27
|
+
for (const port of container.ports) {
|
|
53
28
|
if (typeof port === "object" && port !== null && !port.name) {
|
|
54
29
|
diagnostics.push({
|
|
55
30
|
checkId: "WK8104",
|
|
56
31
|
severity: "warning",
|
|
57
|
-
message: `
|
|
32
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has an unnamed port (containerPort: ${port.containerPort ?? "?"}) — add a name for clarity`,
|
|
58
33
|
entity: resourceName,
|
|
59
34
|
lexicon: "k8s",
|
|
60
35
|
});
|
|
@@ -62,6 +37,26 @@ export const wk8104: PostSynthCheck = {
|
|
|
62
37
|
}
|
|
63
38
|
}
|
|
64
39
|
}
|
|
40
|
+
|
|
41
|
+
// Check Service ports
|
|
42
|
+
if (manifest.kind === "Service") {
|
|
43
|
+
const spec = manifest.spec;
|
|
44
|
+
if (!spec) continue;
|
|
45
|
+
const ports = spec.ports as Array<Record<string, unknown>> | undefined;
|
|
46
|
+
if (!Array.isArray(ports)) continue;
|
|
47
|
+
|
|
48
|
+
for (const port of ports) {
|
|
49
|
+
if (typeof port === "object" && port !== null && !port.name) {
|
|
50
|
+
diagnostics.push({
|
|
51
|
+
checkId: "WK8104",
|
|
52
|
+
severity: "warning",
|
|
53
|
+
message: `Service "${resourceName}" has an unnamed port (port: ${port.port ?? "?"}) — add a name for clarity`,
|
|
54
|
+
entity: resourceName,
|
|
55
|
+
lexicon: "k8s",
|
|
56
|
+
});
|
|
57
|
+
}
|
|
58
|
+
}
|
|
59
|
+
}
|
|
65
60
|
}
|
|
66
61
|
|
|
67
62
|
return diagnostics;
|
package/dist/rules/wk8105.ts
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
9
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
10
|
-
import {
|
|
10
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
11
11
|
|
|
12
12
|
export const wk8105: PostSynthCheck = {
|
|
13
13
|
id: "WK8105",
|
|
@@ -16,26 +16,21 @@ export const wk8105: PostSynthCheck = {
|
|
|
16
16
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
17
17
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
18
18
|
|
|
19
|
-
for (const
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has no explicit imagePullPolicy — set it to "Always", "IfNotPresent", or "Never"`,
|
|
35
|
-
entity: resourceName,
|
|
36
|
-
lexicon: "k8s",
|
|
37
|
-
});
|
|
38
|
-
}
|
|
19
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
20
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
21
|
+
|
|
22
|
+
const containers = extractContainers(manifest);
|
|
23
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
24
|
+
|
|
25
|
+
for (const container of containers) {
|
|
26
|
+
if (!container.imagePullPolicy) {
|
|
27
|
+
diagnostics.push({
|
|
28
|
+
checkId: "WK8105",
|
|
29
|
+
severity: "warning",
|
|
30
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has no explicit imagePullPolicy — set it to "Always", "IfNotPresent", or "Never"`,
|
|
31
|
+
entity: resourceName,
|
|
32
|
+
lexicon: "k8s",
|
|
33
|
+
});
|
|
39
34
|
}
|
|
40
35
|
}
|
|
41
36
|
}
|
package/dist/rules/wk8201.ts
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
9
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
10
|
-
import {
|
|
10
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
11
11
|
|
|
12
12
|
export const wk8201: PostSynthCheck = {
|
|
13
13
|
id: "WK8201",
|
|
@@ -16,36 +16,31 @@ export const wk8201: PostSynthCheck = {
|
|
|
16
16
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
17
17
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
18
18
|
|
|
19
|
-
for (const
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
const
|
|
27
|
-
const
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
if (!limits
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" is missing resource limits for ${missing.join(", ")}`,
|
|
45
|
-
entity: resourceName,
|
|
46
|
-
lexicon: "k8s",
|
|
47
|
-
});
|
|
48
|
-
}
|
|
19
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
20
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
21
|
+
|
|
22
|
+
const containers = extractContainers(manifest);
|
|
23
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
24
|
+
|
|
25
|
+
for (const container of containers) {
|
|
26
|
+
const limits = container.resources?.limits;
|
|
27
|
+
const missing: string[] = [];
|
|
28
|
+
|
|
29
|
+
if (!limits || typeof limits !== "object") {
|
|
30
|
+
missing.push("cpu", "memory");
|
|
31
|
+
} else {
|
|
32
|
+
if (!limits.cpu) missing.push("cpu");
|
|
33
|
+
if (!limits.memory) missing.push("memory");
|
|
34
|
+
}
|
|
35
|
+
|
|
36
|
+
if (missing.length > 0) {
|
|
37
|
+
diagnostics.push({
|
|
38
|
+
checkId: "WK8201",
|
|
39
|
+
severity: "warning",
|
|
40
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" is missing resource limits for ${missing.join(", ")}`,
|
|
41
|
+
entity: resourceName,
|
|
42
|
+
lexicon: "k8s",
|
|
43
|
+
});
|
|
49
44
|
}
|
|
50
45
|
}
|
|
51
46
|
}
|
package/dist/rules/wk8202.ts
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
9
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
10
|
-
import {
|
|
10
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
11
11
|
|
|
12
12
|
export const wk8202: PostSynthCheck = {
|
|
13
13
|
id: "WK8202",
|
|
@@ -16,27 +16,22 @@ export const wk8202: PostSynthCheck = {
|
|
|
16
16
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
17
17
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
18
18
|
|
|
19
|
-
for (const
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
const
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" runs in privileged mode — this grants full host access and should be avoided`,
|
|
36
|
-
entity: resourceName,
|
|
37
|
-
lexicon: "k8s",
|
|
38
|
-
});
|
|
39
|
-
}
|
|
19
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
20
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
21
|
+
|
|
22
|
+
const containers = extractContainers(manifest);
|
|
23
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
24
|
+
|
|
25
|
+
for (const container of containers) {
|
|
26
|
+
const secCtx = container.securityContext;
|
|
27
|
+
if (secCtx && secCtx.privileged === true) {
|
|
28
|
+
diagnostics.push({
|
|
29
|
+
checkId: "WK8202",
|
|
30
|
+
severity: "error",
|
|
31
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" runs in privileged mode — this grants full host access and should be avoided`,
|
|
32
|
+
entity: resourceName,
|
|
33
|
+
lexicon: "k8s",
|
|
34
|
+
});
|
|
40
35
|
}
|
|
41
36
|
}
|
|
42
37
|
}
|
package/dist/rules/wk8203.ts
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
9
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
10
|
-
import {
|
|
10
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
11
11
|
|
|
12
12
|
export const wk8203: PostSynthCheck = {
|
|
13
13
|
id: "WK8203",
|
|
@@ -16,27 +16,22 @@ export const wk8203: PostSynthCheck = {
|
|
|
16
16
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
17
17
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
18
18
|
|
|
19
|
-
for (const
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
const
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not set readOnlyRootFilesystem: true — set it to reduce the attack surface`,
|
|
36
|
-
entity: resourceName,
|
|
37
|
-
lexicon: "k8s",
|
|
38
|
-
});
|
|
39
|
-
}
|
|
19
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
20
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
21
|
+
|
|
22
|
+
const containers = extractContainers(manifest);
|
|
23
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
24
|
+
|
|
25
|
+
for (const container of containers) {
|
|
26
|
+
const secCtx = container.securityContext;
|
|
27
|
+
if (!secCtx || secCtx.readOnlyRootFilesystem !== true) {
|
|
28
|
+
diagnostics.push({
|
|
29
|
+
checkId: "WK8203",
|
|
30
|
+
severity: "warning",
|
|
31
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not set readOnlyRootFilesystem: true — set it to reduce the attack surface`,
|
|
32
|
+
entity: resourceName,
|
|
33
|
+
lexicon: "k8s",
|
|
34
|
+
});
|
|
40
35
|
}
|
|
41
36
|
}
|
|
42
37
|
}
|
package/dist/rules/wk8204.ts
CHANGED
|
@@ -11,7 +11,7 @@
|
|
|
11
11
|
*/
|
|
12
12
|
|
|
13
13
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
14
|
-
import {
|
|
14
|
+
import { docsToManifests, extractContainers, extractPodSpec, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
15
15
|
|
|
16
16
|
export const wk8204: PostSynthCheck = {
|
|
17
17
|
id: "WK8204",
|
|
@@ -20,63 +20,58 @@ export const wk8204: PostSynthCheck = {
|
|
|
20
20
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
21
21
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
22
22
|
|
|
23
|
-
for (const
|
|
24
|
-
|
|
25
|
-
const manifests = parseK8sManifests(yaml);
|
|
23
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
24
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
26
25
|
|
|
27
|
-
|
|
28
|
-
|
|
26
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
27
|
+
const podSpec = extractPodSpec(manifest);
|
|
28
|
+
if (!podSpec) continue;
|
|
29
29
|
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
30
|
+
// Check pod-level securityContext
|
|
31
|
+
const podSecCtx = podSpec.securityContext as Record<string, unknown> | undefined;
|
|
32
|
+
const podRunAsNonRoot = podSecCtx?.runAsNonRoot === true;
|
|
33
|
+
const podRunAsUser = podSecCtx?.runAsUser;
|
|
33
34
|
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
const
|
|
37
|
-
const
|
|
35
|
+
const containers = extractContainers(manifest);
|
|
36
|
+
for (const container of containers) {
|
|
37
|
+
const secCtx = container.securityContext;
|
|
38
|
+
const containerRunAsNonRoot = secCtx?.runAsNonRoot === true;
|
|
39
|
+
const containerRunAsUser = secCtx?.runAsUser;
|
|
38
40
|
|
|
39
|
-
const
|
|
40
|
-
for (const container of containers) {
|
|
41
|
-
const secCtx = container.securityContext;
|
|
42
|
-
const containerRunAsNonRoot = secCtx?.runAsNonRoot === true;
|
|
43
|
-
const containerRunAsUser = secCtx?.runAsUser;
|
|
41
|
+
const hasRunAsNonRoot = podRunAsNonRoot || containerRunAsNonRoot;
|
|
44
42
|
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
continue;
|
|
56
|
-
}
|
|
43
|
+
if (!hasRunAsNonRoot) {
|
|
44
|
+
diagnostics.push({
|
|
45
|
+
checkId: "WK8204",
|
|
46
|
+
severity: "warning",
|
|
47
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not set runAsNonRoot: true — set it at container or pod level`,
|
|
48
|
+
entity: resourceName,
|
|
49
|
+
lexicon: "k8s",
|
|
50
|
+
});
|
|
51
|
+
continue;
|
|
52
|
+
}
|
|
57
53
|
|
|
58
|
-
|
|
59
|
-
|
|
54
|
+
// runAsNonRoot is true — check for explicit runAsUser
|
|
55
|
+
const effectiveRunAsUser = containerRunAsUser ?? podRunAsUser;
|
|
60
56
|
|
|
61
|
-
|
|
62
|
-
|
|
63
|
-
|
|
64
|
-
|
|
65
|
-
|
|
66
|
-
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
71
|
-
|
|
72
|
-
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
}
|
|
57
|
+
if (effectiveRunAsUser === 0) {
|
|
58
|
+
// Contradictory: runAsNonRoot: true + runAsUser: 0
|
|
59
|
+
diagnostics.push({
|
|
60
|
+
checkId: "WK8204",
|
|
61
|
+
severity: "warning",
|
|
62
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has runAsNonRoot: true but runAsUser: 0 — these settings are contradictory and the container will fail to start`,
|
|
63
|
+
entity: resourceName,
|
|
64
|
+
lexicon: "k8s",
|
|
65
|
+
});
|
|
66
|
+
} else if (effectiveRunAsUser === undefined || effectiveRunAsUser === null) {
|
|
67
|
+
// runAsNonRoot: true but no explicit UID
|
|
68
|
+
diagnostics.push({
|
|
69
|
+
checkId: "WK8204",
|
|
70
|
+
severity: "warning",
|
|
71
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" has runAsNonRoot: true but no explicit runAsUser — set a numeric UID to ensure the container doesn't run as root`,
|
|
72
|
+
entity: resourceName,
|
|
73
|
+
lexicon: "k8s",
|
|
74
|
+
});
|
|
80
75
|
}
|
|
81
76
|
}
|
|
82
77
|
}
|
package/dist/rules/wk8205.ts
CHANGED
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
*/
|
|
8
8
|
|
|
9
9
|
import type { PostSynthCheck, PostSynthContext, PostSynthDiagnostic } from "@intentius/chant/lint/post-synth";
|
|
10
|
-
import {
|
|
10
|
+
import { docsToManifests, extractContainers, WORKLOAD_KINDS } from "./k8s-helpers";
|
|
11
11
|
|
|
12
12
|
export const wk8205: PostSynthCheck = {
|
|
13
13
|
id: "WK8205",
|
|
@@ -16,37 +16,32 @@ export const wk8205: PostSynthCheck = {
|
|
|
16
16
|
check(ctx: PostSynthContext): PostSynthDiagnostic[] {
|
|
17
17
|
const diagnostics: PostSynthDiagnostic[] = [];
|
|
18
18
|
|
|
19
|
-
for (const
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
const
|
|
27
|
-
const
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not drop ALL capabilities — add securityContext.capabilities.drop: ["ALL"]`,
|
|
46
|
-
entity: resourceName,
|
|
47
|
-
lexicon: "k8s",
|
|
48
|
-
});
|
|
49
|
-
}
|
|
19
|
+
for (const manifest of docsToManifests(ctx)) {
|
|
20
|
+
if (!manifest.kind || !WORKLOAD_KINDS.has(manifest.kind)) continue;
|
|
21
|
+
|
|
22
|
+
const containers = extractContainers(manifest);
|
|
23
|
+
const resourceName = manifest.metadata?.name ?? manifest.kind;
|
|
24
|
+
|
|
25
|
+
for (const container of containers) {
|
|
26
|
+
const secCtx = container.securityContext;
|
|
27
|
+
const capabilities = secCtx?.capabilities as Record<string, unknown> | undefined;
|
|
28
|
+
const drop = capabilities?.drop;
|
|
29
|
+
|
|
30
|
+
const dropsAll =
|
|
31
|
+
Array.isArray(drop) &&
|
|
32
|
+
drop.some(
|
|
33
|
+
(cap) =>
|
|
34
|
+
(typeof cap === "string" && cap.toUpperCase() === "ALL"),
|
|
35
|
+
);
|
|
36
|
+
|
|
37
|
+
if (!dropsAll) {
|
|
38
|
+
diagnostics.push({
|
|
39
|
+
checkId: "WK8205",
|
|
40
|
+
severity: "warning",
|
|
41
|
+
message: `Container "${container.name ?? "(unnamed)"}" in ${manifest.kind} "${resourceName}" does not drop ALL capabilities — add securityContext.capabilities.drop: ["ALL"]`,
|
|
42
|
+
entity: resourceName,
|
|
43
|
+
lexicon: "k8s",
|
|
44
|
+
});
|
|
50
45
|
}
|
|
51
46
|
}
|
|
52
47
|
}
|