Skip to content
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.

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

feat: Support storing KubeArmor policies to OCI registries #340

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

akshay196
Copy link

@akshay196 akshay196 commented Jun 27, 2023

Part of kubearmor/KubeArmor#1130

Karmor support for interacting with OCI registry for pushing, pulling KubeArmor policies.

Tasks

  • Complete Push
    • Validate policy before pushing
    • Authenticate with OCI registry
  • Complete Pull
    • Validate policy after pulling
    • Authenticate with OCI registry
  • Refactor
  • Testing
  • Logging

@akshay196 akshay196 changed the title feat: Add OCI registry support for KubeArmor policies feat: Support storing KubeArmor policies to OCI registries Jun 27, 2023
@akshay196 akshay196 force-pushed the oci-registry-pull-push branch 7 times, most recently from 3c3d531 to 2d634e2 Compare July 8, 2023 10:50
@akshay196 akshay196 force-pushed the oci-registry-pull-push branch 4 times, most recently from e3565c3 to ca5a188 Compare July 9, 2023 15:32
@akshay196 akshay196 marked this pull request as ready for review July 9, 2023 15:33
@nyrahul
Copy link
Contributor

nyrahul commented Jul 9, 2023

It would be great to document this. Can you do a small writeup on:

  1. What was the problem statement for doing this?
  2. How would users benefit?
  3. How would users use this?
  4. Any refs etc.

Part of #1130

This enhances `karmor` to support interacting with OCI registry, ie.
pushing, pulling KubeArmor policies.

Signed-off-by: Akshay Gaikwad <[email protected]>
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