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

Increasing size of the 'claim-che-workspace' PVC to 2GB #784

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ibuziuk
Copy link
Contributor

@ibuziuk ibuziuk commented Feb 18, 2020

Motivation

After increasing RAM to 7GB Hosted Che user is able to start an advanced workspace with multiple plugins / extensions enabled, but it is still not be possible to simply build a relatively small project in this workspace due to the PVC size that should be also increased.

Reasoning

Since Jenkins / build namespace was removed some time ago which included 1GB PVC we can consider adding this storage to the *-che namespace. Resource wise this should be safe since we do not increase the number of PVC and just relocating the storage that was available initially on openshift.io.

Process

Our current assumption is that after the cluster-wide tenant update during existing PVCs would be automatically resized but we need to verify it against prod-preview:

  • merge this PR
  • proceed with tenant update against prod-preview

@almighty-bot
Copy link

This PR changes one or more of the template files. These changes are now available for testing: Launch in OpenShift.io and click the update tenant button

@codecov
Copy link

codecov bot commented Feb 18, 2020

Codecov Report

Merging #784 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #784   +/-   ##
=======================================
  Coverage   74.82%   74.82%           
=======================================
  Files          38       38           
  Lines        3202     3202           
=======================================
  Hits         2396     2396           
  Misses        602      602           
  Partials      204      204

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bbea362...e94f0a0. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants