Skip to content

Allow efa-device-plugin container to run as root#6435

Closed
DanielJuravski wants to merge 1 commit intoeksctl-io:mainfrom
DanielJuravski:remove-efa-non-root
Closed

Allow efa-device-plugin container to run as root#6435
DanielJuravski wants to merge 1 commit intoeksctl-io:mainfrom
DanielJuravski:remove-efa-non-root

Conversation

@DanielJuravski
Copy link
Copy Markdown

@DanielJuravski DanielJuravski commented Mar 15, 2023

Description

efa-device-plugin container need to run as root.
This Commit is a follow up commit to #6065,
It closes aws-samples/aws-efa-eks#8.

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the userdocs directory)
  • Manually tested
  • Made sure the title of the PR is a good description that can go into the release notes
  • (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

  • Backfilled missing tests for code in same general area 🎉
  • Refactored something and made the world a better place 🌟

Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello DanielJuravski 👋 Thank you for opening a Pull Request in eksctl project. The team will review the Pull Request and aim to respond within 1-10 business days. Meanwhile, please read about the Contribution and Code of Conduct guidelines here. You can find out more information about eksctl on our website

@Himangini
Copy link
Copy Markdown
Contributor

dupe of #6302

@DanielJuravski
Copy link
Copy Markdown
Author

@Himangini #6302 was closed - the bug still exists, this tiny commit fixes this bug, is something missing for a review?

@lipovsek-aws
Copy link
Copy Markdown

hi, I have published helm chart for EFA device plugin (https://github.com/aws-samples/efa-device-plugin-helm) if it helps, I'm also working on pushing this to our public Helm registry.

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.

Issue with container running as root

3 participants