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

Migration: PPPoE alias import fails with invalid argument error #913

Closed
gsanchietti opened this issue Nov 13, 2024 · 2 comments
Closed

Migration: PPPoE alias import fails with invalid argument error #913

gsanchietti opened this issue Nov 13, 2024 · 2 comments
Labels
verified All test cases were verified successfully

Comments

@gsanchietti
Copy link
Member

Steps to reproduce

  • On NethServer 7, create aliases on the PPPoE interface.
  • Export the archive
  • Attempt to import the export.tar.gz archive into NethSecurity 8.

Expected behavior

  • The archive should be imported successfully, with PPPoE aliases correctly set to the wan zone.

Actual behavior

  • Import process fails with an "Invalid argument" error when setting the zone for PPPoE aliases.
  • Error message:
    Creating zone lan
    Traceback (most recent call last):
      File "/usr/share/ns-migration/20network", line 311, in <module>
        u.set("firewall", zname, "network", [utils.sanitize(n) for n in network])
      File "/usr/lib/python3.11/site-packages/euci/__init__.py", line 133, in set
    uci.UciException: Invalid argument
    

Example excerpt from export.json with redacted IPs and MAC addresses:

{
  "aliases": [
    {
      "proto": "static",
      "zone": "pppoe",
      "netmask": "255.255.255.248",
      "hwaddr": "00:xx:xx:xx:xx:xx",
      "ipaddr": "xx.xx.xx.xx"
    },
    {
      "proto": "static",
      "zone": "pppoe",
      "netmask": "255.255.255.248",
      "hwaddr": "00:xx:xx:xx:xx:xx",
      "ipaddr": "xx.xx.xx.xx"
    },
    {
      "proto": "static",
      "zone": "wan",
      "gateway": "",
      "netmask": "255.255.255.248",
      "hwaddr": "00:xx:xx:xx:xx:xx",
      "ipaddr": "xx.xx.xx.xx"
    }
  ]
}

Components
NethSecurity version: 8-23.05.5-ns.1.3.0

@gsanchietti
Copy link
Member Author

In nethserver-testing:
nethserver-firewall-migration-1.0.2-1.5.g1fcf4e1.ns7.noarch.rpm

Test case
Check the issue is not reproducible

@gsanchietti gsanchietti added the testing Packages are available from testing repositories label Nov 13, 2024
@gsanchietti gsanchietti removed their assignment Nov 14, 2024
@francio87 francio87 self-assigned this Nov 18, 2024
@francio87
Copy link
Member

Confirm Fixed

@francio87 francio87 added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Nov 18, 2024
@francio87 francio87 removed their assignment Nov 18, 2024
@nethbot nethbot moved this from Testing to Verified in NethSecurity Nov 18, 2024
@github-project-automation github-project-automation bot moved this from Verified to Done ✅ in NethSecurity Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified All test cases were verified successfully
Projects
Archived in project
Development

No branches or pull requests

2 participants