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

pods priority #393

Merged
merged 7 commits into from
Sep 12, 2023
Merged

pods priority #393

merged 7 commits into from
Sep 12, 2023

Conversation

case0sh
Copy link
Contributor

@case0sh case0sh commented Sep 11, 2023

  • priorityClassName for pods

Related to #379

Description

adding pods priorityClass configurations for all services

How Has This Been Tested?

  • test environment: k8s, oCIS charts
  • test case 1:
  • test case 2:

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 change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation generated (make docs) and committed
  • Documentation ticket raised:
  • Documentation PR created:

@case0sh case0sh marked this pull request as ready for review September 11, 2023 10:28
Copy link
Contributor

@wkloucek wkloucek left a comment

Choose a reason for hiding this comment

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

Tested with the added deployment example and I got this:

kubectl get -n ocis pods -o custom-columns=CONTAINER:.metadata.name,CONTAINER:.spec.priority
CONTAINER                            CONTAINER
appregistry-787986cb7f-f2n94         1001000
audit-5b47d76796-5h4vs               1001000
authmachine-7c5446ff79-6fkjh         1001000
eventhistory-5bbb784c77-fv4wp        1001000
frontend-66877f6cd9-wmb7k            1001000
gateway-58878f9565-92jss             1001000
graph-5b6f699796-g8ngr               1001000
groups-64997db4f5-9xhw8              1001000
idm-97b457f8b-r5jw5                  1001000
idp-7d75b854bb-tmsjt                 1001000
nats-6778787fb-9c877                 1001000
ocdav-64b458c8b8-g8vh6               1001000
ocs-6d44dcdb48-ft56z                 1001000
postprocessing-57f476d4fd-9bdh8      1001000
proxy-797b4496c7-dc5cb               1001000
search-59c78bc69f-n79wg              1000500
settings-76d86676c9-z5wdp            1001000
sharing-86b4d868d6-pjz4b             1001000
storagepubliclink-784478789d-zxgkc   1001000
storageshares-687789b67b-6f4cw       1001000
storagesystem-5c7877d8f8-5zngp       1001000
storageusers-6b86b7b4d4-67sks        1001000
store-848646967-7pj4m                1000500
thumbnails-748c7b999f-gvmnf          1000000
userlog-74b7f78874-9wt7k             1001000
users-55f7d8bf8c-6qqqg               1001000
web-659c6bb678-mk954                 1001000
webdav-8477f8ddd5-gv6rp              1000500

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