Skip to content

avoid infinite loop while unregistering container - #325

Merged
matthyx merged 1 commit into
mainfrom
loop
Jul 11, 2024
Merged

avoid infinite loop while unregistering container#325
matthyx merged 1 commit into
mainfrom
loop

Conversation

@matthyx

@matthyx matthyx commented Jul 10, 2024

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Matthias Bertschy <matthias.bertschy@gmail.com>
@matthyx
matthyx requested a review from amitschendel July 10, 2024 14:57
@github-actions

Copy link
Copy Markdown

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

@matthyx
matthyx merged commit 2224cda into main Jul 11, 2024
@matthyx
matthyx deleted the loop branch July 11, 2024 05:21
matthyx pushed a commit to k8sstormcenter/node-agent that referenced this pull request Jun 26, 2026
…naged lifecycle + signing/tamper detection

Squashed and rebased onto main (resolving conflicts against the merged
kubescape#806 network-wildcards and kubescape#807 exec-args work).

Adds, on top of main's projection surface:
  - Cryptographic signing/verification of ApplicationProfile,
    NetworkNeighborhood, seccomp and rules profiles (pkg/signature,
    cosign-backed, key-based + keyless).
  - Tamper detection for signed user overlays: re-verifies on every
    ContainerProfileCache load and emits R1016 "Signed profile tampered".
  - Rule-signature verification in ruleswatcher.
  - sign-object CLI (sign / verify / generate-keypair / extract-signature).
  - User-defined NetworkNeighborhood overlay support + SyncChecksum
    overlay-identity stamping.
  - enableSignatureVerification config flag.

Conflict resolution: kept main's tested projection design (projectField
isPathSurface, ExecsByPath [][]string) and rule-binding notificationQueue;
the PR's redundant reimplementation of that surface was dropped in favour
of the merged kubescape#806/kubescape#807 versions. Only the net-new features above were
grafted on top.

go.mod: pins the official kubescape/storage v0.0.291 (carries storage kubescape#325),
dropping the temporary k8sstormcenter/storage fork replace; adds sigstore
(fulcio, rekor) dependencies for signing.

Feature documented in docs/features/profile-signing-and-tamper-detection.md.

Squashed from PR kubescape#808 (kubescape/node-agent).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Duck <70207455+entlein@users.noreply.github.com>
matthyx pushed a commit to k8sstormcenter/node-agent that referenced this pull request Jun 26, 2026
…naged lifecycle + signing/tamper detection

Squashed and rebased onto main (resolving conflicts against the merged
kubescape#806 network-wildcards and kubescape#807 exec-args work).

Adds, on top of main's projection surface:
  - Cryptographic signing/verification of ApplicationProfile,
    NetworkNeighborhood, seccomp and rules profiles (pkg/signature,
    cosign-backed, key-based + keyless).
  - Tamper detection for signed user overlays: re-verifies on every
    ContainerProfileCache load and emits R1016 "Signed profile tampered".
  - Rule-signature verification in ruleswatcher.
  - sign-object CLI (sign / verify / generate-keypair / extract-signature).
  - User-defined NetworkNeighborhood overlay support + SyncChecksum
    overlay-identity stamping.
  - enableSignatureVerification config flag.

Conflict resolution: kept main's tested projection design (projectField
isPathSurface, ExecsByPath [][]string) and rule-binding notificationQueue;
the PR's redundant reimplementation of that surface was dropped in favour
of the merged kubescape#806/kubescape#807 versions. Only the net-new features above were
grafted on top.

go.mod: pins the official kubescape/storage v0.0.291 (carries storage kubescape#325),
dropping the temporary k8sstormcenter/storage fork replace; adds sigstore
(fulcio, rekor) dependencies for signing.

Feature documented in docs/features/profile-signing-and-tamper-detection.md.

Syncs R1016 "Signed profile tampered" into the bundled default-rules.yaml
from kubescape/rulelibrary (metadata-only entry, empty ruleExpression since
R1016 is code-emitted, not CEL-evaluated).

Squashed from PR kubescape#808 (kubescape/node-agent).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Duck <70207455+entlein@users.noreply.github.com>
matthyx pushed a commit to k8sstormcenter/node-agent that referenced this pull request Jun 26, 2026
…naged lifecycle + signing/tamper detection

Squashed and rebased onto main (resolving conflicts against the merged
kubescape#806 network-wildcards and kubescape#807 exec-args work).

Adds, on top of main's projection surface:
  - Cryptographic signing/verification of ApplicationProfile,
    NetworkNeighborhood, seccomp and rules profiles (pkg/signature,
    cosign-backed, key-based + keyless).
  - Tamper detection for signed user overlays: re-verifies on every
    ContainerProfileCache load and emits R1016 "Signed profile tampered".
  - Rule-signature verification in ruleswatcher.
  - sign-object CLI (sign / verify / generate-keypair / extract-signature).
  - User-defined NetworkNeighborhood overlay support + SyncChecksum
    overlay-identity stamping.
  - enableSignatureVerification config flag.

Conflict resolution: kept main's tested projection design (projectField
isPathSurface, ExecsByPath [][]string) and rule-binding notificationQueue;
the PR's redundant reimplementation of that surface was dropped in favour
of the merged kubescape#806/kubescape#807 versions. Only the net-new features above were
grafted on top.

go.mod: pins the official kubescape/storage v0.0.291 (carries storage kubescape#325),
dropping the temporary k8sstormcenter/storage fork replace; adds sigstore
(fulcio, rekor) dependencies for signing.

Feature documented in docs/features/profile-signing-and-tamper-detection.md.

Syncs R1016 "Signed profile tampered" into the bundled default-rules.yaml
from kubescape/rulelibrary (metadata-only entry, empty ruleExpression since
R1016 is code-emitted, not CEL-evaluated).

Squashed from PR kubescape#808 (kubescape/node-agent).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Duck <70207455+entlein@users.noreply.github.com>
matthyx pushed a commit to k8sstormcenter/node-agent that referenced this pull request Jun 26, 2026
…naged lifecycle + signing/tamper detection

Squashed and rebased onto main (resolving conflicts against the merged
kubescape#806 network-wildcards and kubescape#807 exec-args work).

Adds, on top of main's projection surface:
  - Cryptographic signing/verification of ApplicationProfile,
    NetworkNeighborhood, seccomp and rules profiles (pkg/signature,
    cosign-backed, key-based + keyless).
  - Tamper detection for signed user overlays: re-verifies on every
    ContainerProfileCache load and emits R1016 "Signed profile tampered".
  - Rule-signature verification in ruleswatcher.
  - sign-object CLI (sign / verify / generate-keypair / extract-signature).
  - User-defined NetworkNeighborhood overlay support + SyncChecksum
    overlay-identity stamping.
  - enableSignatureVerification config flag.

Conflict resolution: kept main's tested projection design (projectField
isPathSurface, ExecsByPath [][]string) and rule-binding notificationQueue;
the PR's redundant reimplementation of that surface was dropped in favour
of the merged kubescape#806/kubescape#807 versions. Only the net-new features above were
grafted on top.

go.mod: pins the official kubescape/storage v0.0.291 (carries storage kubescape#325),
dropping the temporary k8sstormcenter/storage fork replace; adds sigstore
(fulcio, rekor) dependencies for signing.

Feature documented in docs/features/profile-signing-and-tamper-detection.md.

Syncs R1016 "Signed profile tampered" into the bundled default-rules.yaml
from kubescape/rulelibrary (metadata-only entry, empty ruleExpression since
R1016 is code-emitted, not CEL-evaluated).

Squashed from PR kubescape#808 (kubescape/node-agent).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Duck <70207455+entlein@users.noreply.github.com>
matthyx pushed a commit to k8sstormcenter/node-agent that referenced this pull request Jun 27, 2026
…naged lifecycle + signing/tamper detection

Squashed and rebased onto main (resolving conflicts against the merged
kubescape#806 network-wildcards and kubescape#807 exec-args work).

Adds, on top of main's projection surface:
  - Cryptographic signing/verification of ApplicationProfile,
    NetworkNeighborhood, seccomp and rules profiles (pkg/signature,
    cosign-backed, key-based + keyless).
  - Tamper detection for signed user overlays: re-verifies on every
    ContainerProfileCache load and emits R1016 "Signed profile tampered".
  - Rule-signature verification in ruleswatcher.
  - sign-object CLI (sign / verify / generate-keypair / extract-signature).
  - User-defined NetworkNeighborhood overlay support + SyncChecksum
    overlay-identity stamping.
  - enableSignatureVerification config flag.

Conflict resolution: kept main's tested projection design (projectField
isPathSurface, ExecsByPath [][]string) and rule-binding notificationQueue;
the PR's redundant reimplementation of that surface was dropped in favour
of the merged kubescape#806/kubescape#807 versions. Only the net-new features above were
grafted on top.

go.mod: pins the official kubescape/storage v0.0.291 (carries storage kubescape#325),
dropping the temporary k8sstormcenter/storage fork replace; adds sigstore
(fulcio, rekor) dependencies for signing.

Feature documented in docs/features/profile-signing-and-tamper-detection.md.

Syncs R1016 "Signed profile tampered" into the bundled default-rules.yaml
from kubescape/rulelibrary (metadata-only entry, empty ruleExpression since
R1016 is code-emitted, not CEL-evaluated).

Squashed from PR kubescape#808 (kubescape/node-agent).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Duck <70207455+entlein@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants