Skip to content

Set secure-pod-defaults to "enabled" by default #14029

Closed

Description

@nak3

In what area(s)?

/area API

Describe the feature

As per configmap comment:

# This value will default to "enabled" in a future release,
# probably Knative 1.10
secure-pod-defaults: "disabled"

It says "probably Knative 1.10" but it is still disabled by default.
I opened this for tracking issue.

/cc @evankanderson

Activity

  1. added
    kind/featureWell-understood/specified features, ready for coding.
    on May 30, 2023
  2. added this to the v1.11.0 milestone on May 30, 2023
  3. self-assigned this
    on Jun 1, 2023
  4. kauana commented on Jun 15, 2023

    @kauana

    /unassign @pradnyavmw

  5. kauana commented on Jun 15, 2023

    @kauana

    /assign @kauana

  6. modified the milestones: v1.11.0, v1.12.0 on Aug 16, 2023
  7. 13 remaining items

  8. added this to the v1.13.0 milestone on Nov 20, 2023
  9. dprotaso commented on Nov 20, 2023

    @dprotaso
    Member

    Following up on Evan's feedback - we can make secure-pod-defaults an enum

    • Disabled - do nothing
    • Enabled - enable most options but it still works with non-root images - still warn about nonRoot
    • Restricted - enable all options that the restricted profile requires
  10. modified the milestones: v1.13.0, v1.15.0 on May 21, 2024
  11. modified the milestones: v1.15.0, v1.19.0 on Apr 18, 2025
  12. dsimansk commented on Jun 2, 2025

    @dsimansk
    Contributor

    Following up on Evan's feedback - we can make secure-pod-defaults an enum

    • Disabled - do nothing
    • Enabled - enable most options but it still works with non-root images - still warn about nonRoot
    • Restricted - enable all options that the restricted profile requires

    @dprotaso wanna go with this proposal? Or just enable the current restricted profile?

  13. nader-ziada commented on Aug 21, 2025

    @nader-ziada
    Member

    @dprotaso I started looking at this issue, you still want to proceed with the enum approach, just checking since its been a while since this issue has been around

  14. dprotaso commented on Aug 22, 2025

    @dprotaso
    Member

    you still want to proceed with the enum approach, just checking since its been a while since this issue has been around

    Yeah that'd be great

  15. nader-ziada commented on Aug 25, 2025

    @nader-ziada
    Member

    @dprotaso @dsimansk what about kubernetes.podspec-securitycontext? what's the impact on it with this change?

  16. dprotaso commented on Aug 26, 2025

    @dprotaso
    Member

    what about kubernetes.podspec-securitycontext?

    That property allows end-users to set whatever they want on the podspec security context.

    secure-pod-defaults just specifies which defaults to apply

  17. evankanderson commented on Aug 27, 2025

    @evankanderson
    Member

    In particular, Kubernetes chooses compatible but less secure defaults for a number of these security settings (in particular, pods with default settings will fail to run under the restricted pod security admission policy).

    The proposed flag would change the Knative defaults for pods which don't supply security settings to match the restricted requirements. Explicitly setting these security settings would still be honored, it would only affect pods which don't specify anything security-wise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/APIAPI objects and controllerskind/featureWell-understood/specified features, ready for coding.triage/acceptedIssues which should be fixed (post-triage)

    Type

    No type

    Projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions