Skip to content

Add link delete command for nonkube#2150

Merged
nluaces merged 1 commit intoskupperproject:mainfrom
lynnemorrison:v2_nonkube_link_delete
Jul 11, 2025
Merged

Add link delete command for nonkube#2150
nluaces merged 1 commit intoskupperproject:mainfrom
lynnemorrison:v2_nonkube_link_delete

Conversation

@lynnemorrison
Copy link
Collaborator

Add command to delete links.

The file "Link-.yaml" is removed from ~/.local/share/skupper/namesapces//input/resources.
When the site is reloaded the link is removed.

Currently on the specified link is removed, so if the user wants to remove more than one link, the user will need to issue this
command for each link to be removed.

@lynnemorrison lynnemorrison self-assigned this Jun 4, 2025
Copy link
Member

@fgiorgetti fgiorgetti left a comment

Choose a reason for hiding this comment

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

LGTM

Kind: "Link",
},
ObjectMeta: metav1.ObjectMeta{
Name: "link-router-east",
Copy link
Member

Choose a reason for hiding this comment

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

Maybe just rename it to router-east to match the expected file name.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

@lynnemorrison lynnemorrison force-pushed the v2_nonkube_link_delete branch from b953065 to 714f39d Compare June 25, 2025 19:07
@nluaces nluaces merged commit afd71aa into skupperproject:main Jul 11, 2025
2 checks passed
@nluaces nluaces added this to the 2.1.0 milestone Jul 22, 2025
@nluaces nluaces linked an issue Aug 19, 2025 that may be closed by this pull request
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.

link delete for non kube sites

3 participants