Skip to content

Conversation

@opjt
Copy link
Contributor

@opjt opjt commented Dec 2, 2025

related issue #3851
related PR #4618

changes

namespace update

  1. Added validation to require the --label flag for the update command.
pjt@lima-default:/Users/pjt/Projects/PJT/fork-git/nerdctl$ nerdctl namespace update 1
FATA[0000] required flag(s) "label" not set
  1. Added namespace existence check logic
pjt@lima-default:/Users/pjt/Projects/PJT/fork-git/nerdctl$ nerdctl namespace update 11 --label=1
FATA[0000] namespace 11: not found

namespace inspect

  1. Also added namespace existence check logic for inspect
pjt@lima-default:/Users/pjt/Projects/PJT/fork-git/nerdctl$ nerdctl namespace inspect 1 11
[
    {
        "Name": "1",
        "Labels": {
            "1": "true"
        }
    }
]

WARN[0000] namespace 11: not found

opjt added 3 commits December 2, 2025 20:40
Signed-off-by: Park jungtae <jtpark1957@gmail.com>
Signed-off-by: Park jungtae <jtpark1957@gmail.com>
Signed-off-by: Park jungtae <jtpark1957@gmail.com>
@opjt opjt force-pushed the fix/namespace-validation-logic branch from 198649a to 8c96403 Compare December 2, 2025 12:09
@AkihiroSuda AkihiroSuda added this to the v2.2.1 (?) milestone Dec 3, 2025
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit f8f861e into containerd:main Dec 3, 2025
55 of 56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants