-
Notifications
You must be signed in to change notification settings - Fork 552
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
Update k8s dependencies #1440
Update k8s dependencies #1440
Conversation
Welcome @retornam! |
Hi @retornam. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/ok-to-test |
/test pull-aws-efs-csi-driver-e2e |
@mskanth972 I think the
|
Yeah, Will take a look. |
aws/efs-utils#237 should resolve the build failures after its merged. |
@mskanth972 I think aws/efs-utils#237 is still vaild for cases where rust isn't installed with yum. I'll be grateful if you can help get that merged too. Thanks |
Hi @retornam thanks for the PR. Working on merging it. |
5f329f0
to
2c8b38d
Compare
@mskanth972 the other PR has been merged. I amended the commit to force re-build this PR. Can you please approve the workflow. Thanks |
We need a new tag cut from
will work, else we'd have to modify it to use the |
@retornam, we are working on it in making a new tag, it takes time as we need to cut a release for Amazon Linux, my teammate is working on it, will follow up with him to fastrack this. |
Hi @retornam, we are planning to release a feature request in this efs-utils release which is under testing phase and ECD for the new version of driver with your change will be 09/20 and for Addons will be 09/27. |
Thank you. I will keep an eye out for it. |
/retest |
Hi @retornam , can you rebase the PR and trigger the test? |
Signed-off-by: retornam <retornam@users.noreply.github.com>
2c8b38d
to
b6dc9b7
Compare
@mskanth972 done |
Thanks, will merge once tests are passed. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mskanth972, retornam The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Is this a bug fix or adding new feature?
This pr upgrades k8s dependencies to the latest
0.26.x
versionWhat is this PR about? / Why do we need it?
The packages are out of date and this PR aims to bring the project up to date on the package verions
What testing is done?
Ran
make test