Hi.
The CLI smoketests are failing due to changes unrelated to this PR. I checked the "current" branch and found that the same issue occurs there as well, so it seems that it cannot be resolved unless it is addressed separately.
I believe there are no issues with this PR itself, so I would like it to be merged. What do you think?
I have already received a review and approval.
- Feed Queries
- All Stories
- Search
- Feed Search
- Transactions
- Transaction Logs
All Stories
Dec 23 2024
Can't be bothered trying to contribute to this project any more.
But how do we handle container upgrades, will the cache/db then we wiped?
Correct. Bad config on my part then. So it is best practice just to keep stuff internal to docker so to speak?
Dec 22 2024
If someone wants to play with vpncloud
$ wget https://github.com/dswd/vpncloud/releases/download/v2.3.0/vpncloud_2.3.0_amd64.deb $ sudo dpkg -i *.deb && sudo rm *.deb $ $ sudo vpncloud config
Or VyOS container integration https://github.com/sever-sever/docker-vpncloud
Containers use persistent storage and are not copied between releases.
As I understand you bind some cache/databases volumes to /config/xxxx
I'd recommend using FTP server as a container as FTP is a more or less obsolete (to be obsoleted) protocol.
Dec 21 2024
Dec 20 2024
In modern environments, there's hardly a case for it anymore.
PR https://github.com/vyos/vyos-1x/pull/4247
Add combined option
I’ve made various improvements to the code based on the feedback from the review and have moved things forward.
Could you let me know how I can proceed to get further reviews on this PR? 😊
Create a PR to community -- https://github.com/FRRouting/frr/pull/17685
Here is one more related issue: https://github.com/FRRouting/frr/issues/17535
Dec 19 2024
Discussed and POC to be tested.