Skip to content
This repository has been archived by the owner on Mar 2, 2024. It is now read-only.

✅ KM334 Add application resource quota and probes #678

Merged
merged 4 commits into from
Sep 1, 2021

Conversation

deifyed
Copy link
Member

@deifyed deifyed commented Aug 31, 2021

Description

Resource Quota

Added a resources definition to the deployment.yaml file created upon running apply application. Each app will now get the following resource definition as default:

Request Limit
cpu 100m 200m
memory 128Mi 256Mi

Liveness and readiness probes

Added default HTTPGet liveness and readiness probes to the deployment.yaml file created by apply application.

Motivation and Context

KM334

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

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

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the release notes (for the next release).

@deifyed deifyed added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 31, 2021
@deifyed deifyed requested a review from a team August 31, 2021 13:22
@sonarcloud
Copy link

sonarcloud bot commented Aug 31, 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

@codecov
Copy link

codecov bot commented Aug 31, 2021

Codecov Report

Merging #678 (ed3275d) into master (2d6175d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #678   +/-   ##
=======================================
  Coverage   43.41%   43.41%           
=======================================
  Files         233      233           
  Lines        6548     6548           
=======================================
  Hits         2843     2843           
  Misses       3705     3705           

Copy link
Contributor

@yngvark yngvark left a comment

Choose a reason for hiding this comment

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

Looks good!

"k8s.io/apimachinery/pkg/util/intstr"
)

const (
Copy link
Contributor

Choose a reason for hiding this comment

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

sweet comments

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah! thought it would be nice not having to lookup docs everytime we tweak them

@deifyed deifyed merged commit 4e810e7 into master Sep 1, 2021
@deifyed deifyed deleted the KM334-add-application-resources branch September 1, 2021 05:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants