Skip to content

Conversation

Iceber
Copy link
Member

@Iceber Iceber commented Dec 30, 2022

The current ctr content list will print the labels in random order due to the map, and the result will be different each time it is executed, which is inconvenient to use.

Now the labels are sorted to ensure that the result is the same each time

@k8s-ci-robot
Copy link

Hi @Iceber. 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.

Signed-off-by: Iceber Gu <wei.cai-nat@daocloud.io>
@Iceber Iceber force-pushed the sort_content_labels branch from 57ee1eb to 6b333fd Compare December 30, 2022 07:59
@Iceber Iceber closed this Dec 30, 2022
@Iceber Iceber reopened this Dec 30, 2022
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.

5 participants