Skip to content

UI: Fix VNF NIC mapping network select always disabled#13359

Open
Gia-Bursulaia wants to merge 1 commit into
apache:mainfrom
Gia-Bursulaia:fix/vnf-nic-disabled
Open

UI: Fix VNF NIC mapping network select always disabled#13359
Gia-Bursulaia wants to merge 1 commit into
apache:mainfrom
Gia-Bursulaia:fix/vnf-nic-disabled

Conversation

@Gia-Bursulaia
Copy link
Copy Markdown

Description

Fixes VNF NIC mapping network select dropdown which was permanently disabled in the UI.

Root cause:
In ui/src/views/compute/wizard/VnfNicsSelection.vue the disabled attribute
was written as a plain HTML string instead of a Vue reactive binding (missing : prefix).
In Vue/Ant Design, any non-empty string is treated as truthy, making the select
always disabled regardless of state.

Steps to reproduce:

  1. Create an Isolated network (WAN) and L2 network (LAN)
  2. Go to Network → VNF Appliances → Add VNF Appliance
  3. Select a VNF template with NIC definitions
  4. Add networks on the Networks step
  5. Proceed to VNF NIC Mappings step — all dropdowns are disabled

Expected: Network dropdowns are enabled and selectable
Actual: All dropdowns permanently disabled, VNF cannot be deployed via UI

Tested on: CloudStack 4.22.1.0, KVM hypervisor

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Feature/Enhancement Scale or Bug Severity

  • Minor

Remove incorrect disabled attribute from VnfNicsSelection.vue.
The attribute was a plain string instead of Vue binding, causing
the select to always be disabled regardless of state.
@boring-cyborg
Copy link
Copy Markdown

boring-cyborg Bot commented Jun 5, 2026

Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
Here are some useful points:

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant