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

T5695: Add LUA script to build FRR #445

Merged
merged 1 commit into from
Oct 30, 2023
Merged

T5695: Add LUA script to build FRR #445

merged 1 commit into from
Oct 30, 2023

Conversation

sever-sever
Copy link
Member

@sever-sever sever-sever commented Oct 30, 2023

Change Summary

Build FRR with LUA scripts --enable-scripting option.
I think it also requires dependencies which in example

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Component(s) name

FRR, build

Proposed changes

How to test

sudo apt install chrpath gawk install-info libcap-dev libjson-c-dev
sudo apt install libpam-dev libprotobuf-c-dev libpython3-dev:native libsnmp-dev protobuf-c-compiler python3-dev:native texinfo lua5.3

./build-frr.sh

FRR

r4# conf t
r4(config)# 
r4(config)# 
r4(config)# route-map FOO permit 100
r4(config-route-map)# 
r4(config-route-map)# match script 
  WORD  The script name to run, without .lua; e.g. 'myroutemap' to run myroutemap.lua

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@vyosbot vyosbot requested review from a team, dmbaturin, sarthurdev, zdc, jestabro and c-po and removed request for a team October 30, 2023 17:19
Build FRR with LUA scripts --enable-scripting option
@c-po c-po merged commit 32a5fde into vyos:current Oct 30, 2023
3 checks passed
@c-po
Copy link
Member

c-po commented Nov 15, 2023

@Mergifyio backport sagitta

Copy link

mergify bot commented Nov 15, 2023

backport sagitta

✅ Backports have been created

c-po added a commit that referenced this pull request Nov 15, 2023
T5695: Add LUA script to build FRR (backport #445)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants