Skip to content

Conversation

adisky
Copy link
Contributor

@adisky adisky commented May 31, 2023

As discussed in the issue #8596 It is a helpful feature at many places and no replacement readily available

@k8s-ci-robot
Copy link

Hi @adisky. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

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/test-infra repository.

@samuelkarp
Copy link
Member

samuelkarp commented May 31, 2023

   * aaa75a78 "Remove cni conf_template deprecation" ... FAIL
    - FAIL - has whitespace errors. See `git show --check aaa75a78028ae26aab96855ebf2ea85169435092`.

@samuelkarp samuelkarp added area/cri Container Runtime Interface (CRI) cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch labels May 31, 2023
@adisky adisky force-pushed the remove-conf-template-deprecation branch from cc03248 to 9a8e179 Compare May 31, 2023 11:55
As discussed in the issue
containerd#8596
It is a helpful feature at many places and no replacement
readily available

Signed-off-by: Aditi Sharma <adi.sky17@gmail.com>
@adisky adisky force-pushed the remove-conf-template-deprecation branch from 9a8e179 to 3ca5b44 Compare May 31, 2023 12:04
@AkihiroSuda
Copy link
Member

/ok-to-test

@kzys kzys merged commit 3ad032e into containerd:main May 31, 2023
@AkihiroSuda AkihiroSuda added cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch and removed cherry-pick/1.7.x Change to be cherry picked to release/1.7 branch labels Jun 3, 2023
@aojea
Copy link
Contributor

aojea commented Jun 7, 2023

lovely , thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cri Container Runtime Interface (CRI) cherry-picked/1.7.x PR commits are cherry-picked into release/1.7 branch ok-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants