Skip to content

Conversation

@mglotov
Copy link
Contributor

@mglotov mglotov commented Nov 1, 2021

PR Description

Please explain the changes you made here and link to any relevant issues.

Fixes #140

This PR adds new features for kubernetes-namespace module. It allows to configure the next things:

  1. LimitRange - https://kubernetes.io/docs/concepts/policy/limit-range
    The default value is:
    {
      type = "Container"
      default = {
        cpu    = "150m"
        memory = "128Mi"
      }
      default_request = {
        cpu    = "100m"
        memory = "64Mi"
      }
    }
  1. ResourseQuota - https://kubernetes.io/docs/concepts/policy/resource-quotas
    The default value is empty.
  2. NetworkPolicies - https://kubernetes.io/docs/concepts/services-networking/network-policies/
    The default value is empty.

Type of change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

@mglotov mglotov changed the title Feature/k8s ns module feature: add limitrange and resourcequota features for k8s namespace Nov 1, 2021
@mglotov mglotov requested review from a team, halfb00t, khalezin and p1gmale0n and removed request for halfb00t and khalezin November 1, 2021 07:37
@mglotov mglotov changed the title feature: add limitrange and resourcequota features for k8s namespace feat: add limitrange and resourcequota features for k8s namespace Nov 1, 2021
@mglotov mglotov marked this pull request as ready for review November 3, 2021 05:22
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 3, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mglotov mglotov merged commit e58698b into main Nov 3, 2021
@mglotov mglotov deleted the feature/k8s_ns_module branch November 3, 2021 08:36
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.

Improvements for k8s namespaces

4 participants