Skip to content
This repository has been archived by the owner on Nov 21, 2024. It is now read-only.

dev: fix bfd, lan-vpn, acl, traffic-data #830

Merged
merged 4 commits into from
Sep 20, 2024
Merged

dev: fix bfd, lan-vpn, acl, traffic-data #830

merged 4 commits into from
Sep 20, 2024

Conversation

jpkrajewski
Copy link
Collaborator

@jpkrajewski jpkrajewski commented Sep 19, 2024 •
edited by sbasan
Loading

Pull Request summary:

  • fix bfd model
  • fix vpn lan model
  • fix acl model: actions, base_action fields are mutually exclusive, update builder methods
  • fix traffic-policy model (encap value can be list <= 20.12)

Checklist:

  • Make sure to run pre-commit before committing changes
  • Make sure all checks have passed
  • PR description is clear and comprehensive
  • Mentioned the issue that this PR solves (if applicable)
  • Make sure you test the changes

@jpkrajewski jpkrajewski changed the title Fix bfd, lan-vpn dev: Fix bfd, lan-vpn Sep 19, 2024
@jpkrajewski jpkrajewski requested a review from sbasan September 19, 2024 08:13
@sbasan sbasan changed the title dev: Fix bfd, lan-vpn dev: fix bfd, lan-vpn, acl Sep 20, 2024
@sbasan sbasan marked this pull request as ready for review September 20, 2024 13:46
@sbasan sbasan changed the title dev: fix bfd, lan-vpn, acl dev: fix bfd, lan-vpn, acl, traffic-data Sep 20, 2024
@sbasan sbasan merged commit 4057b3e into dev Sep 20, 2024
11 checks passed
@sbasan sbasan deleted the fix/models branch September 20, 2024 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sbasan sbasan sbasan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jpkrajewski @sbasan