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

dev-uxmt: Fix discovered migration flow issues #756

Merged
merged 12 commits into from
Jul 11, 2024
Merged

Conversation

jpkrajewski
Copy link
Collaborator

@jpkrajewski jpkrajewski commented Jul 9, 2024

Pull Request summary:

Fix:

  • VPN INTERFACE: Static IP address with device specific value converted to dynamic IP address
  • VPN: Migration Cisco VPN Feature Template with IPSec route fails and lan/vpn parcel is not created
  • VPN: KeyError: 'aggregate_only' during Cisco VPN feature template migration
  • SECURITY: Constant value convernted to variable value for integrity-type from Cisco Secuirty feature template
  • Interfaces: Incorrect value for static IPv4 address after migration to ux2. Use IP address instead of network address
  • lan/vpn/interface/ethernet: ValidationError: 3 validation errors for VrrpIPv4 #758

Description of changes:

[Add more in depth analysis of what changed, provide logs, examples of usage]

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/conversion results Fix discovered migration flow issues Jul 9, 2024
@jpkrajewski jpkrajewski force-pushed the fix/conversion-results branch from f588178 to 4767d9d Compare July 9, 2024 13:06
@jpkrajewski jpkrajewski self-assigned this Jul 9, 2024
@jpkrajewski jpkrajewski requested a review from sbasan July 9, 2024 15:42
@jpkrajewski jpkrajewski changed the title Fix discovered migration flow issues dev-uxmt: Fix discovered migration flow issues Jul 10, 2024
@jpkrajewski jpkrajewski marked this pull request as ready for review July 11, 2024 08:35
@jpkrajewski jpkrajewski marked this pull request as draft July 11, 2024 08:51
@jpkrajewski jpkrajewski marked this pull request as ready for review July 11, 2024 14:31
@sbasan sbasan force-pushed the fix/conversion-results branch from fafc4e2 to cfc8915 Compare July 11, 2024 15:42
@sbasan sbasan merged commit 121ba1d into dev-uxmt Jul 11, 2024
10 checks passed
@sbasan sbasan deleted the fix/conversion-results branch July 11, 2024 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants