Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimizer has the bug (it cannot modify config file in /var/run/pathvector/cache/) #235

Open
atlanteg opened this issue May 20, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@atlanteg
Copy link

atlanteg commented May 20, 2024

Describe the bug
When running pathvector -v optimizer , got FATAL error when it tries to change policy/config file.
FATA[0012] reading peer file: open /var/run/pathvector/cache/ASNNNNN_IX.conf: no such file or directory
(While pathvector itself working perfectly, generating configs in /var/run/pathvector/cache/ OK, then deleting them.)
I recognized that "pathvector -v optimizer" clears/removes all generated pathvector's files /var/run/pathvector/cache/* just before trying to read one of them.

Environment
Output from pathvector version
Pathvector 6.3.2
Built c35220b on 2023-06-11T07:03:14Z
No plugins
BIRD: ready.
OS version
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.6 LTS"
Minimum reproducible example of your Pathvector config file formatted with triple backticks with yaml language identifier

optimizer:
  targets: [ "<A.B.C.D>" ]
  modifier: 80
  probe-interval: 30
  latency-threshold: 13
  
  routeserver:
    ...
    local-pref: 179
    probe-sources: [ "<L.O.I.P>" ]
    optimize-inbound: true

To Reproduce
Steps to reproduce the behavior:

  1. Copy this config to pathvector.yml...
  2. Run pathvector -v generate
  3. Run pathvector -v optimizer
  4. See error

Expected behavior
I expect optimizer to work normally.

And have one more important question - it seems to be logical to have "latency-threshold" as separate parameter for each probe "target". To optimize really well - it would be per-prefix local-preference change.
Could it be noted as feature request as well please?

@atlanteg atlanteg added the bug Something isn't working label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant