Skip to content

Conversation

@teclator
Copy link
Contributor

@teclator teclator commented Jul 21, 2025

Problem

VLAN connections are skipped from reading because the DeviceType is not identified properly.

Solution

Added the proper conversion from NmDeviceType to DeviceType missed by (#918 & #2580).

Test

Tested manually checking the the configuration is read and the connection details is displayed properly in the UI.

image

Copy link
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@teclator teclator changed the title (backend bug): Added myssed type conversion (VLAN) (backend bug): Added missed VLAN device type conversion from NetworkManager Jul 21, 2025
@teclator teclator merged commit 06c44bc into master Jul 21, 2025
11 checks passed
@teclator teclator deleted the added_mysed_type branch July 21, 2025 11:10
@coveralls
Copy link

coveralls commented Jul 21, 2025

Coverage Status

coverage: 75.96% (+11.7%) from 64.231%
when pulling 89c9088 on added_mysed_type
into e85761d on master.

@imobachgs imobachgs mentioned this pull request Jul 21, 2025
imobachgs added a commit that referenced this pull request Jul 21, 2025
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jul 23, 2025
https://build.opensuse.org/request/show/1295145
by user mvidner + anag_factory
- fix .changes errors reported by obs-service-source_validator
  (gh#agama-project/agama#2593)

- Version 17

- Add support for an inst.script that allows to run an arbitrary
  script (bsc#1246702, gh#agama-project/agama#2589).

- Ensure software cache is updated even if part of its config
  cause error (gh#agama-project/agama#2585)

- Fix device type detection when reading a VLAN connection
  (gh#agama-project/agama#2586).

- Add support to configure VLANs through the CLI or HTTP API
  (gh#agama-project/agama#2580).

- Adjust the information included in the logs (gh#agama-project/agama#2575).

- Fix crash when an Agama profile contains only the zfcp section by
  providing better error report when product selection have to be
  done before or in the same profile (bsc#1246601)

- When search
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants