Skip to content

Fix RADV launch option validation - #1301

Merged
Vysp3r merged 8 commits into
mainfrom
fix/1300-radv-launch-options
Sep 13, 2026
Merged

Vysp3r merged 8 commits into
mainfrom
fix/1300-radv-launch-options

Conversation

@Vysp3r

@Vysp3r Vysp3r commented Sep 13, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes the launch-option failure reported in #1300.

The RADV performance-test widget was still exposing historical values such as sam, gpl, and ruse, while the semantic catalog only accepts the current Mesa RADV values. In addition, list-valued environment options such as RADV_PERFTEST=nggc,rtcps were being validated as one value instead of validating each comma-separated member.

Changes

  • synchronize the RADV performance-test UI with the catalog's supported values
  • add explicit separator metadata for list-valued dynamic environment options
  • validate each member of comma-separated RADV/ACO/VKD3D option lists
  • add regression coverage for valid and invalid multi-value RADV selections
  • surface composition diagnostics in Mass Edit instead of falling back immediately to the generic error message

Expected result

Selections such as:

RADV_PERFTEST=nggc,rtcps

can be prepared and applied, while unsupported list members are still rejected with a useful diagnostic.

Fixes #1300

@Vysp3r
Vysp3r marked this pull request as ready for review September 13, 2026 21:36
@Vysp3r
Vysp3r merged commit 53e9348 into main Sep 13, 2026
2 checks passed
@Vysp3r
Vysp3r deleted the fix/1300-radv-launch-options branch September 13, 2026 21:37
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.

Launch options can not be applied

1 participant