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

feat: ResourceEphemeralStorage #2902

Merged
merged 2 commits into from
Aug 4, 2022

Conversation

dan-cohn-sabre
Copy link
Contributor

Add support for k8s resource type ephemeral-storage

Proposed changes

Add support for specifying "ephemeral-storage" requests and limits in the Okteto manifest. This is a very small improvement but can be critical in cases where namespace defaults are too restrictive.

Add support for k8s resource type ephemeral-storage

Signed-off-by: Dan Cohn <daniel.cohn@sabre.com>
@dan-cohn-sabre dan-cohn-sabre requested a review from a team July 6, 2022 23:00
@derek derek bot added the new-contributor label Jul 6, 2022
@codecov
Copy link

codecov bot commented Jul 6, 2022

Codecov Report

Merging #2902 (553c50f) into master (2d53d76) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2902      +/-   ##
==========================================
+ Coverage   33.02%   33.04%   +0.01%     
==========================================
  Files         162      162              
  Lines       18811    18815       +4     
==========================================
+ Hits         6213     6217       +4     
  Misses      11868    11868              
  Partials      730      730              
Impacted Files Coverage Δ
pkg/k8s/apps/translate.go 86.01% <100.00%> (+0.14%) ⬆️

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 2d53d76...553c50f. Read the comment docs.

@jLopezbarb
Copy link
Contributor

Hi @dan-cohn-sabre,
Could you add some tests on translate_test.go to test that your changes work?

BTW really nice feature, thanks a lot for taking your time 😄

@github-actions
Copy link
Contributor

This pull request is stale because it has been open for 15 days with no activity. Comment on this pull request or it will be closed in 7 days

@github-actions github-actions bot added the stale label Jul 30, 2022
@jmacelroy jmacelroy added keep and removed stale labels Aug 1, 2022
Add tests for k8s resource type ephemeral-storage

Signed-off-by: Dan Cohn <daniel.cohn@sabre.com>
@jLopezbarb jLopezbarb merged commit 10dc30a into okteto:master Aug 4, 2022
@rberrelleza
Copy link
Member

@jLopezbarb could you take care of adding this information to the docs?

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

Successfully merging this pull request may close these issues.

None yet

4 participants