Skip to content

Conversation

ntnn
Copy link

@ntnn ntnn commented Jul 26, 2025

If the containerSecurityContext is set it results in two securityContext keys which causes the helm chart to fail.

      Helm upgrade failed for release mailu/mailu with chart [email protected]: error while running post render on files: map[string]interface {}(nil): yaml: unmarshal errors:
        line 111: mapping key "securityContext" already defined at line 52

@fastlorenzo fastlorenzo requested a review from Copilot August 21, 2025 22:14
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a Helm chart deployment issue where duplicate securityContext keys were causing YAML parsing errors. The fix consolidates the container security context configuration to prevent conflicts between default and user-defined security contexts.

  • Removes the conditional securityContext block that was causing duplication
  • Merges user-defined container security context with the required NET_BIND_SERVICE capability
  • Uses Helm template functions to properly combine security context configurations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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