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

Allow HugePages in emptyDir.medium. #2395

Merged
merged 4 commits into from
Jan 13, 2024
Merged

Allow HugePages in emptyDir.medium. #2395

merged 4 commits into from
Jan 13, 2024

Conversation

gram-signal
Copy link
Contributor

Description

Fixes #2302 by allowing the values HugePages, HugePages-2Mi, and HugePages-1Gi in emptyDir.medium.

Release Note

Release note for CHANGELOG:

emptyDir.medium now allows for HugePages

References

#2302
https://kubernetes.io/docs/tasks/manage-hugepages/scheduling-hugepages/

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

@gram-signal gram-signal requested a review from a team as a code owner January 10, 2024 23:15
@hashicorp-cla
Copy link

hashicorp-cla commented Jan 10, 2024

CLA assistant check
All committers have signed the CLA.

@trunet
Copy link

trunet commented Jan 11, 2024

@gram-signal I tried locally and it's working for me. Would you mind signing the CLA to make the approval/merge faster?

@gram-signal
Copy link
Contributor Author

CLA signed :)

@BBBmau
Copy link
Contributor

BBBmau commented Jan 11, 2024

@gram-signal one last request would be to create a changelog-entry, after that I can approve and merge 👍🏼

@BBBmau
Copy link
Contributor

BBBmau commented Jan 11, 2024

also to fix the test format from the failing checker:
To automatically fix the formatting, run 'make tests-lint-fix' and commit the changes.

@gram-signal
Copy link
Contributor Author

@gram-signal one last request would be to create a changelog-entry, after that I can approve and merge 👍🏼

Looks like someone beat me to it :)

@BBBmau
Copy link
Contributor

BBBmau commented Jan 13, 2024

I went ahead and added the changelog-entry since although we do have a guide its not yet included in CONTRIBUTING.md

It'll be easier to locate once we're done rewriting the contribution guide

Copy link
Contributor

@BBBmau BBBmau left a comment

Choose a reason for hiding this comment

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

Acceptance Test Failure can be ignored. I ran both TestAccKubernetesDeploymentV1_with_empty_dir_volume and TestAccKubernetesDeploymentV1_with_empty_dir_huge_page locally with both passing.

Thanks for contributing!

@BBBmau BBBmau merged commit 61520b4 into hashicorp:main Jan 13, 2024
14 of 15 checks passed
dduportal referenced this pull request in jenkins-infra/azure Feb 19, 2024
<Actions>
<action
id="bcd9b70d7c1eb1e07eb5ad8a958f18dc1bbd81461ee1a2604adeea46e3148a47">
        <h3>Bump Terraform `kubernetes` provider version</h3>
<details
id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24">
            <summary>Update Terraform lock file</summary>
<p>changes detected:&#xA;&#x9;&#34;hashicorp/kubernetes&#34; updated
from &#34;2.25.2&#34; to &#34;2.26.0&#34; in file
&#34;.terraform.lock.hcl&#34;</p>
            <details>
                <summary>2.26.0</summary>
<pre>Changelog retrieved
from:&#xA;&#x9;https://github.com/hashicorp/terraform-provider-kubernetes/releases/tag/v2.26.0&#xA;ENHANCEMENTS:&#xA;&#xA;*
`kubernetes/kubernetes_deployment_v1`: Add support for `HugePages` in
`emptyDir.medium`
[[GH-2395](https://github.com/hashicorp/terraform-provider-kubernetes/issues/2395)]&#xA;*
`resource/kubernetes_job_v1`: add new attribute
`spec.pod_failure_policy` to job spec
[[GH-2394](https://github.com/hashicorp/terraform-provider-kubernetes/issues/2394)]&#xA;&#xA;NOTES:&#xA;&#xA;*
Bump Kubernetes dependencies from x.27.8 to x.28.6.
[[GH-2404](https://github.com/hashicorp/terraform-provider-kubernetes/issues/2404)]&#xA;&#xA;&#xA;</pre>
            </details>
        </details>
<a
href="https://infra.ci.jenkins.io/job/updatecli/job/azure/job/main/3/">Jenkins
pipeline link</a>
    </action>
</Actions>

---

<table>
  <tr>
    <td width="77">
<img src="https://www.updatecli.io/images/updatecli.png" alt="Updatecli
logo" width="50" height="50">
    </td>
    <td>
      <p>
Created automatically by <a
href="https://www.updatecli.io/">Updatecli</a>
      </p>
      <details><summary>Options:</summary>
        <br />
<p>Most of Updatecli configuration is done via <a
href="https://www.updatecli.io/docs/prologue/quick-start/">its
manifest(s)</a>.</p>
        <ul>
<li>If you close this pull request, Updatecli will automatically reopen
it, the next time it runs.</li>
<li>If you close this pull request and delete the base branch, Updatecli
will automatically recreate it, erasing all previous commits made.</li>
        </ul>
        <p>
Feel free to report any issues at <a
href="https://github.com/updatecli/updatecli/issues">github.com/updatecli/updatecli</a>.<br
/>
If you find this tool useful, do not hesitate to star <a
href="https://github.com/updatecli/updatecli/stargazers">our GitHub
repository</a> as a sign of appreciation, and/or to tell us directly on
our <a
href="https://matrix.to/#/#Updatecli_community:gitter.im">chat</a>!
        </p>
      </details>
    </td>
  </tr>
</table>

Co-authored-by: Jenkins Infra Bot (updatecli) <60776566+jenkins-infra-bot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

emptyDir with medium hugepages is not supported
4 participants