Skip to content

Conversation

@larsks
Copy link
Member

@larsks larsks commented May 28, 2025

Previously, attempting to delete a port forwarding that does not exist
would fail with:

$ uv run openstack  esi port forwarding delete 172.17.1.106 128.31.20.118 -p 80
'could not find port forwarding matching 80:80/tcp'
$ echo $?
1

With this change, we log a warning but do not consider it an error:

$ uv run openstack  esi port forwarding delete 172.17.1.106 128.31.20.118 -p 80
port forwarding matching 80:80/tcp does not exist
$ echo $?
0

Previously, attempting to delete a port forwarding that does not exist
would fail with:

    $ uv run openstack  esi port forwarding delete 172.17.1.106 128.31.20.118 -p 80
    'could not find port forwarding matching 80:80/tcp'
    $ echo $?
    1

With this change, we log a warning but do not consider it an error:

    $ uv run openstack  esi port forwarding delete 172.17.1.106 128.31.20.118 -p 80
    port forwarding matching 80:80/tcp does not exist
    $ echo $?
    0
@larsks larsks requested a review from tzumainn May 28, 2025 18:00
@tzumainn tzumainn merged commit 9d28a5f into CCI-MOC:master May 28, 2025
6 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