skupperproject / skupper

Skupper is an implementation of a Virtual Application Network, enabling rich hybrid cloud communication.
http://skupper.io
Apache License 2.0
597 stars 75 forks source link

ConfigMap skupper-network-status shows incorrect linkCost after quickly deleting/recreating link with new cost #1653

Closed Karen-Schoener closed 1 month ago

Karen-Schoener commented 2 months ago

Describe the bug ConfigMap skupper-network-status shows incorrect linkCost after quickly deleting/recreating link with new cost.

Test steps are basically:

How To Reproduce

Create sites: east, west

    skupper -n west init
    skupper -n east init

Link sites (with default cost):

    skupper -n west token create skupper_token_west.out
    skupper -n east link create skupper_token_west.out

In east, view configmap skupper-network-status. Observe that link cost shows as: 1

Delete link.

    skupper -n east link delete link1

Create link (with cost=10). Do not wait for link to age out of the skupper-network-status before creating link (with cost=10).

    skupper -n west token create skupper_token_west.out
    skupper -n east link create skupper_token_west.out --cost=10

In east, view configmap skupper-network-status. Observe that link cost shows as: 1

Expected behavior Expected behavior: linkCost in configmap skupper-network-status matches the configured link cost.

Environment details